From 08b7a4fdd923fd2bab167ac5270111f365282575 Mon Sep 17 00:00:00 2001 From: Bryan Helmkamp Date: Fri, 18 Sep 2026 22:17:15 -0400 Subject: [PATCH] Record the snapshots the Petri pin and the record positions changed The pin to Petri 639ce3e moves the run's format version from 6 to 7, which the attach JSON snapshot records. The other eight snapshots had recorded the run branch and Git identity lines before the Start stage's completion: the order the clock gave them before "Place the run branch and git identity records with their checkpoint" positioned the two records after the firing's finish. That commit refreshed only two files, and these eight already differed the same way at the commit before the pin bump; they now record the one order every run produces. Co-Authored-By: Claude Fable 5.1 --- lib/apps/fabro-cli/tests/it/cmd/attach.rs | 6 +++--- lib/apps/fabro-cli/tests/it/cmd/events.rs | 2 +- lib/apps/fabro-cli/tests/it/cmd/run.rs | 2 +- lib/apps/fabro-cli/tests/it/workflow/dry_run_examples.rs | 6 +++--- 4 files changed, 8 insertions(+), 8 deletions(-) diff --git a/lib/apps/fabro-cli/tests/it/cmd/attach.rs b/lib/apps/fabro-cli/tests/it/cmd/attach.rs index 0475dd88c..f08238db6 100644 --- a/lib/apps/fabro-cli/tests/it/cmd/attach.rs +++ b/lib/apps/fabro-cli/tests/it/cmd/attach.rs @@ -392,8 +392,8 @@ fn attach_replays_completed_detached_run() { ----- stdout ----- ----- stderr ----- Web UI: http://localhost:3000/runs/[ULID] - Base: [BASE] ✓ Start [TIME] + Base: [BASE] ✓ Run Tests [TIME] ✓ Report [TIME] ✓ Exit [TIME] @@ -637,8 +637,8 @@ fn attach_before_completion_streams_to_finished_state() { ----- stdout ----- ----- stderr ----- Web UI: http://localhost:3000/runs/[ULID] - Base: [BASE] ✓ start [DURATION] + Base: [BASE] ✓ wait [DURATION] ✓ exit [DURATION] "); @@ -904,7 +904,7 @@ fn attach_json_errors_without_prompting_for_human_input() { "recorded_at": "[EPOCH_MS]", "body": { "event": "run.started", - "format_version": 6, + "format_version": 7, "key": "[ULID]", "root": 0, "middleware_chain": [ diff --git a/lib/apps/fabro-cli/tests/it/cmd/events.rs b/lib/apps/fabro-cli/tests/it/cmd/events.rs index 15e8ff634..ada175646 100644 --- a/lib/apps/fabro-cli/tests/it/cmd/events.rs +++ b/lib/apps/fabro-cli/tests/it/cmd/events.rs @@ -215,9 +215,9 @@ fn events_pretty_formats_small_run() { [CLOCK] · running [CLOCK] Engine: petri run started [CLOCK] ▶ Start + [CLOCK] ✓ Start [DURATION] [CLOCK] Branch: fabro/run/[ULID] from [SHA] [CLOCK] Git identity: Fabro default - [CLOCK] ✓ Start [DURATION] [CLOCK] ⎘ Checkpoint [SHA] [CLOCK] ▶ Run Tests [CLOCK] start → run_tests continue diff --git a/lib/apps/fabro-cli/tests/it/cmd/run.rs b/lib/apps/fabro-cli/tests/it/cmd/run.rs index 04ef6015f..4c5a7a3c2 100644 --- a/lib/apps/fabro-cli/tests/it/cmd/run.rs +++ b/lib/apps/fabro-cli/tests/it/cmd/run.rs @@ -836,8 +836,8 @@ fn dry_run_simple() { ----- stderr ----- Run: [ULID] Web UI: http://localhost:3000/runs/[ULID] - Base: [BASE] ✓ Start [TIME] + Base: [BASE] ✓ Run Tests [TIME] ✓ Report [TIME] ✓ Exit [TIME] diff --git a/lib/apps/fabro-cli/tests/it/workflow/dry_run_examples.rs b/lib/apps/fabro-cli/tests/it/workflow/dry_run_examples.rs index 49170e99e..d01dee62f 100644 --- a/lib/apps/fabro-cli/tests/it/workflow/dry_run_examples.rs +++ b/lib/apps/fabro-cli/tests/it/workflow/dry_run_examples.rs @@ -16,8 +16,8 @@ fn dry_run_branching() { ----- stderr ----- Run: [ULID] Web UI: http://localhost:3000/runs/[ULID] - Base: [BASE] ✓ Start [TIME] + Base: [BASE] ✓ Plan [TIME] ✓ Implement [TIME] ✓ Validate [TIME] @@ -48,8 +48,8 @@ fn dry_run_conditions() { ----- stderr ----- Run: [ULID] Web UI: http://localhost:3000/runs/[ULID] - Base: [BASE] ✓ start [TIME] + Base: [BASE] ✓ Decide [TIME] ✓ Path B [TIME] ✓ exit [TIME] @@ -145,8 +145,8 @@ fn dry_run_inferred_command() { ----- stderr ----- Run: [ULID] Web UI: http://localhost:3000/runs/[ULID] - Base: [BASE] ✓ Start [TIME] + Base: [BASE] ✓ Echo [TIME] ✓ Exit [TIME]