Skip to content

docs: refresh post-merge product gap baseline - #610

Merged
seonghobae merged 1 commit into
mainfrom
codex/gap-baseline-post-merge-20260825
Aug 25, 2026
Merged

docs: refresh post-merge product gap baseline#610
seonghobae merged 1 commit into
mainfrom
codex/gap-baseline-post-merge-20260825

Conversation

@seonghobae

@seonghobae seonghobae commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Outcome

Refreshes the non-identifying baseline after the protected merge wave. It records current protected-main SHA, all five open PR exact heads, confirmed merge SHAs for #608/#605/#600/#493/#468, the delivered OIDC repair, and the remaining buyer-facing isolation-copy follow-up.

Verification

  • uv run --extra dev pytest -q tests/test_documentation_hygiene.py — 5 passed
  • git diff --check

This document distinguishes protected-main delivery, non-default stack history, branch-local verification, and runtime acceptance. It contains no private records or credentials.


Open in Devin Review

@coderabbitai

coderabbitai Bot commented Aug 25, 2026

Copy link
Copy Markdown

Warning

Review limit reached

Next included review available in 50 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: 6d6b041a-9216-49af-aa6f-908ae306c598

📥 Commits

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

📒 Files selected for processing (1)
  • docs/product-technical-gap-baseline.md

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:05

@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 on lines +56 to 58
Protected main now calls the ADR 0109 OIDC return helpers before
`signinRedirect`; #600 and #605 delivered the repair through the protected
gate. The authenticated-only `accessToken` narrowing remains present.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🔍 Stale OIDC-defect prose contradicts refreshed delivery record

The refresh at product-technical-gap-baseline.md and the delivery table (product-technical-gap-baseline.md) now show #600/#605 merged and the OIDC repair delivered. The untouched sentence at product-technical-gap-baseline.md still says the defect remains until #605 passes the gate and main is not a release candidate. These lines sit outside the diff but now contradict the new state.

Open in Devin Review

Was this helpful? React with 👍 or 👎 to provide feedback.

@seonghobae
seonghobae merged commit 943ec39 into main Aug 25, 2026
21 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