Reported issues for charlotteonthecheap-mcp

Pod holds 7 of 7 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 charlotteonthecheap-mcp.

Most discussed

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

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

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

🔴 Important

• 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 · 2026-07-27 · closed · external user · 0 comments

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

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

🟡 Nits

• 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 · 2026-07-20 · closed · external user · 0 comments

Auto-review follow-ups for PR #15: feat!: serve every On the Cheap city from one server via a required site argument

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

🟡 Nits

• otc_get_post (src/tools/posts.ts) description promises a full URL must match the named site, but the code doesn't enforce host matching — a mismatched-site URL can silently read another city's post with the same slug

When addressing the review on #15, resolve each item and add Closes #<this issue> to the PR once everything is genuinely fixed; deferred items stay open here. S

Read the thread · 2026-07-20 · closed · external user · 1 comment

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

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

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

🔴 Important

• 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

• getPost not-found hint at src/client.ts:237 hardcodes 'charlotteonthecheap.com', wrong for the ot

Read the thread · 2026-07-20 · closed · external user · 0 comments

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

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 · 2026-07-19 · closed · external user · 0 comments

Most recent

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