@vibe-forge/mcp 0.9.1 → 0.9.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@vibe-forge/mcp",
-  "version": "0.9.1",
+  "version": "0.9.2",
   "description": "Vibe Forge MCP server",
   "imports": {
     "#~/*.js": {
@@ -33,13 +33,13 @@
     "@modelcontextprotocol/sdk": "^1.25.3",
     "commander": "^12.1.0",
     "zod": "^3.24.1",
-    "@vibe-forge/register": "^0.9.1",
     "@vibe-forge/config": "^0.9.0",
-    "@vibe-forge/hooks": "^0.9.1",
-    "@vibe-forge/types": "^0.9.0",
-    "@vibe-forge/utils": "^0.9.1",
     "@vibe-forge/cli-helper": "^0.9.0",
-    "@vibe-forge/task": "^0.9.0"
+    "@vibe-forge/task": "^0.9.0",
+    "@vibe-forge/register": "^0.9.0",
+    "@vibe-forge/hooks": "^0.9.1",
+    "@vibe-forge/utils": "^0.9.0",
+    "@vibe-forge/types": "^0.9.0"
   },
   "scripts": {
     "test": "pnpm -C ../.. exec vitest run --workspace vitest.workspace.ts --project bundler packages/mcp/__tests__"

package/src/command.ts

@@ -59,5 +59,5 @@ export const configureMcpCommand = (command: Command, version: string) => (
 )
 
 export function registerMcpCommand(program: Command, version: string) {
-  return configureMcpCommand(program.command('mcp'), version)
+  return configureMcpCommand(program, version)
 }

package/src/tools/interaction/ask-user.ts

@@ -16,7 +16,11 @@ const Schema = z.object({
     currentMode: z.enum(['default', 'acceptEdits', 'plan', 'dontAsk', 'bypassPermissions']).optional(),
     suggestedMode: z.enum(['default', 'acceptEdits', 'plan', 'dontAsk', 'bypassPermissions']).optional(),
     deniedTools: z.array(z.string()).optional(),
-    reasons: z.array(z.string()).optional()
+    reasons: z.array(z.string()).optional(),
+    subjectKey: z.string().optional(),
+    subjectLabel: z.string().optional(),
+    scope: z.enum(['tool']).optional(),
+    projectConfigPath: z.string().optional()
   }).optional().describe('Extra context for permission escalation prompts')
 })
 

package/src/tools/task/index.ts

@@ -65,7 +65,7 @@ export const createTaskRegister = () => {
         await callHook('StartTasks', {
           cwd: process.cwd(),
           sessionId: process.env.__VF_PROJECT_AI_SESSION_ID__!,
-          tasks: resolvedTasks
+          tasks
         })
         const syncResults = parentSessionId
           ? await Promise.allSettled(resolvedTasks.map(task =>

package/src/tools/task/manager.ts

@@ -3,7 +3,7 @@ import process from 'node:process'
 import { loadInjectDefaultSystemPromptValue, mergeSystemPrompts } from '@vibe-forge/config'
 import { callHook } from '@vibe-forge/hooks'
 import { generateAdapterQueryOptions, run } from '@vibe-forge/task'
-import type { ChatMessage, McpTaskOutputEvent, McpTaskSession, SessionPermissionMode } from '@vibe-forge/types'
+import type { AdapterOutputEvent, ChatMessage, McpTaskSession, SessionPermissionMode } from '@vibe-forge/types'
 import { extractTextFromMessage } from '@vibe-forge/utils/chat-message'
 
 import { fetchSessionMessages, postSessionEvent } from '#~/sync.js'
@@ -118,7 +118,7 @@ export class TaskManager {
         mcpServers: resolvedConfig.mcpServers,
         promptAssetIds: resolvedConfig.promptAssetIds,
         assetBundle: resolvedConfig.assetBundle,
-        onEvent: (event: McpTaskOutputEvent) => {
+        onEvent: (event: AdapterOutputEvent) => {
           this.handleEvent(taskId, event)
         }
       })
@@ -165,7 +165,7 @@ export class TaskManager {
     return { taskId }
   }
 
-  private handleEvent(taskId: string, event: McpTaskOutputEvent) {
+  private handleEvent(taskId: string, event: AdapterOutputEvent) {
     const task = this.tasks.get(taskId)
     if (!task) return
 
@@ -221,6 +221,8 @@ export class TaskManager {
         this.stopServerPolling(taskId)
         task.onStop?.()
         break
+      default:
+        break
     }
   }
 
@@ -272,7 +274,7 @@ export class TaskManager {
     }
   }
 
-  private async syncEvent(task: TaskInfo, event: McpTaskOutputEvent) {
+  private async syncEvent(task: TaskInfo, event: AdapterOutputEvent) {
     if (!task.serverSync) return
     try {
       await postSessionEvent(task.serverSync.sessionId, event as unknown as Record<string, unknown>)

package/__tests__/start-tasks.spec.ts

@@ -1,94 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest'
-
-const mocks = vi.hoisted(() => ({
-  callHook: vi.fn(),
-  uuid: vi.fn(),
-  createChildSession: vi.fn(),
-  getParentSessionId: vi.fn(),
-  startTask: vi.fn(),
-  getTask: vi.fn()
-}))
-
-vi.mock('@vibe-forge/hooks', () => ({
-  callHook: mocks.callHook
-}))
-
-vi.mock('@vibe-forge/utils/uuid', () => ({
-  uuid: mocks.uuid
-}))
-
-vi.mock('#~/sync.js', () => ({
-  createChildSession: mocks.createChildSession,
-  getParentSessionId: mocks.getParentSessionId
-}))
-
-vi.mock('#~/tools/task/manager.js', () => ({
-  TaskManager: class {
-    startTask = mocks.startTask
-    getTask = mocks.getTask
-  }
-}))
-
-import { createTaskRegister } from '#~/tools/task/index.js'
-
-import { createToolTester } from './mcp-test-utils.js'
-
-describe('StartTasks hook payload', () => {
-  beforeEach(() => {
-    vi.clearAllMocks()
-    mocks.callHook.mockResolvedValue({ continue: true })
-    mocks.getParentSessionId.mockReturnValue(undefined)
-    mocks.startTask.mockImplementation(async ({ taskId }: { taskId: string }) => ({ taskId }))
-    mocks.getTask.mockImplementation((taskId: string) => ({
-      taskId,
-      status: 'running',
-      logs: []
-    }))
-  })
-
-  it('passes resolved task ids to the StartTasks hook before launching child tasks', async () => {
-    mocks.uuid
-      .mockReturnValueOnce('task-1')
-      .mockReturnValueOnce('task-2')
-
-    const tester = createToolTester()
-    createTaskRegister()(tester.mockRegister)
-
-    await tester.callTool('StartTasks', {
-      tasks: [
-        { description: 'first task', type: 'entity', name: 'alpha' },
-        { description: 'second task', type: 'spec', name: 'beta', background: false }
-      ]
-    })
-
-    expect(mocks.callHook).toHaveBeenCalledWith(
-      'StartTasks',
-      expect.objectContaining({
-        tasks: [
-          {
-            taskId: 'task-1',
-            description: 'first task',
-            type: 'entity',
-            name: 'alpha'
-          },
-          {
-            taskId: 'task-2',
-            description: 'second task',
-            type: 'spec',
-            name: 'beta',
-            background: false
-          }
-        ]
-      })
-    )
-
-    expect(mocks.startTask).toHaveBeenNthCalledWith(
-      1,
-      expect.objectContaining({ taskId: 'task-1' })
-    )
-    expect(mocks.startTask).toHaveBeenNthCalledWith(
-      2,
-      expect.objectContaining({ taskId: 'task-2' })
-    )
-  })
-})