Repository navigation
cmux-next: take Saturn and 27 other grim works out of the backdrop art - #22336
Conversation
Leo found it too grim to keep in the set. Remove the work from art/manifest.json, framing, the bundled and full images, the generated enum, entries, string catalog and plans export, and the settings gallery story. A cmux.json or workspace that still names goya-saturn now shows the default background with no invalid-value diagnostic (BackdropArt.removedIDs), covered by BackdropArtCatalogTests and WorkspaceBackgroundTests. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
cmux status (one comment, edited in place; each line is updated by the job that owns it) Review App of this head (app.zip, kept 1 day)
Gallery before/after for
States
Pick other states with a Pending: CI is running on Last result: Unclear: Written by web and size checks: green
Tour of |
|
Bead |
This comment has been minimized.
This comment has been minimized.
[Medium impact] The changes reach backdrop choices across the macOS app and the published art catalog. Regenerate the CI graph before merging so the generated-files check can pass. Findings
SummaryRemoves 28 backdrop works, leaving 146.
No builds or runtime tests were run. Reviews (5) · Last reviewed commit: "chore(next): PR autofix at aeac3455a631" · Reviewed by Greptile |
…og rotation Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
Correctness review (subagent, read-only) of head c159ee6, base feat-cmux-next. Owner: teamleaderleo (agent lane of coordinator leoli-63). Verdict: no blocking findings.
Findings and resolution:
|
Leo picked from the contact sheet: keep The Death of Marat, remove the rest of the flagged violence, death and horror pieces (Judith, Medusa, Triumph of Death, skeletons, severed heads, drownings). Their ids join BackdropArt.removedIDs so a saved choice falls back to the default background. Forest Interior by Moonlight, the Small Dutch Ship and View of Toledo now ship bundled so the starter set stays at 12 dark works; tests and the gallery story use works still in the set, and the placement thresholds follow the 146-work catalog (framing_sheet.py). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Review finding: a removed work someone already downloaded stayed in the Backgrounds folder and, no longer known to BackdropArt, would be listed under Your images. isDownloadedArt now also matches BackdropArt.removedIDs (and retired ids). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Bugbot needs on-demand usage enabledBugbot 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. |
|
Correctness review of the 28-work removal (subagent, read-only), head 032d260, base feat-cmux-next. Owner: teamleaderleo (agent lane of coordinator leoli-63). Finding (medium, blocking): a removed work someone already downloaded came back as one of their images.
Verified, no issue:
CI on 032d260 (not caused by this PR):
|
- Packages/macOS/CmuxNext/ci-target-graph.json Autofix-Run: https://github.com/manaflow-ai/cmux/actions/runs/39366674602
| "CmuxNextHostedChief": { | ||
| "kind": "regular", | ||
| "path": "Packages/macOS/CmuxNext/Sources/CmuxNextHostedChief", | ||
| "targets": [ | ||
| "CmuxNextCompat", | ||
| "CmuxNextDaemon", | ||
| "CmuxNextHomeEngine" | ||
| ], | ||
| "packages": [] |
There was a problem hiding this comment.
ci-target-graph.json now lists CmuxNextHostedChief, which Package.swift does not declare. It also adds CmuxNextFeed as a direct dependency of CmuxNextAppTests, which the package does not declare either. The generator reads these entries from Package.swift, so its output cannot match this file. CI rejects that mismatch. Regenerate the graph from this PR's checkout.
|
Orphaned: no activity for 14 h, and no lane holds a lease on it. Adopted by the orphan sweep: merged |
|
|
- docs/mdm/com.manaflow.cmux.json - docs/mdm/com.manaflow.cmux.plist - docs/mdm/managed-preferences.md - schemas/settings-export/settings-schema.sources.sha256 - schemas/settings/settings-schema.json Autofix-Run: https://github.com/manaflow-ai/cmux/actions/runs/39475511221
|
|
|
|
|
|
|
|
|
Owned by orphan-sweep on cosmic-gray · session |
Conflicts: BackdropSelectionSetting keeps the removed-id fallback ahead of base's new shuffle parse; BackdropArtPlacementTests stays deleted, as on the base (#22562 dropped framing). Generated settings exports (schema, MDM, sources hash) take the base's copy for the PR autofix to regenerate. The CmuxNextDesign catalog is the base's minus the 28 removed titles. Two tests base added since name removed works: DragGlassTests pinned .goyaSaturn (a compile error after the removal) and sliderRows.test.tsx used "goya-saturn"; both now use Forest Interior by Moonlight. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-663f-7650-b9f7-64d198c6b88b&host=cosmic-gray
|
Review (claude-reviewer-22336): SHIP at 509a9bf Independent read-only review of the merged head against the merged base 5e76238. No builds or tests were run. Checked:
Non-blocking notes:
|
The PR autofix's regenerated exports (run 39588825641, artifact pr-autofix-patch at c34fae6), applied by hand: its push job only runs off pull_request events. Schema, MDM and the sources stamp drop the 28 removed works; the defaults snapshot is rewritten from them. Autofix-Run: https://github.com/manaflow-ai/cmux/actions/runs/39588825641 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-663f-7650-b9f7-64d198c6b88b&host=cosmic-gray
|
Review (claude-reviewer-22336b): SHIP at 083df21 Checked against origin/feat-cmux-next (merge-base 3f206f4):
Non-blocking notes: |
… base's Base's autofix regenerated the settings exports (e323f77), which conflicted line by line with this branch's. The exports are now the base's exactly, minus the 28 removed works (enum values and option objects in the schema, MDM JSON/plist, the managed-preferences table), and the stamp is `defaults_guard.py source-stamp` of the merged sources: only the BackdropArtSetting and BackdropSelectionSetting lines differ from base. The same transform over 3f206f4's exports reproduces the PR autofix's backdrop changes at 083df21 with no difference. schemas/defaults/cmux-next.json stays the base's (bot-only; this PR changes no default). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-663f-7650-b9f7-64d198c6b88b&host=cosmic-gray
|
Review (claude-reviewer-22336c): SHIP at b67255c Checked the net diff against origin/feat-cmux-next and the generated outputs since 5c92af7.
Non-blocking: No blocking findings. |
|
The feat-cmux-next push run https://github.com/manaflow-ai/cmux/actions/runs/39647862464 at 5818254 failed: cmux-next checks (god files, concurrency, crash safety, l10n). |
|
Shipped in:
|
Base merge (orphan sweep, 2026-10-21)
Merged feat-cmux-next at 5e76238.
BackdropSelectionSetting.parsekeeps the removed-id fallback ahead of the base's new shuffle parse. Two tests the base added since name a removed work and now use Forest Interior by Moonlight:DragGlassTests(it pinned.goyaSaturn, which no longer exists) andsliderRows.test.tsx. The CmuxNextDesign catalog is the base's minus the 28 titles; settings exports are the base's, for the autofix.generate_backdrop_art.py --checkreports only key order in the catalog and omissions files (same content as it would write), andexport-backdrop-art.py --checkis current. 12 bundled works, 146 in the catalog and in Shuffle Daily.No capture: the change removes works from the background catalog; no layout, control or styling changes (the picker lists fewer tiles).
risk: normal
Agent session: cosmic-gray