Skip to content

docs(policy): clarify non-Rust allowlist rollout status (#201, rollout PR 1/12)#213

Merged
EffortlessSteven merged 1 commit into
mainfrom
docs/policy-rollout-status-20260511
May 11, 2026
Merged

docs(policy): clarify non-Rust allowlist rollout status (#201, rollout PR 1/12)#213
EffortlessSteven merged 1 commit into
mainfrom
docs/policy-rollout-status-20260511

Conversation

@EffortlessSteven

Copy link
Copy Markdown
Member

Summary

First PR in the 12-PR file-policy rollout decomposition. Docs-only — no policy files yet, no code changes, no enforcement.

Rewrites the rollout-status sections in docs/FILE_POLICY.md and docs/POLICY_ALLOWLISTS.md to reflect the 12-PR ladder rather than the old single-PR view. Adds explicit "receipts, not burn-down" language so future receipts can legitimately carry reason = "scheduled to be converted to Rust/xtask" paired with an expires date.

Adds a new tracker docs/policy/NON_RUST_ROLLOUT.md with:

  • Operating rules (receipts not burn-down; advisory before blocking; no blocking-strict in this rollout).
  • The full ladder table with issue numbers and dependencies.
  • Receipt schema with durable and transitional examples.
  • Definition of done for the ladder.

Issue

Closes #201. Refines #180 (umbrella). Tracks #109.

Decisions

  • Decision: Keep docs/policy/ as a new subdirectory rather than putting the tracker in docs/ci/ or at docs/ root.
    • Rationale: Future policy docs (per-policy operating guides, audit logs) will accumulate here.
    • Alternatives rejected: Single flat docs/NON_RUST_ROLLOUT.md — works for one file but doesn't scale.
  • Decision: Surface "scheduled to be converted to Rust/xtask" as a named valid reason in both FILE_POLICY.md and the tracker.
    • Rationale: Without explicit acceptance, reviewers will treat such entries as inconsistent with the policy. Naming the pattern keeps it visible and bounded (paired with expires).
    • Alternatives rejected: Implicit acceptance — invites churn arguments in every review.

Validation

Follow-ups

PR 2 (issue #202) — add the policy TOML ledger files with first-pass receipts. No checker, no behavior change.

Rewrites the rollout-status sections in FILE_POLICY.md and
POLICY_ALLOWLISTS.md to reflect that the 0.4.0 file-policy work is
decomposed into a 12-PR ladder of small, separately reviewable
changes rather than one large policy-infrastructure PR.

Adds explicit "receipts, not burn-down" language:

- The allowlists mean "known surface, owner, reason, and current
  disposition." They do not mean "approved forever."
- A valid `reason` may include "scheduled to be converted to
  Rust/xtask" when the file exists for legacy compatibility or
  migration staging, paired with an `expires` date.

Adds a new tracker doc, docs/policy/NON_RUST_ROLLOUT.md, with:

- The operating rules (receipts not burn-down; advisory before
  blocking; no blocking-strict in this rollout).
- The full 12-PR ladder with issue numbers, status, and
  dependencies.
- The receipt schema with durable and transitional examples.
- The definition of done for the ladder.

Closes #201.
@gemini-code-assist

Copy link
Copy Markdown

Warning

You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again!

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

@factory-droid

factory-droid Bot commented May 11, 2026

Copy link
Copy Markdown

Droid finished @EffortlessSteven's task —— View job


Validation Complete ✓

PR #213 — docs(policy): clarify non-Rust allowlist rollout status

2 comments posted, 0 rejected, 0 security items.

# Path P Verdict
1 docs/POLICY_ALLOWLISTS.md:151 P2 approved — status for #201 should be "in flight", not "planned"
2 docs/policy/NON_RUST_ROLLOUT.md:25 P2 approved — URL casing mismatch (lowercase vs EffortlessMetrics)

Both are minor consistency issues. No correctness bugs or security issues found — this is a docs-only structural change.

@coderabbitai

coderabbitai Bot commented May 11, 2026

Copy link
Copy Markdown

Warning

Rate limit exceeded

@EffortlessSteven has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 15 minutes and 1 second before requesting another review.

You’ve run out of usage credits. Purchase more in the billing tab.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

ℹ️ Review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro

Run ID: 8450a228-e4c5-458b-9ded-d922f85249f1

📥 Commits

Reviewing files that changed from the base of the PR and between d967dcf and dfc721d.

📒 Files selected for processing (3)
  • docs/FILE_POLICY.md
  • docs/POLICY_ALLOWLISTS.md
  • docs/policy/NON_RUST_ROLLOUT.md
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch docs/policy-rollout-status-20260511

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 and usage tips.

@EffortlessSteven EffortlessSteven merged commit fa0ebb4 into main May 11, 2026
20 checks passed
@EffortlessSteven EffortlessSteven deleted the docs/policy-rollout-status-20260511 branch May 11, 2026 20:08
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.

docs(policy): clarify non-Rust allowlist rollout status (rollout PR 1/12)

1 participant