gsd-pi 2.44.0-dev.73f2fd5 → 2.44.0-dev.8894d5b

package/src/resources/extensions/gsd/prompts/validate-milestone.md

@@ -16,8 +16,6 @@ All relevant context has been preloaded below — the roadmap, all slice summari
 
 {{inlinedContext}}
 
-{{skillActivation}}
-
 ## Validation Steps
 
 1. For each **success criterion** in `{{roadmapPath}}`, check whether slice summaries and UAT results provide evidence that it was met. Record pass/fail per criterion.
@@ -27,15 +25,47 @@ All relevant context has been preloaded below — the roadmap, all slice summari
 5. Determine a verdict:
    - `pass` — all criteria met, all slices delivered, no gaps
    - `needs-attention` — minor gaps that do not block completion (document them)
-   - `needs-remediation` — material gaps found; remediation slices must be added to the roadmap
+   - `needs-remediation` — material gaps found; add remediation slices to the roadmap
+
+## Output
+
+Write `{{validationPath}}` with this structure:
+
+```markdown
+---
+verdict: <pass|needs-attention|needs-remediation>
+remediation_round: {{remediationRound}}
+---
+
+# Milestone Validation: {{milestoneId}}
 
-## Persist Validation
+## Success Criteria Checklist
+- [x] Criterion 1 — evidence: ...
+- [ ] Criterion 2 — gap: ...
 
-**Persist validation results through `gsd_validate_milestone`.** Call it with: `milestoneId`, `verdict`, `remediationRound`, `successCriteriaChecklist`, `sliceDeliveryAudit`, `crossSliceIntegration`, `requirementCoverage`, `verdictRationale`, and `remediationPlan` (if verdict is `needs-remediation`). The tool writes the validation to the DB and renders VALIDATION.md to disk.
+## Slice Delivery Audit
+| Slice | Claimed | Delivered | Status |
+|-------|---------|-----------|--------|
+| S01   | ...     | ...       | pass   |
+
+## Cross-Slice Integration
+(any boundary mismatches)
+
+## Requirement Coverage
+(any unaddressed requirements)
+
+## Verdict Rationale
+(why this verdict was chosen)
+
+## Remediation Plan
+(only if verdict is needs-remediation — list new slices to add to the roadmap)
+```
 
 If verdict is `needs-remediation`:
-- After calling `gsd_validate_milestone`, use `gsd_reassess_roadmap` to add remediation slices. Pass `milestoneId`, a synthetic `completedSliceId` (e.g. "VALIDATION"), `verdict: "roadmap-adjusted"`, `assessment` text, and `sliceChanges` with the new slices in the `added` array. The tool persists the changes to the DB and re-renders ROADMAP.md.
-- These remediation slices will be planned and executed before validation re-runs.
+- Add new slices to `{{roadmapPath}}` with unchecked `[ ]` status
+- These slices will be planned and executed before validation re-runs
+
+**You MUST write `{{validationPath}}` before finishing.**
 
 **File system safety:** When scanning milestone directories for evidence, use `ls` or `find` to list directory contents first — never pass a directory path (e.g. `tasks/`, `slices/`) directly to the `read` tool. The `read` tool only accepts file paths, not directories.
 

package/src/resources/extensions/gsd/provider-error-pause.ts

@@ -19,11 +19,6 @@ export function classifyProviderError(errorMsg: string): {
   const isRateLimit = /rate.?limit|too many requests|429/i.test(errorMsg);
   const isServerError = /internal server error|500|502|503|overloaded|server_error|api_error|service.?unavailable/i.test(errorMsg);
 
-  // Connection/process errors — transient, auto-resume after brief backoff (#2309).
-  // These indicate the process was killed, the connection was reset, or a network
-  // blip occurred. They are NOT permanent failures.
-  const isConnectionError = /terminated|connection.?reset|connection.?refused|other side closed|fetch failed|network.?(?:is\s+)?unavailable|ECONNREFUSED|ECONNRESET|EPIPE/i.test(errorMsg);
-
   // Permanent errors — never auto-resume
   const isPermanent = /auth|unauthorized|forbidden|invalid.*key|invalid.*api|billing|quota exceeded|account/i.test(errorMsg);
 
@@ -42,10 +37,6 @@ export function classifyProviderError(errorMsg: string): {
     return { isTransient: true, isRateLimit: false, suggestedDelayMs: 30_000 }; // 30s for server errors
   }
 
-  if (isConnectionError) {
-    return { isTransient: true, isRateLimit: false, suggestedDelayMs: 15_000 }; // 15s for connection errors
-  }
-
   // Unknown error — treat as permanent (user reviews)
   return { isTransient: false, isRateLimit: false, suggestedDelayMs: 0 };
 }

package/src/resources/extensions/gsd/tests/db-writer.test.ts

@@ -483,85 +483,6 @@ describe('db-writer', () => {
     }
   });
 
-  test('saveArtifactToDb — shrinkage guard preserves larger existing file', async () => {
-    const tmpDir = makeTmpDir();
-    const dbPath = path.join(tmpDir, '.gsd', 'gsd.db');
-    openDatabase(dbPath);
-
-    try {
-      const fullContent = '# Full Research\n\n' + 'x'.repeat(20000) + '\n';
-      const abbreviatedContent = '# Summary\n\nShort version.\n';
-
-      // Pre-create the file with full content (simulating a prior `write` tool call)
-      const relPath = 'milestones/M001/M001-RESEARCH.md';
-      const filePath = path.join(tmpDir, '.gsd', relPath);
-      fs.mkdirSync(path.dirname(filePath), { recursive: true });
-      fs.writeFileSync(filePath, fullContent);
-
-      // Call saveArtifactToDb with abbreviated content — should trigger shrinkage guard
-      await saveArtifactToDb({
-        path: relPath,
-        artifact_type: 'RESEARCH',
-        content: abbreviatedContent,
-        milestone_id: 'M001',
-      }, tmpDir);
-
-      // Disk file should be preserved (not overwritten)
-      assert.deepStrictEqual(
-        fs.readFileSync(filePath, 'utf-8'),
-        fullContent,
-        'disk file preserved — shrinkage guard prevented overwrite',
-      );
-
-      // DB should contain the full disk content, not the abbreviated content
-      const adapter = _getAdapter();
-      const row = adapter!
-        .prepare('SELECT full_content FROM artifacts WHERE path = ?')
-        .get(relPath);
-      assert.deepStrictEqual(
-        row!['full_content'],
-        fullContent,
-        'DB stores the richer disk content instead of abbreviated content',
-      );
-    } finally {
-      closeDatabase();
-      cleanupDir(tmpDir);
-    }
-  });
-
-  test('saveArtifactToDb — allows overwrite when new content is similar size', async () => {
-    const tmpDir = makeTmpDir();
-    const dbPath = path.join(tmpDir, '.gsd', 'gsd.db');
-    openDatabase(dbPath);
-
-    try {
-      const oldContent = '# Summary v1\n\nOriginal content here.\n';
-      const newContent = '# Summary v2\n\nUpdated content here with more details.\n';
-
-      const relPath = 'milestones/M001/M001-SUMMARY.md';
-      const filePath = path.join(tmpDir, '.gsd', relPath);
-      fs.mkdirSync(path.dirname(filePath), { recursive: true });
-      fs.writeFileSync(filePath, oldContent);
-
-      await saveArtifactToDb({
-        path: relPath,
-        artifact_type: 'SUMMARY',
-        content: newContent,
-        milestone_id: 'M001',
-      }, tmpDir);
-
-      // Disk file should be updated (new content is >=50% of old size)
-      assert.deepStrictEqual(
-        fs.readFileSync(filePath, 'utf-8'),
-        newContent,
-        'disk file updated when new content is similar size',
-      );
-    } finally {
-      closeDatabase();
-      cleanupDir(tmpDir);
-    }
-  });
-
   // ═══════════════════════════════════════════════════════════════════════════
   // Full Round-Trip: DB → Markdown → Parse → Compare
   // ═══════════════════════════════════════════════════════════════════════════

package/src/resources/extensions/gsd/tests/infra-error.test.ts

@@ -7,13 +7,10 @@ import { isInfrastructureError, INFRA_ERROR_CODES } from "../auto/infra-errors.j
 // ── INFRA_ERROR_CODES constant ───────────────────────────────────────────────
 
 test("INFRA_ERROR_CODES contains the expected codes", () => {
-  for (const code of [
-    "ENOSPC", "ENOMEM", "EROFS", "EDQUOT", "EMFILE", "ENFILE",
-    "ECONNREFUSED", "ENOTFOUND", "ENETUNREACH",
-  ]) {
+  for (const code of ["ENOSPC", "ENOMEM", "EROFS", "EDQUOT", "EMFILE", "ENFILE"]) {
     assert.ok(INFRA_ERROR_CODES.has(code), `missing ${code}`);
   }
-  assert.equal(INFRA_ERROR_CODES.size, 9, "unexpected extra codes");
+  assert.equal(INFRA_ERROR_CODES.size, 6, "unexpected extra codes");
 });
 
 // ── isInfrastructureError: code property detection ───────────────────────────
@@ -48,21 +45,6 @@ test("detects ENFILE via code property", () => {
   assert.equal(isInfrastructureError(err), "ENFILE");
 });
 
-test("detects ECONNREFUSED via code property", () => {
-  const err = Object.assign(new Error("connect ECONNREFUSED 127.0.0.1:3000"), { code: "ECONNREFUSED" });
-  assert.equal(isInfrastructureError(err), "ECONNREFUSED");
-});
-
-test("detects ENOTFOUND via code property", () => {
-  const err = Object.assign(new Error("getaddrinfo ENOTFOUND api.example.com"), { code: "ENOTFOUND" });
-  assert.equal(isInfrastructureError(err), "ENOTFOUND");
-});
-
-test("detects ENETUNREACH via code property", () => {
-  const err = Object.assign(new Error("connect ENETUNREACH 2607:f8b0:4004::"), { code: "ENETUNREACH" });
-  assert.equal(isInfrastructureError(err), "ENETUNREACH");
-});
-
 // ── isInfrastructureError: message fallback ──────────────────────────────────
 
 test("falls back to message scanning when no code property", () => {

package/src/resources/extensions/gsd/tests/prompt-contracts.test.ts

@@ -147,12 +147,12 @@ test("plan-slice prompt no longer frames direct PLAN writes as the source of tru
   assert.match(prompt, /Do \*\*not\*\* rely on direct `PLAN\.md` writes as the source of truth/i);
 });
 
-test("plan-slice prompt explicitly names gsd_plan_slice as DB-backed planning tool", () => {
+test("plan-slice prompt explicitly names gsd_plan_slice and gsd_plan_task as DB-backed planning tools", () => {
   const prompt = readPrompt("plan-slice");
   assert.match(prompt, /gsd_plan_slice/);
   assert.match(prompt, /gsd_plan_task/);
-  // The prompt should describe the DB-backed tool as the canonical write path
-  assert.match(prompt, /DB-backed tool is the canonical write path/i);
+  // The prompt should describe these as the canonical write path
+  assert.match(prompt, /DB-backed tools are the canonical write path/i);
 });
 
 test("plan-slice prompt does not instruct direct file writes as a primary step", () => {
@@ -161,18 +161,14 @@ test("plan-slice prompt does not instruct direct file writes as a primary step",
   assert.doesNotMatch(prompt, /^\d+\.\s+Write `?\{\{outputPath\}\}`?\s*$/m);
 });
 
-test("plan-slice prompt clarifies gsd_plan_slice handles task persistence", () => {
+test("plan-slice prompt instructs calling gsd_plan_task for each task", () => {
   const prompt = readPrompt("plan-slice");
-  // gsd_plan_slice persists tasks in its transaction — no separate gsd_plan_task calls needed
-  assert.match(prompt, /gsd_plan_task/);
-  assert.match(prompt, /gsd_plan_slice` handles task persistence/i);
+  assert.match(prompt, /call `gsd_plan_task` for each task/i);
 });
 
-test("replan-slice prompt uses gsd_replan_slice as canonical DB-backed tool", () => {
+test("replan-slice prompt requires DB-backed planning state when available", () => {
   const prompt = readPrompt("replan-slice");
-  assert.match(prompt, /gsd_replan_slice/);
-  // Degraded fallback (direct file writes) was removed — DB tools are always available
-  assert.doesNotMatch(prompt, /Degraded fallback/i);
+  assert.match(prompt, /DB-backed planning tool exists for this phase, use it as the source of truth/i);
 });
 
 test("reassess-roadmap prompt references gsd_reassess_roadmap tool", () => {

package/src/resources/extensions/gsd/tests/tool-naming.test.ts

@@ -34,7 +34,6 @@ const RENAME_MAP: Array<{ canonical: string; alias: string }> = [
   { canonical: "gsd_replan_slice", alias: "gsd_slice_replan" },
   { canonical: "gsd_reassess_roadmap", alias: "gsd_roadmap_reassess" },
   { canonical: "gsd_complete_milestone", alias: "gsd_milestone_complete" },
-  { canonical: "gsd_validate_milestone", alias: "gsd_milestone_validate" },
 ];
 
 // ─── Registration count ──────────────────────────────────────────────────────
@@ -44,7 +43,7 @@ console.log('\n── Tool naming: registration count ──');
 const pi = makeMockPi();
 registerDbTools(pi);
 
-assert.deepStrictEqual(pi.tools.length, 26, 'Should register exactly 26 tools (13 canonical + 13 aliases)');
+assert.deepStrictEqual(pi.tools.length, 24, 'Should register exactly 24 tools (12 canonical + 12 aliases)');
 
 // ─── Both names exist for each pair ──────────────────────────────────────────
 

package/src/resources/extensions/gsd/tools/plan-slice.ts

@@ -20,7 +20,6 @@ export interface PlanSliceTaskInput {
   inputs: string[];
   expectedOutput: string[];
   observabilityImpact?: string;
-  fullPlanMd?: string;
 }
 
 export interface PlanSliceParams {
@@ -168,7 +167,6 @@ export async function handlePlanSlice(
           inputs: task.inputs,
           expectedOutput: task.expectedOutput,
           observabilityImpact: task.observabilityImpact ?? "",
-          fullPlanMd: task.fullPlanMd,
         });
       }
     });

package/src/resources/extensions/gsd/tools/plan-task.ts

@@ -15,7 +15,6 @@ export interface PlanTaskParams {
   inputs: string[];
   expectedOutput: string[];
   observabilityImpact?: string;
-  fullPlanMd?: string;
 }
 
 export interface PlanTaskResult {
@@ -95,7 +94,6 @@ export async function handlePlanTask(
         inputs: params.inputs,
         expectedOutput: params.expectedOutput,
         observabilityImpact: params.observabilityImpact ?? "",
-        fullPlanMd: params.fullPlanMd,
       });
     });
   } catch (err) {

package/src/resources/extensions/gsd/tools/replan-slice.ts

@@ -21,7 +21,6 @@ export interface ReplanSliceTaskInput {
   verify: string;
   inputs: string[];
   expectedOutput: string[];
-  fullPlanMd?: string;
 }
 
 export interface ReplanSliceParams {
@@ -137,7 +136,6 @@ export async function handleReplanSlice(
             verify: updatedTask.verify || "",
             inputs: updatedTask.inputs || [],
             expectedOutput: updatedTask.expectedOutput || [],
-            fullPlanMd: updatedTask.fullPlanMd,
           });
         } else {
           // Insert new task then set planning fields
@@ -156,7 +154,6 @@ export async function handleReplanSlice(
             verify: updatedTask.verify || "",
             inputs: updatedTask.inputs || [],
             expectedOutput: updatedTask.expectedOutput || [],
-            fullPlanMd: updatedTask.fullPlanMd,
           });
         }
       }

package/src/resources/extensions/gsd/worktree-resolver.ts

@@ -17,7 +17,6 @@ import { existsSync, unlinkSync } from "node:fs";
 import { join } from "node:path";
 import type { AutoSession } from "./auto/session.js";
 import { debugLog } from "./debug-logger.js";
-import { MergeConflictError } from "./git-service.js";
 
 // ─── Dependency Interface ──────────────────────────────────────────────────
 
@@ -434,12 +433,6 @@ export class WorktreeResolver {
           /* best-effort */
         }
       }
-
-      // Re-throw MergeConflictError so the auto loop can detect real code
-      // conflicts and stop instead of retrying forever (#2330).
-      if (err instanceof MergeConflictError) {
-        throw err;
-      }
     }
 
     // Always restore basePath and rebuild — whether merge succeeded or failed

package/src/resources/extensions/mcp-client/index.ts

@@ -213,26 +213,6 @@ function formatToolList(serverName: string, tools: McpToolSchema[]): string {
 	return lines.join("\n");
 }
 
-// ─── Status helper (consumed by /gsd mcp) ─────────────────────────────────────
-
-/**
- * Return the live connection status for a named MCP server.
- * Safe to call even when the server has never been connected.
- */
-export function getConnectionStatus(name: string): {
-	connected: boolean;
-	tools: string[];
-	error?: string;
-} {
-	const conn = connections.get(name);
-	const cached = toolCache.get(name);
-	return {
-		connected: !!conn,
-		tools: cached ? cached.map((t) => t.name) : [],
-		error: undefined,
-	};
-}
-
 // ─── Extension ────────────────────────────────────────────────────────────────
 
 export default function (pi: ExtensionAPI) {

package/dist/resources/extensions/gsd/commands-mcp-status.js

@@ -1,187 +0,0 @@
-/**
- * MCP Status — `/gsd mcp` command handler.
- *
- * Shows configured MCP servers, their connection status, and available tools.
- *
- * Subcommands:
- *   /gsd mcp             — Overview of all servers (alias: /gsd mcp status)
- *   /gsd mcp status      — Same as bare /gsd mcp
- *   /gsd mcp check <srv> — Detailed status for a specific server
- */
-import { existsSync, readFileSync } from "node:fs";
-import { join } from "node:path";
-function readMcpConfigs() {
-    const servers = [];
-    const seen = new Set();
-    const configPaths = [
-        join(process.cwd(), ".mcp.json"),
-        join(process.cwd(), ".gsd", "mcp.json"),
-    ];
-    for (const configPath of configPaths) {
-        try {
-            if (!existsSync(configPath))
-                continue;
-            const raw = readFileSync(configPath, "utf-8");
-            const data = JSON.parse(raw);
-            const mcpServers = (data.mcpServers ?? data.servers);
-            if (!mcpServers || typeof mcpServers !== "object")
-                continue;
-            for (const [name, config] of Object.entries(mcpServers)) {
-                if (seen.has(name))
-                    continue;
-                seen.add(name);
-                const hasCommand = typeof config.command === "string";
-                const hasUrl = typeof config.url === "string";
-                const transport = hasCommand
-                    ? "stdio"
-                    : hasUrl
-                        ? "http"
-                        : "unknown";
-                servers.push({
-                    name,
-                    transport,
-                    ...(hasCommand && {
-                        command: config.command,
-                        args: Array.isArray(config.args) ? config.args : undefined,
-                    }),
-                    ...(hasUrl && { url: config.url }),
-                });
-            }
-        }
-        catch {
-            // Non-fatal — config file may not exist or be malformed
-        }
-    }
-    return servers;
-}
-// ─── Formatters (exported for testing) ──────────────────────────────────────
-export function formatMcpStatusReport(servers) {
-    if (servers.length === 0) {
-        return [
-            "No MCP servers configured.",
-            "",
-            "Add servers to .mcp.json or .gsd/mcp.json to enable MCP integrations.",
-            "See: https://modelcontextprotocol.io/quickstart",
-        ].join("\n");
-    }
-    const lines = [`MCP Server Status — ${servers.length} server(s)\n`];
-    for (const s of servers) {
-        const icon = s.error ? "✗" : s.connected ? "✓" : "○";
-        const status = s.error
-            ? `error: ${s.error}`
-            : s.connected
-                ? `connected — ${s.toolCount} tools`
-                : "disconnected";
-        lines.push(`  ${icon} ${s.name} (${s.transport}) — ${status}`);
-    }
-    lines.push("");
-    lines.push("Use /gsd mcp check <server> for details on a specific server.");
-    lines.push("Use mcp_discover to connect and list tools for a server.");
-    return lines.join("\n");
-}
-export function formatMcpServerDetail(server) {
-    const lines = [`MCP Server: ${server.name}\n`];
-    lines.push(`  Transport: ${server.transport}`);
-    if (server.error) {
-        lines.push(`  Status:    error`);
-        lines.push(`  Error:     ${server.error}`);
-    }
-    else if (server.connected) {
-        lines.push(`  Status:    connected`);
-        lines.push(`  Tools:     ${server.toolCount}`);
-        if (server.tools.length > 0) {
-            lines.push("");
-            lines.push("  Available tools:");
-            for (const tool of server.tools) {
-                lines.push(`    - ${tool}`);
-            }
-        }
-    }
-    else {
-        lines.push(`  Status:    disconnected`);
-        lines.push("");
-        lines.push(`  Run mcp_discover("${server.name}") to connect and list tools.`);
-    }
-    return lines.join("\n");
-}
-// ─── Command handler ────────────────────────────────────────────────────────
-/**
- * Handle `/gsd mcp [status|check <server>]`.
- */
-export async function handleMcpStatus(args, ctx) {
-    const trimmed = args.trim().toLowerCase();
-    const configs = readMcpConfigs();
-    // /gsd mcp check <server>
-    if (trimmed.startsWith("check ")) {
-        const serverName = args.trim().slice("check ".length).trim();
-        const config = configs.find((c) => c.name === serverName);
-        if (!config) {
-            const available = configs.map((c) => c.name).join(", ") || "(none)";
-            ctx.ui.notify(`Unknown MCP server: "${serverName}"\n\nAvailable: ${available}`, "warning");
-            return;
-        }
-        // Try to get connection/tool info from the mcp-client module if available
-        let connected = false;
-        let toolNames = [];
-        let error;
-        try {
-            const mcpClient = await import("../mcp-client/index.js");
-            // Access the module's connection state if exported; fall back gracefully
-            const mod = mcpClient;
-            if (typeof mod.getConnectionStatus === "function") {
-                const status = mod.getConnectionStatus(serverName);
-                connected = status.connected;
-                toolNames = status.tools;
-                error = status.error;
-            }
-        }
-        catch {
-            // mcp-client may not expose status helpers — that's fine
-        }
-        ctx.ui.notify(formatMcpServerDetail({
-            name: config.name,
-            transport: config.transport,
-            connected,
-            toolCount: toolNames.length,
-            tools: toolNames,
-            error,
-        }), "info");
-        return;
-    }
-    // /gsd mcp or /gsd mcp status
-    if (!trimmed || trimmed === "status") {
-        // Build status for each server
-        const statuses = [];
-        for (const config of configs) {
-            let connected = false;
-            let toolCount = 0;
-            let error;
-            try {
-                const mcpClient = await import("../mcp-client/index.js");
-                const mod = mcpClient;
-                if (typeof mod.getConnectionStatus === "function") {
-                    const status = mod.getConnectionStatus(config.name);
-                    connected = status.connected;
-                    toolCount = status.tools.length;
-                    error = status.error;
-                }
-            }
-            catch {
-                // Fall back to unknown state
-            }
-            statuses.push({
-                name: config.name,
-                transport: config.transport,
-                connected,
-                toolCount,
-                error,
-            });
-        }
-        ctx.ui.notify(formatMcpStatusReport(statuses), "info");
-        return;
-    }
-    // Unknown subcommand
-    ctx.ui.notify("Usage: /gsd mcp [status|check <server>]\n\n" +
-        "  status           Show all MCP server statuses (default)\n" +
-        "  check <server>   Detailed status for a specific server", "warning");
-}

package/dist/resources/extensions/gsd/tools/validate-milestone.js

@@ -1,88 +0,0 @@
-/**
- * validate-milestone handler — the core operation behind gsd_validate_milestone.
- *
- * Persists milestone validation results to the assessments table,
- * renders VALIDATION.md to disk, and invalidates caches.
- */
-import { join } from "node:path";
-import { transaction, _getAdapter, } from "../gsd-db.js";
-import { resolveMilestonePath, clearPathCache } from "../paths.js";
-import { saveFile, clearParseCache } from "../files.js";
-import { invalidateStateCache } from "../state.js";
-function renderValidationMarkdown(params) {
-    let md = `---
-verdict: ${params.verdict}
-remediation_round: ${params.remediationRound}
----
-
-# Milestone Validation: ${params.milestoneId}
-
-## Success Criteria Checklist
-${params.successCriteriaChecklist}
-
-## Slice Delivery Audit
-${params.sliceDeliveryAudit}
-
-## Cross-Slice Integration
-${params.crossSliceIntegration}
-
-## Requirement Coverage
-${params.requirementCoverage}
-
-## Verdict Rationale
-${params.verdictRationale}
-`;
-    if (params.verdict === "needs-remediation" && params.remediationPlan) {
-        md += `\n## Remediation Plan\n${params.remediationPlan}\n`;
-    }
-    return md;
-}
-export async function handleValidateMilestone(params, basePath) {
-    if (!params.milestoneId || typeof params.milestoneId !== "string" || params.milestoneId.trim() === "") {
-        return { error: "milestoneId is required and must be a non-empty string" };
-    }
-    const validVerdicts = ["pass", "needs-attention", "needs-remediation"];
-    if (!validVerdicts.includes(params.verdict)) {
-        return { error: `verdict must be one of: ${validVerdicts.join(", ")}` };
-    }
-    // ── Filesystem render ──────────────────────────────────────────────────
-    const validationMd = renderValidationMarkdown(params);
-    let validationPath;
-    const milestoneDir = resolveMilestonePath(basePath, params.milestoneId);
-    if (milestoneDir) {
-        validationPath = join(milestoneDir, `${params.milestoneId}-VALIDATION.md`);
-    }
-    else {
-        const gsdDir = join(basePath, ".gsd");
-        const manualDir = join(gsdDir, "milestones", params.milestoneId);
-        validationPath = join(manualDir, `${params.milestoneId}-VALIDATION.md`);
-    }
-    try {
-        await saveFile(validationPath, validationMd);
-    }
-    catch (renderErr) {
-        process.stderr.write(`gsd-db: validate_milestone — disk render failed: ${renderErr.message}\n`);
-        return { error: `disk render failed: ${renderErr.message}` };
-    }
-    // ── DB write — store in assessments table ──────────────────────────────
-    const validatedAt = new Date().toISOString();
-    transaction(() => {
-        const adapter = _getAdapter();
-        adapter.prepare(`INSERT OR REPLACE INTO assessments (path, milestone_id, slice_id, task_id, status, scope, full_content, created_at)
-       VALUES (:path, :mid, NULL, NULL, :verdict, 'milestone-validation', :content, :created_at)`).run({
-            ":path": validationPath,
-            ":mid": params.milestoneId,
-            ":verdict": params.verdict,
-            ":content": validationMd,
-            ":created_at": validatedAt,
-        });
-    });
-    invalidateStateCache();
-    clearPathCache();
-    clearParseCache();
-    return {
-        milestoneId: params.milestoneId,
-        verdict: params.verdict,
-        validationPath,
-    };
-}

package/packages/pi-coding-agent/dist/core/local-model-check.d.ts

@@ -1,15 +0,0 @@
-/**
- * local-model-check.ts — Utility to detect if a model baseUrl is local.
- *
- * Leaf module with zero transitive dependencies on TypeScript parameter properties.
- * Used by ModelRegistry and tests.
- */
-/**
- * Check if a model's baseUrl points to a local endpoint.
- * Returns true for localhost, 127.0.0.1, 0.0.0.0, ::1, or unix socket paths.
- * Returns false if baseUrl is empty (cloud provider) or points to a remote host.
- */
-export declare function isLocalModel(model: {
-    baseUrl: string;
-}): boolean;
-//# sourceMappingURL=local-model-check.d.ts.map

package/packages/pi-coding-agent/dist/core/local-model-check.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"local-model-check.d.ts","sourceRoot":"","sources":["../../src/core/local-model-check.ts"],"names":[],"mappings":"AAAA;;;;;GAKG;AAEH;;;;GAIG;AACH,wBAAgB,YAAY,CAAC,KAAK,EAAE;IAAE,OAAO,EAAE,MAAM,CAAA;CAAE,GAAG,OAAO,CAgChE"}