Prepare FutolStructure public launch#5
Merged
Conversation
…ve/load data loss Phase 1 stabilization before modularization: - Remove 8 duplicate function definitions (finishBeamDraw, toggleSlabVoid x2, getFloorCustomBeams, generateETABSContent, exportToETABS, generateIFCGUID, generateIFCContent, exportToIFC) — kept last (active) definition of each - Remove misplaced IFC material code injected inside init3D() by upgrade script that caused ReferenceError on nid/ifc/ownerHistoryId, breaking 3D view entirely - Fix save/load data loss: saveProject() now serializes per-floor cantilevers, customBeams, voidSlabs, deletedBeams, deletedColumns, lockedBeams, lockedSlabs; loadProject() initializes missing fields for backward compat with old saves - Rename first ETABS/IFC generator blocks to _superseded_*_v1 (dead code, kept for reference until Phase 2 cleanup) https://claude.ai/code/session_017DPBHvPnz7ubhZozZUn6Sx
- Add deletedColumns, lockedBeams, lockedSlabs to addFloor(), toggleGFSuspended(), and resetProject() floor creation paths to match authoritative floor shape in state init and save/load - Eliminate temporary state.cantilevers swap-and-restore pattern from calculate() per-floor loop, display regeneration, and STAAD export by threading floor cantilevers explicitly through generateBeams() → generateCantileverBeams(floorCantilevers) - This removes the failure mode where an exception inside the calculate loop would leave state.cantilevers permanently wrong https://claude.ai/code/session_017DPBHvPnz7ubhZozZUn6Sx
Extract 9 pure structural calculation functions from the monolith into engine/loads.js (IIFE pattern, zero DOM deps, zero state refs). Replace all 10 inline call sites with thin delegating wrappers via getLoadParams(). Add createFloor() factory used at all 4 floor creation sites, fixing missing wallLoad field in addFloor() and toggleGFSuspended(). Add dumpStateSnapshot() for regression checking. https://claude.ai/code/session_017DPBHvPnz7ubhZozZUn6Sx
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
michaelfutol
changed the title
michaelfutol
changed the title
michaelfutol
marked this pull request as ready for review
July 12, 2026 21:31
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
Validation
Deployment