refactor: remove local mcp placeholders

Author: thymikee

src/commands/semantic-command-surface.ts

@@ -4,7 +4,6 @@ import { semanticClientCommands } from './semantic-client-commands.ts';
 import type { JsonSchema } from './semantic-contract.ts';
 import { bootSemanticCommand } from './semantic-device.ts';
 import { interactionSemanticCommands } from './semantic-interactions.ts';
-import { semanticLocalCommands } from './semantic-local-commands.ts';
 import {
   isSemanticBatchCommand as isSemanticGrammarBatchCommand,
   semanticBatchCommandNames,
@@ -23,7 +22,6 @@ type CommandSurfaceEntry<TDefinition extends AnySemanticCommandDefinition> = {
   definition: TDefinition;
   batch: boolean;
   genericCli: boolean;
-  mcp: 'tool' | 'local-boundary';
 };
 
 function commandSurfaceEntry<
@@ -100,13 +98,6 @@ const baseCommandSurface = [
   ...semanticClientCommands.map((definition) =>
     commandSurfaceEntry(definition, commandMetadata(definition.name)),
   ),
-  ...semanticLocalCommands.map((definition) =>
-    commandSurfaceEntry(definition, {
-      batch: false,
-      genericCli: false,
-      mcp: 'local-boundary',
-    }),
-  ),
 ] as const;
 
 const batchSemanticCommand = createBatchSemanticCommand(semanticBatchCommandNames);
@@ -116,7 +107,6 @@ const semanticCommandSurface = [
   commandSurfaceEntry(batchSemanticCommand, {
     batch: false,
     genericCli: true,
-    mcp: 'tool',
   }),
 ] as const;
 
@@ -134,14 +124,11 @@ function commandMetadata(
   return {
     batch: isSemanticGrammarBatchCommand(name),
     genericCli: genericCliNames.has(name),
-    mcp: 'tool',
   };
 }
 
 export function listSemanticMcpToolDefinitions(): AnySemanticCommandDefinition[] {
-  return semanticCommandSurface
-    .filter((entry) => entry.mcp === 'tool' || entry.mcp === 'local-boundary')
-    .map((entry) => entry.definition);
+  return semanticCommandSurface.map((entry) => entry.definition);
 }
 
 export function listSemanticGenericCliCommands(): SemanticCliCommand[] {

src/commands/semantic-local-commands.ts

@@ -3,7 +3,15 @@ import { test } from 'vitest';
 import { getCliCommandNames } from '../../utils/command-schema.ts';
 import { handleMcpMessage } from '../router.ts';
 
-test('MCP exposes every CLI command as a semantic direct tool except the MCP transport command', async () => {
+const LOCAL_ONLY_CLI_COMMANDS = new Set([
+  'auth',
+  'connect',
+  'connection',
+  'disconnect',
+  'react-devtools',
+]);
+
+test('MCP exposes every automatable CLI command as a semantic direct tool', async () => {
   const response = await handleMcpMessage({
     jsonrpc: '2.0',
     id: 1,
@@ -15,7 +23,7 @@ test('MCP exposes every CLI command as a semantic direct tool except the MCP tra
     (tool) => tool.name,
   );
   const expectedToolNames = getCliCommandNames()
-    .filter((command) => command !== 'mcp')
+    .filter((command) => command !== 'mcp' && !LOCAL_ONLY_CLI_COMMANDS.has(command))
     .sort();
 
   assert.deepEqual(tools.filter((name) => name !== 'status').sort(), expectedToolNames);