Skip to content

chore: remove unused starter scaffolding - #16

Open
PouyanJay wants to merge 1 commit into
thebuggeddev:mainfrom
PouyanJay:fix/13-dead-code
Open

chore: remove unused starter scaffolding#16
PouyanJay wants to merge 1 commit into
thebuggeddev:mainfrom
PouyanJay:fix/13-dead-code

Conversation

@PouyanJay

Copy link
Copy Markdown

Fixes #13.

Scaffolding inherited from the vinext template that nothing references. Ten files deleted, 210 lines removed, ~600 KB off the shipped assets, and three npm advisories gone.

Database — never wired up

drizzle isn't imported anywhere in app/, and .openai/hosting.json sets "d1": null, so no binding is ever created.

Removed drizzle-kit, drizzle-orm, db/, examples/d1/, drizzle/, drizzle.config.ts, and the db:generate script. drizzle-kit was pulling the vulnerable @esbuild-kit/* chain, so this takes moderate-severity advisories to zero.

Assets

  • Removed public/basis/ — 576 KB of KTX2 transcoder that is never registered. loaders.ts sets the meshopt decoder and nothing else.
  • Kept public/draco/ — also unloaded today, but it's exactly what segmented-asset work needs, and registering DRACOLoader is on the roadmap. Deleting it now would only mean re-adding it later.

Dead modules

  • app/lib/three/tsl-materials.ts — exports medicalRimNode, which nothing imports
  • app/chatgpt-auth.ts — the starter's SIWC helper, never called

README

Was still the untouched starter readme: it documented D1 bindings and ChatGPT sign-in, and never mentioned Anatomy Atelier. Rewritten around what the project actually is, including the two things worth knowing before touching the viewer — it renders on demand, and organs are normalised into a fixed cube so hotspot coordinates mean the same thing for every model.

Deliberately left alone

Verification

⚠️ Same note as #15npm test fails on this branch in isolation because main still carries the broken starter test. #14 fixes that. The 6/6 above comes from a scratch branch with all three merged.

Touches package.json and the lockfile, as does #15 — whichever merges second will need a rebase and an npm install.

Scaffolding from the vinext template that nothing references.

Database: drizzle is not imported anywhere in app/, and
.openai/hosting.json sets "d1": null so no binding is ever created.
Removes drizzle-kit and drizzle-orm, db/, examples/d1/, drizzle/,
drizzle.config.ts and the db:generate script. drizzle-kit pulled the
@esbuild-kit chain, so this clears 3 of the 18 npm advisories and takes
moderate-severity findings to zero.

Assets: public/basis/ is 576 KB of KTX2 transcoder that is never
registered — loaders.ts sets the meshopt decoder and nothing else.
public/draco/ is deliberately kept: also unloaded today, but it is what
segmented-asset work needs.

Modules: tsl-materials.ts exports a node nothing imports, and
chatgpt-auth.ts is the starter's sign-in helper, never called.

README described D1 bindings and ChatGPT sign-in and never mentioned
Anatomy Atelier. Rewritten around what the project actually is, with the
two things worth knowing before touching the viewer: it renders on
demand, and organs are normalised into a fixed cube so hotspot
coordinates mean the same thing for every model.

The drizzle copy step in build/sites-vite-plugin.ts is left alone — it is
guarded by an existence check, so it is harmless with the directory gone,
and it is generic build infrastructure rather than app code.

Fixes thebuggeddev#13
@vercel

vercel Bot commented Aug 6, 2026

Copy link
Copy Markdown

@PouyanJay is attempting to deploy a commit to the thebuggeddev-9222's projects Team on Vercel.

A member of the Team first needs to authorize it.

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.

Remove unused starter scaffolding: drizzle/D1, basis decoder, dead modules

1 participant