Repository navigation
docs(governance): mandate universal governed project and local/cloud admission - #240
Conversation
|
Important
This repository does not receive automatic reviews because it has fewer than 10 stars.
|
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
🟡 Changes recommended
The acceptance case incorrectly requires paid-tier protection universally, blocking valid enrollment for GitHub Free public repositories with effective required checks and branch protection.
1 open issue
What changed in this PR
Expands the existing BDD rollout proposal with mandatory project governance enrollment and equivalent local/cloud admission requirements. This remains documentation only; enforcement requires a separately approved specification and implementation.
Changes:
- Defines governed project intake, quarantine, coverage reporting, and positive and negative acceptance checks.
- Requires equivalent local/cloud authorization, isolation, identity verification, and publication controls.
- Updates status wording to reflect merged PR #239 without claiming deployed enforcement.
| File | Description |
|---|---|
docs/operations/cucumber-bdd-rollout.md |
Adds universal admission and local/cloud parity requirements; clarifies proposal status and implementation gates. |
Review effort: Balanced
| The spec must assign stable EARS IDs, responsibilities, actual enforcement surfaces and tested rollback to **every** case, preserving G0–G6 and V1–V6: | ||
|
|
||
| 1. Unregistered new venture/repository attempts to enter code-writing before bootstrap; deny with observable no-write/no-publish evidence while allowing authorized governance-only initialization. | ||
| 2. Existing repo missing the caller, required verdict, up-to-date SHA, trusted approval, or available paid-tier protection; do not mark governed or enable code. |

Relates to #238. Follows the merged documentation-only PR #239 (main
0b137992a45873beaf6e68764649237d2bb31fd0). No second issue, tracker, or automated coordinator.Authorship: GPT-6, 2026-11-01, using the connected GitHub account. This is AI-authored documentation proposed for Ky's review; GitHub account appearance is not an owner-authenticated G-stage approval or Kiro specification.
Scope and actual changes
Update only the existing
docs/operations/cucumber-bdd-rollout.md(one file, doc-only).Explicit non-goals and gates
No spec was authored, no code/test/CI/roles/pinned caller or protected security rule edited, no Cucumber installation, no cloud/local runtime provisioned or billed, and no local vs cloud parity pass claimed. The Kiro-first spec-only PR with EARS criteria and owner merge remains required before Codex gets implementation authority. Preserve existing
AI_MONETIZATION_GATE, Human Skill Retention,ai-governance / verdict, role separation, distinct verifier and owner-only merge/deploy.Verification
Against exact main
0b137992a45873beaf6e68764649237d2bb31fd0, the branch is ahead by one commit; the diff is one existing documentation file only. No project test suite was executed for this documentation-only edit. CI status, human review and approval remain to be observed. DRAFT, NOT MERGE-AUTHORIZED BY AN AGENT.Universal mandatory enrollment addendum — 2026-11-01 (GPT-6, owner request)
The owner has now made mandatory by-default governance of every new venture, project, repository, and code change an explicit non-negotiable requirement. The same existing documentation file now carries the proposed governed project-creation/admission contract, scope inventory, trusted bootstrap/quarantine and no-code-before-GOVERNED_READY rule, local/cloud equivalence, role/identity enforcement, fail-closed owner-approved applicability, and adversarial tests for ungoverned creation, skipped checks and bypasses. Only the Cucumber runner is selectively optional; the governance process isn't. No new issue, PR, policy runtime or automation was created for this expansion.
Current observed blockers, not deployment claims
scripts/optin.shandscripts/sweep.share retroactive/opt-in workflows with excluded scopes. They cannot stop the creation of ungoverned repos by themselves.24443511viewed viaKyLamportLogic/buildlogic-governancerequires PRs on default branches but has no requiredai-governance / verdictstatus check, CODEOWNERS review or approving-review count. That repository's shared workflow caller is still pinned to old SHAefd0e11dde73a70d4867293e4ac2308d28be057e.KyLamportLogic/LamportLogicreturned a GitHub plan-related 403; inability to verify protection is not evidence of protection. Organization-creation limits cannot restrict organization owners from creating repos; personal-account repos lie outside organization policies. Unmanaged local source creation likewise cannot be prevented by GitHub rules.Required next gated action
Kiro must incorporate this broadened invariant into the existing #238 specification-only PR (EARS, design, tasks, fail-closed/positive controls, per-account/private-plan limitations), which must pass current policy and be Ky-reviewed/merged before protected enforcement/runtime changes. Do not edit a ruleset/role map/workflow or create a new bootstrapper by treating this documentation change as implementation authorization. No automatic cost or provider setup.