Repository navigation
feat(skills): add portable SDLC orchestration - #1090
sbalabanov wants to merge 3 commits into
Conversation
|
|
sbalabanov
left a comment
There was a problem hiding this comment.
Add instructions for PEW output:
- Prefer one row per each subagent, even if they do not have linked PRs yet.
- Output PR number and also PR link as a separate column
- First column Issue / work has a clear indication of the worker state - working, dormant, etc. Define colors for each of them.
- Merged PRs should have the whole row crossed-out font. Agents implementing them could be archived after one week and removed from the ledger.
- The list of issues should appear in a reversed order of updates to that particular issue. The example of update is code change, PR status change or new comments, new instructions sent to subagent, etc.
- Subagent name and issue/work should be consolidated as first field
- Define output for PR State / CI field and define colors for different PR states
- Agent state / next step field should have an indication if operator input is required. Operator input is required when a decision is awaited on a PR comment or in the agent response, or if PR comments are unresolved, or if draft version of the PR needs operator's review. It should also indicate the last action on the work item in this row.
All outputs (not just PEW) should be instructed to have a date and time in local time with short timezone abbreviation, when displayed.
91cedf0 to
4414202
Compare
|
2026-11-14 02:06 UTC - Added the requested PEW layout, state colors, update ordering, operator-input/last-action markers, safe one-week archival, and local timestamps in 4414202. [addressed by agent] |
sbalabanov
left a comment
There was a problem hiding this comment.
More corrections to PEW output:
Make worker only two states: working or idle. working should display some indicator (working icon). idle should not display anything specific.
Make PR number and PR link field a one field. If output supports links without displaying them on terminal, make PR number a link to PR. Otherwise display PR number immediately followed by PR link.
Operator input: make it an icon. Operator is the author of the change and someone who runs orchestrator. Other human reviewers are not operators. The icon should indicate if the current worker/agent waits for the author's to do something.
For PR state, use something other than green for "merged". Use green for "ready".
Make date and time output in PST by default and make it configurable in the skill.
Summary: Coordinate independent issue owners, draft PRs, review feedback, CI, and authorized merges through one reusable skill. Share the instructions between Codex and Claude Code and add bounded STARTWATCH/STOPWATCH control with run-ID and deadline fencing. Test Plan: Bazel watch-control tests/build, isolated built-CLI concurrent-start/stop/restart checks, canonical and Claude-alias skill validation, make gazelle, make lint, and make check-gazelle pass. The control helper does not launch a watcher or access GitHub. Revert Plan: Revert this commit.
4414202 to
112ad11
Compare
|
2026-11-14 00:41 PST - Simplified PEW to working/idle, linked PR numbers, author-action icons, and green Ready/purple Merged in 112ad11. Display time now defaults to configurable Pacific time (PST/PDT); absolute deadlines are unchanged. [addressed by agent] |
Summary
Adds a shared skill for delegation, review, CI/comments, readiness, and authorized merging. PEW shows all retained agents and PRs. It includes linked PR numbers, author-action icons, and last actions in latest-update order; displayed output defaults to configurable Pacific time. STARTWATCH defaults to 48 hours; STOPWATCH fences new watch actions. A small control helper persists deadlines and run IDs; it does not poll or wake agents by itself.
Repro
orchestrate-sdlcfor a multi-issue campaign.PEW,STARTWATCH 48h, andSTOPWATCH; follow the common lifecycle and verified runtime protocol instead of reconstructing it for each agent.Test Plan
Revert Plan
Revert this PR.
Issues