Repository navigation
feat(framework): ship generic sub-agents across six hosts - #1027
Merged
Merged
Conversation
1 task
Merged
1 task
alexsoyes
commented
Oct 21, 2026
alexsoyes
left a comment
Contributor
Author
There was a problem hiding this comment.
@blafourcade si tu peux commencer à regarder les décisions prises (pas le code) ou si tu vois une énormité mais je suis encore en train de bosser dessus
| "invocation": "aidd", | ||
| "exitCode": 0, | ||
| "help": "Usage: aidd [options] [command]\n\nGenerate AI coding assistant configurations from the AIDD framework\n\nOptions:\n -V, --version Show version number\n --verbose Show detailed diagnostic output (default: false)\n -h, --help display help for command\n\nCommands:\n setup [options] Set up or update the project to a correct state\n — bootstraps the whole project (marketplace,\n framework, tools, plugins); see `framework\n install`, which acts on the framework alone\n framework Manage the framework's lifecycle on installed\n tools\n translate [options] <source> Convert an arbitrary source into a target-native\n plugin tree — records nothing (see `sync` for\n the manifest-driven, tracked version)\n plugin Manage plugins for AI tools\n marketplace Manage plugin marketplaces\n auth Manage authentication\n sync [options] [files...] Rewrite owned files from what is already there —\n regenerate tracked files, driven by the manifest\n (see `translate`, which converts a source\n without recording anything)\n update|upgrade [options] Update the aidd CLI itself to the latest version\n doctor [options] Detected and equipped tools, plugins, drift, and\n problems — across all tools or one\n clean [options] Remove all AIDD-managed files from the project —\n retires every part of AIDD; see `framework\n remove`, which removes the framework only\n telemetry Measure what Claude Code sessions consume,\n locally\n help [command] display help for command" | ||
| "help": "Usage: aidd [options] [command]\n\nGenerate AI coding assistant configurations from the AIDD framework\n\nOptions:\n -V, --version Show version number\n --verbose Show detailed diagnostic output (default: false)\n -h, --help display help for command\n\nCommands:\n setup [options] Set up or update the project to a correct state\n — bootstraps the whole project (marketplace,\n framework, tools, plugins); see `framework\n install`, which acts on the framework alone\n framework Manage the framework's lifecycle on installed\n tools\n translate [options] <source> Convert an arbitrary source into a target-native\n plugin tree — records nothing (see `sync` for\n the manifest-driven, tracked version)\n plugin Manage plugins for AI tools\n marketplace Manage plugin marketplaces\n auth Manage authentication\n sync [options] [files...] Rewrite owned files from what is already there —\n regenerate tracked files, driven by the manifest\n (see `translate`, which converts a source\n without recording anything)\n update|upgrade [options] Update the aidd CLI itself to the latest version\n doctor [options] Detected and equipped tools, plugins, drift, and\n problems — across all tools or one\n clean [options] Remove all AIDD-managed files from the project —\n retires every part of AIDD; see `framework\n remove`, which removes the framework only\n telemetry Measure what Claude Code sessions consume,\n locally\n delegation Read this project's delegation suggestions\n help [command] display help for command" |
Contributor
Author
There was a problem hiding this comment.
très difficile à lire
| "help": "Usage: aidd clean [options]\n\nRemove all AIDD-managed files from the project — retires every part of AIDD; see\n`framework remove`, which removes the framework only\n\nOptions:\n --force Confirm file removal (skip dry-run) (default: false)\n --scope <scope> project (default) cleans this project alone; user undoes the\n machine-wide registration setup --scope user wrote and purges\n the shared source itself\n -h, --help display help for command" | ||
| }, | ||
| { | ||
| "invocation": "aidd delegation", |
Contributor
Author
There was a problem hiding this comment.
pourquoi y'a 2 commandes ?
vraiment utile ?
| { path: "src/presentation/commands", count: 15 }, | ||
| // Command groups, including opt-in agents, and their command-only helpers. | ||
| // Keeping each group at its CLI entry point avoids combining unrelated command surfaces. | ||
| { path: "src/presentation/commands", count: 16 }, |
| // Host plugin registries prove ref scope/enablement; the added native source reader independently | ||
| // proves the current catalogue before these flows mutate it. Neither witness replaces the other. | ||
| { path: "src/contexts/framework/application/clean-use-case.ts", injected: 13 }, | ||
| // Independent opt-in agents have their own ownership record and participate in full cleanup. |
Contributor
Author
There was a problem hiding this comment.
je veux pas te flinguer tes règles
| 3. **Fill.** Fill the plan and each phase from their templates, following the inline contracts. Slice the projection across the phases. | ||
| 4. **Show.** Display the written paths. | ||
| 5. **Review.** Score the complete plan and its phases from 0 to 10, with ✓ reasons and ✗ risks. In interactive mode, show them and revise until approved. Under an autonomous orchestrator, revise against the source without waiting for approval; ask only when a product decision cannot be resolved from the source. The score is never written to the plan. | ||
| 5. **Review.** Challenge source coverage, acceptance criteria, phase boundaries, and dependencies; revise gaps. In interactive mode, show them and revise until approved. Under an autonomous orchestrator, revise against the source without waiting for approval; ask only when a product decision cannot be resolved from the source. |
Contributor
Author
There was a problem hiding this comment.
Les scores à la con comme ça si on borne pas l'IA c'st random
|
|
||
| ## Applicable rules | ||
|
|
||
| <!-- Generated by the project's rule-inventory step. The checker uses this list to verify the implementation. `none` if no installed tool has rules. --> |
Contributor
Author
There was a problem hiding this comment.
tu vois ça hop ça vire
blafourcade
reviewed
Oct 22, 2026
alexsoyes
added a commit
that referenced
this pull request
Oct 23, 2026
…rced Review on #1027: agents belong to the marketplace, not the CLI. They are generic roles a skill gives a concern to, and only the orchestrator spawns them, so the four agents move to aidd-orchestrator/agents. `aidd agents install/remove` and per-provider model profiles leave this PR; they come back as a layer over the plugin agents in a follow-up. Read-only is declared, not only written: explorer and validator carry a Claude `tools` allowlist. Claude and Copilot read it as is; OpenCode gets V2 `permissions` denying edits, and loses `name`, which made V2 ignore the agent's permissions. Codex and Cursor have no per-agent restriction. Verified by dispatching the explorer on Claude Code and OpenCode: the write was refused.
This was referenced Oct 26, 2026
1 task
alexsoyes
marked this pull request as ready for review
October 28, 2026 03:29
alexsoyes
force-pushed
the
feature/sub-agents
branch
from
October 28, 2026 07:57
f97b1d2 to
61917e8
Compare
1 task
alexsoyes
enabled auto-merge
October 29, 2026 02:18
Squash of the branch rebased on next. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
blafourcade
force-pushed
the
feature/sub-agents
branch
from
October 29, 2026 08:39
dd8dcf0 to
c38776b
Compare
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
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.
🎯 What & why
Ship four generic sub-agents (explorer, executor, validator, reviewer) in
aidd-orchestrator, translated to each host's native agent format, so workflows can delegate by responsibility instead of naming an agent.🛠️ How it works
Agents:
plugins/aidd-orchestrator/agents/, no tool list and no model: every host gives an agent all tools and the caller's model by default. Delegation guidance lives in the## Agentssection ofAGENTS.md/CLAUDE.md; without an agent, the host's default subagent takes the brief. The formeraidd-devagents are removed.Translation (
cli/): each host receives only the keys its documentation lists.nameeffortname, TOMLmodel_reasoning_effortname, plusinclude-custom-instructions: truereasoningEffortname, double-quotedOne Codex writer (
codexAgentToml) serves both the built distributions and the plugin installer.make reload: installs every plugin locally (Claude, Codex, OpenCode, Copilot, Cursor), copies the agents into Codex and OpenCode (neither loads them from a plugin), and syncs the## Agentsblock into each host's global instructions.Workflows: 01-sdlc, 10-batch and 01-plan delegate by role, never by agent name.
Docs:
docs/SUBAGENTS.md.🧪 How to verify
make reload, then in Claude Code, Codex or OpenCode ask for one of the four agents by role.make reloadinstall them.make reloadused to copyexecutor.tomlandchecker.tomlunprefixed into~/.codex/agents; delete them by hand, the new copies areaidd-orchestrator-*.toml.aidd-orchestrator-explorer),aidd setupand the native plugin install keepexplorer; the native Codex install also writes to.codex/plugins/<p>/agents/, which Codex never reads (already true onnext). Left for a follow-up.--mainand theaidd setupmodel options come in feat(cli): choose the agents' provider and install the framework machine-wide #1119, stacked on this PR.02-deliver.md,03-check.mdand10-batch; whichever merges second resyncs.✅ I certify