Skip to content

refactor(runtime): keep site tools clockless#148

Merged
jBernavaPrah merged 1 commit into
mainfrom
refactor/39-clockless-tools
Jul 16, 2026
Merged

refactor(runtime): keep site tools clockless#148
jBernavaPrah merged 1 commit into
mainfrom
refactor/39-clockless-tools

Conversation

@jBernavaPrah

Copy link
Copy Markdown
Contributor

Summary

  • consume the released framework clockless-tool contract from phoxal 0.34.0
  • stop injecting PHOXAL_CLOCK into router, joypad, and telemetry in Run, Webots, and Deploy
  • remove the Webots/joypad clock-selection branch entirely
  • cover every STANDARD_SITE_TOOLS entry and deployed tool environment

Refs phoxal/.github#39

Validation

  • cargo check --workspace
  • cargo test --workspace --quiet (432 unit tests plus all integration suites; one existing manual test ignored)
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo fmt --all -- --check
  • git diff --check
  • independent cross-review: approved, no blocker

Released clean-room evidence with framework snapshot build-20260716-0000201:

  • doctor: passed
  • update: passed and selected released tool/motion/simulator artifacts
  • check --strict: passed, 18 participants
  • simulation run default --dry-run: passed
  • live simulation run default --env dev: router, joypad, and telemetry launched with no clock flag and all reached Ready; services retained --clock simulation

The live world later failed in an unchanged clocked robot participant (one run: Webots controller; another: safety). Cross-review confirmed those failures are outside this diff. They leave the tracker’s broader full-world clean-room acceptance item open for separate simulation-startup diagnosis.

Risk

The behavior change is limited to standard site-tool process environments. Services, drivers, and simulators retain their existing ParticipantLaunch clock encoding. Tool supervision/readiness is unchanged.

Cross-repository order

  1. Framework PR #268 and its corrected release PR #269 are merged.
  2. phoxal 0.34.0, phoxal-macros 0.23.0, tool artifacts, and coherent catalog build-20260716-0000201 are published.
  3. This consumer PR lands next.
  4. Canonical organization documentation follows after this merge.

@jBernavaPrah
jBernavaPrah merged commit 7f54222 into main Jul 16, 2026
4 checks passed
@jBernavaPrah
jBernavaPrah deleted the refactor/39-clockless-tools branch July 16, 2026 14:03
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