Skip to content

Expand cSpell coverage to more file types - #42760

Merged
julien-deramond merged 1 commit into
mainfrom
main-jd-expand-cspell-coverage
Jul 28, 2026
Merged

Expand cSpell coverage to more file types#42760
julien-deramond merged 1 commit into
mainfrom
main-jd-expand-cspell-coverage

Conversation

@julien-deramond

Copy link
Copy Markdown
Member

Description

Backport of 565eee6

Copilot AI review requested due to automatic review settings July 28, 2026 06:18

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR expands spellchecking coverage (cSpell) across additional file types in the repository, adds a local npm run spellcheck entrypoint, and updates the dictionary/config to accommodate the broader scan surface.

Changes:

  • Add a spellcheck npm script and include cspell as a dev dependency.
  • Expand the GitHub Actions cspell workflow to scan additional file types (and .gitignore).
  • Update .cspell.json words/ignore settings to reduce false positives and exclude noisy paths.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
package.json Adds cspell devDependency and a spellcheck npm script targeting more file types.
package-lock.json Updates lockfile to include cspell and its transitive dependencies.
.github/workflows/cspell.yml Expands the workflow’s scanned file globs (includes .gitignore and more extensions).
.cspell.json Extends dictionary/ignore settings to support the expanded scan scope.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .cspell.json
Comment on lines +194 to +196
"ignoreRegExpList": [
"\\u001[Bb]\\[[0-9;]*m"
],
@julien-deramond
julien-deramond force-pushed the main-jd-expand-cspell-coverage branch from 48c5aaf to 9ba5678 Compare July 28, 2026 06:27
Copilot AI review requested due to automatic review settings July 28, 2026 06:27
@julien-deramond
julien-deramond marked this pull request as ready for review July 28, 2026 06:28
@julien-deramond
julien-deramond merged commit 832d8ec into main Jul 28, 2026
15 checks passed
@julien-deramond
julien-deramond deleted the main-jd-expand-cspell-coverage branch July 28, 2026 06:29

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 3 out of 4 changed files in this pull request and generated no new comments.

Comments suppressed due to low confidence (1)

.cspell.json:175

  • ignoreRegExpList entry looks like an invalid Unicode escape: \u001[Bb] will be interpreted by the RegExp parser as a malformed \u escape (it expects 4 hex digits), which can cause cspell to error when loading the config. If the intent is to ignore ANSI color sequences (ESC + [...m), use \u001b and escape the literal [ properly.
  "ignoreRegExpList": [
    "\\u001[Bb]\\[[0-9;]*m"
  ],

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants