@unbrained/pm-cli 2026.7.13 → 2026.7.14

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

@@ -147,7 +147,7 @@ declare function buildCloseReasonBackfillRows(metadataPolicy: ValidateMetadataPo
 declare function buildEstimateBackfillRows(metadataPolicy: ValidateMetadataPolicy, missingByField: Record<ValidateMetadataRequiredField, string[]>, itemsById: Map<string, ItemWithBody>): EstimateBackfillRow[];
 declare function escapeRegExp(value: string): string;
 declare function buildLifecycleDependencyGraph(activeItems: ItemWithBody[], idPrefix?: string): Map<string, string[]>;
-declare function buildDependencyReferencesCheck(items: ItemWithBody[], verboseDiagnostics: boolean): {
+declare function buildDependencyReferencesCheck(items: ItemWithBody[], verboseDiagnostics: boolean, statusRegistry?: RuntimeStatusRegistry): {
     check: ValidateCheck;
     warnings: string[];
 };

package/dist/cli/commands/validate.js

@@ -4,7 +4,7 @@
  * Implements the pm validate command surface and its agent-facing runtime behavior.
  */
 
-!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]="288ee244-f1ab-5413-82fd-0c53e76f5aad")}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]="fef78b9a-56c5-57af-8bc3-4407b251edfa")}catch(e){}}();
 import fs from "node:fs/promises";
 import { realpathSync } from "node:fs";
 import { execFile } from "node:child_process";
@@ -31,7 +31,11 @@ import { buildMissingByTypeCounts, } from "../../core/validate/missing-by-type.j
 import { classifyStaleLinkedPaths, summarizeStaleLinkedPathClassifications, } from "../../core/validate/stale-file-classification.js";
 import { isRemoteLinkedArtifactReference } from "../../core/validate/linked-artifact-reference.js";
 import { buildMissingLinkedPathRows, summarizeMissingLinkedPathRows, } from "../../core/validate/missing-link-owners.js";
+import { collectDanglingDependencyReferences } from "../../sdk/dependencies.js";
+import { runDocs } from "../../sdk/docs.js";
+import { runFiles } from "../../sdk/files.js";
 import { extractReferencedPmItemIdsFromCommand } from "./test.js";
+import { runUpdate } from "./update.js";
 const FILE_SCAN_DIRECTORIES = ["src", "tests", "docs"];
 const FILE_SCAN_ROOT_FILES = [
     "README.md",
@@ -938,50 +942,46 @@ function buildLifecycleDependencyGraph(activeItems, idPrefix = "pm") {
     return graph;
 }
 /* c8 ignore stop */
-function buildDependencyReferencesCheck(items, verboseDiagnostics) {
-    const knownIds = new Set(items.map((item) => item.id.trim().toLowerCase()));
-    const rows = [];
-    for (const item of items) {
-        const parent = toMeaningfulString(item.parent);
-        if (parent && !knownIds.has(parent.toLowerCase()))
-            rows.push(`${item.id}:${parent}:parent`);
-        const scalarBlocker = toMeaningfulString(item.blocked_by);
-        if (scalarBlocker && !knownIds.has(scalarBlocker.toLowerCase())) {
-            rows.push(`${item.id}:${scalarBlocker}:blocked_by`);
-        }
-        for (const dependency of item.dependencies ?? []) {
-            const dependencyId = toMeaningfulString(dependency.id);
-            if (dependencyId && !knownIds.has(dependencyId.toLowerCase())) {
-                rows.push(`${item.id}:${dependencyId}:${dependency.kind}`);
-            }
-        }
-    }
-    const uniqueRows = [...new Set(rows)].sort((left, right) => left.localeCompare(right));
+function buildDependencyReferencesCheck(items, verboseDiagnostics, statusRegistry) {
+    const classified = collectDanglingDependencyReferences(items, statusRegistry
+        ? (status) => isTerminalStatus(status, statusRegistry)
+        : undefined);
+    const activeRows = classified.active.map((row) => `${row.holder_id}:${row.target_id}:${row.kind}`);
+    const legacyRows = classified.legacy_terminal.map((row) => `${row.holder_id}:${row.target_id}:${row.kind}:${row.holder_status}`);
     const diagnosticLimit = verboseDiagnostics
         ? Number.POSITIVE_INFINITY
         : DIAGNOSTIC_LIST_SUMMARY_LIMIT;
-    const summarizedRows = summarizeList(uniqueRows, diagnosticLimit);
-    const hints = summarizeList(uniqueRows.map((row) => {
-        const [itemId, , kind] = row.split(":");
-        return kind === "parent"
-            ? `pm update ${itemId} --unset parent`
-            : `pm update ${itemId} --replace-deps '<correct dependency edges>'`;
+    const summarizedRows = summarizeList(activeRows, diagnosticLimit);
+    const summarizedLegacyRows = summarizeList(legacyRows, diagnosticLimit);
+    const hints = summarizeList(classified.active.map((row) => {
+        if (row.source === "parent") {
+            return `pm update ${row.holder_id} --unset parent`;
+        }
+        return row.source === "blocked_by"
+            ? `pm update ${row.holder_id} --unset blocked_by`
+            : `pm update ${row.holder_id} --replace-deps '<correct dependency edges>'`;
     }), diagnosticLimit);
     return {
         check: {
             name: "dependency_references",
-            status: uniqueRows.length === 0 ? "ok" : "warn",
+            status: activeRows.length === 0 ? "ok" : "warn",
             details: {
                 checked_items: items.length,
-                dangling_reference_count: uniqueRows.length,
+                dangling_reference_count: activeRows.length + legacyRows.length,
+                active_dangling_reference_count: activeRows.length,
                 dangling_reference_rows: summarizedRows.values,
                 dangling_reference_rows_truncated: summarizedRows.truncated,
+                legacy_terminal_dangling_reference_count: legacyRows.length,
+                legacy_closed_dangling_reference_count: classified.legacy_terminal.filter((row) => row.holder_status === "closed").length,
+                legacy_terminal_dangling_reference_rows: summarizedLegacyRows.values,
+                legacy_terminal_dangling_reference_rows_truncated: summarizedLegacyRows.truncated,
+                no_active_blocker_sentinel_count: classified.no_active_blocker_sentinels.length,
                 remediation_hints: hints.values,
                 remediation_hints_truncated: hints.truncated,
             },
         },
-        warnings: uniqueRows.length > 0
-            ? [`validate_dangling_dependency_references:${uniqueRows.length}`]
+        warnings: activeRows.length > 0
+            ? [`validate_dangling_dependency_references:${activeRows.length}`]
             : [],
     };
 }
@@ -1854,7 +1854,6 @@ async function applyValidateFix(fix, global) {
         case "set_estimate":
         case "reparent":
         case "unset_parent": {
-            const { runUpdate } = await import("./update.js");
             const updateOptions = {
                 message: VALIDATE_AUTO_FIX_MESSAGE,
             };
@@ -1927,11 +1926,9 @@ async function applyValidateFixes(applicable, global) {
             .filter((value) => typeof value === "string" && value.length > 0);
         try {
             if (first.kind === "prune_file_link") {
-                const { runFiles } = await import("./files.js");
                 await runFiles(first.item_id, { remove, message: VALIDATE_AUTO_FIX_MESSAGE }, global);
             }
             else {
-                const { runDocs } = await import("./docs.js");
                 await runDocs(first.item_id, { remove, message: VALIDATE_AUTO_FIX_MESSAGE }, global);
             }
             applied.push(...batch);
@@ -2018,7 +2015,7 @@ async function executeRequestedValidateChecks(params) {
         const built = buildLifecycleCheck(params.items, Boolean(params.options.checkStaleBlockers), params.dependencyCycleSeverity, params.parentCycleSeverity, params.statusRegistry, params.lifecyclePatternPolicy, fullDiagnostics, params.settings.id_prefix);
         state.terminalParentFixRows = built.terminalParentFixRows;
         recordValidateCheck(state, built, fixHintsEnabled);
-        recordValidateCheck(state, buildDependencyReferencesCheck(params.items, fullDiagnostics), fixHintsEnabled);
+        recordValidateCheck(state, buildDependencyReferencesCheck(params.items, fullDiagnostics, params.statusRegistry), fixHintsEnabled);
     }
     if (params.requestedChecks.has("files")) {
         const built = await buildFilesCheck(params.items, params.workspaceRoot, params.pmRoot, params.fileScanMode, Boolean(params.options.includePmInternals), Boolean(params.options.verboseFileLists));
@@ -2157,4 +2154,4 @@ export async function runValidate(options, global) {
 }
 /* c8 ignore stop */
 //# sourceMappingURL=validate.js.map
-//# debugId=288ee244-f1ab-5413-82fd-0c53e76f5aad
+//# debugId=fef78b9a-56c5-57af-8bc3-4407b251edfa