[teamleaderleo]

Fixes the acpmux base red that #24739 (2584a85) introduced on feat-cmux-next. It hides every acpmux test: Rust MSRV (1.91), clippy, cargo test and the cross-build fail on every acpmux PR (for example #24811, run 40068104184, which changes none of these files).

error[E0308]: mismatched types
   --> crates/acpmux/src/hub/prompt_images.rs:382:67
382 |             msg.get("images").and_then(Value::as_array).unwrap_or(&[]),
    |                                                                   ^^^ expected `&Vec<Value>`, found `&[_; 0]`

The same error is at hub/queue.rs:370 and hub/queue_edit.rs:188.

Change. At all three sites, .and_then(Value::as_array).unwrap_or(&[]) becomes .and_then(Value::as_array).map(Vec::as_slice).unwrap_or_default(). inline_parts takes &[Value], and no images is still an empty slice, so behaviour doesn't change. It's 3 lines. rustfmt (edition 2024) is clean, and CI's Rust jobs on this PR are the compile check.

Risk. Low. It's a type-level fix with no behaviour change.

Agent session: cosmic-gray

🤖 Generated with Claude Code

---

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

---

Summary by cubic

Fixes the acpmux compile error that #24739 introduced on feat-cmux-next, which was breaking Rust MSRV, clippy, cargo test, and cross-build checks on every acpmux PR. At three sites, .and_then(Value::as_array).unwrap_or(&[]) didn't type-check because unwrap_or expects &Vec<Value> while &[] is a slice literal. The fix maps to a slice first (.map(Vec::as_slice).unwrap_or_default()), matching what inline_parts takes. This is a type-level fix only—no images still means an empty slice, so behavior is unchanged.

^{Written for commit af9f554. Summary will update on new commits.}

[cursor]

Bugbot needs on-demand usage enabled

Bugbot uses usage-based billing for this team and requires on-demand usage to be enabled.

A team admin can enable on-demand usage in the Cursor dashboard.

[github-actions]

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

Review af9f554: 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=pr24813 -f ref=deputy-2/acpmux-images-slice -f platform=macos -f backend_mode=local

red: cmux-next checks (god files, concurrency, crash safety, l10n); base red: the same failure on the base d79dbaa; does not block

[blacksmith-sh]

Found 9 test failures on Blacksmith runners:

Failures

Test	View Logs
/tmp/tmppbxdofc7/summary-fail.json (live):/summary-fail.json (live):	View Logs
/tmp/tmpyl4ymix3/summary-missing.json (live):/summary-missing.json (live):	View Logs
[new_workspaces] debug.new_workspace timed 0 of 50 new workspace rows/ [new_workspaces] debug.new_workspace timed 0 of 50 new workspace rows	View Logs
[new_workspaces] the window draws no frames/[new_workspaces] the window draws no frames	View Logs
[tabs] debug.tab_switch timed 0 of 40 switches/ [tabs] debug.tab_switch timed 0 of 40 switches	View Logs
[tabs] the window draws no frames/[tabs] the window draws no frames	View Logs
convention violations found (ERROR lines above)./ convention violations found (ERROR lines above).	View Logs
test_bot_attempt_three_drops_stale_owned_pool_placement (main.PathRoutingStructure. test_bot_attempt_three_drops_stale_owned_pool_placement)/ test_bot_attempt_three_drops_stale_owned_pool_placement (main.PathRoutingStructure. test_bot_attempt_three_drops_stale_owned_pool_placement)	View Logs
test_the_session_lane_label_comes_after_the_fork_clause (main.GeneratedFilesRunner. test_the_session_lane_label_comes_after_the_fork_clause)/ test_the_session_lane_label_comes_after_the_fork_clause (main.GeneratedFilesRunner. test_the_session_lane_label_comes_after_the_fork_clause)	View Logs

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

[teamleaderleo]

Bead cx-34m4y created for this PR by the cmux coordinator.

[teamleaderleo]

Review (codex xhigh, correctness first): SHIP at af9f554

No findings: Vec::as_slice turns Option<&Vec<Value>> into Option<&[Value]>, and <&[Value]>::default() is &[], so behaviour is unchanged and it type-checks on 1.91. acpmux compiles at this head (build-windows / build acpmux x86_64-pc-windows-gnu is green). Every remaining red is also red on base: web bundles, cmux-next checks, release-notary, Guard status, and clippy (changed cmux-tui packages). The clippy lints are in dispatch.rs, requests.rs and an older if in prompt_images.rs; base hid them behind this compile error, and a follow-up PR fixes them.

[github-actions]

Shipped in:

• cmux NEXT nightly 1.0.0-nightly.4007654379801 (03846a44af)