Repository navigation
web: clear two feat-cmux-next Web tests base reds (devbox awaiting-rebake, docs shortcuts) - #22003
Conversation
… (no bead) #20818 brought main's freestyle-cmux-devbox-38669994342 ladder (baked from desktop sources 28fce1e49495) into feat-cmux-next, and the desktop sources are now d77b614be8fb. tests/vm-image-manifest.test.ts failed on every feat-cmux-next PR's Web tests shard. Same procedure as the entry's comment: move baked and current when a main sync brings a new ladder. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
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. Comment |
|
Receipt for
|
|
Bead |
|
all web/size checks green |
|
cmux status (one comment, edited in place; each line is updated by the job that owns it) Passes: CI passes on CI passes on Written by
|
|
cmux status (one comment, edited in place; each line is updated by the job that owns it)
|
tests/docs-shortcuts.test.ts failed on every feat-cmux-next PR's Web tests shard: web/data/cmux-shortcuts.generated.json was stale against the catalog export. Regenerated with bun web/scripts/export-docs-shortcuts.ts. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Folded in the docs-shortcuts regeneration from #22016. Each fix's PR failed on the other base red, so they land together. |
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
[Low impact] The PR appears safe to merge; no actionable issues were found. SummaryUpdates the desktop entry in
Reviews (1) · Last reviewed commit: "web: regenerate docs shortcuts after bas..." · Reviewed by Greptile |
|
The only red check is |
|
Shipped in:
|
Base red on feat-cmux-next:
checked-in image manifestin tests/vm-image-manifest.test.ts fails on every PR's Web tests shard (e.g. #21634, #21952), because #20818 brought main'sfreestyle-cmux-devbox-38669994342ladder (baked from desktop sources28fce1e49495) while this branch's desktop sources ared77b614be8fb.Following the procedure in
DEVBOX_AWAITING_REBAKE's comment, this movesbakedandcurrentto the new ladder. Promotion stays strict.Verified:
bun test tests/vm-image-manifest.test.tsgives 23 pass, 0 fail (it failed 2 before).🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need.Summary by cubic
Fixes two base reds that failed on every Web tests shard on
feat-cmux-next:tests/vm-image-manifest.test.tsandtests/docs-shortcuts.test.ts.web/scripts/devbox-image-common.tsto the38669994342ladder, whose baked sources were synced from main.web/data/cmux-shortcuts.generated.jsonfrom the docs shortcuts catalog export, and again after the base merge so it stays in sync.Written for commit fd8ccff. Summary will update on new commits.