Consolidate workspace + refactor biome visualization engine#25
Merged
Conversation
Made-with: Cursor
… consolidate shaders, simplify handlers - Extract duplicated code between WebGL/WebGPU engines into engine-shared.ts (~80 lines removed) - Eliminate 12+ per-frame heap allocations in 60fps render loop (cached Color, Fog, scratch vectors) - Move ~120 lines of inline GLSL into shaders/index.ts barrel with standalone .glsl reference files - Replace ~25 boilerplate event handlers with 4 factory functions in +page.svelte (~140 lines reduced) Net change: -491 / +184 lines across 6 modified files + 2 new modules + 11 .glsl files https://claude.ai/code/session_018k8MaN75eTLbCfA8zrtGfe
…le checkout to CI - Remove 29 submodule entries from git index that had no .gitmodules mapping (caused `fatal: no submodule mapping found` during checkout) - Regenerate uv.lock to resolve empirica from PyPI instead of stale `_unified/empirica` editable path that no longer exists - Add `submodules: recursive` to all 3 CI checkout steps https://claude.ai/code/session_018k8MaN75eTLbCfA8zrtGfe
The xuehai template submodule has broken nested submodules that cause recursive checkout to fail. Since CI only runs Python tests, linting, and project verification, no submodules are needed. https://claude.ai/code/session_018k8MaN75eTLbCfA8zrtGfe
awakening_app was referenced on line 1041 but never imported, causing NameError on `waft verify` in CI. https://claude.ai/code/session_018k8MaN75eTLbCfA8zrtGfe
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
2 participants
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
integration/all-code-20260327branch — oracle-cycle endpoints, sitrep hub guardrails, biome visualization engine, meme lab, cognitive prosthetics CLI, pantheon, promotion review, visualizer packages, tests, docs, scripts, and work effortsef5ab266): Extracts shared utilities, fixes per-frame allocations, consolidates shaders, and simplifies event handlers across the 3D aquatic visualization engine92322718): Standalone configurator updateBiome refactor details
engine-shared.ts— shared constants, presets, debug helpers, ripple logicColor,Fog, scratchVector3in both engines + causticsshaders/index.tsbarrel + 11 standalone.glslreference filesslider(),checkbox(),select(),textInput()in+page.svelteNet biome change: -491 / +184 lines across 6 modified files + 13 new files
Key biome files changed
visualizer/src/lib/biome/engine.tsvisualizer/src/lib/biome/engine-webgpu.tsvisualizer/src/lib/biome/caustics.tsvisualizer/src/lib/biome/rain-system.tsvisualizer/src/lib/biome/engine-shared.ts(new)visualizer/src/lib/biome/shaders/index.ts(new)visualizer/src/routes/biome/+page.svelteTest plan
npm run checkinvisualizer/— 0 TS errors in biome files (confirmed)/biomeroute — water renders, orbit controls work, rain overlay visible?debugBiome=1query param logginghttps://claude.ai/code/session_018k8MaN75eTLbCfA8zrtGfe