Skip to content

Remove cspell.json files - #178

Merged
clarkd merged 1 commit into
mainfrom
claude/remove-cspell
Nov 5, 2026
Merged

clarkd merged 1 commit into
mainfrom
claude/remove-cspell

Conversation

@clarkd

@clarkd clarkd commented Nov 5, 2026 •

Copy link
Copy Markdown
Member

📋 Summary

Removes every cspell.json from the repository: the root file and the copies in 15 plugin folders. They're spell-checker word lists, which are personal tooling config and not needed here. Reviewers have asked for them to be removed on recent PRs (#134, #165, #176).

Nothing references these files: no workflow, setting or skill uses cspell. No plugin behaviour changes. The deletions don't touch metadata.json, data streams or docs, and nothing is deployed differently.

Plugins affected: AutoTask, Checkly, DattoRMM, FantasyPremierLeague (v1, v2), Healthchecks.io, Huntress, Jaeger, NinjaOne, Phare, Postcoder, RDAP, RealTimeCloud, Rootly, UniFi.

Expected CI failure: pr-run.yaml will fail the scope and version check, because this touches several plugin folders without version bumps. Bumping 15 plugins to delete a file that never ships would trigger 15 releases for no user-facing change. This needs a maintainer merge past that check, as with the repo-wide ignoreCertificateErrors cleanup in #148.

A separate PR updates REVIEW.md so that CodeRabbit flags cspell.json in future PRs.

🔍 Scope of change

  • Documentation only
  • Repository metadata or configuration
  • CI / automation
  • Other (please describe):

📚 Checklist


Generated by Claude Code

Summary by CodeRabbit

  • Chores
    • Removed custom spelling-word lists for the project and several integrations. Specialized product names and technical terms previously included in those lists may no longer be recognized by spelling checks. These changes affect spelling-check configuration only; no changes to product features or runtime behavior are indicated.

Spell-check config is personal tooling and does not belong in the
repository. Deletes the root file and the copies in 15 plugin folders.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01FsDuPT1b16nKH9GT1WWhws
@github-actions

github-actions Bot commented Nov 5, 2026

Copy link
Copy Markdown

🧩 Plugin PR Summary

📦 Modified Plugins

  • plugins/AutoTask/v1
  • plugins/Checkly/v1
  • plugins/DattoRMM/v1
  • plugins/FantasyPremierLeague/v1
  • plugins/FantasyPremierLeague/v2
  • plugins/Healthchecks.io/v1
  • plugins/Huntress/v1
  • plugins/Jaeger/v1
  • plugins/NinjaOne/v1
  • plugins/Phare/v1
  • plugins/Postcoder/v1
  • plugins/RDAP/v1
  • plugins/RealTimeCloud/v1
  • plugins/Rootly/v1
  • plugins/UniFi/v1

📋 Results

Step Status
Scope & version ❌ Failed
Validation ⏭️ Skipped
Deployment ⏭️ Skipped

@coderabbitai

coderabbitai Bot commented Nov 5, 2026 •

Copy link
Copy Markdown
Contributor

Explain Change →

No actionable comments were generated in the recent review.

Walkthrough

The pull request deletes the root cspell configuration and 15 plugin-specific cspell configuration files. The removed files contained custom spelling word lists.

Changes: cspell configuration removals

Layer / File(s) Summary
Delete cspell word lists
cspell.json, plugins/*/v1/cspell.json, plugins/FantasyPremierLeague/v2/cspell.json
The root and plugin-specific cspell configuration files are deleted. The files contained custom spelling words.
Merge Risk: Moderate · up to 63c23

The PR cannot complete its normal plugin validation and preview deployment checks. Resolve the workflow exception or explicitly accept the missing checks before merging.

Pre-merge checks: Passed 4

Check name Status Explanation
Title check Passed The title "Remove cspell.json files" matches the main change. The PR deletes the root cspell.json and the cspell.json copies in 15 plugin folders. The title is short, specific, and names the file type…
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Description check Passed The description identifies the change, affected files and plugins, scope, expected CI impact, rationale, and checklist status. It is appropriate for a miscellaneous repository configuration change.
Full details: Title check

Explanation

The title "Remove cspell.json files" matches the main change. The PR deletes the root cspell.json and the cspell.json copies in 15 plugin folders. The title is short, specific, and names the file type removed. It does not name the plugin folders, but that detail is not required.

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Recent review info

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Repository UI (base), Organization UI (inherited)
  • Review profile: ASSERTIVE
  • Plan: Essentials
  • Run ID: e67b5e9a-876b-4643-aad9-eef047d73854
Commits

Reviewing files that changed from the base of the PR and between 9f597aa and 63c23d3.

Files selected for processing: 16

  • cspell.json
  • plugins/AutoTask/v1/cspell.json
  • plugins/Checkly/v1/cspell.json
  • plugins/DattoRMM/v1/cspell.json
  • plugins/FantasyPremierLeague/v1/cspell.json
  • plugins/FantasyPremierLeague/v2/cspell.json
  • plugins/Healthchecks.io/v1/cspell.json
  • plugins/Huntress/v1/cspell.json
  • plugins/Jaeger/v1/cspell.json
  • plugins/NinjaOne/v1/cspell.json
  • plugins/Phare/v1/cspell.json
  • plugins/Postcoder/v1/cspell.json
  • plugins/RDAP/v1/cspell.json
  • plugins/RealTimeCloud/v1/cspell.json
  • plugins/Rootly/v1/cspell.json
  • plugins/UniFi/v1/cspell.json
Files with no reviewable changes: 16

  • cspell.json
  • plugins/Rootly/v1/cspell.json
  • plugins/Phare/v1/cspell.json
  • plugins/Jaeger/v1/cspell.json
  • plugins/NinjaOne/v1/cspell.json
  • plugins/UniFi/v1/cspell.json
  • plugins/FantasyPremierLeague/v2/cspell.json
  • plugins/AutoTask/v1/cspell.json
  • plugins/Huntress/v1/cspell.json
  • plugins/RealTimeCloud/v1/cspell.json
  • plugins/Postcoder/v1/cspell.json
  • plugins/RDAP/v1/cspell.json
  • plugins/DattoRMM/v1/cspell.json
  • plugins/FantasyPremierLeague/v1/cspell.json
  • plugins/Healthchecks.io/v1/cspell.json
  • plugins/Checkly/v1/cspell.json

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

@clarkd
clarkd marked this pull request as ready for review November 5, 2026 10:33
@clarkd
clarkd requested a review from a team November 5, 2026 10:33
@clarkd clarkd mentioned this pull request Nov 5, 2026
3 of 9 tasks
@clarkd
clarkd merged commit a01f9db into main Nov 5, 2026
1 of 2 checks passed
@clarkd
clarkd deleted the claude/remove-cspell branch November 5, 2026 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants