fix: reject relation-only resolve handoffs

Keep live CLI and MCP smoke guidance from treating relation-only group leads as direct actions.\nClarify contradictory availability signals and structurally inapplicable evidence.

Author: jlitola

docs/implementation/cli-commands.md

@@ -330,13 +330,14 @@ confidence) or related.
 Each target keeps its normalized description, capped at 240 characters, plus
 available popularity, repository, license, docs, and code evidence. Positive
 docs/code counts render when content is available; otherwise availability is
-stated without inventing a count. Available content with a zero count renders
-that zero; unavailable content with zero renders `no docs` / `no code`. Missing
-licenses/counts produce no placeholder, and structurally inapplicable negatives
-are omitted (`no docs` is not shown for repositories and `no code` is not shown
-for sites). If availability is false despite a positive recorded count, output
-keeps both facts as `docs unavailable (<n> pages recorded)` or
-`code unavailable (<n> files recorded)`. A package's
+stated without inventing a count unless the backend supplies a contradictory
+positive recorded count. Available content with a zero count renders that zero;
+unavailable content with zero renders `no docs` / `no code`. Missing
+licenses/counts produce no placeholder. Structurally inapplicable evidence
+dimensions are omitted entirely: repositories show no docs field and sites show
+no code field. If availability is false despite a positive recorded count,
+output keeps both facts as `docs unavailable (<n> pages recorded)` or `code
+unavailable (<n> files recorded)`. A package's
 compact linked-repository fallback appears only when its group does not already
 contain the canonical repository target. `targetsTruncated` produces one note
 that additional related targets were omitted and direct matches are complete.

scripts/cli-smoke.ts

@@ -1012,7 +1012,7 @@ export function assertExperimentalCliResolveText(resolveText: string): void {
   if (directTarget) {
     assert(
       isResolveDirectTargetUnwarned(resolveText, directTarget),
-      "experimental direct resolve action should target a listed candidate without a warning",
+      "experimental direct resolve action should target a listed direct candidate without a warning",
     );
     return;
   }

scripts/mcp-smoke.ts

@@ -308,7 +308,7 @@ export function assertExperimentalMcpResolveText(
   if (directTarget) {
     assert(
       isResolveDirectTargetUnwarned(resolveTextBody, directTarget),
-      "experimental direct resolve action should target a listed candidate without a warning",
+      "experimental direct resolve action should target a listed direct candidate without a warning",
     );
     return;
   }

scripts/resolve-smoke-guidance.ts

@@ -15,9 +15,8 @@ export function isResolveDirectTargetUnwarned(
   const candidateLine = lines[candidateIndex] ?? "";
   const nested = candidateLine.startsWith("       ");
   if (
-    nested &&
-    (candidateLine.includes(" · related ") ||
-      candidateLine.includes("[related;"))
+    candidateLine.includes(" · related ") ||
+    candidateLine.includes("[related;")
   ) {
     return false;
   }

scripts/smoke-scripts.test.ts

@@ -537,7 +537,33 @@ Next: pass the canonical target "github:owner/project" to the next MCP tool.
         "       github:owner/project [related; repository]",
       );
     expect(() => assertExperimentalMcpResolveText(relatedMcpAction)).toThrow(
-      "without a warning",
+      "listed direct candidate without a warning",
+    );
+
+    const relatedCliLead = relatedCliAction
+      .replace(
+        "  1. npm:project [exact] · package",
+        "  1. github:owner/project · related repository",
+      )
+      .replace(
+        "\n     Related:\n       github:owner/project · related repository",
+        "",
+      );
+    expect(() => assertExperimentalCliResolveText(relatedCliLead)).toThrow(
+      "listed direct candidate without a warning",
+    );
+
+    const relatedMcpLead = relatedMcpAction
+      .replace(
+        "  1. npm:express [exact; package]",
+        "  1. github:owner/project [related; repository]\n  2. npm:express [exact; package]",
+      )
+      .replace(
+        "\n     Related:\n       github:owner/project [related; repository]",
+        "",
+      );
+    expect(() => assertExperimentalMcpResolveText(relatedMcpLead)).toThrow(
+      "listed direct candidate without a warning",
     );
   });