Skip to content

chore: remove the stray root CNAME - #12

Merged
TechLuddite merged 1 commit into
mainfrom
claude/last-commit-error-9pqqkh
Aug 17, 2026
Merged

TechLuddite merged 1 commit into
mainfrom
claude/last-commit-error-9pqqkh

Conversation

@TechLuddite

Copy link
Copy Markdown
Owner

GitHub's Settings → Pages UI wrote a CNAME at the repo root when the custom domain was first set (commit bc22fd9, "Create CNAME"). It has never been part of the deploy.

Pages publishes the Actions artifact, which is dist/, and dist/CNAME is copied from public/CNAME. The deploy workflow already asserts that file survives the build, so the root copy does nothing.

Two copies of the same setting invites the wrong one being edited later, and they had already drifted apart — the root file had no trailing newline, public/CNAME does. public/CNAME is the one that ships, so it is the one that stays.

No changelog entry: nothing a player can see changes.

Checks

  • npm run test:run — 353 tests across 8 files, all passing
  • npm run build — clean
  • dist/CNAME present after build, containing jbl.opsvibe.systems

Unrelated: the "Cannot retrieve latest commit" banner

This PR does not fix that, because there is nothing in the repo causing it. Commit 38fdd24 resolves fine through the API, git fsck is clean, and the deploy on that commit was green. GitHub is in an active incident (opened 2026-08-17 13:40 UTC) with API Requests, Actions, Webhooks, Issues and Pull Requests all degraded, which is why the banner follows you across browsers and private windows.


Generated by Claude Code

GitHub's Settings -> Pages UI wrote a CNAME at the repo root when the
custom domain was first set. It has never been part of the deploy: Pages
publishes the Actions artifact, which is dist/, and dist/CNAME comes from
public/CNAME. The workflow already asserts that file exists.

Two copies of the same setting invites the wrong one being edited later,
and they had already drifted apart on trailing newline. public/CNAME is
the one that ships, so it is the one that stays.

Co-authored-by: Claude <noreply@anthropic.com>
@TechLuddite
TechLuddite merged commit b14359a into main Aug 17, 2026
2 checks passed
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.

2 participants