Skip to content

fix(physics): var RigidBody param for rigid integrate + composable smoke#68

Closed
cap-jmk-real wants to merge 8 commits into
mainfrom
cursor/rigid-var-param-57b4
Closed

fix(physics): var RigidBody param for rigid integrate + composable smoke#68
cap-jmk-real wants to merge 8 commits into
mainfrom
cursor/rigid-var-param-57b4

Conversation

@cap-jmk-real
Copy link
Copy Markdown
Contributor

Summary

rigid_integrate_semi_implicit mutates its body in place via b: var RigidBody; composable import_physics_runtime.li runs one semi-implicit gravity step after import.

Agent continuation

  1. Read packages/li-physics-rigid/src/lib.li and li-tests/composable/import_physics_runtime.li.
  2. Run CI check after merge.
  3. Next: composable smoke for physics.runtime / physics_step if PhysicsWorld needs var world params.
  4. Blocked on: publishing li-physics-* org mirrors (lic [Ecosystem gap] Publish li-std-physics-* org mirrors (12 packages) #50).

Breaking

N/A

Release note: docs/release-notes/2026-05-19-rigid-var-param-composable.md

rigid_integrate_semi_implicit mutates the body in place; use b: var RigidBody
so borrowck does not mark caller locals moved. Composable import_physics_runtime
runs one gravity substep after version check.
Pass holder.body to rigid_integrate_semi_implicit so borrowck does not mark
a local RigidBody moved (check_call_moves only tracks Ident args).
Use physics.runtime package API without referencing imported object types
in the fixture (RigidBody-local integrate blocked in composable build).
CI shows composable compile_ok fails when fixtures use imported object
types or physics_world_* calls; keep import_physics_runtime as rigid
version smoke until compiler/import surface is fixed.
@cap-jmk-real
Copy link
Copy Markdown
Contributor Author

Superseded by #69 — root cause was workspace members parse + multiline def params, not borrowck-only. Composable rigid integrate smoke is on #69.

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