Skip to content

feat: submit TEPP runs through a fail-closed HTTP port (v0.79.0) - #214

Open
seonghobae wants to merge 1 commit into
mainfrom
feat/tepp-http-fail-closed-outbox-v0720
Open

feat: submit TEPP runs through a fail-closed HTTP port (v0.79.0)#214
seonghobae wants to merge 1 commit into
mainfrom
feat/tepp-http-fail-closed-outbox-v0720

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 17, 2026

Copy link
Copy Markdown
Contributor

Buyer impact

After make seed, Period reports shows the last Valkey outbox row. When TEPP is unconfigured, Demo Analyst sees that TEPP is not available and no score was invented. post_admin can Request TEPP measurement; LineageWeave POSTs the published AnalysisRunRequest to TEPP_BASE_URL/v1/analysis-runs when that URL is set. Never invent a theta.

  • Fail-closed HTTP port + Valkey outbox (ADR 0022 / 0023).
  • Thetas still come only from TEPP or from calibrate_period_report on the IRT panel.

Exact head

193ba55941846c93a503211cc393971a3ab0d148 on feat/tepp-http-fail-closed-outbox-v0720.

Base: main @ 7464abb (merged #211 leftover pairs).

Review gate

Not in this slice

Leftover pairs landed as #211. naruon mailbox is #217. ThreadWeave conversations are #219. RankWeave rankings are #220. Keyverse identity is #221.

@cursor

cursor Bot commented Aug 17, 2026

Copy link
Copy Markdown

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@coderabbitai

coderabbitai Bot commented Aug 17, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@seonghobae, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 14 minutes

Limit details: You’ve used all 1 included review currently available under your plan.

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 0522d27e-a05b-4345-9bc9-156021e74818

📥 Commits

Reviewing files that changed from the base of the PR and between 8c020aa and 52c16ad.

⛔ Files ignored due to path filters (1)
  • uv.lock is excluded by !**/*.lock
📒 Files selected for processing (23)
  • .env.example
  • AGENTS.md
  • ARCHITECTURE.md
  • CHANGELOG.d/0.79.0-tepp-fail-closed.md
  • CHANGELOG.md
  • backend/app/config.py
  • backend/app/main.py
  • backend/app/tepp_outbox.py
  • backend/tests/test_config.py
  • docker-compose.yml
  • docs/adr/0022-tepp-http-fail-closed.md
  • docs/adr/0023-tepp-valkey-outbox.md
  • frontend/package.json
  • frontend/src/App.test.tsx
  • frontend/src/App.tsx
  • frontend/src/api.ts
  • lineageweave/__init__.py
  • lineageweave/fail_closed.py
  • lineageweave/tepp_client.py
  • pyproject.toml
  • scripts/seed_demo_data.py
  • tests/test_tepp_client.py
  • tests/test_tepp_fail_closed.py

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

G3 unique bits on feat/tepp-orchestrator-valkey-outbox-v0713 @ 9d4a1c42c7c684f116c1b3a66084eb2f92111a82 stay off this PR.

This PR already lands the buyer-visible TEPP slice: published HTTP /v1/analysis-runs, fail-closed envelope with no invented theta, Valkey outbox:tepp, and the Period reports Request TEPP measurement control.

What G3 still has that this head does not:

  • 3NF connector_outbox_event (Postgres) before the Valkey XADD;
  • orchestrator_envelope.py fail-closed parser for a missing/invalid orchestrator chat body.

Those are later increments if Valkey-only outbox or orchestrator envelope parsing become buyer-visible. No competing TEPP PR will be opened. Independent exact-head APPROVE is still required here. This comment is not an approval.

Copy link
Copy Markdown
Contributor Author

@cwl-noema-review

Independently review exact head 98d5266abe23668459fa0194a8224f3b838a51c0.

Buyer-visible TEPP slice: published HTTP /v1/analysis-runs, fail-closed envelope with no invented theta, Valkey outbox:tepp, Period reports Request TEPP measurement. Do not treat this comment as approval. Do not merge without an independent exact-head APPROVE plus green required checks on this SHA. Owner/self COMMENT is not APPROVE.

Copy link
Copy Markdown
Contributor Author

Write-first cycle 2026-08-17T09:05Z (KST 18:05).

Exact head 98d5266 still needs an independent exact-head APPROVE before squash. Owner/self COMMENT is not APPROVE. I will not self-approve and I will not merge this from the PR author.

Do not wait on Checks for the next buyer gap. ThreadWeave conversation port (v0.74.0 / ADR 0021 on main) is the next independent slice; leftover pairs stay on #211 and naruon mailbox stays on #217. Do not mix this into #74 or #92.

Copy link
Copy Markdown
Contributor Author

Exact head 193ba55 is waiting on independent current-head APPROVE. This COMMENT is not APPROVE and is not a merge. Do not self-approve. Do not invent a theta.

POST the published AnalysisRunRequest when TEPP_BASE_URL is set.
Otherwise the Period reports panel shows the Valkey outbox next
action and invents no theta.
@seonghobae
seonghobae force-pushed the feat/tepp-http-fail-closed-outbox-v0720 branch from 193ba55 to 52c16ad Compare August 17, 2026 10:14

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-17T19:13 KST: TEPP fail-closed HTTP port (#214) still needs an independent exact-head APPROVE. This COMMENT is owner/self and is not APPROVE. Do not squash-merge from the PR author. Do not mix into #74 or #92. Never invent a theta.

@seonghobae seonghobae changed the title feat: submit TEPP runs through a fail-closed HTTP port (v0.72.0) feat: submit TEPP runs through a fail-closed HTTP port (v0.79.0) Aug 17, 2026

Copy link
Copy Markdown
Contributor Author

Rebased onto main@8c020aa (RankWeave v0.75.0). Version bumped 0.72.0 → 0.79.0 so this does not downgrade main. Rankings port and TEPP outbox both stay. Independent exact-head APPROVE still required. Do not mix into #74 or #92.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-17T20:12 KST: exact head 52c16ad still waiting on independent current-head APPROVE. This COMMENT is owner/self and is not APPROVE. Do not self-approve. Do not invent a theta.

Copy link
Copy Markdown
Contributor Author

Cycle 2026-08-17T22:00 KST: TEPP fail-closed port still waiting on independent exact-head APPROVE. Do not self-approve. Copilot review requested. Do not mix into #74 or #92.

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