Repository navigation
docs(quest): Smoke substitutes main's relay and CLI from nix.moq.dev - #8161
Conversation
Quest-Claim: 7ba4367a6b928b5a
Scheduled Smoke run 42382978607 built pinned main b98833c and copied moq-relay-0.17.2 and moq-cli-0.14.2 from https://nix.moq.dev with no building lines, so the runtime-cache quest is done. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
kixelated
changed the title
|
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 14 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. |
kixelated
left a comment
kixelated
left a comment
There was a problem hiding this comment.
Automated review by review (OpenAI)
Reviewed commit de5ed66 against base 619f5c8.
No actionable findings. The deleted quest's waiting condition is supported by the linked scheduled Smoke run 42382978607: I checked its GitHub metadata and job 128434330416's log, which pins b98833c and copies both moq-relay-0.17.2 and moq-cli-0.14.2 from https://nix.moq.dev. There is no building ' line in that fetched job log. The documentation-only deletion and removal of its related links are consistent with closing this main-cache verification task.
Direction: retain the separate cache-expiry and release/Cachix follow-ups; this successful substitution does not establish long-term retention or release-tag coverage.
Verification limits: GitHub-only diff and hosted-run/log inspection. I did not run quest validation, build binaries, dispatch CI, or test cache behavior independently. This is a COMMENT, not an approval.
(Written by review (OpenAI))
![pullfrog[bot]](https://avatars.githubusercontent.com/in/1768019?s=60&v=4){kind=small}
There was a problem hiding this comment.
✅ No new issues found.
Reviewed changes
- Deleted
quest/m0/ci/nix-cache-verify.md— the finished "Verify the Nix runtime cache" quest, now that its Waiting condition is met. - Dropped its entry from
quest/m0/ci/README.md— the## Requiredlist no longer points at the deleted file. - Removed three
## Relatedsections — incachix-retire.md,mbx-r2-verify.md, andnix-cache-expiry.md, each of which held only the deleted quest, so the heading went too.
I verified the evidence rather than taking it on trust: run 42382978607 is a successful scheduled run on main whose nix job pins the same b98833c rev and logs copying path '...moq-relay-0.17.2' from 'https://nix.moq.dev' and copying path '...moq-cli-0.14.2' from ..., with no building '/nix/store/... line for either binary. No reference to the deleted quest path or title remains anywhere in the tree, and the Quest check passes on de5ed663. Removing the now-empty ## Related sections matches how earlier removals were handled, and expiry coverage still lives in quest/m0/ci/nix-cache-expiry.md.
This is exactly the outcome the quest was waiting on.
Deletes Verify the Nix runtime cache: its Waiting condition cleared.
Evidence
After moq-dev/moq#8114 merged (2026-11-07 09:08Z), the scheduled Smoke run 42382978607 (
ubuntu-latest / nix, job 128434330416) logged in "Build moq from the flake":The job log has no
building '...'line. Both binaries were substituted, not compiled.Choices made without asking
quest/m0/ci/README.md, and from the Related sections ofcachix-retire.md,mbx-r2-verify.md, andnix-cache-expiry.md. Each of those sections then had no entries, so the heading went too (quest checkpasses).quest/m0/ci/nix-cache-verifyalone. It holds the earlier ci: cache Smoke's Nix runtime packages on main #7662-era attempt, which was squash-merged already.Review
Skipped at de5ed66: the change only deletes a finished quest and its links (13 lines removed, no code changed).
quest checkpasses.Suggested follow-ups
🤖 Generated with Claude Code
(Written by Claude Opus 5.5)