Skip to content

feat(api): inspect stored analysis-run requests via loopback CLI - #395

Draft
seonghobae wants to merge 1 commit into
feat/analysis-run-stored-request-consumer-parity-gap-003afrom
feat/analysis-run-stored-request-cli-gap-003a
Draft

feat(api): inspect stored analysis-run requests via loopback CLI#395
seonghobae wants to merge 1 commit into
feat/analysis-run-stored-request-consumer-parity-gap-003afrom
feat/analysis-run-stored-request-cli-gap-003a

Conversation

@seonghobae

Copy link
Copy Markdown
Contributor

Why

GAP-003A / issue #166 still required operators to write raw HTTP/1.1 after stored-request GET. Operators cannot inspect snapshot, cutoff, model contract, and output profile of a listed run from a client without guessing framing. Stdout stays metric-free. tepp.scientific_acceptance.v1 never appears.

Live slices that this PR does not duplicate:

What

GAP-003A stored-request CLI stacked on #387 over stored-request GET #377 / protected main 1bc02f580cf48e1d39da239f0e818453437c31c3:

  • tepp-analysis-runs stored-request client of GET /v1/analysis-runs/{run_id}/request
  • metric-free snapshot/cutoff/model/profile stdout
  • tepp.scientific_acceptance.v1 never prints
  • another consumer cannot inspect the first consumer's stored request
  • empty stdin required; nonempty body fails closed
  • loopback-only; unpublished consumers, credential flags, collection pagination flags, and unknown verbs fail closed
  • ADR 0041 on the stored-request lineage

Not an ADR 0014 scientific claim. Not persistence.

Verification

  • cargo clippy -p tepp_api --all-targets -- -D warnings
  • cargo test -p tepp_api --lib analysis_run_stored_request_cli
  • cargo test -p tepp_api --test analysis_run_stored_request_cli_contract
  • RUSTDOCFLAGS='-D warnings' cargo doc -p tepp_api --no-deps
  • python3 scripts/validate_documentation.py
  • python3 scripts/check_docstrings.py

Exact head: a99cdc77d26488f01c2deb7501dc691bc820ad02

Do not merge without two independent APPROVE reviews on this exact head and current Checks. Author COMMENTED is not independent APPROVE.

GAP-003A / issue #166: operators can inspect metric-free snapshot, cutoff,
model contract, and output profile through `tepp-analysis-runs stored-request`
without writing raw HTTP. tepp.scientific_acceptance.v1 never appears.
Stacked on stored-request consumer-parity (#387) over stored-request GET
(#377). Does not duplicate retry CLI, cancel/create/status CLIs, or GET-by-id.
@coderabbitai

coderabbitai Bot commented Aug 31, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Copy link
Copy Markdown
Contributor Author

Hour-24 exact-head: a99cdc77d26488f01c2deb7501dc691bc820ad02. GAP-003A stored-request CLI tepp-analysis-runs stored-request stacked on #387 over stored-request GET #377. Metric-free snapshot/cutoff/model/profile; tepp.scientific_acceptance.v1 never appears. Devin/CodeRabbit/author COMMENTED is not independent APPROVE. Ruleset 18156473 needs two independent APPROVE reviews. No self-approve.

@seonghobae seonghobae left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Exact-head COMMENT on a99cdc7 (draft). Unique occupied: tepp-analysis-runs stored-request for GET /v1/analysis-runs/{run_id}/request (ADR 0041) stacked on stored-request HTTP #377. Metric-free snapshot/cutoff/model-contract/output-profile inspect. tepp.scientific_acceptance.v1 never appears. Do not duplicate stored-request HTTP/CLI. Do not un-quarantine unscoped stored-request-by-client-key (ADR 0099 lives on #466). Do not un-draft. Zero reviews on this SHA before this COMMENT. Never self-approve. Persistence remains GAP-003B. Do not weaken fail-closed. No Buyer language.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant