chore: v1 repo hygiene + tooling gate#17
Merged
Conversation
Remove dead config and junk, wire Biome as a real gate, add CI. - delete stale .eslintrc.js (referenced eslint/standard/prettier/ @typescript-eslint — none of which are installed; superseded by Biome) - delete bash.exe.stackdump (committed Cygwin crash dump) + ignore *.stackdump - delete orphaned vite.config.js and the dead docs/ mini-site it served (vite isn't a dependency; the playground is the demo now) - package.json: add `lint`/`format`/`check` scripts (Biome was configured but never script-wired) and `engines: node >=18` - biome.json: ignore generated/non-source paths (.astro, .vscode, .tldr) - .gitignore: drop stale entries for removed packages (snap, vue), dead dist-new/ and docs/dist; add *.stackdump - .gitattributes: normalize text to LF - CI: build + lint the library as the v1 gate; playground build advisory
Biome formatting pass over packages/. The large diff is purely CRLF→LF line-ending normalization (the source was committed with Windows CRLF); the only non-whitespace change is one import re-sort in tempus.ts.
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.
What this does
Cleans up dead config and junk that accumulated in the repo, and turns Biome (already configured but never script-wired) into an actual CI gate ahead of v1. No changes to the shipped library's behavior or its dependency tree — this is tooling and repo hygiene only.
This is the first of two v1 prep PRs. The companion #15 handles the playground's Astro 5 upgrade and its dependency moves (kept separate so this PR touches no lockfile).
Summary
.eslintrc.jsreferencedeslint/standard/prettier/@typescript-eslint— none installed; Biome replaced ESLint. Deleting it removes a second, broken lint mental-model.bash.exe.stackdump(a committed Cygwin crash dump), orphanedvite.config.js+ the deaddocs/mini-site it served (vite isn't a dependency; the playground is the demo).lint(gatespackages/),format, andcheckscripts. Ignored generated paths (.astro,.vscode,.tldr).engines: node >=18declared..gitignore: dropped stale entries for long-removed packages (snap,vue),dist-new/,docs/dist; added*.stackdump..gitattributes: normalize text to LF. The library source was committed with Windows CRLF — the second commit normalizespackages/to LF (that's why its diff looks large; the only non-whitespace change is one import re-sort).libraryjob builds + lints the published package (the v1 gate);playgroundjob builds the demo as advisory (continue-on-error) until chore(playground): upgrade to Astro 5 #15's Astro 5 + demo fixes land.Test Plan
pnpm build(library) succeedspnpm lint(biome check packages) passes — 0 errorspnpm install --frozen-lockfilevalid (no dep/lockfile changes)Follow-ups (not in this PR)
lottie-web/motionfrom root devDeps → playground (they're only used byplayground/core/test.ts) — folded into chore(playground): upgrade to Astro 5 #15 since it owns the lockfile.noExplicitAnywarning remains intempus.ts(windowversion global) — left as a tracked warning.