chore: remove cspell - #597
Conversation
Drop the spell checker from the repo: delete `cspell.json`, take `cspell` out of the `lint` script and devDependencies, and remove the inline `cspell:disable-next-line` / `cspell:ignore` directives it required. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Essentials Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (7)
💤 Files with no reviewable changes (4)
Included review availability: 4 reviews are currently available. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour. WalkthroughThe lint workflow now runs ESLint and Prettier checks without cspell. The cspell development dependency and configuration were removed. Pre-ship documentation was updated to match the lint command. Unused cspell suppression comments were removed from feedback documents and HTML keyword extraction code. Runtime HTML extraction logic was unchanged. Priority: ⬇️ Low Merge Risk: ⚪ Minimal · up to Linting now runs ESLint and Prettier without cspell, with repository documentation updated to match. The change is ready to merge. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
Removes the cspell spell checker from the repo. Deletes
cspell.json, dropscspellfrom the rootlintscript and devDependencies, and removes the inlinecspell:disable-next-line/cspell:ignoredirectives that only existed to satisfy it.AGENTS.mdandagent-feedback/README.mdare updated to describe whatpnpm run lintactually runs now.🤖 Generated with Claude Code