scriptoriumadmin/fabro
1
0
Fork 0
mirror of https://github.com/fabro-sh/fabro.git synced 2026-10-01 02:04:24 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 3

Settle clippy and formatting after the sandbox merge

Browse source 
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
This commit is contained in:
Bryan Helmkamp 2026-09-18 18:39:58 -04:00
parent b354a2f948
commit 6aec33c4f5
No known key found for this signature in database
4 changed files with 13 additions and 16 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

17
lib/apps/fabro-server/src/server/handler/artifacts.rs
View file

@ -417,16 +417,13 @@ async fn download_run_artifacts(
Ok(projection) => projection,
Err(error) => return error.into_response(),
};
let entries = match run_artifacts(state.as_ref(), &id, &projection).await {
Ok(entries) => entries,
Err(_) => {
warn!(run_id = %id, "failed to list artifacts for ZIP download");
return ApiError::new(
StatusCode::INTERNAL_SERVER_ERROR,
"Artifact archive could not be prepared.",
)
.into_response();
}
let Ok(entries) = run_artifacts(state.as_ref(), &id, &projection).await else {
warn!(run_id = %id, "failed to list artifacts for ZIP download");
return ApiError::new(
StatusCode::INTERNAL_SERVER_ERROR,
"Artifact archive could not be prepared.",
)
.into_response();
};
let artifacts = latest_run_artifacts(entries, &projection);

2
lib/components/fabro-dump/src/lib.rs
View file

@ -766,7 +766,7 @@ mod tests {
fn hydrate_referenced_blobs_takes_a_plain_reference_as_text() {
// A large string leaves the run context as its own bytes, under a
// plain reference: the bytes are the text, not JSON.
let blob = b"x".repeat(12).to_vec();
let blob = b"x".repeat(12);
let blob_hash = fabro_types::BlobHash::new(&blob);
let blob_ref = fabro_types::format_blob_ref(&blob_hash);
let mut dump = RunDump {

9
lib/components/fabro-petri/src/projection.rs
View file

@ -38,11 +38,10 @@ use fabro_types::{
ModelRef, ModelUsage, ParallelBranchId, ParallelBranchResult, PendingInterviewRecord,
PullRequestCreation, PullRequestCreationStatus, PullRequestLink, RunApproval, RunApprovalState,
RunArtifact, RunControlAction, RunDiff, RunFailure, RunId, RunProjection, RunSandbox,
RunSandboxFailure, RunSandboxInstance, RunSandboxPlan, RunSandboxRuntime, RunStatus,
RunTiming, SandboxProviderKind, StageCompletion, StageHandler, StageId,
StageInferenceProjection, StageModelUsage, StageOutcome, StageProjection, StageState,
StageTiming, StartRecord, SuccessReason, first_event_seq, format_blob_ref, parse_blob_ref,
timing, usage_rollup,
RunSandboxFailure, RunSandboxInstance, RunSandboxPlan, RunSandboxRuntime, RunStatus, RunTiming,
SandboxProviderKind, StageCompletion, StageHandler, StageId, StageInferenceProjection,
StageModelUsage, StageOutcome, StageProjection, StageState, StageTiming, StartRecord,
SuccessReason, first_event_seq, format_blob_ref, parse_blob_ref, timing, usage_rollup,
};
use lithos_llm::catalog::{ModelId, ProviderId};
use lithos_llm::types::Usage;

1
lib/foundation/fabro-config/migrations/2026091801_remove_server_slatedb.rs
View file

@ -13,6 +13,7 @@
#![expect(
clippy::disallowed_methods,
clippy::disallowed_types,
reason = "temporary startup config migration uses synchronous file I/O before config is loaded"
)]