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.

Most discussed

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

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

🟡 Nits

• 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). Ticks survive later rounds, but a round that reviewed an older

Read the thread · 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

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 · 2026-08-27 · closed · outside contributor · 0 comments

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

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 · 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

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 · 2026-08-03 · closed · outside contributor · 0 comments

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

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 · 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

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 · 2026-08-03 · closed · outside contributor · 0 comments

Most recent

The remaining reports are on the project's issue tracker.