Merged
Conversation
Pure relocation per Spec C: textureAtlas.ts and cloudFade.ts move under src/services/gpu/resources/, with their test mirror under tests/services/gpu/resources/. Imports updated at every consumer. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Pure relocation per Spec C: postProcess.ts moves under src/services/gpu/passes/, with postProcess.test.ts and the legacy toneMap.test.ts mirrored under tests/services/gpu/passes/. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Pure relocation per Spec C: fontMetrics.ts, labelLayout.ts, and youAreHereVisibility.ts move under src/services/gpu/labels/, with matching tests under tests/services/gpu/labels/. Sibling import in labelLayout.ts is unchanged because both files moved together. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Pure relocation per Spec C: pointRenderer, pickRenderer, quadRenderer, diskRenderer, proceduralDiskRenderer, milkyWayRenderer, and filamentRenderer move under src/services/gpu/renderers/, with their test mirrors under tests/services/gpu/renderers/. Imports updated at every consumer (engine phases/frame/subsystems/interaction, engine GPU handles type, click handler test). Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
skymap | e48f74d | Commit Preview URL Branch Preview URL |
May 08 2026, 01:54 AM |
Five module-header / inline doc references still pointed at pre-Spec-C paths. Refresh: - src/data/defaults.ts: milkyWayRenderer + filamentRenderer paths - shaders/filaments/fragment.wesl: cloudFade path - shaders/lib/cloudFade.wesl: 3 occurrences of cloudFade path Co-Authored-By: Claude Opus 4.7 <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.
Summary
Pure relocation of
src/services/gpu/from a flat 16-file directory into the four-subfolder taxonomy from Spec C (renderers/,passes/,resources/,labels/), withtests/services/gpu/mirrored to match. Mirrors the engine restructure that landed in #47.No file renames, no logic changes, no new barrels, no re-export shims. Only import paths shift.
Final layout
Commits (4, one per subfolder)
chore(gpu): move resources/ + testschore(gpu): move passes/ + testschore(gpu): move labels/ + testschore(gpu): move renderers/ + testsEach commit was followed by
npx tsc --noEmit && npx vitest run— all 931 tests passing at every commit boundary.Test mirror
tests/services/gpu/passes/toneMap.test.tskeeps its legacy name per the spec (pre-Spec-A artefact, separate cleanup).Test plan
npx tsc --noEmitclean at every commitnpx vitest run931 tests passing at every commit🤖 Generated with Claude Code