@unbrained/pm-cli 2026.5.18 → 2026.5.24

package/dist/cli/commands/validate.d.ts

@@ -1,6 +1,6 @@
 import type { GlobalOptions } from "../../core/shared/command-types.js";
 type ValidateCheckName = "metadata" | "resolution" | "lifecycle" | "files" | "command_references" | "history_drift";
-type ValidateStatus = "ok" | "warn";
+type ValidateStatus = "ok" | "warn" | "error";
 export interface ValidateCommandOptions {
     checkMetadata?: boolean;
     checkResolution?: boolean;
@@ -10,6 +10,7 @@ export interface ValidateCommandOptions {
     checkFiles?: boolean;
     includePmInternals?: boolean;
     verboseFileLists?: boolean;
+    verboseDiagnostics?: boolean;
     checkHistoryDrift?: boolean;
     checkCommandReferences?: boolean;
     scanMode?: string;
@@ -22,6 +23,7 @@ export interface ValidateCheck {
 }
 export interface ValidateResult {
     ok: boolean;
+    has_warnings: boolean;
     checks: ValidateCheck[];
     warnings: string[];
     generated_at: string;

package/dist/cli/commands/validate.js

@@ -1,5 +1,5 @@
 
-!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="d8cd1c7e-f286-5f29-9404-1e7242621e73")}catch(e){}}();
+!function(){try{var e="undefined"!=typeof window?window:"undefined"!=typeof global?global:"undefined"!=typeof globalThis?globalThis:"undefined"!=typeof self?self:{},n=(new e.Error).stack;n&&(e._sentryDebugIds=e._sentryDebugIds||{},e._sentryDebugIds[n]="9884a203-9cc7-581e-a8e1-fd43052be74d")}catch(e){}}();
 import fs from "node:fs/promises";
 import { realpathSync } from "node:fs";
 import { execFile } from "node:child_process";
@@ -7,7 +7,7 @@ import path from "node:path";
 import { promisify } from "node:util";
 import { getActiveExtensionRegistrations } from "../../core/extensions/index.js";
 import { pathExists } from "../../core/fs/fs-utils.js";
-import { hashDocument } from "../../core/history/history.js";
+import { scanHistoryDrift } from "../../core/history/drift-scan.js";
 import { normalizeStatusInput } from "../../core/item/status.js";
 import { resolveItemTypeRegistry } from "../../core/item/type-registry.js";
 import { resolveRuntimeStatusRegistry } from "../../core/schema/runtime-schema.js";
@@ -16,9 +16,8 @@ import { PmCliError } from "../../core/shared/errors.js";
 import { toNonEmptyStringOrUndefined } from "../../core/shared/primitives.js";
 import { nowIso } from "../../core/shared/time.js";
 import { listAllFrontMatterWithBody } from "../../core/store/item-store.js";
-import { getHistoryPath, getSettingsPath, resolvePmRoot } from "../../core/store/paths.js";
+import { getSettingsPath, resolvePmRoot } from "../../core/store/paths.js";
 import { readSettings } from "../../core/store/settings.js";
-import { verifyHistoryChain } from "./history.js";
 import { extractReferencedPmItemIdsFromCommand } from "./test.js";
 const FILE_SCAN_DIRECTORIES = ["src", "tests", "docs"];
 const FILE_SCAN_ROOT_FILES = [
@@ -110,6 +109,7 @@ const METADATA_TRUNCATED_KEY_BY_FIELD = {
 };
 const GIT_LS_FILES_MAX_BUFFER = 32 * 1024 * 1024;
 const FILE_LIST_SUMMARY_LIMIT = 40;
+const DIAGNOSTIC_LIST_SUMMARY_LIMIT = 40;
 const execFileAsync = promisify(execFile);
 function normalizeRelativePath(value) {
     return value.replaceAll("\\", "/").replace(/^\.\/+/, "").replace(/^\/+/, "");
@@ -446,7 +446,7 @@ async function collectProjectFileCandidates(workspaceRoot, scanMode) {
         candidateScanned: defaultCandidates.length,
     };
 }
-function summarizeList(values, limit = 200) {
+function summarizeList(values, limit = DIAGNOSTIC_LIST_SUMMARY_LIMIT) {
     /* c8 ignore start -- truncation behavior only surfaces with very large synthetic datasets. */
     if (values.length <= limit) {
         return { values, truncated: false };
@@ -518,7 +518,7 @@ function resolveRequestedChecks(options) {
     }
     return requested;
 }
-function buildMetadataCheck(items, metadataPolicy, statusRegistry) {
+function buildMetadataCheck(items, metadataPolicy, statusRegistry, verboseDiagnostics) {
     const missingByField = Object.fromEntries(SUPPORTED_METADATA_REQUIRED_FIELDS.map((field) => [field, []]));
     for (const item of items) {
         for (const field of SUPPORTED_METADATA_REQUIRED_FIELDS) {
@@ -561,7 +561,7 @@ function buildMetadataCheck(items, metadataPolicy, statusRegistry) {
         if (missingByField[field].length === 0) {
             continue;
         }
-        const summarized = summarizeList(missingByField[field]);
+        const summarized = summarizeList(missingByField[field], verboseDiagnostics ? missingByField[field].length : DIAGNOSTIC_LIST_SUMMARY_LIMIT);
         details[METADATA_ITEM_IDS_KEY_BY_FIELD[field]] = summarized.values;
         details[METADATA_TRUNCATED_KEY_BY_FIELD[field]] = summarized.truncated;
     }
@@ -574,7 +574,7 @@ function buildMetadataCheck(items, metadataPolicy, statusRegistry) {
         warnings: warningTokens,
     };
 }
-function buildResolutionCheck(items, statusRegistry) {
+function buildResolutionCheck(items, statusRegistry, verboseDiagnostics) {
     const terminalDoneStatuses = new Set(statusRegistry.terminal_done_statuses);
     terminalDoneStatuses.add(statusRegistry.close_status);
     const closedItems = items.filter((item) => terminalDoneStatuses.has(normalizeStatusForRegistry(item.status, statusRegistry)));
@@ -590,9 +590,10 @@ function buildResolutionCheck(items, statusRegistry) {
         });
     }
     const warnings = missingResolutionRows.length > 0 ? [`validate_resolution_missing_fields:${missingResolutionRows.length}`] : [];
-    const summarizedRows = summarizeList(missingResolutionRows.map((row) => `${row.id}:${row.missing_fields.join(",")}`));
+    const diagnosticLimit = verboseDiagnostics ? Number.POSITIVE_INFINITY : DIAGNOSTIC_LIST_SUMMARY_LIMIT;
+    const summarizedRows = summarizeList(missingResolutionRows.map((row) => `${row.id}:${row.missing_fields.join(",")}`), diagnosticLimit);
     const remediationHints = missingResolutionRows.map((row) => buildResolutionRemediationCommand(row));
-    const summarizedHints = summarizeList(remediationHints);
+    const summarizedHints = summarizeList(remediationHints, diagnosticLimit);
     return {
         check: {
             name: "resolution",
@@ -822,7 +823,11 @@ function buildLifecycleCheck(items, includeStaleBlockers, dependencyCycleSeverit
     return {
         check: {
             name: "lifecycle",
-            status: warnings.length === 0 ? "ok" : "warn",
+            status: dependencyCycleDiagnostics.cycle_count > 0 && dependencyCycleSeverity === "error"
+                ? "error"
+                : warnings.length === 0
+                    ? "ok"
+                    : "warn",
             details: {
                 checked_active_items: activeItems.length,
                 active_closure_like_metadata_items: closureLikeRows.length,
@@ -961,62 +966,7 @@ async function buildFilesCheck(items, workspaceRoot, pmRoot, fileScanMode, inclu
     };
 }
 async function buildHistoryDriftCheck(pmRoot, items) {
-    const missingStreams = [];
-    const unreadableStreams = [];
-    const hashMismatches = [];
-    const chainMismatches = [];
-    for (const item of items) {
-        const historyPath = getHistoryPath(pmRoot, item.id);
-        let latestAfterHash = null;
-        try {
-            const raw = await fs.readFile(historyPath, "utf8");
-            if (raw.trim().length === 0) {
-                missingStreams.push(item.id);
-                continue;
-            }
-            const entries = [];
-            for (const line of raw.split(/\r?\n/)) {
-                const trimmed = line.trim();
-                if (trimmed.length === 0) {
-                    continue;
-                }
-                const parsed = JSON.parse(trimmed);
-                if (typeof parsed.after_hash !== "string" || parsed.after_hash.trim().length === 0) {
-                    throw new Error("missing after_hash");
-                }
-                entries.push(parsed);
-                latestAfterHash = parsed.after_hash;
-            }
-            const chainVerification = verifyHistoryChain(entries);
-            if (!chainVerification.ok) {
-                chainMismatches.push(item.id);
-            }
-        }
-        catch (error) {
-            if (typeof error === "object" && error !== null && "code" in error && error.code === "ENOENT") {
-                missingStreams.push(item.id);
-            }
-            else {
-                unreadableStreams.push(item.id);
-            }
-            continue;
-        }
-        /* c8 ignore start -- defensive guard for future history schema changes. */
-        if (!latestAfterHash) {
-            missingStreams.push(item.id);
-            continue;
-        }
-        /* c8 ignore stop */
-        const { body, ...frontMatter } = item;
-        const currentHash = hashDocument({
-            metadata: frontMatter,
-            body,
-        });
-        if (currentHash !== latestAfterHash) {
-            hashMismatches.push(item.id);
-        }
-    }
-    const driftedItems = [...new Set([...missingStreams, ...unreadableStreams, ...hashMismatches, ...chainMismatches])].sort((a, b) => a.localeCompare(b));
+    const { missingStreams, unreadableStreams, hashMismatches, chainMismatches, driftedItems } = await scanHistoryDrift(pmRoot, items);
     const warnings = [];
     if (missingStreams.length > 0) {
         warnings.push(`validate_history_drift_missing_streams:${missingStreams.length}`);
@@ -1132,12 +1082,12 @@ export async function runValidate(options, global) {
     const checks = [];
     const warnings = [...new Set(itemReadWarnings)];
     if (requestedChecks.has("metadata")) {
-        const metadataCheck = buildMetadataCheck(items, metadataPolicy, statusRegistry);
+        const metadataCheck = buildMetadataCheck(items, metadataPolicy, statusRegistry, Boolean(options.verboseDiagnostics));
         checks.push(metadataCheck.check);
         warnings.push(...metadataCheck.warnings);
     }
     if (requestedChecks.has("resolution")) {
-        const resolutionCheck = buildResolutionCheck(items, statusRegistry);
+        const resolutionCheck = buildResolutionCheck(items, statusRegistry, Boolean(options.verboseDiagnostics));
         checks.push(resolutionCheck.check);
         warnings.push(...resolutionCheck.warnings);
     }
@@ -1162,12 +1112,14 @@ export async function runValidate(options, global) {
         warnings.push(...historyDriftCheck.warnings);
     }
     const normalizedWarnings = [...new Set(warnings)].sort((left, right) => left.localeCompare(right));
+    const hasErrors = checks.some((check) => check.status === "error");
     return {
-        ok: normalizedWarnings.length === 0,
+        ok: !hasErrors,
+        has_warnings: normalizedWarnings.length > 0,
         checks,
         warnings: normalizedWarnings,
         generated_at: nowIso(),
     };
 }
 //# sourceMappingURL=validate.js.map
-//# debugId=d8cd1c7e-f286-5f29-9404-1e7242621e73
+//# debugId=9884a203-9cc7-581e-a8e1-fd43052be74d