Feat/1 movement foundation #5

2026-06-03T21:33:44-07:00

Dotts commented

2026-06-03 21:33:44 -07:00

No description provided.

Dotts added the movement label 2026-06-03 21:33:44 -07:00

Dotts added 39 commits 2026-06-03 21:33:44 -07:00

feat(movement): add movement state machine with air/ground/wallrun/slide/dash/ground states 202f67fa32

- MovementStateMachine: core state machine base
- MovementParams: export-rich tunable resource
- StateGround/StateAir: grounded & aeriel locomotion with coyote, buffered jump, bunny hop
- StateWallRun / StateWallCling: wall-based traversal with wall-jump off
- StateSlide: sprint-parallel slide with friction fallout
- StateDash: burst dash with momentum brake
- PlayerMovementController: entry point wiring to CharacterBody3D
- input_map.cfg: full action map reference file
- characters/player/player_base.tscn: placeholder scene
- movement/tests/test_fsm_runner.gd: 9-state unit test runner

Closes #1

fix: resolve GDScript parse errors in movement states 6cac1f4032

- state_dash.gd: Basis*Vector4 → Basis*Vector3 (no Vec4 in Godot 4.2)
- state_ground.gd: shape_owner_get_shape(0) → (0, 0); typed Variant vars; typed ray/normal
- state_slide.gd: same shape API fix + unique shape vars per branch
- state_wall_run.gd: split 'and' line into single typed bool expression

feat(debug): add TestLevel debug scene for playable testing ebb6d1a750

- scenes/test_level/test_level.tscn — playable test environment (30x30 floor + 4 walls + ramp)
  Floor at y=-1, Player spawn at (0, 1.5, 8), FpsCamera at head height (y=1.7)
  DirectionalLight + WorldEnvironment + UI label overlay (speed/state/chain)
- debug/debug_test_env.sh — launch wrapper for manual playtesting

No code changes: raw physics testbed. Intended to be opened as first scene in Godot window.

Closes #2 (test-level scene task)

fix(scene): remove broken debug script ref from test_level.tscn 9d043c02e0

Rewrote scene without ext_resource / subresource UIDs so it loads from the repo root.

fix(scene): replace BoxShape3D.new() with SubResource blocks in test_level d6ca58932e

SubResource UIDs: box_floor, box_wall_north, box_wall_east, box_wall_west
Removes parse error at line 9

fix: make test_level.tscn load without subresources; align project features to Godot 4.2 52bb5e02c7

- Removed all SubResource BoxShape3D references from test_level.tscn
- Changed PackedStringArray feature from 4.6 to 4.2
Scene is now pure nodes + transforms, no external resource IDs

fix(scene): collision + light scale in TestLevel f2bcbb6ac2

- Use numeric SubResource(id) instead of uid:// strings (avoids parser error)
- Light transform: remove non-uniform scale, set only rotation + position
- Keep floor@y=-1.1, walls thin (0.4), PlayerSpawn unchanged

fix: make TestLevel load on 4.6.3; build geometry from GDScript 5a45f3e9bb

- test_level.tscn: minimal Node3D root (zero SubResource references)
- debug/test_level_builder.gd: runtime floor/walls/lighting/camera/UI
  shapes/meshes created from BoxShape3D/BoxMesh — no .tscn subresources
- project.godot: revert features to 4.6 (installed build is 4.6.3)

fix(debug): multi-line string literal in test_level_builder.gd 559c4bde8f

fix(movement): typed Vector3 infer in state_wall_run.detect_wall() eeb074e58a

test: full builder rewrite with player state machine, camera, and manual _ready() bb6cac9d7a

- _build_player: creates Player (CharacterBody3D) with capsule,
  MovementStateMachine + 6 state children, attaches movement script
- Calls player._ready() manually after set_script (GDScript
  set_script does not auto-trigger _ready)
- Camera parented under Player so it follows movement
- Restores _build_camera() which was accidentally dropped

fix: unused param warning + @onready type mismatch for MovementStateMachine dfed237a63

- state_wall_run.gd: typed ray_origin/ray_end as Vector3
- test_level_builder.gd: restructure builder to put MovementStateMachine
  + state children under Player, attach camera under Player too
- player_movement_controller.gd: @onready uses get_node so type is not inferred

fix: avoid typed @onready infer crash for MovementStateMachine 872201aa85

player_movement_controller.gd: use Node + get_node() instead of
typed MovementStateMachine + $ shorthand so runtime set_script paths
don't crash on @implicit_ready

fix: drop type hint on @onready _machine so runtime set_script path accepts MovementStateMachine 9bdbc69ddd

test: load MovementStateMachine + state scripts via set_script() at runtime 1feefbf811

Root cause: builder created plain Node instances for the state machine
and states, so _machine.player = self in PlayerMovementController
crashed with 'Invalid assignment ... on base object of type Node'.
Now each node gets its real script attached before _ready() runs.

fix: add smoke verifier, camera to TestLevel, fix player collision shape ceeff6a1ab

fix: guard input forwarding in PlayerMovementController; add movement_runner verifier 24a8651a6d

fix: wire movement controller before state machine; harden machine lookup c0a90df690

refactor: remove print_debug and switch to preload for movement scripts 693145fd69

fix: replace movement controller with parse-safe version for 4.6.3 60091932bd

fix: manually bootstrap state machine after runtime script attachment ea7169048f

test: self-bootstrap InputMap actions in TestLevelBuilder f47fc4827a

fix: rewrite TestLevelBuilder with parse-safe InputMap bootstrapping 6d2f31d962

revert bogus debug edit in state_ground; restore clean update() entry c463c29031

fix: ensure player._ready() runs before MovementStateMachine._ready() 9c39d79696

fix: clean state_ground and ensure duck-type state machine bootstrap 6c57f70fb4

debug: log movement controller inputs/state/velocity each frame 5cb2013d6c