@chrischall/pickuppatrol-mcp 0.1.1 → 0.2.0

package/dist/tools/_confirm.js

@@ -1,4 +1,4 @@
-import { schemaConfirm, textResult } from '@chrischall/mcp-utils';
+import { minifiedResult, schemaConfirm } from '@chrischall/mcp-utils';
 export { schemaConfirm };
 /**
  * Confirm-gate for a mutating tool (the fleet convention). Without
@@ -12,7 +12,7 @@ export { schemaConfirm };
 export function previewUnlessConfirmed(confirm, action, method, dto, body) {
     if (confirm === true)
         return null;
-    return textResult({
+    return minifiedResult({
         dryRun: true,
         action,
         method,

package/dist/tools/account.js

@@ -1,5 +1,5 @@
 import { z } from 'zod';
-import { textResult, messageOf } from '@chrischall/mcp-utils';
+import { messageOf, minifiedResult } from '@chrischall/mcp-utils';
 import { dayIdToName } from '../dates.js';
 import { VERSION } from '../version.js';
 /** Project a student down to the fields a parent actually asks about. */
@@ -36,7 +36,7 @@ export function registerAccountTools(server, client) {
         annotations: { readOnlyHint: true },
     }, async () => {
         const session = await client.getSession();
-        return textResult({
+        return minifiedResult({
             userId: session.UserId ?? null,
             name: session.DisplayName ?? [session.FirstName, session.LastName].filter(Boolean).join(' '),
             email: session.Email ?? session.PrimaryEmail ?? null,
@@ -55,7 +55,7 @@ export function registerAccountTools(server, client) {
             client.getDefaultPlansReviewNeeded(),
         ]);
         const needsReview = new Map(review.map((r) => [r.StudentId, r.NeedsReview]));
-        return textResult(students.map((student) => ({
+        return minifiedResult(students.map((student) => ({
             ...summarizeStudent(student),
             needsDefaultsReview: needsReview.get(student.StudentId) ?? false,
         })));
@@ -72,7 +72,7 @@ export function registerAccountTools(server, client) {
         },
     }, async ({ student_id, raw }) => {
         const student = await client.getStudent(student_id);
-        return textResult(raw === true ? student : summarizeStudent(student));
+        return minifiedResult(raw === true ? student : summarizeStudent(student));
     });
     server.registerTool('pup_healthcheck', {
         description: 'Verify the configured credentials sign in and the PickUp Patrol API answers. Reports the server version and the students the account can see.',
@@ -80,7 +80,7 @@ export function registerAccountTools(server, client) {
     }, async () => {
         try {
             const session = await client.getSession();
-            return textResult({
+            return minifiedResult({
                 ok: true,
                 version: VERSION,
                 signedInAs: session.Email ?? session.PrimaryEmail ?? session.DisplayName ?? null,
@@ -91,7 +91,7 @@ export function registerAccountTools(server, client) {
             // A healthcheck reports rather than throws: the whole point is to say
             // what is wrong, and an exception here reads to the host as the tool
             // itself being broken.
-            return textResult({ ok: false, version: VERSION, error: messageOf(err) });
+            return minifiedResult({ ok: false, version: VERSION, error: messageOf(err) });
         }
     });
 }

package/dist/tools/defaults.js

@@ -1,11 +1,10 @@
 import { z } from 'zod';
-import { McpToolError, textResult } from '@chrischall/mcp-utils';
+import { McpToolError, minifiedResult } from '@chrischall/mcp-utils';
 import { applyDefaultPlans, clearDefaultPlans } from '../plans.js';
 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
@@ -43,7 +42,7 @@ export function registerDefaultPlanTools(server, client) {
             client.getStudent(student_id),
             client.getDefaultPlansReviewNeeded(),
         ]);
-        return textResult({
+        return minifiedResult({
             studentId: student.StudentId,
             name: [student.FirstName, student.LastName].filter(Boolean).join(' '),
             schoolId: student.SchoolId,
@@ -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
@@ -92,7 +91,7 @@ export function registerDefaultPlanTools(server, client) {
             await client.updateStudent(payload);
             const after = await client.getStudent(student_id);
             const remaining = after.DefaultPlans ?? [];
-            return textResult({
+            return minifiedResult({
                 action: 'Cleared every weekday default',
                 defaultPlans: summarizeDefaultPlans(remaining),
                 verified: remaining.length === 0,
@@ -139,7 +138,7 @@ export function registerDefaultPlanTools(server, client) {
                 return true;
             return !proofsMatch({ transportationId: plan.TransportationId ?? null, note: plan.Note ?? null }, { transportationId: transportation.TransportationId, note: sentNote });
         });
-        return textResult({
+        return minifiedResult({
             action,
             defaultPlans: summarizeDefaultPlans(after.DefaultPlans),
             verified: unchanged.length === 0,
@@ -166,7 +165,7 @@ export function registerDefaultPlanTools(server, client) {
         await client.setDefaultsReviewed(student_id, value);
         const review = await client.getDefaultPlansReviewNeeded();
         const needsReview = review.find((r) => r.StudentId === student_id)?.NeedsReview ?? false;
-        return textResult({
+        return minifiedResult({
             studentId: student_id,
             needsDefaultsReview: needsReview,
             verified: needsReview === !value,

package/dist/tools/plans.js

@@ -1,9 +1,8 @@
 import { z } from 'zod';
-import { McpToolError, textResult } from '@chrischall/mcp-utils';
+import { McpToolError, minifiedResult } 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();
@@ -46,7 +45,7 @@ export function registerPlanTools(server, client) {
             start_date: z.string().describe('YYYY-MM-DD'),
             end_date: z.string().describe('YYYY-MM-DD'),
         },
-    }, async ({ start_date, end_date }) => textResult(await client.getParentPlans(start_date, end_date)));
+    }, async ({ start_date, end_date }) => minifiedResult(await client.getParentPlans(start_date, end_date)));
     server.registerTool('pup_get_plan', {
         description: 'The dismissal plan for one student on one date — the option in force, any note, the early-dismissal time, and whether the date is locked because the cutoff has passed.',
         annotations: { readOnlyHint: true },
@@ -54,7 +53,7 @@ export function registerPlanTools(server, client) {
             student_id: z.number().int().describe('Student id, from pup_list_students'),
             date: z.string().describe('YYYY-MM-DD'),
         },
-    }, async ({ student_id, date }) => textResult(await client.getPlanEdit(date, student_id)));
+    }, async ({ student_id, date }) => minifiedResult(await client.getPlanEdit(date, student_id)));
     server.registerTool('pup_set_plan', {
         description: "Change how a student is dismissed on one or more specific dates, or clear those dates back to the student's weekly default. This changes how a child actually leaves school, so it requires confirm: true; without it you get a dry-run of the exact payload. Read pup_list_transportations first — options differ in whether they require a note, a car number or an early-dismissal time.",
         inputSchema: {
@@ -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
@@ -132,7 +131,7 @@ export function registerPlanTools(server, client) {
             };
         }));
         const failed = verification.filter((v) => !v.verified);
-        return textResult({
+        return minifiedResult({
             action,
             applied: verification,
             verified: failed.length === 0,

package/dist/tools/school.js

@@ -1,5 +1,5 @@
 import { z } from 'zod';
-import { textResult } from '@chrischall/mcp-utils';
+import { minifiedResult } from '@chrischall/mcp-utils';
 /**
  * Project a transportation to its identity plus the rules that decide what a
  * plan using it must carry — an agent choosing an option needs those rules up
@@ -32,7 +32,7 @@ export function registerSchoolTools(server, client) {
     }, async ({ school_id, include_inactive }) => {
         const options = await client.getTransportations(school_id);
         const visible = include_inactive === true ? options : options.filter((o) => o.IsActive !== false);
-        return textResult(visible.map(summarizeTransportation));
+        return minifiedResult(visible.map(summarizeTransportation));
     });
     server.registerTool('pup_get_school', {
         description: 'A school profile together with its per-weekday notify times and plan cutoff times, and the settings that decide whether parents may set plans at all.',
@@ -46,7 +46,7 @@ export function registerSchoolTools(server, client) {
             client.getSchoolNotifyTimes(school_id),
             client.getSchoolSettings(school_id),
         ]);
-        return textResult({ school, notifyTimes, settings });
+        return minifiedResult({ school, notifyTimes, settings });
     });
     server.registerTool('pup_list_non_school_days', {
         description: 'Dates a plan cannot be set for at a school (holidays, closures, weekends), and optionally the dates in a range that already differ from the student defaults.',
@@ -62,10 +62,10 @@ export function registerSchoolTools(server, client) {
     }, async ({ school_id, start_date, end_date }) => {
         const invalidDates = await client.getInvalidPlanDates(school_id);
         if (start_date === undefined || end_date === undefined) {
-            return textResult({ invalidDates });
+            return minifiedResult({ invalidDates });
         }
         const changedDates = await client.getBoldedDates(start_date, end_date);
-        return textResult({ invalidDates, changedDates });
+        return minifiedResult({ invalidDates, changedDates });
     });
     server.registerTool('pup_list_car_numbers', {
         description: 'The car numbers a school has issued to this account, for dismissal options where usesCarNumbers is true.',
@@ -73,5 +73,5 @@ export function registerSchoolTools(server, client) {
         inputSchema: {
             school_id: z.number().int().describe('School id, from pup_list_students'),
         },
-    }, async ({ school_id }) => textResult(await client.getCarNumbers(school_id)));
+    }, async ({ school_id }) => minifiedResult(await client.getCarNumbers(school_id)));
 }

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.2.0";

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.2.0'; // x-release-please-version

package/mint.yaml

@@ -0,0 +1,22 @@
+version: 1
+name: PickUp Patrol
+slug: pickuppatrol
+summary: >-
+  Read and change your children's school dismissal plans in PickUp Patrol —
+  defaults, day-by-day changes and school cutoff times
+env:
+  - name: PICKUPPATROL_USERNAME
+    required: false
+    help: >-
+      The email address you sign in to app.pickuppatrol.net with.
+  - name: PICKUPPATROL_PASSWORD
+    secret: true
+    required: false
+    help: >-
+      Your PickUp Patrol password. Stored by the host, never logged by this
+      server.
+egress:
+  allow:
+    # Only hosts the SERVER process actually fetches. Hosts that appear
+    # solely in a URL this server BUILDS and returns are excluded.
+    - app.pickuppatrol.net

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@chrischall/pickuppatrol-mcp",
-  "version": "0.1.1",
+  "version": "0.2.0",
   "license": "MIT",
   "mcpName": "io.github.chrischall/pickuppatrol-mcp",
   "description": "PickUp Patrol MCP server for Claude — developed and maintained by AI (Claude Code)",
@@ -44,19 +44,20 @@
     ".claude-plugin",
     "skills",
     ".mcp.json",
-    "server.json"
+    "server.json",
+    "mint.yaml"
   ],
   "scripts": {
     "build": "tsc && npm run bundle",
     "bundle": "esbuild src/index.ts --bundle --platform=node --format=esm --external:dotenv --outfile=dist/bundle.js",
     "dev": "node dist/index.js",
     "typecheck": "tsc --noEmit",
-    "test": "vitest run",
-    "test:coverage": "vitest run --coverage",
+    "test": "npm run typecheck && vitest run",
+    "test:coverage": "npm run typecheck && vitest run --coverage",
     "test:watch": "vitest"
   },
   "dependencies": {
-    "@chrischall/mcp-utils": "^0.14.0",
+    "@chrischall/mcp-utils": "^0.23.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.2.0",
   "packages": [
     {
       "registryType": "npm",
       "identifier": "@chrischall/pickuppatrol-mcp",
-      "version": "0.1.1",
+      "version": "0.2.0",
       "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;
-        }
-    };
-}