tines 0.0.176 → 0.0.178

package/README.md

@@ -39,7 +39,7 @@ tines issues show <project>/<number>
 tines issues move <project>/<number> <action>        # a workflow transition
 tines issues transfer <project>/<number> --project <dest>   # move to another project (keeps ID, record and old refs)
 tines issues transfer <ref> --project <dest> --dry-run      # review only: no number allocated, nothing written
-tines issues transfer <ref> --project <dest> --inspect 0    # print reviewed guidance item [0] in full
+tines issues transfer <ref> --project <dest> --inspect 0    # print any reviewed guidance item, including retained, in full
 tines issues transfer <ref> --project <dest> --yes          # skip the prompt (still commits only the fetched review)
 tines issues comment <project>/<number> - <<'EOF'    # body from stdin; @file also works
 …

package/dist/index.js

@@ -5048,11 +5048,12 @@ var CHANGE_LABEL = {
   removed: "removed with source",
   retained: "retained",
   rescoped: "moves with the issue",
-  replaced: "overridden at destination"
+  replaced: "effective selection changes"
 };
 function inspectableChanges(preview) {
-  return preview.context.changes.filter((c) => c.change !== "retained");
+  return preview.context.changes;
 }
+var effectiveness = (present, effective) => !present ? "not present" : effective ? "effective" : "overridden candidate";
 function repoLine(change) {
   if (change.kind !== "repo") return null;
   const before = change.repo_before;
@@ -5074,16 +5075,85 @@ function routingLines(preview) {
     lines.push(`${label}: ${explainer.verdict}`);
     if (explainer.pin) {
       lines.push(
-        `           pinned runner ${explainer.pin.runner_name ?? explainer.pin.runner_id}${explainer.pin.tier ? ` (${explainer.pin.tier})` : ""} \u2014 retained by the move`
+        `           pin: runner ${explainer.pin.runner_name ?? explainer.pin.runner_id}, tier ${explainer.pin.tier ?? "default"}`
       );
     }
+    for (const check of explainer.checks) {
+      lines.push(
+        `           ${check.ok ? "pass" : "FAIL"} ${check.name}: ${check.detail}${check.action?.label ? ` (${check.action.label})` : ""}`
+      );
+    }
+    if (explainer.matched_rule)
+      lines.push(`           matched rule ${explainer.matched_rule.scope_label}`);
     for (const tie of explainer.ambiguous_rules) {
       lines.push(`           tied rule ${tie.scope_label}`);
     }
+    if (explainer.runner_rule)
+      lines.push(`           runner source rule ${explainer.runner_rule.scope_label}`);
+    if (explainer.tier_override) lines.push(`           tier override ${explainer.tier_override}`);
+    for (const target of explainer.targets) {
+      lines.push(
+        `           target ${target.runner_name} \u2014 ${target.tier}${target.model ? ` / ${target.model}` : ""}: ${target.verdict} \u2014 ${target.detail}`
+      );
+    }
+    if (explainer.active_run)
+      lines.push(
+        `           active run ${explainer.active_run.id} (${explainer.active_run.status})`
+      );
+    if (explainer.queue_position !== null)
+      lines.push(`           queue position ${explainer.queue_position}`);
+    lines.push(
+      `           attempts ${explainer.attempt_count}/${explainer.attempt_limit}; parked ${explainer.parked ? "yes" : "no"}`
+    );
   }
   lines.push("  Capacity, heartbeats and spending are advisory and may change at any moment.");
   return lines;
 }
+function repoDetail(repo) {
+  return `${repo.url}; branch ${repo.branch ?? "repository default branch"}; directory ${repo.dir}`;
+}
+function effectiveRepositoryLines(preview) {
+  const lines = ["", "Effective repositories:"];
+  for (const [label, context] of [
+    ["  before", preview.context.before],
+    ["  after ", preview.context.after]
+  ]) {
+    lines.push(`${label}:`);
+    if (!context.repos.length) lines.push("    none");
+    for (const repo of context.repos) {
+      lines.push(`    ${repo.name} \u2014 effective (${repo.scope.label})`);
+      lines.push(`      ${repoDetail(repo)}`);
+      for (const loser of context.overridden.filter(
+        (item) => item.kind === "repo" && item.name === repo.name
+      )) {
+        lines.push(
+          `      overridden candidate ${loser.name} (${loser.scope.label}) \u2014 overridden by ${loser.overridden_by}`
+        );
+        lines.push(
+          `        ${loser.repo ? repoDetail(loser.repo) : "checkout details unavailable from this server"}`
+        );
+      }
+    }
+  }
+  return lines;
+}
+function conflictLines(preview) {
+  const lines = ["", "Repository checkout conflicts:"];
+  for (const [label, context] of [
+    ["  before", preview.context.before],
+    ["  after ", preview.context.after]
+  ]) {
+    if (!context.conflicts.length) lines.push(`${label}: none`);
+    for (const conflict of context.conflicts) {
+      const participants = conflict.item_ids.map((id) => {
+        const repo = context.repos.find((item) => item.item_id === id);
+        return repo ? `${repo.name} (${repo.scope.label})` : id;
+      }).join(", ");
+      lines.push(`${label}: ${conflict.dir} \u2014 ${participants}`);
+    }
+  }
+  return lines;
+}
 function formatTransferPreview(preview) {
   const p = preview.preserved;
   const lines = [
@@ -5097,20 +5167,19 @@ function formatTransferPreview(preview) {
     `  pins: ${p.pinned_runner_id ?? "none"}${p.pinned_tier ? ` (${p.pinned_tier})` : ""}; attempts ${p.attempt_count}${p.parked ? "; parked" : ""}`
   ];
   const changes = inspectableChanges(preview);
-  lines.push("", changes.length ? "Guidance changes:" : "Guidance: unchanged by this move.");
+  lines.push("", changes.length ? "Guidance:" : "Guidance: none.");
   changes.forEach((change, index) => {
     lines.push(`  [${index}] ${change.kind} "${change.name}" \u2014 ${CHANGE_LABEL[change.change]}`);
     const scopeBefore = change.scope_before?.label ?? "\u2014";
     const scopeAfter = change.scope_after?.label ?? "\u2014";
     lines.push(`      scope: ${scopeBefore} => ${scopeAfter}`);
+    lines.push(
+      `      before: ${effectiveness(Boolean(change.scope_before), change.effective_before)}; after: ${effectiveness(Boolean(change.scope_after), change.effective_after)}`
+    );
     const repo = repoLine(change);
     if (repo) lines.push(repo);
   });
-  for (const conflict of preview.context.after.conflicts) {
-    lines.push(
-      `  ! ${conflict.item_ids.length} repositories still want the "${conflict.dir}" directory at the destination`
-    );
-  }
+  lines.push(...effectiveRepositoryLines(preview), ...conflictLines(preview));
   lines.push(...routingLines(preview));
   if (preview.schedule) {
     lines.push(
@@ -5133,11 +5202,24 @@ function formatTransferPreview(preview) {
 function formatTransferItem(preview, index) {
   const change = inspectableChanges(preview)[index];
   if (!change) return `no item [${index}] in this review`;
-  const side = change.change === "removed" ? preview.context.before : preview.context.after;
-  const prompt = side.prompt.parts.find((part) => part.item_id === change.item_id);
-  if (prompt) return `${change.name} (prompt, ${prompt.scope.label})
-${prompt.body}`;
-  const skill = side.skills.find((s) => s.item_id === change.item_id);
+  const beforePrompt = preview.context.before.prompt.parts.find(
+    (part) => part.item_id === change.item_id
+  );
+  const afterPrompt = preview.context.after.prompt.parts.find(
+    (part) => part.item_id === change.item_id
+  );
+  if (beforePrompt || afterPrompt) {
+    if (beforePrompt?.body === afterPrompt?.body)
+      return `${change.name} (prompt, Before and after)
+${beforePrompt?.body ?? afterPrompt?.body}`;
+    return [
+      beforePrompt ? `${change.name} (prompt, Before \u2014 ${beforePrompt.scope.label})
+${beforePrompt.body}` : "",
+      afterPrompt ? `${change.name} (prompt, After \u2014 ${afterPrompt.scope.label})
+${afterPrompt.body}` : ""
+    ].filter(Boolean).join("\n\n");
+  }
+  const skill = preview.context.after.skills.find((s) => s.item_id === change.item_id) ?? preview.context.before.skills.find((s) => s.item_id === change.item_id);
   if (skill) {
     return [
       `${change.name} (skill, ${skill.scope.label})`,
@@ -5145,11 +5227,17 @@ ${prompt.body}`;
 ${f.content}`)
     ].join("\n");
   }
-  const repo = side.repos.find((r) => r.item_id === change.item_id);
-  if (repo)
-    return `${change.name} (repo, ${repo.scope.label})
-${repo.url} ${repo.branch ?? ""} \u2192 ${repo.dir}`;
-  return `${change.name} (${change.kind}) is not effective on either side; nothing to show.`;
+  const repo = preview.context.after.repos.find((r) => r.item_id === change.item_id) ?? preview.context.before.repos.find((r) => r.item_id === change.item_id);
+  if (repo) return `${change.name} (repo, ${repo.scope.label})
+${repoDetail(repo)}`;
+  const candidate = [
+    ...preview.context.after.overridden,
+    ...preview.context.before.overridden
+  ].find((item) => item.item_id === change.item_id);
+  if (candidate?.repo)
+    return `${change.name} (overridden repo candidate, ${candidate.scope.label})
+${repoDetail(candidate.repo)}`;
+  return `${change.name} (${change.kind}) is an overridden candidate; content details are unavailable from this server.`;
 }
 function formatTransferResult(result) {
   if (result.status === "noop") {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "tines",
-  "version": "0.0.176",
+  "version": "0.0.178",
   "description": "CLI for Tines, an orchestration layer for AI agents",
   "repository": {
     "type": "git",