diff --git a/run.json b/run.json
index b5997e042..41c46a2f9 100644
--- a/run.json
+++ b/run.json
@@ -505,7 +505,7 @@
     "kind": "running"
   },
   "status_updated_at": "2026-06-03T15:55:32.289943Z",
-  "last_event_at": "2026-06-03T17:03:55.857368Z",
+  "last_event_at": "2026-06-03T17:12:22.695009Z",
   "pending_control": null,
   "checkpoints": [
     {
@@ -1361,9 +1361,9 @@
       }
     },
     {
-      "seq": 0,
+      "seq": 1449,
       "checkpoint": {
-        "timestamp": "2026-06-03T17:11:59.403168Z",
+        "timestamp": "2026-06-03T17:12:22.694599Z",
         "current_node": "verify",
         "completed_nodes": [
           "start",
@@ -1377,87 +1377,45 @@
         ],
         "node_retries": {},
         "context_values": {
-          "internal.run_id": "01KT734BXFV007VWXT3G5PFXYA",
-          "response.simplify_gpt": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues found.\n  - Code quality: no issues found.\n  - Efficiency: no issues found.\n- No fixes were needed.\n- Final `git status --short` is clean.",
-          "thread.simplify_gpt.current_node": "verify",
-          "internal.work_dir": "/home/daytona/workspace/fabro",
-          "response.simplify_opus": "All frontend tests pass.\n\n## Summary\n\nReviewed both commits with three parallel agents (reuse, quality, efficiency). Findings triaged and applied:\n\n**Fixed:**\n- **Inconsistent indentation** in `apps/fabro-web/app/routes/run-files.render.test.tsx` (`created_by` fixture had under-indented inner keys) — re-indented to match surrounding nesting.\n- **Inconsistent `<EmptyValue />` indentation** in `apps/fabro-web/app/components/run-summary-panel.tsx` — the two cells touched by the diff had been fixed, but three other cells (Sandbox, Cost, Artifacts) still had the same under-indented `<EmptyValue />`. Made all five consistent.\n\n**Skipped (with rationale):**\n- *Duplicated `test_run_provenance` in `fabro-workflow/src/test_support.rs`* — intentionally uses `Principal::System { Engine }` rather than a user, since workflow-engine tests model the engine itself as the creator. Not a true duplicate.\n- *Unifying `test_principal()` across crates* — existing per-crate helpers use the `https://github.com` issuer with `AuthMethod::Github`, while the new `fabro_types::test_support::test_principal()` deliberately uses `fabro:test`/`DevToken`. Consolidating would require widening the helper API beyond the scope of this PR.\n- *Match arm `None | Some(Principal::Agent { .. } | Principal::System { .. })` in `server.rs`* — exhaustive, concise, and exhaustiveness-checked over `Principal` variants. Splitting would add noise.\n- *Stringly-typed `\"none\"` for `principal_kind`* — used in exactly one place; a named constant would be over-engineering.\n- *`<EmptyValue />` fallback on \"Created by\"* — still reachable when `run` is undefined but not loading (e.g. error states), so not dead code.\n- *Per-request `DEMO_PRINCIPAL.clone()` allocations in demo summaries* — pre-existing pattern, the `LazyLock` already eliminates the per-request `IdpIdentity::new` validation cost. Promoting summaries to a static `Vec<Run>` is a larger refactor unrelated to this PR's scope.\n\nFrontend `typecheck` and `bun test` (508 pass) both green.",
-          "internal.retry_count.start": 0,
-          "failure_signature": "",
-          "last_response": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues",
-          "last_stage": "simplify_gpt",
-          "internal.retry_count.preflight_compile": 0,
-          "response.implement": "Implemented the two-commit plan and committed both changes.\n\nCommits:\n- `43a12a3e1 refactor(auth): remove anonymous principal`\n- `9ae80132d refactor(runs): require creator provenance`\n\nSummary:\n- Removed `Principal::Anonymous`; unauthenticated/invalid request state is now represented as `None`, with HTTP logs using `principal_kind=\"none\"`.\n- Made persisted run actors/provenance total:\n  - `Run.created_by: Principal`\n  - `RunSpec.provenance: RunProvenance`\n  - `RunProvenance.subject: Principal`\n  - `run.created` provenance required/non-null\n- Updated OpenAPI and regenerated Rust/TS API outputs.\n- Added `fabro-types::test_support` helpers behind `test-support`.\n- Updated server/workflow/store projections, create/retry flows, demo principal, frontend UI/data assumptions, and test fixtures/snapshots.\n- Final working tree is clean.\n\nVerification run:\n- `cargo build -p fabro-api`\n- `cd lib/packages/fabro-api-client && bun run generate`\n- `cd lib/packages/fabro-api-client && bun run typecheck`\n- `cd apps/fabro-web && bun run typecheck && bun test && bun run build`\n- `cargo +nightly-2026-04-14 fmt --check --all`\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings`\n- `cargo build --workspace`\n- `cargo nextest run --workspace`\n- `cargo nextest run -p fabro-server`\n- Final sweeps for anonymous principal and nullable creator/provenance patterns:\n  - only unrelated `crossorigin=\"anonymous\"` and unrelated todo/web-auth `subject` fields remain.\n\nNotes:\n- `cargo insta pending-snapshots` could not run because `cargo-insta` is not installed in this environment; I checked for pending `*.snap.new` files and found none.\n- Manual browser/server E2E verification was not performed in this environment.",
-          "internal.fidelity": "compact",
-          "graph.model_stylesheet": "\n            * { model: claude-opus-4-7; }\n        ",
           "internal.thread_id": "simplify_gpt",
-          "thread.simplify_opus.current_node": "simplify_gpt",
-          "internal.retry_count.toolchain": 0,
+          "response.simplify_gpt": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues found.\n  - Code quality: no issues found.\n  - Efficiency: no issues found.\n- No fixes were needed.\n- Final `git status --short` is clean.",
+          "failure_class": "",
           "internal.retry_count.simplify_gpt": 0,
           "command.output": "blob://sha256/14a657f055256f714ba70d6898170e78b8ad9d9fac61f27d4b1445719ab6b2d5",
+          "internal.work_dir": "/home/daytona/workspace/fabro",
           "internal.retry_count.simplify_opus": 0,
-          "graph.goal": "# Plan: Make run actors and provenance total\n\n## Context\n\nThis is a greenfield app. Backward compatibility with old serialized runs, old API clients, old generated models, and old tests is not a constraint. Prefer the clean invariant and remove all traces of the placeholder shape.\n\n`Principal::Anonymous` currently represents \"no authenticated actor on this request\" inside auth middleware. That is auth state, not an actor. A `Principal` should only mean \"who acted.\"\n\nLikewise, a persisted run should always have a creator. `Run.created_by`, `RunSpec.provenance`, `RunProvenance.subject`, and `run.created` event provenance should all be total. No `Option<Principal>`, no nullable OpenAPI fields, no legacy deserialization defaults, and no fallback creator in projection code.\n\nTwo commits, in order.\n\n---\n\n## Commit 1 - Remove `Principal::Anonymous`\n\nBreaking cleanup. `Principal` becomes actor-only. Missing/invalid auth is represented as absent request principal, not as an anonymous principal variant.\n\n### Rust\n\n`lib/crates/fabro-types/src/principal.rs`:\n- Drop `Anonymous`.\n- Drop `Anonymous` arms in `kind()` and `display()`.\n- Delete anonymous serialization/round-trip test coverage.\n\n`lib/crates/fabro-server/src/principal_middleware.rs`:\n- `RequestAuthContext.principal: Principal` -> `Option<Principal>`.\n- `RequestAuthLogContext.principal: Principal` -> `Option<Principal>`.\n- `initial()` and `rejected()` set `principal: None`.\n- `authenticated(...)`, `authenticated_worker(...)`, and `authenticated_user(...)` set `principal: Some(...)`.\n- Update `principal_without_log_unused_fields` to preserve `None` and strip user avatar data only inside `Some(Principal::User(...))`.\n- Update all gate helpers to match `Option<Principal>`:\n  - `require_user`\n  - `require_authenticated_user`\n  - `require_run_management_actor`\n  - `require_worker_or_user_for_run`\n  - `require_run_management_target`\n- `None` routes to the existing `auth_rejection(context.auth_status, context.auth_error_code)` behavior.\n- `Some(Principal::Worker { .. })` keeps the current forbidden-vs-auth-rejection distinctions.\n- Update tests that assert the initial/rejected principal to assert `None`.\n\n`lib/crates/fabro-server/src/server.rs` HTTP logging:\n- Keep the `principal_kind` field on every HTTP log line.\n- Compute `principal_kind` as `auth_context.principal.as_ref().map(Principal::kind).unwrap_or(\"none\")`.\n- Match `auth_context.principal` as an `Option<Principal>`:\n  - `Some(User(...))`, `Some(Worker { ... })`, `Some(Webhook { ... })`, `Some(Slack { ... })` keep their extra fields.\n  - `None | Some(Agent { .. } | System { .. })` emits only the common HTTP fields.\n\n`docs/internal/logging-strategy.md`:\n- Replace the `anonymous` HTTP caller category guidance with `none` for requests that have no principal.\n- Keep `auth_status` as the field that distinguishes missing, invalid, expired, and authenticated auth state.\n\n### OpenAPI and generated clients\n\n`docs/public/api-reference/fabro-api.yaml`:\n- Remove `PrincipalAnonymous` from the `Principal` `oneOf`.\n- Remove `anonymous` from the `Principal` discriminator mapping.\n- Delete the `PrincipalAnonymous` schema.\n\nRegenerate:\n- `cargo build -p fabro-api`\n- `cd lib/packages/fabro-api-client && bun run generate`\n\nExpected generated cleanup:\n- `lib/packages/fabro-api-client/src/models/principal-anonymous.ts` disappears.\n- `Principal` union no longer includes `{ kind: \"anonymous\" }`.\n- `lib/packages/fabro-api-client/src/models/index.ts` no longer exports `principal-anonymous`.\n\n### Frontend\n\n`apps/fabro-web/app/lib/principal-display.tsx`:\n- Remove the `\"anonymous\"` switch case and unused icon import.\n\n`apps/fabro-web/app/components/run-summary-panel.test.tsx` and API-client exhaustiveness tests:\n- Remove anonymous principal cases.\n\n### Documentation sweep\n\nRemove anonymous-principal references from product/API docs and tests. Be careful not to touch unrelated uses of \"anonymous\" such as telemetry anonymous IDs or Git's `remote_anonymous` API.\n\nUseful sweep:\n- `rg -n \"Principal::Anonymous|PrincipalAnonymous|kind: 'anonymous'|kind: \\\"anonymous\\\"|anonymous actor|anonymous subject|principal_kind.*anonymous|\\\"anonymous\\\"\" lib/crates apps/fabro-web lib/packages/fabro-api-client docs/public docs/internal`\n\n### Verification\n\n- `cargo +nightly-2026-04-14 fmt --check --all`\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings`\n- `cargo build --workspace`\n- `cargo nextest run --workspace`\n- `cd apps/fabro-web && bun run typecheck && bun test`\n- Manual: start `fabro server start`, hit a protected endpoint without a token, confirm 401 and an HTTP log with `principal_kind=\"none\"` and `auth_status=\"missing\"`.\n\n---\n\n## Commit 2 - Make run provenance and creator non-optional\n\nFull-chain invariant. Every persisted run has exactly one creator principal. No nullable schema fields, no legacy defaults, no projection fallbacks.\n\n### Core type changes\n\n`lib/crates/fabro-types/src/run_summary.rs`:\n- `Run.created_by: Option<Principal>` -> `Principal`.\n- Drop `#[serde(default)]`.\n\n`lib/crates/fabro-types/src/run.rs`:\n- `RunProvenance.subject: Option<Principal>` -> `Principal`.\n- Drop `#[serde(default, skip_serializing_if = \"Option::is_none\")]`.\n- Drop `Default` derive on `RunProvenance`.\n- `RunSpec.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- Drop `#[serde(default, skip_serializing_if = \"Option::is_none\")]` on `RunSpec.provenance`.\n\n`lib/crates/fabro-types/src/run_event/run.rs`:\n- `RunCreatedProps.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- Drop default/skip serialization attributes for provenance.\n\n`lib/crates/fabro-workflow/src/event/events.rs`:\n- `Event::RunCreated.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- Drop default/skip serialization attributes for provenance.\n\n### Creation and retry flow\n\n`lib/crates/fabro-workflow/src/operations/create.rs`:\n- `CreateRunInput.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- `PersistCreateOptions.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- `RunSpec { provenance }` stores the total provenance directly.\n- `Event::RunCreated { provenance }` emits total provenance directly.\n\n`lib/crates/fabro-server/src/server/handler/runs.rs`:\n- `run_provenance(headers, subject)` returns `RunProvenance { subject: subject.clone(), ... }`.\n- Build provenance before creating `CreateRunInput`.\n\n`lib/crates/fabro-server/src/run_manifest.rs`:\n- Change `create_run_input(...)` to accept `provenance: RunProvenance` and set it directly, or stop using the helper for the final `CreateRunInput` construction. Do not create a temporary input with missing provenance.\n\n`lib/crates/fabro-workflow/src/operations/retry.rs`:\n- `RetryRunInput.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- `retry_run(...)` writes the new run's `run.created` event with total provenance.\n\n`lib/crates/fabro-server/src/server/handler/lifecycle.rs`:\n- Pass `run_provenance(&headers, &actor)` directly into `RetryRunInput`.\n\n### Event conversion and projections\n\n`lib/crates/fabro-workflow/src/event/convert.rs`:\n- Convert `Event::RunCreated.provenance` into `RunCreatedProps.provenance` directly.\n- Remove `Some(...)` wrapping for run-created provenance.\n\n`lib/crates/fabro-workflow/src/event/stored_fields.rs`:\n- `Event::RunCreated { provenance, .. }` sets `actor: Some(provenance.subject.clone())`.\n\n`lib/crates/fabro-store/src/run_state.rs`:\n- `projection_from_created(...)` builds `RunSpec { provenance: props.provenance.clone(), ... }`.\n- `build_summary(...)` sets `created_by: state.spec.provenance.subject.clone()`.\n- Delete or rewrite tests that deserialize projections with `\"provenance\": null`.\n\n`lib/crates/fabro-types/src/run_projection.rs` and projection tests:\n- Replace all test `RunSpec` literals with total provenance.\n- Remove tests whose only purpose is legacy/null provenance tolerance.\n\n### OpenAPI\n\n`docs/public/api-reference/fabro-api.yaml`:\n- `Run.created_by` references `Principal` directly. Remove `oneOf [..., null]`.\n- `RunProvenance.required` includes `subject`.\n- `RunProvenance.subject` references `Principal` directly. Remove `oneOf [..., null]`.\n- `RunSpec.required` includes `provenance`.\n- `RunSpec.provenance` references `RunProvenance` directly. Remove `oneOf [..., null]`.\n- If `run.created` event properties are represented separately in the spec, make that event provenance required and non-nullable too.\n\nRegenerate:\n- `cargo build -p fabro-api`\n- `cd lib/packages/fabro-api-client && bun run generate`\n\nDo not hand-edit generated client files.\n\n### Demo mode\n\n`lib/crates/fabro-server/src/demo/mod.rs`:\n- Add a clearly synthetic demo principal using `AuthMethod::DevToken`, not GitHub:\n  ```rust\n  static DEMO_PRINCIPAL: LazyLock<Principal> = LazyLock::new(|| {\n      Principal::user(\n          IdpIdentity::new(\"fabro:demo\", \"demo\").unwrap(),\n          \"demo\".to_string(),\n          AuthMethod::DevToken,\n      )\n  });\n  ```\n- Replace `created_by: None` with `created_by: DEMO_PRINCIPAL.clone()`.\n- If demo creates any full `RunSpec` or `run.created` event data, give it `RunProvenance { subject: DEMO_PRINCIPAL.clone(), ... }`.\n\n### Test support\n\nDo not add fake auth helpers to `fabro_types::fixtures`; that module is run-id constants.\n\nUse the existing `fabro-types` `test-support` feature:\n- Add `#[cfg(any(test, feature = \"test-support\"))] pub mod test_support;` in `lib/crates/fabro-types/src/lib.rs` if it does not already exist.\n- Add `lib/crates/fabro-types/src/test_support.rs` with:\n  - `test_principal() -> Principal`\n  - `test_run_provenance() -> RunProvenance`\n- Use an obviously fake dev-token identity, e.g. issuer `fabro:test`, subject `test-user`, login `test`.\n- In crates that need the helper from integration tests or cross-crate tests, dual-list `fabro-types` in `dev-dependencies` with `features = [\"test-support\"]`, following existing repo patterns.\n\nUpdate all constructors:\n- Replace `provenance: None` in `RunSpec`, `CreateRunInput`, `RetryRunInput`, `Event::RunCreated`, and `RunCreatedProps` literals with `test_run_provenance()` or a locally meaningful provenance.\n- Replace `subject: Some(...)` with `subject: ...`.\n- Replace `subject: None` only when it is actually `RunProvenance.subject`; leave unrelated todo/commit/message `subject` fields alone.\n- Replace `created_by: None` / `created_by: null` with `test_principal()` or a frontend TS principal fixture.\n- Delete tests that assert nullable or omitted creator/provenance behavior.\n\nRepresentative Rust areas:\n- `lib/crates/fabro-store/src/run_state.rs`\n- `lib/crates/fabro-store/tests/serializable_projection.rs`\n- `lib/crates/fabro-workflow/src/operations/{create,retry,start}.rs`\n- `lib/crates/fabro-workflow/src/event/{convert,sink,stored_fields}.rs`\n- `lib/crates/fabro-workflow/src/handler/**`\n- `lib/crates/fabro-workflow/src/pipeline/**`\n- `lib/crates/fabro-workflow/src/run_{lookup,metadata}.rs`\n- `lib/crates/fabro-server/src/server/tests.rs`\n- `lib/crates/fabro-server/src/server/handler/**`\n- `lib/crates/fabro-server/tests/it/**`\n- `lib/crates/fabro-cli/tests/it/support/mod.rs`\n- `lib/crates/fabro-dump/src/lib.rs`\n- `lib/crates/fabro-tool/src/{common,create,interact,search}.rs`\n- `lib/crates/fabro-api/tests/{principal_round_trip,run_summary_round_trip,run_projection_round_trip,run_event_round_trip}.rs`\n- `lib/crates/fabro-types/tests/{run_spec_serde,run_spec_methods,run_event_serde}.rs`\n\nRepresentative TypeScript areas:\n- `apps/fabro-web/app/**` tests with `created_by: null`\n- `apps/fabro-web/app/data/runs.ts`\n- `apps/fabro-web/app/components/run-summary-panel.tsx`\n- `apps/fabro-web/app/components/runs-list/**`\n- `lib/packages/fabro-api-client/tests/principal-exhaustive.ts`\n\nUseful sweep after edits:\n- `rg -n \"Principal::Anonymous|PrincipalAnonymous|principal-anonymous|kind: ['\\\"]anonymous|created_by:\\\\s*(None|null)|provenance:\\\\s*None|subject:\\\\s*Some\\\\(|subject:\\\\s*None\" lib/crates apps/fabro-web lib/packages/fabro-api-client docs/public docs/internal`\n\nReview each hit. The only acceptable remaining matches should be unrelated uses of \"anonymous\" and unrelated non-principal `subject` fields.\n\n### Frontend\n\n`apps/fabro-web/app/components/run-summary-panel.tsx`:\n- `run?.created_by` may still be guarded by `run` loading state, but `created_by` itself is non-null once `run` exists.\n- Pass `run.created_by` directly to `principalDisplay(...)` inside loaded-run branches.\n\n`apps/fabro-web/app/data/runs.ts` and run-list components:\n- Treat `createdBy` as a total principal in UI data derived from a loaded API run.\n- Remove empty/fallback rendering that only existed for missing creator data.\n\n### Verification\n\n- `cargo +nightly-2026-04-14 fmt --check --all`\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings`\n- `cargo build --workspace`\n- `cargo nextest run --workspace`\n- `cargo nextest run -p fabro-server`\n- `cd apps/fabro-web && bun run typecheck && bun test && bun run build`\n- Manual end-to-end:\n  - `fabro server start`\n  - `cd apps/fabro-web && bun run dev`\n  - Authenticate and create a run through the UI.\n  - Confirm `/api/v1/runs/:id` has non-null `created_by`.\n  - Confirm `/api/v1/runs/:id/state` has non-null `spec.provenance.subject`.\n  - Retry a failed run and confirm the retried run has the retrying user as creator.\n  - Hit demo mode with `X-Fabro-Demo: 1` and confirm the run summary renders the synthetic `demo` dev-token user.\n",
-          "graph.rankdir": "LR",
           "thread.preflight_compile.current_node": "preflight_lint",
-          "thread.preflight_lint.current_node": "implement",
-          "internal.retry_count.implement": 0,
-          "outcome": "succeeded",
-          "failure_class": "",
-          "current_node": "verify",
-          "thread.implement.current_node": "simplify_opus",
-          "internal.retry_count.verify": 0,
-          "internal.retry_count.preflight_lint": 0,
           "thread.start.current_node": "toolchain",
+          "thread.simplify_gpt.current_node": "verify",
+          "graph.goal": "# Plan: Make run actors and provenance total\n\n## Context\n\nThis is a greenfield app. Backward compatibility with old serialized runs, old API clients, old generated models, and old tests is not a constraint. Prefer the clean invariant and remove all traces of the placeholder shape.\n\n`Principal::Anonymous` currently represents \"no authenticated actor on this request\" inside auth middleware. That is auth state, not an actor. A `Principal` should only mean \"who acted.\"\n\nLikewise, a persisted run should always have a creator. `Run.created_by`, `RunSpec.provenance`, `RunProvenance.subject`, and `run.created` event provenance should all be total. No `Option<Principal>`, no nullable OpenAPI fields, no legacy deserialization defaults, and no fallback creator in projection code.\n\nTwo commits, in order.\n\n---\n\n## Commit 1 - Remove `Principal::Anonymous`\n\nBreaking cleanup. `Principal` becomes actor-only. Missing/invalid auth is represented as absent request principal, not as an anonymous principal variant.\n\n### Rust\n\n`lib/crates/fabro-types/src/principal.rs`:\n- Drop `Anonymous`.\n- Drop `Anonymous` arms in `kind()` and `display()`.\n- Delete anonymous serialization/round-trip test coverage.\n\n`lib/crates/fabro-server/src/principal_middleware.rs`:\n- `RequestAuthContext.principal: Principal` -> `Option<Principal>`.\n- `RequestAuthLogContext.principal: Principal` -> `Option<Principal>`.\n- `initial()` and `rejected()` set `principal: None`.\n- `authenticated(...)`, `authenticated_worker(...)`, and `authenticated_user(...)` set `principal: Some(...)`.\n- Update `principal_without_log_unused_fields` to preserve `None` and strip user avatar data only inside `Some(Principal::User(...))`.\n- Update all gate helpers to match `Option<Principal>`:\n  - `require_user`\n  - `require_authenticated_user`\n  - `require_run_management_actor`\n  - `require_worker_or_user_for_run`\n  - `require_run_management_target`\n- `None` routes to the existing `auth_rejection(context.auth_status, context.auth_error_code)` behavior.\n- `Some(Principal::Worker { .. })` keeps the current forbidden-vs-auth-rejection distinctions.\n- Update tests that assert the initial/rejected principal to assert `None`.\n\n`lib/crates/fabro-server/src/server.rs` HTTP logging:\n- Keep the `principal_kind` field on every HTTP log line.\n- Compute `principal_kind` as `auth_context.principal.as_ref().map(Principal::kind).unwrap_or(\"none\")`.\n- Match `auth_context.principal` as an `Option<Principal>`:\n  - `Some(User(...))`, `Some(Worker { ... })`, `Some(Webhook { ... })`, `Some(Slack { ... })` keep their extra fields.\n  - `None | Some(Agent { .. } | System { .. })` emits only the common HTTP fields.\n\n`docs/internal/logging-strategy.md`:\n- Replace the `anonymous` HTTP caller category guidance with `none` for requests that have no principal.\n- Keep `auth_status` as the field that distinguishes missing, invalid, expired, and authenticated auth state.\n\n### OpenAPI and generated clients\n\n`docs/public/api-reference/fabro-api.yaml`:\n- Remove `PrincipalAnonymous` from the `Principal` `oneOf`.\n- Remove `anonymous` from the `Principal` discriminator mapping.\n- Delete the `PrincipalAnonymous` schema.\n\nRegenerate:\n- `cargo build -p fabro-api`\n- `cd lib/packages/fabro-api-client && bun run generate`\n\nExpected generated cleanup:\n- `lib/packages/fabro-api-client/src/models/principal-anonymous.ts` disappears.\n- `Principal` union no longer includes `{ kind: \"anonymous\" }`.\n- `lib/packages/fabro-api-client/src/models/index.ts` no longer exports `principal-anonymous`.\n\n### Frontend\n\n`apps/fabro-web/app/lib/principal-display.tsx`:\n- Remove the `\"anonymous\"` switch case and unused icon import.\n\n`apps/fabro-web/app/components/run-summary-panel.test.tsx` and API-client exhaustiveness tests:\n- Remove anonymous principal cases.\n\n### Documentation sweep\n\nRemove anonymous-principal references from product/API docs and tests. Be careful not to touch unrelated uses of \"anonymous\" such as telemetry anonymous IDs or Git's `remote_anonymous` API.\n\nUseful sweep:\n- `rg -n \"Principal::Anonymous|PrincipalAnonymous|kind: 'anonymous'|kind: \\\"anonymous\\\"|anonymous actor|anonymous subject|principal_kind.*anonymous|\\\"anonymous\\\"\" lib/crates apps/fabro-web lib/packages/fabro-api-client docs/public docs/internal`\n\n### Verification\n\n- `cargo +nightly-2026-04-14 fmt --check --all`\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings`\n- `cargo build --workspace`\n- `cargo nextest run --workspace`\n- `cd apps/fabro-web && bun run typecheck && bun test`\n- Manual: start `fabro server start`, hit a protected endpoint without a token, confirm 401 and an HTTP log with `principal_kind=\"none\"` and `auth_status=\"missing\"`.\n\n---\n\n## Commit 2 - Make run provenance and creator non-optional\n\nFull-chain invariant. Every persisted run has exactly one creator principal. No nullable schema fields, no legacy defaults, no projection fallbacks.\n\n### Core type changes\n\n`lib/crates/fabro-types/src/run_summary.rs`:\n- `Run.created_by: Option<Principal>` -> `Principal`.\n- Drop `#[serde(default)]`.\n\n`lib/crates/fabro-types/src/run.rs`:\n- `RunProvenance.subject: Option<Principal>` -> `Principal`.\n- Drop `#[serde(default, skip_serializing_if = \"Option::is_none\")]`.\n- Drop `Default` derive on `RunProvenance`.\n- `RunSpec.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- Drop `#[serde(default, skip_serializing_if = \"Option::is_none\")]` on `RunSpec.provenance`.\n\n`lib/crates/fabro-types/src/run_event/run.rs`:\n- `RunCreatedProps.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- Drop default/skip serialization attributes for provenance.\n\n`lib/crates/fabro-workflow/src/event/events.rs`:\n- `Event::RunCreated.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- Drop default/skip serialization attributes for provenance.\n\n### Creation and retry flow\n\n`lib/crates/fabro-workflow/src/operations/create.rs`:\n- `CreateRunInput.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- `PersistCreateOptions.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- `RunSpec { provenance }` stores the total provenance directly.\n- `Event::RunCreated { provenance }` emits total provenance directly.\n\n`lib/crates/fabro-server/src/server/handler/runs.rs`:\n- `run_provenance(headers, subject)` returns `RunProvenance { subject: subject.clone(), ... }`.\n- Build provenance before creating `CreateRunInput`.\n\n`lib/crates/fabro-server/src/run_manifest.rs`:\n- Change `create_run_input(...)` to accept `provenance: RunProvenance` and set it directly, or stop using the helper for the final `CreateRunInput` construction. Do not create a temporary input with missing provenance.\n\n`lib/crates/fabro-workflow/src/operations/retry.rs`:\n- `RetryRunInput.provenance: Option<RunProvenance>` -> `RunProvenance`.\n- `retry_run(...)` writes the new run's `run.created` event with total provenance.\n\n`lib/crates/fabro-server/src/server/handler/lifecycle.rs`:\n- Pass `run_provenance(&headers, &actor)` directly into `RetryRunInput`.\n\n### Event conversion and projections\n\n`lib/crates/fabro-workflow/src/event/convert.rs`:\n- Convert `Event::RunCreated.provenance` into `RunCreatedProps.provenance` directly.\n- Remove `Some(...)` wrapping for run-created provenance.\n\n`lib/crates/fabro-workflow/src/event/stored_fields.rs`:\n- `Event::RunCreated { provenance, .. }` sets `actor: Some(provenance.subject.clone())`.\n\n`lib/crates/fabro-store/src/run_state.rs`:\n- `projection_from_created(...)` builds `RunSpec { provenance: props.provenance.clone(), ... }`.\n- `build_summary(...)` sets `created_by: state.spec.provenance.subject.clone()`.\n- Delete or rewrite tests that deserialize projections with `\"provenance\": null`.\n\n`lib/crates/fabro-types/src/run_projection.rs` and projection tests:\n- Replace all test `RunSpec` literals with total provenance.\n- Remove tests whose only purpose is legacy/null provenance tolerance.\n\n### OpenAPI\n\n`docs/public/api-reference/fabro-api.yaml`:\n- `Run.created_by` references `Principal` directly. Remove `oneOf [..., null]`.\n- `RunProvenance.required` includes `subject`.\n- `RunProvenance.subject` references `Principal` directly. Remove `oneOf [..., null]`.\n- `RunSpec.required` includes `provenance`.\n- `RunSpec.provenance` references `RunProvenance` directly. Remove `oneOf [..., null]`.\n- If `run.created` event properties are represented separately in the spec, make that event provenance required and non-nullable too.\n\nRegenerate:\n- `cargo build -p fabro-api`\n- `cd lib/packages/fabro-api-client && bun run generate`\n\nDo not hand-edit generated client files.\n\n### Demo mode\n\n`lib/crates/fabro-server/src/demo/mod.rs`:\n- Add a clearly synthetic demo principal using `AuthMethod::DevToken`, not GitHub:\n  ```rust\n  static DEMO_PRINCIPAL: LazyLock<Principal> = LazyLock::new(|| {\n      Principal::user(\n          IdpIdentity::new(\"fabro:demo\", \"demo\").unwrap(),\n          \"demo\".to_string(),\n          AuthMethod::DevToken,\n      )\n  });\n  ```\n- Replace `created_by: None` with `created_by: DEMO_PRINCIPAL.clone()`.\n- If demo creates any full `RunSpec` or `run.created` event data, give it `RunProvenance { subject: DEMO_PRINCIPAL.clone(), ... }`.\n\n### Test support\n\nDo not add fake auth helpers to `fabro_types::fixtures`; that module is run-id constants.\n\nUse the existing `fabro-types` `test-support` feature:\n- Add `#[cfg(any(test, feature = \"test-support\"))] pub mod test_support;` in `lib/crates/fabro-types/src/lib.rs` if it does not already exist.\n- Add `lib/crates/fabro-types/src/test_support.rs` with:\n  - `test_principal() -> Principal`\n  - `test_run_provenance() -> RunProvenance`\n- Use an obviously fake dev-token identity, e.g. issuer `fabro:test`, subject `test-user`, login `test`.\n- In crates that need the helper from integration tests or cross-crate tests, dual-list `fabro-types` in `dev-dependencies` with `features = [\"test-support\"]`, following existing repo patterns.\n\nUpdate all constructors:\n- Replace `provenance: None` in `RunSpec`, `CreateRunInput`, `RetryRunInput`, `Event::RunCreated`, and `RunCreatedProps` literals with `test_run_provenance()` or a locally meaningful provenance.\n- Replace `subject: Some(...)` with `subject: ...`.\n- Replace `subject: None` only when it is actually `RunProvenance.subject`; leave unrelated todo/commit/message `subject` fields alone.\n- Replace `created_by: None` / `created_by: null` with `test_principal()` or a frontend TS principal fixture.\n- Delete tests that assert nullable or omitted creator/provenance behavior.\n\nRepresentative Rust areas:\n- `lib/crates/fabro-store/src/run_state.rs`\n- `lib/crates/fabro-store/tests/serializable_projection.rs`\n- `lib/crates/fabro-workflow/src/operations/{create,retry,start}.rs`\n- `lib/crates/fabro-workflow/src/event/{convert,sink,stored_fields}.rs`\n- `lib/crates/fabro-workflow/src/handler/**`\n- `lib/crates/fabro-workflow/src/pipeline/**`\n- `lib/crates/fabro-workflow/src/run_{lookup,metadata}.rs`\n- `lib/crates/fabro-server/src/server/tests.rs`\n- `lib/crates/fabro-server/src/server/handler/**`\n- `lib/crates/fabro-server/tests/it/**`\n- `lib/crates/fabro-cli/tests/it/support/mod.rs`\n- `lib/crates/fabro-dump/src/lib.rs`\n- `lib/crates/fabro-tool/src/{common,create,interact,search}.rs`\n- `lib/crates/fabro-api/tests/{principal_round_trip,run_summary_round_trip,run_projection_round_trip,run_event_round_trip}.rs`\n- `lib/crates/fabro-types/tests/{run_spec_serde,run_spec_methods,run_event_serde}.rs`\n\nRepresentative TypeScript areas:\n- `apps/fabro-web/app/**` tests with `created_by: null`\n- `apps/fabro-web/app/data/runs.ts`\n- `apps/fabro-web/app/components/run-summary-panel.tsx`\n- `apps/fabro-web/app/components/runs-list/**`\n- `lib/packages/fabro-api-client/tests/principal-exhaustive.ts`\n\nUseful sweep after edits:\n- `rg -n \"Principal::Anonymous|PrincipalAnonymous|principal-anonymous|kind: ['\\\"]anonymous|created_by:\\\\s*(None|null)|provenance:\\\\s*None|subject:\\\\s*Some\\\\(|subject:\\\\s*None\" lib/crates apps/fabro-web lib/packages/fabro-api-client docs/public docs/internal`\n\nReview each hit. The only acceptable remaining matches should be unrelated uses of \"anonymous\" and unrelated non-principal `subject` fields.\n\n### Frontend\n\n`apps/fabro-web/app/components/run-summary-panel.tsx`:\n- `run?.created_by` may still be guarded by `run` loading state, but `created_by` itself is non-null once `run` exists.\n- Pass `run.created_by` directly to `principalDisplay(...)` inside loaded-run branches.\n\n`apps/fabro-web/app/data/runs.ts` and run-list components:\n- Treat `createdBy` as a total principal in UI data derived from a loaded API run.\n- Remove empty/fallback rendering that only existed for missing creator data.\n\n### Verification\n\n- `cargo +nightly-2026-04-14 fmt --check --all`\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings`\n- `cargo build --workspace`\n- `cargo nextest run --workspace`\n- `cargo nextest run -p fabro-server`\n- `cd apps/fabro-web && bun run typecheck && bun test && bun run build`\n- Manual end-to-end:\n  - `fabro server start`\n  - `cd apps/fabro-web && bun run dev`\n  - Authenticate and create a run through the UI.\n  - Confirm `/api/v1/runs/:id` has non-null `created_by`.\n  - Confirm `/api/v1/runs/:id/state` has non-null `spec.provenance.subject`.\n  - Retry a failed run and confirm the retried run has the retrying user as creator.\n  - Hit demo mode with `X-Fabro-Demo: 1` and confirm the run summary renders the synthetic `demo` dev-token user.\n",
+          "failure_signature": "",
+          "outcome": "succeeded",
+          "internal.retry_count.toolchain": 0,
+          "thread.preflight_lint.current_node": "implement",
+          "current_node": "verify",
+          "last_stage": "simplify_gpt",
+          "internal.run_id": "01KT734BXFV007VWXT3G5PFXYA",
+          "last_response": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues",
+          "internal.retry_count.preflight_compile": 0,
+          "internal.retry_count.implement": 0,
+          "internal.retry_count.verify": 0,
+          "thread.implement.current_node": "simplify_opus",
+          "graph.model_stylesheet": "\n            * { model: claude-opus-4-7; }\n        ",
+          "internal.retry_count.start": 0,
+          "response.simplify_opus": "All frontend tests pass.\n\n## Summary\n\nReviewed both commits with three parallel agents (reuse, quality, efficiency). Findings triaged and applied:\n\n**Fixed:**\n- **Inconsistent indentation** in `apps/fabro-web/app/routes/run-files.render.test.tsx` (`created_by` fixture had under-indented inner keys) — re-indented to match surrounding nesting.\n- **Inconsistent `<EmptyValue />` indentation** in `apps/fabro-web/app/components/run-summary-panel.tsx` — the two cells touched by the diff had been fixed, but three other cells (Sandbox, Cost, Artifacts) still had the same under-indented `<EmptyValue />`. Made all five consistent.\n\n**Skipped (with rationale):**\n- *Duplicated `test_run_provenance` in `fabro-workflow/src/test_support.rs`* — intentionally uses `Principal::System { Engine }` rather than a user, since workflow-engine tests model the engine itself as the creator. Not a true duplicate.\n- *Unifying `test_principal()` across crates* — existing per-crate helpers use the `https://github.com` issuer with `AuthMethod::Github`, while the new `fabro_types::test_support::test_principal()` deliberately uses `fabro:test`/`DevToken`. Consolidating would require widening the helper API beyond the scope of this PR.\n- *Match arm `None | Some(Principal::Agent { .. } | Principal::System { .. })` in `server.rs`* — exhaustive, concise, and exhaustiveness-checked over `Principal` variants. Splitting would add noise.\n- *Stringly-typed `\"none\"` for `principal_kind`* — used in exactly one place; a named constant would be over-engineering.\n- *`<EmptyValue />` fallback on \"Created by\"* — still reachable when `run` is undefined but not loading (e.g. error states), so not dead code.\n- *Per-request `DEMO_PRINCIPAL.clone()` allocations in demo summaries* — pre-existing pattern, the `LazyLock` already eliminates the per-request `IdpIdentity::new` validation cost. Promoting summaries to a static `Vec<Run>` is a larger refactor unrelated to this PR's scope.\n\nFrontend `typecheck` and `bun test` (508 pass) both green.",
+          "response.implement": "Implemented the two-commit plan and committed both changes.\n\nCommits:\n- `43a12a3e1 refactor(auth): remove anonymous principal`\n- `9ae80132d refactor(runs): require creator provenance`\n\nSummary:\n- Removed `Principal::Anonymous`; unauthenticated/invalid request state is now represented as `None`, with HTTP logs using `principal_kind=\"none\"`.\n- Made persisted run actors/provenance total:\n  - `Run.created_by: Principal`\n  - `RunSpec.provenance: RunProvenance`\n  - `RunProvenance.subject: Principal`\n  - `run.created` provenance required/non-null\n- Updated OpenAPI and regenerated Rust/TS API outputs.\n- Added `fabro-types::test_support` helpers behind `test-support`.\n- Updated server/workflow/store projections, create/retry flows, demo principal, frontend UI/data assumptions, and test fixtures/snapshots.\n- Final working tree is clean.\n\nVerification run:\n- `cargo build -p fabro-api`\n- `cd lib/packages/fabro-api-client && bun run generate`\n- `cd lib/packages/fabro-api-client && bun run typecheck`\n- `cd apps/fabro-web && bun run typecheck && bun test && bun run build`\n- `cargo +nightly-2026-04-14 fmt --check --all`\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings`\n- `cargo build --workspace`\n- `cargo nextest run --workspace`\n- `cargo nextest run -p fabro-server`\n- Final sweeps for anonymous principal and nullable creator/provenance patterns:\n  - only unrelated `crossorigin=\"anonymous\"` and unrelated todo/web-auth `subject` fields remain.\n\nNotes:\n- `cargo insta pending-snapshots` could not run because `cargo-insta` is not installed in this environment; I checked for pending `*.snap.new` files and found none.\n- Manual browser/server E2E verification was not performed in this environment.",
+          "thread.toolchain.current_node": "preflight_compile",
+          "internal.retry_count.preflight_lint": 0,
           "internal.node_visit_count": 1,
-          "thread.toolchain.current_node": "preflight_compile"
+          "graph.rankdir": "LR",
+          "thread.simplify_opus.current_node": "simplify_gpt",
+          "internal.fidelity": "compact"
         },
         "node_outcomes": {
-          "preflight_compile": {
-            "status": "succeeded",
-            "context_updates": {
-              "command.output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126"
-            },
-            "notes": "Script completed: cargo check -q --workspace 2>&1",
-            "usage": null,
-            "timing": {
-              "wall_time_ms": 0,
-              "inference_time_ms": 0,
-              "tool_time_ms": 134621,
-              "active_time_ms": 134621
-            }
-          },
           "start": {
             "status": "succeeded",
             "usage": null
           },
-          "preflight_lint": {
-            "status": "succeeded",
-            "context_updates": {
-              "command.output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126"
-            },
-            "notes": "Script completed: cargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
-            "usage": null,
-            "timing": {
-              "wall_time_ms": 0,
-              "inference_time_ms": 0,
-              "tool_time_ms": 147496,
-              "active_time_ms": 147496
-            }
-          },
-          "toolchain": {
-            "status": "succeeded",
-            "context_updates": {
-              "command.output": "blob://sha256/fc14b2ba2d770e5cd3169df7a29525c962adfc4cfa3097b9098c63ebd61a748c"
-            },
-            "notes": "Script completed: command -v cargo >/dev/null || { curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y && sudo ln -sf $HOME/.cargo/bin/* /usr/local/bin/; }; cargo --version 2>&1",
-            "usage": null,
-            "timing": {
-              "wall_time_ms": 0,
-              "inference_time_ms": 0,
-              "tool_time_ms": 1316,
-              "active_time_ms": 1316
-            }
-          },
           "simplify_opus": {
             "status": "succeeded",
             "context_updates": {
@@ -1500,6 +1458,70 @@
               "active_time_ms": 390128
             }
           },
+          "verify": {
+            "status": "succeeded",
+            "context_updates": {
+              "command.output": "blob://sha256/14a657f055256f714ba70d6898170e78b8ad9d9fac61f27d4b1445719ab6b2d5"
+            },
+            "notes": "Script completed: git fetch origin main 2>&1 && git merge --no-edit --no-stat origin/main 2>&1 && cargo +nightly-2026-04-14 fmt --all 2>&1 && cargo dev docs refresh 2>&1 && cargo +nightly-2026-04-14 fmt --check --all 2>&1 && { command -v rg >/dev/null 2>&1 || { echo 'rg is required for verify'; exit 127; }; } && ! rg -n 'AuthMode::Disabled|RunAuthMethod|RunSubjectProvenance|\\bActorRef\\b|\\bActorKind\\b|AuthenticatedSubject|AuthenticatedService|AuthorizeRunScoped|AuthorizeRunBlob|AuthorizeStageArtifact|AuthorizeCommandLog|auth_method\\s*==\\s*\"disabled\"' lib/crates apps lib/packages docs/public/api-reference/fabro-api.yaml 2>&1 && cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings 2>&1 && cargo nextest run --workspace --status-level slow --profile ci 2>&1 && cargo dev docs check 2>&1 && bun install --frozen-lockfile 2>&1 && (cd apps/fabro-web && bun run typecheck) 2>&1 && (cd apps/fabro-web && bun run test) 2>&1 && (cd lib/packages/fabro-api-client && bun run typecheck) 2>&1 && cargo dev build -- -p fabro-cli --release 2>&1",
+            "usage": null,
+            "timing": {
+              "wall_time_ms": 0,
+              "inference_time_ms": 0,
+              "tool_time_ms": 483510,
+              "active_time_ms": 483510
+            }
+          },
+          "simplify_gpt": {
+            "status": "succeeded",
+            "context_updates": {
+              "last_response": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues",
+              "response.simplify_gpt": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues found.\n  - Code quality: no issues found.\n  - Efficiency: no issues found.\n- No fixes were needed.\n- Final `git status --short` is clean.",
+              "last_stage": "simplify_gpt"
+            },
+            "notes": "Stage completed: simplify_gpt",
+            "usage": {
+              "input": {
+                "usage": {
+                  "model": {
+                    "provider": "openai",
+                    "model_id": "gpt-5.5"
+                  },
+                  "tokens": {
+                    "input_tokens": 78198,
+                    "output_tokens": 2542,
+                    "reasoning_tokens": 284,
+                    "cache_read_tokens": 10752,
+                    "cache_write_tokens": 0
+                  }
+                },
+                "facts": {
+                  "algorithm": "openai"
+                }
+              },
+              "total_usd_micros": 481146
+            },
+            "timing": {
+              "wall_time_ms": 0,
+              "inference_time_ms": 62247,
+              "tool_time_ms": 1144,
+              "active_time_ms": 63391
+            }
+          },
+          "preflight_compile": {
+            "status": "succeeded",
+            "context_updates": {
+              "command.output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126"
+            },
+            "notes": "Script completed: cargo check -q --workspace 2>&1",
+            "usage": null,
+            "timing": {
+              "wall_time_ms": 0,
+              "inference_time_ms": 0,
+              "tool_time_ms": 134621,
+              "active_time_ms": 134621
+            }
+          },
           "implement": {
             "status": "succeeded",
             "context_updates": {
@@ -1539,73 +1561,173 @@
               "active_time_ms": 3204625
             }
           },
-          "simplify_gpt": {
+          "toolchain": {
             "status": "succeeded",
             "context_updates": {
-              "last_response": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues",
-              "response.simplify_gpt": "Review complete.\n\n- Working tree had no uncommitted changes, so I reviewed the latest implementation diff (`9ae80132d..HEAD`).\n- Launched all three review agents in parallel:\n  - Code reuse: no issues found.\n  - Code quality: no issues found.\n  - Efficiency: no issues found.\n- No fixes were needed.\n- Final `git status --short` is clean.",
-              "last_stage": "simplify_gpt"
+              "command.output": "blob://sha256/fc14b2ba2d770e5cd3169df7a29525c962adfc4cfa3097b9098c63ebd61a748c"
             },
-            "notes": "Stage completed: simplify_gpt",
-            "usage": {
-              "input": {
-                "usage": {
-                  "model": {
-                    "provider": "openai",
-                    "model_id": "gpt-5.5"
-                  },
-                  "tokens": {
-                    "input_tokens": 78198,
-                    "output_tokens": 2542,
-                    "reasoning_tokens": 284,
-                    "cache_read_tokens": 10752,
-                    "cache_write_tokens": 0
-                  }
-                },
-                "facts": {
-                  "algorithm": "openai"
-                }
-              },
-              "total_usd_micros": 481146
-            },
-            "timing": {
-              "wall_time_ms": 0,
-              "inference_time_ms": 62247,
-              "tool_time_ms": 1144,
-              "active_time_ms": 63391
-            }
-          },
-          "verify": {
-            "status": "succeeded",
-            "context_updates": {
-              "command.output": "blob://sha256/14a657f055256f714ba70d6898170e78b8ad9d9fac61f27d4b1445719ab6b2d5"
-            },
-            "notes": "Script completed: git fetch origin main 2>&1 && git merge --no-edit --no-stat origin/main 2>&1 && cargo +nightly-2026-04-14 fmt --all 2>&1 && cargo dev docs refresh 2>&1 && cargo +nightly-2026-04-14 fmt --check --all 2>&1 && { command -v rg >/dev/null 2>&1 || { echo 'rg is required for verify'; exit 127; }; } && ! rg -n 'AuthMode::Disabled|RunAuthMethod|RunSubjectProvenance|\\bActorRef\\b|\\bActorKind\\b|AuthenticatedSubject|AuthenticatedService|AuthorizeRunScoped|AuthorizeRunBlob|AuthorizeStageArtifact|AuthorizeCommandLog|auth_method\\s*==\\s*\"disabled\"' lib/crates apps lib/packages docs/public/api-reference/fabro-api.yaml 2>&1 && cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings 2>&1 && cargo nextest run --workspace --status-level slow --profile ci 2>&1 && cargo dev docs check 2>&1 && bun install --frozen-lockfile 2>&1 && (cd apps/fabro-web && bun run typecheck) 2>&1 && (cd apps/fabro-web && bun run test) 2>&1 && (cd lib/packages/fabro-api-client && bun run typecheck) 2>&1 && cargo dev build -- -p fabro-cli --release 2>&1",
+            "notes": "Script completed: command -v cargo >/dev/null || { curl --proto '=https' --tlsv1.2 -sSf https://sh.rustup.rs | sh -s -- -y && sudo ln -sf $HOME/.cargo/bin/* /usr/local/bin/; }; cargo --version 2>&1",
             "usage": null,
             "timing": {
               "wall_time_ms": 0,
               "inference_time_ms": 0,
-              "tool_time_ms": 483510,
-              "active_time_ms": 483510
+              "tool_time_ms": 1316,
+              "active_time_ms": 1316
+            }
+          },
+          "preflight_lint": {
+            "status": "succeeded",
+            "context_updates": {
+              "command.output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126"
+            },
+            "notes": "Script completed: cargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
+            "usage": null,
+            "timing": {
+              "wall_time_ms": 0,
+              "inference_time_ms": 0,
+              "tool_time_ms": 147496,
+              "active_time_ms": 147496
             }
           }
         },
         "next_node_id": "exit",
+        "git_commit_sha": "bb1a3b17de15b32d73183259a45d164ea46f6a3c",
         "node_visits": {
+          "preflight_lint": 1,
           "simplify_opus": 1,
+          "simplify_gpt": 1,
           "preflight_compile": 1,
           "start": 1,
-          "implement": 1,
-          "simplify_gpt": 1,
           "toolchain": 1,
-          "preflight_lint": 1,
+          "implement": 1,
           "verify": 1
         }
       },
-      "diff": {}
+      "diff": {
+        "patch": "diff --git a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx\nindex f7bcda6bf..e746780ab 100644\n--- a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx\n+++ b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx\n@@ -159,6 +159,7 @@ describe(\"VncPanel render\", () => {\n       \"https://preview.example.com/sb-1/6080?token=abc\",\n     );\n     expect(iframes[0]?.props.allow).toContain(\"clipboard-write\");\n+    expect(iframes[0]?.props.sandbox).toContain(\"allow-same-origin\");\n   });\n \n   test(\"renders an actionable error state for 409 startup failures\", () => {\ndiff --git a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx\nindex 9721a48d4..514b80042 100644\n--- a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx\n+++ b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx\n@@ -246,9 +246,9 @@ function VncBody({\n       src={url}\n       title=\"Sandbox VNC desktop\"\n       // Daytona's signed preview already pins the iframe to the noVNC service;\n-      // clipboard + fullscreen let the embedded session feel native.\n+      // noVNC reads localStorage, which requires the child to keep its origin.\n       allow=\"clipboard-read; clipboard-write; fullscreen\"\n-      sandbox=\"allow-forms allow-pointer-lock allow-scripts\"\n+      sandbox=\"allow-forms allow-pointer-lock allow-same-origin allow-scripts\"\n       className=\"size-full border-0\"\n     />\n   );\n",
+        "summary": {
+          "files_changed": 99,
+          "additions": 410,
+          "deletions": 338
+        }
+      }
     }
   ],
-  "conclusion": null,
+  "conclusion": {
+    "timestamp": "2026-06-03T17:12:22.774893Z",
+    "status": "succeeded",
+    "timing": {
+      "wall_time_ms": 4610392,
+      "inference_time_ms": 1977560,
+      "tool_time_ms": 2447527,
+      "active_time_ms": 4425087
+    },
+    "final_git_commit_sha": "bb1a3b17de15b32d73183259a45d164ea46f6a3c",
+    "stages": [
+      {
+        "stage_id": "start",
+        "stage_label": "start",
+        "timing": {
+          "wall_time_ms": 0,
+          "inference_time_ms": 0,
+          "tool_time_ms": 0,
+          "active_time_ms": 0
+        },
+        "retries": 0
+      },
+      {
+        "stage_id": "toolchain",
+        "stage_label": "toolchain",
+        "timing": {
+          "wall_time_ms": 1329,
+          "inference_time_ms": 0,
+          "tool_time_ms": 1316,
+          "active_time_ms": 1316
+        },
+        "retries": 0
+      },
+      {
+        "stage_id": "preflight_compile",
+        "stage_label": "preflight_compile",
+        "timing": {
+          "wall_time_ms": 134629,
+          "inference_time_ms": 0,
+          "tool_time_ms": 134621,
+          "active_time_ms": 134621
+        },
+        "retries": 0
+      },
+      {
+        "stage_id": "preflight_lint",
+        "stage_label": "preflight_lint",
+        "timing": {
+          "wall_time_ms": 147502,
+          "inference_time_ms": 0,
+          "tool_time_ms": 147496,
+          "active_time_ms": 147496
+        },
+        "retries": 0
+      },
+      {
+        "stage_id": "implement",
+        "stage_label": "implement",
+        "timing": {
+          "wall_time_ms": 3320136,
+          "inference_time_ms": 1761337,
+          "tool_time_ms": 1443288,
+          "active_time_ms": 3204625
+        },
+        "billing_usd_micros": 35826537,
+        "retries": 0
+      },
+      {
+        "stage_id": "simplify_opus",
+        "stage_label": "simplify_opus",
+        "timing": {
+          "wall_time_ms": 393885,
+          "inference_time_ms": 153976,
+          "tool_time_ms": 236152,
+          "active_time_ms": 390128
+        },
+        "billing_usd_micros": 2020680,
+        "retries": 0
+      },
+      {
+        "stage_id": "simplify_gpt",
+        "stage_label": "simplify_gpt",
+        "timing": {
+          "wall_time_ms": 63759,
+          "inference_time_ms": 62247,
+          "tool_time_ms": 1144,
+          "active_time_ms": 63391
+        },
+        "billing_usd_micros": 481146,
+        "retries": 0
+      },
+      {
+        "stage_id": "verify",
+        "stage_label": "verify",
+        "timing": {
+          "wall_time_ms": 483542,
+          "inference_time_ms": 0,
+          "tool_time_ms": 483510,
+          "active_time_ms": 483510
+        },
+        "retries": 0
+      }
+    ],
+    "billing": {
+      "input_tokens": 4226069,
+      "output_tokens": 45881,
+      "total_tokens": 32967630,
+      "reasoning_tokens": 15394,
+      "cache_read_tokens": 28494592,
+      "cache_write_tokens": 185694,
+      "total_usd_micros": 38328363
+    },
+    "total_retries": 0,
+    "diff": {}
+  },
   "sandbox": {
     "kind": "ready",
     "plan": {
@@ -2568,7 +2690,12 @@
       "first_event_seq": 1442,
       "prompt": null,
       "response": null,
-      "completion": null,
+      "completion": {
+        "outcome": "succeeded",
+        "notes": "Script completed: git fetch origin main 2>&1 && git merge --no-edit --no-stat origin/main 2>&1 && cargo +nightly-2026-04-14 fmt --all 2>&1 && cargo dev docs refresh 2>&1 && cargo +nightly-2026-04-14 fmt --check --all 2>&1 && { command -v rg >/dev/null 2>&1 || { echo 'rg is required for verify'; exit 127; }; } && ! rg -n 'AuthMode::Disabled|RunAuthMethod|RunSubjectProvenance|\\bActorRef\\b|\\bActorKind\\b|AuthenticatedSubject|AuthenticatedService|AuthorizeRunScoped|AuthorizeRunBlob|AuthorizeStageArtifact|AuthorizeCommandLog|auth_method\\s*==\\s*\"disabled\"' lib/crates apps lib/packages docs/public/api-reference/fabro-api.yaml 2>&1 && cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings 2>&1 && cargo nextest run --workspace --status-level slow --profile ci 2>&1 && cargo dev docs check 2>&1 && bun install --frozen-lockfile 2>&1 && (cd apps/fabro-web && bun run typecheck) 2>&1 && (cd apps/fabro-web && bun run test) 2>&1 && (cd lib/packages/fabro-api-client && bun run typecheck) 2>&1 && cargo dev build -- -p fabro-cli --release 2>&1",
+        "failure_reason": null,
+        "timestamp": "2026-06-03T17:11:59.401885Z"
+      },
       "provider_used": null,
       "diff": null,
       "script_invocation": {
@@ -2576,11 +2703,27 @@
         "command": "exec 2>&1\ngit fetch origin main 2>&1 && git merge --no-edit --no-stat origin/main 2>&1 && cargo +nightly-2026-04-14 fmt --all 2>&1 && cargo dev docs refresh 2>&1 && cargo +nightly-2026-04-14 fmt --check --all 2>&1 && { command -v rg >/dev/null 2>&1 || { echo 'rg is required for verify'; exit 127; }; } && ! rg -n 'AuthMode::Disabled|RunAuthMethod|RunSubjectProvenance|\\bActorRef\\b|\\bActorKind\\b|AuthenticatedSubject|AuthenticatedService|AuthorizeRunScoped|AuthorizeRunBlob|AuthorizeStageArtifact|AuthorizeCommandLog|auth_method\\s*==\\s*\"disabled\"' lib/crates apps lib/packages docs/public/api-reference/fabro-api.yaml 2>&1 && cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings 2>&1 && cargo nextest run --workspace --status-level slow --profile ci 2>&1 && cargo dev docs check 2>&1 && bun install --frozen-lockfile 2>&1 && (cd apps/fabro-web && bun run typecheck) 2>&1 && (cd apps/fabro-web && bun run test) 2>&1 && (cd lib/packages/fabro-api-client && bun run typecheck) 2>&1 && cargo dev build -- -p fabro-cli --release 2>&1",
         "language": "shell"
       },
-      "script_timing": null,
+      "script_timing": {
+        "output": "blob://sha256/14a657f055256f714ba70d6898170e78b8ad9d9fac61f27d4b1445719ab6b2d5",
+        "exit_code": 0,
+        "duration_ms": 483510,
+        "termination": "exited",
+        "output_bytes": 201705,
+        "live_streaming": true
+      },
       "parallel_results": null,
       "output": null,
+      "output_bytes": 201705,
+      "live_streaming": true,
+      "termination": "exited",
       "started_at": "2026-06-03T17:03:55.856785Z",
       "handler": "command",
+      "timing": {
+        "wall_time_ms": 483542,
+        "inference_time_ms": 0,
+        "tool_time_ms": 483510,
+        "active_time_ms": 483510
+      },
       "usage": {
         "input_tokens": 0,
         "output_tokens": 0,
@@ -2589,7 +2732,41 @@
         "cache_read_tokens": 0,
         "cache_write_tokens": 0
       },
-      "state": "running"
+      "state": "succeeded"
+    },
+    "exit@1": {
+      "first_event_seq": 1452,
+      "prompt": null,
+      "response": null,
+      "completion": {
+        "outcome": "succeeded",
+        "notes": null,
+        "failure_reason": null,
+        "timestamp": "2026-06-03T17:12:22.695009Z"
+      },
+      "provider_used": null,
+      "diff": null,
+      "script_invocation": null,
+      "script_timing": null,
+      "parallel_results": null,
+      "output": null,
+      "started_at": "2026-06-03T17:12:22.694911Z",
+      "handler": "exit",
+      "timing": {
+        "wall_time_ms": 0,
+        "inference_time_ms": 0,
+        "tool_time_ms": 0,
+        "active_time_ms": 0
+      },
+      "usage": {
+        "input_tokens": 0,
+        "output_tokens": 0,
+        "total_tokens": 0,
+        "reasoning_tokens": 0,
+        "cache_read_tokens": 0,
+        "cache_write_tokens": 0
+      },
+      "state": "succeeded"
     }
   }
 }
\ No newline at end of file
diff --git a/stages/008-verify@1/diff.patch b/stages/008-verify@1/diff.patch
new file mode 100644
index 000000000..408cd0a95
--- /dev/null
+++ b/stages/008-verify@1/diff.patch
@@ -0,0 +1,28 @@
+diff --git a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx
+index f7bcda6bf..e746780ab 100644
+--- a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx
++++ b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.test.tsx
+@@ -159,6 +159,7 @@ describe("VncPanel render", () => {
+       "https://preview.example.com/sb-1/6080?token=abc",
+     );
+     expect(iframes[0]?.props.allow).toContain("clipboard-write");
++    expect(iframes[0]?.props.sandbox).toContain("allow-same-origin");
+   });
+ 
+   test("renders an actionable error state for 409 startup failures", () => {
+diff --git a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx
+index 9721a48d4..514b80042 100644
+--- a/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx
++++ b/apps/fabro-web/app/routes/run-sandbox/vnc-panel.tsx
+@@ -246,9 +246,9 @@ function VncBody({
+       src={url}
+       title="Sandbox VNC desktop"
+       // Daytona's signed preview already pins the iframe to the noVNC service;
+-      // clipboard + fullscreen let the embedded session feel native.
++      // noVNC reads localStorage, which requires the child to keep its origin.
+       allow="clipboard-read; clipboard-write; fullscreen"
+-      sandbox="allow-forms allow-pointer-lock allow-scripts"
++      sandbox="allow-forms allow-pointer-lock allow-same-origin allow-scripts"
+       className="size-full border-0"
+     />
+   );
diff --git a/stages/008-verify@1/output.log b/stages/008-verify@1/output.log
new file mode 100644
index 000000000..134d2f11e
--- /dev/null
+++ b/stages/008-verify@1/output.log
@@ -0,0 +1 @@
+blob://sha256/14a657f055256f714ba70d6898170e78b8ad9d9fac61f27d4b1445719ab6b2d5
\ No newline at end of file
diff --git a/stages/008-verify@1/script_timing.json b/stages/008-verify@1/script_timing.json
new file mode 100644
index 000000000..52f029ae0
--- /dev/null
+++ b/stages/008-verify@1/script_timing.json
@@ -0,0 +1,8 @@
+{
+  "output": "blob://sha256/14a657f055256f714ba70d6898170e78b8ad9d9fac61f27d4b1445719ab6b2d5",
+  "exit_code": 0,
+  "duration_ms": 483510,
+  "termination": "exited",
+  "output_bytes": 201705,
+  "live_streaming": true
+}
\ No newline at end of file
diff --git a/stages/008-verify@1/status.json b/stages/008-verify@1/status.json
new file mode 100644
index 000000000..4fc609beb
--- /dev/null
+++ b/stages/008-verify@1/status.json
@@ -0,0 +1,6 @@
+{
+  "outcome": "succeeded",
+  "notes": "Script completed: git fetch origin main 2>&1 && git merge --no-edit --no-stat origin/main 2>&1 && cargo +nightly-2026-04-14 fmt --all 2>&1 && cargo dev docs refresh 2>&1 && cargo +nightly-2026-04-14 fmt --check --all 2>&1 && { command -v rg >/dev/null 2>&1 || { echo 'rg is required for verify'; exit 127; }; } && ! rg -n 'AuthMode::Disabled|RunAuthMethod|RunSubjectProvenance|\\bActorRef\\b|\\bActorKind\\b|AuthenticatedSubject|AuthenticatedService|AuthorizeRunScoped|AuthorizeRunBlob|AuthorizeStageArtifact|AuthorizeCommandLog|auth_method\\s*==\\s*\"disabled\"' lib/crates apps lib/packages docs/public/api-reference/fabro-api.yaml 2>&1 && cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings 2>&1 && cargo nextest run --workspace --status-level slow --profile ci 2>&1 && cargo dev docs check 2>&1 && bun install --frozen-lockfile 2>&1 && (cd apps/fabro-web && bun run typecheck) 2>&1 && (cd apps/fabro-web && bun run test) 2>&1 && (cd lib/packages/fabro-api-client && bun run typecheck) 2>&1 && cargo dev build -- -p fabro-cli --release 2>&1",
+  "failure_reason": null,
+  "timestamp": "2026-06-03T17:11:59.401885Z"
+}
\ No newline at end of file
diff --git a/stages/009-exit@1/status.json b/stages/009-exit@1/status.json
new file mode 100644
index 000000000..641e36119
--- /dev/null
+++ b/stages/009-exit@1/status.json
@@ -0,0 +1,6 @@
+{
+  "outcome": "succeeded",
+  "notes": null,
+  "failure_reason": null,
+  "timestamp": "2026-06-03T17:12:22.695009Z"
+}
\ No newline at end of file