Skip to content

organize CilboxSceneBasis, add CilboxPropBasis, modify BasisLocalPlayer#553

Merged
dooly123 merged 1 commit intoBasisVR:developerfrom
BoatFloater:cilbox_shim_updates
Feb 27, 2026
Merged

organize CilboxSceneBasis, add CilboxPropBasis, modify BasisLocalPlayer#553
dooly123 merged 1 commit intoBasisVR:developerfrom
BoatFloater:cilbox_shim_updates

Conversation

@BoatFloater
Copy link
Contributor

A few cilbox related changes in this PR.

  • CilboxSceneBasis
    • I sorted all the types allowed. This should make it easier to edit and review the whitelist (well, after this PR..)
    • The only actual change to the whitelist is BasisLocalPlayer with tight restrictions:
      • GetPositionAndRotation, Teleport, Respawn, get_Instance
  • CilboxPropBasis
    • I saw that the content police for props currently reference the CilboxScene from the Cilbox package, but it's probably better to control the rules internally to the Basis project, so I created CilboxPropBasis with the exact rules from CilboxScene unchanged.
  • PropContentPoliceSelector
    • Reference CilboxPropBasis instead of CilboxScene
  • BasisLocalPlayer
    • Add GetPositionAndRotation
    • Change Instance from a field to a field accessor with private setter
      • Generally this is source-compatible but binary incompatible, so I can approach this another way if you'd like.

@dooly123 dooly123 merged commit 1590a00 into BasisVR:developer Feb 27, 2026
8 checks passed
@dooly123
Copy link
Collaborator

thankyou for the time put into this

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.

2 participants