@chrischall/pickuppatrol-mcp 0.1.1 → 0.1.2

package/.claude-plugin/marketplace.json

@@ -6,7 +6,7 @@
   },
   "metadata": {
     "description": "PickUp Patrol school-dismissal tools for Claude Code",
-    "version": "0.1.1"
+    "version": "0.1.2"
   },
   "plugins": [
     {
@@ -14,7 +14,7 @@
       "displayName": "PickUp Patrol",
       "source": "./",
       "description": "Read and change your children's school dismissal plans in PickUp Patrol — defaults, day-by-day changes and school cutoff times — via MCP",
-      "version": "0.1.1",
+      "version": "0.1.2",
       "author": {
         "name": "Chris Chall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "pickuppatrol",
   "displayName": "PickUp Patrol",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "Read and change your children's school dismissal plans in PickUp Patrol — defaults, day-by-day changes and school cutoff times — via MCP",
   "author": {
     "name": "Chris Chall"

package/dist/bundle.js

@@ -31000,53 +31000,6 @@ var StdioServerTransport = class {
   }
 };
 
-// node_modules/@chrischall/mcp-utils/dist/server/index.js
-async function createMcpServer(opts) {
-  const server = new McpServer({ name: opts.name, version: opts.version });
-  if (opts.banner !== void 0) {
-    console.error(opts.banner);
-  }
-  const deps = opts.deps;
-  for (const register of opts.tools) {
-    await register(server, deps);
-  }
-  return server;
-}
-function withGracefulShutdown(server, opts = {}) {
-  const shouldExit = opts.exit ?? true;
-  let shuttingDown = false;
-  const handler = (signal) => {
-    if (shuttingDown)
-      return;
-    shuttingDown = true;
-    void (async () => {
-      try {
-        if (opts.onSignal)
-          await opts.onSignal(signal);
-        await server.close();
-      } catch (err) {
-        console.error(`[mcp-utils] error during graceful shutdown on ${signal}: ${err instanceof Error ? err.message : String(err)}`);
-      } finally {
-        if (shouldExit)
-          process.exit(0);
-      }
-    })();
-  };
-  process.on("SIGINT", () => handler("SIGINT"));
-  process.on("SIGTERM", () => handler("SIGTERM"));
-}
-async function runMcp(opts) {
-  const server = await createMcpServer(opts);
-  const shutdown = opts.shutdown ?? true;
-  if (shutdown !== false) {
-    withGracefulShutdown(server, shutdown === true ? {} : shutdown);
-  }
-  const spec = opts.transport ?? "stdio";
-  const transport = spec === "stdio" ? new StdioServerTransport() : spec;
-  await server.connect(transport);
-  return server;
-}
-
 // node_modules/@chrischall/mcp-utils/dist/errors/index.js
 var McpToolError = class extends Error {
   /** Actionable remediation text, when one applies. */
@@ -31105,6 +31058,75 @@ function errorResult(message) {
   };
 }
 
+// node_modules/@chrischall/mcp-utils/dist/server/index.js
+function hintResultOrRethrow(err) {
+  if (err instanceof McpToolError && err.hint) {
+    return errorResult(`${err.message}
+
+Hint: ${err.hint}`);
+  }
+  throw err;
+}
+function surfaceToolHints(server) {
+  const register = server.registerTool.bind(server);
+  server.registerTool = (name, config2, cb) => register(name, config2, (...args) => {
+    let result;
+    try {
+      result = cb(...args);
+    } catch (err) {
+      return hintResultOrRethrow(err);
+    }
+    return result instanceof Promise ? result.catch(hintResultOrRethrow) : result;
+  });
+}
+async function createMcpServer(opts) {
+  const server = new McpServer({ name: opts.name, version: opts.version });
+  if (opts.surfaceHints !== false)
+    surfaceToolHints(server);
+  if (opts.banner !== void 0) {
+    console.error(opts.banner);
+  }
+  const deps = opts.deps;
+  for (const register of opts.tools) {
+    await register(server, deps);
+  }
+  return server;
+}
+function withGracefulShutdown(server, opts = {}) {
+  const shouldExit = opts.exit ?? true;
+  let shuttingDown = false;
+  const handler = (signal) => {
+    if (shuttingDown)
+      return;
+    shuttingDown = true;
+    void (async () => {
+      try {
+        if (opts.onSignal)
+          await opts.onSignal(signal);
+        await server.close();
+      } catch (err) {
+        console.error(`[mcp-utils] error during graceful shutdown on ${signal}: ${err instanceof Error ? err.message : String(err)}`);
+      } finally {
+        if (shouldExit)
+          process.exit(0);
+      }
+    })();
+  };
+  process.on("SIGINT", () => handler("SIGINT"));
+  process.on("SIGTERM", () => handler("SIGTERM"));
+}
+async function runMcp(opts) {
+  const server = await createMcpServer(opts);
+  const shutdown = opts.shutdown ?? true;
+  if (shutdown !== false) {
+    withGracefulShutdown(server, shutdown === true ? {} : shutdown);
+  }
+  const spec = opts.transport ?? "stdio";
+  const transport = spec === "stdio" ? new StdioServerTransport() : spec;
+  await server.connect(transport);
+  return server;
+}
+
 // node_modules/@chrischall/mcp-utils/dist/config/index.js
 var PLACEHOLDER_RE = /^\$\{[^}]*\}$/;
 function readEnvVar(key, opts = {}) {
@@ -31503,7 +31525,7 @@ var PickUpPatrolClient = class {
 var client = new PickUpPatrolClient();
 
 // src/version.ts
-var VERSION = "0.1.1";
+var VERSION = "0.1.2";
 
 // src/dates.ts
 var WEEKDAY_NAMES = [
@@ -31861,22 +31883,6 @@ function previewUnlessConfirmed(confirm, action, method, dto, body) {
   });
 }
 
-// src/tools/_errors.ts
-function withHints(handler) {
-  return async (args) => {
-    try {
-      return await handler(args);
-    } catch (err) {
-      if (err instanceof McpToolError && err.hint) {
-        return errorResult(`${err.message}
-
-Hint: ${err.hint}`);
-      }
-      throw err;
-    }
-  };
-}
-
 // src/tools/plans.ts
 function proofsMatch(a, b) {
   return a.transportationId === b.transportationId && (a.note ?? "").trim() === (b.note ?? "").trim();
@@ -31935,7 +31941,7 @@ function registerPlanTools(server, client2) {
         confirm: schemaConfirm
       }
     },
-    withHints(async ({ student_id, dates, transportation_id, note, early_dismissal_time, car_number, confirm }) => {
+    (async ({ student_id, dates, transportation_id, note, early_dismissal_time, car_number, confirm }) => {
       const student = await client2.getStudent(student_id);
       const transportation = transportation_id === null ? null : await resolveTransportation(client2, student.SchoolId, transportation_id);
       const plans = buildPlanUpdates({
@@ -32048,7 +32054,7 @@ function registerDefaultPlanTools(server, client2) {
         confirm: schemaConfirm
       }
     },
-    withHints(async ({ student_id, days, transportation_id, note, early_dismissal_time, clear_all, confirm }) => {
+    (async ({ student_id, days, transportation_id, note, early_dismissal_time, clear_all, confirm }) => {
       const student = await client2.getStudent(student_id);
       if (clear_all === true) {
         const payload2 = clearDefaultPlans(student);

package/dist/tools/defaults.js

@@ -5,7 +5,6 @@ import { dayIdToName, nameToDayId } from '../dates.js';
 import { summarizeDefaultPlans } from './account.js';
 import { proofsMatch, resolveTransportation } from './plans.js';
 import { previewUnlessConfirmed, schemaConfirm } from './_confirm.js';
-import { withHints } from './_errors.js';
 /**
  * Accept weekdays as names ("Monday") or ids (1 = Sunday … 7 = Saturday).
  * A name that is not a weekday is rejected rather than coerced — silently
@@ -78,7 +77,7 @@ export function registerDefaultPlanTools(server, client) {
                 .describe('Remove every weekday default instead of setting one (days is ignored)'),
             confirm: schemaConfirm,
         },
-    }, withHints(async ({ student_id, days, transportation_id, note, early_dismissal_time, clear_all, confirm }) => {
+    }, (async ({ student_id, days, transportation_id, note, early_dismissal_time, clear_all, confirm }) => {
         // Read-modify-write: PickUp Patrol has no default-plans endpoint, so the
         // whole student record round-trips. Reading it here (before the confirm
         // gate) is what makes the dry-run show the real payload; it mutates

package/dist/tools/plans.js

@@ -3,7 +3,6 @@ import { McpToolError, textResult } from '@chrischall/mcp-utils';
 import { buildPlanUpdates } from '../plans.js';
 import { weekdayOf } from '../dates.js';
 import { previewUnlessConfirmed, schemaConfirm } from './_confirm.js';
-import { withHints } from './_errors.js';
 /** Compare two proofs, ignoring whitespace the service may normalise off a note. */
 export function proofsMatch(a, b) {
     return a.transportationId === b.transportationId && (a.note ?? '').trim() === (b.note ?? '').trim();
@@ -79,7 +78,7 @@ export function registerPlanTools(server, client) {
                 .describe('Car number, for options where usesCarNumbers is true'),
             confirm: schemaConfirm,
         },
-    }, withHints(async ({ student_id, dates, transportation_id, note, early_dismissal_time, car_number, confirm }) => {
+    }, (async ({ student_id, dates, transportation_id, note, early_dismissal_time, car_number, confirm }) => {
         // The reads below resolve and validate the payload; they mutate nothing.
         // Running them before the confirm gate is deliberate: it makes the
         // dry-run show the exact bytes that would be sent, already checked

package/dist/version.d.ts

@@ -7,4 +7,4 @@
  * annotation, so there is exactly one line to keep in sync and
  * `versionSyncTest` has exactly one line to check.
  */
-export declare const VERSION = "0.1.1";
+export declare const VERSION = "0.1.2";

package/dist/version.js

@@ -7,4 +7,4 @@
  * annotation, so there is exactly one line to keep in sync and
  * `versionSyncTest` has exactly one line to check.
  */
-export const VERSION = '0.1.1'; // x-release-please-version
+export const VERSION = '0.1.2'; // x-release-please-version

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrischall/pickuppatrol-mcp",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "license": "MIT",
   "mcpName": "io.github.chrischall/pickuppatrol-mcp",
   "description": "PickUp Patrol MCP server for Claude — developed and maintained by AI (Claude Code)",
@@ -56,7 +56,7 @@
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@chrischall/mcp-utils": "^0.14.0",
+    "@chrischall/mcp-utils": "^0.15.0",
     "@modelcontextprotocol/sdk": "^1.29.0",
     "dotenv": "^17.4.0",
     "zod": "^4.4.3"

package/server.json

@@ -6,12 +6,12 @@
     "url": "https://github.com/chrischall/pickuppatrol-mcp",
     "source": "github"
   },
-  "version": "0.1.1",
+  "version": "0.1.2",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@chrischall/pickuppatrol-mcp",
-      "version": "0.1.1",
+      "version": "0.1.2",
       "transport": {
         "type": "stdio"
       }

package/dist/tools/_errors.d.ts

@@ -1,14 +0,0 @@
-import type { CallToolResult } from '@modelcontextprotocol/sdk/types.js';
-/**
- * Surface an `McpToolError`'s `hint` to the caller.
- *
- * The MCP tool boundary turns a thrown error into a result carrying only
- * `message`, so a `hint` — which is where the actionable half lives here ("the
- * available options are …", "the school describes the note as …") — is
- * otherwise dropped on the floor. Wrapping the handler folds it into the text
- * the caller actually sees.
- *
- * Only `McpToolError` is handled: an unexpected error keeps propagating, so a
- * genuine bug still reads as one instead of being flattened into advice.
- */
-export declare function withHints<A>(handler: (args: A) => Promise<CallToolResult>): (args: A) => Promise<CallToolResult>;

package/dist/tools/_errors.js

@@ -1,26 +0,0 @@
-import { McpToolError, errorResult } from '@chrischall/mcp-utils';
-/**
- * Surface an `McpToolError`'s `hint` to the caller.
- *
- * The MCP tool boundary turns a thrown error into a result carrying only
- * `message`, so a `hint` — which is where the actionable half lives here ("the
- * available options are …", "the school describes the note as …") — is
- * otherwise dropped on the floor. Wrapping the handler folds it into the text
- * the caller actually sees.
- *
- * Only `McpToolError` is handled: an unexpected error keeps propagating, so a
- * genuine bug still reads as one instead of being flattened into advice.
- */
-export function withHints(handler) {
-    return async (args) => {
-        try {
-            return await handler(args);
-        }
-        catch (err) {
-            if (err instanceof McpToolError && err.hint) {
-                return errorResult(`${err.message}\n\nHint: ${err.hint}`);
-            }
-            throw err;
-        }
-    };
-}