Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 require-ready-label-and-ci

Greptile Summary

This PR adds SQLite-backed fixture resolution to the live dogfood pipeline. When the CLI exposes a sync command, a quiet pre-sync (capped at 5 s) primes the local store, then each companion-failure fallback site queries the store for a real resource ID before falling back to the synthetic example placeholder.

• Store resolution uses the sqlite3 CLI with a hard 2 s cap; the storeResolved == nPlaceholders guard correctly emits \"store\" only when every placeholder came from the DB.
• fixtureSource propagation covers happy_path, json_fidelity, and the credential-skip JSON result.
• Test isolation is achieved by deriving a unique binary name from the test function name, preventing the shared-DB contamination flagged in prior review threads.

Confidence Score: 4/5

Safe to merge for CLIs whose resource type names are plain nouns; the store lookup path has a candidate-generation quirk for path segments ending in 'id' that would misclassify those results as synthetic rather than store-backed.

The three concerns raised in earlier review rounds are addressed: cross-test DB contamination is eliminated by per-test unique binary names, the pre-sync timeout is capped at 5 s, and the mixed store/companion fixture_source tag is handled correctly by the storeResolved counter. The remaining open item is that storeResourceCandidates applies the same id-suffix stripping to CLI path segments as to placeholder names, so a command path like [android, get] would search for resource_type 'andr' rather than 'android' and incorrectly produce synthetic results.

internal/pipeline/live_dogfood.go — storeResourceCandidates and the table-existence check in liveDogfoodStoreFixtureID

Important Files Changed

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[RunLiveDogfood] --> B[liveDogfoodDefaultDBPath]
    B --> C[runLiveDogfoodPreSync]
    C -->|sync found| D[run sync capped 5s]
    D --> E[Loop over commands]
    E --> F[resolveCommandPositionals]
    F --> G{companion found?}
    G -->|no| K[resolveStoreFixtureID]
    G -->|yes| M[run companion CLI]
    M -->|fail| K
    M -->|ok| N[companion-resolved]
    K -->|ok=true| O[storeResolved++]
    K -->|storeAvail no match| P[return synthetic]
    K -->|no store| Q[skip command]
    O --> R{storeResolved == nPlaceholders?}
    N --> R
    R -->|yes| T[source=store]
    R -->|no| U[source=empty]
    T --> V[set FixtureSource on results]
    U --> V
    P --> W[FixtureSource=synthetic]

_{Reviews (2): Last reviewed commit: "fix(cli): address live dogfood fixture r..." | Re-trigger Greptile}

@Mergifyio queue

Merge Queue Status

• ✅ Entered queue — 2026-06-07 23:12 UTC · Rule: default
• ✅ Checks passed · on draft merge queue: embarking main (d0a5d24), #2817 and #2826 together #2850
• ✅ Merged — 2026-06-07 23:36 UTC · at 10e9dced79af935fc3ce1ee1bb74d880212d1b63 · squash

This pull request spent 23 minutes 57 seconds in the queue, including 23 minutes 38 seconds running CI.