product-discovery-cli 0.0.14 → 0.0.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "product-discovery-cli",
-  "version": "0.0.14",
+  "version": "0.0.15",
   "description": "CLI to consume the Product Discovery Agent API",
   "license": "Apache-2.0",
   "author": "AuronForge",

package/src/application/RunDiscoveryFlow.spec.js

@@ -0,0 +1,222 @@
+const { RunDiscoveryFlow } = require("./RunDiscoveryFlow");
+
+const mockI18n = {
+  t: jest.fn((key) => key)
+};
+
+describe("RunDiscoveryFlow", () => {
+  test("runs discovery, saves, and stops", async () => {
+    const prompt = {
+      askInput: jest.fn().mockResolvedValue("My idea"),
+      askYesNo: jest.fn().mockResolvedValue(false)
+    };
+    const presenter = {
+      printHeader: jest.fn(),
+      info: jest.fn(),
+      error: jest.fn(),
+      json: jest.fn(),
+      success: jest.fn(),
+      goodbye: jest.fn(),
+      spinner: jest.fn().mockReturnValue({ succeed: jest.fn(), fail: jest.fn() })
+    };
+    const apiClient = {
+      runDiscovery: jest.fn().mockResolvedValue({ name: "Test Product" })
+    };
+    const storage = {
+      saveJson: jest.fn().mockResolvedValue({ saved: true, fullPath: "/tmp/file.json" })
+    };
+
+    const useCase = new RunDiscoveryFlow({ prompt, apiClient, storage, presenter });
+    await useCase.execute({
+      apiUrl: "http://localhost/api",
+      lang: "pt-br",
+      saveDefaults: { autoSave: true },
+      i18n: mockI18n
+    });
+
+    expect(presenter.printHeader).toHaveBeenCalled();
+    expect(prompt.askInput).toHaveBeenCalled();
+    expect(apiClient.runDiscovery).toHaveBeenCalledWith("My idea", "http://localhost/api", "pt-br");
+    expect(storage.saveJson).toHaveBeenCalled();
+    expect(presenter.goodbye).toHaveBeenCalled();
+  });
+
+  test("asks to improve when not saving and exits on no", async () => {
+    const prompt = {
+      askInput: jest.fn().mockResolvedValue("My idea"),
+      askYesNo: jest.fn().mockResolvedValueOnce(false)
+    };
+    const presenter = {
+      printHeader: jest.fn(),
+      info: jest.fn(),
+      error: jest.fn(),
+      json: jest.fn(),
+      success: jest.fn(),
+      goodbye: jest.fn(),
+      spinner: jest.fn().mockReturnValue({ succeed: jest.fn(), fail: jest.fn() })
+    };
+    const apiClient = {
+      runDiscovery: jest.fn().mockResolvedValue({ name: "Test Product" })
+    };
+    const storage = {
+      saveJson: jest.fn()
+    };
+
+    const useCase = new RunDiscoveryFlow({ prompt, apiClient, storage, presenter });
+    await useCase.execute({
+      apiUrl: "http://localhost/api",
+      lang: "pt-br",
+      saveDefaults: { autoSave: false },
+      i18n: mockI18n
+    });
+
+    expect(prompt.askYesNo).toHaveBeenCalledWith("askImprove");
+    expect(storage.saveJson).not.toHaveBeenCalled();
+    expect(presenter.goodbye).toHaveBeenCalled();
+  });
+
+  test("retries when discovery fails and user chooses to retry", async () => {
+    const prompt = {
+      askInput: jest.fn().mockResolvedValue("My idea"),
+      askYesNo: jest
+        .fn()
+        .mockResolvedValueOnce(true)
+        .mockResolvedValueOnce(false)
+    };
+    const presenter = {
+      printHeader: jest.fn(),
+      info: jest.fn(),
+      error: jest.fn(),
+      json: jest.fn(),
+      success: jest.fn(),
+      goodbye: jest.fn(),
+      spinner: jest.fn().mockReturnValue({ succeed: jest.fn(), fail: jest.fn() })
+    };
+    const apiClient = {
+      runDiscovery: jest
+        .fn()
+        .mockRejectedValueOnce(new Error("API down"))
+        .mockResolvedValueOnce({ name: "Recovered" })
+    };
+    const storage = {
+      saveJson: jest.fn().mockResolvedValue({ saved: true, fullPath: "/tmp/file.json" })
+    };
+
+    const useCase = new RunDiscoveryFlow({ prompt, apiClient, storage, presenter });
+    await useCase.execute({
+      apiUrl: "http://localhost/api",
+      lang: "pt-br",
+      saveDefaults: { autoSave: true },
+      i18n: mockI18n
+    });
+
+    expect(apiClient.runDiscovery).toHaveBeenCalledTimes(2);
+    expect(presenter.error).toHaveBeenCalledWith("API down");
+    expect(presenter.goodbye).toHaveBeenCalled();
+  });
+
+  test("prompts to improve after save declined", async () => {
+    const prompt = {
+      askInput: jest.fn().mockResolvedValue("My idea"),
+      askYesNo: jest
+        .fn()
+        .mockResolvedValueOnce(true)
+        .mockResolvedValueOnce(false)
+    };
+    const presenter = {
+      printHeader: jest.fn(),
+      info: jest.fn(),
+      error: jest.fn(),
+      json: jest.fn(),
+      success: jest.fn(),
+      goodbye: jest.fn(),
+      spinner: jest.fn().mockReturnValue({ succeed: jest.fn(), fail: jest.fn() })
+    };
+    const apiClient = {
+      runDiscovery: jest.fn().mockResolvedValue({ name: "Test Product" })
+    };
+    const storage = {
+      saveJson: jest.fn().mockResolvedValue({ saved: false })
+    };
+
+    const useCase = new RunDiscoveryFlow({ prompt, apiClient, storage, presenter });
+    await useCase.execute({
+      apiUrl: "http://localhost/api",
+      lang: "pt-br",
+      saveDefaults: { autoSave: true },
+      i18n: mockI18n
+    });
+
+    expect(storage.saveJson).toHaveBeenCalled();
+    expect(prompt.askYesNo).toHaveBeenCalledWith("askImprove");
+    expect(presenter.goodbye).toHaveBeenCalled();
+  });
+
+  test("stops when discovery fails and user declines retry", async () => {
+    const prompt = {
+      askInput: jest.fn().mockResolvedValue("My idea"),
+      askYesNo: jest.fn().mockResolvedValue(false)
+    };
+    const presenter = {
+      printHeader: jest.fn(),
+      info: jest.fn(),
+      error: jest.fn(),
+      json: jest.fn(),
+      success: jest.fn(),
+      goodbye: jest.fn(),
+      spinner: jest.fn().mockReturnValue({ succeed: jest.fn(), fail: jest.fn() })
+    };
+    const apiClient = {
+      runDiscovery: jest.fn().mockRejectedValue(new Error("API down"))
+    };
+    const storage = {
+      saveJson: jest.fn()
+    };
+
+    const useCase = new RunDiscoveryFlow({ prompt, apiClient, storage, presenter });
+    await useCase.execute({
+      apiUrl: "http://localhost/api",
+      lang: "pt-br",
+      saveDefaults: { autoSave: true },
+      i18n: mockI18n
+    });
+
+    expect(prompt.askYesNo).toHaveBeenCalledWith("askRetry");
+    expect(storage.saveJson).not.toHaveBeenCalled();
+    expect(presenter.goodbye).toHaveBeenCalled();
+  });
+
+  test("continues when autoSave false and user wants to improve", async () => {
+    const prompt = {
+      askInput: jest.fn().mockResolvedValue("My idea"),
+      askYesNo: jest.fn().mockResolvedValueOnce(true).mockResolvedValueOnce(false)
+    };
+    const presenter = {
+      printHeader: jest.fn(),
+      info: jest.fn(),
+      error: jest.fn(),
+      json: jest.fn(),
+      success: jest.fn(),
+      goodbye: jest.fn(),
+      spinner: jest.fn().mockReturnValue({ succeed: jest.fn(), fail: jest.fn() })
+    };
+    const apiClient = {
+      runDiscovery: jest.fn().mockResolvedValue({ name: "Test Product" })
+    };
+    const storage = {
+      saveJson: jest.fn()
+    };
+
+    const useCase = new RunDiscoveryFlow({ prompt, apiClient, storage, presenter });
+    await useCase.execute({
+      apiUrl: "http://localhost/api",
+      lang: "pt-br",
+      saveDefaults: { autoSave: false },
+      i18n: mockI18n
+    });
+
+    expect(apiClient.runDiscovery).toHaveBeenCalledTimes(2);
+    expect(prompt.askYesNo).toHaveBeenCalledWith("askImprove");
+    expect(presenter.goodbye).toHaveBeenCalled();
+  });
+});

package/src/domain/DiscoverySession.spec.js

@@ -0,0 +1,12 @@
+const { DiscoverySession } = require("./DiscoverySession");
+
+describe("DiscoverySession", () => {
+  test("appends additional details after base idea", () => {
+    const session = new DiscoverySession();
+    const first = session.addIdea("Base idea");
+    const second = session.addIdea("More details");
+
+    expect(first).toBe("Base idea");
+    expect(second).toBe("Base idea\n\nAdditional details: More details");
+  });
+});

package/src/domain/PathNaming.spec.js

@@ -0,0 +1,26 @@
+const {
+  sanitizeForFolderName,
+  timestampForPath,
+  ensureJsonExtension
+} = require("./PathNaming");
+
+describe("PathNaming", () => {
+  test("sanitizeForFolderName normalizes input", () => {
+    expect(sanitizeForFolderName("My Product!! 2024")).toBe("my-product-2024");
+  });
+
+  test("sanitizeForFolderName falls back to product", () => {
+    expect(sanitizeForFolderName("###")).toBe("product");
+  });
+
+  test("timestampForPath returns a safe string", () => {
+    const value = timestampForPath();
+    expect(value).toMatch(/\d{4}-\d{2}-\d{2}T/);
+    expect(value).not.toMatch(/[:.]/);
+  });
+
+  test("ensureJsonExtension appends extension", () => {
+    expect(ensureJsonExtension("discovery")).toBe("discovery.json");
+    expect(ensureJsonExtension("report.JSON")).toBe("report.JSON");
+  });
+});

package/src/index.js

@@ -7,7 +7,7 @@ const { ProductDiscoveryApi } = require("./infrastructure/ProductDiscoveryApi");
 const { JsonFileStorage } = require("./infrastructure/JsonFileStorage");
 const { RunDiscoveryFlow } = require("./application/RunDiscoveryFlow");
 
-const DEFAULT_API_URL = process.env.API_URL || "http://localhost:3001/api/v1/discovery";
+const DEFAULT_API_URL = process.env.API_URL || "http://localhost:3000/api/v1/discovery";
 
 const prompt = new PromptService();
 const presenter = new ConsolePresenter(null);