@bretwardjames/ghp-mcp 0.4.0 → 0.5.0

package/.turbo/turbo-build.log

@@ -1,5 +1,5 @@
 
-> @bretwardjames/ghp-mcp@0.4.0 build /home/bretwardjames/IdeaProjects/ghp/packages/mcp
+> @bretwardjames/ghp-mcp@0.5.0 build /home/bretwardjames/IdeaProjects/ghp/packages/mcp
 > tsup src/index.ts --format esm --dts
 
 CLI Building entry: src/index.ts
@@ -7,8 +7,8 @@ CLI Using tsconfig: tsconfig.json
 CLI tsup v8.5.1
 CLI Target: es2022
 ESM Build start
-ESM dist/index.js 97.52 KB
-ESM ⚡️ Build success in 30ms
+ESM dist/index.js 103.19 KB
+ESM ⚡️ Build success in 35ms
 DTS Build start
-DTS ⚡️ Build success in 1029ms
+DTS ⚡️ Build success in 1054ms
 DTS dist/index.d.ts 20.00 B

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @bretwardjames/ghp-mcp
 
+## 0.5.0
+
+### Minor Changes
+
+- Add field discovery, hotfix branching, and named flags
+
+  - `ghp fields` command to discover project fields and valid values
+  - `--hotfix [ref]` flag on `ghp start` to branch from tags/commits
+  - `--priority` and `--size` named flags on `ghp add`
+  - `--body-file` and `--body-stdin` flags for issue body input
+  - Verbose creation summary showing field assignments
+  - Fix silent flag dropping after positional title arg
+  - `get_fields` and `get_tags` MCP tools
+  - `validateRefString()` for shell injection prevention in git refs
+
+### Patch Changes
+
+- Updated dependencies
+  - @bretwardjames/ghp-core@0.9.0
+
 ## 0.4.0
 
 ### Minor Changes

package/README.md

@@ -73,8 +73,10 @@ The server uses the same GitHub authentication as the CLI. Run `ghp auth` to aut
 | `update_issue` | Update an issue's title and/or body |
 | `move_issue` | Move an issue to a different status |
 | `mark_done` | Mark an issue as done |
-| `start_work` | Start working on an issue |
+| `start_work` | Start working on an issue (supports `hotfix` param for branching from tags) |
 | `create_worktree` | Create a worktree for parallel development |
+| `get_fields` | List all project fields and their valid values |
+| `get_tags` | List git tags sorted newest first (for hotfix discovery) |
 | `assign_issue` | Assign users to an issue |
 | `add_comment` | Add a comment to an issue |
 | `set_field` | Set a field value on an issue |

package/dist/index.js

@@ -674,7 +674,8 @@ import * as z5 from "zod";
 import {
   getCurrentBranch,
   executeHooksForEvent,
-  hasHooksForEvent
+  hasHooksForEvent,
+  resolveRef
 } from "@bretwardjames/ghp-core";
 var meta5 = {
   name: "start_work",
@@ -688,10 +689,11 @@ function register5(server, context) {
       description: 'Start working on an issue by setting its status to "In Progress". Note: Branch creation is not supported in MCP context - use the CLI for that.',
       inputSchema: {
         issue: z5.number().describe("Issue number to start working on"),
-        updateStatus: z5.boolean().optional().describe("Whether to update the status (default: true)")
+        updateStatus: z5.boolean().optional().describe("Whether to update the status (default: true)"),
+        hotfix: z5.string().optional().describe("Branch from a specific tag or commit for hotfix branches. Use get_tags tool to discover available tags.")
       }
     },
-    async ({ issue, updateStatus = true }) => {
+    async ({ issue, updateStatus = true, hotfix }) => {
       const authenticated = await context.ensureAuthenticated();
       if (!authenticated) {
         return {
@@ -717,6 +719,24 @@ function register5(server, context) {
         };
       }
       try {
+        let hotfixInfo = "";
+        if (hotfix) {
+          const resolved = await resolveRef(hotfix);
+          if (!resolved) {
+            return {
+              content: [
+                {
+                  type: "text",
+                  text: `Error: Hotfix ref "${hotfix}" does not exist. Use get_tags to discover available tags.`
+                }
+              ],
+              isError: true
+            };
+          }
+          hotfixInfo = `
+
+Hotfix: Branch from "${hotfix}" (${resolved.substring(0, 7)}). Use the CLI to create the branch: ghp start ${issue} --hotfix ${hotfix}`;
+        }
         const item = await context.api.findItemByNumber(repo, issue);
         if (!item) {
           return {
@@ -738,7 +758,7 @@ WARNING: This issue is blocked by: ${openBlockers.map((b) => `#${b.number} (${b.
             content: [
               {
                 type: "text",
-                text: `Started work on issue #${issue} "${item.title}" (status not updated).${blockingWarning}`
+                text: `Started work on issue #${issue} "${item.title}" (status not updated).${blockingWarning}${hotfixInfo}`
               }
             ]
           };
@@ -820,7 +840,7 @@ Hooks: ${successCount} succeeded`;
           content: [
             {
               type: "text",
-              text: `Started work on issue #${issue} "${item.title}" - status set to "${inProgressOption.name}".${blockingWarning}${hookInfo}`
+              text: `Started work on issue #${issue} "${item.title}" - status set to "${inProgressOption.name}".${blockingWarning}${hotfixInfo}${hookInfo}`
             }
           ]
         };
@@ -1700,7 +1720,7 @@ function register13(server, context) {
         deleteHead: z13.boolean().optional().describe("Delete head branch after merge (default: true)")
       }
     },
-    async ({ number: number19, method = "squash", deleteHead = true }) => {
+    async ({ number: number20, method = "squash", deleteHead = true }) => {
       const authenticated = await context.ensureAuthenticated();
       if (!authenticated) {
         return {
@@ -1726,7 +1746,7 @@ function register13(server, context) {
         };
       }
       try {
-        const safePrNumber = validateNumericInput2(number19, "PR number");
+        const safePrNumber = validateNumericInput2(number20, "PR number");
         const args = ["pr", "merge", String(safePrNumber)];
         if (method === "squash") {
           args.push("--squash");
@@ -1755,7 +1775,7 @@ function register13(server, context) {
             isError: true
           };
         }
-        let message = `Merged PR #${number19} using ${method} strategy.`;
+        let message = `Merged PR #${number20} using ${method} strategy.`;
         if (deleteHead) {
           message += " Head branch deleted.";
         }
@@ -2993,6 +3013,191 @@ function register24(server, context) {
   );
 }
 
+// src/tools/fields.ts
+var fields_exports = {};
+__export(fields_exports, {
+  meta: () => meta25,
+  register: () => register25
+});
+import * as z24 from "zod";
+var meta25 = {
+  name: "get_fields",
+  category: "read"
+};
+function register25(server, context) {
+  server.registerTool(
+    "get_fields",
+    {
+      title: "Get Project Fields",
+      description: "Get all project fields and their valid values. Useful for discovering what values can be used with set_field, move_issue, or add_issue --field flags.",
+      inputSchema: {
+        project: z24.string().optional().describe("Project name to query (defaults to first project)")
+      }
+    },
+    async ({ project: projectName }) => {
+      const authenticated = await context.ensureAuthenticated();
+      if (!authenticated) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: "Error: Not authenticated. Please ensure gh CLI is authenticated or set GITHUB_TOKEN."
+            }
+          ],
+          isError: true
+        };
+      }
+      const repo = await context.getRepo();
+      if (!repo) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: "Error: Not in a git repository with a GitHub remote."
+            }
+          ],
+          isError: true
+        };
+      }
+      try {
+        const projects = await context.api.getProjects(repo);
+        if (projects.length === 0) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "No projects found for this repository."
+              }
+            ],
+            isError: true
+          };
+        }
+        let selectedProject = projects[0];
+        if (projectName) {
+          const match = projects.find(
+            (p) => p.title.toLowerCase() === projectName.toLowerCase()
+          );
+          if (!match) {
+            const available = projects.map((p) => p.title).join(", ");
+            return {
+              content: [
+                {
+                  type: "text",
+                  text: `Project "${projectName}" not found. Available projects: ${available}`
+                }
+              ],
+              isError: true
+            };
+          }
+          selectedProject = match;
+        }
+        const fields = await context.api.getProjectFields(selectedProject.id);
+        const result = {
+          project: selectedProject.title,
+          fields: fields.map((f) => ({
+            name: f.name,
+            type: f.type || "Text",
+            options: f.options?.map((o) => o.name)
+          }))
+        };
+        return {
+          content: [
+            {
+              type: "text",
+              text: JSON.stringify(result, null, 2)
+            }
+          ]
+        };
+      } catch (error) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: `Error fetching project fields: ${error instanceof Error ? error.message : String(error)}`
+            }
+          ],
+          isError: true
+        };
+      }
+    }
+  );
+}
+
+// src/tools/tags.ts
+var tags_exports = {};
+__export(tags_exports, {
+  meta: () => meta26,
+  register: () => register26
+});
+import * as z25 from "zod";
+import { listTags } from "@bretwardjames/ghp-core";
+var meta26 = {
+  name: "get_tags",
+  category: "read"
+};
+function register26(server, context) {
+  server.registerTool(
+    "get_tags",
+    {
+      title: "Get Tags",
+      description: "List git tags in the repository, sorted newest first. Useful for discovering available tags for hotfix branches with start_work.",
+      inputSchema: {
+        limit: z25.number().optional().describe("Maximum number of tags to return (default: 20)")
+      }
+    },
+    async ({ limit = 20 }) => {
+      const repo = await context.getRepo();
+      if (!repo) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: "Error: Not in a git repository with a GitHub remote."
+            }
+          ],
+          isError: true
+        };
+      }
+      try {
+        const tags = await listTags();
+        const limited = tags.slice(0, limit);
+        if (limited.length === 0) {
+          return {
+            content: [
+              {
+                type: "text",
+                text: "No tags found in this repository."
+              }
+            ]
+          };
+        }
+        return {
+          content: [
+            {
+              type: "text",
+              text: JSON.stringify({
+                total: tags.length,
+                showing: limited.length,
+                tags: limited
+              }, null, 2)
+            }
+          ]
+        };
+      } catch (error) {
+        return {
+          content: [
+            {
+              type: "text",
+              text: `Error listing tags: ${error instanceof Error ? error.message : String(error)}`
+            }
+          ],
+          isError: true
+        };
+      }
+    }
+  );
+}
+
 // src/tool-registry.ts
 var TOOLS = [
   // Read tools
@@ -3002,6 +3207,8 @@ var TOOLS = [
   get_progress_exports,
   get_issue_exports,
   standup_exports,
+  fields_exports,
+  tags_exports,
   // Action tools
   move_exports,
   done_exports,
@@ -3334,7 +3541,7 @@ function registerIssueResource(server, context) {
       description: "Detailed information about a specific issue including comments",
       mimeType: "application/json"
     },
-    async (uri, { number: number19 }) => {
+    async (uri, { number: number20 }) => {
       const authenticated = await context.ensureAuthenticated();
       if (!authenticated) {
         return {
@@ -3360,7 +3567,7 @@ function registerIssueResource(server, context) {
         };
       }
       try {
-        const issueNumber = parseInt(number19, 10);
+        const issueNumber = parseInt(number20, 10);
         if (isNaN(issueNumber)) {
           return {
             contents: [

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@bretwardjames/ghp-mcp",
-  "version": "0.4.0",
+  "version": "0.5.0",
   "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.8.0"
+    "@bretwardjames/ghp-core": "0.9.0"
   },
   "devDependencies": {
     "@types/node": "^20.10.0",

package/src/tool-registry.ts

@@ -35,6 +35,8 @@ import * as unlinkBranchTool from './tools/unlink-branch.js';
 // Phase 3: Lower Priority Tools
 import * as getIssueTool from './tools/get-issue.js';
 import * as standupTool from './tools/standup.js';
+import * as fieldsTool from './tools/fields.js';
+import * as tagsTool from './tools/tags.js';
 
 // Re-export types
 export type { ToolCategory, McpConfig, McpToolsConfig } from './types.js';
@@ -58,6 +60,8 @@ const TOOLS: ToolModule[] = [
     getProgressTool,
     getIssueTool,
     standupTool,
+    fieldsTool,
+    tagsTool,
     // Action tools
     moveTool,
     doneTool,

package/src/tools/fields.ts

@@ -0,0 +1,124 @@
+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';
+
+/** Tool metadata for registry */
+export const meta: ToolMeta = {
+    name: 'get_fields',
+    category: 'read',
+};
+
+/**
+ * Registers the get_fields tool.
+ * Returns project fields and their valid values (options for single-select, type for others).
+ */
+export function register(server: McpServer, context: ServerContext): void {
+    server.registerTool(
+        'get_fields',
+        {
+            title: 'Get Project Fields',
+            description:
+                'Get all project fields and their valid values. Useful for discovering what values can be used with set_field, move_issue, or add_issue --field flags.',
+            inputSchema: {
+                project: z
+                    .string()
+                    .optional()
+                    .describe('Project name to query (defaults to first project)'),
+            },
+        },
+        async ({ project: projectName }) => {
+            const authenticated = await context.ensureAuthenticated();
+            if (!authenticated) {
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: 'Error: Not authenticated. Please ensure gh CLI is authenticated or set GITHUB_TOKEN.',
+                        },
+                    ],
+                    isError: true,
+                };
+            }
+
+            const repo = await context.getRepo();
+            if (!repo) {
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: 'Error: Not in a git repository with a GitHub remote.',
+                        },
+                    ],
+                    isError: true,
+                };
+            }
+
+            try {
+                const projects = await context.api.getProjects(repo);
+                if (projects.length === 0) {
+                    return {
+                        content: [
+                            {
+                                type: 'text',
+                                text: 'No projects found for this repository.',
+                            },
+                        ],
+                        isError: true,
+                    };
+                }
+
+                // Select project by name if provided, otherwise first
+                let selectedProject = projects[0];
+                if (projectName) {
+                    const match = projects.find(
+                        p => p.title.toLowerCase() === projectName.toLowerCase()
+                    );
+                    if (!match) {
+                        const available = projects.map(p => p.title).join(', ');
+                        return {
+                            content: [
+                                {
+                                    type: 'text',
+                                    text: `Project "${projectName}" not found. Available projects: ${available}`,
+                                },
+                            ],
+                            isError: true,
+                        };
+                    }
+                    selectedProject = match;
+                }
+
+                const fields = await context.api.getProjectFields(selectedProject.id);
+
+                const result = {
+                    project: selectedProject.title,
+                    fields: fields.map(f => ({
+                        name: f.name,
+                        type: f.type || 'Text',
+                        options: f.options?.map(o => o.name),
+                    })),
+                };
+
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: JSON.stringify(result, null, 2),
+                        },
+                    ],
+                };
+            } catch (error) {
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: `Error fetching project fields: ${error instanceof Error ? error.message : String(error)}`,
+                        },
+                    ],
+                    isError: true,
+                };
+            }
+        }
+    );
+}

package/src/tools/index.ts

@@ -27,6 +27,8 @@ import { register as registerLinkBranch } from './link-branch.js';
 import { register as registerUnlinkBranch } from './unlink-branch.js';
 // Phase 3: Lower Priority Tools
 import { register as registerGetIssue } from './get-issue.js';
+import { register as registerGetFields } from './fields.js';
+import { register as registerGetTags } from './tags.js';
 
 /**
  * @deprecated Use registerEnabledTools from '../tool-registry.js' instead.
@@ -39,6 +41,8 @@ export function registerAllTools(server: McpServer, context: ServerContext): voi
     registerListWorktrees(server, context);
     registerGetProgress(server, context);
     registerGetIssue(server, context);
+    registerGetFields(server, context);
+    registerGetTags(server, context);
 
     // Action tools
     registerMove(server, context);

package/src/tools/start.ts

@@ -7,6 +7,7 @@ import {
     getCurrentBranch,
     executeHooksForEvent,
     hasHooksForEvent,
+    resolveRef,
     type IssueStartedPayload,
 } from '@bretwardjames/ghp-core';
 
@@ -33,9 +34,13 @@ export function register(server: McpServer, context: ServerContext): void {
                     .boolean()
                     .optional()
                     .describe('Whether to update the status (default: true)'),
+                hotfix: z
+                    .string()
+                    .optional()
+                    .describe('Branch from a specific tag or commit for hotfix branches. Use get_tags tool to discover available tags.'),
             },
         },
-        async ({ issue, updateStatus = true }) => {
+        async ({ issue, updateStatus = true, hotfix }) => {
             const authenticated = await context.ensureAuthenticated();
             if (!authenticated) {
                 return {
@@ -63,6 +68,24 @@ export function register(server: McpServer, context: ServerContext): void {
             }
 
             try {
+                // Validate hotfix ref if provided
+                let hotfixInfo = '';
+                if (hotfix) {
+                    const resolved = await resolveRef(hotfix);
+                    if (!resolved) {
+                        return {
+                            content: [
+                                {
+                                    type: 'text',
+                                    text: `Error: Hotfix ref "${hotfix}" does not exist. Use get_tags to discover available tags.`,
+                                },
+                            ],
+                            isError: true,
+                        };
+                    }
+                    hotfixInfo = `\n\nHotfix: Branch from "${hotfix}" (${resolved.substring(0, 7)}). Use the CLI to create the branch: ghp start ${issue} --hotfix ${hotfix}`;
+                }
+
                 // Find the issue in projects
                 const item = await context.api.findItemByNumber(repo, issue);
                 if (!item) {
@@ -88,7 +111,7 @@ export function register(server: McpServer, context: ServerContext): void {
                         content: [
                             {
                                 type: 'text',
-                                text: `Started work on issue #${issue} "${item.title}" (status not updated).${blockingWarning}`,
+                                text: `Started work on issue #${issue} "${item.title}" (status not updated).${blockingWarning}${hotfixInfo}`,
                             },
                         ],
                     };
@@ -182,7 +205,7 @@ export function register(server: McpServer, context: ServerContext): void {
                     content: [
                         {
                             type: 'text',
-                            text: `Started work on issue #${issue} "${item.title}" - status set to "${inProgressOption.name}".${blockingWarning}${hookInfo}`,
+                            text: `Started work on issue #${issue} "${item.title}" - status set to "${inProgressOption.name}".${blockingWarning}${hotfixInfo}${hookInfo}`,
                         },
                     ],
                 };

package/src/tools/tags.ts

@@ -0,0 +1,85 @@
+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 { listTags } from '@bretwardjames/ghp-core';
+
+/** Tool metadata for registry */
+export const meta: ToolMeta = {
+    name: 'get_tags',
+    category: 'read',
+};
+
+/**
+ * Registers the get_tags tool.
+ * Lists git tags in the repository, sorted newest first.
+ */
+export function register(server: McpServer, context: ServerContext): void {
+    server.registerTool(
+        'get_tags',
+        {
+            title: 'Get Tags',
+            description:
+                'List git tags in the repository, sorted newest first. Useful for discovering available tags for hotfix branches with start_work.',
+            inputSchema: {
+                limit: z
+                    .number()
+                    .optional()
+                    .describe('Maximum number of tags to return (default: 20)'),
+            },
+        },
+        async ({ limit = 20 }) => {
+            const repo = await context.getRepo();
+            if (!repo) {
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: 'Error: Not in a git repository with a GitHub remote.',
+                        },
+                    ],
+                    isError: true,
+                };
+            }
+
+            try {
+                const tags = await listTags();
+                const limited = tags.slice(0, limit);
+
+                if (limited.length === 0) {
+                    return {
+                        content: [
+                            {
+                                type: 'text',
+                                text: 'No tags found in this repository.',
+                            },
+                        ],
+                    };
+                }
+
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: JSON.stringify({
+                                total: tags.length,
+                                showing: limited.length,
+                                tags: limited,
+                            }, null, 2),
+                        },
+                    ],
+                };
+            } catch (error) {
+                return {
+                    content: [
+                        {
+                            type: 'text',
+                            text: `Error listing tags: ${error instanceof Error ? error.message : String(error)}`,
+                        },
+                    ],
+                    isError: true,
+                };
+            }
+        }
+    );
+}