mirror of
https://github.com/fabro-sh/fabro.git
synced 2026-10-09 03:20:56 +00:00
parent
f12eac42b7
commit
0b7a989171
5 changed files with 347 additions and 17 deletions
339
run.json
339
run.json
File diff suppressed because one or more lines are too long
13
stages/009-fixup@1/diff.patch
Normal file
13
stages/009-fixup@1/diff.patch
Normal file
|
|
@ -0,0 +1,13 @@
|
|||
diff --git a/lib/crates/fabro-server/tests/it/api/runs.rs b/lib/crates/fabro-server/tests/it/api/runs.rs
|
||||
index c6521ee7c..0c2982598 100644
|
||||
--- a/lib/crates/fabro-server/tests/it/api/runs.rs
|
||||
+++ b/lib/crates/fabro-server/tests/it/api/runs.rs
|
||||
@@ -139,7 +139,7 @@ _version = 1
|
||||
);
|
||||
let state = fabro_server::test_support::TestAppStateBuilder::new()
|
||||
.runtime_settings(settings.server_settings, settings.manifest_run_defaults)
|
||||
- .env_lookup(|name| (name == "OPENAI_API_KEY").then(|| "test-key".to_string()))
|
||||
+ .vault_entries([("OPENAI_API_KEY", "test-key")])
|
||||
.build();
|
||||
let app = fabro_server::test_support::build_test_router(state);
|
||||
let created = create_run(&app, minimal_manifest_json(MINIMAL_DOT)).await;
|
||||
1
stages/009-fixup@1/response.md
Normal file
1
stages/009-fixup@1/response.md
Normal file
|
|
@ -0,0 +1 @@
|
|||
All 6405 tests pass. The fix was a single-line change to `lib/crates/fabro-server/tests/it/api/runs.rs` — the test was injecting `OPENAI_API_KEY` via `env_lookup`, but after the secrets rationalization the server LLM credentials are vault-only, so it needed to use `vault_entries` instead.
|
||||
6
stages/009-fixup@1/status.json
Normal file
6
stages/009-fixup@1/status.json
Normal file
|
|
@ -0,0 +1,6 @@
|
|||
{
|
||||
"outcome": "succeeded",
|
||||
"notes": "Stage completed: fixup",
|
||||
"failure_reason": null,
|
||||
"timestamp": "2026-05-25T18:53:44.550725Z"
|
||||
}
|
||||
5
stages/010-verify@2/script_invocation.json
Normal file
5
stages/010-verify@2/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