Repository navigation
Tidy the public repo: drop stale generated sprites, move proof folders to nori/pr-proofs - #192
Merged
Merged
Conversation
…roof folders - Stop tracking tools/sprites/.build/ (obsolete per-sprite intermediates nothing reads; the pipeline writes only to the ignored public/sprites/) and ignore it. - Move unreferenced PR review captures (pr-69/, pr-143/, screenshots/, docs/pr-proofs/, docs/proofs/, docs/art-scale/, docs/audio-evidence/, e2e/evidence/, smoke/evidence/) to the nori/pr-proofs branch under from-main/, byte-identical. - Remove the stale tools/art/preview.html (expects the old single-page atlas). - E2e specs capture through docsScreenshot: test output by default, committed docs images only with UPDATE_SCREENSHOTS=1, so test runs no longer dirty the tree. - CONTRIBUTING "Where files go"; neutral CI-permission wording; fix four broken doc links. Co-authored-by: Nori <contact@tilework.tech>
…; lint ignores .local Co-authored-by: Nori <contact@tilework.tech>
Contributor
Author
|
Proof: production build from a fresh clone (no
Post-merge on |
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
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
🤖 Generated with Nori
Repository hygiene now that the repo is public. Game code and art inputs are unchanged.
maintools/sprites/.build/pr-69/,pr-143/(root)nori/pr-proofs→from-main/screenshots/(root)placement-ghosts.spec.tsre-created it on every e2e rundocs/pr-proofs/,docs/proofs/,docs/art-scale/,docs/audio-evidence/,e2e/evidence/,smoke/evidence/tools/art/preview.html/sprites/atlas.pngthat the paged packer no longer writesTotal: 5,579 → 1,123 tracked files. The archived copies are byte-identical (tree hashes match). History is not rewritten, so the full clone size is unchanged.
Other changes:
e2e/support/docs-screenshot.ts: specs that keep committed docs images write to Playwright output by default and only refreshdocs/withUPDATE_SCREENSHOTS=1. Before this, an e2e run modified 6 tracked images and re-created rootscreenshots/. Only capture locations changed; assertions are untouched..local/, the gitignored scratch directory.Fixture pages (
depth-fixture.html,occlusion-fixture.html),docs/screenshots/**,docs/playtest/assets/**,docs/art/**, source sheets, prompts, generators, LICENSE and README are kept.Test Plan
pnpm install --frozen-lockfilethenpnpm buildwith no API keys.public/sprites/is regenerated: 24,882 files, 68 atlas pages, 8,422 manifest sprites. Its SHA-256 matches a checkout that still had.build.git statusis empty afterwards andpublic/sprites/is ignored.pnpm lint,format:check,typecheckpass.pnpm test: 2,715 passed. 3bench/balance.test.tscases timed out at 30 s under parallel load and pass when run alone (10/10).UPDATE_SCREENSHOTS=1still refreshes committed images.land-ownership"shows land rights…" already fails onmain: tile (13,39) is no longer for sale in Starter Park after Hard scenario remixes: irregular land, shaped terrain, woods and park rules (+ Mossgrove Isle) #190. Not touched here.