Skip to content

Add north star release readiness gate#340

Merged
TraderSamwise merged 3 commits into
masterfrom
chore/core-sidecar-next-65
Jul 6, 2026
Merged

Add north star release readiness gate#340
TraderSamwise merged 3 commits into
masterfrom
chore/core-sidecar-next-65

Conversation

@TraderSamwise

@TraderSamwise TraderSamwise commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Summary

  • add a north-star completion tracker that measures all remaining architecture epics
  • add a release-readiness gate doc for source verification plus live install/restart rehearsal
  • add yarn verify:north-star and yarn release:readiness so the tracker/gate is enforced

Verification

  • yarn verify:north-star
  • yarn release:readiness
  • push hook: yarn typecheck && yarn lint && yarn test

Summary by CodeRabbit

  • Documentation

    • Added new guidance for North Star completion tracking, release readiness checks, and runtime authority completion criteria.
    • Expanded existing planning docs with a link to the new completion tracker.
  • Chores

    • Added verification and release-readiness commands to streamline consistency checks across the docs and project scripts.

@vercel

vercel Bot commented Jul 6, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
app Ready Ready Preview, Comment Jul 6, 2026 1:44pm

@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown

Review Change Stack

Warning

Review limit reached

You’ve reached a temporary PR review limit under our Fair Usage Limits Policy.

Your recent review volume is higher than typical usage, so adaptive limits are currently applied.

Next review available in: 33 minutes

Enable usage-based reviews in Billing to review now. Otherwise, wait until the next included review is available.
You're only billed for reviews past your plan's rate limits ($0.25/file).

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: a95bc472-1341-4d78-813c-71e5de43b90b

📥 Commits

Reviewing files that changed from the base of the PR and between e900612 and 90bd4e7.

📒 Files selected for processing (6)
  • .github/workflows/release.yml
  • app/app/cli-auth.tsx
  • docs/release-readiness-gate.md
  • docs/runtime-authority-dead-paths.md
  • package.json
  • scripts/verify-north-star-tracker.mjs
📝 Walkthrough

Walkthrough

This PR adds documentation defining a North Star Completion Tracker (epics A–J, progress vocabulary, executive snapshot) and a Release Readiness Gate process, cross-links them with existing docs, adds a Completion Gate checklist to runtime-authority-dead-paths.md, and introduces a verification script plus two npm scripts to enforce their consistency.

Changes

North Star Tracker and Verification

Layer / File(s) Summary
North Star Completion Tracker
docs/north-star-completion-tracker.md
New tracker document defining progress labels, executive snapshot table, and completion epics A–J with goals, remaining tasks, and "Done when" criteria, plus a release-ready vs north-star-complete comparison, scorecard template, and measurement process.
Release readiness gate and cross-links
docs/release-readiness-gate.md, docs/core-sidecar-north-star.md, docs/runtime-authority-dead-paths.md
New release readiness gate doc covering automated gate, runtime rehearsal, multi-project smoke, and evidence recording; adds a link to the tracker and a new "Completion Gate" checklist subsection.
Verification script and npm scripts
scripts/verify-north-star-tracker.mjs, package.json
New CLI script asserting required doc markers, epic headings, and snapshot rows, plus validating package.json script strings; adds verify:north-star and release:readiness npm scripts.

Estimated code review effort: 2 (Simple) | ~15 minutes

Sequence Diagram(s)

sequenceDiagram
  participant Developer
  participant Yarn as "yarn release:readiness"
  participant VerifyScript as "verify-north-star-tracker.mjs"
  participant Docs as "Tracker/Gate/Ownership docs"

  Developer->>Yarn: run release:readiness
  Yarn->>Yarn: run yarn verify
  Yarn->>VerifyScript: run verify:north-star
  VerifyScript->>Docs: read and assert markers
  Docs-->>VerifyScript: content
  VerifyScript-->>Yarn: exit 0 or exit 1 with errors
  Yarn-->>Developer: pass/fail result
Loading

Possibly related PRs

  • TraderSamwise/aimux#29: Both PRs modify the "Completion Gate" requirements inside docs/runtime-authority-dead-paths.md, referenced by this PR's verifier.
  • TraderSamwise/aimux#38: Both PRs touch the same runtime authority dead-paths checklist content.
  • TraderSamwise/aimux#316: This PR's verifier asserts markers in docs/command-ownership-inventory.md, which #316 also modifies.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly matches the main change: adding a north star release readiness gate and related documentation/scripts.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/core-sidecar-next-65

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

@TraderSamwise

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@docs/runtime-authority-dead-paths.md`:
- Around line 11-17: The file contains two conflicting “Completion Gate”
sections, which makes the release-gate contract ambiguous. Merge this new gate
text into the existing Completion Gate section, or rename one heading so only
one authoritative target remains. Use the existing Completion Gate heading in
docs/runtime-authority-dead-paths.md as the single place for the consolidated
rules.
🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: e9f3bda8-1658-46a7-96ba-75238687f8b8

📥 Commits

Reviewing files that changed from the base of the PR and between 9ca159b and e900612.

📒 Files selected for processing (6)
  • docs/core-sidecar-north-star.md
  • docs/north-star-completion-tracker.md
  • docs/release-readiness-gate.md
  • docs/runtime-authority-dead-paths.md
  • package.json
  • scripts/verify-north-star-tracker.mjs

Comment thread docs/runtime-authority-dead-paths.md Outdated
@TraderSamwise

Copy link
Copy Markdown
Owner Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Jul 6, 2026

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@TraderSamwise
TraderSamwise merged commit f9f909a into master Jul 6, 2026
3 checks passed
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