Skip to content

Add the coordination skill and task-worker profile - #2422

Draft
Aaronontheweb wants to merge 90 commits into
feature/background-subagentsfrom
feature/agent-coordination
Draft

Aaronontheweb wants to merge 90 commits into
feature/background-subagentsfrom
feature/agent-coordination

Conversation

@Aaronontheweb

@Aaronontheweb Aaronontheweb commented Oct 9, 2026 •

Copy link
Copy Markdown
Collaborator

Base: #2421. This is the third draft PR in the stack.

Netclaw gains delegation guidance, the agent-coordination skill, and a default task-worker profile that uses the Main model role. Small resources cover source analysis, complete plans, independent code review, defect analysis, and parallel research.

The parent reviews complete child artifacts before conclusions or file delivery. The guide explains status, cancellation, partial results, and live-log access through existing file tools. The parent ends its turn when it needs a canonical child continuation and has no independent work.

Integrated verification repairs spawn-field alias checks, a plan-column assertion, and sidecar template suppression. The CI matrix moves one complete mutation gate to another group. All 14 gates and the 25-minute timeout remain intact.

Why this is safe: The profile adds no authority grant. Existing project, path, and tool policies remain decisive. The adapter changes only explicit suppression requests for its declared dialect. It retains unrelated template values. The oracle repairs retain ownership, behavior, scope, and output-format requirements. Peer messages and live steering remain deferred.

Validation at c56fd8fa:

  • Root passes 111 affected spawn-case tests and 65 artifact tests.
  • The adapter candidate passes 77 targeted .NET tests. Nine controls fail against the old adapter. Independent review approves its exact source and evidence.
  • Slopwatch, copyright headers, and whitespace checks pass. Independent review approves all current repairs.
  • Four earlier process cases pass independent review: routed activation, approval prompts, cancellation, and journal-only owner recovery.
  • Each model trial uses a fresh container and data home. The trivial case passes twice. The unavailable-profile case passes once, then fails because prose follows its JSON object.
  • The code workflow fails report, command-scope, and final-format requirements. Both children complete.
  • The analysis artifacts pass the corrected oracle offline. The original complete-trial false verdict remains unchanged.
  • Compaction remains inconclusive because no recovery-model handoff occurs. No fresh model trial validates the adapter fix.
  • The prior head 91ef75bc finishes with 28 successful CI checks and one mutation-job timeout. The current head needs fresh CI proof.

Fresh model acceptance, private replay, shadow review, the productive holdout, and final verification remain open. This PR remains a draft. No merge, release, live skill publication, or deployment occurs.

Comment thread src/Netclaw.Cli.Tests/Tui/Wizard/IdentityStepViewModelTests.cs Fixed
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from 8407840 to d734d05 Compare October 9, 2026 02:03
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from d734d05 to ce010f2 Compare October 9, 2026 03:13
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from dde46e1 to 1eb81c1 Compare October 9, 2026 04:11
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch 3 times, most recently from 70e5acf to 7924e96 Compare October 9, 2026 05:51
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from 7924e96 to 3d970ea Compare October 9, 2026 06:13
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from 3d970ea to 1dae792 Compare October 9, 2026 06:58
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from 1dae792 to 35abc7c Compare October 9, 2026 23:00
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch 3 times, most recently from 0964f53 to 76fa29b Compare October 10, 2026 01:52
Comment thread evals/test_productive_parent_child_evals.py Fixed
Comment thread evals/test_coordination_implement_review_evals.py Fixed
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch 2 times, most recently from 3dd2c1a to 6d0d750 Compare October 10, 2026 04:22
Comment thread evals/test_terminal_rejection_evals.py Fixed
…vidence

Remove the durable loop-detector state and its replay reconstruction while
keeping the in-memory per-call recurrence guard, the pending-job exemption,
the hard-ceiling removal, and the child-acceptance authority check.

- Remove ToolLoopCheckpoint/ToolLoopDelta/ToolLoopEntry/ToolLoopKey/ToolLoopAdjacent
  and the checkpoint/delta/restore methods.
- Remove CompleteEvidence replay reconstruction.
- Remove detector-state propagation into jobs and child runs (OriginCheckpoint,
  ParentCheckpoint, ParentReceiptFailure, SameCheckpoint).
- Keep LoopAdmission/LoopObservations for the child-acceptance attribution check.
- Delete durability/replay-only tests.
@Aaronontheweb
Aaronontheweb force-pushed the feature/agent-coordination branch from 1e2013c to de568a1 Compare October 11, 2026 01:54
def spawn_fields(self, spellings):
args = copy.deepcopy(self.args)
def change(value):
task = original = value.pop("Task")
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant