Repository navigation
Fix pool registry resource-owned paths - #152
Merged
Merged
Conversation
dop42
added
the
needs-in-game-test
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
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 live reconnect after adding props.register exposed the next native refusal: invalid_model_paths. The op77.152 path validator requires both registered paths to start with open77/assets//, have the expected extension, and contain neither traversal nor doubled separators. Vanilla base/... mesh paths and the previous opx/pool/... hosts do not satisfy that contract.
Register resource-owned mesh and visual-entity paths under open77/assets/opx_infinity/pool/. Require companion asset pack 0.3.1, which places the fifteen cooked hosts in that namespace and declares ArchiveXL links from fifteen namespaced mesh identities to the vanilla ball meshes. Preserve the manifest props.register permission and the full rack/camera fixes. Update the native double to reproduce invalid_model_paths for mismatched namespaces, and correct the operator/native documentation.
Validation: 68 Lua audits pass; full suite: 6,722 checks, zero failures. GitHub CI required before merge. Companion asset tests validate every registry path against a packed host and a vanilla mesh link, all cooked checksums, deterministic packaging and original cleanup guards. Actual game rendering remains unverified: fully exit the game and restart through the launcher to mount 0.3.1 after deployment. ArchiveXL must remain enabled.
Follow-up #149 and opx77-framework/opx_pool_assets#8. Documentation updated in MANUAL and POOL_MODELS.md; docs-site tracking opx77-framework/opx77_doc#19.
Companion asset pack: opx77-framework/opx_pool_assets#10