Skip to content

Add codespell support with configuration and fixes#65

Open
yarikoptic wants to merge 4 commits intoanthropics:mainfrom
yarikoptic:enh-codespell
Open

Add codespell support with configuration and fixes#65
yarikoptic wants to merge 4 commits intoanthropics:mainfrom
yarikoptic:enh-codespell

Conversation

@yarikoptic
Copy link

Add codespell configuration and fix existing typos.

More about codespell: https://github.com/codespell-project/codespell

I personally introduced it to dozens if not hundreds of projects already and so far only positive feedback.

CI workflow has 'permissions' set only to 'read' so also should be safe.

Changes

Configuration & Infrastructure

  • Added .codespellrc with skip patterns for .git*, .codespellrc, .npm, .cache
  • Created GitHub Actions workflow to check spelling on push and PRs to main
  • Added camelCase/PascalCase ignore-regex to prevent false positives on identifiers like prFiles
  • Added .npm/ and .local/ to .gitignore (cache directories that shouldn't be tracked)

Typo Fixes (3 fixes in 3 files)

Non-ambiguous typos fixed automatically via codespell -w ([DATALAD RUNCMD] commit):

  • deseralization -> deserialization (.claude/commands/security-review.md, claudecode/prompts.py)
  • Outputing -> Outputting (claudecode/claude_api_client.py)

All fixes are in documentation/prompt strings, no functional code changes.

Testing

Codespell passes with zero errors after all fixes.


Generated with Claude Code

Fixed typos:
- deseralization -> deserialization (.claude/commands/security-review.md, claudecode/prompts.py)
- Outputing -> Outputting (claudecode/claude_api_client.py)

=== Do not change lines below ===
{
 "chain": [],
 "cmd": "uvx codespell -w",
 "exit": 0,
 "extra_inputs": [],
 "inputs": [],
 "outputs": [],
 "pwd": "."
}
^^^ Do not change lines above ^^^
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