@unbrained/pm-cli 2026.7.17 → 2026.7.18

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

@@ -454,8 +454,8 @@ export declare const _testOnly: {
     toContextFocusItem: typeof toContextFocusItem;
 };
 declare function buildActivity(activityLimit: number, global: GlobalOptions): Promise<CompactActivityEntry[]>;
-declare function buildProgress(allItems: ItemMetadata[], activeItems: ItemMetadata[], statusRegistry: RuntimeStatusRegistry, limit: number): ProgressEntry[];
-declare function buildBlockers(blockedItems: ItemMetadata[], itemMap: Map<string, ItemMetadata>, limit: number): BlockerEntry[];
+declare function buildProgress(allItems: ItemMetadata[], activeItems: ItemMetadata[], blockedIds: ReadonlySet<string>, statusRegistry: RuntimeStatusRegistry, limit: number): ProgressEntry[];
+declare function buildBlockers(blockedItems: ItemMetadata[], fullCorpus: ItemMetadata[], statusRegistry: RuntimeStatusRegistry, limit: number): BlockerEntry[];
 declare function buildHotFiles(activeItems: ItemMetadata[], limit: number): HotFile[];
 declare function buildWorkload(activeItems: ItemMetadata[], statusRegistry: RuntimeStatusRegistry, limit: number): WorkloadEntry[];
 declare function buildStaleness(allNonTerminal: ItemMetadata[], staleThresholdDays: number, now: string, limit: number): StaleEntry[];

package/dist/cli/commands/context.js

@@ -4,11 +4,12 @@
  * Implements the pm context 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]="36bded4c-6a70-5b72-bd69-49bdc0cf1332")}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]="2df694f8-45bb-5a5c-aea9-30da2ed1633d")}catch(e){}}();
 import { SETTINGS_DEFAULTS, EXIT_CODE } from "../../core/shared/constants.js";
 import { PmCliError } from "../../core/shared/errors.js";
 import { compareTimestampStrings, nowIso } from "../../core/shared/time.js";
 import { isTerminalStatus, normalizeStatusForRegistry, normalizeStatusInput, } from "../../core/item/status.js";
+import { collectDependencyBlockedIds, resolveItemBlockers, } from "../../sdk/actionability.js";
 import { resolveRuntimeStatusRegistry, } from "../../core/schema/runtime-schema.js";
 import { resolvePmRoot } from "../../core/store/paths.js";
 import { readSettings } from "../../core/store/settings.js";
@@ -655,7 +656,7 @@ async function buildActivity(activityLimit, global) {
     const result = await runActivity({ compact: true, limit: String(activityLimit) }, global);
     return result.compact_activity ?? [];
 }
-function buildProgress(allItems, activeItems, statusRegistry, limit) {
+function buildProgress(allItems, activeItems, blockedIds, statusRegistry, limit) {
     const childrenByParent = new Map();
     for (const item of allItems) {
         if (!item.parent)
@@ -677,12 +678,14 @@ function buildProgress(allItems, activeItems, statusRegistry, limit) {
         let blocked = 0;
         /* c8 ignore start -- chained status-classification branch accounting is noisy under v8 for mixed descendant sets */
         for (const desc of descendants) {
+            // Blocked is edge-aware (GH-578): dependency-blocked descendants count as
+            // blocked regardless of lifecycle status, matching pm next and list-blocked.
             if (isClosedStatus(desc.status, statusRegistry))
                 closed++;
+            else if (blockedIds.has(desc.id.trim().toLowerCase()))
+                blocked++;
             else if (isInProgressStatus(desc.status, statusRegistry))
                 inProgress++;
-            else if (isBlockedStatus(desc.status, statusRegistry))
-                blocked++;
             else if (isOpenStatus(desc.status, statusRegistry))
                 open++;
         }
@@ -703,15 +706,22 @@ function buildProgress(allItems, activeItems, statusRegistry, limit) {
         .sort((a, b) => a.completion_pct - b.completion_pct)
         .slice(0, limit);
 }
-function buildBlockers(blockedItems, itemMap, limit) {
+function buildBlockers(blockedItems, fullCorpus, statusRegistry, limit) {
+    const itemsById = new Map(fullCorpus.map((item) => [item.id.trim().toLowerCase(), item]));
     return blockedItems.slice(0, limit).map((item) => {
-        const blockerItem = item.blocked_by
-            ? itemMap.get(item.blocked_by)
+        // Prefer a currently open blocker edge. The legacy scalar may still name a
+        // resolved predecessor, so it is only a fallback when no open edge exists.
+        const blockers = resolveItemBlockers(item, itemsById, statusRegistry);
+        const blockedBy = blockers.find((blocker) => !blocker.resolved)?.id ??
+            blockers[0]?.id ??
+            null;
+        const blockerItem = blockedBy
+            ? itemsById.get(blockedBy.trim().toLowerCase())
             : undefined;
         return {
             id: item.id,
             title: item.title,
-            blocked_by: item.blocked_by ?? null,
+            blocked_by: blockedBy,
             blocked_by_title: blockerItem?.title ?? null,
             blocked_by_status: blockerItem?.status ?? null,
             blocked_reason: item.blocked_reason ?? null,
@@ -1128,17 +1138,18 @@ function buildUnpaginatedContextListOptions(baseListOptions, extra) {
 }
 async function loadContextCorpus(options, global, runtime) {
     const needsAllItems = contextNeedsAllItems(runtime.sectionsIncluded);
-    const listOptions = buildUnpaginatedContextListOptions(runtime.baseListOptions, { excludeTerminal: true });
-    const listed = await runList(undefined, listOptions, global);
-    let listedItemMetadata = listed.items;
-    let allItems = listedItemMetadata;
-    if (needsAllItems || runtime.parentScope !== undefined) {
-        const allListed = await runList(undefined, buildUnpaginatedContextListOptions(runtime.baseListOptions, {
-            excludeTerminal: false,
-        }), global);
-        allItems = allListed.items;
-    }
-    const fullCorpus = allItems;
+    // One full-corpus scan serves every consumer: runList always reads all item
+    // files before filtering, so deriving the non-terminal subset in memory is
+    // free and keeps terminal blocker targets available for the shared
+    // edge-aware blocked classification (GH-578).
+    const listed = await runList(undefined, buildUnpaginatedContextListOptions(runtime.baseListOptions, {
+        excludeTerminal: false,
+    }), global);
+    const fullCorpus = listed.items;
+    let listedItemMetadata = fullCorpus.filter((item) => !isTerminalStatus(item.status, runtime.statusRegistry));
+    let allItems = needsAllItems || runtime.parentScope !== undefined
+        ? fullCorpus
+        : listedItemMetadata;
     const subtreeIds = resolveContextSubtreeIds(runtime.parentScope, fullCorpus);
     if (subtreeIds) {
         listedItemMetadata = listedItemMetadata.filter((item) => subtreeIds.has(item.id.trim().toLowerCase()));
@@ -1156,7 +1167,7 @@ function resolveContextSubtreeIds(parentScope, fullCorpus) {
     }
     return subtree.ids;
 }
-async function resolveContextFocusGroups(listedItemMetadata, allItems, statusRegistry, sectionsIncluded, limit, now, author, cursor, cursorFingerprint, useBoundedPage, pmRoot) {
+async function resolveContextFocusGroups(listedItemMetadata, allItems, blockedIds, statusRegistry, sectionsIncluded, limit, now, author, cursor, cursorFingerprint, useBoundedPage, pmRoot) {
     const structural = [...listedItemMetadata].sort((left, right) => compareCriticalItems(left, right, statusRegistry));
     const usage = await readContextUsageAffinity({
         pmRoot,
@@ -1184,7 +1195,7 @@ async function resolveContextFocusGroups(listedItemMetadata, allItems, statusReg
     const nextCursor = hasMore && focusPage.length > 0
         ? encodeQueryCursor(cursorFingerprint, focusPage[focusPage.length - 1].id, pageStart + focusPage.length - 1)
         : undefined;
-    const rankedBlockedItems = ranked.filter((item) => statusRegistry.blocked_statuses.has(normalizeStatusForRegistry(item.status, statusRegistry)));
+    const rankedBlockedItems = ranked.filter((item) => blockedIds.has(item.id.trim().toLowerCase()));
     const packingIds = new Set([
         ...focusPage.map((item) => item.id),
         ...rankedBlockedItems.slice(0, limit).map((item) => item.id),
@@ -1192,7 +1203,7 @@ async function resolveContextFocusGroups(listedItemMetadata, allItems, statusReg
     const packing = packRankedContextItems({ ...ranking, ranked: ranking.ranked.filter((entry) => packingIds.has(entry.id)) }, Math.max(256, limit * 160));
     const packedItems = packing.included.map((entry) => entry.item);
     const activeItems = packedItems.filter((item) => activeStatuses.has(normalizeStatusForRegistry(item.status, statusRegistry)));
-    const blockedItems = packedItems.filter((item) => statusRegistry.blocked_statuses.has(normalizeStatusForRegistry(item.status, statusRegistry)));
+    const blockedItems = packedItems.filter((item) => blockedIds.has(item.id.trim().toLowerCase()));
     const childrenByParent = buildChildrenByParent(allItems);
     const focusChildrenByParent = contextNeedsAllItems(sectionsIncluded)
         ? childrenByParent
@@ -1264,9 +1275,8 @@ function countContextStatus(items, status, statusRegistry) {
     return items.filter((item) => normalizeStatusForRegistry(item.status, statusRegistry) === targetStatus).length;
 }
 async function buildOptionalContextSections(params) {
-    const { runtime, allItems, fullCorpus, focusGroups, now, global } = params;
+    const { runtime, allItems, fullCorpus, blockedIds, focusGroups, now, global } = params;
     const childrenByParent = buildChildrenByParent(allItems);
-    const itemMap = new Map(fullCorpus.map((item) => [item.id, item]));
     const allNonTerminal = allItems.filter((item) => !isTerminalStatus(item.status, runtime.statusRegistry));
     const has = (section) => runtime.sectionsIncluded.includes(section);
     return {
@@ -1277,10 +1287,10 @@ async function buildOptionalContextSections(params) {
             ? await buildActivity(runtime.activityLimit, global)
             : undefined,
         progress: has("progress")
-            ? buildProgress(allItems, focusGroups.activeItems, runtime.statusRegistry, runtime.limit)
+            ? buildProgress(allItems, focusGroups.activeItems, blockedIds, runtime.statusRegistry, runtime.limit)
             : undefined,
         blockersSection: has("blockers")
-            ? buildBlockers(focusGroups.blockedItems, itemMap, runtime.limit)
+            ? buildBlockers(focusGroups.blockedItems, fullCorpus, runtime.statusRegistry, runtime.limit)
             : undefined,
         recentlyCreated: has("recently_created")
             ? buildRecentlyCreated(allNonTerminal, runtime.statusRegistry, childrenByParent, runtime.limit)
@@ -1394,7 +1404,10 @@ export async function runContext(options, global) {
         limit: resolveContextLimitAtScale(resolvedRuntime.limit, corpus.fullCorpus.length),
     };
     const author = process.env.PM_AUTHOR ?? runtime.settings.author_default;
-    const focusGroups = await resolveContextFocusGroups(corpus.listedItemMetadata, corpus.allItems, runtime.statusRegistry, runtime.sectionsIncluded, runtime.limit, nowIso(), author, options.after, buildContextCursorFingerprint(options), shouldPageContextFocus(options, corpus.fullCorpus.length), pmRoot);
+    // Single shared edge-aware blocked classification (GH-578): resolved against
+    // the full corpus so terminal blocker targets count as satisfied at every depth.
+    const blockedIds = collectDependencyBlockedIds(corpus.fullCorpus, runtime.statusRegistry);
+    const focusGroups = await resolveContextFocusGroups(corpus.listedItemMetadata, corpus.allItems, blockedIds, runtime.statusRegistry, runtime.sectionsIncluded, runtime.limit, nowIso(), author, options.after, buildContextCursorFingerprint(options), shouldPageContextFocus(options, corpus.fullCorpus.length), pmRoot);
     const agendaContext = await buildContextAgenda(options, global, runtime, corpus.subtreeIds);
     const warnings = mergeSortedWarnings(corpus.listed.warnings, agendaContext.agenda.warnings, focusGroups.ranking.warnings);
     const inProgressCount = countContextStatus(focusGroups.activeItems, normalizeStatusInput("in_progress", runtime.statusRegistry), runtime.statusRegistry);
@@ -1403,6 +1416,7 @@ export async function runContext(options, global) {
         runtime,
         allItems: corpus.allItems,
         fullCorpus: corpus.fullCorpus,
+        blockedIds,
         focusGroups,
         now: agendaContext.agenda.now,
         global,
@@ -1469,4 +1483,4 @@ export async function runContext(options, global) {
     return result;
 }
 //# sourceMappingURL=context.js.map
-//# debugId=36bded4c-6a70-5b72-bd69-49bdc0cf1332
+//# debugId=2df694f8-45bb-5a5c-aea9-30da2ed1633d