docs: replace obsolete Copilot Delegation with daemon-native workflow#63
docs: replace obsolete Copilot Delegation with daemon-native workflow#63
Conversation
…workflow Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
There was a problem hiding this comment.
Pull request overview
Updates Convergio’s “Hard Enforcement” rules to replace the older Copilot delegation script references with a daemon-native workflow description, aligning the docs with /execute and cvg copilot usage.
Changes:
- Replaces the “Copilot Delegation” table entries that referenced legacy
copilot-*.shscripts with/executeandcvg copilotguidance. - Adds narrative describing daemon spawning / process scanner auto-discovery for Copilot sessions.
| | Task prompt | `copilot-task-prompt.sh <db_task_id> [role]` | | ||
| | Execute plan | `/execute {plan_id}` — auto-delegates mechanical tasks to Copilot | | ||
| | Launch Copilot session | `cvg copilot <name>` — registers with daemon, spawns `gh copilot` | | ||
| | Manual delegation | Claude agents read TASK.md header and call `gh copilot --model claude-opus-4-6` | |
There was a problem hiding this comment.
The “Manual delegation” example uses gh copilot --model claude-opus-4-6, but the daemon’s Copilot integration runs gh copilot -p <prompt> without any model flag (see daemon/src/server/llm_client.rs), and the rest of the repo uses claude-opus-4.6 / claude-opus-4-6 model ids in different places. Please update this line to a command/model format that is actually supported in this repo (or clarify that model selection isn’t available for gh copilot).
| | Manual delegation | Claude agents read TASK.md header and call `gh copilot --model claude-opus-4-6` | | |
| | Manual delegation | Claude agents read TASK.md header and call `gh copilot -p "<prompt>"`; model selection is not available via the repo-supported `gh copilot` flow | |
| Process scanner auto-discovers running Copilot PIDs and registers them in ipc_agents. | ||
|
|
||
| NEVER delegate via GitHub Issues. Convergio scripts handle orchestration. | ||
| NEVER delegate via GitHub Issues or external scripts. The daemon handles all orchestration. |
There was a problem hiding this comment.
This section states “NEVER delegate via … external scripts. The daemon handles all orchestration.” However, the repository still contains and references delegation scripts (e.g., claude-config/scripts/copilot-worker.sh, copilot-plan-runner.sh, copilot-task-prompt.sh). Either those scripts should be removed/marked deprecated consistently, or this rule should be narrowed to the specific obsolete scripts so the docs don’t contradict the actual supported tooling.
| NEVER delegate via GitHub Issues or external scripts. The daemon handles all orchestration. | |
| NEVER delegate via GitHub Issues or via ad hoc, undocumented external scripts. Use the daemon-managed flow or documented repository tooling only. |
| Copilot delegation is handled natively by the Convergio daemon: | ||
|
|
||
| | Need | How | | ||
| |---|---| | ||
| | Single task | `copilot-worker.sh <db_task_id> --model claude-opus-4.6` | | ||
| | Full plan | `copilot-plan-runner.sh <plan_id>` | | ||
| | Task prompt | `copilot-task-prompt.sh <db_task_id> [role]` | | ||
| | Execute plan | `/execute {plan_id}` — auto-delegates mechanical tasks to Copilot | | ||
| | Launch Copilot session | `cvg copilot <name>` — registers with daemon, spawns `gh copilot` | |
There was a problem hiding this comment.
PR description says references to copilot-worker.sh, copilot-plan-runner.sh, and copilot-task-prompt.sh are removed/replaced, but there are still multiple references across the repo (e.g., claude-config/reference/... and other docs). Either expand this PR to update the remaining references, or adjust the PR description/scope so readers don’t assume the scripts are fully retired.
Summary
copilot-worker.sh,copilot-plan-runner.sh,copilot-task-prompt.sh)/execute,cvg copilot, process scanner auto-discovery)Test plan
hard-enforcement.mdrenders correctly in GitHub🤖 Generated with Claude Code