@programinglive/dev-workflow-mcp-server 1.8.1 → 1.10.0

package/tests/handlers-release.test.js

@@ -1,212 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { handleToolCall } from "../tools/handlers.js";
-import { withWorkflowState, testUtils, createRequest, createGitMock } from "./test-helpers.js";
-
-test("perform_release runs release command before pushing with tags", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.currentPhase = "release";
-    workflowState.state.commitAndPushCompleted = true;
-    workflowState.state.readyCheckCompleted = true;
-    workflowState.state.lastPushBranch = "feature/split";
-    workflowState.state.lastCommitMessage = "fix: address issue";
-    await workflowState.save();
-
-    const commands = [];
-    const execStub = async (command) => {
-      commands.push(command);
-      if (command === "git status --porcelain") {
-        return { stdout: "" };
-      }
-      return { stdout: "" };
-    };
-
-    const response = await handleToolCall({
-      request: createRequest("perform_release", { command: "npm run release" }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: execStub,
-      git: createGitMock(),
-      utils: testUtils,
-    });
-
-    const releaseCommand = commands.find((command) => command.startsWith("npm run release"));
-    assert.ok(releaseCommand, "release command should run");
-    assert.ok(
-      commands.some((command) => command.startsWith("git push --follow-tags")),
-      "release should push tags"
-    );
-  });
-});
-
-test("perform_release accepts shorthand patch command", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.currentPhase = "release";
-    workflowState.state.commitAndPushCompleted = true;
-    workflowState.state.readyCheckCompleted = true;
-    workflowState.state.lastCommitMessage = "fix: adjust labels";
-    await workflowState.save();
-
-    const commands = [];
-    const execStub = async (command) => {
-      commands.push(command);
-      if (command === "git status --porcelain") {
-        return { stdout: "" };
-      }
-      return { stdout: "" };
-    };
-
-    await handleToolCall({
-      request: createRequest("perform_release", { command: "patch" }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: execStub,
-      git: createGitMock(),
-      utils: testUtils,
-    });
-
-    const releaseCommand = commands.find((command) => command.startsWith("npm run release"));
-    assert.ok(releaseCommand, "release command should run with patch");
-  });
-});
-
-test("perform_release respects explicit releaseType option", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.currentPhase = "release";
-    workflowState.state.commitAndPushCompleted = true;
-    workflowState.state.readyCheckCompleted = true;
-    await workflowState.save();
-
-    const commands = [];
-    const execStub = async (command) => {
-      commands.push(command);
-      if (command === "git status --porcelain") {
-        return { stdout: "" };
-      }
-      return { stdout: "" };
-    };
-
-    await handleToolCall({
-      request: createRequest("perform_release", {
-        command: "npm run release",
-        releaseType: "minor",
-      }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: execStub,
-      git: createGitMock(),
-      utils: testUtils,
-    });
-
-    const releaseCommand = commands.find((command) => command.startsWith("npm run release"));
-    assert.ok(releaseCommand, "release command should run");
-    assert.ok(
-      releaseCommand.includes("-- --release-as minor"),
-      "release command should append provided releaseType"
-    );
-  });
-});
-
-test("perform_release blocks when new changes detected", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.currentPhase = "release";
-    workflowState.state.commitAndPushCompleted = true;
-    workflowState.state.readyCheckCompleted = true;
-    await workflowState.save();
-
-    const commands = [];
-    const execStub = async (command) => {
-      commands.push(command);
-      if (command === "git status --porcelain") {
-        return { stdout: "?? new-file.js" };
-      }
-      return { stdout: "" };
-    };
-
-    const response = await handleToolCall({
-      request: createRequest("perform_release", { command: "patch" }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: execStub,
-      git: createGitMock({
-        hasWorkingChanges: async () => true,
-        getStatusOutput: async () => "?? new-file.js",
-        containsTestFilesInStatus: (statusOutput) => statusOutput.includes("tests/"),
-        workingTreeSummary: () => ({ hasChanges: true, lines: ["?? new-file.js"] }),
-      }),
-      utils: testUtils,
-    });
-
-    assert.ok(
-      response.content[0].text.includes("run 'commit_and_push' again"),
-      "response should instruct to recommit"
-    );
-    assert.equal(workflowState.state.currentPhase, "commit");
-    assert.equal(workflowState.state.commitAndPushCompleted, false);
-  });
-});
-
-test("perform_release requires commit_and_push completion when tree is clean", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.currentPhase = "release";
-    workflowState.state.commitAndPushCompleted = false;
-    workflowState.state.readyCheckCompleted = true;
-    workflowState.state.lastPushBranch = "";
-    workflowState.state.lastCommitMessage = "";
-    await workflowState.save();
-
-    const commands = [];
-    const execStub = async (command) => {
-      commands.push(command);
-      return { stdout: "" };
-    };
-
-    const response = await handleToolCall({
-      request: createRequest("perform_release", { command: "npm run release" }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: execStub,
-      git: createGitMock({
-        getLastCommitMessage: async () => "chore: already pushed",
-        getStatusOutput: async () => "",
-        containsTestFilesInStatus: (statusOutput) => statusOutput.includes("tests/"),
-      }),
-      utils: testUtils,
-    });
-
-    assert.ok(
-      response.content[0].text.includes("commit_and_push"),
-      "response should mention commit_and_push requirement"
-    );
-  });
-});
-
-test("skip_release records justification and advances to completion phase", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.currentPhase = "release";
-    workflowState.state.commitAndPushCompleted = true;
-    workflowState.state.readyCheckCompleted = true;
-    workflowState.state.lastCommitMessage = "chore: docs";
-    workflowState.state.releaseSkipped = false;
-    workflowState.state.released = false;
-    workflowState.state.releaseCommand = "";
-    await workflowState.save();
-
-    const response = await handleToolCall({
-      request: createRequest("skip_release", { reason: "Python project – no npm release" }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: async () => ({ stdout: "" }),
-      git: createGitMock(),
-      utils: testUtils,
-    });
-
-    const text = response.content[0].text;
-    assert.ok(text.includes("Release step skipped"), "response should mention release was skipped");
-    assert.equal(workflowState.state.releaseSkipped, true);
-    assert.equal(workflowState.state.releaseSkippedReason, "Python project – no npm release");
-    assert.equal(workflowState.state.releaseCommand, "(skipped)");
-    assert.equal(workflowState.state.currentPhase, "ready_to_complete");
-    assert.equal(workflowState.state.released, false);
-  });
-});

package/tests/handlers-workflow.test.js

@@ -1,68 +0,0 @@
-import test from "node:test";
-import assert from "node:assert/strict";
-import { handleToolCall } from "../tools/handlers.js";
-import { withWorkflowState, testUtils, createRequest, createGitMock } from "./test-helpers.js";
-
-test("run_tests guidance mentions core documentation types", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.bugFixed = true;
-    workflowState.state.testsCreated = true;
-    await workflowState.save();
-
-    const response = await handleToolCall({
-      request: createRequest("run_tests", {
-        passed: true,
-        testCommand: "npm test",
-      }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: async () => ({ stdout: "" }),
-      git: createGitMock(),
-      utils: testUtils,
-    });
-
-    const text = response.content[0].text;
-    assert.ok(
-      text.includes("create or update documentation using 'create_documentation'"),
-      "run_tests guidance should mention create or update documentation",
-    );
-    assert.ok(
-      text.includes('"PRD", "README", "RELEASE_NOTES"'),
-      "run_tests guidance should list core documentation types",
-    );
-  });
-});
-
-test("create_documentation verifies PRD file exists", async () => {
-  await withWorkflowState(async (workflowState) => {
-    workflowState.state.testsPassed = true;
-    await workflowState.save();
-
-    const response = await handleToolCall({
-      request: createRequest("create_documentation", {
-        documentationType: "PRD",
-        summary: "Updated PRD for documentation workflow expectations",
-      }),
-      normalizeRequestArgs: (args) => ({ args, error: null }),
-      workflowState,
-      exec: async () => ({ stdout: "" }),
-      git: createGitMock(),
-      utils: testUtils,
-    });
-
-    assert.equal(workflowState.state.documentationCreated, true);
-    assert.equal(workflowState.state.documentationType, "PRD");
-    assert.equal(
-      workflowState.state.documentationSummary,
-      "Updated PRD for documentation workflow expectations",
-    );
-    assert.ok(
-      response.content[0].text.includes("Documentation created/updated!"),
-      "create_documentation response should mention created/updated",
-    );
-    assert.ok(
-      response.content[0].text.includes("PRD verified"),
-      "create_documentation response should verify PRD file",
-    );
-  });
-});