Skip to content

docs(templates): keep 3D smoke bridge read-only - #50

Closed
wh000wh000 wants to merge 11 commits into
leigest519:mainfrom
wh000wh000:fix/threed-v1-4-readonly-smoke-bridge
Closed

docs(templates): keep 3D smoke bridge read-only#50
wh000wh000 wants to merge 11 commits into
leigest519:mainfrom
wh000wh000:fix/threed-v1-4-readonly-smoke-bridge

Conversation

@wh000wh000

Copy link
Copy Markdown

结论

threed_basic verification must measure the playable route, not a shipped debug backdoor. This change freezes the existing window.__opengame3d probe as read-only and requires keyboard/mouse/DOM evidence.

Production evidence

The one-POST v1.3 game aurora-island-treasure-3d-v1-3 is genuinely playable: an independent keyboard/mouse route collected 6/6 and reached the completion screen without calling teleport. Its generated self-test nevertheless added teleport, setRemaining, and mutable state to the public bridge, creating a path for future false-positive acceptance.

Change

Path Contract added
agent-test/docs/modules/threed_basic/template_api.md Exact bridge shape: isPaused + renderer only; no state mutation helpers
agent-test/docs/modules/threed_basic/threed_basic.md Manual acceptance must use real keyboard/mouse input and visible DOM

The complementary L-layer control lives in non-Git opengame-server: check_generated_code.py --fail-bridge rejects extra runtime bridge keys for three.js games. It is intentionally not duplicated in this frozen repository.

Verification

Gate Result
git diff --check PASS
L-layer unit checks PASS: readonly fixture accepted; teleport fixture rejected
Negative production sample PASS: v1.3 is rejected with G4 keys setRemaining/state/teleport
Existing 3D sample PASS: Crystal v1.2 bridge and full v2-accept
Existing Phaser sample PASS: no new FAIL

Risk and rollback

Documentation-only frozen-layer change; runtime code and 2D routes are untouched. Revert commit b49c27f to restore prior guidance. Upstream dependencies remain PRs #47-#49.

蝉息 and others added 11 commits August 4, 2026 17:17
Add the frozen classifier/GDD contract for a minimal three.js archetype while preserving every existing Phaser route. Includes regression fixtures for explicit 3D, 2D platformer, and 2D top-down prompts.
Provide the core3d skeleton, threed_basic gameplay layer, pinned npm lockfile, and builder-facing manuals. The template keeps DOM lifecycle screens and the three-key ESC fallback while limiting assets to the existing image pipeline.
feat(3d): add threed_basic v1 production path
Keep the player on authored floor patches and outside explicit static obstacle circles without introducing a physics dependency. Substepped resolution prevents high-delta tunnelling and the self-check covers road, obstacle, and slide behavior.
Teach the builder to preserve the pure XZ resolver, explicit collision radii, and v1 no-physics boundary so generated games consume the new capability instead of bypassing it.
feat(templates): add threed_basic manual collision
The three.js renderer sizes from its container and never consumes the inherited 2D reference dimensions. Removing the unused leaves gives both standalone core3d and threed_basic a zero-dead-config baseline.
Generated v1.1 code duplicated fog, pickup, and count values. Make the builder keep one canonical path per runtime value, remove superseded aliases, and record the leaf-to-consumer map in GDD completion notes.
The universal GDD reminder contradicted threed_basic zero-dead-config rules by forbidding deletion of inherited fields. Keep the Phaser reminder unchanged while requiring consumed, single-source leaves for 3D.
fix(templates): enforce consumed 3D config
A generated v1.3 game shipped teleport and time mutation hooks solely to make verification easy. Freeze the existing probe surface and require completion evidence through real browser input so acceptance measures gameplay instead of a debug backdoor.
@wh000wh000

Copy link
Copy Markdown
Author

Superseded by #57, which rebases the complete threed_basic v1 milestone onto current origin/main and excludes unrelated fork-only changes.

@wh000wh000 wh000wh000 closed this Aug 6, 2026
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.

1 participant