feat(aidd-vcs): ci-watch skill with compact GitHub status script
Chưa có ai nhận issue này.
Đánh giá
- Độ khó
- 4/5
- Thời gian dự kiến
- 3-5 ngày
- Mức phù hợp với người mới
- 55/100
- Loại issue
- Tính năng
- Độ rõ ràng
- Khá rõ ràng
- Mức độ hoạt động
- Sôi nổi
- Công nghệ
- github, github-actions
- Lĩnh vực
- ci-cd, developer-experience, tooling
Hướng nghiên cứu
Start by reading plugins/aidd-vcs/skills/02-pull-request/, plugins/aidd-dev/skills/00-sdlc/SKILL.md, and the async review path under plugins/aidd-orchestrator/skills/00-async-dev/actions/review/. Trace the existing ship/finalize flow and inspect how .aidd/config.yml features are handled. Done means the read-only ci-watch script and skill provide tested exit-code behavior, and the flag-enabled ship path blocks success on red checks without changing the default path.
Do mô hình lập chỉ mục viết ra từ nội dung của issue.
Mô tả
Problem
Ship / open-PR paths do not wait on CI deterministically. Free-form agent polling is verbose and inconsistent. Sessions can treat work as done while checks are still red. The async orchestrator path (#149) does not give interactive sessions a compact exit-code contract.
Not a duplicate of #149
| #149 | This issue | |
|---|---|---|
| Caller | Async bot / orchestrator | Interactive session skill |
| Mechanism | React to checks/comments in bot loop | Local script + skill mapping exit codes |
| Integration | Auto finalize rules | Opt-in via features.ci_watch_before_ship |
Scope
- Skill
ci-watch: resolve PR | branch | run id → run script → map exit code. - Bundled script (GitHub-first), e.g. plugin
scripts/ci-watch:- deps:
gh,jq; - flags:
--repo,--pr|--run|--branch, optional--workflow,--interval; - compact stdout (state-change lines; failed job logs only on red);
- stable exit codes documented in script header + skill (pass / fail / cancelled / other / pass-but-not-merged if merge watch);
- pure classifiers unit-testable without network.
- deps:
- Integration contract (non-vacuous):
- Feature flag
features.ci_watch_before_shipin.aidd/config.yml(default false = non-breaking). - When true, ship/finalize path must run ci-watch and must not claim success on non-zero failure exit.
- When false, skill remains manually invocable (standalone value).
- Feature flag
- Skill does not re-implement polling in prose as the primary path.
Standalone value: Any session can wait on CI with stable exit codes without an async bot.
Acceptance criteria
- Green PR: exit 0; short summary.
- Red PR: non-zero failure; failed check names (+ failed logs only).
- Skill invokes script and maps exit codes (no primary agent poll loop).
- Read-only (no commits / force-push).
- With
features.ci_watch_before_ship: trueand red checks, ship path does not report success (doc + action test or fixture). - With flag false/absent, ship behavior unchanged; skill still runnable alone.
- Issue body states non-duplicate vs #149 (this section).
Prior art in this repo
- #149 — async bot reacts to failing checks / quality gates (
plugins/aidd-orchestrator/skills/00-async-dev/actions/review/) plugins/aidd-vcs/skills/02-pull-request/— open PR / ship-adjacent external pathplugins/aidd-dev/skills/00-sdlc/SKILL.md— ship step in interactive SDLC (no CI wait today)
Relations
Intended graph (same fields as GitHub parent / blocked-by / blocks). Formal GraphQL edges need maintainer write — see comment on #584.
| Field | Value |
|---|---|
| parent | #584 |
| blocked-by | #585 |
| blocks | — |
| related | #149 |
| MVP | 1 |
Out of scope
- Auto-fix on red CI
- Non-GitHub forges in v1 (same exit-code contract later)
- Async bot loop (#149)
- Ngôn ngữ chính
- TypeScript
- Star
- 481
- Fork
- 45
- Merge trung bình
- 17 giờ 40 phút
- Pull request đã merge (30 ngày)
- 106
Hướng dẫn đóng góp
Bắt đầu từ đâu
- Đọc hết issue, rồi đọc hướng dẫn đóng góp của dự án.
- Bình luận trên issue rằng bạn sẽ nhận — tránh hai người làm cùng một việc.
- Fork repository và làm thay đổi trên một nhánh.
- Mở pull request có tham chiếu số hiệu của issue.
Issue khác của ai-driven-dev/framework
-
refactor(aidd-orchestrator): the check zone says when to stop, and reviews its axes in one round Đang mở
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
ai-driven-dev/framework#887 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
ai-driven-dev/framework#873 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 85/100
ai-driven-dev/framework#872 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
ai-driven-dev/framework#625 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 76/100
ai-driven-dev/framework#467 · 1 bình luận ·
Tất cả issue của ai-driven-dev/framework
Issue tương tự
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 84/100
Eynzof/Hermes-CN-Desktop#610 ·
-
[Bug]: Matrix progress drafts fail with "Matrix runtime not initialized" during tool activity Đang mởbug clawsweeper:linked-pr-open clawsweeper:needs-live-repro clawsweeper:no-new-fix-pr impact:message-loss issue-rating: 🐚 platinum hermit P2 regression
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 78/100
-
Client support matrix inclusion Đang mởenhancement
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 68/100
-
calcite-components needs triage refactor
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 75/100
Esri/calcite-design-system#15203 ·
-
Độ khó 2/5 1-3 giờ Mức phù hợp với người mới 90/100
danielmiessler/LifeOS#2218 ·