# Reported issues for infinitecampus-mcp

Pod holds 6 of 6 problems reported by people outside the maintainer team. Issues filed by the project's own owners, members and collaborators are excluded entirely — a maintainer's release checklist is not a warning to a prospective user.

Back to [infinitecampus-mcp](/mcp/infinitecampus-mcp).

## Most discussed

### Auto-review follow-ups for PR #152: feat: add ic_healthcheck, and surface district linkage

<!-- auto-review-followup:PR-152 -->

Tracking auto-review follow-ups for #152 (verdict: **warn**).

### 🟡 Nits
- [x] CLAUDE.md, README.md, and skills/ic/SKILL.md still say "19 tools" after this PR adds the 20th tool (ic_healthcheck); SKILL.md's tool tables also lack a row for ic_healthcheck

---
Checklist regenerated from the auto-review of `aee82ae` ([run](https://github.com/chrischall/infinitecampus-mcp/actions/runs/33336991502)). Ticks survive later rounds, but a round that reviewed an older

[Read the thread](https://github.com/chrischall/infinitecampus-mcp/issues/153) · 2026-08-30 · closed · outside contributor · 1 comment

### Auto-review follow-ups for PR #144: feat: cache the primary district's session so a restart skips the login

<!-- auto-review-followup:PR-144 -->

Tracking auto-review follow-ups for #144 (verdict: **warn**).

### 🟡 Nits
- [ ] src/client.ts:447 comments a non-latching `return true` as "a real, permanent answer" while line 460 calls the same non-latching outcome "transient" — make the permanent/transient wording match the actual `discoveryDone` behaviour
- [ ] tests/client.test.ts:1899-1919 ("retries discovery when the CUPS probe failed rather than latching") has a dead first `mockImplementation` and de

[Read the thread](https://github.com/chrischall/infinitecampus-mcp/issues/145) · 2026-08-27 · closed · outside contributor · 0 comments

### Auto-review follow-ups for PR #134: chore: declare a mint.yaml hosting manifest

<!-- auto-review-followup:PR-134 -->

Tracking auto-review follow-ups for #134 (verdict: **warn**).

### 🟡 Nits
- [ ] `.mcpbignore`'s new `mint.yaml` line is redundant — already excluded by the pre-existing `*.yaml`/`*.yml` glob
- [ ] `mint.yaml` isn't added to CLAUDE.md's 'Plugin / Marketplace / Registry' inventory of distribution-only manifest files, unlike its siblings

---
Checklist regenerated from the auto-review of `23615e3` ([run](https://github.com/chrischall/infinitecampus-mcp/actions/

[Read the thread](https://github.com/chrischall/infinitecampus-mcp/issues/135) · 2026-08-25 · closed · outside contributor · 0 comments

### Auto-review follow-ups for PR #118: test(auth): hold the per-host lifter cache in place, and drop a stale docstring

<!-- auto-review-followup:PR-118 -->

Tracking auto-review follow-ups for #118 (verdict: **warn**).

### 🟡 Nits
- [ ] Test isolation: lifterFactoryMock not reset in beforeEach, causing test-order dependency (tests/auth.test.ts:51)

---
When addressing the review on #118, resolve each item and add `Closes #<this issue>` to the PR once everything is genuinely fixed; deferred items stay open here. See the auto-review follow-up convention in CLAUDE.md.

[Read the thread](https://github.com/chrischall/infinitecampus-mcp/issues/119) · 2026-08-03 · closed · outside contributor · 0 comments

### Auto-review follow-ups for PR #116: refactor(auth): collapse the hand-rolled lifter onto createSessionLifter

<!-- auto-review-followup:PR-116 -->

Tracking auto-review follow-ups for #116 (verdict: **warn**).

### 🟡 Nits
- [ ] Stale duplicate JSDoc comment left above lifterFor's cache in src/auth.ts:141-148, describing the wrong declaration
- [ ] tests/auth.test.ts doesn't verify the single-flight lifter caching the PR calls load-bearing — the mock can't distinguish a cached lifter from a freshly-constructed one per call

---
When addressing the review on #116, resolve each item and add `Closes #<this 

[Read the thread](https://github.com/chrischall/infinitecampus-mcp/issues/117) · 2026-08-03 · closed · outside contributor · 0 comments

### Auto-review follow-ups for PR #113: fix(auth): renew the browser session instead of dead-ending on expiry

<!-- auto-review-followup:PR-113 -->

Tracking auto-review follow-ups for #113 (verdict: **warn**).

### 🟡 Nits
- [ ] src/client.ts:54 — the `fetchproxyMode` private field is assigned but never read after both discovery guards were removed; delete it, and fix its doc comment which still claims the client cannot re-login on cookie expiry and that discovery runs lazily on the first `ensureDiscovery()` call
- [ ] src/client.ts:137 — `ensureDiscovery()` binds `const session = await …ensure()` only t

[Read the thread](https://github.com/chrischall/infinitecampus-mcp/issues/114) · 2026-08-03 · closed · outside contributor · 0 comments

## Most recent

The remaining reports are on [the project's issue tracker](https://github.com/chrischall/infinitecampus-mcp/issues).