# Reported issues for onthecheap-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 [onthecheap-mcp](/mcp/onthecheap-mcp).

## Most discussed

### Auto-review follow-ups for PR #33: refactor(connector): retire the standalone Cloudflare Worker connector

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

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

### 🟡 Nits
- [ ] vitest.config.ts:1 — `configDefaults` is now an unused import; trim to `import { defineConfig } from 'vitest/config';`
- [ ] package.json:5 — revert the `—` escape in `description` back to a literal em dash, matching manifest.json:7 and server.json:4
- [ ] src/client.ts:115 — stale "the hosted connector's every request would fail" comment; reword to "a deployed server" to match the

[Read the thread](https://github.com/chrischall/onthecheap-mcp/issues/34) · 2026-08-07 · closed · external user · 0 comments

### Auto-review follow-ups for PR #27: fix(posts): reject a full URL belonging to a different site

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

Tracking auto-review follow-ups for #27 (verdict: **fail**).

### 🔴 Important
- [x] src/client.ts assertSameSite compares refHost/ownHost without stripping a leading www., unlike siteForBaseUrl used two lines later — causes false-positive site-mismatch rejection for charlotte/denver when the URL's www-form differs from the canonical baseUrl's

---
When addressing the review on #27, resolve each item and add `Closes #<this issue>` to the PR once everything is 

[Read the thread](https://github.com/chrischall/onthecheap-mcp/issues/28) · 2026-07-27 · closed · external user · 0 comments

### Auto-review follow-ups for PR #17: ci: deploy the connector Worker on release

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

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

### 🟡 Nits
- [x] docs/DEPLOY-CONNECTOR.md and README.md still describe the connector deploy as manual-only / "no CI deploy for connectors," which this PR's new release-triggered deploy-connector job makes inaccurate

---
When addressing the review on #17, 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

[Read the thread](https://github.com/chrischall/onthecheap-mcp/issues/18) · 2026-07-20 · closed · external user · 0 comments

### Auto-review follow-ups for PR #12: fix(ci): typecheck the Worker entry and run its test suite

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

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

### 🟡 Nits
- [ ] README.md:110 Development section still says `npm test # node suite`, stale now that `npm test` runs both node and Workers pools
- [ ] vitest.workers.config.ts:7 comment claims `npm test` never touches this file, no longer true after package.json's test script change

---
When addressing the review on #12, resolve each item and add `Closes #<this issue>` to the PR once everything i

[Read the thread](https://github.com/chrischall/onthecheap-mcp/issues/13) · 2026-07-20 · closed · external user · 1 comment

### Auto-review follow-ups for PR #9: feat!: serve the whole On the Cheap network, resolving term ids per site

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

Tracking auto-review follow-ups for #9 (verdict: **fail**).

### 🔴 Important
- [x] Event tools (otc_list_events / otc_events_month_overview) registered unconditionally for the national hub in src/index.ts:17 and src/worker.ts:33 — national has no events calendar, so calls silently return empty and misinform the model; gate on client.site.national

### 🟡 Nits
- [x] getPost not-found hint at src/client.ts:237 hardcodes 'charlotteonthecheap.com', wrong for the ot

[Read the thread](https://github.com/chrischall/onthecheap-mcp/issues/10) · 2026-07-20 · closed · external user · 0 comments

### Auto-review follow-ups for PR #4: feat: hosted Cloudflare connector for claude.ai

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

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

### 🟡 Nits
- [ ] src/worker.ts has no automated typecheck: excluded from tsconfig.json and worker:test is not in ci.yml, so a type regression in the Worker entry only surfaces at manual deploy — add a tsconfig.worker.json (noEmit) + tsc --noEmit CI step

---
When addressing the review on #4, resolve each item and add `Closes #<this issue>` to the PR once everything is genuinely fixed; deferred items 

[Read the thread](https://github.com/chrischall/onthecheap-mcp/issues/5) · 2026-07-19 · closed · external user · 0 comments

## Most recent

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