T1750: Taskcore control plane fixes — metadata, queue, decomposition, cleanup#66
Open
krandder wants to merge 12 commits into
Open
T1750: Taskcore control plane fixes — metadata, queue, decomposition, cleanup#66krandder wants to merge 12 commits into
krandder wants to merge 12 commits into
Conversation
- verifyCompletion() rejects done if task has metadata.repo but stateRef is missing or zeroed (returns 422 missing_state_ref) - Rejects done if task has completionRule='and' but not all children are terminal=done (returns 422 children_not_done) - Added isZeroedStateRef() helper - Tests: stateRef missing, stateRef zeroed, children not all done, and happy path with valid stateRef Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Makes verifyCompletion tests independent of insertion order. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Investigated all 10 POST endpoints in middle/http.ts and created a reference doc covering routes, body schemas, status transitions, decomposition flow, and key behaviors (fencing, cost enforcement, completion checks, notifications).
Strategy doc and decomposition committed: - T2271: metadata inheritance for child tasks - T2272: queue claimability false positives / review deadlocks - T2273: decomposition state reconciliation (depends on T2271) - T2274: attention formatter crash fix - T2275: obsolete blocked task cleanup Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…lready materialized; parent stuck in decomposition.active (Bug 3 in action)
…ssment - Created 3-wave execution plan for 6 remaining child tasks - Cancelled T2272 as duplicate of T2268 (scope merged) - Identified T2274→T2270 dependency chain - Wave 1 (parallel): T2271, T2274, T2275 - Wave 2: T2268 (expanded), T2269 - Wave 3: T2270 (depends on T2274) Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Updated T2268 description to merge canceled T2272's claimability.js/CLI scope - Updated T2271 description to include full metadata propagation (repo, base_branch, informed, tags, custom fields) per T2267 analysis - Canceled T2269 (superseded by completed T2273) - Rewrote docs/t1750-decomposition-strategy.md v3 with accurate child inventory, bug coverage matrix, and sequencing Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Task 1750
Taskcore control plane fixes — metadata, queue, decomposition, cleanup
Branch:
task/T1750→main