infinitecampus-mcp 2.8.2 → 2.8.4

package/dist/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 import { dirname, join } from 'path';
 import { fileURLToPath } from 'url';
-import { loadDotenvSafely, runMcp } from '@chrischall/mcp-utils';
+import { loadDotenvSafely, McpToolError, runMcp } from '@chrischall/mcp-utils';
 // quiet load of the local .env (no-throw, silent when the file is absent —
 // e.g. inside an mcpb bundle). Path is resolved next to dist/ so the same
 // `..`/.env that the previous inline dotenv call used still applies.
@@ -35,7 +35,7 @@ const AI_NOTICE = '[infinitecampus-mcp] Developed and maintained by AI (Claude).
 // Defer config errors so the server can still start cleanly when env vars
 // aren't set (e.g. during the host's install-time smoke test, before the
 // user has filled in user_config OR the user hasn't yet signed into their
-// IC portal in the browser). When not configured we register no tools and
+// IC portal in the browser). When not configured every tool still registers and
 // log a clear stderr message — far better than the previous crash loop.
 let account = null;
 let refreshSession;
@@ -51,21 +51,61 @@ catch (e) {
     configError = e;
 }
 // Hand off to mcp-utils' runMcp for the connect + stdio transport + graceful
-// shutdown. We keep the deferred-config-error pattern by deciding the registrar
-// list and banner up here: when configured we wire every
-// register<Domain>Tools(server, client) call; when not, we register no tools so
-// the host's install-time tools/list still succeeds and the user gets an
-// actionable stderr message (banner) instead of a crash loop.
+// shutdown. The registrar list is the SAME either way; only the client and the
+// banner differ. A tool surface that follows the environment is what this
+// avoids — see `unconfiguredClient`.
 const COMMON = {
     name: 'infinitecampus',
-    version: '2.8.2', // x-release-please-version
+    version: '2.8.4', // x-release-please-version
 };
 // Shared with the healthcheck so it can report the SAME state the tools see,
 // including on the unconfigured path where there are no tools at all.
 const healthState = { account, source, configError, client: null };
-if (account) {
-    const client = new ICClient(account, { refreshSession });
-    healthState.client = client;
+/**
+ * Stands in for the client when configuration is missing, so that every tool
+ * registers either way and the failure surfaces WHERE IT IS ACTIONABLE — in the
+ * answer to the tool somebody called, not in a tool list they never saw.
+ *
+ * The surface must not follow the environment. This server is hosted with
+ * `perUserChild` + `authFields`, so mcp-host spawns a principal-less child with
+ * none of the per-user config for prewarm, auto-update, restart and every
+ * admin-portal tool listing — and THAT child's list is what the portal
+ * publishes as the connector's surface. It advertised one tool for a connector
+ * that has twenty.
+ *
+ * This continues the reasoning that put the healthcheck on this path rather
+ * than reversing it: a server that can explain itself beats one that cannot,
+ * and nineteen tools that each name the missing variables explain it better
+ * than nineteen absences do. The healthcheck still reports the same state, and
+ * `healthState.client` deliberately stays null so it reports "unconfigured"
+ * rather than mistaking this stand-in for a live client.
+ */
+function unconfiguredClient(reason) {
+    const fail = () => {
+        throw new McpToolError(`infinitecampus-mcp is not configured: ${reason?.message ?? 'unknown error'}`, {
+            hint: 'Set IC_BASE_URL (your portal URL) and IC_DISTRICT (the app-name path segment), ' +
+                'plus IC_USERNAME and IC_PASSWORD together if you are not signing in through the ' +
+                'browser. Call ic_healthcheck for the resolved state.',
+        });
+    };
+    return new Proxy({}, {
+        get(_target, prop) {
+            // `then` must answer undefined or this object is thenable, and any
+            // `await client` would resolve to something other than the client.
+            // Symbols (toStringTag, node's inspect) are asked for by the runtime
+            // rather than by tool code, so they get the same treatment.
+            if (typeof prop === 'symbol' || prop === 'then')
+                return undefined;
+            return fail;
+        },
+    });
+}
+{
+    const client = account
+        ? new ICClient(account, { refreshSession })
+        : unconfiguredClient(configError);
+    if (account)
+        healthState.client = client;
     const tools = [
         (server) => registerHealthcheckTools(server, healthState),
         registerDistrictTools,
@@ -87,19 +127,10 @@ if (account) {
         registerFeaturesTools,
     ];
     const suffix = source === 'fetchproxy' ? ' [via fetchproxy]' : '';
-    const districtLine = `[infinitecampus-mcp] District: ${account.name} (${account.baseUrl})${suffix}`;
-    await runMcp({ ...COMMON, deps: client, tools, banner: `${districtLine}\n${AI_NOTICE}` });
-}
-else {
-    const notConfigured = `[infinitecampus-mcp] Not configured: ${configError?.message ?? 'unknown error'}\n` +
-        '[infinitecampus-mcp] Server is running with no tools registered. Set the required env vars and reinstall.';
-    // NOT `tools: []` any more. A server that registers nothing explains itself
-    // only on stderr, which a hosted connector never sees — so a misconfigured
-    // server was indistinguishable from a broken one. The healthcheck is the one
-    // tool that must survive this path, because this is the path that needs it.
-    await runMcp({
-        ...COMMON,
-        tools: [(server) => registerHealthcheckTools(server, healthState)],
-        banner: `${notConfigured}\n${AI_NOTICE}`,
-    });
+    const banner = account
+        ? `[infinitecampus-mcp] District: ${account.name} (${account.baseUrl})${suffix}`
+        : `[infinitecampus-mcp] Not configured: ${configError?.message ?? 'unknown error'}\n` +
+            '[infinitecampus-mcp] Every tool is registered and will report this until the ' +
+            'required env vars are set. Call ic_healthcheck for the resolved state.';
+    await runMcp({ ...COMMON, deps: client, tools, banner: `${banner}\n${AI_NOTICE}` });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "infinitecampus-mcp",
-  "version": "2.8.2",
+  "version": "2.8.4",
   "mcpName": "io.github.chrischall/infinitecampus-mcp",
   "description": "Infinite Campus (Campus Parent) MCP server — multi-district read + message/document write",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -45,13 +45,13 @@
     "typecheck": "tsc -p tsconfig.json --noEmit"
   },
   "dependencies": {
-    "@chrischall/mcp-utils": "^0.23.2",
-    "@fetchproxy/bootstrap": "^2.2.0",
-    "@fetchproxy/server": "^2.2.0",
-    "@modelcontextprotocol/sdk": "^1.29.0",
+    "@chrischall/mcp-utils": "^0.26.1",
+    "@fetchproxy/bootstrap": "^3.0.0",
+    "@fetchproxy/server": "^3.0.0",
+    "@modelcontextprotocol/sdk": "^1.30.0",
     "dotenv": "^17.4.2",
     "node-html-parser": "^9.0.0",
-    "zod": "^4.4.3"
+    "zod": "^4.5.4"
   },
   "devDependencies": {
     "@types/node": "^26.0.0",

package/server.json

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