@phi-code-admin/phi-code 0.96.1 → 0.97.0

package/dist/modes/interactive/interactive-mode.js

@@ -20,7 +20,7 @@ import { DefaultPackageManager } from "../../core/package-manager.js";
 import { BUILT_IN_PROVIDER_DISPLAY_NAMES } from "../../core/provider-display-names.js";
 import { formatMissingSessionCwdPrompt, MissingSessionCwdError } from "../../core/session-cwd.js";
 import { SessionManager } from "../../core/session-manager.js";
-import { BUILTIN_SLASH_COMMANDS } from "../../core/slash-commands.js";
+import { BUILTIN_SLASH_COMMANDS, matchBareBuiltinWithArgs } from "../../core/slash-commands.js";
 import { getChangelogPath, getNewEntries, parseChangelog } from "../../utils/changelog.js";
 import { copyToClipboard } from "../../utils/clipboard.js";
 import { extensionForImageMimeType, readClipboardImage } from "../../utils/clipboard-image.js";
@@ -2098,9 +2098,17 @@ export class InteractiveMode {
                 return;
             }
             if (text === "/debug") {
-                this.handleDebugCommand();
-                this.editor.setText("");
-                return;
+                // An extension may own /debug (the debug orchestrator does). The TUI
+                // debug overlay only claims the bare form when nothing else does —
+                // otherwise it would silently shadow the extension command.
+                const extensionOwnsDebug = this.session.extensionRunner
+                    .getRegisteredCommands()
+                    .some((command) => command.name === "debug" || command.invocationName === "debug");
+                if (!extensionOwnsDebug) {
+                    this.handleDebugCommand();
+                    this.editor.setText("");
+                    return;
+                }
             }
             if (text === "/arminsayshi") {
                 this.handleArminSaysHi();
@@ -2122,6 +2130,18 @@ export class InteractiveMode {
                 await this.shutdown();
                 return;
             }
+            // A bare builtin invoked with arguments used to leak to the model as
+            // prose ("/new please"). Surface it instead — unless an extension
+            // registered a command with that name (it then owns the arguments).
+            const bareWithArgs = matchBareBuiltinWithArgs(text);
+            if (bareWithArgs &&
+                !this.session.extensionRunner
+                    .getRegisteredCommands()
+                    .some((command) => command.name === bareWithArgs || command.invocationName === bareWithArgs)) {
+                this.showWarning(`/${bareWithArgs} takes no arguments. Run \`/${bareWithArgs}\` alone.`);
+                this.editor.setText(text);
+                return;
+            }
             // Handle bash command (! for normal, !! for excluded from context)
             if (text.startsWith("!")) {
                 const isExcluded = text.startsWith("!!");