docs: define reusable repository README quality standard - #1694
Open
seonghobae wants to merge 2 commits into
Open
docs: define reusable repository README quality standard#1694seonghobae wants to merge 2 commits into
seonghobae wants to merge 2 commits into
Conversation
|
Warning Review limit reachedNext included review available in 29 seconds. View limit detailsLimit 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. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Team Run ID: 📒 Files selected for processing (3)
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 |
Adopt the current protected main lineage without rewriting history, link the shared README quality contract from the repository documentation map, and record the policy addition in the Unreleased changelog.
seonghobae
marked this pull request as draft
September 7, 2026 05:39
seonghobae
marked this pull request as ready for review
September 8, 2026 02:09
Contributor
There was a problem hiding this comment.
Pull request overview
OpenCode could not approve from deterministic current-head evidence because GitHub Checks have failed.
Findings
1. HIGH Current-head GitHub Checks - Fix failed required checks before approval
- Problem: Failed same-head checks remain for
bd56a9cc599e60be1b5d2a9729dea36b0e215e9f. - Root cause: The model-unavailable evidence fallback is allowed only when peer GitHub Checks are complete and clean.
- Fix: Read and fix the failed check logs below, then rerun the current-head checks.
- Regression test: Keep the model-unavailable fallback gated on an empty failed-check rollup.
Failed checks:
- Required Noema Review/noema-review: FAILURE (https://github.com/ContextualWisdomLab/.github/actions/runs/34179093452/job/101915615008)
- noema-review check run: failure (https://github.com/ContextualWisdomLab/.github/actions/runs/34179093452/job/101915615008)
Changed-File Evidence Map
flowchart LR
PR["PR changed files"] --> Evidence["OpenCode bounded evidence"]
Evidence --> S1["Repository file: CHANGELOG.md"]
S1 --> I1["repository behavior"]
I1 --> R1["Review risk: Repository file: CHANGELOG.md"]
R1 --> V1["required checks"]
Evidence --> S2["Repository file: README.md"]
S2 --> I2["repository behavior"]
I2 --> R2["Review risk: Repository file: README.md"]
R2 --> V2["required checks"]
Evidence --> S3["Docs: repository-readme-quality-standard.md"]
S3 --> I3["operator or user guidance"]
I3 --> R3["Review risk: Docs: repository-readme-quality-standard.md"]
R3 --> V3["docs review"]
Contributor
OpenCode Review Overview
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Outcome
Add one organization-level README quality standard while preserving the existing ownership rule that each product repository owns its own README prose.
The standard captures the recurring high-leverage pattern discovered across active README lanes:
Review repairs
The initial review found that the new standard was not discoverable from the repository documentation map and was missing from release history. The current head:
docs/repository-readme-quality-standard.mdfrom the root README's Related documents table; andCHANGELOG.md→ Unreleased.The informational copyleft-policy review is preserved: inherited obligations remain in force until sufficient relicensing rights or a safe replacement exists.
Ownership and licensing boundary
This is a review/checklist contract, not a copy-and-paste template or README generator. Repository-specific product language, security boundaries, installation reality, release state, support path, and licensing obligations remain authoritative in the owning repository.
The
.githubrepository already carries a ContextualWisdomLab MIT root license. This documentation change introduces no dependency, copied third-party source, asset, model, or new license grant, and does not change sibling-repository licensing.Current exact authority — 2026-09-07
main:c9052e607e5f3cc76e73207e7786b21500721b79;bd56a9cc599e60be1b5d2a9729dea36b0e215e9f;mainas merge base;README.md,CHANGELOG.md, anddocs/repository-readme-quality-standard.md;Exact-head CodeQL PR
34066085855, Security Scan34066085856, SAST Semgrep34066085876, Python Security34066085882, OpenCode coverage jobs, and current-head admission completed successfully.Required
noema-reviewjob101592715849failed only afterorchestrator/freereturned HTTP 502 following 780.1 seconds while servingdeepseek-ai/deepseek-v4-flash-0731; it produced no README-standard finding. The transient transport failure was rerun without source churn, and replacement job101609741798is queued on the unchanged exact head.Required
opencode-reviewjob101592718134failed closed because no authenticatedopencode-agentverdict existed when it checked. Dispatch succeeded and created central owner run34075986499, which completed failure: the free model pool was exhausted and produced no authenticated current-head verdict. Do not classify either reviewer/runtime state as a README source failure, and do not transfer predecessor evidence.Merge boundary
Merge only through ordinary protected governance after the unchanged exact head has terminal-success applicable checks, authenticated exact-head reviewer verdicts, all valid current-head findings and threads resolved, base ancestry remains current, and the then-live review/ruleset requirements are satisfied. No self-approval, routine administrator bypass, gate weakening, source-neutral retrigger churn, or predecessor-evidence transfer.
Current review-runtime blocker — 2026-09-08
A newer exact-head admission refresh supersedes the earlier queued/retry snapshot. Noema job
101915615008reachedorchestrator/freeon unchanged headbd56a9cc599e60be1b5d2a9729dea36b0e215e9fbut failed closed with HTTP 429 after 47.1 seconds while servingdeepseek-ai/deepseek-v4-flash-0731; its preflight evidence also records exhausted 429, 404, and timeout candidates. Current OpenCode job101918287128admitted the same head and failed closed because no authenticated verdict was published, while both coverage jobs succeeded. These are review-runtime failures, not README/source findings. Gateway-owned failover remains proposed on contextual-orchestrator#1049 and is not integrated or released. Do not perform further source-neutral retries or admit this README head as successful until the immutable owner fix is integrated and the unchanged head receives fresh authenticated verdicts.This PR is Ready for review admission. The owner transport repair and authenticated current-head reviewer verdicts remain merge gates, not reasons to suppress review by keeping this documentation PR Draft.