-
Notifications
You must be signed in to change notification settings - Fork 2
feat(cross-llm): Group I + III skill rewrites for cross-host portability #16
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
f96905f
16ae09f
787a6b1
b8e5584
50f070a
79100b6
078fd79
2d1a831
8f1f77b
cdfd497
fafe4e8
4a62410
5fb77d4
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||||
|---|---|---|---|---|---|---|---|---|
|
|
@@ -19,7 +19,15 @@ Load plan, review critically, execute tasks in batches, report for review betwee | |||||||
| 1. Read plan file | ||||||||
| 2. Review critically - identify any questions or concerns about the plan | ||||||||
| 3. If concerns: Raise them with your human partner before starting | ||||||||
| 4. If no concerns: Create TodoWrite and proceed | ||||||||
| 4. If no concerns: create one task entry per plan task in your host's task system, then proceed to Step 2 | ||||||||
|
|
||||||||
|
||||||||
| Create one task entry per plan task in your host's task system before starting. |
Copilot
AI
Apr 25, 2026
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Step 2 instructs to mark tasks as in_progress/completed, but Step 1 no longer establishes any task list/tracking mechanism for non-claude-code hosts (the only tracking instruction is now inside the <host: claude-code> block). Add a host-neutral instruction in Step 1 to create/track tasks in the current host’s task system so the status updates in Step 2 are actionable everywhere.
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -17,10 +17,13 @@ Invoked automatically by `finishing-a-development-branch` in autonomous mode aft | |
|
|
||
| ## The Process | ||
|
|
||
| Spawn a background agent that monitors the PR in a loop: | ||
| Run a `balanced`-tier agent that monitors the PR in a loop until all CI checks pass and no unresolved reviews remain. | ||
|
|
||
| ``` | ||
| Agent tool (general-purpose, model: sonnet, run_in_background: true): | ||
| <host: claude-code> | ||
| Use the Agent tool to run the monitor in the background: | ||
|
|
||
| ```` | ||
| Agent tool (general-purpose, model: balanced, run_in_background: true): | ||
| description: "Monitor PR #N for CI and reviews" | ||
|
Comment on lines
+22
to
27
|
||
| prompt: | | ||
| You are monitoring PR #<number> on <repo> and automatically fixing issues. | ||
|
|
@@ -33,9 +36,7 @@ Agent tool (general-purpose, model: sonnet, run_in_background: true): | |
| Design doc: <path> | ||
| Plan doc: <path> | ||
|
|
||
| ## Monitor Loop | ||
|
|
||
| Repeat until exit conditions met: | ||
| Repeat the Monitor Loop until exit conditions are met: | ||
|
|
||
| ### 1. Check CI Status | ||
|
|
||
|
|
@@ -88,6 +89,20 @@ Agent tool (general-purpose, model: sonnet, run_in_background: true): | |
| ### 4. Wait Between Checks | ||
|
|
||
| Sleep 60 seconds between check cycles. Do not poll more frequently. | ||
| ```` | ||
| </host> | ||
|
|
||
| <host: codex, opencode, cursor> | ||
|
|
||
| Use your host's equivalent mechanism to periodically poll the following in a loop: | ||
| - `gh pr checks <number>` — fix any failing CI checks | ||
| - `gh api repos/<owner>/<repo>/pulls/<number>/comments` — respond to inline review comments | ||
| - `gh api repos/<owner>/<repo>/pulls/<number>/reviews` — handle any "CHANGES_REQUESTED" reviews | ||
|
|
||
| Continue until all checks pass, no unresolved inline comments remain, and no "changes requested" reviews are pending. | ||
|
|
||
| </host> | ||
|
|
||
|
|
||
| ## Safety Limits | ||
|
|
||
|
|
||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The alignment-agent dispatch instructions are now completely wrapped in
<host: claude-code>, leaving no host-neutral guidance for other environments even though this skill is part of the autonomous pipeline. Add a generic instruction outside the host block (e.g., “use a balanced-tier model / your host’s subagent mechanism”) or provide explicit per-host alternatives so the skill remains usable cross-host.