Skip to content

fix(ci): install eslint deps + strip ChatGPT compat claim#9

Merged
tattarw merged 4 commits into
mainfrom
fix/eslint-and-chatgpt-claim
Jun 10, 2026
Merged

fix(ci): install eslint deps + strip ChatGPT compat claim#9
tattarw merged 4 commits into
mainfrom
fix/eslint-and-chatgpt-claim

Conversation

@tattarw

@tattarw tattarw commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Two fixes:

1. Make CI green (closes #128)
The npm run lint --if-present step calls eslint src --ext .ts from package.json, but eslint was never installed. Every CI run on main has been failing on this.

Fix:

  • Add eslint@^8.57, @typescript-eslint/parser@^7.18, @typescript-eslint/eslint-plugin@^7.18 to devDependencies.
  • Switch lint script to eslint src (flat config supersedes --ext).
  • Add eslint.config.mjs with a minimal TS-aware ruleset.

2. Strip ChatGPT compatibility claim (part of #133)
README L12 pull-quote claimed Claude/Cursor/ChatGPT support. ChatGPT does not run stdio MCP servers, so that claim is incorrect. Replaced with the accurate "Claude, Cursor, Windsurf, or any MCP-compatible host" phrasing.

Deliberately NOT changed:

  • Repo description (also says ChatGPT) — needs Settings edit, not a PR.
  • README L22 verdict enum (PROCEED / DUE_DILIGENCE / ITERATE / ABANDON) — left for the enum-cascade PR (#126).
  • Tool naming: spec ↔ README already aligned on verify_brand_name.

@tattarw tattarw merged commit 709dcd6 into main Jun 10, 2026
7 checks passed
@tattarw tattarw deleted the fix/eslint-and-chatgpt-claim branch June 10, 2026 08:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant