Skip to content

web: refresh the docs shortcuts copy (base red) - #22016

Closed
teamleaderleo wants to merge 1 commit into
feat-cmux-nextfrom
l5/docs-shortcuts-fresh
Closed

teamleaderleo wants to merge 1 commit into
feat-cmux-nextfrom
l5/docs-shortcuts-fresh

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Oct 19, 2026 •

Copy link
Copy Markdown
Collaborator

Base red on feat-cmux-next: keyboard shortcuts docs in web/tests/docs-shortcuts.test.ts fails on PRs' Web tests shard (e.g. #22003, run 39210921669), because web/data/cmux-shortcuts.generated.json is stale against the catalog export. Regenerated with bun web/scripts/export-docs-shortcuts.ts, and bun test tests/docs-shortcuts.test.ts passes.

🤖 Generated with Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need.


Summary by cubic

Fixes the docs shortcuts test failure on feat-cmux-next PRs by regenerating web/data/cmux-shortcuts.generated.json from the catalog export. The refresh removes the obsolete paletteKey.toggleActions entry, so tests/docs-shortcuts.test.ts now passes.

Written for commit dc39138. Summary will update on new commits.

View guided diff Turn on auto-fix

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>
@coderabbitai

coderabbitai Bot commented Oct 19, 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: Repository: manaflow-ai/cmux/.coderabbit.yaml
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: d8f8ac52-ea58-46e3-8c98-89f20bd9c953

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
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

@github-actions

Copy link
Copy Markdown
Contributor

Receipt for dc39138 (this PR's head; a new push replaces it)

  • Visual review: HQ feed card · gallery before/after runs
  • Checks: this head
  • App of this head (app.zip, kept 1 day): gh workflow run reload-build.yml -R manaflow-ai/cmux --ref feat-cmux-next -f tag=pr22016 -f ref=l5/docs-shortcuts-fresh -f platform=macos -f backend_mode=local

@github-actions

Copy link
Copy Markdown
Contributor

red: web / Web status; do not merge until green; see lane-rules
red: web / Web tests (2/4); do not merge until green; see lane-rules
red: web / web-instant-navigation; do not merge until green; see lane-rules

@github-actions

github-actions Bot commented Oct 19, 2026 •

Copy link
Copy Markdown
Contributor

cmux status (one comment, edited in place; each line is updated by the job that owns it)

Probably yours: vm-image-manifest.test.ts, vm-image-manifest.test.ts fails here but not on main's latest full suite.

CI failed on dc39138ee9 (run 39212117775 attempt 1): 1 code, 1 unknown.

Job Verdict Why
web / Web tests (2/4) code a web test failed
web / web-instant-navigation unknown no known signature; failed step: Instant navigation tests

Failures:

  • new in this PR vm-image-manifest.test.ts checked-in image manifest > every default was baked from this checkout's devbox sources: failed (job)
  • new in this PR vm-image-manifest.test.ts checked-in image manifest > every drift awaiting its rebake is still this checkout's and still unbak...: failed (job)
Matched log lines
web / Web tests (2/4): (fail) checked-in image manifest > every default was baked from this checkout's devbox sources [3.55ms]

Not re-run automatically: web / Web tests (2/4), web / web-instant-navigation are not machine failures.

Written by scripts/ci/classify_failures.py (ci-failure-attribution.yml); signatures are its SIGNATURES table. A machine verdict is the runner's fault, not this PR's; yours means the failing file is one this PR changes, also red on main that main's latest full suite fails the same way, seen on other PRs that it failed on another pull request's run lately.

CI fast guards failed on dc39138ee9 (https://github.com/manaflow-ai/cmux/actions/runs/39212117775). It does not block the merge; a red guard merged into main breaks it for every open PR.
The log named no failed step; see the run.

Agents: python3 scripts/ci/guard_attribution.py fix applies the mechanical fixes locally.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Bead cx-64li3 created for this PR by the cmux coordinator.

@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Folded into #22003, which carries this commit. Each fix failed on the other's base red.

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