@cyanheads/git-mcp-server 2.1.0 → 2.1.2

package/dist/mcp-server/tools/gitWrapupInstructions/logic.js

@@ -1,13 +1,16 @@
-import { z } from 'zod';
-import { logger } from '../../../utils/index.js'; // Added logger
-import { getGitStatus } from '../gitStatus/logic.js'; // Corrected path
+import { z } from "zod";
+import { logger } from "../../../utils/index.js"; // Added logger
+import { getGitStatus } from "../gitStatus/logic.js"; // Corrected path
 // Define the input schema
 export const GitWrapupInstructionsInputSchema = z.object({
-    acknowledgement: z.enum(['Y', 'y', 'Yes', 'yes'], {
-        required_error: 'Acknowledgement is required.',
+    acknowledgement: z.enum(["Y", "y", "Yes", "yes"], {
+        required_error: "Acknowledgement is required.",
         description: 'Acknowledgement that you have permission (implicit allowed, explicit preferred) from the user to initiate this tool. Must be "Y" or "Yes" (case-insensitive).',
     }),
-    updateAgentMetaFiles: z.enum(['Y', 'y', 'Yes', 'yes']).optional().describe("If set to 'Y' or 'Yes', include an extra instruction to review and update agent-specific meta files like .clinerules or claude.md if present. Only use this if the user explicitly requested it."),
+    updateAgentMetaFiles: z
+        .enum(["Y", "y", "Yes", "yes"])
+        .optional()
+        .describe("If set to 'Y' or 'Yes', include an extra instruction to review and update agent-specific meta files like .clinerules or claude.md if present. Only use this if the user explicitly requested it."),
 });
 // The predefined instructions string.
 const WRAPUP_INSTRUCTIONS = `
@@ -17,6 +20,8 @@ Perform all actions for our git wrapup workflow:
 3. Update the CHANGELOG with concise, descriptive entries detailing all modifications, clearly indicating their purpose (e.g., bug fix, feature implementation, refactoring). Include specific metrics or identifiers where applicable, such as issue numbers or pull request links, to provide context and traceability for each change. This will help maintain a clear history of changes and their impacts on the project.
 4. Proceed to commit all changes; based on your review of the git_diff and readme, group these changes into logical, atomic commits, each accompanied by a clear and descriptive message adhering to Conventional Commits standards (e.g. "docs(readme): updated readme to include xyz."). Note the 'git_commit' tool allows you to also stage the files while commiting. Ensure commit messages accurately convey the scope and impact of the changes, incorporating specific metrics or identifiers where applicable.
 Note: Be sure to set 'git_set_working_dir' if not already set.
+
+Instructions: Now write a concise list of what you must do to complete the git wrapup workflow, then perform all actions. Do not push unless requested.
 `;
 /**
  * Core logic for the git_wrapup_instructions tool.
@@ -30,8 +35,11 @@ export async function getWrapupInstructions(input,
 // The context is now expected to be enhanced by the registration layer
 // to include session-specific methods like getWorkingDirectory.
 context) {
+    const operation = "getWrapupInstructions";
+    logger.debug(`Executing ${operation}`, { ...context, input });
     let finalInstructions = WRAPUP_INSTRUCTIONS;
-    if (input.updateAgentMetaFiles && ['Y', 'y', 'Yes', 'yes'].includes(input.updateAgentMetaFiles)) {
+    if (input.updateAgentMetaFiles &&
+        ["Y", "y", "Yes", "yes"].includes(input.updateAgentMetaFiles)) {
         finalInstructions += ` Extra request: review and update if needed the .clinerules and claude.md files if present.`;
     }
     let statusResult = undefined;
@@ -42,16 +50,20 @@ context) {
             // The `getGitStatus` function expects `path` and a context with `getWorkingDirectory`.
             // Passing `path: '.'` signals `getGitStatus` to use the working directory from the context.
             // The `registration.ts` for this tool will be responsible for ensuring `context.getWorkingDirectory` is correctly supplied.
-            statusResult = await getGitStatus({ path: '.' }, context);
+            statusResult = await getGitStatus({ path: "." }, context);
         }
         catch (error) {
-            logger.warning(`Failed to get git status while generating wrapup instructions (working dir: ${workingDir}). Tool will proceed without it.`, { ...context, tool: 'gitWrapupInstructions', originalError: error.message });
+            logger.warning(`Failed to get git status while generating wrapup instructions (working dir: ${workingDir}). Tool will proceed without it.`, {
+                ...context,
+                tool: "gitWrapupInstructions",
+                originalError: error.message,
+            });
             statusError = error instanceof Error ? error.message : String(error);
         }
     }
     else {
-        logger.info('No working directory set for session, skipping git status for wrapup instructions.', { ...context, tool: 'gitWrapupInstructions' });
-        statusError = 'No working directory set for session, git status skipped.';
+        logger.info("No working directory set for session, skipping git status for wrapup instructions.", { ...context, tool: "gitWrapupInstructions" });
+        statusError = "No working directory set for session, git status skipped.";
     }
     return {
         instructions: finalInstructions,

package/dist/mcp-server/tools/gitWrapupInstructions/registration.js

@@ -1,6 +1,6 @@
-import { BaseErrorCode } from '../../../types-global/errors.js';
-import { ErrorHandler, logger, requestContextService } from '../../../utils/index.js';
-import { getWrapupInstructions, GitWrapupInstructionsInputSchema, } from './logic.js';
+import { BaseErrorCode } from "../../../types-global/errors.js";
+import { ErrorHandler, logger, requestContextService, } from "../../../utils/index.js";
+import { getWrapupInstructions, GitWrapupInstructionsInputSchema, } from "./logic.js";
 let _getWorkingDirectory;
 let _getSessionId;
 /**
@@ -12,10 +12,10 @@ let _getSessionId;
 export function initializeGitWrapupInstructionsStateAccessors(getWdFn, getSidFn) {
     _getWorkingDirectory = getWdFn;
     _getSessionId = getSidFn;
-    logger.info('State accessors initialized for git_wrapup_instructions tool registration.');
+    logger.info("State accessors initialized for git_wrapup_instructions tool registration.");
 }
-const TOOL_NAME = 'git_wrapup_instructions';
-const TOOL_DESCRIPTION = 'Provides a standard Git wrap-up workflow. This involves reviewing changes with `git_diff`, updating documentation (README, CHANGELOG), and making logical, descriptive commits using the `git_commit` tool. The tool\'s response also includes the current `git status` output.';
+const TOOL_NAME = "git_wrapup_instructions";
+const TOOL_DESCRIPTION = "Provides a standard Git wrap-up workflow. This involves reviewing changes with `git_diff`, updating documentation (README, CHANGELOG), and making logical, descriptive commits using the `git_commit` tool. The tool's response also includes the current `git status` output. You should set the working directory using `git_set_working_dir` before running this tool.";
 /**
  * Registers the git_wrapup_instructions tool with the MCP server.
  *
@@ -25,14 +25,16 @@ const TOOL_DESCRIPTION = 'Provides a standard Git wrap-up workflow. This involve
  */
 export const registerGitWrapupInstructionsTool = async (server) => {
     if (!_getWorkingDirectory || !_getSessionId) {
-        throw new Error('State accessors for git_wrapup_instructions must be initialized before registration.');
+        throw new Error("State accessors for git_wrapup_instructions must be initialized before registration.");
     }
-    const operation = 'registerGitWrapupInstructionsTool';
+    const operation = "registerGitWrapupInstructionsTool";
     // Context for the registration operation itself
-    const registrationOpContext = requestContextService.createRequestContext({ operation });
+    const registrationOpContext = requestContextService.createRequestContext({
+        operation,
+    });
     await ErrorHandler.tryCatch(async () => {
         server.tool(TOOL_NAME, TOOL_DESCRIPTION, GitWrapupInstructionsInputSchema.shape, async (validatedArgs, callContext) => {
-            const toolOperation = 'tool:git_wrapup_instructions';
+            const toolOperation = "tool:git_wrapup_instructions";
             // Create a base RequestContext for this specific tool call,
             // potentially linking to the callContext provided by the McpServer.
             // Pass callContext directly; createRequestContext will handle it appropriately
@@ -60,9 +62,9 @@ export const registerGitWrapupInstructionsTool = async (server) => {
                 // Call the core logic function with validated arguments and the prepared logicContext.
                 const result = await getWrapupInstructions(validatedArgs, logicContext);
                 const resultContent = {
-                    type: 'text',
+                    type: "text",
                     text: JSON.stringify(result, null, 2),
-                    contentType: 'application/json',
+                    contentType: "application/json",
                 };
                 logger.info(`Tool ${TOOL_NAME} executed successfully, returning JSON`, logicContext);
                 return { content: [resultContent] };