agentHost: hide routine Codex approval review notices

[Giuseppe Cianci (Giuspepe)]

Summary

• suppress routine Codex automatic-approval notices from chat transcripts
• surface timed-out and aborted reviews as compact, collapsible status rows
• preserve denied-review rationale, the Approve anyway flow, and the circuit-breaker warning
• add regression coverage for handler, formatting, metadata, and UI adaptation

Fixes #333180

Testing

• npm run compile-client
• npm run hygiene
• ./scripts/test.sh --grep CodexAgent|codexGuardianReview|stateToProgressAdapter (324 passing)
• ESLint on the changed files
• live Code OSS Agents UI verification for approved, timed-out, aborted, denied, and interrupted states

[Copilot]

Copilot review overview

🟡 Changes recommended

Timeout and failure warnings are suppressed without alternative user feedback.

Once you've addressed the issues Copilot identified, you can request another Copilot review.

Review tier: Balanced
Findings: 1

New issues introduced by this change (1)

Severity	Finding
	src/​vs/​platform/​agentHost/​node/​codex/​codexAgent.ts — This allowlist also drops non-routine failures. In particular, timedOut is a structured review…

What changed in this PR

Suppresses routine Codex guardian review notices while retaining circuit-breaker interruption warnings.

Changes:

• Filters guardian warnings by message prefix.
• Adds coverage for review outcomes and interruptions.
• Updates handler documentation.

File	Description
codexAgent.ts	Filters guardian notifications.
codexAgent.test.ts	Adds regression tests.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.