Skip to content

feat: add P1 work-item delivery orchestration - #284

Merged
Wibias merged 1 commit into
feat/p0-open-work-publication-safetyfrom
feat/p1-work-item-delivery
Aug 18, 2026
Merged

feat: add P1 work-item delivery orchestration#284
Wibias merged 1 commit into
feat/p0-open-work-publication-safetyfrom
feat/p1-work-item-delivery

Conversation

@Wibias

@Wibias Wibias commented Aug 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • add tracker-agnostic work-item delivery state planning, with Linear as the first normalized tracker shape
  • derive tracker milestones from verified GitHub delivery evidence instead of hardcoded tracker status names or IDs
  • fail closed on missing or ambiguous review/status mappings
  • bind tracker reconciliation plans to the observed current status ID so stale writes must re-plan
  • route external work-item requests through a dedicated controller without allowing tracker content to grant merge authority
  • expose external work-item delivery in SKILL.md discovery metadata so the host can select the skill before the internal router runs

Hardening found during self-review

  • work on ENG-42 and open a PR now binds the explicitly requested push_code and create_pr actions
  • implementation-only requests such as implement ENG-42 do not silently gain publication authority
  • ship ENG-42 carries publication plus explicit merge intent into delegated phases without weakening the normal merge gate
  • review-like tracker names are accepted for the review milestone only when the tracker type is started, so a completed status such as Review Complete cannot be selected accidentally

Stack

Validation

  • focused unit contracts cover work-item keys, milestone derivation, ambiguous status maps, tracker-type safety, expected-status transitions, covering-PR reuse, publication-action binding, routing, mutation modes, and controller phases
  • repository-wide local suite is intentionally left to the user's local run; GitHub Actions is not used as the current gate

Provenance

The lifecycle concept was informed by OutThisLife/brooklyn-skills ticket-ship and redesigned for GitHub Delivery's existing evidence, publication, and merge gates.

@coderabbitai

coderabbitai Bot commented Aug 18, 2026

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 94bf8d05-9c51-4b80-9570-d63ee8bfcd9d

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@Wibias
Wibias force-pushed the feat/p1-work-item-delivery branch from 61b3c44 to 7e00cd2 Compare August 18, 2026 05:50
@Wibias
Wibias marked this pull request as ready for review August 18, 2026 07:46
@Wibias
Wibias force-pushed the feat/p1-work-item-delivery branch from 7e00cd2 to f056134 Compare August 18, 2026 07:54
@Wibias
Wibias force-pushed the feat/p1-work-item-delivery branch from f056134 to 12b088c Compare August 18, 2026 08:02
@Wibias
Wibias merged commit 165c5a7 into main Aug 18, 2026
10 checks passed
@Wibias
Wibias deleted the feat/p1-work-item-delivery branch August 18, 2026 08:12
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.

1 participant