Reported issues for signupgenius-mcp
Pod holds 9 of 9 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 signupgenius-mcp.
Most discussed
Auto-review follow-ups for PR #143: feat: add signupgenius_healthcheck
Tracking auto-review follow-ups for #143 (verdict: fail).
🔴 Important
- src/tools/healthcheck.ts:33 — credential_rejected hint points users at SIGNUPGENIUS_API_KEY, which doesn't exist; the real key-mode env var (documented in CLAUDE.md and used everywhere else in the repo) is SIGNUPGENIUS_USER_KEY
Checklist regenerated from the auto-review of 35687d2 (run). Ticks
Read the thread · 2026-08-30 · closed · outside contributor · 1 comment
Auto-review follow-ups for PR #136: docs: list the cache env vars in server.json and .env.example
Tracking auto-review follow-ups for #136 (verdict: fail).
🔴 Important
- SIGNUPGENIUS_SESSION_CACHE description exceeds 100-character MCP Registry limit (114 chars) — server.json:71
🟡 Nits
- Inconsistent comment formatting: .env.example lines 51 & 54 use 'Optional:' prefix not found in other optional vars
Checklist regenerated from the auto-review of 6e45f88 ([run](https://github.com/chrischall/signupgenius-mcp/actions/runs/331258
Read the thread · 2026-08-27 · closed · outside contributor · 1 comment
Auto-review follow-ups for PR #125: chore: declare a mint.yaml hosting manifest
Tracking auto-review follow-ups for #125 (verdict: warn).
🟡 Nits
- mint.yaml egress.allow lists bare signupgenius.com apex domain, but no fetch() in src/client.ts targets it — only api.signupgenius.com and www.signupgenius.com are actually fetched, inconsistent with the manifest's own 'only hosts the server process actually fetches' comment
Checklist regenerated from the auto-review of c38f1f0 ([run](https://github.com/chrischall/signupge
Read the thread · 2026-08-25 · closed · outside contributor · 0 comments
Auto-review follow-ups for PR #116: feat: read slot availability and participants with no auth, and claim or release slots
Tracking auto-review follow-ups for #116 (verdict: warn).
🟡 Nits
- Stale docs describe the ModeMismatchError arm removed from client.requireKeyMode: fix src/tools/reports.ts:13, CLAUDE.md:108/112/151, README.md:30, skills/signupgenius/SKILL.md:104 to say KeyModeRequiredError is now thrown unconditionally
- CLAUDE.md:112 ('## Tool surface') still says '15 read + 2 write' while the file's TL;DR and the tool table both list 4 write tools (add_g
Read the thread · 2026-08-10 · closed · outside contributor · 0 comments
Auto-review follow-ups for PR #103: feat: add unauthenticated slot listing and fix the fetchproxy session lifecycle
Tracking auto-review follow-ups for #103 (verdict: warn).
🟡 Nits
- skills/signupgenius/SKILL.md not updated for signupgenius_get_signup_slots — the tool is missing from the "Public sign-up" list and the trigger example "What slots are still open…" still routes to the key-only, owner-scoped signupgenius_report_available
- src/tools/public-signup.ts: SlotItemSummary.participants JSDoc and the get_signup_slots tool description both promise name
Read the thread · 2026-08-03 · closed · outside contributor · 0 comments
Auto-review follow-ups for PR #89: docs: replace duplicated fleet policy with a pointer
Tracking auto-review follow-ups for #89 (verdict: warn).
🟡 Nits
- PR lacks a release-notes label (e.g.
documentation) per CLAUDE.md's 'apply exactly one label' convention — onlyauto-reviewis currently set
When addressing the review on #89, 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-07-19 · closed · outside contributor · 1 comment
Auto-review follow-ups for PR #82: feat(skill): add signupgenius api access skill
Tracking auto-review follow-ups for #82 (verdict: fail).
🔴 Important
- skills/signupgenius-api/references/sug-endpoints.md:139 — Step 2 preview jq filter reads useRSVP/owner/id/title off the top level of the response instead of under .DATA, so it always prints null and silently defeats the doc's own pre-RSVP safety gate (still unfixed as of the empty re-trigger commit 3d12d88)
When addressing the review on #82, resolve each item and add `Clo
Read the thread · 2026-07-13 · closed · outside contributor · 0 comments
Auto-review follow-ups for PR #72: refactor: adopt shared error classes from mcp-utils
Tracking auto-review follow-ups for #72 (verdict: warn).
🟡 Nits
- Stale comment in src/tools/reports.ts:15 says 'pointing the user at SIGNUPGENIUS_USER_KEY' but the shared ModeMismatchError.hint now says 'Switch to key mode to use {feature}.' without naming the env var
- Stale test description in tests/tools/reports.test.ts:28 says 'throws ModeMismatchError pointing the user at SIGNUPGENIUS_USER_KEY' but the assertion only checks instanceof a
Read the thread · 2026-07-06 · closed · outside contributor · 0 comments
Most recent
Auto-review follow-ups for PR #80: fix: bump @chrischall/mcp-utils to 0.12.0
Tracking auto-review follow-ups for #80 (verdict: fail).
🔴 Important
- Manual dependency bump violates CLAUDE.md versioning policy — versioning should be automated via release-please, not manual
🟡 Nits
- PR label 'bug' may be misleading for a dependency bump (consider 'dependencies' or confirming first-party bump label strategy)
When addressing the review on #80, resolve each item and add Closes #<this issue> to the PR once everyt
Read the thread · 2026-07-07 · closed · outside contributor · 0 comments
The remaining reports are on the project's issue tracker.