[nwparker]

	Files	Added	Deleted	Net
Test	2	$\color{#1a7f37}{\Huge{\mathbf{+}}}$​6	$\color{#cf222e}{\Huge{\mathbf{−}}}$​4	$\color{#1a7f37}{\Huge{\mathbf{+}}}$​2
Prod	0	0	0	0

ELI5

Two tests on main checked a rule using Grok as "an agent whose chats have no permission modes". A change merged the same day gave Grok chats their own modes, so the tests now fail on every PR. They use Gemini instead, which still has none.

What Changed

Test-only. No product behaviour changes.

• fix(native-chat): never run a chat on an older host with more access than Ask for approval allows #29442 (merged 2026-10-15) added a check that stops a chat from running on an older paired host with more access than the person asked for. Its tests used Grok as the agent the check skips.
• fix(acp): OMP, OpenCode and Grok chats follow their own permission mode #29335 (merged 2026-10-16) gave Grok, OpenCode and OMP chats Ask for approval and Full access modes. So the check now applies to Grok chats too: on an older host Grok runs with that host's Agent Permissions, which can mean full access.
• Before: structured-chat-host-permission.test.ts ("asks nothing ... for an agent without chat modes") and agent-chat-permission-host-support.test.ts ("names no mode for an agent without chat modes") fail on main. After: both pass, with Gemini as the agent that has no modes.

Why

#29335 meant to give Grok modes, and putting Grok's chats through the older-host check is the right result, so the product stays as is and the tests change. No released tag includes #29442 without #29335, so no released host advertises per-chat modes but ignores Grok's mode.

The other reported breakage, max-lines on structured-agent-session-host.ts, was already fixed on main by #29869. Root oxlint passes on current main.

Linked Issue

N/A: main was red, no issue filed.

Visual Proof

N/A: test-only change.

Testing

• pnpm test src/renderer/src/runtime src/main/native-chat --maxWorkers=1: 656 files, 5969 tests passed.

• pnpm test src/shared/agent-chat-permission --maxWorkers=1: passed.

• Static-analysis job steps run locally (oxlint, anti-slop, native and type-aware plugins, changed-code quality, React Doctor, max-lines / ts-nocheck / runtime-electron / focus-read / layout-writer ratchets, catalogs, localization, etc.): all passed. Typecheck passed through pnpm tc (the bun-compiler variant typecheck:ci fails locally for an environment reason, "Script not found check").

• I manually tested these changes locally

• Automated tests added/updated, or explained why not below