codecartographer-pi 0.22.2 → 0.22.3

package/.codecarto/workflow/scaffold-version.yaml

@@ -3,4 +3,4 @@
 # workspace's framework-owned files (GUIDE.md, templates/, workflow/ pipelines
 # and VALIDATE.md) predate the running release. Written at release time and
 # copied verbatim by init — never edit by hand.
-scaffold_version: 0.22.2
+scaffold_version: 0.22.3

package/dist/extensions/codecarto/completions.d.ts

@@ -0,0 +1,18 @@
+export interface CompletionCandidate {
+    value: string;
+    label?: string;
+    description?: string;
+}
+export interface CompletionItem {
+    value: string;
+    label: string;
+    description?: string;
+}
+/**
+ * Complete the last whitespace-separated token of `prefix` from `candidates`,
+ * returning items whose `value` is the full argument text Pi should put in
+ * the line — every earlier token, then the candidate. A candidate already
+ * typed as an earlier token is not offered again. Returns null when nothing
+ * matches, which is what Pi expects for "no popup".
+ */
+export declare function completeLastToken(prefix: string, candidates: readonly CompletionCandidate[]): CompletionItem[] | null;

package/dist/extensions/codecarto/completions.js

@@ -0,0 +1,36 @@
+// Argument completion for the /codecarto-* slash commands.
+//
+// Pi hands `getArgumentCompletions` everything typed after the command name,
+// and when the user accepts an item it replaces all of that text with the
+// item's `value` (pi 0.85: `argumentText = textBeforeCursor.slice(spaceIndex
+// + 1)` on the way in, `applyCompletion` slicing `cursorCol - prefix.length`
+// on the way out). A completer that matches the *last* token but returns a
+// bare flag therefore erases every flag typed before it: with the popup open,
+// `/codecarto-next --auto --llm-steer` became `/codecarto-next --llm-steer` on
+// Enter, the steered phase ran alone, and the auto run never started. The
+// same shape turned `--auto --strict` into `--strict`, the one combination
+// the parser rejects.
+//
+// So: match the last token, but return the whole line.
+/**
+ * Complete the last whitespace-separated token of `prefix` from `candidates`,
+ * returning items whose `value` is the full argument text Pi should put in
+ * the line — every earlier token, then the candidate. A candidate already
+ * typed as an earlier token is not offered again. Returns null when nothing
+ * matches, which is what Pi expects for "no popup".
+ */
+export function completeLastToken(prefix, candidates) {
+    const tokens = prefix.split(/\s+/);
+    const last = tokens.pop() ?? "";
+    const earlier = tokens.filter((token) => token.length > 0);
+    const head = earlier.join(" ");
+    const typed = new Set(earlier);
+    const items = candidates
+        .filter((candidate) => candidate.value.startsWith(last) && !typed.has(candidate.value))
+        .map((candidate) => ({
+        value: head ? `${head} ${candidate.value}` : candidate.value,
+        label: candidate.label ?? candidate.value,
+        ...(candidate.description && { description: candidate.description }),
+    }));
+    return items.length > 0 ? items : null;
+}

package/dist/extensions/codecarto/index.js

@@ -6,6 +6,7 @@ import { parseDashboardFlags } from "./dashboard-flags.js";
 import { narrateDashboard } from "./dashboard-narrator.js";
 import { parseBroadsideFlags, KNOWN_BROADSIDE_TOKENS } from "./broadside-flags.js";
 import { parseNextFlags } from "./next-flags.js";
+import { completeLastToken } from "./completions.js";
 import { buildPiGuideMessage } from "./guide-framing.js";
 import { isCtxLive, notifyCtx } from "./notify.js";
 import { phaseCompactionExtension } from "./phase-compaction.js";
@@ -432,12 +433,7 @@ export default function codeCartographerExtension(pi) {
     });
     pi.registerCommand("codecarto-init", {
         description: "Initialize .codecarto/ in the current repository",
-        getArgumentCompletions: (prefix) => {
-            const items = Object.keys(PIPELINE_ALIASES)
-                .filter((value) => value.startsWith(prefix))
-                .map((value) => ({ value, label: value }));
-            return items.length > 0 ? items : null;
-        },
+        getArgumentCompletions: (prefix) => completeLastToken(prefix, Object.keys(PIPELINE_ALIASES).map((value) => ({ value }))),
         handler: async (args, ctx) => {
             const trimmedArgs = args.trim();
             const pipelineChoice = resolvePipelineChoice(trimmedArgs);
@@ -536,12 +532,7 @@ export default function codeCartographerExtension(pi) {
     });
     pi.registerCommand("codecarto-switch-pipeline", {
         description: "Switch the active pipeline without losing findings or progress: /codecarto-switch-pipeline <variant>",
-        getArgumentCompletions: (prefix) => {
-            const items = Object.keys(PIPELINE_ALIASES)
-                .filter((value) => value.startsWith(prefix))
-                .map((value) => ({ value, label: value }));
-            return items.length > 0 ? items : null;
-        },
+        getArgumentCompletions: (prefix) => completeLastToken(prefix, Object.keys(PIPELINE_ALIASES).map((value) => ({ value }))),
         handler: async (args, ctx) => {
             const trimmedArgs = args.trim();
             if (!trimmedArgs) {
@@ -595,16 +586,17 @@ export default function codeCartographerExtension(pi) {
             // --strict is offered only once --auto is present, because on its own
             // it is rejected — suggesting it standalone invites the one error the
             // parser has.
+            // Each item's value is the whole argument line (see completions.ts):
+            // accepting `--llm-steer` after `--auto ` keeps the `--auto`.
             const autoAlreadyTyped = prefix.includes("--auto");
-            const items = [
-                { value: "--auto", label: "--auto", description: "run every remaining phase back to back (recommended with --llm-steer)" },
-                { value: "--llm-steer", label: "--llm-steer", description: "seed each phase from the previous phase's closeout; no effect on the first phase" },
-                { value: "--no-llm-steer", label: "--no-llm-steer", description: "force steering off when the workspace config turns it on" },
+            return completeLastToken(prefix, [
+                { value: "--auto", description: "run every remaining phase back to back (recommended with --llm-steer)" },
+                { value: "--llm-steer", description: "seed each phase from the previous phase's closeout; no effect on the first phase" },
+                { value: "--no-llm-steer", description: "force steering off when the workspace config turns it on" },
                 ...(autoAlreadyTyped
-                    ? [{ value: "--strict", label: "--strict", description: "with --auto: stop on PASS WITH GAPS instead of advancing" }]
+                    ? [{ value: "--strict", description: "with --auto: stop on PASS WITH GAPS instead of advancing" }]
                     : []),
-            ].filter((item) => item.value.startsWith(prefix.split(/\s+/).pop() ?? prefix));
-            return items.length > 0 ? items : null;
+            ]);
         },
         handler: async (args, ctx) => {
             const flags = parseNextFlags(args);
@@ -967,10 +959,7 @@ export default function codeCartographerExtension(pi) {
         description: "Read the packaged CodeCartographer agent guide into the session: /codecarto-guide [topic]",
         getArgumentCompletions: async (prefix) => {
             const topics = await listGuideTopics().catch(() => ["overview"]);
-            const items = topics
-                .filter((value) => value.startsWith(prefix))
-                .map((value) => ({ value, label: value }));
-            return items.length > 0 ? items : null;
+            return completeLastToken(prefix, topics.map((value) => ({ value })));
         },
         handler: async (args, ctx) => {
             // The guide is packaged with the extension, not copied into a
@@ -1004,12 +993,9 @@ export default function codeCartographerExtension(pi) {
     });
     pi.registerCommand("codecarto-broadside", {
         description: "Batch reconnaissance (Broad-Side): /codecarto-broadside [submit|collect|status|models] [lenses…] [--model=ID] [--lens-model=LENS:ID] [flags]",
-        getArgumentCompletions: (prefix) => {
-            const items = KNOWN_BROADSIDE_TOKENS
-                .filter((value) => value.startsWith(prefix))
-                .map((value) => ({ value, label: value }));
-            return items.length > 0 ? items : null;
-        },
+        // Completes the token under the cursor, so lens names and flags are
+        // offered after the action too, and keeps everything typed before it.
+        getArgumentCompletions: (prefix) => completeLastToken(prefix, KNOWN_BROADSIDE_TOKENS.map((value) => ({ value }))),
         handler: async (args, ctx) => {
             const flags = parseBroadsideFlags(args);
             if (flags.unknown.length > 0) {
@@ -1439,12 +1425,7 @@ export default function codeCartographerExtension(pi) {
     });
     pi.registerCommand("codecarto-dashboard", {
         description: "Regenerate .codecarto/dashboard.html (use --narrate for an LLM executive summary)",
-        getArgumentCompletions: (prefix) => {
-            const items = ["--narrate"]
-                .filter((value) => value.startsWith(prefix))
-                .map((value) => ({ value, label: value }));
-            return items.length > 0 ? items : null;
-        },
+        getArgumentCompletions: (prefix) => completeLastToken(prefix, [{ value: "--narrate" }]),
         handler: async (args, ctx) => {
             const flags = parseDashboardFlags(args);
             if (flags.unknown.length > 0) {
@@ -1515,10 +1496,7 @@ export default function codeCartographerExtension(pi) {
         description: "Apply a post-pipeline amendment from .codecarto/scratch/amendments/, after a preview: /codecarto-amend <name | scratch/amendments/name.yaml>",
         getArgumentCompletions: async (prefix) => {
             const names = await listAmendmentNames(join(sessionCwd ?? process.cwd(), ".codecarto"));
-            const items = names
-                .filter((value) => value.startsWith(prefix))
-                .map((value) => ({ value, label: value }));
-            return items.length > 0 ? items : null;
+            return completeLastToken(prefix, names.map((value) => ({ value })));
         },
         handler: async (args, ctx) => {
             const state = await ensureWorkspaceState(ctx);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "codecartographer-pi",
-  "version": "0.22.2",
+  "version": "0.22.3",
   "mcpName": "io.github.HuginnIndustries/codecartographer",
   "description": "Turn an unfamiliar codebase into a validated reimplementation spec, then synthesize confirmed specs and a product vision into a traceable plan.",
   "type": "module",