Skip to content

feat(skills): add portable SDLC orchestration - #1090

Draft
sbalabanov wants to merge 3 commits into
mainfrom
skill/orchestrate-sdlc
Draft

sbalabanov wants to merge 3 commits into
mainfrom
skill/orchestrate-sdlc

Conversation

@sbalabanov

@sbalabanov sbalabanov commented Nov 14, 2026 •

Copy link
Copy Markdown
Contributor

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

  1. Load orchestrate-sdlc for a multi-issue campaign.
  2. Request PEW, STARTWATCH 48h, and STOPWATCH; follow the common lifecycle and verified runtime protocol instead of reconstructing it for each agent.

Test Plan

  • Five Bazel behavior tests and the skill/CLI build pass.
  • Isolated built-CLI checks and an independent 33-operation scenario run pass; no live watcher or native wake integration was exercised.
  • Canonical/Claude-alias skill validation, local links, Gazelle, lint, and BUILD synchronization pass.

Revert Plan

Revert this PR.

Issues

@cla-assistant

cla-assistant Bot commented Nov 14, 2026

Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@sbalabanov sbalabanov left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

@sbalabanov
sbalabanov force-pushed the skill/orchestrate-sdlc branch from 91cedf0 to 4414202 Compare November 14, 2026 02:05
@sbalabanov

Copy link
Copy Markdown
Contributor Author

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 sbalabanov left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.
@sbalabanov
sbalabanov force-pushed the skill/orchestrate-sdlc branch from 4414202 to 112ad11 Compare November 14, 2026 08:40
@sbalabanov

Copy link
Copy Markdown
Contributor Author

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]

This branch has not been deployed

No deployments
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.

2 participants