Skip to content

fix(lineage): render focused isolation reasons - #609

Merged
seonghobae merged 1 commit into
mainfrom
codex/isolation-reason-buyer-followup
Aug 25, 2026
Merged

fix(lineage): render focused isolation reasons#609
seonghobae merged 1 commit into
mainfrom
codex/isolation-reason-buyer-followup

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Outcome

Completes the buyer-facing half of ADR 0143 after #493 merged its backend-only exact head. The focused Event Lineage API type and post dialog now distinguish visible comparison candidates from an honestly unavailable comparison group, with localized next actions and backward-compatible generic copy.

The accepted ADR keeps the batched multi-post Ask graph reason unavailable because attributing one focused-post reason there would be unsupported.

Verification

  • backend lineage plus documentation hygiene: 31 passed
  • focused empty-state regressions: 2 passed (before restacking onto protected main)
  • i18n: 64 passed
  • frontend lint and production build passed on the protected-main follow-up

A later local Vitest worker startup attempt timed out under host contention before collecting tests; this is recorded separately from the successful focused run. No real records or credentials are included.


Open in Devin Review

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 53 minutes.

View limit details

Limit details: You’ve used the included review currently available.

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 7a6226fa-48f4-4550-9a9d-ad83cac2197c

📥 Commits

Reviewing files that changed from the base of the PR and between 3d6d718 and 820fe64.

📒 Files selected for processing (5)
  • docs/adr/0143-lineage-isolation-reason.md
  • frontend/src/App.test.tsx
  • frontend/src/App.tsx
  • frontend/src/api.ts
  • frontend/src/i18n.ts

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.

@seonghobae
seonghobae enabled auto-merge (squash) August 25, 2026 09:02

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Devin Review found 1 potential issue.

Open in Devin Review

Comment thread frontend/src/App.tsx
@seonghobae
seonghobae merged commit ed6b941 into main Aug 25, 2026
20 of 22 checks passed
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