infinitecampus-mcp 3.1.9 → 3.2.0

package/.claude-plugin/marketplace.json

@@ -7,7 +7,7 @@
   },
   "metadata": {
     "description": "MCP server for Infinite Campus (Campus Parent) — grades, attendance, assignments, messages, documents",
-    "version": "3.1.9"
+    "version": "3.2.0"
   },
   "plugins": [
     {
@@ -15,7 +15,7 @@
       "displayName": "Infinite Campus",
       "source": "./",
       "description": "Infinite Campus (Campus Parent) MCP server for Claude — grades, attendance, assignments, messages, and documents via natural language",
-      "version": "3.1.9",
+      "version": "3.2.0",
       "author": {
         "name": "Chris Hall"
       },

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "infinitecampus-mcp",
   "displayName": "Infinite Campus",
-  "version": "3.1.9",
+  "version": "3.2.0",
   "description": "Infinite Campus (Campus Parent) MCP server for Claude — grades, attendance, assignments, messages, and documents",
   "author": {
     "name": "Chris Hall",

package/dist/bundle.js

@@ -27078,7 +27078,7 @@ function declaredDialect(schema, remedy) {
   throw new Error(`JSON Schema declares an unsupported dialect ("$schema": "${schema.$schema.slice(0, 200)}"). The default validator supports JSON Schema 2020-12, 2019-09, draft-07, and draft-06; ${remedy}`);
 }
 
-// node_modules/@modelcontextprotocol/core/dist/auth-BNDyLTqp.mjs
+// node_modules/@modelcontextprotocol/server/node_modules/@modelcontextprotocol/core/dist/auth-BNDyLTqp.mjs
 var z = __toESM(require_v4(), 1);
 var LATEST_PROTOCOL_VERSION = "2025-11-25";
 var SUPPORTED_PROTOCOL_VERSIONS = [
@@ -68700,7 +68700,7 @@ function resolveDownloadDir(env = process.env) {
 // package.json
 var package_default = {
   name: "infinitecampus-mcp",
-  version: "3.1.9",
+  version: "3.2.0",
   mcpName: "io.github.chrischall/infinitecampus-mcp",
   description: "Infinite Campus (Campus Parent) MCP server \u2014 multi-district, read-only (plus local document download)",
   author: "Claude Code (AI) <https://www.anthropic.com/claude>",
@@ -69427,6 +69427,10 @@ var ICClient = class {
   // linkedDistrictName → primaryDistrictName
   primaryName;
   featuresCache = /* @__PURE__ */ new Map();
+  /** Campus version per district, read once. A failed read is dropped, to be retried after VERSION_RETRY_MS. */
+  versions = /* @__PURE__ */ new Map();
+  /** When each district's last version read failed — see VERSION_RETRY_MS. */
+  versionFailedAt = /* @__PURE__ */ new Map();
   /**
    * True when the primary account has empty creds (fetchproxy mode). The
    * client uses pre-loaded cookies in place of the `verify.jsp` POST and
@@ -69597,7 +69601,37 @@ var ICClient = class {
   async request(district, path, opts = {}) {
     const account2 = await this.resolveAccount(district);
     await this.managers.get(account2.name).ensure();
-    return this.doRequest(account2, path, opts);
+    try {
+      return await this.doRequest(account2, path, opts);
+    } catch (e) {
+      if (e instanceof PortalUnreachableError) e.setCampusVersion(await this.campusVersion(account2.name));
+      throw e;
+    }
+  }
+  /**
+   * The district's Campus version (e.g. "2635.16"), or null when it cannot be
+   * read. DIAGNOSTIC ONLY — nothing chooses behaviour by it. The only place IC
+   * states it is the parent login page footer ("Version:Campus-2635.16"),
+   * which needs no session, so this works in fetchproxy mode too and never
+   * touches a district's cookies.
+   */
+  campusVersion(district) {
+    const account2 = this.accounts.get(district);
+    if (!account2) return Promise.resolve(null);
+    const failedAt = this.versionFailedAt.get(account2.name);
+    if (failedAt !== void 0 && Date.now() - failedAt < VERSION_RETRY_MS) return Promise.resolve(null);
+    let pending = this.versions.get(account2.name);
+    if (!pending) {
+      pending = fetchCampusVersion(account2).then((v) => {
+        if (v === null) {
+          this.versions.delete(account2.name);
+          this.versionFailedAt.set(account2.name, Date.now());
+        }
+        return v;
+      });
+      this.versions.set(account2.name, pending);
+    }
+    return pending;
   }
   /**
    * The account for `district`, running CUPS discovery once on a miss.
@@ -69973,7 +70007,30 @@ var PortalUnreachableError = class extends Error {
   }
   district;
   status;
+  /** The district's Campus version, when it could be read — for bug reports. */
+  campusVersion;
+  setCampusVersion(version2) {
+    if (version2 === null) return;
+    this.campusVersion = version2;
+    this.message += ` [Campus ${version2}]`;
+  }
 };
+function parseCampusVersion(html) {
+  return /Version:\s*Campus-([0-9][\w.-]*)/.exec(html)?.[1] ?? null;
+}
+async function fetchCampusVersion(account2) {
+  try {
+    const res = await icFetch(
+      `${account2.baseUrl}/campus/portal/parents/${encodeURIComponent(account2.district)}.jsp`,
+      {},
+      VERSION_TIMEOUT_MS
+    );
+    if (!res.ok) return null;
+    return parseCampusVersion(await res.text());
+  } catch {
+    return null;
+  }
+}
 var SessionExpiredError = class extends Error {
   constructor(district) {
     super(`Session expired for district '${district}' after re-login retry`);
@@ -69995,6 +70052,8 @@ var DocumentOriginNotAllowedError = class extends Error {
   allowedOrigin;
 };
 var REQUEST_TIMEOUT_MS = 3e4;
+var VERSION_TIMEOUT_MS = 5e3;
+var VERSION_RETRY_MS = 5 * 60 * 1e3;
 var DOWNLOAD_TIMEOUT_MS = 12e4;
 var MAX_DOWNLOAD_BYTES = 50 * 1024 * 1024;
 function icFetch(url4, init = {}, timeoutMs = REQUEST_TIMEOUT_MS) {
@@ -70119,12 +70178,19 @@ function registerHealthcheckTools(server, state) {
       const detail = {
         base_url: state.account.baseUrl,
         configured_district: state.account.district,
-        auth_source: state.source ?? "unknown"
+        auth_source: state.source ?? "unknown",
+        // Read off the login page, so it needs no session and is reported
+        // even when discovery cannot run. Diagnostic only (#253).
+        campus_version: await state.client.campusVersion(state.account.name)
       };
       try {
         await state.client.ensureDiscovery();
         const districts = state.client.listDistricts();
-        detail.districts = districts.map((d) => ({ name: d.name, linked: d.linked }));
+        detail.districts = await Promise.all(districts.map(async (d) => ({
+          name: d.name,
+          linked: d.linked,
+          campus_version: await state.client.campusVersion(d.name)
+        })));
         const linked = districts.filter((d) => d.linked).map((d) => d.name);
         detail.configured_district_is_linked = districts.some(
           (d) => d.name === state.account.district && d.linked
@@ -70637,6 +70703,61 @@ function registerDocumentTools(server, client) {
 }
 
 // src/tools/calendar.ts
+function isRouteFailure(e) {
+  return e instanceof PortalUnreachableError || is404(e);
+}
+var preferDayRoute = /* @__PURE__ */ new WeakMap();
+async function fetchDayRoute(client, district, calendarID, route) {
+  if (route === "instructionalDay") {
+    return toArray(await client.request(
+      district,
+      `/campus/resources/calendar/instructionalDay?calendarID=${calendarID}`
+    ));
+  }
+  const all = toArray(await client.request(
+    district,
+    `/campus/resources/calendar/day?calendarID=${calendarID}`
+  ));
+  return all.filter((d) => d.isInstructionalDay);
+}
+async function fetchInstructionalDays(client, district, calendarID) {
+  let preferred = preferDayRoute.get(client);
+  if (!preferred) preferDayRoute.set(client, preferred = /* @__PURE__ */ new Set());
+  const routes = preferred.has(district) ? ["day", "instructionalDay"] : ["instructionalDay", "day"];
+  for (const route of routes) {
+    try {
+      const days = await fetchDayRoute(client, district, calendarID, route);
+      if (route === "day") preferred.add(district);
+      else preferred.delete(district);
+      return days;
+    } catch (e) {
+      if (!isRouteFailure(e)) throw e;
+    }
+  }
+  return null;
+}
+async function fetchDayEvents(client, district, personID) {
+  let groups;
+  try {
+    groups = toArray(await client.request(
+      district,
+      `/campus/resources/portal/dayEvent/byEnrollment?personID=${encodeURIComponent(personID)}`
+    ));
+  } catch (e) {
+    if (!isRouteFailure(e)) throw e;
+    groups = [];
+  }
+  const byCalendar = /* @__PURE__ */ new Map();
+  for (const ev of groups.flatMap((g) => toArray(g.events))) {
+    let byDate = byCalendar.get(ev.calendarID);
+    if (!byDate) byCalendar.set(ev.calendarID, byDate = /* @__PURE__ */ new Map());
+    byDate.set(ev.date, [...byDate.get(ev.date) ?? [], ev]);
+  }
+  return byCalendar;
+}
+function joinLabels(labels) {
+  return [...new Set(labels.filter((l) => !!l))].join("; ");
+}
 var argsSchema8 = external_exports.object({
   district: external_exports.string(),
   studentId: external_exports.string().describe("Student personID from ic_list_students"),
@@ -70645,7 +70766,7 @@ var argsSchema8 = external_exports.object({
 });
 function registerCalendarTools(server, client) {
   server.registerTool("ic_list_school_days", {
-    description: "List a student's school days (instructional calendar) grouped by term. Returns one entry per enrollment, with term boundaries (Q1-Q4 start/end dates) and the school days inside each term \u2014 including comments like 'Teacher Workday' or 'Spring Break'. Use since/until to narrow the range.",
+    description: "List a student's school days (instructional calendar) grouped by term. Returns one entry per enrollment, with term boundaries (Q1-Q4 start/end dates) and the school days inside each term \u2014 a day carries comments when the calendar labels it (e.g. 'Early Release Day'). Each enrollment also lists its labelled daysOff (e.g. 'Holiday', 'Teacher Work Day', 'Spring Break'), including breaks that fall between terms. Use since/until to narrow the range. An enrollment whose day list the district won't serve comes back with warning 'SchoolDaysUnavailable' (plus campusVersion when known), its terms and days off, and no school days.",
     annotations: { readOnlyHint: true, openWorldHint: true },
     inputSchema: external_exports.object({ ...argsSchema8.shape, view: viewArg() })
   }, async (rawArgs) => {
@@ -70653,18 +70774,18 @@ function registerCalendarTools(server, client) {
     const student = await findStudent(client, args.district, args.studentId);
     if (!student) return studentNotFound(args.studentId);
     const result = [];
-    for (const enr of student.enrollments ?? []) {
-      const [termsRaw, daysRaw] = await Promise.all([
+    const enrollments = student.enrollments ?? [];
+    const events = enrollments.length > 0 ? await fetchDayEvents(client, args.district, args.studentId) : /* @__PURE__ */ new Map();
+    for (const enr of enrollments) {
+      const [termsRaw, days] = await Promise.all([
         client.request(args.district, `/campus/resources/term?structureID=${enr.structureID}`),
-        client.request(args.district, `/campus/resources/calendar/instructionalDay?calendarID=${enr.calendarID}`)
+        fetchInstructionalDays(client, args.district, enr.calendarID)
       ]);
       const terms = toArray(termsRaw);
-      const days = toArray(daysRaw);
-      const filteredDays = days.filter((d) => {
-        if (args.since && d.date < args.since) return false;
-        if (args.until && d.date > args.until) return false;
-        return true;
-      });
+      const inRange2 = (date5) => !(args.since && date5 < args.since) && !(args.until && date5 > args.until);
+      const filteredDays = (days ?? []).filter((d) => inRange2(d.date));
+      const calendarEvents = events.get(enr.calendarID) ?? /* @__PURE__ */ new Map();
+      const daysOff = [...calendarEvents.entries()].filter(([date5, evs]) => inRange2(date5) && evs.some((e) => e.instruction === false)).map(([date5, evs]) => ({ date: date5, comments: joinLabels(evs.map((e) => e.name)) })).sort((a, b) => a.date.localeCompare(b.date));
       const enrollmentTerms = terms.filter((t) => t.structureID === enr.structureID).sort((a, b) => a.seq - b.seq);
       const trimmedTerms = enrollmentTerms.map((t) => ({
         termID: t.termID,
@@ -70673,16 +70794,21 @@ function registerCalendarTools(server, client) {
         endDate: t.endDate,
         days: filteredDays.filter((d) => d.date >= t.startDate && d.date <= t.endDate).map((d) => {
           const out = { date: d.date, requiresAttendance: d.requiresAttendance };
-          if (d.comments) out.comments = d.comments;
+          const comments = joinLabels([d.comments, ...(calendarEvents.get(d.date) ?? []).map((e) => e.name)]);
+          if (comments) out.comments = comments;
           return out;
         })
-      })).filter((t) => t.days.length > 0 || !args.since && !args.until);
+      })).filter((t) => days === null ? !(args.since && t.endDate < args.since) && !(args.until && t.startDate > args.until) : t.days.length > 0 || !args.since && !args.until);
+      const campusVersion = days === null ? await client.campusVersion(args.district) : null;
       result.push({
         enrollmentID: enr.enrollmentID,
         calendarID: enr.calendarID,
         structureID: enr.structureID,
         calendarName: enr.calendarName,
-        terms: trimmedTerms
+        ...days === null ? { warning: "SchoolDaysUnavailable" } : {},
+        ...campusVersion ? { campusVersion } : {},
+        terms: trimmedTerms,
+        ...daysOff.length > 0 ? { daysOff } : {}
       });
     }
     return viewResponse(rawArgs.view, result);
@@ -71034,7 +71160,7 @@ try {
 }
 var COMMON = {
   name: "infinitecampus",
-  version: "3.1.9"
+  version: "3.2.0"
   // x-release-please-version
 };
 var healthState = { account, source, configError, client: null };

package/dist/client.js

@@ -32,6 +32,10 @@ export class ICClient {
     linkedTo = new Map(); // linkedDistrictName → primaryDistrictName
     primaryName;
     featuresCache = new Map();
+    /** Campus version per district, read once. A failed read is dropped, to be retried after VERSION_RETRY_MS. */
+    versions = new Map();
+    /** When each district's last version read failed — see VERSION_RETRY_MS. */
+    versionFailedAt = new Map();
     /**
      * True when the primary account has empty creds (fetchproxy mode). The
      * client uses pre-loaded cookies in place of the `verify.jsp` POST and
@@ -248,7 +252,43 @@ export class ICClient {
     async request(district, path, opts = {}) {
         const account = await this.resolveAccount(district);
         await this.managers.get(account.name).ensure();
-        return this.doRequest(account, path, opts);
+        try {
+            return await this.doRequest(account, path, opts);
+        }
+        catch (e) {
+            // A 5xx is the failure most likely to be a Campus release changing
+            // behaviour under us (#253), so it carries the version into the report.
+            if (e instanceof PortalUnreachableError)
+                e.setCampusVersion(await this.campusVersion(account.name));
+            throw e;
+        }
+    }
+    /**
+     * The district's Campus version (e.g. "2635.16"), or null when it cannot be
+     * read. DIAGNOSTIC ONLY — nothing chooses behaviour by it. The only place IC
+     * states it is the parent login page footer ("Version:Campus-2635.16"),
+     * which needs no session, so this works in fetchproxy mode too and never
+     * touches a district's cookies.
+     */
+    campusVersion(district) {
+        const account = this.accounts.get(district);
+        if (!account)
+            return Promise.resolve(null);
+        const failedAt = this.versionFailedAt.get(account.name);
+        if (failedAt !== undefined && Date.now() - failedAt < VERSION_RETRY_MS)
+            return Promise.resolve(null);
+        let pending = this.versions.get(account.name);
+        if (!pending) {
+            pending = fetchCampusVersion(account).then((v) => {
+                if (v === null) {
+                    this.versions.delete(account.name);
+                    this.versionFailedAt.set(account.name, Date.now());
+                }
+                return v;
+            });
+            this.versions.set(account.name, pending);
+        }
+        return pending;
     }
     /**
      * The account for `district`, running CUPS discovery once on a miss.
@@ -792,12 +832,35 @@ export class AuthFailedError extends Error {
 export class PortalUnreachableError extends Error {
     district;
     status;
+    /** The district's Campus version, when it could be read — for bug reports. */
+    campusVersion;
     constructor(district, status) {
         super(`Portal unreachable for district '${district}' (status ${status})`);
         this.district = district;
         this.status = status;
         this.name = 'PortalUnreachableError';
     }
+    setCampusVersion(version) {
+        if (version === null)
+            return;
+        this.campusVersion = version;
+        this.message += ` [Campus ${version}]`;
+    }
+}
+/** The version from a Campus login page footer ("Version:Campus-2635.16"), or null. */
+export function parseCampusVersion(html) {
+    return /Version:\s*Campus-([0-9][\w.-]*)/.exec(html)?.[1] ?? null;
+}
+async function fetchCampusVersion(account) {
+    try {
+        const res = await icFetch(`${account.baseUrl}/campus/portal/parents/${encodeURIComponent(account.district)}.jsp`, {}, VERSION_TIMEOUT_MS);
+        if (!res.ok)
+            return null;
+        return parseCampusVersion(await res.text());
+    }
+    catch {
+        return null;
+    }
 }
 export class SessionExpiredError extends Error {
     district;
@@ -821,6 +884,13 @@ export class DocumentOriginNotAllowedError extends Error {
 }
 /** Per-request ceiling for every IC call except a document download. */
 export const REQUEST_TIMEOUT_MS = 30_000;
+/**
+ * The Campus version read rides on failures (request() reads it on every 5xx),
+ * so it must not add the full request timeout to a call that already failed.
+ */
+export const VERSION_TIMEOUT_MS = 5_000;
+/** How long a failed version read is remembered before the page is read again. */
+export const VERSION_RETRY_MS = 5 * 60 * 1000;
 /** Ceiling for a document download, which moves a whole file. */
 export const DOWNLOAD_TIMEOUT_MS = 120_000;
 /**

package/dist/index.js

@@ -56,7 +56,7 @@ catch (e) {
 // avoids — see `unconfiguredClient`.
 const COMMON = {
     name: 'infinitecampus',
-    version: '3.1.9', // x-release-please-version
+    version: '3.2.0', // 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.

package/dist/tools/calendar.js

@@ -1,6 +1,84 @@
 import { viewArg, viewResponse } from '../view.js';
 import { z } from 'zod';
-import { findStudent, studentNotFound, toArray } from './_shared.js';
+import { PortalUnreachableError } from '../client.js';
+import { findStudent, is404, studentNotFound, toArray } from './_shared.js';
+/** The portal answered, but not with this resource: worth trying another route. */
+function isRouteFailure(e) {
+    return e instanceof PortalUnreachableError || is404(e);
+}
+/**
+ * Districts (per client) that last answered on calendar/day rather than
+ * instructionalDay. Asking the broken route first on every call would cost one
+ * guaranteed-failing request per enrollment. It is a preference, not a
+ * verdict: if calendar/day stops answering, instructionalDay is tried again.
+ */
+const preferDayRoute = new WeakMap();
+async function fetchDayRoute(client, district, calendarID, route) {
+    if (route === 'instructionalDay') {
+        return toArray(await client.request(district, `/campus/resources/calendar/instructionalDay?calendarID=${calendarID}`));
+    }
+    const all = toArray(await client.request(district, `/campus/resources/calendar/day?calendarID=${calendarID}`));
+    return all.filter((d) => d.isInstructionalDay);
+}
+/**
+ * The calendar's instructional days, or null when no route to them answers.
+ *
+ * Some districts now 500 on calendar/instructionalDay: Campus matches it to
+ * calendar/{calendarID} and reads "instructionalDay" as the ID (#253).
+ * calendar/day returns the same days plus the non-instructional ones, flagged
+ * by isInstructionalDay. Session and auth errors are not route failures and
+ * propagate, so a dead session never reads as "days unknown".
+ */
+async function fetchInstructionalDays(client, district, calendarID) {
+    let preferred = preferDayRoute.get(client);
+    if (!preferred)
+        preferDayRoute.set(client, (preferred = new Set()));
+    const routes = preferred.has(district) ? ['day', 'instructionalDay'] : ['instructionalDay', 'day'];
+    for (const route of routes) {
+        try {
+            const days = await fetchDayRoute(client, district, calendarID, route);
+            if (route === 'day')
+                preferred.add(district);
+            else
+                preferred.delete(district);
+            return days;
+        }
+        catch (e) {
+            if (!isRouteFailure(e))
+                throw e;
+        }
+    }
+    return null;
+}
+/**
+ * Event labels by calendarID → date, from portal/dayEvent/byEnrollment. Neither
+ * day route carries `comments` any more, so this is where "Holiday" or
+ * "Teacher Work Day" lives now. Labels are a nicety: a route failure costs only
+ * the labels (an empty map), but a session error still propagates.
+ */
+async function fetchDayEvents(client, district, personID) {
+    let groups;
+    try {
+        groups = toArray(await client.request(district, `/campus/resources/portal/dayEvent/byEnrollment?personID=${encodeURIComponent(personID)}`));
+    }
+    catch (e) {
+        if (!isRouteFailure(e))
+            throw e;
+        groups = [];
+    }
+    const byCalendar = new Map();
+    for (const ev of groups.flatMap((g) => toArray(g.events))) {
+        let byDate = byCalendar.get(ev.calendarID);
+        if (!byDate)
+            byCalendar.set(ev.calendarID, (byDate = new Map()));
+        byDate.set(ev.date, [...(byDate.get(ev.date) ?? []), ev]);
+    }
+    return byCalendar;
+}
+/** Distinct, non-empty labels joined the way IC's old `comments` read. */
+function joinLabels(labels) {
+    return [...new Set(labels.filter((l) => !!l))].join('; ');
+}
 const argsSchema = z.object({
     district: z.string(),
     studentId: z.string().describe('Student personID from ic_list_students'),
@@ -9,7 +87,7 @@ const argsSchema = z.object({
 });
 export function registerCalendarTools(server, client) {
     server.registerTool('ic_list_school_days', {
-        description: "List a student's school days (instructional calendar) grouped by term. Returns one entry per enrollment, with term boundaries (Q1-Q4 start/end dates) and the school days inside each term — including comments like 'Teacher Workday' or 'Spring Break'. Use since/until to narrow the range.",
+        description: "List a student's school days (instructional calendar) grouped by term. Returns one entry per enrollment, with term boundaries (Q1-Q4 start/end dates) and the school days inside each term — a day carries comments when the calendar labels it (e.g. 'Early Release Day'). Each enrollment also lists its labelled daysOff (e.g. 'Holiday', 'Teacher Work Day', 'Spring Break'), including breaks that fall between terms. Use since/until to narrow the range. An enrollment whose day list the district won't serve comes back with warning 'SchoolDaysUnavailable' (plus campusVersion when known), its terms and days off, and no school days.",
         annotations: { readOnlyHint: true, openWorldHint: true },
         inputSchema: z.object({ ...argsSchema.shape, view: viewArg() }),
     }, async (rawArgs) => {
@@ -19,22 +97,27 @@ export function registerCalendarTools(server, client) {
         if (!student)
             return studentNotFound(args.studentId);
         const result = [];
+        const enrollments = student.enrollments ?? [];
+        // One call covers every enrollment; skip it when there are none.
+        const events = enrollments.length > 0
+            ? await fetchDayEvents(client, args.district, args.studentId)
+            : new Map();
         // 2. For each enrollment, fetch terms + instructional days
-        for (const enr of student.enrollments ?? []) {
-            const [termsRaw, daysRaw] = await Promise.all([
+        for (const enr of enrollments) {
+            const [termsRaw, days] = await Promise.all([
                 client.request(args.district, `/campus/resources/term?structureID=${enr.structureID}`),
-                client.request(args.district, `/campus/resources/calendar/instructionalDay?calendarID=${enr.calendarID}`),
+                fetchInstructionalDays(client, args.district, enr.calendarID),
             ]);
             const terms = toArray(termsRaw);
-            const days = toArray(daysRaw);
             // 3. Filter days by since/until range
-            const filteredDays = days.filter((d) => {
-                if (args.since && d.date < args.since)
-                    return false;
-                if (args.until && d.date > args.until)
-                    return false;
-                return true;
-            });
+            const inRange = (date) => !(args.since && date < args.since) && !(args.until && date > args.until);
+            const filteredDays = (days ?? []).filter((d) => inRange(d.date));
+            const calendarEvents = events.get(enr.calendarID) ?? new Map();
+            // Labelled dates IC itself marks as out of instruction, oldest first.
+            const daysOff = [...calendarEvents.entries()]
+                .filter(([date, evs]) => inRange(date) && evs.some((e) => e.instruction === false))
+                .map(([date, evs]) => ({ date, comments: joinLabels(evs.map((e) => e.name)) }))
+                .sort((a, b) => a.date.localeCompare(b.date));
             // 4. Group days into terms (only terms in this structureID; sorted by seq)
             const enrollmentTerms = terms
                 .filter((t) => t.structureID === enr.structureID)
@@ -49,19 +132,27 @@ export function registerCalendarTools(server, client) {
                     .filter((d) => d.date >= t.startDate && d.date <= t.endDate)
                     .map((d) => {
                     const out = { date: d.date, requiresAttendance: d.requiresAttendance };
-                    if (d.comments)
-                        out.comments = d.comments;
+                    const comments = joinLabels([d.comments, ...(calendarEvents.get(d.date) ?? []).map((e) => e.name)]);
+                    if (comments)
+                        out.comments = comments;
                     return out;
                 }),
             }))
-                // Drop empty terms (term entirely outside the since/until range)
-                .filter((t) => t.days.length > 0 || (!args.since && !args.until));
+                // Drop terms entirely outside the since/until range. With no days to
+                // go on, judge a term by whether its own dates overlap the range.
+                .filter((t) => days === null
+                ? !(args.since && t.endDate < args.since) && !(args.until && t.startDate > args.until)
+                : t.days.length > 0 || (!args.since && !args.until));
+            const campusVersion = days === null ? await client.campusVersion(args.district) : null;
             result.push({
                 enrollmentID: enr.enrollmentID,
                 calendarID: enr.calendarID,
                 structureID: enr.structureID,
                 calendarName: enr.calendarName,
+                ...(days === null ? { warning: 'SchoolDaysUnavailable' } : {}),
+                ...(campusVersion ? { campusVersion } : {}),
                 terms: trimmedTerms,
+                ...(daysOff.length > 0 ? { daysOff } : {}),
             });
         }
         return viewResponse(rawArgs.view, result);

package/dist/tools/healthcheck.js

@@ -28,6 +28,9 @@ export function registerHealthcheckTools(server, state) {
                 base_url: state.account.baseUrl,
                 configured_district: state.account.district,
                 auth_source: state.source ?? 'unknown',
+                // Read off the login page, so it needs no session and is reported
+                // even when discovery cannot run. Diagnostic only (#253).
+                campus_version: await state.client.campusVersion(state.account.name),
             };
             try {
                 // Discovery is what knows about LINKED districts, and it needs a live
@@ -35,7 +38,11 @@ export function registerHealthcheckTools(server, state) {
                 // is known and let the probe below decide.
                 await state.client.ensureDiscovery();
                 const districts = state.client.listDistricts();
-                detail.districts = districts.map((d) => ({ name: d.name, linked: d.linked }));
+                detail.districts = await Promise.all(districts.map(async (d) => ({
+                    name: d.name,
+                    linked: d.linked,
+                    campus_version: await state.client.campusVersion(d.name),
+                })));
                 const linked = districts.filter((d) => d.linked).map((d) => d.name);
                 detail.configured_district_is_linked = districts.some((d) => d.name === state.account.district && d.linked);
                 if (linked.length > 0)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "infinitecampus-mcp",
-  "version": "3.1.9",
+  "version": "3.2.0",
   "mcpName": "io.github.chrischall/infinitecampus-mcp",
   "description": "Infinite Campus (Campus Parent) MCP server — multi-district, read-only (plus local document download)",
   "author": "Claude Code (AI) <https://www.anthropic.com/claude>",

package/server.json

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

package/skills/ic/SKILL.md

@@ -96,7 +96,8 @@ registered even when the server is unconfigured (every other tool is not), and
 it reports whether the configured `IC_DISTRICT` is one the account is actually
 LINKED to. A district you are not linked to fails every other tool for a reason
 that looks nothing like "wrong district", so run this first when things fail
-inexplicably. Most student-scoped tools also take `studentId` (the personID from `ic_list_students`).
+inexplicably. It also reports each district's Campus version (`campus_version`,
+e.g. `2635.16`) — quote it in any bug report about an endpoint that changed. Most student-scoped tools also take `studentId` (the personID from `ic_list_students`).
 
 ## Tools
 
@@ -119,7 +120,7 @@ inexplicably. Most student-scoped tools also take `studentId` (the personID from
 ### Daily life
 | Tool | Notes |
 |------|-------|
-| `ic_list_school_days(district, studentId, view?)` | Instructional calendar with term boundaries. |
+| `ic_list_school_days(district, studentId, since?, until?, view?)` | Instructional calendar with term boundaries; labelled days carry `comments`, and each enrollment lists its `daysOff` (holidays, workdays, breaks — including ones between terms). `warning: 'SchoolDaysUnavailable'` means the district wouldn't serve the day list; terms and `daysOff` still come back. |
 | `ic_list_attendance(district, studentId, since?, until?, view?)` | Per-course attendance summary grouped by term. |
 | `ic_list_attendance_events(district, studentId, since?, until?, excusedOnly?, view?)` | Individual absence/tardy events with codes and human comments. |
 | `ic_list_behavior(district, studentId, since?, until?, view?)` | Returns a `FeatureDisabled` warning if the district has no behavior module enabled — this is not an error. |

package/skills/infinitecampus-api/references/endpoints.md

@@ -97,6 +97,12 @@ Two calls per enrollment, then join term windows to days client-side:
 ```sh
 ic_get "/campus/resources/term?structureID=$STRUCTURE_ID" | jq .          # term boundaries
 ic_get "/campus/resources/calendar/instructionalDay?calendarID=$CALENDAR_ID" | jq .  # day list
+# some districts 500 on instructionalDay (#253) — same days via calendar/day:
+ic_get "/campus/resources/calendar/day?calendarID=$CALENDAR_ID" | jq '[.[] | select(.isInstructionalDay)]'
+# day labels (holidays, workdays, early release) — the day lists no longer carry comments:
+ic_get "/campus/resources/portal/dayEvent/byEnrollment?personID=$PID" | jq '[.[].events[] | {date, name, instruction}]'
+# Campus version, no session needed (footer of the parent login page):
+curl -s "$IC_BASE_URL/campus/portal/parents/$IC_DISTRICT.jsp" | grep -o 'Version:Campus-[0-9.]*'
 
 # days within term N's [startDate,endDate]:
 jq --slurpfile terms terms.json --arg t "$TERM_ID" \