docs: make README links registry-safe - #1117
seonghobae wants to merge 67 commits into
Conversation
|
Important Draft PR not reviewedDraft PRs are not automatically reviewed by default.
To automatically review draft PRs, update your CodeRabbit configuration: reviews:
auto_review:
drafts: trueNo actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
Included review availability: Your plan provides up to 1 included review per hour; 0 remain after this review. 📝 WalkthroughWalkthroughREADME의 내부 문서 링크를 canonical GitHub 절대 URL로 변경했습니다. OIDC 라운드트립 검증 안내는 관련 스크립트 설명에서 ChangesREADME 링크 정규화
Priority: ⬇️ Low Estimated code review effort: 1 (Trivial) | ~3 minutes Change: Other Merge Risk: ⚪ Minimal · up to The README-only update has no identified merge-blocking risk. 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
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. Comment |
|
Fresh maintainer finding: the original README-only registry-link delta was mechanically mergeable on protected Repair applied without force/rebase or source duplication: ordinary two-parent convergence Predecessor |
|
Fresh operator-contract review found that the |
seonghobae
left a comment
There was a problem hiding this comment.
Current-head stack review at 1f0cbceb3445aeea76271ed7640bf6c25c102d1d: #1120 advanced test-only to 0808b081e7ac500c0181ed395743bd46b4c8b5b0. This README child adopts that exact parent via ordinary two-parent, non-force convergence. Fresh compare has merge-base 0808b081..., behind_by=0, and still only README.md plus tests/test_readme_auth_contract.py as child-owned delta. No valid README/test evidence was lost; previous receipts remain predecessor-only.
seonghobae
left a comment
There was a problem hiding this comment.
Current-head stack review at 87b2f9fafb524beb0c6b99d30af6a4ad59c4e6d9: parent #1120 moved to 82723adb2d1fded5f3ba741ac957d3dd5bdd8891 for the JWKS-selector contract repair. This README child adopted that exact parent through an ordinary two-parent non-force merge. Fresh compare has merge-base 82723adb..., behind_by=0, and still only README.md plus tests/test_readme_auth_contract.py as child-owned delta. Exact-head Tests 35665060477 is Draft-policy skipped; predecessor receipts do not transfer.
Current stacked authority — 2026-09-22 KST
This README follows the executable auth semantics it documents.
c10b6545520afb342e68d01ea4bcfce75a6e5bab→ fix(dev): make OIDC smoke target install its declared JWT dependency #11180dd0fd9a74165a63a081d1435123a79be6feefca→ security(auth): migrate public frontend off direct grants #112073ab7ea24517bfcb1edfc83676f9fb56ea524bc0→ this README child72341358f7edf01360a5da69b01fe87684b8076b73ab7ea...,behind_by=0; effective child delta remains onlyREADME.mdplustests/test_readme_auth_contract.pyThe buyer/operator documentation repair remains unchanged: RED
94f0f510c4495263faec4764bd598fd1f8bd65d4pinned the current auth contract, and causal fix2769b92172dd089ae9ffec207c9130abe057a728documents Authorization Code + S256 PKCE for humans and separate confidential Client Credentials actors for machine integration while preserving registry-safe links.The latest parent movement is ancestry-only: #1120 adopted #1118/#899 source-neutral live ancestry. Ordinary two-parent/non-force convergence
72341358...preserves this README lane exactly and inherits the current parent; no parent test/source is copied into the child-only delta.Exact
72341358...must reacquire applicable hosted evidence. Keep Draft until the prerequisite stack is promotable and qualifying independent review exists.No force push, destructive rebase, self-approval, gate weakening, stale-receipt transfer, owner-source duplication, new no-op wake commit, protected-main merge, or release is used.