[austinywang]

Summary

cmux-next checks fails on feat-cmux-next and on every PR, at the step "No literal state roots (per-build isolation S7, cx-is1kf.8)":

check-state-roots: cmux-tui/crates/acpmux/src/prompt_uploads.rs has 1 literal state root line(s), baseline 0:
  cmux-tui/crates/acpmux/src/prompt_uploads.rs:345:            link(format!("/home/me/.acpmux/uploads/chat/{id}/a.png")),

That line is a fixture in the inline #[cfg(test)] module, which the ratchet counts. It came in with #25840. upload_ids only takes the path's parent folder as the upload id, so this PR changes the fixture to /home/me/uploads/chat/{id}/a.png. The test still checks the same behaviour, and the baseline stays at 0 instead of gaining a row.

Testing

• bash scripts/cmux-next/check-state-roots.sh . exits 1 on the base tip and 0 on this head (check-state-roots: ok).

Observed / Not run

• Observed: the state-roots check above.
• Not run locally: cargo test -p acpmux a_link_names_its_upload_on_any_platform. Rust isn't built on this Mac, so this PR's cmux-tui CI runs it. Only a string literal in the test changed, and the parent folder is still the uuid.

Changelog

none

🤖 Generated with Claude Code

---

^{Need help on this PR? Tag @codesmith-bot with what you need.}

[github-actions]

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

Review 771c6ab: HQ feed card · gallery before/after runs · checks

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=pr26087 -f ref=next-state-roots-upload-fixture -f platform=macos -f backend_mode=local

web and size checks: green

[blacksmith-sh]

Found 6 test failures on Blacksmith runners:

Failures

Test	View Logs
main.DeveloperIdAppTests.test_download_prints_the_active_profile_only/ test_download_prints_the_active_profile_only	View Logs
stubs: /System/Library/Frameworks/Security.framework/Versions/A/Security has no stub/ Security has no stub	View Logs
stubs: CoreFoundation.tbd: install-name /System/Library/Frameworks/ CoreFoundation.framework/Versions/A/CoreFoundation, missing 1: ['_CFRetain']/ CoreFoundation, missing 1: ['_CFRetain']	View Logs
stubs: CoreFoundation.tbd: install-name /System/Library/Frameworks/ CoreFoundation.framework/Versions/A/CoreFoundation, missing 1: ['_log2']/ CoreFoundation, missing 1: ['_log2']	View Logs
test_the_nightly_is_covered (main.NightlyStepCoverageTests.test_the_nightly_is_cove red)/ test_the_nightly_is_covered (main.NightlyStepCoverageTests.test_the_nightly_is_cove red)	View Logs
test_the_repository_highlights_validate (main.DigestTests.test_the_repository_highl ights_validate)/ test_the_repository_highlights_validate (main.DigestTests.test_the_repository_highl ights_validate)	View Logs

^{Need help on this PR? Tag @codesmith-bot with what you need.}

[austinywang]

Review (claude-subagent correctness review): SHIP at 771c6ab

An independent subagent reviewed the exact head and found no defects.

• The test checks the same behaviour. upload_ids (prompt_uploads.rs:91-99) takes the uuid from the path's parent folder. Dropping .acpmux/ changes only folders higher up, so the parent of a.png is still {id}. The Windows fixture, the bare a.png case and the assertion are unchanged.
• The check clears. bash scripts/cmux-next/check-state-roots.sh . passes on this head. On the base tip 3a04f3c29ba it fails, and prompt_uploads.rs:345 is the only line it reports.
• Nothing else matches. --list finds no other line in the file, so state-roots-baseline.tsv needs no row. The Windows path C:\Users\me\acpmux\... doesn't match the pattern.
• No duplicate. No open PR fixes the same line.
• Base moved. feat-cmux-next has moved one commit since the merge base (nightly-next: read the Dev ring switch live when publishing #26077, nightly ring files only). That commit doesn't touch this file.

Not run locally: cargo test -p acpmux. This PR's CI runs it.

Resolution: nothing to change.

[austinywang]

The same fixture fix is now part of #25839 (commit d66d8d5). Each PR was red on the other's base red, so they're combined. Once 25839 lands, this PR's diff against the base becomes empty, and I'll close it.

[austinywang]

Closing as redundant. #26278 fixed the same state-roots fixture on feat-cmux-next, using /home/me/state/uploads/..., so 'No literal state roots' is no longer red on the base. #25839 dropped this file when it merged the base.