Skip to content

Add comprehensive spell checking integration with CSpell#7

Merged
plx merged 4 commits intomainfrom
plx/spell-check-integration
Aug 20, 2025
Merged

Add comprehensive spell checking integration with CSpell#7
plx merged 4 commits intomainfrom
plx/spell-check-integration

Conversation

@plx
Copy link
Copy Markdown
Owner

@plx plx commented Aug 19, 2025

Summary

This PR introduces intelligent spell checking to the Astro-based static site using CSpell, providing comprehensive coverage for both source files and generated HTML output.

Key Features Added:

Smart Technical Dictionary: Understands camelCase, PascalCase, kebab-case, and other technical naming conventions
Specialized Dictionaries: Includes CSS, HTML, npm, and TypeScript-specific terms
Dual-Phase Checking: Validates both source files during development and built HTML in CI/CD
Custom Dictionary: Project-specific terms stored in cspell.json for easy maintenance
Developer-Friendly: Integrated with justfile for convenient local usage

Changes Made:

• Added CSpell with specialized dictionaries (@cspell/dict-css, @cspell/dict-html, @cspell/dict-npm, @cspell/dict-typescript)
• Created comprehensive cspell.json configuration with custom project dictionary
• Fixed typo in personal website project description ("sytem" → "system")
• Added npm scripts: spellcheck, spellcheck:html, spellcheck:all, validate:all
• Integrated spell checking into GitHub Actions workflow with proper staging
• Updated .gitignore to exclude CSpell cache files
• Added SPELLCHECK.md documentation with usage examples and maintenance guidelines
• Enhanced justfile with spell-check commands for streamlined development workflow

CI/CD Integration:

The GitHub Actions workflow now includes:

  1. Lint checking (existing)
  2. Source file spell checking (new)
  3. Build process (existing)
  4. HTML output spell checking (new)
  5. Deployment (existing)

This ensures all content is validated before deployment while maintaining fast feedback loops.

Developer Experience:

# Quick spell check during development
just spellcheck

# Check built HTML output
just spellcheck-html

# Run complete validation suite
just validate

The spell checker integrates seamlessly with the existing ESLint workflow and respects the project's strict quality standards.

🤖 Generated with Claude Code

@github-actions
Copy link
Copy Markdown
Contributor

PR Validation ✅

All checks passed! Ready for review.

Checks Performed:

  • ✓ Linting
  • ✓ Type checking
  • ✓ Build verification
  • ✓ Internal link validation

This commit introduces intelligent spell checking to the project:

- Added CSpell package with specialized dictionaries for CSS, HTML, npm, and TypeScript
- Created comprehensive cspell.json configuration with custom project dictionary
- Fixed typo in personal website project description ("sytem" -> "system")
- Added spell-check npm scripts for source files and built HTML
- Integrated spell checking into GitHub Actions CI/CD pipeline
- Updated .gitignore to exclude CSpell cache files
- Added SPELLCHECK.md documentation for usage and maintenance
- Enhanced justfile with spell-check commands for convenient development

The spell checker understands technical naming conventions (camelCase, PascalCase, etc.)
and integrates seamlessly with the existing linting and build process.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@plx plx force-pushed the plx/spell-check-integration branch from d2489a7 to 4504471 Compare August 20, 2025 10:10
@github-actions
Copy link
Copy Markdown
Contributor

PR Validation ❌

Some checks failed. Please review the errors above.

Checks Performed:

  • ✓ Linting
  • ✓ Type checking
  • ✓ Build verification
  • ✓ Internal link validation

Regenerated package-lock.json to resolve @rollup/rollup-linux-x64-gnu
module loading issue in GitHub Actions Ubuntu environment.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

PR Validation ❌

Some checks failed. Please review the errors above.

Checks Performed:

  • ✓ Linting
  • ✓ Type checking
  • ✓ Build verification
  • ✓ Internal link validation

- Remove stale pnpm-lock.yaml that was causing version conflicts
- Regenerate package-lock.json with consistent dependency resolution
- Fix prettier version mismatch between package.json and lockfile

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

PR Validation ❌

Some checks failed. Please review the errors above.

Checks Performed:

  • ✓ Linting
  • ✓ Type checking
  • ✓ Build verification
  • ✓ Internal link validation

- @astrojs/language-server requires prettier@^3.0.0
- Previous lockfile had prettier@2.8.7 which conflicted with CI expectations
- Added prettier@^3.6.2 as explicit devDependency to satisfy all peer deps
- Fixes npm ci lockfile sync errors in GitHub Actions

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@github-actions
Copy link
Copy Markdown
Contributor

PR Validation ✅

All checks passed! Ready for review.

Checks Performed:

  • ✓ Linting
  • ✓ Type checking
  • ✓ Build verification
  • ✓ Internal link validation

@plx plx merged commit e50cda2 into main Aug 20, 2025
1 check passed
@plx plx deleted the plx/spell-check-integration branch August 20, 2025 10:34
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