Skip to content

chore: replace all em dashes with hyphen-minus#1

Merged
BENZOOgataga merged 2 commits into
mainfrom
copilot/remove-all-em-dashes
Apr 24, 2026
Merged

chore: replace all em dashes with hyphen-minus#1
BENZOOgataga merged 2 commits into
mainfrom
copilot/remove-all-em-dashes

Conversation

Copilot AI commented Apr 24, 2026

Copy link
Copy Markdown
Contributor

Removes all occurrences of the Unicode em dash character () from repository documentation, replacing each with - (space-hyphen-space).

Note: this PR targets em dashes only. Other non-ASCII characters (e.g., the box-drawing glyphs ├──/└── in CLAUDE.md's package-structure diagram) are intentional and left untouched.

Files changed

  • .gitignore - 3 IDE section comments
  • README.md - tagline and feature bullet list
  • CLAUDE.md - prose, package structure table, build quirk notes
  • docs/superpowers/plans/2026-04-24-schematic-parsing-list-info.md - plan prose, inline code comments and string literals within code examples
  • docs/superpowers/specs/2026-04-24-schematic-parsing-list-info-design.md - spec prose

No functional code changed; all replacements are in documentation, comments, and string literals inside planning/spec markdown.

Original prompt

Create a pull request in BENZOOgataga/FrameShift that removes all em dashes (—) from the codebase.

Requirements:

  • Find all occurrences of the Unicode em dash character across the repository.
  • Replace with a hyphen-minus - where a punctuation substitution is appropriate (e.g., prose in README/docs, comments, strings).
  • Where an em dash was used to imply a parenthetical or clause break and a simple - would be awkward, rephrase the surrounding text to remove the em dash (use commas/parentheses or reword the sentence).
  • Preserve meaning and readability; do not change functional code behavior.
  • Be careful with source code and string literals used in tests/snapshots; update expected outputs accordingly.
  • Avoid modifying generated or lock files unless they contain em dashes in human-facing text (prefer leaving lockfiles untouched if possible).

Deliverables:

  • PR with all changes committed.
  • Include a concise PR description summarizing what was changed and how to review.

Notes:

  • The user did not specify exact hyphen formatting; default to replacing with - (space-hyphen-space) in prose when a direct substitution is acceptable, otherwise rephrase.

The following is the prior conversation context from the user's chat exploration (may be truncated):

User: remove any em dashes in this codebase, and replace it with either a - , or rephrase it

Assistant: To do that, I need you to explicitly ask for a PR (or tell me you don’t want a PR and I’ll instead give you a local/scripted approach), because “remove any em dashes in this codebase” requires making repo-wide code changes.

Please confirm one of these:

  1. Open a PR in BENZOOgataga/FrameShift that removes/rephrases all em dashes (—) across the repository, replacing them with either - or rewritten text where appropriate.
  2. No PR: you want a script/commands you can run locally (I’ll provide a safe approach with a reviewable diff).

Also, two quick preferences so I apply the right transformation:

  • When it’s just punctuation between words (e.g., foo — bar), should I always convert to foo - bar (with spaces) or foo-bar (no spaces)?
  • Should I touch generated/lock files (like package-lock.json, yarn.lock, pnpm-lock.yaml) or leave them alone?

User: make a PR

Assistant: I’ve kicked off the pull request creation, and we’re now waiting on your confirmation in the approval dialog before I can proceed with scanning the repo and committing the em-dash replacements.

This pull request was created from Copilot chat.

Copilot AI changed the title [WIP] Remove all em dashes from the codebase chore: replace all em dashes with hyphen-minus Apr 24, 2026
Copilot AI requested a review from BENZOOgataga April 24, 2026 22:36
@BENZOOgataga BENZOOgataga marked this pull request as ready for review April 24, 2026 22:36
@BENZOOgataga BENZOOgataga requested a review from Copilot April 24, 2026 22:36

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 removes Unicode em dashes () from repository documentation/config text, replacing them with ASCII hyphen-minus (-) to standardize punctuation and avoid Unicode em dashes in prose and examples.

Changes:

  • Replaced em dashes with - in project-facing docs (README.md, CLAUDE.md) and internal plan/spec markdown.
  • Updated .gitignore IDE section comments to use - instead of .

Reviewed changes

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

Show a summary per file
File Description
.gitignore Replaces em dashes in IDE section comments with ASCII hyphen-minus.
README.md Replaces em dash punctuation in tagline and feature list with -.
CLAUDE.md Replaces em dash punctuation throughout guidance text and tables with -.
docs/superpowers/plans/2026-04-24-schematic-parsing-list-info.md Replaces em dashes in plan prose and code-example comments/string literals with -.
docs/superpowers/specs/2026-04-24-schematic-parsing-list-info-design.md Replaces em dashes in spec prose with -.

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

Comment thread CLAUDE.md
@BENZOOgataga

Copy link
Copy Markdown
Owner

@copilot apply changes based on the comments in this thread

Copilot AI commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

@copilot apply changes based on the comments in this thread

Updated the PR description to clarify that only Unicode em dashes were removed - the box-drawing characters (├──/└──) in CLAUDE.md are intentional and left untouched.

@BENZOOgataga

Copy link
Copy Markdown
Owner

@copilot apply changes based on the comments in this thread

Updated the PR description to clarify that only Unicode em dashes were removed - the box-drawing characters (├──/└──) in CLAUDE.md are intentional and left untouched.

I read your thread too fast, don't remove those characters, they are here for structuring folders and files in the README.md file. Revert.

Copilot AI commented Apr 24, 2026

Copy link
Copy Markdown
Contributor Author

@copilot apply changes based on the comments in this thread

Updated the PR description to clarify that only...

No characters were removed - the box-drawing glyphs (├──/└──) were never touched by this PR or by my changes. The only thing I did was update the PR description to clarify that those characters are intentional and left untouched. The files are already in the correct state; there is nothing to revert.

@BENZOOgataga BENZOOgataga left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

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

ready to merge

@BENZOOgataga BENZOOgataga merged commit 1c9f020 into main Apr 24, 2026
7 checks passed
@BENZOOgataga BENZOOgataga deleted the copilot/remove-all-em-dashes branch April 24, 2026 22:42
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.

3 participants