gsd-pi 2.50.0-dev.d210a87 → 2.50.0

package/src/resources/extensions/gsd/undo.ts

@@ -7,7 +7,6 @@ import type { ExtensionCommandContext, ExtensionAPI } from "@gsd/pi-coding-agent
 import { existsSync, readFileSync, writeFileSync, unlinkSync, readdirSync } from "node:fs";
 import { join } from "node:path";
 import { nativeRevertCommit, nativeRevertAbort } from "./native-git-bridge.js";
-import { parseUnitId } from "./unit-id.js";
 import { deriveState } from "./state.js";
 import { invalidateAllCaches } from "./cache.js";
 import { gsdRoot, resolveTasksDir, resolveSlicePath, resolveTaskFile, buildTaskFileName, buildSliceFileName } from "./paths.js";
@@ -66,11 +65,11 @@ export async function handleUndo(args: string, ctx: ExtensionCommandContext, _pi
   }
 
   // 1. Delete summary artifact
-  const { milestone, slice, task } = parseUnitId(unitId);
+  const parts = unitId.split("/");
   let summaryRemoved = false;
-  if (task !== undefined && slice !== undefined) {
+  if (parts.length === 3) {
     // Task-level: M001/S01/T01
-    const [mid, sid, tid] = [milestone, slice, task];
+    const [mid, sid, tid] = parts;
     const tasksDir = resolveTasksDir(basePath, mid, sid);
     if (tasksDir) {
       const summaryFile = join(tasksDir, buildTaskFileName(tid, "SUMMARY"));
@@ -79,9 +78,9 @@ export async function handleUndo(args: string, ctx: ExtensionCommandContext, _pi
         summaryRemoved = true;
       }
     }
-  } else if (slice !== undefined) {
+  } else if (parts.length === 2) {
     // Slice-level: M001/S01
-    const [mid, sid] = [milestone, slice];
+    const [mid, sid] = parts;
     const slicePath = resolveSlicePath(basePath, mid, sid);
     if (slicePath) {
       for (const suffix of ["SUMMARY", "COMPLETE"]) {
@@ -96,8 +95,8 @@ export async function handleUndo(args: string, ctx: ExtensionCommandContext, _pi
 
   // 2. Uncheck task in PLAN if execute-task
   let planUpdated = false;
-  if (unitType === "execute-task" && task !== undefined && slice !== undefined) {
-    const [mid, sid, tid] = [milestone, slice, task];
+  if (unitType === "execute-task" && parts.length === 3) {
+    const [mid, sid, tid] = parts;
     planUpdated = uncheckTaskInPlan(basePath, mid, sid, tid);
   }
 

package/src/resources/extensions/gsd/unit-runtime.ts

@@ -8,7 +8,6 @@ import {
   resolveTaskFile,
 } from "./paths.js";
 import { loadFile, parseTaskPlanMustHaves, countMustHavesMentionedInSummary } from "./files.js";
-import { parseUnitId } from "./unit-id.js";
 
 export type UnitRuntimePhase =
   | "dispatched"
@@ -129,7 +128,7 @@ export async function inspectExecuteTaskDurability(
   basePath: string,
   unitId: string,
 ): Promise<ExecuteTaskRecoveryStatus | null> {
-  const { milestone: mid, slice: sid, task: tid } = parseUnitId(unitId);
+  const [mid, sid, tid] = unitId.split("/");
   if (!mid || !sid || !tid) return null;
 
   const planAbs = resolveSliceFile(basePath, mid, sid, "PLAN");

package/src/resources/extensions/gsd/verification-gate.ts

@@ -8,7 +8,6 @@ import { existsSync, readFileSync } from "node:fs";
 import { join, basename } from "node:path";
 import type { AuditWarning, RuntimeError, VerificationCheck, VerificationResult } from "./types.js";
 import { DEFAULT_COMMAND_TIMEOUT_MS } from "./constants.js";
-import { rewriteCommandWithRtk } from "../shared/rtk.js";
 
 /** Maximum bytes of stdout/stderr to retain per command (10 KB). */
 const MAX_OUTPUT_BYTES = 10 * 1024;
@@ -258,11 +257,10 @@ export function runVerificationGate(options: RunVerificationGateOptions): Verifi
 
   for (const command of commands) {
     const start = Date.now();
-    const rewrittenCommand = rewriteCommandWithRtk(command);
     // Pass the command string as an argument to the shell explicitly
     // to avoid Node.js DEP0190 (spawnSync with shell: true and no args).
     const shellBin = process.platform === "win32" ? "cmd" : "sh";
-    const shellArgs = process.platform === "win32" ? ["/c", rewrittenCommand] : ["-c", rewrittenCommand];
+    const shellArgs = process.platform === "win32" ? ["/c", command] : ["-c", command];
     const result: SpawnSyncReturns<string> = spawnSync(shellBin, shellArgs, {
       cwd: options.cwd,
       stdio: "pipe",

package/dist/resources/extensions/gsd/auto-observability.js

@@ -1,54 +0,0 @@
-/**
- * Pre-dispatch observability checks for auto-mode units.
- * Validates plan/summary file quality and builds repair instructions
- * for the agent to fix gaps before proceeding with the unit.
- */
-import { validatePlanBoundary, validateExecuteBoundary, validateCompleteBoundary, formatValidationIssues, } from "./observability-validator.js";
-import { parseUnitId } from "./unit-id.js";
-export async function collectObservabilityWarnings(ctx, basePath, unitType, unitId) {
-    // Hook units have custom artifacts — skip standard observability checks
-    if (unitType.startsWith("hook/"))
-        return [];
-    const { milestone: mid, slice: sid, task: tid } = parseUnitId(unitId);
-    if (!mid || !sid)
-        return [];
-    let issues = [];
-    if (unitType === "plan-slice") {
-        issues = await validatePlanBoundary(basePath, mid, sid);
-    }
-    else if (unitType === "execute-task" && tid) {
-        issues = await validateExecuteBoundary(basePath, mid, sid, tid);
-    }
-    else if (unitType === "complete-slice") {
-        issues = await validateCompleteBoundary(basePath, mid, sid);
-    }
-    if (issues.length > 0) {
-        ctx.ui.notify(`Observability check (${unitType}) found ${issues.length} warning${issues.length === 1 ? "" : "s"}:\n${formatValidationIssues(issues)}`, "warning");
-    }
-    return issues;
-}
-export function buildObservabilityRepairBlock(issues) {
-    if (issues.length === 0)
-        return "";
-    const items = issues.map(issue => {
-        const fileName = issue.file.split("/").pop() || issue.file;
-        let line = `- **${fileName}**: ${issue.message}`;
-        if (issue.suggestion)
-            line += ` → ${issue.suggestion}`;
-        return line;
-    });
-    return [
-        "",
-        "---",
-        "",
-        "## Pre-flight: Observability gaps to fix FIRST",
-        "",
-        "The following issues were detected in plan/summary files for this unit.",
-        "**Read each flagged file, apply the fix described, then proceed with the unit.**",
-        "",
-        ...items,
-        "",
-        "---",
-        "",
-    ].join("\n");
-}

package/dist/resources/extensions/gsd/auto-utils.js

@@ -1,20 +0,0 @@
-// Shared utilities for the auto-loop modules (auto-post-unit, auto, etc.).
-import { debugLog } from "./debug-logger.js";
-/**
- * Run a non-fatal operation, logging any error via `debugLog` and continuing.
- *
- * Replaces the repeated try-catch-debugLog-continue boilerplate that wraps
- * operations whose failure should not abort the post-unit pipeline.
- *
- * @param context - The debugLog event name (e.g. "postUnit")
- * @param phase   - The phase label attached to the debug entry
- * @param fn      - The operation to execute (may be sync or async)
- */
-export async function runSafely(context, phase, fn) {
-    try {
-        await fn();
-    }
-    catch (e) {
-        debugLog(context, { phase, error: String(e) });
-    }
-}

package/dist/resources/extensions/gsd/observability-validator.js

@@ -1,422 +0,0 @@
-import { loadFile } from "./files.js";
-import { resolveSliceFile, resolveTaskFile, resolveTasksDir, resolveTaskFiles } from "./paths.js";
-function getSection(content, heading, level = 2) {
-    const prefix = "#".repeat(level) + " ";
-    const escaped = heading.replace(/[.*+?^${}()|[\]\\]/g, "\\$&");
-    const regex = new RegExp(`^${prefix}${escaped}\\s*$`, "m");
-    const match = regex.exec(content);
-    if (!match)
-        return null;
-    const start = match.index + match[0].length;
-    const rest = content.slice(start);
-    const nextHeading = rest.match(new RegExp(`^#{1,${level}} `, "m"));
-    const end = nextHeading ? nextHeading.index : rest.length;
-    return rest.slice(0, end).trim();
-}
-function getFrontmatter(content) {
-    const trimmed = content.trimStart();
-    if (!trimmed.startsWith("---"))
-        return null;
-    const afterFirst = trimmed.indexOf("\n");
-    if (afterFirst === -1)
-        return null;
-    const rest = trimmed.slice(afterFirst + 1);
-    const endIdx = rest.indexOf("\n---");
-    if (endIdx === -1)
-        return null;
-    return rest.slice(0, endIdx);
-}
-function hasFrontmatterKey(content, key) {
-    const fm = getFrontmatter(content);
-    if (!fm)
-        return false;
-    return new RegExp(`^${key}:`, "m").test(fm);
-}
-function normalizeMeaningfulLines(text) {
-    return text
-        .split("\n")
-        .map(line => line.trim())
-        .filter(line => line.length > 0)
-        .filter(line => !line.startsWith("<!--"))
-        .filter(line => !line.endsWith("-->"))
-        .filter(line => !/^[-*]\s*\{\{.+\}\}$/.test(line))
-        .filter(line => !/^\{\{.+\}\}$/.test(line));
-}
-function sectionLooksPlaceholderOnly(text) {
-    if (!text)
-        return true;
-    const lines = normalizeMeaningfulLines(text)
-        .map(line => line.replace(/^[-*]\s+/, "").trim())
-        .filter(line => line.length > 0);
-    if (lines.length === 0)
-        return true;
-    return lines.every(line => {
-        const lower = line.toLowerCase();
-        return lower === "none" ||
-            lower.endsWith(": none") ||
-            lower.includes("{{") ||
-            lower.includes("}}") ||
-            lower.startsWith("required for non-trivial") ||
-            lower.startsWith("describe how a future agent") ||
-            lower.startsWith("prefer:") ||
-            lower.startsWith("keep this section concise");
-    });
-}
-function textSuggestsObservabilityRelevant(content) {
-    const lower = content.toLowerCase();
-    const needles = [
-        " api", "route", "server", "worker", "queue", "job", "sync", "import",
-        "webhook", "auth", "db", "database", "migration", "cache", "background",
-        "polling", "realtime", "socket", "stateful", "integration", "ui", "form",
-        "submit", "status", "service", "pipeline", "health endpoint", "error path"
-    ];
-    return needles.some(needle => lower.includes(needle));
-}
-function verificationMentionsDiagnostics(section) {
-    if (!section)
-        return false;
-    const lower = section.toLowerCase();
-    const needles = [
-        "error", "failure", "diagnostic", "status", "health", "inspect", "log",
-        "network", "console", "retry", "last error", "correlation", "readiness"
-    ];
-    return needles.some(needle => lower.includes(needle));
-}
-export function validateSlicePlanContent(file, content) {
-    const issues = [];
-    // ── Plan quality rules (always run, not gated by runtime relevance) ──
-    const tasksSection = getSection(content, "Tasks", 2);
-    if (tasksSection) {
-        const lines = tasksSection.split("\n");
-        const taskLinePattern = /^- \[[ x]\] \*\*T\d+:/;
-        const taskLineIndices = [];
-        for (let i = 0; i < lines.length; i++) {
-            if (taskLinePattern.test(lines[i]))
-                taskLineIndices.push(i);
-        }
-        for (let t = 0; t < taskLineIndices.length; t++) {
-            const start = taskLineIndices[t];
-            const end = t + 1 < taskLineIndices.length ? taskLineIndices[t + 1] : lines.length;
-            // Check lines between this task header and the next (or section end)
-            const bodyLines = lines.slice(start + 1, end);
-            const meaningful = bodyLines.filter(l => l.trim().length > 0);
-            if (meaningful.length === 0) {
-                issues.push({
-                    severity: "warning",
-                    scope: "slice-plan",
-                    file,
-                    ruleId: "empty_task_entry",
-                    message: "Inline task entry has no description content beneath the checkbox line.",
-                    suggestion: "Add at least a Why/Files/Do/Verify summary so the task is self-describing.",
-                });
-            }
-        }
-    }
-    // ── Observability rules (gated by runtime relevance) ──
-    const relevant = textSuggestsObservabilityRelevant(content);
-    if (!relevant)
-        return issues;
-    const obs = getSection(content, "Observability / Diagnostics", 2);
-    const verification = getSection(content, "Verification", 2);
-    if (!obs) {
-        issues.push({
-            severity: "warning",
-            scope: "slice-plan",
-            file,
-            ruleId: "missing_observability_section",
-            message: "Slice plan appears non-trivial but is missing `## Observability / Diagnostics`.",
-            suggestion: "Add runtime signals, inspection surfaces, failure visibility, and redaction constraints.",
-        });
-    }
-    else if (sectionLooksPlaceholderOnly(obs)) {
-        issues.push({
-            severity: "warning",
-            scope: "slice-plan",
-            file,
-            ruleId: "observability_section_placeholder_only",
-            message: "Slice plan has `## Observability / Diagnostics` but it still looks like placeholder text.",
-            suggestion: "Replace placeholders with concrete signals and inspection surfaces a future agent should trust.",
-        });
-    }
-    if (!verificationMentionsDiagnostics(verification)) {
-        issues.push({
-            severity: "warning",
-            scope: "slice-plan",
-            file,
-            ruleId: "verification_missing_diagnostic_check",
-            message: "Slice verification does not appear to include any diagnostic or failure-path check.",
-            suggestion: "Add at least one verification step for inspectable failure state, structured error output, status surface, or equivalent.",
-        });
-    }
-    return issues;
-}
-export function validateTaskPlanContent(file, content) {
-    const issues = [];
-    // ── Plan quality rules (always run, not gated by runtime relevance) ──
-    // Rule: empty or missing Steps section
-    const stepsSection = getSection(content, "Steps", 2);
-    if (stepsSection === null || sectionLooksPlaceholderOnly(stepsSection)) {
-        issues.push({
-            severity: "warning",
-            scope: "task-plan",
-            file,
-            ruleId: "empty_steps_section",
-            message: "Task plan has an empty or missing `## Steps` section.",
-            suggestion: "Add concrete numbered implementation steps so execution has a clear sequence.",
-        });
-    }
-    // Rule: placeholder-only Verification section
-    const verificationSection = getSection(content, "Verification", 2);
-    if (verificationSection !== null && sectionLooksPlaceholderOnly(verificationSection)) {
-        issues.push({
-            severity: "warning",
-            scope: "task-plan",
-            file,
-            ruleId: "placeholder_verification",
-            message: "Task plan has `## Verification` but it still looks like placeholder text.",
-            suggestion: "Replace placeholders with concrete verification commands, test runs, or observable checks.",
-        });
-    }
-    // Rule: scope estimate thresholds
-    const fm = getFrontmatter(content);
-    if (fm) {
-        const stepsMatch = fm.match(/^estimated_steps:\s*(\d+)/m);
-        const filesMatch = fm.match(/^estimated_files:\s*(\d+)/m);
-        if (stepsMatch) {
-            const estimatedSteps = parseInt(stepsMatch[1], 10);
-            if (estimatedSteps >= 10) {
-                issues.push({
-                    severity: "warning",
-                    scope: "task-plan",
-                    file,
-                    ruleId: "scope_estimate_steps_high",
-                    message: `Task plan estimates ${estimatedSteps} steps (threshold: 10). Consider splitting into smaller tasks.`,
-                    suggestion: "Break the task into sub-tasks or reduce scope so each task stays focused and completable in one pass.",
-                });
-            }
-        }
-        if (filesMatch) {
-            const estimatedFiles = parseInt(filesMatch[1], 10);
-            if (estimatedFiles >= 12) {
-                issues.push({
-                    severity: "warning",
-                    scope: "task-plan",
-                    file,
-                    ruleId: "scope_estimate_files_high",
-                    message: `Task plan estimates ${estimatedFiles} files (threshold: 12). Consider splitting into smaller tasks.`,
-                    suggestion: "Break the task into sub-tasks or reduce scope to keep the change footprint manageable.",
-                });
-            }
-        }
-    }
-    // Rule: Inputs and Expected Output should contain backtick-wrapped file paths
-    const inputsSection = getSection(content, "Inputs", 2);
-    const outputSection = getSection(content, "Expected Output", 2);
-    const backtickPathPattern = /`[^`]*[./][^`]*`/;
-    if (outputSection === null || !backtickPathPattern.test(outputSection)) {
-        issues.push({
-            severity: "warning",
-            scope: "task-plan",
-            file,
-            ruleId: "missing_output_file_paths",
-            message: "Task plan `## Expected Output` is missing or has no backtick-wrapped file paths.",
-            suggestion: "List concrete output file paths in backticks (e.g. `src/types.ts`). These are machine-parsed to derive task dependencies.",
-        });
-    }
-    if (inputsSection !== null && inputsSection.trim().length > 0 && !backtickPathPattern.test(inputsSection)) {
-        issues.push({
-            severity: "info",
-            scope: "task-plan",
-            file,
-            ruleId: "missing_input_file_paths",
-            message: "Task plan `## Inputs` has content but no backtick-wrapped file paths.",
-            suggestion: "List input file paths in backticks (e.g. `src/config.json`). These are machine-parsed to derive task dependencies.",
-        });
-    }
-    // ── Observability rules (gated by runtime relevance) ──
-    const relevant = textSuggestsObservabilityRelevant(content);
-    if (!relevant)
-        return issues;
-    const obs = getSection(content, "Observability Impact", 2);
-    if (!obs) {
-        issues.push({
-            severity: "warning",
-            scope: "task-plan",
-            file,
-            ruleId: "missing_observability_impact",
-            message: "Task plan appears runtime-relevant but is missing `## Observability Impact`.",
-            suggestion: "Explain what signals change, how a future agent inspects this task, and what failure state becomes visible.",
-        });
-    }
-    else if (sectionLooksPlaceholderOnly(obs)) {
-        issues.push({
-            severity: "warning",
-            scope: "task-plan",
-            file,
-            ruleId: "observability_impact_placeholder_only",
-            message: "Task plan has `## Observability Impact` but it still looks empty or placeholder-only.",
-            suggestion: "Fill in concrete inspection surfaces or explicitly justify why observability is not applicable.",
-        });
-    }
-    return issues;
-}
-export function validateTaskSummaryContent(file, content) {
-    const issues = [];
-    if (!hasFrontmatterKey(content, "observability_surfaces")) {
-        issues.push({
-            severity: "warning",
-            scope: "task-summary",
-            file,
-            ruleId: "missing_observability_frontmatter",
-            message: "Task summary is missing `observability_surfaces` in frontmatter.",
-            suggestion: "List the durable status/log/error surfaces a future agent should use.",
-        });
-    }
-    const diagnostics = getSection(content, "Diagnostics", 2);
-    if (!diagnostics) {
-        issues.push({
-            severity: "warning",
-            scope: "task-summary",
-            file,
-            ruleId: "missing_diagnostics_section",
-            message: "Task summary is missing `## Diagnostics`.",
-            suggestion: "Document how to inspect what this task built later.",
-        });
-    }
-    else if (sectionLooksPlaceholderOnly(diagnostics)) {
-        issues.push({
-            severity: "warning",
-            scope: "task-summary",
-            file,
-            ruleId: "diagnostics_placeholder_only",
-            message: "Task summary diagnostics section still looks like placeholder text.",
-            suggestion: "Replace placeholders with concrete commands, endpoints, logs, error shapes, or failure artifacts.",
-        });
-    }
-    const evidence = getSection(content, "Verification Evidence", 2);
-    if (!evidence) {
-        issues.push({
-            severity: "warning",
-            scope: "task-summary",
-            file,
-            ruleId: "evidence_block_missing",
-            message: "Task summary is missing `## Verification Evidence`.",
-            suggestion: "Add a verification evidence table showing gate check results (command, exit code, verdict, duration).",
-        });
-    }
-    else if (sectionLooksPlaceholderOnly(evidence)) {
-        issues.push({
-            severity: "warning",
-            scope: "task-summary",
-            file,
-            ruleId: "evidence_block_placeholder",
-            message: "Task summary verification evidence section still looks like placeholder text.",
-            suggestion: "Replace placeholders with actual gate results or note that no verification commands were discovered.",
-        });
-    }
-    return issues;
-}
-export function validateSliceSummaryContent(file, content) {
-    const issues = [];
-    if (!hasFrontmatterKey(content, "observability_surfaces")) {
-        issues.push({
-            severity: "warning",
-            scope: "slice-summary",
-            file,
-            ruleId: "missing_observability_frontmatter",
-            message: "Slice summary is missing `observability_surfaces` in frontmatter.",
-            suggestion: "List the authoritative diagnostics and durable inspection surfaces for this slice.",
-        });
-    }
-    const diagnostics = getSection(content, "Authoritative diagnostics", 3);
-    if (!diagnostics) {
-        issues.push({
-            severity: "warning",
-            scope: "slice-summary",
-            file,
-            ruleId: "missing_authoritative_diagnostics",
-            message: "Slice summary is missing `### Authoritative diagnostics` in Forward Intelligence.",
-            suggestion: "Tell future agents where to look first and why that signal is trustworthy.",
-        });
-    }
-    else if (sectionLooksPlaceholderOnly(diagnostics)) {
-        issues.push({
-            severity: "warning",
-            scope: "slice-summary",
-            file,
-            ruleId: "authoritative_diagnostics_placeholder_only",
-            message: "Slice summary includes authoritative diagnostics but it still looks like placeholder text.",
-            suggestion: "Replace placeholders with the real first-stop diagnostic surface for this slice.",
-        });
-    }
-    return issues;
-}
-export async function validatePlanBoundary(basePath, milestoneId, sliceId) {
-    const issues = [];
-    const slicePlan = resolveSliceFile(basePath, milestoneId, sliceId, "PLAN");
-    if (slicePlan) {
-        const content = await loadFile(slicePlan);
-        if (content)
-            issues.push(...validateSlicePlanContent(slicePlan, content));
-    }
-    const tasksDir = resolveTasksDir(basePath, milestoneId, sliceId);
-    const taskPlans = tasksDir ? resolveTaskFiles(tasksDir, "PLAN") : [];
-    for (const file of taskPlans) {
-        const taskId = file.split("-")[0];
-        const taskPlan = resolveTaskFile(basePath, milestoneId, sliceId, taskId, "PLAN");
-        if (!taskPlan)
-            continue;
-        const content = await loadFile(taskPlan);
-        if (content)
-            issues.push(...validateTaskPlanContent(taskPlan, content));
-    }
-    return issues;
-}
-export async function validateExecuteBoundary(basePath, milestoneId, sliceId, taskId) {
-    const issues = [];
-    const slicePlan = resolveSliceFile(basePath, milestoneId, sliceId, "PLAN");
-    if (slicePlan) {
-        const content = await loadFile(slicePlan);
-        if (content)
-            issues.push(...validateSlicePlanContent(slicePlan, content));
-    }
-    const taskPlan = resolveTaskFile(basePath, milestoneId, sliceId, taskId, "PLAN");
-    if (taskPlan) {
-        const content = await loadFile(taskPlan);
-        if (content)
-            issues.push(...validateTaskPlanContent(taskPlan, content));
-    }
-    return issues;
-}
-export async function validateCompleteBoundary(basePath, milestoneId, sliceId) {
-    const issues = [];
-    const tasksDir = resolveTasksDir(basePath, milestoneId, sliceId);
-    const taskSummaries = tasksDir ? resolveTaskFiles(tasksDir, "SUMMARY") : [];
-    for (const file of taskSummaries) {
-        const taskId = file.split("-")[0];
-        const taskSummary = resolveTaskFile(basePath, milestoneId, sliceId, taskId, "SUMMARY");
-        if (!taskSummary)
-            continue;
-        const content = await loadFile(taskSummary);
-        if (content)
-            issues.push(...validateTaskSummaryContent(taskSummary, content));
-    }
-    const sliceSummary = resolveSliceFile(basePath, milestoneId, sliceId, "SUMMARY");
-    if (sliceSummary) {
-        const content = await loadFile(sliceSummary);
-        if (content)
-            issues.push(...validateSliceSummaryContent(sliceSummary, content));
-    }
-    return issues;
-}
-export function formatValidationIssues(issues, limit = 4) {
-    if (issues.length === 0)
-        return "";
-    const lines = issues.slice(0, limit).map(issue => {
-        const fileName = issue.file.split("/").pop() || issue.file;
-        return `- ${fileName}: ${issue.message}`;
-    });
-    if (issues.length > limit)
-        lines.push(`- ...and ${issues.length - limit} more`);
-    return lines.join("\n");
-}