Skip to content

docs: make README links registry-safe - #45

Open
seonghobae wants to merge 4 commits into
fix/ci-protected-ref-concurrencyfrom
docs/registry-readme-links
Open

seonghobae wants to merge 4 commits into
fix/ci-protected-ref-concurrencyfrom
docs/registry-readme-links

Conversation

@seonghobae

@seonghobae seonghobae commented Sep 18, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Convert public README links to absolute GitHub URLs.
  • Keep package-facing documentation links usable from the registry page.

Verification

  • README package links audited.
  • python -m compileall -q src.

No packaging manifest or README assertion required changes.

Summary by CodeRabbit

  • 문서
    • README에 있는 문서 링크를 특정 커밋을 가리키는 GitHub 절대 URL로 변경했습니다.
    • 변경된 링크에는 공급망 문서, 문서 홈과 조사 자료, 자율 유지보수 문서 및 라이선스가 포함됩니다.
    • 링크를 통해 관련 문서와 라이선스 페이지에 접근할 수 있습니다.

Current exact authority — 2026-09-19

  • Exact head: f1d0b9999540797cd341a2aa77b75d6694c32cb6
  • Direct prerequisite/base: #44 at 3190776243e545013738fd6a0f30cd9f966d7632
  • Ordinary two-parent prerequisite merge: f1d0b9999540797cd341a2aa77b75d6694c32cb6; no force update or destructive rebase.
  • Compare against the exact prerequisite is ahead 3 / behind 0 with only README.md in the effective diff.
  • README blob: b4ea16d13ff771784cd7254ff5fd664d21ca5e98
  • Mutable package links: RED 5 → GREEN 0; all repository targets bind to protected main@0fda6e60c2c80ec7b2aa2d58dac6b944dec6a6d0.
  • Predecessor CI 35289558903 failed because protected main still used run_id concurrency while its regression required stable protected-ref identity. fix(ci): restore protected-ref concurrency cancellation #44 is the canonical RED→GREEN owner and is now ancestry-preserved in this child.
  • Exact-head CI 35448287478 and SAST 35448287499 are terminal GREEN.
  • Security 35448287531 is terminal RED: exact base/head checkout matched, but Dependency Review support returned HTTP 403 and failed closed; ContextualWisdomLab/.github#810 remains open.
  • CodeQL 35448287496 is terminal RED for both actions and python: dispatch succeeded but VERDICT_STATE=pending; terminal status publication remains open in ContextualWisdomLab/.github#1929.
  • State at the 2026-10-02 re-fetch: open Draft, mechanically mergeable, unresolved review threads 0, qualifying independent approvals 0.

#44 remains open Draft and must integrate first. This child stays Draft; after protected truth is adopted it must reacquire terminal required Checks and a qualifying independent current-head approval before ordinary merge. No predecessor evidence, self-approval, gate weakening, or publication claim transfers.

Latest prerequisite integration — 2026-10-02

Parent #44 advanced from 3190776243e545013738fd6a0f30cd9f966d7632 to 91161c64a620f0a3c660db94490a174ad780dd7f. Its intervening delta modifies only docs/product-technical-gap-baseline.md; this child modifies only README.md, so the changes have no path overlap.

  • exact current head: 832a9e3efbf15bb6f7ff06359f43c5fe3aeb8c7b;
  • exact tree: a2166550842d71c433c2f18a3f05ec6908b06ad5;
  • ordinary two-parent merge: prior child f1d0b9999540797cd341a2aa77b75d6694c32cb6 plus current parent 91161c64a620f0a3c660db94490a174ad780dd7f;
  • comparison against the current parent is ahead 4 / behind 0 and the effective file set remains only README.md;
  • README blob remains b4ea16d13ff771784cd7254ff5fd664d21ca5e98, so no valid registry-link delta was lost;
  • fresh exact-head CI 36933450471 is terminal success: lock integrity and package build passed; Python 3.10–3.14 each passed 432 tests with 1,895/1,895 statements and 660/660 branches covered, and the autonomous-guard sub-suite passed 215 tests with 100% measured coverage.

The child is Ready only to admit fresh exact-head review and applicable Checks while preserving the stack on #44. Ready is not approval or merge authorization. Keep merge and auto-merge disabled until #44 integrates first and this exact head then reacquires every required Check, a qualifying independent approval, resolved-thread state, unchanged ancestry/mergeability, and then-live protected governance; do not reuse predecessor-head success or bypass failed central gates.

@coderabbitai

coderabbitai Bot commented Sep 18, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Warning

Review limit reached

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

Next included review available in 57 minutes.

Check out review usage here.

View limit details

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

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: 2ac0577e-e8d9-4cbe-ba72-65ff253c96d7

📥 Commits

Reviewing files that changed from the base of the PR and between f1d0b99 and 832a9e3.

📒 Files selected for processing (1)
  • README.md

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Advanced

Run ID: c9c7d3f9-2668-4d4b-b663-02e190249c80

📥 Commits

Reviewing files that changed from the base of the PR and between a18aa01 and f1d0b99.

📒 Files selected for processing (1)
  • README.md
🚧 Files skipped from review as they are similar to previous changes (1)
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.


📝 Walkthrough

Walkthrough

README.md의 공급망, 문서 홈, 조사 자료, 자율 유지보수 및 라이선스 링크를 상대 경로에서 특정 커밋을 가리키는 GitHub 절대 URL로 변경했습니다.

Changes

문서 링크 업데이트

Layer / File(s) Summary
README 링크 대상 변경
README.md
공급망 안내, 문서 홈, 조사 자료, 자율 유지보수 문서 및 라이선스 링크를 GitHub 절대 URL로 변경했습니다. 관련 설명 문구와 문서 경로 표기는 유지했습니다.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to f1d0b

The absolute documentation links point to existing repository targets, with no actionable merge-blocking defect identified. The reported Security and CodeQL status failures still need resolution through the normal merge workflow.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed 제목은 README 링크를 레지스트리에서 사용할 수 있도록 변경한 주요 내용을 정확하고 간결하게 설명합니다.
✨ Finishing Touches 💡 1
🛠️ Fix failing CI checks 💡
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

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 added documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work labels Sep 19, 2026 — with ChatGPT Codex Connector
@seonghobae
seonghobae changed the base branch from main to fix/ci-protected-ref-concurrency September 19, 2026 14:17

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

Please review exact head f1d0b9999540797cd341a2aa77b75d6694c32cb6. The child now ancestry-preserves canonical concurrency owner #44 and its effective diff remains README-only with immutable package links.

@coderabbitai

coderabbitai Bot commented Sep 19, 2026

Copy link
Copy Markdown

Rate Limit Exceeded

@seonghobae have exceeded the limit for the number of chat messages per hour. Please wait 1 minutes and 26 seconds before sending another message.

Copy link
Copy Markdown
Contributor Author

Exact-head admission correction — f1d0b9999540797cd341a2aa77b75d6694c32cb6

Ready is review admission only. Fresh audit against base 3190776243e545013738fd6a0f30cd9f966d7632 found:

  • latest terminal workflow blockers: Security Scan 35448287531=failure; CodeQL PR 35448287496=failure

This PR is moved to Draft/Proposed until the causal owner repair is present on a successor exact head and re-audited. Queued/pending work is neither an additional blocker nor passing evidence. No Close, force push, destructive rebase, manual rerun, synthetic status/approval, merge, auto-merge, or bypass was performed.

@seonghobae
seonghobae marked this pull request as draft September 30, 2026 05:03

Copy link
Copy Markdown
Contributor Author

@coderabbitai review

Please review unchanged exact head f1d0b9999540797cd341a2aa77b75d6694c32cb6. The PR body formatting and current-state timestamp were repaired without changing the branch; the effective diff remains README-only on prerequisite #44.

Fresh admission readback: open Draft, mechanically mergeable, Security 35448287531 RED on Dependency Review HTTP 403, CodeQL 35448287496 RED on VERDICT_STATE=pending, current-head approvals 0, unresolved threads 0. No rerun, merge, close, or bypass was performed.

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

seonghobae commented Oct 1, 2026 •

Copy link
Copy Markdown
Contributor Author

Integrated the latest #44 prerequisite without force-updating history.

  • New exact head: 832a9e3efbf15bb6f7ff06359f43c5fe3aeb8c7b
  • Parents: prior child f1d0b9999540797cd341a2aa77b75d6694c32cb6 and current fix(ci): restore protected-ref concurrency cancellation #44 head 91161c64a620f0a3c660db94490a174ad780dd7f
  • Effective diff against the current parent: README.md only; ahead 4 / behind 0
  • README blob preserved: b4ea16d13ff771784cd7254ff5fd664d21ca5e98
  • Fresh CI 36933450471 is terminal SUCCESS.
  • Lock integrity and package build passed; Python 3.10–3.14 each passed 432 tests at 100% statement/branch coverage, and the autonomous-guard sub-suite passed 215 tests at 100% measured coverage.

PR remains Draft because required exact-head Security/CodeQL and qualifying independent review evidence are absent. No merge, auto-merge, bypass, or Close was requested.

@seonghobae
seonghobae marked this pull request as ready for review October 1, 2026 23:06

Copy link
Copy Markdown
Contributor Author

Restored Ready for stacked review on unchanged exact head 832a9e3efbf15bb6f7ff06359f43c5fe3aeb8c7b.

  • Direct parent fix(ci): restore protected-ref concurrency cancellation #44 is now Ready at 91161c64a620f0a3c660db94490a174ad780dd7f; this child remains based exactly on that head.
  • CI 36933450471 is terminal SUCCESS across lock integrity, package build, and Python 3.10–3.14.
  • Effective diff against the parent remains README.md only, ahead 4 / behind 0.
  • Unresolved threads: 0; qualifying current-head approvals: 0.
  • Fresh exact-head Security/CodeQL evidence is absent and is not inferred from predecessor runs.

Ready admits review and applicable Checks only. #44 must integrate first; merge and auto-merge remain disabled. @coderabbitai review exact head 832a9e3efbf15bb6f7ff06359f43c5fe3aeb8c7b. @opencode-agent please issue a fresh exact-head review only after applicable hosted checks are terminal.

@coderabbitai

coderabbitai Bot commented Oct 1, 2026 •

Copy link
Copy Markdown

@seonghobae I’ll request review of exact head 832a9e3efbf15bb6f7ff06359f43c5fe3aeb8c7b.

Thanks for clarifying the admission rules. Ready permits review and applicable checks only. Predecessor Security/CodeQL results do not establish passing evidence for this head. #44 must integrate first. This request does not authorize merge, auto-merge, or bypass.


✏️ Learnings added
Learnt from: seonghobae
URL: https://github.com/ContextualWisdomLab/ThreadWeave/pull/45

Timestamp: 2026-10-01T23:07:29.883Z
Learning: For ContextualWisdomLab/ThreadWeave stacked PR #45, Ready is admission for review and applicable checks only, not merge authorization. Prerequisite PR #44 must integrate first. Security/CodeQL evidence must apply to the exact current head; predecessor results must not be treated as current-head passing evidence.

Note: Learnings are effective only in the context of similar code segments. To apply general coding guidelines, please configure review instructions. You can manage existing learnings in the app.

⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation priority: medium Normal-priority or P2 work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant