Address review: prettier, narrow fixture exclusion, fix stale imports

- Run prettier on the 7 files CI flagged.
- Narrow the tests/fixtures/** typecheck exclusion so it no longer covers
  tests/fixtures/crash-run or tests/fixtures/plugins/implement-feature —
  both import real production modules by relative path and crash-run is
  spawned by tests/integration/crash-finalize.test.ts as a live exercise
  of that code. Excluding them would have recreated the gap this PR
  closes. List the remaining hermetic eval fixtures individually instead
  of a blanket glob.
- Typechecking implement-feature surfaced exactly the kind of drift this
  PR targets: its relative imports were two directories short (stale),
  so CommandPlugin/WorkflowPlugin resolved to nothing. Fixed the path
  depth. The fixture isn't wired into any test today, so this was
  invisible until now.

Author: TheGreatAxios

tests/fixtures/plugins/implement-feature/src/index.ts

@@ -1,6 +1,6 @@
 import { implementFeature } from "./workflows/implement-feature.js";
-import type { CommandPlugin } from "../../../src/tui/commands/registry.js";
-import type { WorkflowPlugin } from "../../../src/workflows/definition.js";
+import type { CommandPlugin } from "../../../../../src/tui/commands/registry.js";
+import type { WorkflowPlugin } from "../../../../../src/workflows/definition.js";
 
 export const workflowPlugin: WorkflowPlugin = {
   workflows: [implementFeature],

tests/fixtures/plugins/implement-feature/src/workflows/implement-feature.ts

@@ -1,4 +1,4 @@
-import type { Workflow } from "../../../../src/workflows/definition.js";
+import type { Workflow } from "../../../../../../src/workflows/definition.js";
 
 export const implementFeature: Workflow = {
   name: "implement-feature",

tests/unit/codex-session.test.ts

@@ -122,7 +122,8 @@ describe("getValidCodexToken", () => {
         { name: "p", createdAt: 0, tokens: { access: "old", refresh: "bad", expiresAt: 1_000 } },
         home,
       );
-      globalThis.fetch = (async () => new Response("revoked", { status: 400 })) as unknown as typeof fetch;
+      globalThis.fetch = (async () =>
+        new Response("revoked", { status: 400 })) as unknown as typeof fetch;
       const err = await getValidCodexToken("p", 5_000, home).catch((e: unknown) => e);
       expect(err).toBeInstanceOf(CodexAuthError);
       expect((err as CodexAuthError).reason).toBe("refresh-failed");

tests/unit/resolve-inference-spec.test.ts

@@ -8,7 +8,10 @@ import type { InferenceSpec } from "../../src/agent/profile-types.js";
 
 const baseSettings: Settings = {
   providers: {
-    anthropic: { baseURL: "https://api.anthropic.com", models: ["claude-sonnet-4", "claude-haiku-4"] },
+    anthropic: {
+      baseURL: "https://api.anthropic.com",
+      models: ["claude-sonnet-4", "claude-haiku-4"],
+    },
     xai: { baseURL: "https://api.x.ai", models: ["grok-4"] },
     local: { baseURL: "http://localhost:11434", models: [] }, // empty models list = unrestricted
   },

tests/unit/subagent.test.ts

@@ -742,7 +742,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider,
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [
         {
           id: "p",
@@ -779,7 +781,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider,
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [
         {
           id: "p",
@@ -811,7 +815,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider,
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [
         {
           id: "p",
@@ -852,7 +858,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider: { ...provider, reasoningEffort: "high" },
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [
         {
           id: "p",
@@ -883,7 +891,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider: { ...provider, reasoningEffort: "high" },
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [
         {
           id: "p",
@@ -912,7 +922,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider: { ...provider, reasoningEffort: "low" },
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [
         {
           id: "orch",
@@ -947,7 +959,9 @@ describe("createTaskTool profile resolution", () => {
       cwd: "/repo",
       getWorkdirBase: () => "/repo/.ctx",
       provider,
-      settings: baseSettings as unknown as NonNullable<Parameters<typeof createTaskTool>[0]["settings"]>,
+      settings: baseSettings as unknown as NonNullable<
+        Parameters<typeof createTaskTool>[0]["settings"]
+      >,
       profiles: [{ id: "karen", systemPromptRole: "You are karen.", orchestrator: true }],
       run: async (params) => {
         received = params;

tests/unit/telemetry-product-events.test.ts

@@ -137,10 +137,7 @@ test("skill_used carries no skill name, so an employer-named skill cannot leak",
 
   const tool = createUseSkillTool(cwd, [], telemetry);
   if (tool.kind !== "string") throw new Error(`expected string tool, got ${tool.kind}`);
-  const result = await tool.handler(
-    { name: "acme-internal-deploy" },
-    new AbortController().signal,
-  );
+  const result = await tool.handler({ name: "acme-internal-deploy" }, new AbortController().signal);
 
   // Guard against the test passing because resolution failed: the event only
   // fires on a resolved skill, so a silent miss would trivially "not leak".

tests/unit/tui/agent-tools.test.ts

@@ -70,7 +70,10 @@ mock.module("../../../src/agent/posix-tool-plugins.js", () => ({
 }));
 
 const mockConnectMCPServer = mock(
-  async (config: { name: string }, _options?: import("../../../src/mcp/client.js").MCPConnectOptions) => ({
+  async (
+    config: { name: string },
+    _options?: import("../../../src/mcp/client.js").MCPConnectOptions,
+  ) => ({
     ok: false as const,
     serverName: config.name,
     error: "not connected",

tests/unit/workflows-director.test.ts

@@ -101,7 +101,12 @@ test("the active step directive is injected into the inferred system prompt", as
     type: "message.received",
     message: {
       ref: { uid: 1, mailbox: "INBOX" },
-      headers: { from: "user@test", to: ["agent@test"], date: "1970-01-01T00:00:00Z", messageId: "m1" },
+      headers: {
+        from: "user@test",
+        to: ["agent@test"],
+        date: "1970-01-01T00:00:00Z",
+        messageId: "m1",
+      },
       flags: [],
       signatureStatus: "missing",
       content: "go",

tsconfig.json

@@ -11,14 +11,48 @@
       "@intx/types/*": ["./vendor/intx-types/src/*"]
     }
   },
-  "include": ["src/**/*.ts", "packages/**/*.ts", "tests/**/*.ts", "evals/**/*.ts", "scripts/**/*.ts"],
+  "include": [
+    "src/**/*.ts",
+    "packages/**/*.ts",
+    "tests/**/*.ts",
+    "evals/**/*.ts",
+    "scripts/**/*.ts"
+  ],
   // Excluded: hermetic sandbox repos for the capability eval suite, not real
-  // application code. They own their own package.json/tsconfig-free layout,
-  // are copied into per-run workdirs rather than imported, and some
-  // deliberately contain invalid syntax (e.g. tests/fixtures/broken-toolchain)
-  // as fixture content for an eval case.
+  // application code. They own their own package.json and are copied into
+  // per-run eval workdirs rather than imported, and tests/fixtures/buggy-service
+  // deliberately ships a bug as fixture content for an eval case.
+  //
+  // tests/fixtures/crash-run and tests/fixtures/plugins/implement-feature are
+  // NOT listed here on purpose: they import real production modules by
+  // relative path (src/index.ts, src/session/active-run.ts, src/session/state.ts,
+  // src/tui/runner.ts, src/workflows/definition.ts, src/tui/commands/registry.ts)
+  // and tests/integration/crash-finalize.test.ts spawns them as live exercises
+  // of that code. Excluding them would recreate exactly the silent-drift gap
+  // this config change exists to close.
   "exclude": [
-    "tests/fixtures/**",
+    "tests/fixtures/broken-toolchain/**",
+    "tests/fixtures/buggy-service/**",
+    "tests/fixtures/cart-service/**",
+    "tests/fixtures/codex-sse/**",
+    "tests/fixtures/demo-comparison/**",
+    "tests/fixtures/env-config-build/**",
+    "tests/fixtures/flaky-baseline/**",
+    "tests/fixtures/flaky-cache/**",
+    "tests/fixtures/inventory-service/**",
+    "tests/fixtures/large-read/**",
+    "tests/fixtures/marketplace/**",
+    "tests/fixtures/multi-file-service/**",
+    "tests/fixtures/multiline-edit/**",
+    "tests/fixtures/plugins/exa/**",
+    "tests/fixtures/plugins/example-agent/**",
+    "tests/fixtures/plugins/example-commands/**",
+    "tests/fixtures/plugins/example-tool/**",
+    "tests/fixtures/rawmode-sigint/**",
+    "tests/fixtures/report-pipeline/**",
+    "tests/fixtures/skill-workspace/**",
+    "tests/fixtures/slow-command/**",
+    "tests/fixtures/web-note/**",
     "evals/capability/cases/**"
   ]
 }