@bretwardjames/ghp-mcp 0.1.3 → 0.1.4

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @bretwardjames/ghp-mcp@0.1.2 build /home/bretwardjames/IdeaProjects/ghp/packages/mcp
+> @bretwardjames/ghp-mcp@0.1.3 build /home/bretwardjames/IdeaProjects/ghp/packages/mcp
 > tsup src/index.ts --format esm --dts
 
 CLI Building entry: src/index.ts
@@ -8,7 +8,7 @@ CLI tsup v8.5.1
 CLI Target: es2022
 ESM Build start
 ESM dist/index.js 52.39 KB
-ESM ⚡️ Build success in 38ms
+ESM ⚡️ Build success in 54ms
 DTS Build start
-DTS ⚡️ Build success in 1676ms
+DTS ⚡️ Build success in 1637ms
 DTS dist/index.d.ts 20.00 B

package/CHANGELOG.md

@@ -1,5 +1,49 @@
 # @bretwardjames/ghp-mcp
 
+## 0.1.4
+
+### Patch Changes
+
+- f0e69f8: Centralize hook firing in core workflows
+
+  ## @bretwardjames/ghp-core (minor)
+
+  - Add workflow layer with functions that combine operations + hook firing:
+
+    - `createIssueWorkflow` - Create issue and fire `issue-created` hook
+    - `startIssueWorkflow` - Start working on issue and fire `issue-started` hook
+    - `createPRWorkflow` - Create PR and fire `pr-created` hook
+    - `createWorktreeWorkflow` - Create worktree and fire `worktree-created` hook
+    - `removeWorktreeWorkflow` - Remove worktree and fire `worktree-removed` hook
+
+  - Add `cwd` option to hook executor for firing hooks from inside worktrees
+  - Add tests for all workflow functions (24 tests)
+  - Add vitest test runner
+
+  ## @bretwardjames/ghp-cli (patch)
+
+  - Hook firing order improved: `worktree-created` fires before `issue-started` in parallel mode
+  - Hooks now fire from inside the worktree directory when using `--parallel`
+
+  ## @bretwardjames/ghp-mcp (patch)
+
+  - MCP `start` tool now fires `issue-started` hook
+  - MCP `add-issue` tool now fires `issue-created` hook
+
+  ## gh-projects (patch)
+
+  - VS Code extension now fires `issue-started` hook when starting work
+  - VS Code extension now fires `worktree-created` and `issue-started` hooks when creating worktrees
+  - Hooks fire from inside the worktree directory for correct file placement
+
+- Updated dependencies [62b7941]
+- Updated dependencies [16c3603]
+- Updated dependencies [f0e69f8]
+- Updated dependencies [3fce458]
+- Updated dependencies [c5b3627]
+- Updated dependencies [25143fe]
+  - @bretwardjames/ghp-core@0.5.0
+
 ## 0.1.3
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bretwardjames/ghp-mcp",
-  "version": "0.1.3",
+  "version": "0.1.4",
   "description": "MCP server for ghp (GitHub Projects)",
   "publishConfig": {
     "access": "public"
@@ -29,7 +29,7 @@
   "dependencies": {
     "@modelcontextprotocol/sdk": "^1.0.0",
     "zod": "^3.22.0",
-    "@bretwardjames/ghp-core": "0.4.0"
+    "@bretwardjames/ghp-core": "0.5.0"
   },
   "devDependencies": {
     "@types/node": "^20.10.0",

package/src/tools/add-issue.ts

@@ -2,6 +2,11 @@ import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import * as z from 'zod';
 import type { ServerContext } from '../server.js';
 import type { ToolMeta } from '../types.js';
+import {
+    executeHooksForEvent,
+    hasHooksForEvent,
+    type IssueCreatedPayload,
+} from '@bretwardjames/ghp-core';
 
 /** Tool metadata for registry */
 export const meta: ToolMeta = {
@@ -115,6 +120,30 @@ export function register(server: McpServer, context: ServerContext): void {
                     }
                 }
 
+                // Fire issue-created hook
+                if (hasHooksForEvent('issue-created')) {
+                    const payload: IssueCreatedPayload = {
+                        repo: `${repo.owner}/${repo.name}`,
+                        issue: {
+                            number: result.number,
+                            title,
+                            body: body || '',
+                            url: issueUrl,
+                        },
+                    };
+
+                    const hookResults = await executeHooksForEvent('issue-created', payload);
+                    const successCount = hookResults.filter(r => r.success).length;
+                    const failCount = hookResults.length - successCount;
+
+                    if (hookResults.length > 0) {
+                        message += `\n\nHooks: ${successCount} succeeded`;
+                        if (failCount > 0) {
+                            message += `, ${failCount} failed`;
+                        }
+                    }
+                }
+
                 return {
                     content: [
                         {

package/src/tools/start.ts

@@ -2,6 +2,12 @@ import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
 import * as z from 'zod';
 import type { ServerContext } from '../server.js';
 import type { ToolMeta } from '../types.js';
+import {
+    getCurrentBranch,
+    executeHooksForEvent,
+    hasHooksForEvent,
+    type IssueStartedPayload,
+} from '@bretwardjames/ghp-core';
 
 /** Tool metadata for registry */
 export const meta: ToolMeta = {
@@ -129,16 +135,7 @@ export function register(server: McpServer, context: ServerContext): void {
                     inProgressOption.id
                 );
 
-                if (success) {
-                    return {
-                        content: [
-                            {
-                                type: 'text',
-                                text: `Started work on issue #${issue} "${item.title}" - status set to "${inProgressOption.name}".${blockingWarning}`,
-                            },
-                        ],
-                    };
-                } else {
+                if (!success) {
                     return {
                         content: [
                             {
@@ -149,6 +146,42 @@ export function register(server: McpServer, context: ServerContext): void {
                         isError: true,
                     };
                 }
+
+                // Fire issue-started hook
+                let hookInfo = '';
+                if (hasHooksForEvent('issue-started')) {
+                    const branch = await getCurrentBranch() || '';
+                    const payload: IssueStartedPayload = {
+                        repo: `${repo.owner}/${repo.name}`,
+                        issue: {
+                            number: issue,
+                            title: item.title,
+                            body: '', // Body not available from ProjectItem
+                            url: `https://github.com/${repo.owner}/${repo.name}/issues/${issue}`,
+                        },
+                        branch,
+                    };
+
+                    const hookResults = await executeHooksForEvent('issue-started', payload);
+                    const successCount = hookResults.filter(r => r.success).length;
+                    const failCount = hookResults.length - successCount;
+
+                    if (hookResults.length > 0) {
+                        hookInfo = `\n\nHooks: ${successCount} succeeded`;
+                        if (failCount > 0) {
+                            hookInfo += `, ${failCount} failed`;
+                        }
+                    }
+                }
+
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: `Started work on issue #${issue} "${item.title}" - status set to "${inProgressOption.name}".${blockingWarning}${hookInfo}`,
+                        },
+                    ],
+                };
             } catch (error) {
                 return {
                     content: [