test: assert directory-outline honesty at the rendered plain-text surface

The e2e expectations still parsed the legacy directory-JSON envelope.
Assert the plain-text contract instead, at the surface the agent sees:
partial-result and skipped-file footers rather than raw JSON fields.

Author: ualtinok

packages/opencode-plugin/src/__tests__/e2e/honest-reporting.test.ts

@@ -122,16 +122,14 @@ export function runHonestReportingSuite(
       const output = toolResultText(
         await tools.aft_outline.execute({ target: "outline-small" }, sdkCtx),
       );
-      const response = JSON.parse(output) as Record<string, unknown>;
-
-      expect(response.complete).toBe(true);
-      expect(response.walk_truncated).toBe(false);
-      const skipped = response.skipped_files as Array<{ file: string; reason: string }>;
-      expect(skipped).toHaveLength(1);
-      expect(skipped[0].file).toMatch(/outline-small[/\\]bad\.ts$/);
-      expect(skipped[0].reason).toBe("parse_error");
-      expect(String(response.text)).toContain("good.ts");
-      expect(String(response.text)).toContain("good");
+
+      // Directory outlines are plain text; honesty data arrives as footers.
+      expect(() => JSON.parse(output)).toThrow();
+      expect(output).toContain("good.ts");
+      expect(output).toContain("good");
+      expect(output).not.toContain("⚠ Partial result");
+      expect(output).toContain("Skipped 1 file(s):");
+      expect(output).toMatch(/bad\.ts — parse_error/);
     });
 
     test("aft_outline directory mode returns complete false when Rust walk truncates", async () => {
@@ -148,12 +146,11 @@ export function runHonestReportingSuite(
       const output = toolResultText(
         await tools.aft_outline.execute({ target: "outline-large" }, sdkCtx),
       );
-      const response = JSON.parse(output) as Record<string, unknown>;
 
-      expect(response.complete).toBe(false);
-      expect(response.walk_truncated).toBe(true);
-      expect(Array.isArray(response.skipped_files)).toBe(true);
-      expect(String(response.text)).toContain("file-000.ts");
+      // Truncation is disclosed in the plain-text partial-result footer.
+      expect(() => JSON.parse(output)).toThrow();
+      expect(output).toContain("file-000.ts");
+      expect(output).toContain("⚠ Partial result: walk truncated at 200 files.");
     });
 
     test("aft_outline single file target keeps text output behavior", async () => {

packages/opencode-plugin/src/__tests__/e2e/read-only-spine-toolcall.test.ts

@@ -174,25 +174,20 @@ export function runReadOnlySpineToolcallSuite(
       expect(output).toContain("ToolCallOutlineService");
     });
 
-    test("aft_outline returns structured directory JSON through tool_call", async () => {
+    test("aft_outline returns plain-text directory outline through tool_call", async () => {
       const h = await harness();
       const tools = readingTools(createPluginContext(h));
 
       const output = await tools.aft_outline.execute({ target: "src" }, createToolContext(h));
-      const parsed = JSON.parse(output) as {
-        success?: boolean;
-        complete?: boolean;
-        text?: string;
-        skipped_files?: unknown[];
-      };
-
-      expect(parsed.success).toBe(true);
-      expect(parsed.complete).toBe(true);
-      expect(parsed.text).toContain("src/");
-      expect(parsed.text).toContain("hit.ts");
-      expect(parsed.text).toContain("toolCallOutlineFunction");
-      expect(parsed.text).not.toContain("hit.test.ts");
-      expect(parsed.skipped_files).toEqual([]);
+
+      // Directory outlines render as plain text like every other outline
+      // mode; a complete walk carries no partial-result footer.
+      expect(() => JSON.parse(output)).toThrow();
+      expect(output).toContain("src/");
+      expect(output).toContain("hit.ts");
+      expect(output).toContain("toolCallOutlineFunction");
+      expect(output).not.toContain("hit.test.ts");
+      expect(output).not.toContain("⚠ Partial result");
     });
 
     test("aft_outline files:true returns the server-rendered files tree through tool_call", async () => {
@@ -230,20 +225,19 @@ export function runReadOnlySpineToolcallSuite(
       const h = await harness();
       const tools = readingTools(createPluginContext(h));
 
-      const withoutTests = JSON.parse(
-        await tools.aft_outline.execute({ target: "src" }, createToolContext(h)),
-      ) as { text?: string };
-      const withTests = JSON.parse(
-        await tools.aft_outline.execute(
-          { target: "src", includeTests: true },
-          createToolContext(h),
-        ),
-      ) as { text?: string };
-
-      expect(withoutTests.text).not.toContain("hit.test.ts");
-      expect(withoutTests.text).not.toContain("toolCallOutlineTestOnly");
-      expect(withTests.text).toContain("hit.test.ts");
-      expect(withTests.text).toContain("toolCallOutlineTestOnly");
+      const withoutTests = await tools.aft_outline.execute(
+        { target: "src" },
+        createToolContext(h),
+      );
+      const withTests = await tools.aft_outline.execute(
+        { target: "src", includeTests: true },
+        createToolContext(h),
+      );
+
+      expect(withoutTests).not.toContain("hit.test.ts");
+      expect(withoutTests).not.toContain("toolCallOutlineTestOnly");
+      expect(withTests).toContain("hit.test.ts");
+      expect(withTests).toContain("toolCallOutlineTestOnly");
     });
   });
 }