Skip to content

Chief prompt: import suggestion cards are on Settings' Import page, not Home/New Tab - #33465

Merged
lawrencecchen merged 1 commit into
feat-cmux-nextfrom
chief/import-suggest-settings-wording
Nov 3, 2026
Merged

lawrencecchen merged 1 commit into
feat-cmux-nextfrom
chief/import-suggest-settings-wording

Conversation

@teamleaderleo

@teamleaderleo teamleaderleo commented Nov 3, 2026 •

Copy link
Copy Markdown
Collaborator

The Chief prompt tells subagents that cmux import suggest raises a card "on Home and New Tab". Since #24352 (Leo, 2026-10-21: import only in Settings and the command palette), a raised card shows only on the Settings › Import page, so the prompt sends agents and the person looking in the wrong place.

Change: one line in Native/OptChat/chief-kernel/src/prompt.rs now says the card lands on Settings' Import page. No behaviour change; prompt text only. optchat-chief/toolpacks/browser/SKILL.md already says only "to offer it later" and needs no change.

Risk: none beyond prompt wording (Chief area, so the owner is requested for awareness).

Agent session: agent session

🤖 Generated with Claude Code


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


Summary by cubic

Fixes the Chief prompt's outdated instruction that cmux import suggest raises a card on Home and New Tab. Cards now appear only on Settings' Import page, so the prompt text has been updated to match. Prompt wording only; no behavior change.

Written for commit 02f206e. Summary will update on new commits.

View guided diff Turn on auto-fix

#24352 removed import suggestion cards from Home and New Tab (Leo, 2026-10-21),
so a card raised with `cmux import suggest` shows only in Settings > Import.
The Chief prompt still told subagents it lands on Home and New Tab.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@github-actions

github-actions Bot commented Nov 3, 2026 •

Copy link
Copy Markdown
Contributor

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

Review 02f206e: 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=pr33465 -f ref=chief/import-suggest-settings-wording -f platform=macos -f backend_mode=local

Shipped in:

@github-actions github-actions Bot added the risk:normal Risk tier (docs/ci/risk-tiers.md) label Nov 3, 2026
@teamleaderleo

Copy link
Copy Markdown
Collaborator Author

Review (subagent claude-sonnet-5-5): SHIP at 02f206e
Visual verdict: unchanged by this diff (prompt text only).
Checked: ImportActionCatalog.swift on feat-cmux-next confirms cards live only in Settings › Import (#24352); the edited line is in a plain format! literal, so the apostrophe is safe; no test pins the old sentence.
Follow-ups: none.

@lawrencecchen for awareness: one-line Chief prompt wording fix in chief-kernel/src/prompt.rs (you're the most recent author there). Not a wait gate.

@teamleaderleo teamleaderleo added the ready-to-land Reviewed at this head; next-autoland merges it into feat-cmux-next when its only reds are base reds label Nov 3, 2026
@blacksmith-sh

blacksmith-sh Bot commented Nov 3, 2026

Copy link
Copy Markdown

Found 1 test failure on Blacksmith runners:

Failure

Test View Logs
nightly preparation must run on Blacksmith Linux, keep its fallback, and reuse a succes
sful artifact/
nightly preparation must run on Blacksmith Linux, keep its fallback, and reuse a succes
sful artifact
View Logs

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

@lawrencecchen
lawrencecchen merged commit 227f2dd into feat-cmux-next Nov 3, 2026
191 of 201 checks passed
@lawrencecchen
lawrencecchen deleted the chief/import-suggest-settings-wording branch November 3, 2026 09:43

This branch was previously deployed

1 inactive deployment
ci-routing-agent — 02f206e9 Deployed Nov 3, 2026 by teamleaderleo via wait for same-key tree publisher #105098
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ready-to-land Reviewed at this head; next-autoland merges it into feat-cmux-next when its only reds are base reds risk:normal Risk tier (docs/ci/risk-tiers.md)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants