Repository navigation
fix(cmux-next): let sessions.promptSuggestions turn suggested tasks off - #23284
Conversation
#21897 added the setting but ORed it with the agent.suggestedTasks Debug toggle, which #22089 had made default on, so turning the setting off did nothing. The setting now decides and defaults on (today's behavior); the Debug toggle defaults off and only forces suggestions on. The host's default closure reads forNextDaemon alone, which already carries both. The old test compared isOn to the Debug toggle's value, which hid the bug; the new one turns the setting off and expects suggestions off. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-5eec-7700-94d7-9a3a1f7d889a&host=cosmic-gray
|
Owned by composer-scroll-2 on cosmic-gray · session |
|
codex-review: SHIP at 4ae319d
|
|
Found 17 test failures on Blacksmith runners: Failures
...and 7 more test failures. View all on Blacksmith: run 39708706791, run 39708706801, run 39708706868
|
|
Bead |
|
|
The comments still called the Debug toggle on by default; it is now a force-on, and Settings > Suggest Follow-Up Tasks decides. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-5eec-7700-94d7-9a3a1f7d889a&host=cosmic-gray
|
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 Unclear: CI failed on
Failures:
Matched log linesNot re-run automatically: Written by red: cmux-next checks (god files, concurrency, crash safety, l10n); no finished base run on 6dc664a to compare; do not merge until green; see lane-rules
Tour of Path-selected interactions
agent-pane-before: the media could not be published (refused 3 file(s) that show private names: 23284/c3c5390c/next-agent-pane-before/shots/02-open-agent-chat.png (mini-name: an agent's pull request); 23284/c3c5390c/next-agent-pane-before/shots/03-queue)agent-pane-after: the media could not be published (refused 3 file(s) that show private names: 23284/c3c5390c/next-agent-pane-after/shots/02-open-agent-chat.png (mini-name: an agent's pull request); 23284/c3c5390c/next-agent-pane-after/shots/03-queue-a)core-before: the media could not be published (refused 12 file(s) that show private names: 23284/c3c5390c/next-core-before/shots/05-split-right.png (mini-name, private-term, shell-prompt: an agent's pull request); 23284/c3c5390c/next-core-before/s)core-after: the media could not be published (refused 12 file(s) that show private names: 23284/c3c5390c/next-core-after/shots/05-split-right.png (mini-name, private-term, shell-prompt: an agent's pull request); 23284/c3c5390c/next-core-after/sho)Tours are data in |
…-suggestions-setting-decides
From the cmux-next PR autofix artifact (run 39623858039): the exports now list sessions.promptSuggestions with default true, plus base's stale agents.subsessions.answerQuestions row. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-5eec-7700-94d7-9a3a1f7d889a&host=cosmic-gray
Default changesThis pull request changes 1 default at
Signed off on this commit by Merge receipt: merged as Regenerate the snapshot with |
|
Review (subagent, correctness first): SHIP at 0a301ca
|
…-suggestions-setting-decides
…-suggestions-setting-decides
…-suggestions-setting-decides # Conflicts: # schemas/settings-export/settings-schema.sources.sha256 # schemas/settings/settings-schema.json
…ebug toggle Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-5eec-7700-94d7-9a3a1f7d889a&host=Air-Blue
…Debug toggle Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-5eec-7700-94d7-9a3a1f7d889a&host=Air-Blue
…-suggestions-setting-decides
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Agent-Session: https://cmux.com/deeplink/agent?harness=claude&session=01a15cfc-5eec-7700-94d7-9a3a1f7d889a&host=Air-Blue
|
Review (subagent, correctness first): SHIP at c3c5390
|
|
Shipped in:
|
![Fix with [code]smith](https://pr-comments-assets.blacksmith.sh/codesmith/fix-with-codesmith-light-3x.png)
Fix forward for #21897, whose merge-audit revert is #23173.
Takes back #23173: #21897 was reverted because its setting could not turn suggestions off (the Debug toggle, on by default, was ORed in). This PR makes the setting decide, so the reason for the revert is gone.
Takes back #22089: it made the Debug toggle
agent.suggestedTaskson by default, and that toggle was ORed in, so the setting could not turn suggestions off. The default now lives in the settingsessions.promptSuggestions(still on), and the toggle only forces suggestions on, so default behaviour is unchanged.#21897 added
sessions.promptSuggestions, but ORed it with theagent.suggestedTasksDebug toggle in two places:AgentPaneSuggestedTasks.isOnandAcpmuxHost's defaultoffersSessionTools. #22089 had made that toggle default to on, so turning the setting off did nothing. #21897's test comparedisOnwith the toggle's value, which hid the bug.What changes:
sessions.promptSuggestionsnow defaults to true, so the default behavior is the same as today. I regeneratedSettingsSchema+Sessions.generated.swiftwithscripts/cmux-next/gen-settings-sources.py.agent.suggestedTasksnow defaults to false and is renamed "Force suggested tasks on": the setting decides, and the toggle only forces suggestions on.AcpmuxHost's default closure now readsAgentPaneSuggestedTasks.forNextDaemonalone. That flag already holdsisOn, so the second OR is gone.forNextDaemonandsettingstart at the setting's default instead offalse.isOn == debugOncheck.AcpmuxOfferToolsTestsnow builds its host the way the app does (setting OR toggle).Exports and the defaults guard:
schemas/settings/settings-schema.json,schemas/defaults/cmux-next.jsonand the docs/mdm exports come from a Swift test. The last commit applies the PR autofix's patch artifact at the final base merge (run 39705775066), so they listpromptSuggestionswithdefault: true.agent.suggestedTasksDebug tunable (on by default since cmux NEXT: suggested tasks on by default #22089), which overrode the setting's oldfalse. They stay on by default; the setting now just decides.setting.sessions.promptSuggestionsas a changed default; since defaults guard: a SHIP review receipt on the head passes; a new setting that defaults off is free #23761 a SHIP receipt at the head clears it.suggestedTasksbool.No capture: no layout or styling changes. The suggested-tasks card is the same card; the only visible changes are that turning Settings > Suggest Follow-Up Tasks off now hides it, and the Debug Settings row's label and help text.
Note: a nightly user who had explicitly turned the Debug toggle off will see suggestions come back on, because that override no longer forces anything off. To turn them off, use Settings > Suggest Follow-Up Tasks.
Note: turning the setting off hides the card at once, but a daemon that is already running keeps offering the session tools (
ACPMUX_OFFER_TOOLS=1) until it restarts.No local build or tests (Leo's Macs never build); CI and
cmux-ci compile-check --appcompile and run the suites.Agent session: Air-Blue
🤖 Generated with Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need.Summary by cubic
Fixes the forward for the reverted #21897 so
sessions.promptSuggestionscan actually turn suggested tasks off. The setting was ORed with theagent.suggestedTasksDebug toggle, which #22089 had made default on, so turning the setting off did nothing.sessions.promptSuggestionsnow defaults to true and decides alone;agent.suggestedTasksdefaults to false and only forces suggestions on. Default behavior is unchanged, andAcpmuxHostreadsAgentPaneSuggestedTasks.forNextDaemonalone. Until the App wires the setting,forNextDaemonandsettingstart at the setting's default.agents.subsessions.answerQuestionsrow.isOnto the toggle's value, hiding the bug; the new test turns the setting off and expects suggestions off.Note: a nightly user who had explicitly turned the Debug toggle off will see suggestions come back on, since that override no longer forces anything off. To turn them off, use Settings > Suggest Follow-Up Tasks.
Written for commit c3c5390. Summary will update on new commits.