@dexto/tools-plan 1.6.0 → 1.6.2

package/dist/tools/plan-create-tool.js

@@ -1,83 +1,78 @@
-/**
- * Plan Create Tool
- *
- * Creates a new implementation plan for the current session.
- * Shows a preview for approval before saving.
- */
-import { z } from 'zod';
-import { defineTool } from '@dexto/core';
-import { PlanError } from '../errors.js';
-const PlanCreateInputSchema = z
-    .object({
-    title: z.string().describe('Plan title (e.g., "Add User Authentication")'),
-    content: z
-        .string()
-        .describe('Plan content in markdown format. Use - [ ] and - [x] for checkboxes to track progress.'),
-})
-    .strict();
-/**
- * Creates the plan_create tool
- */
-export function createPlanCreateTool(getPlanService) {
-    return defineTool({
-        id: 'plan_create',
-        displayName: 'Plan',
-        description: 'Create a new implementation plan for the current session. Shows the plan for approval before saving. Use markdown format for the plan content with clear steps and file references.',
-        inputSchema: PlanCreateInputSchema,
-        /**
-         * Generate preview for approval UI
-         */
-        generatePreview: async (input, context) => {
-            const { content } = input;
-            if (!context.sessionId) {
-                throw PlanError.sessionIdRequired();
-            }
-            const resolvedPlanService = await getPlanService(context);
-            // Check if plan already exists
-            const exists = await resolvedPlanService.exists(context.sessionId);
-            if (exists) {
-                throw PlanError.planAlreadyExists(context.sessionId);
-            }
-            // Return preview for approval UI
-            const lineCount = content.split('\n').length;
-            const planPath = resolvedPlanService.getPlanPath(context.sessionId);
-            return {
-                type: 'file',
-                path: planPath,
-                operation: 'create',
-                content,
-                size: Buffer.byteLength(content, 'utf8'),
-                lineCount,
-            };
-        },
-        async execute(input, context) {
-            const { title, content } = input;
-            if (!context.sessionId) {
-                throw PlanError.sessionIdRequired();
-            }
-            const resolvedPlanService = await getPlanService(context);
-            // Keep consistent with generatePreview: fail early if plan already exists.
-            // (PlanService.create also guards this, but this keeps the control flow obvious.)
-            const exists = await resolvedPlanService.exists(context.sessionId);
-            if (exists) {
-                throw PlanError.planAlreadyExists(context.sessionId);
-            }
-            const plan = await resolvedPlanService.create(context.sessionId, content, { title });
-            const planPath = resolvedPlanService.getPlanPath(context.sessionId);
-            const _display = {
-                type: 'file',
-                path: planPath,
-                operation: 'create',
-                size: Buffer.byteLength(content, 'utf8'),
-                lineCount: content.split('\n').length,
-            };
-            return {
-                success: true,
-                path: planPath,
-                status: plan.meta.status,
-                title: plan.meta.title,
-                _display,
-            };
-        },
-    });
+import { z } from "zod";
+import { createLocalToolCallHeader, defineTool, truncateForHeader } from "@dexto/core";
+import { PlanError } from "../errors.js";
+const PlanCreateInputSchema = z.object({
+  title: z.string().describe('Plan title (e.g., "Add User Authentication")'),
+  content: z.string().describe(
+    "Plan content in markdown format. Use - [ ] and - [x] for checkboxes to track progress."
+  )
+}).strict();
+function createPlanCreateTool(getPlanService) {
+  return defineTool({
+    id: "plan_create",
+    description: "Create a new implementation plan for the current session. Shows the plan for approval before saving. Use markdown format for the plan content with clear steps and file references.",
+    inputSchema: PlanCreateInputSchema,
+    presentation: {
+      describeHeader: (input) => createLocalToolCallHeader({
+        title: "Create Plan",
+        argsText: truncateForHeader(input.title, 140)
+      }),
+      /**
+       * Generate preview for approval UI
+       */
+      preview: async (input, context) => {
+        const { content } = input;
+        if (!context.sessionId) {
+          throw PlanError.sessionIdRequired();
+        }
+        const resolvedPlanService = await getPlanService(context);
+        const exists = await resolvedPlanService.exists(context.sessionId);
+        if (exists) {
+          throw PlanError.planAlreadyExists(context.sessionId);
+        }
+        const lineCount = content.split("\n").length;
+        const planPath = resolvedPlanService.getPlanPath(context.sessionId);
+        return {
+          type: "file",
+          title: "Create Plan",
+          path: planPath,
+          operation: "create",
+          content,
+          size: Buffer.byteLength(content, "utf8"),
+          lineCount
+        };
+      }
+    },
+    async execute(input, context) {
+      const { title, content } = input;
+      if (!context.sessionId) {
+        throw PlanError.sessionIdRequired();
+      }
+      const resolvedPlanService = await getPlanService(context);
+      const exists = await resolvedPlanService.exists(context.sessionId);
+      if (exists) {
+        throw PlanError.planAlreadyExists(context.sessionId);
+      }
+      const plan = await resolvedPlanService.create(context.sessionId, content, { title });
+      const planPath = resolvedPlanService.getPlanPath(context.sessionId);
+      const _display = {
+        type: "file",
+        title: "Create Plan",
+        path: planPath,
+        operation: "create",
+        size: Buffer.byteLength(content, "utf8"),
+        lineCount: content.split("\n").length
+      };
+      return {
+        success: true,
+        path: planPath,
+        status: plan.meta.status,
+        title: plan.meta.title,
+        _display
+      };
+    }
+  });
 }
+export {
+  createPlanCreateTool
+};

package/dist/tools/plan-create-tool.test.cjs

@@ -0,0 +1,174 @@
+"use strict";
+var __create = Object.create;
+var __defProp = Object.defineProperty;
+var __getOwnPropDesc = Object.getOwnPropertyDescriptor;
+var __getOwnPropNames = Object.getOwnPropertyNames;
+var __getProtoOf = Object.getPrototypeOf;
+var __hasOwnProp = Object.prototype.hasOwnProperty;
+var __copyProps = (to, from, except, desc) => {
+  if (from && typeof from === "object" || typeof from === "function") {
+    for (let key of __getOwnPropNames(from))
+      if (!__hasOwnProp.call(to, key) && key !== except)
+        __defProp(to, key, { get: () => from[key], enumerable: !(desc = __getOwnPropDesc(from, key)) || desc.enumerable });
+  }
+  return to;
+};
+var __toESM = (mod, isNodeMode, target) => (target = mod != null ? __create(__getProtoOf(mod)) : {}, __copyProps(
+  // If the importer is in node compatibility mode or this is not an ESM
+  // file that has been converted to a CommonJS file using a Babel-
+  // compatible transform (i.e. "__esModule" has not been set), then set
+  // "default" to the CommonJS "module.exports" for node compatibility.
+  isNodeMode || !mod || !mod.__esModule ? __defProp(target, "default", { value: mod, enumerable: true }) : target,
+  mod
+));
+var import_vitest = require("vitest");
+var path = __toESM(require("node:path"), 1);
+var fs = __toESM(require("node:fs/promises"), 1);
+var os = __toESM(require("node:os"), 1);
+var import_plan_create_tool = require("./plan-create-tool.js");
+var import_plan_service = require("../plan-service.js");
+var import_errors = require("../errors.js");
+var import_core = require("@dexto/core");
+const createMockLogger = () => {
+  const logger = {
+    debug: import_vitest.vi.fn(),
+    silly: import_vitest.vi.fn(),
+    info: import_vitest.vi.fn(),
+    warn: import_vitest.vi.fn(),
+    error: import_vitest.vi.fn(),
+    trackException: import_vitest.vi.fn(),
+    createChild: import_vitest.vi.fn(() => logger),
+    createFileOnlyChild: import_vitest.vi.fn(() => logger),
+    setLevel: import_vitest.vi.fn(),
+    getLevel: import_vitest.vi.fn(() => "debug"),
+    getLogFilePath: import_vitest.vi.fn(() => null),
+    destroy: import_vitest.vi.fn(async () => void 0)
+  };
+  return logger;
+};
+function createToolContext(logger, overrides = {}) {
+  return { logger, ...overrides };
+}
+(0, import_vitest.describe)("plan_create tool", () => {
+  let logger;
+  let tempDir;
+  let planService;
+  (0, import_vitest.beforeEach)(async () => {
+    logger = createMockLogger();
+    const rawTempDir = await fs.mkdtemp(path.join(os.tmpdir(), "dexto-plan-create-test-"));
+    tempDir = await fs.realpath(rawTempDir);
+    planService = new import_plan_service.PlanService({ basePath: tempDir }, logger);
+    import_vitest.vi.clearAllMocks();
+  });
+  (0, import_vitest.afterEach)(async () => {
+    try {
+      await fs.rm(tempDir, { recursive: true, force: true });
+    } catch {
+    }
+  });
+  (0, import_vitest.describe)("generatePreview", () => {
+    (0, import_vitest.it)("should return FileDisplayData for new plan", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      const sessionId = "test-session";
+      const content = "# Implementation Plan\n\n## Steps\n1. First step";
+      const previewFn = tool.presentation?.preview;
+      (0, import_vitest.expect)(previewFn).toBeDefined();
+      const preview = await previewFn(
+        { title: "Test Plan", content },
+        createToolContext(logger, { sessionId })
+      );
+      (0, import_vitest.expect)(preview.type).toBe("file");
+      (0, import_vitest.expect)(preview.title).toBe("Create Plan");
+      (0, import_vitest.expect)(preview.operation).toBe("create");
+      (0, import_vitest.expect)(preview.path).toContain(sessionId);
+      (0, import_vitest.expect)(preview.path).toMatch(/plan\.md$/);
+      (0, import_vitest.expect)(preview.content).toBe(content);
+      (0, import_vitest.expect)(preview.lineCount).toBe(4);
+    });
+    (0, import_vitest.it)("should throw error when sessionId is missing", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      const previewFn = tool.presentation?.preview;
+      (0, import_vitest.expect)(previewFn).toBeDefined();
+      try {
+        await previewFn({ title: "Test", content: "# Plan" }, createToolContext(logger));
+        import_vitest.expect.fail("Should have thrown an error");
+      } catch (error) {
+        (0, import_vitest.expect)(error).toBeInstanceOf(import_core.DextoRuntimeError);
+        (0, import_vitest.expect)(error.code).toBe(import_errors.PlanErrorCode.SESSION_ID_REQUIRED);
+      }
+    });
+    (0, import_vitest.it)("should throw error when plan already exists", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      const sessionId = "test-session";
+      const previewFn = tool.presentation?.preview;
+      (0, import_vitest.expect)(previewFn).toBeDefined();
+      await planService.create(sessionId, "# Existing Plan");
+      try {
+        await previewFn(
+          { title: "New Plan", content: "# New Content" },
+          createToolContext(logger, { sessionId })
+        );
+        import_vitest.expect.fail("Should have thrown an error");
+      } catch (error) {
+        (0, import_vitest.expect)(error).toBeInstanceOf(import_core.DextoRuntimeError);
+        (0, import_vitest.expect)(error.code).toBe(import_errors.PlanErrorCode.PLAN_ALREADY_EXISTS);
+      }
+    });
+  });
+  (0, import_vitest.describe)("execute", () => {
+    (0, import_vitest.it)("should create plan and return success", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      const sessionId = "test-session";
+      const content = "# Implementation Plan";
+      const title = "My Plan";
+      const result = await tool.execute(
+        { title, content },
+        createToolContext(logger, { sessionId })
+      );
+      (0, import_vitest.expect)(result.success).toBe(true);
+      (0, import_vitest.expect)(result.path).toContain(sessionId);
+      (0, import_vitest.expect)(result.path).toMatch(/plan\.md$/);
+      (0, import_vitest.expect)(result.status).toBe("draft");
+      (0, import_vitest.expect)(result.title).toBe(title);
+    });
+    (0, import_vitest.it)("should throw error when plan already exists", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      const sessionId = "test-session";
+      await planService.create(sessionId, "# Existing Plan");
+      try {
+        await tool.execute(
+          { title: "New Plan", content: "# New content" },
+          createToolContext(logger, { sessionId })
+        );
+        import_vitest.expect.fail("Should have thrown an error");
+      } catch (error) {
+        (0, import_vitest.expect)(error).toBeInstanceOf(import_core.DextoRuntimeError);
+        (0, import_vitest.expect)(error.code).toBe(import_errors.PlanErrorCode.PLAN_ALREADY_EXISTS);
+      }
+    });
+    (0, import_vitest.it)("should throw error when sessionId is missing", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      try {
+        await tool.execute({ title: "Test", content: "# Plan" }, createToolContext(logger));
+        import_vitest.expect.fail("Should have thrown an error");
+      } catch (error) {
+        (0, import_vitest.expect)(error).toBeInstanceOf(import_core.DextoRuntimeError);
+        (0, import_vitest.expect)(error.code).toBe(import_errors.PlanErrorCode.SESSION_ID_REQUIRED);
+      }
+    });
+    (0, import_vitest.it)("should include _display data in result", async () => {
+      const tool = (0, import_plan_create_tool.createPlanCreateTool)(async () => planService);
+      const sessionId = "test-session";
+      const content = "# Plan\n## Steps";
+      const result = await tool.execute(
+        { title: "Plan", content },
+        createToolContext(logger, { sessionId })
+      );
+      (0, import_vitest.expect)(result._display).toBeDefined();
+      (0, import_vitest.expect)(result._display.type).toBe("file");
+      (0, import_vitest.expect)(result._display.title).toBe("Create Plan");
+      (0, import_vitest.expect)(result._display.operation).toBe("create");
+      (0, import_vitest.expect)(result._display.lineCount).toBe(2);
+    });
+  });
+});

package/dist/tools/plan-create-tool.test.js

@@ -1,143 +1,151 @@
-/**
- * Plan Create Tool Tests
- *
- * Tests for the plan_create tool including preview generation.
- */
-import { describe, it, expect, beforeEach, afterEach, vi } from 'vitest';
-import * as path from 'node:path';
-import * as fs from 'node:fs/promises';
-import * as os from 'node:os';
-import { createPlanCreateTool } from './plan-create-tool.js';
-import { PlanService } from '../plan-service.js';
-import { PlanErrorCode } from '../errors.js';
-import { DextoRuntimeError } from '@dexto/core';
-// Create mock logger
+import { describe, it, expect, beforeEach, afterEach, vi } from "vitest";
+import * as path from "node:path";
+import * as fs from "node:fs/promises";
+import * as os from "node:os";
+import { createPlanCreateTool } from "./plan-create-tool.js";
+import { PlanService } from "../plan-service.js";
+import { PlanErrorCode } from "../errors.js";
+import { DextoRuntimeError } from "@dexto/core";
 const createMockLogger = () => {
-    const logger = {
-        debug: vi.fn(),
-        silly: vi.fn(),
-        info: vi.fn(),
-        warn: vi.fn(),
-        error: vi.fn(),
-        trackException: vi.fn(),
-        createChild: vi.fn(() => logger),
-        setLevel: vi.fn(),
-        getLevel: vi.fn(() => 'debug'),
-        getLogFilePath: vi.fn(() => null),
-        destroy: vi.fn(async () => undefined),
-    };
-    return logger;
+  const logger = {
+    debug: vi.fn(),
+    silly: vi.fn(),
+    info: vi.fn(),
+    warn: vi.fn(),
+    error: vi.fn(),
+    trackException: vi.fn(),
+    createChild: vi.fn(() => logger),
+    createFileOnlyChild: vi.fn(() => logger),
+    setLevel: vi.fn(),
+    getLevel: vi.fn(() => "debug"),
+    getLogFilePath: vi.fn(() => null),
+    destroy: vi.fn(async () => void 0)
+  };
+  return logger;
 };
 function createToolContext(logger, overrides = {}) {
-    return { logger, ...overrides };
+  return { logger, ...overrides };
 }
-describe('plan_create tool', () => {
-    let logger;
-    let tempDir;
-    let planService;
-    beforeEach(async () => {
-        logger = createMockLogger();
-        // Create temp directory for testing
-        const rawTempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'dexto-plan-create-test-'));
-        tempDir = await fs.realpath(rawTempDir);
-        planService = new PlanService({ basePath: tempDir }, logger);
-        vi.clearAllMocks();
+describe("plan_create tool", () => {
+  let logger;
+  let tempDir;
+  let planService;
+  beforeEach(async () => {
+    logger = createMockLogger();
+    const rawTempDir = await fs.mkdtemp(path.join(os.tmpdir(), "dexto-plan-create-test-"));
+    tempDir = await fs.realpath(rawTempDir);
+    planService = new PlanService({ basePath: tempDir }, logger);
+    vi.clearAllMocks();
+  });
+  afterEach(async () => {
+    try {
+      await fs.rm(tempDir, { recursive: true, force: true });
+    } catch {
+    }
+  });
+  describe("generatePreview", () => {
+    it("should return FileDisplayData for new plan", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      const sessionId = "test-session";
+      const content = "# Implementation Plan\n\n## Steps\n1. First step";
+      const previewFn = tool.presentation?.preview;
+      expect(previewFn).toBeDefined();
+      const preview = await previewFn(
+        { title: "Test Plan", content },
+        createToolContext(logger, { sessionId })
+      );
+      expect(preview.type).toBe("file");
+      expect(preview.title).toBe("Create Plan");
+      expect(preview.operation).toBe("create");
+      expect(preview.path).toContain(sessionId);
+      expect(preview.path).toMatch(/plan\.md$/);
+      expect(preview.content).toBe(content);
+      expect(preview.lineCount).toBe(4);
     });
-    afterEach(async () => {
-        try {
-            await fs.rm(tempDir, { recursive: true, force: true });
-        }
-        catch {
-            // Ignore cleanup errors
-        }
+    it("should throw error when sessionId is missing", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      const previewFn = tool.presentation?.preview;
+      expect(previewFn).toBeDefined();
+      try {
+        await previewFn({ title: "Test", content: "# Plan" }, createToolContext(logger));
+        expect.fail("Should have thrown an error");
+      } catch (error) {
+        expect(error).toBeInstanceOf(DextoRuntimeError);
+        expect(error.code).toBe(PlanErrorCode.SESSION_ID_REQUIRED);
+      }
     });
-    describe('generatePreview', () => {
-        it('should return FileDisplayData for new plan', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            const sessionId = 'test-session';
-            const content = '# Implementation Plan\n\n## Steps\n1. First step';
-            const preview = (await tool.generatePreview({ title: 'Test Plan', content }, createToolContext(logger, { sessionId })));
-            expect(preview.type).toBe('file');
-            expect(preview.operation).toBe('create');
-            // Path is now absolute, check it ends with the expected suffix
-            expect(preview.path).toContain(sessionId);
-            expect(preview.path).toMatch(/plan\.md$/);
-            expect(preview.content).toBe(content);
-            expect(preview.lineCount).toBe(4);
-        });
-        it('should throw error when sessionId is missing', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            try {
-                await tool.generatePreview({ title: 'Test', content: '# Plan' }, createToolContext(logger));
-                expect.fail('Should have thrown an error');
-            }
-            catch (error) {
-                expect(error).toBeInstanceOf(DextoRuntimeError);
-                expect(error.code).toBe(PlanErrorCode.SESSION_ID_REQUIRED);
-            }
-        });
-        it('should throw error when plan already exists', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            const sessionId = 'test-session';
-            // Create existing plan
-            await planService.create(sessionId, '# Existing Plan');
-            try {
-                await tool.generatePreview({ title: 'New Plan', content: '# New Content' }, createToolContext(logger, { sessionId }));
-                expect.fail('Should have thrown an error');
-            }
-            catch (error) {
-                expect(error).toBeInstanceOf(DextoRuntimeError);
-                expect(error.code).toBe(PlanErrorCode.PLAN_ALREADY_EXISTS);
-            }
-        });
+    it("should throw error when plan already exists", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      const sessionId = "test-session";
+      const previewFn = tool.presentation?.preview;
+      expect(previewFn).toBeDefined();
+      await planService.create(sessionId, "# Existing Plan");
+      try {
+        await previewFn(
+          { title: "New Plan", content: "# New Content" },
+          createToolContext(logger, { sessionId })
+        );
+        expect.fail("Should have thrown an error");
+      } catch (error) {
+        expect(error).toBeInstanceOf(DextoRuntimeError);
+        expect(error.code).toBe(PlanErrorCode.PLAN_ALREADY_EXISTS);
+      }
     });
-    describe('execute', () => {
-        it('should create plan and return success', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            const sessionId = 'test-session';
-            const content = '# Implementation Plan';
-            const title = 'My Plan';
-            const result = (await tool.execute({ title, content }, createToolContext(logger, { sessionId })));
-            expect(result.success).toBe(true);
-            // Path is now absolute, check it ends with the expected suffix
-            expect(result.path).toContain(sessionId);
-            expect(result.path).toMatch(/plan\.md$/);
-            expect(result.status).toBe('draft');
-            expect(result.title).toBe(title);
-        });
-        it('should throw error when plan already exists', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            const sessionId = 'test-session';
-            await planService.create(sessionId, '# Existing Plan');
-            try {
-                await tool.execute({ title: 'New Plan', content: '# New content' }, createToolContext(logger, { sessionId }));
-                expect.fail('Should have thrown an error');
-            }
-            catch (error) {
-                expect(error).toBeInstanceOf(DextoRuntimeError);
-                expect(error.code).toBe(PlanErrorCode.PLAN_ALREADY_EXISTS);
-            }
-        });
-        it('should throw error when sessionId is missing', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            try {
-                await tool.execute({ title: 'Test', content: '# Plan' }, createToolContext(logger));
-                expect.fail('Should have thrown an error');
-            }
-            catch (error) {
-                expect(error).toBeInstanceOf(DextoRuntimeError);
-                expect(error.code).toBe(PlanErrorCode.SESSION_ID_REQUIRED);
-            }
-        });
-        it('should include _display data in result', async () => {
-            const tool = createPlanCreateTool(async () => planService);
-            const sessionId = 'test-session';
-            const content = '# Plan\n## Steps';
-            const result = (await tool.execute({ title: 'Plan', content }, createToolContext(logger, { sessionId })));
-            expect(result._display).toBeDefined();
-            expect(result._display.type).toBe('file');
-            expect(result._display.operation).toBe('create');
-            expect(result._display.lineCount).toBe(2);
-        });
+  });
+  describe("execute", () => {
+    it("should create plan and return success", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      const sessionId = "test-session";
+      const content = "# Implementation Plan";
+      const title = "My Plan";
+      const result = await tool.execute(
+        { title, content },
+        createToolContext(logger, { sessionId })
+      );
+      expect(result.success).toBe(true);
+      expect(result.path).toContain(sessionId);
+      expect(result.path).toMatch(/plan\.md$/);
+      expect(result.status).toBe("draft");
+      expect(result.title).toBe(title);
     });
+    it("should throw error when plan already exists", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      const sessionId = "test-session";
+      await planService.create(sessionId, "# Existing Plan");
+      try {
+        await tool.execute(
+          { title: "New Plan", content: "# New content" },
+          createToolContext(logger, { sessionId })
+        );
+        expect.fail("Should have thrown an error");
+      } catch (error) {
+        expect(error).toBeInstanceOf(DextoRuntimeError);
+        expect(error.code).toBe(PlanErrorCode.PLAN_ALREADY_EXISTS);
+      }
+    });
+    it("should throw error when sessionId is missing", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      try {
+        await tool.execute({ title: "Test", content: "# Plan" }, createToolContext(logger));
+        expect.fail("Should have thrown an error");
+      } catch (error) {
+        expect(error).toBeInstanceOf(DextoRuntimeError);
+        expect(error.code).toBe(PlanErrorCode.SESSION_ID_REQUIRED);
+      }
+    });
+    it("should include _display data in result", async () => {
+      const tool = createPlanCreateTool(async () => planService);
+      const sessionId = "test-session";
+      const content = "# Plan\n## Steps";
+      const result = await tool.execute(
+        { title: "Plan", content },
+        createToolContext(logger, { sessionId })
+      );
+      expect(result._display).toBeDefined();
+      expect(result._display.type).toBe("file");
+      expect(result._display.title).toBe("Create Plan");
+      expect(result._display.operation).toBe("create");
+      expect(result._display.lineCount).toBe(2);
+    });
+  });
 });