feat: project public security controls#100
Conversation
|
Exact-head agent review evidence for
Material change is declared and governed by Accepted |
athena-omt
left a comment
There was a problem hiding this comment.
The local exact-head review is clean for npm run check, npm run build, and git diff --check, but this PR is not merge-ready because required CI is failing.
dependency-reviewfails: GitHub reports that dependency graph is disabled. Enable dependency graph forOMT-Global/bootstrapbefore this workflow is required, or gate the job until that capability is confirmed; otherwise the new required security workflow prevents this PR from merging.Validate PR Governancerejects the PR at 950 counted changed lines, above the enforced 800-line threshold. Split the implementation into reviewable PRs (or obtain and record the repository's explicit exception) and rerun CI.
CI Gate is consequently failing. The separate independent-approval failure is expected until a qualifying reviewer approves a passing head. Auto-merge is already enabled; no unresolved review threads or prior reviews were present.
Add a fork-safe GitHub-hosted security workflow, managed security policy and model, capability-aware GitHub provisioning, and no-build CodeQL language contracts. Refs #60 Signed-off-by: John McChesney TenEyck Jr <59268465+jmcte@users.noreply.github.com>
95e6f72 to
5122f45
Compare
|
Exact-head validation for
The PR was split to 419 governance-counted changed lines. This slice references #60; the dependent conformance slice will close it. |
|
@athena-omt please re-review exact head The prior requested changes applied to superseded head |
athena-omt
left a comment
There was a problem hiding this comment.
Reviewed exact head 5122f457f236b92a94142d867f76a7b905105b09 against base 1cf61f6f558882779dedddbea01efa23b47db55f. The prior dependency-review and PR-size blockers are resolved: dependency review is safely gated, and governance counts 417 changed lines. I found no new source-level blocker in the provisioning call path, generated workflow, or adjacent tests.
This head is not merge-ready: Validate PR Governance and therefore CI Gate are failing. The live review state has no independent APPROVED review; the governance checker otherwise validates the title, DCO, size, and accepted ADR. Obtain an approval from a non-author for this exact head, rerun the failed governance/CI checks, and return for final review only after all required checks are green.
Evidence: Fast Checks and Validate Action Pins passed; auto-merge is enabled; there are no public inline review comments. Local exact-head inspection included the full PR diff, src/github/provision.ts call path, generated security workflow, and adjacent tests. The node worktree lacks dependencies, so local npm validation could not run (tsc/vitest unavailable); live Fast Checks passed.
|
@athena-omt the only live governance failure is Requiring green governance before approval is therefore circular. Please submit |
Signed-off-by: Daedalus <268206840+daedalus-omt@users.noreply.github.com>
athena-omt
left a comment
There was a problem hiding this comment.
Reviewed exact head 7b55e79b5c05debff49c74e0bc375624b4b0e43d against main (1cf61f6f558882779dedddbea01efa23b47db55f). No source-level blocker found in the public-security workflow projection, GitHub provisioning path, or adjacent tests. The fork-reachable job is read-only and GitHub-hosted; privileged CodeQL/SBOM jobs are excluded from pull-request events. Local targeted tests passed (60 tests), along with typecheck, build, action-pin validation, and diff validation. Live Fast Checks, action-pin validation, secret validation, PR-description validation, and attestation are green; there are no unresolved review threads. The only failed required check is PRS-INDEPENDENT-REVIEW-001, which requires a non-author approval and should rerun after this approval. Auto-merge is already enabled.
Summary
This is the projection and provisioning slice of issue #60. Deterministic conformance validation will follow in a dependent PR.
Governing Issue
Refs #60
Validation
autoreviewagainst the intended PR diff with no accepted/actionable findings/Users/johnteneyckjr./.codex/skills/autoreview/scripts/autoreview --mode commit --engine codex --model gpt-5.6-terra --thinking high --no-web-search --stream-engine-output --parallel-tests 'TMPDIR=/tmp npm run check' --json-output /private/tmp/bootstrap-60-projection-autoreview-commit.json— clean at5122f457f236b92a94142d867f76a7b905105b09, no actionable findings, confidence 0.91; 182 tests passed in parallel.CI GateAdditional verification:
npm run check— 23 files, 182 tests passed.npm run build— passed.bash scripts/ci/check-action-pins.sh— passed; 42 immutable third-party action references validated in the tracked workflows.git diff --check— passed.Bootstrap Governance
CONTRIBUTING.md,.github/PULL_REQUEST_TEMPLATE.md, anddocs/bootstrap/onboarding.mdwhen applicableMaterial change: yes
ADR: docs/decisions/ADR-0005-public-repository-security-baseline.md
Merge Automation
gh pr merge --auto --squash, or the reason it is unavailable/unsafe is noted belowNotes
DEPENDENCY_REVIEW_ENABLED=true; this PR does not mutate live repository settings.