# 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](/mcp/signupgenius-mcp).

## Most discussed

### Auto-review follow-ups for PR #143: feat: add signupgenius_healthcheck

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

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](https://github.com/chrischall/signupgenius-mcp/actions/runs/33337039911)). Ticks 

[Read the thread](https://github.com/chrischall/signupgenius-mcp/issues/144) · 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

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

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
- [x] 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](https://github.com/chrischall/signupgenius-mcp/issues/137) · 2026-08-27 · closed · outside contributor · 1 comment

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

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

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](https://github.com/chrischall/signupgenius-mcp/issues/126) · 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

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

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](https://github.com/chrischall/signupgenius-mcp/issues/117) · 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

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

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](https://github.com/chrischall/signupgenius-mcp/issues/104) · 2026-08-03 · closed · outside contributor · 0 comments

### Auto-review follow-ups for PR #89: docs: replace duplicated fleet policy with a pointer

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

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 — only `auto-review` is 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](https://github.com/chrischall/signupgenius-mcp/issues/90) · 2026-07-19 · closed · outside contributor · 1 comment

### Auto-review follow-ups for PR #82: feat(skill): add signupgenius api access skill

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

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](https://github.com/chrischall/signupgenius-mcp/issues/83) · 2026-07-13 · closed · outside contributor · 0 comments

### Auto-review follow-ups for PR #72: refactor: adopt shared error classes from mcp-utils

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

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](https://github.com/chrischall/signupgenius-mcp/issues/73) · 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

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

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](https://github.com/chrischall/signupgenius-mcp/issues/81) · 2026-07-07 · closed · outside contributor · 0 comments

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