fix(web): correct pending question attachment message

[dominic-r]

What Changed

Changed the attachment toast to say "Attach files after answering pending questions." Updated the nearby comment to use the same mode-independent terminology.

Why

Attaching a file while an agent question awaits an answer showed "Attach files after answering plan questions," even outside plan mode. The guard checks for pending questions, so the message now describes that condition. Attachment behavior is unchanged.

Validation

• git diff --check passed after rebasing onto upstream main.
• Targeted lint could not run because this checkout is missing vite-plus.
• Copy-only change; no browser verification or screenshots captured.

Model: GPT-6. Harness: Codex.

Note

Fix attachment toast message in ChatComposer to say "pending questions"

Updates the attachment rejection toast and inline paste comments in ChatComposer.tsx. Changes the wording from "plan questions" to "pending questions" or "agent questions" without changing any routing or validation logic.

^{Macroscope summarized ca1f763.}

Summary by CodeRabbit

• Style
  • Updated wording in the attachment notification to refer to pending questions.
  • Clarified related terminology for agent questions.

[macroscopeapp]

Approvability

Verdict: Approved at ca1f763

Macroscope's review found this PR approvable — This PR is limited to correcting terminology in an existing attachment warning and its explanatory comment. No control flow, data model, configuration, defaults, or production infrastructure behavior changes.

^{You can add or adjust custom eligibility rules. Learn more.}

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Team

Run ID: 347fac8c-10c4-4278-85e1-d0fcc8494e1e

📥 Commits

Reviewing files that changed from the base of the PR and between 5ec6f77 and ca1f763.

📒 Files selected for processing (1)

• apps/web/src/components/chat/ChatComposer.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

---

📝 Walkthrough

Walkthrough

ChatComposer.tsx updates the attachment toast and paste-handler comment from “plan questions” terminology to “pending questions” and “agent questions.” No logic or control flow changed.

Changes

Chat composer wording

Layer / File(s)	Summary
Update pending-question wording apps/web/src/components/chat/ChatComposer.tsx	The attachment toast and paste-handler comment now use updated question terminology.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Merge Risk: ⚪ Minimal · up to ca1f7

The attachment warning now correctly refers to pending questions in all relevant modes, without changing attachment or paste behavior. The change is ready to merge.

Suggested reviewers: juliusmarminge, t3dotgg, maria-rcks

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and concisely describes the attachment message fix.
Description check	✅ Passed	The description explains what changed, why it changed, scope, and validation. It omits the template checklist and UI-change section, but the small copy-only change is otherwise documented sufficiently…
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 1…
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

_{Comment @coderabbitai help to get the list of available commands.}