chore(config,docs): cover .pmtiles.png in artifact guard; record #118 learnings#120
Merged
Conversation
…learnings - .claude/settings.json: the PreToolUse artifact-guard deny pattern now also matches *.pmtiles.png — the #118 rename (basemap.pmtiles -> basemap.pmtiles.png, PR #119) had moved the committed basemap archive out of *.pmtiles coverage, leaving the generated binary hand-editable without a prompt. - CLAUDE.md: document the basemap.pmtiles.png masquerade rationale (Pages CDN gzips application/octet-stream at every size and range-slices the compressed body; image/png is the only probe-proven exempt type; preflight+Blob fallback absorbs future CDN flips) and the stale-PR-head merge-verification law from the PR #119 incident (verify head.sha == pushed SHA + check-runs exist for that exact SHA before any merge; close/reopen resync fires a duplicate stale event whose concurrency clash cancels fresh CI - cancel stale run, then rerun). Spec check: docs/superpowers/specs/ contains no basemap filename references (verified) - only dated historical plan documents mention the old name, and those correctly record what was true at the time; deliberately untouched. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
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.
Follow-ups from the #118 fix (PR #119), user-requested.
What
.claude/settings.jsonartifact guard: the PreToolUse deny pattern now also matches*.pmtiles.png. The First-load vector basemap broken: GitHub Pages CDN gzips basemap.pmtiles and range-slices the compressed form (ERR_CONTENT_DECODING_FAILED) #118 rename (basemap.pmtiles→basemap.pmtiles.png) had moved the committed basemap archive out of*.pmtilescoverage, leaving the 27 MB generated binary hand-editable without a prompt. One-line change; the user's local (uncommitted) settings churn is deliberately not included.basemap.pmtiles.pngmasquerade rationale (Pages CDN gzipsapplication/octet-streamat every size and range-slices the compressed body;image/pngis the only probe-proven exempt type; the runtime preflight + Blob fallback absorbs future CDN flips) — so a future "cleanup" doesn't resurrect First-load vector basemap broken: GitHub Pages CDN gzips basemap.pmtiles and range-slices the compressed form (ERR_CONTENT_DECODING_FAILED) #118;head.sha== pushed SHA and check-runs exist for that exact SHA before any merge; close/reopen resyncs but fires a duplicate stale event whose concurrency clash cancels fresh CI — cancel the stale run first, then rerun).Spec check (resolves the "spec still names basemap.pmtiles" flag)
docs/superpowers/specs/contains no basemap filename references (verified by grep; the onlypmtilesmention is a generic route-ordering note in the GPX spec). The old name appears only in dated historical plan documents, which correctly record what was true at the time — deliberately untouched. No spec edit needed.Refs #118, #119 (no code changes; docs/config only).
🤖 Generated with Claude Code