examples: polish the demo, contraption builder, and slingshot#75
Merged
Conversation
…ally verified) A safe-cosmetic polish pass over the non-platformer example stacks, applying the lessons banked from the platformer (build-once chrome, the performance playbook, the OXT gotchas). All changes are example-side (outside the embedded Kit), pass tools/check-livecodescript.py, and are flagged for an OXT feel/look pass. Demo (box2dxt-demo): - Fix a gotcha-11 landmine: hoist the nested `local tOldest` in registerDrop to the top of the handler (a block-nested local can break OXT compilation; the static checker does not catch it). - Lidar scene now fills the whole arena (spawn bounds derived from gArenaL/R instead of hardcoded literals that clustered the bodies left-of-centre). - Seed the FPS readout so the first HUD frame isn't blank. Contraption builder (box2dxt-contraption-builder): - Remove the dead `partTabs` function (a never-called near-duplicate of partTabsForInspector). - tickHud reads `the milliseconds` once per pass into a local (the "one clock read per pass" rule) instead of up to four times. Slingshot (box2dxt-slingshot): - A built-once SKY BACKDROP (sky + horizon haze + sun + two hills), persistent decor behind every level, created at the back of the chrome and left in place by sgWipeStage; bumps kSgUIVersion 2 -> 3 so older saved stacks rebuild once. - The aim-preview dots now clamp to the turf instead of burying into the ground. - Star-rated level-clear cards (from leftover ammo) and a star flourish on the win card. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01KcQmrtT7Vun4pwnkvHbhxX
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.
A safe-cosmetic polish pass over the three non-platformer example stacks, applying the lessons banked from the platformer. Three Opus reviewers combed each stack; this PR lands the high-confidence, statically-safe wins. Every change is example-side (outside the embedded Kit), passes
tools/check-livecodescript.py+sync-embedded-kit.py --check, and is flagged for an OXT feel/look pass — none of it could be run here, so the runtime confirmation is yours.Demo (
box2dxt-demo)local tOldestinregisterDropto the handler top. A block-nestedlocalcan break OXT compilation, and the static checker doesn't catch it — this was a real latent risk.gArenaL/Rinstead of hardcoded literals that clustered the bodies left-of-centre (the right strip was always empty).Contraption builder (
box2dxt-contraption-builder)Already playbook-clean — only two items:
partTabsfunction (a never-called near-duplicate ofpartTabsForInspector, ~30 lines).tickHudreads the clock once per pass into a local (the "one clock read per pass" rule) instead of up to 4×.Slingshot (
box2dxt-slingshot)The graphics-only game with the most visual headroom:
sgWipeStageacross level rebuilds, zero per-frame cost. BumpskSgUIVersion2 → 3 so older saved stacks rebuild once.Deliberately deferred to your OXT pass (riskier / additive — not in this PR)
spawnRingeffect graphic (live render path).●○spent indicator (the loaded-shot count has a quirk I didn't want to risk blind), tower texturing, and screen-shake (b2kCamShakeis a no-op without a camera).Sky-backdrop note
The hill/sun rects are placed by calculation, not by eye — please confirm/tune their framing on the OXT pass. They're pure decor (no physics body), so worst case is a cosmetic nudge.
🤖 Generated with Claude Code
https://claude.ai/code/session_01KcQmrtT7Vun4pwnkvHbhxX
Generated by Claude Code