Repository navigation
docs(spec-007): R1 — records honesty for phase C owner decisions - #724
Merged
Merged
Conversation
Master stated the owner's 2026-10-06 reverse (open-read → SystemHigh) as still reversible, omitted dated decisions-log rows, promised bare ::1, and claimed stacking makes CI a fast-forward. Align SPEC-007, inventory, CHANGELOG, process docs and the readiness-push gate with what master actually ships; code follow-ups stay in 4.10b/4.3b/4.5b. Co-authored-by: IanFrelinger <IanFrelinger@users.noreply.github.com>
Reject a missing push filter even when a later pull_request event has the expected branch list. Co-authored-by: Codex <noreply@openai.com>
IanFrelinger
marked this pull request as ready for review
October 8, 2026 20:49
This was referenced Oct 8, 2026
3 tasks done
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.
Summary
Makes master honest about the three 2026-10-06 owner decisions and related records drift after #720/#722/#721/#723. No behaviour change except tightening the readiness-push shell gate. Code follow-ups stay in 4.10b / 4.3b / 4.5b. This records PR does not settle the separate owner questions in the review.
Changes
SystemHighData; Q8 clarified/O1; O2); intro amended; 4.3/4.5/4.10 bullets carry merge SHAs; reverse "the owner can reverse it"; Known limits for O2 /ASPNETCORE_HTTP_PORTS/ multi-frame on AG;[::1]wordingdocs/EgressInventory.md+CHANGELOG.md: same decision wording; ci: speed multi-PR merges — no readiness double-fire on cursor/** #723 CHANGELOG entry; past-tense 4.4 "no production frame" sentence; stale Program.cs /EgressGuardcitations; TSV notes for SNS/telemetry linesCLAUDE.md,CONTRIBUTING.md,.github/workflows/README.md; drop post-hoc merge-commit exception; reconcile ~30–40 min timingtests/scripts/readiness-push-no-cursor-branches.test.sh: pinpush.branchesto exactly[master, main]and prevent reading another event's branch filterAshlarInboundListenerPolicydoc comment: bare::1refused;[::1]succeeds.cs/ 139 entries as of 2026-10-08Testing
ef5920bdfd7857ba7eeda71dcf94d08b407aa546: cert-gate, build-core, shell-lint, lychee, and Readiness summary.bash tests/scripts/readiness-push-no-cursor-branches.test.shpasses onef5920bd.on.push: a valid fixture with no push branch filter and a laterpull_request.branches: [master, main]falsely passed at1d376c04, and is rejected atef5920bd. Four fixture cases pass: baseline, added agent glob, missing push filter, and later PR filter.ef5920bd:repo-gates: all 28 gate(s) passed. The exploratory Windows suite is not acceptance evidence: Linux-specific fixture checks failed, and its branch-gate invocation overlapped the intentional workflow mutation. The branch gate passed separately after restoration; CI checked the clean committed tree.Testing strategy (blast radius)
readiness-push-…)claude/**mutation redmake kernel-coverage-gate(if touchingsrc/Ashlar.Core.*orsrc/Ashlar.Infrastructure) — N/A (XML doc comment only)make kernel-gate— N/Amake test-prod-style— N/AChecklist
Records touched (for Grok drift audit)
docs/specs/SPEC-007-security-labels-and-reference-monitor.md,docs/EgressInventory.md,CHANGELOG.md,docs/Configuration.md,docs/Federation.md,ci/cert-gate-assertions.md,ci/egress-inventory.tsv,CLAUDE.md,CONTRIBUTING.md,.github/workflows/README.md,AshlarInboundListenerPolicy.cs(doc), readiness-push testRelease