mirror of
https://github.com/fabro-sh/fabro.git
synced 2026-09-08 22:21:45 +00:00
Fabro
parent
3612259caa
commit
050a928849
3 changed files with 359 additions and 60 deletions
408
run.json
408
run.json
|
|
@ -505,7 +505,7 @@
|
|||
"kind": "running"
|
||||
},
|
||||
"status_updated_at": "2026-05-28T00:17:17.965930Z",
|
||||
"last_event_at": "2026-05-28T00:59:11.375203Z",
|
||||
"last_event_at": "2026-05-28T00:59:14.909882Z",
|
||||
"pending_control": null,
|
||||
"checkpoints": [
|
||||
{
|
||||
|
|
@ -1124,9 +1124,9 @@
|
|||
}
|
||||
},
|
||||
{
|
||||
"seq": 0,
|
||||
"seq": 975,
|
||||
"checkpoint": {
|
||||
"timestamp": "2026-05-28T00:59:11.421006Z",
|
||||
"timestamp": "2026-05-28T00:59:14.908062Z",
|
||||
"current_node": "simplify_gpt",
|
||||
"completed_nodes": [
|
||||
"start",
|
||||
|
|
@ -1138,6 +1138,239 @@
|
|||
"simplify_gpt"
|
||||
],
|
||||
"node_retries": {},
|
||||
"context_values": {
|
||||
"internal.retry_count.start": 0,
|
||||
"last_stage": "simplify_gpt",
|
||||
"graph.rankdir": "LR",
|
||||
"thread.simplify_opus.current_node": "simplify_gpt",
|
||||
"internal.thread_id": "simplify_opus",
|
||||
"internal.run_id": "01KSNZ26XXV9Y70FDJQHBQ0CC4",
|
||||
"graph.goal": "# Issue #399: Add automation run endpoints\n\n- URL: https://github.com/fabro-sh/fabro/issues/399\n- State: OPEN\n- Author: Bryan Helmkamp (@brynary)\n- Created: 2026-05-25T15:06:27Z\n- Updated: 2026-05-25T15:06:27Z\n- Labels: None\n- Assignees: None\n- Milestone: None\n- Comments: 0\n\n---\n\n## Goal\n\nExpose API endpoints for listing runs associated with an automation and starting a run through an enabled API trigger.\n\n## Scope\n\nImplement these endpoints:\n\n```http\nGET /automations/{id}/runs\nPOST /automations/{id}/runs\n```\n\n`GET /automations/{id}/runs` behavior:\n\n- Require the automation definition to exist; return 404 when it does not.\n- List cached runs from the existing run store.\n- Filter by `run.automation.as_ref().is_some_and(|a| a.id == id)`.\n- Sort newest first.\n- Support `page[limit]` and `page[offset]` using existing pagination behavior.\n- Return the existing paginated run list envelope:\n\n```json\n{\n \"data\": [],\n \"meta\": { \"has_more\": false, \"total\": 0 }\n}\n```\n\n`POST /automations/{id}/runs` behavior:\n\n- Use `RequiredRunToolActor`.\n- Require the automation to exist and be enabled.\n- Find an enabled trigger where `type = \"api\"`.\n- Return 409 with API error code `automation_api_trigger_disabled` when the automation is disabled or no enabled API trigger is available.\n- Materialize the run manifest using the configured `AutomationRunMaterializer`.\n- Call the shared create-run helper with:\n\n```rust\nAutomationRef {\n id: automation.id.to_string(),\n name: Some(automation.name.clone()),\n trigger_id: Some(api_trigger.id.to_string()),\n}\n```\n\n- Return 201 and the normal `Run` response shape with automation metadata populated.\n\nFinal integration expectations:\n\n- Automation-created runs are visible through normal run APIs.\n- Automation-created runs are visible through `GET /automations/{id}/runs`.\n- Run history is derived from persisted/cached runs; no runtime automation state store is introduced.\n- Schedule trigger expressions are stored and validated by earlier phases but are not scheduled by this endpoint work.\n\n## Files\n\nModify:\n\n- `lib/crates/fabro-server/src/server/handler/automations.rs`\n- `lib/crates/fabro-server/src/server/handler/runs.rs`, only if additional helper exposure is needed from the previous phase\n- `lib/crates/fabro-server/tests/it/api/automations.rs`\n- `lib/crates/fabro-server/tests/it/api/mod.rs`\n\n## Acceptance Criteria\n\n- Disabled automations cannot start runs through the automation run endpoint.\n- Automations without an enabled API trigger cannot start runs through the automation run endpoint.\n- A successful API-triggered automation run returns a normal `Run` response with `automation.id`, `automation.name`, and `automation.trigger_id`.\n- The automation run listing endpoint returns only runs linked to that automation.\n- Automation run listings are newest-first and paginate correctly.\n- No scheduler, web UI route/component, or CLI command is added.\n\n## Verification\n\nAdd integration tests using the fake materializer for:\n\n- Disabled automation returns 409.\n- Disabled API trigger returns 409.\n- Missing API trigger returns 409.\n- Successful run creation returns 201.\n- Created run persists `Run.automation`.\n- Associated run listing includes the run.\n- Run listing excludes runs from other automations.\n- Run listing pagination and newest-first sorting.\n\nRun:\n\n```bash\ncargo nextest run -p fabro-automation\ncargo nextest run -p fabro-api\ncargo nextest run -p fabro-server automations\ncargo nextest run -p fabro-server openapi_conformance\ncargo +nightly-2026-04-14 fmt --check --all\ncargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings\ngit diff -- apps/fabro-web lib/crates/fabro-cli\n```\n\nExpected: focused tests and checks pass; web UI and CLI command modules remain unchanged.\n",
|
||||
"internal.retry_count.toolchain": 0,
|
||||
"internal.fidelity": "compact",
|
||||
"thread.implement.current_node": "simplify_opus",
|
||||
"internal.retry_count.simplify_opus": 0,
|
||||
"response.simplify_opus": "Web UI and CLI remain untouched.\n\n## Summary\n\nReviewed 3 changed files via parallel reuse/quality/efficiency agents and applied the following cleanups:\n\n**Removed dead/redundant code:**\n- Deleted `require_run_tool_actor` helper in `principal_middleware.rs` — was a trivial alias for `require_run_management_actor`; `RequiredRunToolActor::from_request_parts` now calls the real function directly.\n- Removed stale `#[allow(dead_code, reason = \"...\")]` on `AppState::materialize_automation_run` in `server.rs` (now called from `create_automation_run`).\n\n**Inlined single-use helpers in `automations.rs`:**\n- `automation_api_trigger_disabled_error` + the matching const `AUTOMATION_API_TRIGGER_DISABLED_CODE` — only used by one `else` branch.\n- `automation_materialization_temp_root` — one-line `Storage::new(...).scratch_dir().join(\"automations\")`.\n- `automation_materialize_error` — one-line wrapper around `ApiError::new(UNPROCESSABLE_ENTITY, …)`.\n\n**Moved logic to its proper home:**\n- Promoted `enabled_api_trigger` from a private function in the HTTP handler to `Automation::enabled_api_trigger()` in the `fabro-automation` crate, where future trigger consumers (scheduler) can reuse it.\n\n**Quality tweaks:**\n- `chrono::Utc::now()` → `Utc::now()` via `use chrono::Utc;` to match the idiom in `runs.rs`.\n- Paginate before decorate in `list_automation_runs` so `decorate_run_summaries` only runs over the returned page instead of all filtered runs.\n\n**Considered and intentionally skipped:**\n- Adding `automation_id` to `fabro_store::ListRunsQuery` to push filtering into the store (would scale better but is explicitly out of scope — issue spec mandates in-memory filtering).\n- Sharing a list-runs envelope/sort helper between `list_runs` and `list_automation_runs` (would require exposing `RunsSortKey`/`RunsSortDirection` and a generic helper; the 4-line inline sort is short enough).\n- Collapsing `RequiredRunToolActor` into `RequiredRunManagementActor` (issue explicitly mandates the `RequiredRunToolActor` name).\n\n**Verification:** `cargo check --workspace --all-targets`, `cargo nextest run -p fabro-automation -p fabro-api` (176/176 pass), `cargo nextest run -p fabro-server --test it --features test-support` (171/171 pass, including all 24 automation API tests), `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` clean, `cargo +nightly-2026-04-14 fmt --check --all` clean. Net change: 37 inserts / 54 deletes across 4 files; `apps/fabro-web` and `lib/crates/fabro-cli` untouched.",
|
||||
"internal.retry_count.simplify_gpt": 0,
|
||||
"internal.retry_count.preflight_lint": 0,
|
||||
"outcome": "succeeded",
|
||||
"thread.start.current_node": "toolchain",
|
||||
"failure_class": "",
|
||||
"internal.work_dir": "/home/daytona/workspace/fabro",
|
||||
"response.implement": "Implemented Issue #399 automation run endpoints.\n\nSummary:\n- Added `GET /automations/{id}/runs`\n - Requires automation existence.\n - Lists cached persisted runs only.\n - Filters by `Run.automation.id`.\n - Sorts newest-first.\n - Uses existing `page[limit]` / `page[offset]` pagination envelope.\n\n- Added `POST /automations/{id}/runs`\n - Uses new `RequiredRunToolActor`.\n - Requires automation to exist, be enabled, and have an enabled API trigger.\n - Returns `409` with code `automation_api_trigger_disabled` for disabled automation/API trigger or missing API trigger.\n - Uses configured `AutomationRunMaterializer`.\n - Calls the shared run creation helper with populated `AutomationRef`.\n\n- Added integration coverage using the fake materializer for:\n - Disabled automation.\n - Disabled API trigger.\n - Missing API trigger.\n - Missing automation 404s.\n - Successful API-triggered run creation.\n - Persisted/retrievable `Run.automation`.\n - Automation run listing filtering.\n - Newest-first sorting and pagination.\n\nChanged files:\n- `lib/crates/fabro-server/src/server/handler/automations.rs`\n- `lib/crates/fabro-server/src/principal_middleware.rs`\n- `lib/crates/fabro-server/tests/it/api/automations.rs`\n\nVerification run:\n- `cargo nextest run -p fabro-automation` ✅\n- `cargo nextest run -p fabro-api` ✅\n- `cargo nextest run -p fabro-server --features test-support automations` ✅\n- `cargo nextest run -p fabro-server --features test-support openapi_conformance` ✅\n- `cargo check -p fabro-server` ✅\n- `cargo +nightly-2026-04-14 fmt --check --all` ✅\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` ✅\n- `git diff -- apps/fabro-web lib/crates/fabro-cli` ✅ empty\n\nNote: the `fabro-server` integration test binary is gated behind the crate’s `test-support` feature, so I used `--features test-support` for the focused `automations` and `openapi_conformance` integration runs.",
|
||||
"internal.retry_count.implement": 0,
|
||||
"response.simplify_gpt": "Review complete.\n\n- Working tree was already clean; reviewed committed changes vs `origin/main`.\n- Launched the three requested review agents. The tool reported completion but did not return actionable findings, so I also performed a direct reuse/quality/efficiency review of the diff.\n- No cleanup changes were needed; implementation looked focused and consistent.\n\nVerification run:\n\n- `cargo nextest run -p fabro-server --features test-support api::automations` ✅\n- `cargo nextest run -p fabro-server --features test-support openapi_conformance` ✅\n- `cargo nextest run -p fabro-automation` ✅\n- `cargo nextest run -p fabro-api` ✅\n- `cargo +nightly-2026-04-14 fmt --check --all` ✅\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` ✅\n- `git diff -- apps/fabro-web lib/crates/fabro-cli` ✅ no output\n- Final `git status --short` ✅ clean\n\nNote: Running `cargo nextest run -p fabro-server automations` without `--features test-support` found no integration tests because the `it` test target requires the `test-support` feature, so I reran with the required feature enabled.",
|
||||
"command.output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126",
|
||||
"thread.preflight_lint.current_node": "implement",
|
||||
"current_node": "simplify_gpt",
|
||||
"thread.toolchain.current_node": "preflight_compile",
|
||||
"thread.preflight_compile.current_node": "preflight_lint",
|
||||
"last_response": "Review complete.\n\n- Working tree was already clean; reviewed committed changes vs `origin/main`.\n- Launched the three requested review agents. The tool reported completion but did not return actionabl",
|
||||
"failure_signature": "",
|
||||
"internal.node_visit_count": 1,
|
||||
"internal.retry_count.preflight_compile": 0,
|
||||
"graph.model_stylesheet": "\n * { model: claude-opus-4-7; }\n "
|
||||
},
|
||||
"node_outcomes": {
|
||||
"simplify_gpt": {
|
||||
"status": "succeeded",
|
||||
"context_updates": {
|
||||
"response.simplify_gpt": "Review complete.\n\n- Working tree was already clean; reviewed committed changes vs `origin/main`.\n- Launched the three requested review agents. The tool reported completion but did not return actionable findings, so I also performed a direct reuse/quality/efficiency review of the diff.\n- No cleanup changes were needed; implementation looked focused and consistent.\n\nVerification run:\n\n- `cargo nextest run -p fabro-server --features test-support api::automations` ✅\n- `cargo nextest run -p fabro-server --features test-support openapi_conformance` ✅\n- `cargo nextest run -p fabro-automation` ✅\n- `cargo nextest run -p fabro-api` ✅\n- `cargo +nightly-2026-04-14 fmt --check --all` ✅\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` ✅\n- `git diff -- apps/fabro-web lib/crates/fabro-cli` ✅ no output\n- Final `git status --short` ✅ clean\n\nNote: Running `cargo nextest run -p fabro-server automations` without `--features test-support` found no integration tests because the `it` test target requires the `test-support` feature, so I reran with the required feature enabled.",
|
||||
"last_stage": "simplify_gpt",
|
||||
"last_response": "Review complete.\n\n- Working tree was already clean; reviewed committed changes vs `origin/main`.\n- Launched the three requested review agents. The tool reported completion but did not return actionabl"
|
||||
},
|
||||
"notes": "Stage completed: simplify_gpt",
|
||||
"usage": {
|
||||
"input": {
|
||||
"usage": {
|
||||
"model": {
|
||||
"provider": "openai",
|
||||
"model_id": "gpt-5.5"
|
||||
},
|
||||
"tokens": {
|
||||
"input_tokens": 383859,
|
||||
"output_tokens": 3670,
|
||||
"reasoning_tokens": 1261,
|
||||
"cache_read_tokens": 1061888,
|
||||
"cache_write_tokens": 0
|
||||
}
|
||||
},
|
||||
"facts": {
|
||||
"algorithm": "openai"
|
||||
}
|
||||
},
|
||||
"total_usd_micros": 2598169
|
||||
},
|
||||
"timing": {
|
||||
"wall_time_ms": 0,
|
||||
"inference_time_ms": 201837,
|
||||
"tool_time_ms": 97609,
|
||||
"active_time_ms": 299446
|
||||
}
|
||||
},
|
||||
"start": {
|
||||
"status": "succeeded",
|
||||
"usage": null
|
||||
},
|
||||
"implement": {
|
||||
"status": "succeeded",
|
||||
"context_updates": {
|
||||
"response.implement": "Implemented Issue #399 automation run endpoints.\n\nSummary:\n- Added `GET /automations/{id}/runs`\n - Requires automation existence.\n - Lists cached persisted runs only.\n - Filters by `Run.automation.id`.\n - Sorts newest-first.\n - Uses existing `page[limit]` / `page[offset]` pagination envelope.\n\n- Added `POST /automations/{id}/runs`\n - Uses new `RequiredRunToolActor`.\n - Requires automation to exist, be enabled, and have an enabled API trigger.\n - Returns `409` with code `automation_api_trigger_disabled` for disabled automation/API trigger or missing API trigger.\n - Uses configured `AutomationRunMaterializer`.\n - Calls the shared run creation helper with populated `AutomationRef`.\n\n- Added integration coverage using the fake materializer for:\n - Disabled automation.\n - Disabled API trigger.\n - Missing API trigger.\n - Missing automation 404s.\n - Successful API-triggered run creation.\n - Persisted/retrievable `Run.automation`.\n - Automation run listing filtering.\n - Newest-first sorting and pagination.\n\nChanged files:\n- `lib/crates/fabro-server/src/server/handler/automations.rs`\n- `lib/crates/fabro-server/src/principal_middleware.rs`\n- `lib/crates/fabro-server/tests/it/api/automations.rs`\n\nVerification run:\n- `cargo nextest run -p fabro-automation` ✅\n- `cargo nextest run -p fabro-api` ✅\n- `cargo nextest run -p fabro-server --features test-support automations` ✅\n- `cargo nextest run -p fabro-server --features test-support openapi_conformance` ✅\n- `cargo check -p fabro-server` ✅\n- `cargo +nightly-2026-04-14 fmt --check --all` ✅\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` ✅\n- `git diff -- apps/fabro-web lib/crates/fabro-cli` ✅ empty\n\nNote: the `fabro-server` integration test binary is gated behind the crate’s `test-support` feature, so I used `--features test-support` for the focused `automations` and `openapi_conformance` integration runs.",
|
||||
"last_response": "Implemented Issue #399 automation run endpoints.\n\nSummary:\n- Added `GET /automations/{id}/runs`\n - Requires automation existence.\n - Lists cached persisted runs only.\n - Filters by `Run.automation.",
|
||||
"last_stage": "implement"
|
||||
},
|
||||
"notes": "Stage completed: implement",
|
||||
"usage": {
|
||||
"input": {
|
||||
"usage": {
|
||||
"model": {
|
||||
"provider": "openai",
|
||||
"model_id": "gpt-5.5"
|
||||
},
|
||||
"tokens": {
|
||||
"input_tokens": 1645117,
|
||||
"output_tokens": 13240,
|
||||
"reasoning_tokens": 11182,
|
||||
"cache_read_tokens": 8178176,
|
||||
"cache_write_tokens": 0
|
||||
}
|
||||
},
|
||||
"facts": {
|
||||
"algorithm": "openai"
|
||||
}
|
||||
},
|
||||
"total_usd_micros": 13047333
|
||||
},
|
||||
"timing": {
|
||||
"wall_time_ms": 0,
|
||||
"inference_time_ms": 818239,
|
||||
"tool_time_ms": 427741,
|
||||
"active_time_ms": 1245980
|
||||
}
|
||||
},
|
||||
"simplify_opus": {
|
||||
"status": "succeeded",
|
||||
"context_updates": {
|
||||
"last_response": "Web UI and CLI remain untouched.\n\n## Summary\n\nReviewed 3 changed files via parallel reuse/quality/efficiency agents and applied the following cleanups:\n\n**Removed dead/redundant code:**\n- Deleted `req",
|
||||
"response.simplify_opus": "Web UI and CLI remain untouched.\n\n## Summary\n\nReviewed 3 changed files via parallel reuse/quality/efficiency agents and applied the following cleanups:\n\n**Removed dead/redundant code:**\n- Deleted `require_run_tool_actor` helper in `principal_middleware.rs` — was a trivial alias for `require_run_management_actor`; `RequiredRunToolActor::from_request_parts` now calls the real function directly.\n- Removed stale `#[allow(dead_code, reason = \"...\")]` on `AppState::materialize_automation_run` in `server.rs` (now called from `create_automation_run`).\n\n**Inlined single-use helpers in `automations.rs`:**\n- `automation_api_trigger_disabled_error` + the matching const `AUTOMATION_API_TRIGGER_DISABLED_CODE` — only used by one `else` branch.\n- `automation_materialization_temp_root` — one-line `Storage::new(...).scratch_dir().join(\"automations\")`.\n- `automation_materialize_error` — one-line wrapper around `ApiError::new(UNPROCESSABLE_ENTITY, …)`.\n\n**Moved logic to its proper home:**\n- Promoted `enabled_api_trigger` from a private function in the HTTP handler to `Automation::enabled_api_trigger()` in the `fabro-automation` crate, where future trigger consumers (scheduler) can reuse it.\n\n**Quality tweaks:**\n- `chrono::Utc::now()` → `Utc::now()` via `use chrono::Utc;` to match the idiom in `runs.rs`.\n- Paginate before decorate in `list_automation_runs` so `decorate_run_summaries` only runs over the returned page instead of all filtered runs.\n\n**Considered and intentionally skipped:**\n- Adding `automation_id` to `fabro_store::ListRunsQuery` to push filtering into the store (would scale better but is explicitly out of scope — issue spec mandates in-memory filtering).\n- Sharing a list-runs envelope/sort helper between `list_runs` and `list_automation_runs` (would require exposing `RunsSortKey`/`RunsSortDirection` and a generic helper; the 4-line inline sort is short enough).\n- Collapsing `RequiredRunToolActor` into `RequiredRunManagementActor` (issue explicitly mandates the `RequiredRunToolActor` name).\n\n**Verification:** `cargo check --workspace --all-targets`, `cargo nextest run -p fabro-automation -p fabro-api` (176/176 pass), `cargo nextest run -p fabro-server --test it --features test-support` (171/171 pass, including all 24 automation API tests), `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` clean, `cargo +nightly-2026-04-14 fmt --check --all` clean. Net change: 37 inserts / 54 deletes across 4 files; `apps/fabro-web` and `lib/crates/fabro-cli` untouched.",
|
||||
"last_stage": "simplify_opus"
|
||||
},
|
||||
"notes": "Stage completed: simplify_opus",
|
||||
"usage": {
|
||||
"input": {
|
||||
"usage": {
|
||||
"model": {
|
||||
"provider": "anthropic",
|
||||
"model_id": "claude-opus-4-7"
|
||||
},
|
||||
"tokens": {
|
||||
"input_tokens": 91335,
|
||||
"output_tokens": 20965,
|
||||
"reasoning_tokens": 0,
|
||||
"cache_read_tokens": 3664894,
|
||||
"cache_write_tokens": 461333
|
||||
}
|
||||
},
|
||||
"facts": {
|
||||
"algorithm": "anthropic",
|
||||
"cache_write_5m_tokens": 461333,
|
||||
"cache_write_1h_tokens": 0
|
||||
}
|
||||
},
|
||||
"total_usd_micros": 5696578
|
||||
},
|
||||
"files_touched": [
|
||||
"/home/daytona/workspace/fabro/lib/crates/fabro-automation/src/model.rs",
|
||||
"/home/daytona/workspace/fabro/lib/crates/fabro-server/src/principal_middleware.rs",
|
||||
"/home/daytona/workspace/fabro/lib/crates/fabro-server/src/server.rs",
|
||||
"/home/daytona/workspace/fabro/lib/crates/fabro-server/src/server/handler/automations.rs"
|
||||
],
|
||||
"timing": {
|
||||
"wall_time_ms": 0,
|
||||
"inference_time_ms": 364281,
|
||||
"tool_time_ms": 300326,
|
||||
"active_time_ms": 664607
|
||||
}
|
||||
},
|
||||
"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": 143345,
|
||||
"active_time_ms": 143345
|
||||
}
|
||||
},
|
||||
"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": 1398,
|
||||
"active_time_ms": 1398
|
||||
}
|
||||
},
|
||||
"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": 128511,
|
||||
"active_time_ms": 128511
|
||||
}
|
||||
}
|
||||
},
|
||||
"next_node_id": "verify",
|
||||
"git_commit_sha": "153297b34627cc953ba64a97051abb2f8e0cd8d2",
|
||||
"node_visits": {
|
||||
"implement": 1,
|
||||
"preflight_lint": 1,
|
||||
"simplify_opus": 1,
|
||||
"simplify_gpt": 1,
|
||||
"toolchain": 1,
|
||||
"preflight_compile": 1,
|
||||
"start": 1
|
||||
}
|
||||
},
|
||||
"diff": {
|
||||
"summary": {
|
||||
"files_changed": 5,
|
||||
"additions": 363,
|
||||
"deletions": 13
|
||||
}
|
||||
}
|
||||
},
|
||||
{
|
||||
"seq": 0,
|
||||
"checkpoint": {
|
||||
"timestamp": "2026-05-28T01:08:19.665710Z",
|
||||
"current_node": "verify",
|
||||
"completed_nodes": [
|
||||
"start",
|
||||
"toolchain",
|
||||
"preflight_compile",
|
||||
"preflight_lint",
|
||||
"implement",
|
||||
"simplify_opus",
|
||||
"simplify_gpt",
|
||||
"verify"
|
||||
],
|
||||
"node_retries": {},
|
||||
"context_values": {
|
||||
"response.simplify_opus": "Web UI and CLI remain untouched.\n\n## Summary\n\nReviewed 3 changed files via parallel reuse/quality/efficiency agents and applied the following cleanups:\n\n**Removed dead/redundant code:**\n- Deleted `require_run_tool_actor` helper in `principal_middleware.rs` — was a trivial alias for `require_run_management_actor`; `RequiredRunToolActor::from_request_parts` now calls the real function directly.\n- Removed stale `#[allow(dead_code, reason = \"...\")]` on `AppState::materialize_automation_run` in `server.rs` (now called from `create_automation_run`).\n\n**Inlined single-use helpers in `automations.rs`:**\n- `automation_api_trigger_disabled_error` + the matching const `AUTOMATION_API_TRIGGER_DISABLED_CODE` — only used by one `else` branch.\n- `automation_materialization_temp_root` — one-line `Storage::new(...).scratch_dir().join(\"automations\")`.\n- `automation_materialize_error` — one-line wrapper around `ApiError::new(UNPROCESSABLE_ENTITY, …)`.\n\n**Moved logic to its proper home:**\n- Promoted `enabled_api_trigger` from a private function in the HTTP handler to `Automation::enabled_api_trigger()` in the `fabro-automation` crate, where future trigger consumers (scheduler) can reuse it.\n\n**Quality tweaks:**\n- `chrono::Utc::now()` → `Utc::now()` via `use chrono::Utc;` to match the idiom in `runs.rs`.\n- Paginate before decorate in `list_automation_runs` so `decorate_run_summaries` only runs over the returned page instead of all filtered runs.\n\n**Considered and intentionally skipped:**\n- Adding `automation_id` to `fabro_store::ListRunsQuery` to push filtering into the store (would scale better but is explicitly out of scope — issue spec mandates in-memory filtering).\n- Sharing a list-runs envelope/sort helper between `list_runs` and `list_automation_runs` (would require exposing `RunsSortKey`/`RunsSortDirection` and a generic helper; the 4-line inline sort is short enough).\n- Collapsing `RequiredRunToolActor` into `RequiredRunManagementActor` (issue explicitly mandates the `RequiredRunToolActor` name).\n\n**Verification:** `cargo check --workspace --all-targets`, `cargo nextest run -p fabro-automation -p fabro-api` (176/176 pass), `cargo nextest run -p fabro-server --test it --features test-support` (171/171 pass, including all 24 automation API tests), `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` clean, `cargo +nightly-2026-04-14 fmt --check --all` clean. Net change: 37 inserts / 54 deletes across 4 files; `apps/fabro-web` and `lib/crates/fabro-cli` untouched.",
|
||||
"internal.retry_count.simplify_gpt": 0,
|
||||
|
|
@ -1145,10 +1378,11 @@
|
|||
"internal.fidelity": "compact",
|
||||
"internal.node_visit_count": 1,
|
||||
"last_stage": "simplify_gpt",
|
||||
"internal.thread_id": "simplify_opus",
|
||||
"internal.retry_count.verify": 0,
|
||||
"internal.thread_id": "simplify_gpt",
|
||||
"internal.retry_count.implement": 0,
|
||||
"failure_class": "",
|
||||
"command.output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126",
|
||||
"command.output": "blob://sha256/a197e7ab024f17d58dff943b5a0eeaa545ba6b50528d00269ea25c5ff79810da",
|
||||
"graph.model_stylesheet": "\n * { model: claude-opus-4-7; }\n ",
|
||||
"graph.rankdir": "LR",
|
||||
"internal.retry_count.toolchain": 0,
|
||||
|
|
@ -1157,7 +1391,7 @@
|
|||
"failure_signature": "",
|
||||
"internal.retry_count.start": 0,
|
||||
"thread.simplify_opus.current_node": "simplify_gpt",
|
||||
"current_node": "simplify_gpt",
|
||||
"current_node": "verify",
|
||||
"internal.retry_count.simplify_opus": 0,
|
||||
"internal.retry_count.preflight_compile": 0,
|
||||
"internal.run_id": "01KSNZ26XXV9Y70FDJQHBQ0CC4",
|
||||
|
|
@ -1168,6 +1402,7 @@
|
|||
"response.simplify_gpt": "Review complete.\n\n- Working tree was already clean; reviewed committed changes vs `origin/main`.\n- Launched the three requested review agents. The tool reported completion but did not return actionable findings, so I also performed a direct reuse/quality/efficiency review of the diff.\n- No cleanup changes were needed; implementation looked focused and consistent.\n\nVerification run:\n\n- `cargo nextest run -p fabro-server --features test-support api::automations` ✅\n- `cargo nextest run -p fabro-server --features test-support openapi_conformance` ✅\n- `cargo nextest run -p fabro-automation` ✅\n- `cargo nextest run -p fabro-api` ✅\n- `cargo +nightly-2026-04-14 fmt --check --all` ✅\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` ✅\n- `git diff -- apps/fabro-web lib/crates/fabro-cli` ✅ no output\n- Final `git status --short` ✅ clean\n\nNote: Running `cargo nextest run -p fabro-server automations` without `--features test-support` found no integration tests because the `it` test target requires the `test-support` feature, so I reran with the required feature enabled.",
|
||||
"thread.start.current_node": "toolchain",
|
||||
"graph.goal": "# Issue #399: Add automation run endpoints\n\n- URL: https://github.com/fabro-sh/fabro/issues/399\n- State: OPEN\n- Author: Bryan Helmkamp (@brynary)\n- Created: 2026-05-25T15:06:27Z\n- Updated: 2026-05-25T15:06:27Z\n- Labels: None\n- Assignees: None\n- Milestone: None\n- Comments: 0\n\n---\n\n## Goal\n\nExpose API endpoints for listing runs associated with an automation and starting a run through an enabled API trigger.\n\n## Scope\n\nImplement these endpoints:\n\n```http\nGET /automations/{id}/runs\nPOST /automations/{id}/runs\n```\n\n`GET /automations/{id}/runs` behavior:\n\n- Require the automation definition to exist; return 404 when it does not.\n- List cached runs from the existing run store.\n- Filter by `run.automation.as_ref().is_some_and(|a| a.id == id)`.\n- Sort newest first.\n- Support `page[limit]` and `page[offset]` using existing pagination behavior.\n- Return the existing paginated run list envelope:\n\n```json\n{\n \"data\": [],\n \"meta\": { \"has_more\": false, \"total\": 0 }\n}\n```\n\n`POST /automations/{id}/runs` behavior:\n\n- Use `RequiredRunToolActor`.\n- Require the automation to exist and be enabled.\n- Find an enabled trigger where `type = \"api\"`.\n- Return 409 with API error code `automation_api_trigger_disabled` when the automation is disabled or no enabled API trigger is available.\n- Materialize the run manifest using the configured `AutomationRunMaterializer`.\n- Call the shared create-run helper with:\n\n```rust\nAutomationRef {\n id: automation.id.to_string(),\n name: Some(automation.name.clone()),\n trigger_id: Some(api_trigger.id.to_string()),\n}\n```\n\n- Return 201 and the normal `Run` response shape with automation metadata populated.\n\nFinal integration expectations:\n\n- Automation-created runs are visible through normal run APIs.\n- Automation-created runs are visible through `GET /automations/{id}/runs`.\n- Run history is derived from persisted/cached runs; no runtime automation state store is introduced.\n- Schedule trigger expressions are stored and validated by earlier phases but are not scheduled by this endpoint work.\n\n## Files\n\nModify:\n\n- `lib/crates/fabro-server/src/server/handler/automations.rs`\n- `lib/crates/fabro-server/src/server/handler/runs.rs`, only if additional helper exposure is needed from the previous phase\n- `lib/crates/fabro-server/tests/it/api/automations.rs`\n- `lib/crates/fabro-server/tests/it/api/mod.rs`\n\n## Acceptance Criteria\n\n- Disabled automations cannot start runs through the automation run endpoint.\n- Automations without an enabled API trigger cannot start runs through the automation run endpoint.\n- A successful API-triggered automation run returns a normal `Run` response with `automation.id`, `automation.name`, and `automation.trigger_id`.\n- The automation run listing endpoint returns only runs linked to that automation.\n- Automation run listings are newest-first and paginate correctly.\n- No scheduler, web UI route/component, or CLI command is added.\n\n## Verification\n\nAdd integration tests using the fake materializer for:\n\n- Disabled automation returns 409.\n- Disabled API trigger returns 409.\n- Missing API trigger returns 409.\n- Successful run creation returns 201.\n- Created run persists `Run.automation`.\n- Associated run listing includes the run.\n- Run listing excludes runs from other automations.\n- Run listing pagination and newest-first sorting.\n\nRun:\n\n```bash\ncargo nextest run -p fabro-automation\ncargo nextest run -p fabro-api\ncargo nextest run -p fabro-server automations\ncargo nextest run -p fabro-server openapi_conformance\ncargo +nightly-2026-04-14 fmt --check --all\ncargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings\ngit diff -- apps/fabro-web lib/crates/fabro-cli\n```\n\nExpected: focused tests and checks pass; web UI and CLI command modules remain unchanged.\n",
|
||||
"thread.simplify_gpt.current_node": "verify",
|
||||
"last_response": "Review complete.\n\n- Working tree was already clean; reviewed committed changes vs `origin/main`.\n- Launched the three requested review agents. The tool reported completion but did not return actionabl",
|
||||
"response.implement": "Implemented Issue #399 automation run endpoints.\n\nSummary:\n- Added `GET /automations/{id}/runs`\n - Requires automation existence.\n - Lists cached persisted runs only.\n - Filters by `Run.automation.id`.\n - Sorts newest-first.\n - Uses existing `page[limit]` / `page[offset]` pagination envelope.\n\n- Added `POST /automations/{id}/runs`\n - Uses new `RequiredRunToolActor`.\n - Requires automation to exist, be enabled, and have an enabled API trigger.\n - Returns `409` with code `automation_api_trigger_disabled` for disabled automation/API trigger or missing API trigger.\n - Uses configured `AutomationRunMaterializer`.\n - Calls the shared run creation helper with populated `AutomationRef`.\n\n- Added integration coverage using the fake materializer for:\n - Disabled automation.\n - Disabled API trigger.\n - Missing API trigger.\n - Missing automation 404s.\n - Successful API-triggered run creation.\n - Persisted/retrievable `Run.automation`.\n - Automation run listing filtering.\n - Newest-first sorting and pagination.\n\nChanged files:\n- `lib/crates/fabro-server/src/server/handler/automations.rs`\n- `lib/crates/fabro-server/src/principal_middleware.rs`\n- `lib/crates/fabro-server/tests/it/api/automations.rs`\n\nVerification run:\n- `cargo nextest run -p fabro-automation` ✅\n- `cargo nextest run -p fabro-api` ✅\n- `cargo nextest run -p fabro-server --features test-support automations` ✅\n- `cargo nextest run -p fabro-server --features test-support openapi_conformance` ✅\n- `cargo check -p fabro-server` ✅\n- `cargo +nightly-2026-04-14 fmt --check --all` ✅\n- `cargo +nightly-2026-04-14 clippy --workspace --all-targets -- -D warnings` ✅\n- `git diff -- apps/fabro-web lib/crates/fabro-cli` ✅ empty\n\nNote: the `fabro-server` integration test binary is gated behind the crate’s `test-support` feature, so I used `--features test-support` for the focused `automations` and `openapi_conformance` integration runs."
|
||||
},
|
||||
|
|
@ -1294,6 +1529,20 @@
|
|||
"active_time_ms": 128511
|
||||
}
|
||||
},
|
||||
"verify": {
|
||||
"status": "succeeded",
|
||||
"context_updates": {
|
||||
"command.output": "blob://sha256/a197e7ab024f17d58dff943b5a0eeaa545ba6b50528d00269ea25c5ff79810da"
|
||||
},
|
||||
"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": 544732,
|
||||
"active_time_ms": 544732
|
||||
}
|
||||
},
|
||||
"start": {
|
||||
"status": "succeeded",
|
||||
"usage": null
|
||||
|
|
@ -1335,15 +1584,16 @@
|
|||
}
|
||||
}
|
||||
},
|
||||
"next_node_id": "verify",
|
||||
"next_node_id": "exit",
|
||||
"node_visits": {
|
||||
"toolchain": 1,
|
||||
"simplify_gpt": 1,
|
||||
"implement": 1,
|
||||
"preflight_lint": 1,
|
||||
"preflight_compile": 1,
|
||||
"simplify_opus": 1,
|
||||
"start": 1,
|
||||
"implement": 1,
|
||||
"preflight_lint": 1,
|
||||
"simplify_gpt": 1
|
||||
"verify": 1
|
||||
}
|
||||
},
|
||||
"diff": {}
|
||||
|
|
@ -1379,7 +1629,12 @@
|
|||
"first_event_seq": 748,
|
||||
"prompt": null,
|
||||
"response": null,
|
||||
"completion": null,
|
||||
"completion": {
|
||||
"outcome": "succeeded",
|
||||
"notes": "Stage completed: simplify_gpt",
|
||||
"failure_reason": null,
|
||||
"timestamp": "2026-05-28T00:59:11.420414Z"
|
||||
},
|
||||
"provider_used": {
|
||||
"mode": "agent",
|
||||
"provider": "openai",
|
||||
|
|
@ -1392,6 +1647,12 @@
|
|||
"output": null,
|
||||
"started_at": "2026-05-28T00:54:11.295100Z",
|
||||
"handler": "agent",
|
||||
"timing": {
|
||||
"wall_time_ms": 300123,
|
||||
"inference_time_ms": 201837,
|
||||
"tool_time_ms": 97609,
|
||||
"active_time_ms": 299446
|
||||
},
|
||||
"usage": {
|
||||
"input_tokens": 383859,
|
||||
"output_tokens": 3670,
|
||||
|
|
@ -1635,7 +1896,7 @@
|
|||
],
|
||||
"warnings": []
|
||||
},
|
||||
"state": "running"
|
||||
"state": "succeeded"
|
||||
},
|
||||
"preflight_compile@1": {
|
||||
"first_event_seq": 31,
|
||||
|
|
@ -1685,6 +1946,54 @@
|
|||
},
|
||||
"state": "succeeded"
|
||||
},
|
||||
"preflight_lint@1": {
|
||||
"first_event_seq": 41,
|
||||
"prompt": null,
|
||||
"response": null,
|
||||
"completion": {
|
||||
"outcome": "succeeded",
|
||||
"notes": "Script completed: cargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
|
||||
"failure_reason": null,
|
||||
"timestamp": "2026-05-28T00:22:01.214434Z"
|
||||
},
|
||||
"provider_used": null,
|
||||
"diff": null,
|
||||
"script_invocation": {
|
||||
"script": "cargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
|
||||
"command": "exec 2>&1\ncargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
|
||||
"language": "shell"
|
||||
},
|
||||
"script_timing": {
|
||||
"output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126",
|
||||
"exit_code": 0,
|
||||
"duration_ms": 143345,
|
||||
"termination": "exited",
|
||||
"output_bytes": 0,
|
||||
"live_streaming": false
|
||||
},
|
||||
"parallel_results": null,
|
||||
"output": null,
|
||||
"output_bytes": 0,
|
||||
"live_streaming": false,
|
||||
"termination": "exited",
|
||||
"started_at": "2026-05-28T00:19:37.857664Z",
|
||||
"handler": "command",
|
||||
"timing": {
|
||||
"wall_time_ms": 143356,
|
||||
"inference_time_ms": 0,
|
||||
"tool_time_ms": 143345,
|
||||
"active_time_ms": 143345
|
||||
},
|
||||
"usage": {
|
||||
"input_tokens": 0,
|
||||
"output_tokens": 0,
|
||||
"total_tokens": 0,
|
||||
"reasoning_tokens": 0,
|
||||
"cache_read_tokens": 0,
|
||||
"cache_write_tokens": 0
|
||||
},
|
||||
"state": "succeeded"
|
||||
},
|
||||
"toolchain@1": {
|
||||
"first_event_seq": 21,
|
||||
"prompt": null,
|
||||
|
|
@ -1945,54 +2254,6 @@
|
|||
},
|
||||
"state": "succeeded"
|
||||
},
|
||||
"preflight_lint@1": {
|
||||
"first_event_seq": 41,
|
||||
"prompt": null,
|
||||
"response": null,
|
||||
"completion": {
|
||||
"outcome": "succeeded",
|
||||
"notes": "Script completed: cargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
|
||||
"failure_reason": null,
|
||||
"timestamp": "2026-05-28T00:22:01.214434Z"
|
||||
},
|
||||
"provider_used": null,
|
||||
"diff": null,
|
||||
"script_invocation": {
|
||||
"script": "cargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
|
||||
"command": "exec 2>&1\ncargo +nightly-2026-04-14 clippy -q --workspace --all-targets -- -D warnings 2>&1",
|
||||
"language": "shell"
|
||||
},
|
||||
"script_timing": {
|
||||
"output": "blob://sha256/12ae32cb1ec02d01eda3581b127c1fee3b0dc53572ed6baf239721a03d82e126",
|
||||
"exit_code": 0,
|
||||
"duration_ms": 143345,
|
||||
"termination": "exited",
|
||||
"output_bytes": 0,
|
||||
"live_streaming": false
|
||||
},
|
||||
"parallel_results": null,
|
||||
"output": null,
|
||||
"output_bytes": 0,
|
||||
"live_streaming": false,
|
||||
"termination": "exited",
|
||||
"started_at": "2026-05-28T00:19:37.857664Z",
|
||||
"handler": "command",
|
||||
"timing": {
|
||||
"wall_time_ms": 143356,
|
||||
"inference_time_ms": 0,
|
||||
"tool_time_ms": 143345,
|
||||
"active_time_ms": 143345
|
||||
},
|
||||
"usage": {
|
||||
"input_tokens": 0,
|
||||
"output_tokens": 0,
|
||||
"total_tokens": 0,
|
||||
"reasoning_tokens": 0,
|
||||
"cache_read_tokens": 0,
|
||||
"cache_write_tokens": 0
|
||||
},
|
||||
"state": "succeeded"
|
||||
},
|
||||
"simplify_opus@1": {
|
||||
"first_event_seq": 402,
|
||||
"prompt": null,
|
||||
|
|
@ -2263,6 +2524,33 @@
|
|||
},
|
||||
"state": "succeeded"
|
||||
},
|
||||
"verify@1": {
|
||||
"first_event_seq": 978,
|
||||
"prompt": null,
|
||||
"response": null,
|
||||
"completion": null,
|
||||
"provider_used": null,
|
||||
"diff": null,
|
||||
"script_invocation": {
|
||||
"script": "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",
|
||||
"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,
|
||||
"parallel_results": null,
|
||||
"output": null,
|
||||
"started_at": "2026-05-28T00:59:14.909667Z",
|
||||
"handler": "command",
|
||||
"usage": {
|
||||
"input_tokens": 0,
|
||||
"output_tokens": 0,
|
||||
"total_tokens": 0,
|
||||
"reasoning_tokens": 0,
|
||||
"cache_read_tokens": 0,
|
||||
"cache_write_tokens": 0
|
||||
},
|
||||
"state": "running"
|
||||
},
|
||||
"start@1": {
|
||||
"first_event_seq": 17,
|
||||
"prompt": null,
|
||||
|
|
|
|||
6
stages/007-simplify_gpt@1/status.json
Normal file
6
stages/007-simplify_gpt@1/status.json
Normal file
|
|
@ -0,0 +1,6 @@
|
|||
{
|
||||
"outcome": "succeeded",
|
||||
"notes": "Stage completed: simplify_gpt",
|
||||
"failure_reason": null,
|
||||
"timestamp": "2026-05-28T00:59:11.420414Z"
|
||||
}
|
||||
5
stages/008-verify@1/script_invocation.json
Normal file
5
stages/008-verify@1/script_invocation.json
Normal file
|
|
@ -0,0 +1,5 @@
|
|||
{
|
||||
"script": "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",
|
||||
"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"
|
||||
}
|
||||
Loading…
Add table
Reference in a new issue