afi / afi · cli reviewed proxy at 9b51f95e with Default profile / Google · gemini 3.7 flash high on afi 0.30.0
Unavailable values retain their exported state and reason
Lifecycle, reviewer, build, settings, and normalized execution limits
ccf2033be64fconfig-962dd9893b4bf224b0f313c0b59a66ecc7612396dc8db0ea5da13a7acomparison-99aa0f2063f28380build-5c6619f952748951cohort-fdab2adfb8a4bd39c4bc25bc962489fe5de7f71b59f19b2f426564c0 dirtysha256:a1a935298956020ee6d767a756847abb4c00694c88c2eb80d454886ebc4acf8cNormalized reviewer-tool usage projection
Normalized assessment summary
Review text is escaped; normalized decisions follow each finding
candidates.sort_by_key(|candidate| candidate.age)` sorts in ascending order of elapsed duration, causing the newest blobs to be evicted first instead of the oldest.
services/proxy/src/sweep.rs:102
Evidence quote
Not recorded
Suggested fix
Not recorded
forget` deletes index files from disk unconditionally even when `dry_run` is true, destroying cache index entries during a dry run.
services/proxy/src/sweep.rs:140
Evidence quote
Not recorded
Suggested fix
Not recorded
Metrics::values` loads `sweep_bytes_reclaimed` before `sweep_blobs_removed`, reversing the declaration order in `COUNTERS` and swapping the Prometheus values for reclaimed bytes and removed blob… Metrics::values` loads `sweep_bytes_reclaimed` before `sweep_blobs_removed`, reversing the declaration order in `COUNTERS` and swapping the Prometheus values for reclaimed bytes and removed blob count.
services/proxy/src/metrics.rs:94
Evidence quote
Not recorded
Suggested fix
Not recorded
tokio::time::interval` yields its first tick immediately, causing the proxy to sweep the blob store at startup rather than waiting for the configured sweep interval.
services/proxy/src/main.rs:108
Evidence quote
Not recorded
Suggested fix
Not recorded
reclaim` calls `app.sweeper.sweep(dry_run)` directly without acquiring the `running` mutex, allowing manual sweeps to run concurrently with background sweeps or other manual sweeps.
services/proxy/src/routes/admin.rs:76
Evidence quote
Not recorded
Suggested fix
Not recorded
Open a row to inspect or download evidence from this atomically published local run
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link
Open this path at the exported commit
The artifact is present; its content is available through the verified source link