Repository navigation
Conversation
…alidator, dispatch, aggregator)
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 54 minutes. View limit details
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. |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 7d449cc363
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| const OPENCODE = "developing-today-forks/opencode"; | ||
| const OPENCHAMBER = "developing-today-forks/openchamber"; | ||
| const NIX = "dezren39/nix"; | ||
| const NIX = "dezren39/darwin-nix"; |
There was a problem hiding this comment.
Preserve roadmap attribution for the renamed repository
When using the shipped Grist seed or an existing roadmap initialized from it, this rename removes the Nix roadmap row from its fleet repository card. pm-stack/grist/seed/roadmap.json still names the row Repo fleet — dezren39/nix, while buildProjectDetail in project-config.ts:836–837 attributes rows only when their title contains the exact card repository or a link starts with that repository's ref prefix. The unchanged row satisfies neither condition for dezren39/darwin-nix, so the card now displays no roadmap work despite the row remaining present. Normalize the historical repository alias during attribution or migrate the roadmap data alongside this constant.
Useful? React with 👍 / 👎.
Repo rename follow-through for dezren39/nix -> dezren39/darwin-nix (renamed on GitHub 2026-11-04).
Functional constants only:
Deliberately NOT changed: narrative prose in project-config.ts / program.ts / READMEs (dated lineage text), mcpx docs/decisions (historical issue references), grist/pad seed data. Existing Supabase task dedupe keys keep the old repo segment (history); new packets validate against the new name.
Verified on the laptop:
bun test aggregator/test dispatch/test— failure set is a subset of clean main's pre-existing/flaky failures (zero rename-caused failures remaining);cargo test -p fleet-queuecompiles/passes. Committed via the GitHub API because laptop git push auth is down today (keyring locked); local commit 8c0b97659d3e has identical content.Open in [code]smith · Need help on this PR? Tag
@codesmith-botwith what you need.