Skip to content
This repository was archived by the owner on Sep 10, 2026. It is now read-only.

Drop every em dash - #46

Merged
seenu-k merged 4 commits into
mainfrom
docs/no-em-dashes
Aug 8, 2026
Merged

seenu-k merged 4 commits into
mainfrom
docs/no-em-dashes

Conversation

@seenu-k

@seenu-k seenu-k commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator

1046 of them, across 150 tracked files. Recast per sentence as a comma, colon, semicolon, parenthesis or a second sentence, not swept with a regex. No spaced hyphens were introduced.

Everything, not just the published surface: source comments and TSDoc, tests, MAINTAINERS.md / CONTRIBUTING.md / AGENTS.md, docs/blockers.md, the workflows, the plugin skill, the scaffolder's templates, and the changelogs.

Text that actually ships

Error messages State updated, try again (kernel stateUpdated), plus the rate-limit, avatar-type and account-deletion messages. Dispatch is on code, never error, so nothing following that rule breaks.
OpenAPI descriptions Re-emitted via pnpm openapi, and clients/dart regenerated from them. Wording only: no operation, schema or status code moved.
Engine-rendered pages A title reads Terms of Service: My Game; the /j share description separates the versus line with a comma.
create-eigen-game The greeting, the missing-tooling report, and the scaffolded project's README.

Deliberately untouched

packages/create-eigen-game/templates/worker/worker-configuration.d.ts and examples/rps/worker-configuration.d.ts, 20 instances between them. That is Cloudflare's generated runtime types; wrangler types reproduces it verbatim, so editing it would only be undone.

Changesets

One patch changeset across the five published packages. clients/dart has no version of its own; it is stamped from the engine's.

Verification

biome check . clean over 170 files · all six tsc projects clean · 323 tests pass (kernel 73, testkit 5, server 160, create-eigen-game 71, rps 2 + 12).

Stacking

Based on docs/firebase-installer (#45), which touches the same three files. Merge that first and this retargets to main on its own.

The eigen-web and eigen-flutter halves follow separately. The generated TypeScript reference on the docs site is downstream of this repo's TSDoc, so it clears once this lands and sync-api runs.

🤖 Generated with Claude Code

seenu-k and others added 2 commits August 8, 2026 14:03
`curl -sL https://firebase.tools | bash` replaces
`npm install -g firebase-tools` in the two places the scaffolder suggests
it: the missing-tooling report and the scaffolded project's README.

Google's own installer, it picks a standalone binary or npm to suit the
machine, and it is the one the Firebase documentation leads with. The npm
install still works; it just needs a global npm prefix the reader may not
have write access to.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
1046 of them across 150 tracked files, which is enough that the prose
reads as machine-written whatever it actually says. Recast per sentence
as a comma, colon, semicolon, parenthesis or a second sentence, not swept
with a regex. No spaced hyphens were introduced.

Everything, not just the published surface: source comments and TSDoc,
tests, MAINTAINERS/CONTRIBUTING/AGENTS, docs/blockers.md, the workflows,
the plugin skill, the scaffolder's templates, the changelogs.

Text that actually ships changed with it:

  · Error messages. `State updated, try again` (kernel stateUpdated),
    the rate-limit, avatar-type and account-deletion messages. Dispatch
    is on `code`, never `error`, so nothing following that rule breaks.
  · OpenAPI descriptions, re-emitted, and the Dart client regenerated
    from them. Wording only; no operation, schema or status moved.
  · Engine-rendered pages: a title reads `Terms of Service: My Game`,
    and the /j share description separates the versus line with a comma.
  · create-eigen-game's greeting, missing-tooling report and README.

Left alone: `worker-configuration.d.ts` in both worker trees, which is
Cloudflare's generated runtime types and comes back on `wrangler types`.

biome clean over 170 files, all six typecheck projects clean, 323 tests
pass.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
seenu-k added a commit to eigeninteractive/eigen-web that referenced this pull request Aug 8, 2026
The first pass took the 537 in authored docs. This takes the rest of the
repository: docusaurus.config.ts comments, CONTRIBUTING.md, the four
scripts, sidebars.ts, wrangler.jsonc, the workflows and dependabot.yml.

The generated reference clears by regeneration rather than by hand.
`pnpm sync-api` against the fixed engine rewrites docs/reference/
typescript (138 of them, all from eigen-server's TSDoc), the HTTP API
pages, and both copies of openapi.json.

That part depends on eigeninteractive/eigen-server#46 landing; the
committed output here is what a sync run produces once it has.

check-admonitions, check-docs-version, lint, typecheck and build all
clean. Zero em dashes left in any tracked file.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@seenu-k
seenu-k changed the base branch from docs/firebase-installer to main August 8, 2026 11:00
@seenu-k

seenu-k commented Aug 8, 2026

Copy link
Copy Markdown
Collaborator Author

Retargeted from docs/firebase-installer to main: checks.yml only triggers on pull_request: branches: [main], so the stacked form got no CI at all.

That means this PR now carries #45's three-line Firebase-installer change and its changeset as well. Merging this supersedes #45 — close that one rather than merging both.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@seenu-k
seenu-k merged commit 37b4ad1 into main Aug 8, 2026
3 checks passed
@seenu-k
seenu-k deleted the docs/no-em-dashes branch August 8, 2026 11:05
This was referenced Aug 8, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant