feat: remove JetBrains IDEs, keep Cursor for agentic workflow#16
Merged
Conversation
GoLand, IntelliJ IDEA, and PyCharm removed from Brewfile. Shifting fully to Claude Code + Cursor for development — AI-native tooling replaces the traditional IDE suite. Cursor stays: works alongside Claude Code, fits the vibe coding workflow, and is small enough (~500MB) to include in Brewfile.vm for VM tests. Brewfile.vm updated to include Cursor now that the disk budget is no longer constrained by the JetBrains suite. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
amcheste
added a commit
that referenced
this pull request
May 12, 2026
## Summary Brand-alignment pass against the [alanchester-brand voice rules](https://github.com/amcheste/alanchester-brand/blob/main/docs/voice.md). 105 em-dashes addressed across 7 prose files, plus the now-standard Scorecard `publish_results` fix. ## 1. Em-dash sweep (7 files, 105 dashes addressed) | File | Em-dashes | |---|---| | README.md | 33 | | CONTRIBUTING.md | 11 | | CLAUDE.md | 20 | | TESTING.md | 27 | | VERSIONING.md | 8 | | CHANGELOG.md | 4 | | SECURITY.md | 2 | **87 swept mechanically** via `re.sub(r' — ([a-z])', lambda: '. ' + capitalize, ...)` with code-block and table-row protection. Same approach as [engineering-handbook PR #16](amcheste/engineering-handbook#16) and [claude-teams-operator PR #228](amcheste/claude-teams-operator#228). **18 swept manually** with contextual judgment: - Mid-table appositives (`X — Y` where Y describes X) → commas. E.g. `pyenv — switch versions` → `pyenv, switch versions`. - Longer table-cell continuations → period + capitalize. E.g. `Persist facts across sessions — supplements CLAUDE.md` → `Persist facts across sessions. Supplements CLAUDE.md.` - List-introducing em-dashes → colons. E.g. `Breaking changes — tools removed, ...` → `Breaking changes: tools removed, ...` - Stable-state appositives → semicolons. E.g. `Pre-stable — actively being developed` → `Pre-stable; actively being developed`. - Two end-of-line em-dashes in CLAUDE.md (`...productive —\ntools, dotfiles...`) → commas with continuation. **One em-dash deliberately preserved**: `TESTING.md:73` (`# Dry run — show what would execute without running`) is inside a `` ```bash `` code block (a shell comment example for `act --dryrun`). Per theming-prompt rules on code structure, this stays. **Post-sweep audit clean**: 1 match on `grep -nE '\. [a-z]'` — `README.md:149` "Region (e.g. iad): iad" — abbreviation false positive (`e.g.`), left as-is. ## 2. Scorecard publish_results fix Same one-pattern fix as engineering-handbook, repo-template, and claude-teams-operator: ```diff push: - branches: [main] + branches: [main, develop] results_format: sarif - # Only publish to scorecard.dev from main — the action enforces this - publish_results: ${{ github.ref == 'refs/heads/main' }} + # Publish to scorecard.dev only from the default branch. + publish_results: ${{ github.ref_name == github.event.repository.default_branch }} - uses: github/codeql-action/upload-sarif@... # v4 - if: github.ref == 'refs/heads/main' + if: github.ref_name == github.event.repository.default_branch ``` This repo's default is currently `main`, so no behavior change today. But the workflow is now correct regardless of which branch is set as default — consistent with the family pattern and forward-compatible if you ever flip to develop as default for a pre-stable-release phase. ## Note on the codeql-action SHA The `github/codeql-action/upload-sarif@e46ed2cbd01164d986452f91f178727624ae40d7` pin on develop is **real and correct** — verified via the GitHub API. No imposter-commit issue on develop. `main` still has the older imposter SHA (`d4b3ca9f...` from when it was last updated from repo-template). That'll resolve on its own at the next release promotion (`develop → main`), since develop already has the real SHA. ## What's NOT in this PR - **Banner migration** (`assets/logo.png` → `assets/banner.svg` per banner-spec). Same follow-up as pokemon-red-ai, overleaf-mcp, etc. - **Repo topics** (`repositoryTopics: null`). Manual `gh repo edit`. ## Verification - [x] `git diff origin/develop --name-only` → 8 files. No edits to `setup.sh`, `Brewfile*`, `Formula/`, `claude-skills/`, `dotfiles/`, or `scripts/`. - [x] `grep -rn "—" *.md` → 1 result (TESTING.md:73, shell-comment example in code block). - [x] Em-dashes in newly-rewritten prose: 0. - [x] Em-dashes in code blocks / shell comments: preserved. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Removing GoLand, IntelliJ IDEA, and PyCharm from the dev environment. Shifting to Claude Code + Cursor as the primary development workflow — AI-native tooling replaces the traditional IDE suite.
Cursor stays — works alongside Claude Code, fits the vibe coding workflow, and is small enough (~500MB) to include in
Brewfile.vmfor VM acceptance tests.Brewfile— removegoland,intellij-idea,pycharm; keepcursorBrewfile.vm— addcursorback (now fits in 50GB VM disk without the JetBrains suite)README.md— remove JetBrains row from GUI apps table🤖 Generated with Claude Code