Skip to content

feat(api): enumerate analysis runs via loopback collection CLI - #371

Draft
seonghobae wants to merge 1 commit into
feat/analysis-run-collection-get-gap-003afrom
feat/analysis-run-collection-cli-gap-003a
Draft

feat(api): enumerate analysis runs via loopback collection CLI#371
seonghobae wants to merge 1 commit into
feat/analysis-run-collection-get-gap-003afrom
feat/analysis-run-collection-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 collection GET. Operators cannot enumerate accepted, running, cancelled, or terminal runs from a client without guessing framing. Collection CLI stdout stays metric-free. tepp.scientific_acceptance.v1 never appears.

Live slices that this PR does not duplicate:

What

Eighth GAP-003A slice stacked on #368 over cancel HTTP #361 / protected main 1bc02f580cf48e1d39da239f0e818453437c31c3:

  • tepp-analysis-runs list client of GET /v1/analysis-runs
  • metric-free collection rows: run_id, run_state, idempotency_key
  • accepted and cancelled rows list; succeeded rows still omit the artifact
  • tepp.scientific_acceptance.v1 never prints
  • bounded cursor pagination via --page-cursor / --page-limit
  • loopback-only; unpublished consumers, credential flags, GET-by-id flags, nonempty stdin, and unknown verbs fail closed
  • ADR 0032 (0026–0031 live on other GAP-003A PRs)

Not an ADR 0014 scientific claim. Not persistence. Not GET-by-id. Not lifecycle POST. Not cancel HTTP. Not scientific-acceptance CLI.

Verification

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

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 list accepted, running, cancelled, and
terminal runs through `tepp-analysis-runs list` without writing raw HTTP or
guessing run identities. Stdout stays metric-free; tepp.scientific_acceptance.v1
never appears. Stacked on collection GET (#368). Does not duplicate GET-by-id,
lifecycle POST, cancel HTTP, or the scientific-acceptance CLI.
@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-20 exact-head review request.

Current head f7d5e5a391eb2caa55bd5d337e90e719137b5041. Devin/CodeRabbit/author COMMENTED is not independent APPROVE. OpenCode DISMISSED/REQUEST_CHANGES on predecessor heads does not transfer. Ruleset 18156473 needs two independent APPROVE reviews on this exact SHA.

GAP-003A collection CLI draft stacked on #368. ADR 0032. tepp-analysis-runs list only. Metric-free stdout. Do not duplicate #362 scientific-acceptance CLI or #368 collection GET.

Do not self-approve. Do not --admin merge. Checks/reviews are not a reason to weaken fail-closed gates.

Copy link
Copy Markdown
Contributor Author

Hour-21 exact-head review request.

Current head f7d5e5a391eb2caa55bd5d337e90e719137b5041.
Devin/CodeRabbit/author COMMENTED is not independent APPROVE. Copilot review requests are not APPROVE. Ruleset 18156473 needs two independent approvals. Do not self-approve. Do not merge without two independent APPROVE reviews on this exact head.

@opencode-agent review

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