feat(architecture): delete machine authorization, parity, guardSemantics (PR 3/6) - #296
Merged
Conversation
hzcheng
force-pushed
the
agent-pivot/arch-refact
branch
6 times, most recently
from
August 19, 2026 07:30
e8ac758 to
e915050
Compare
hzcheng
force-pushed
the
agent-pivot/arch-refact
branch
from
August 19, 2026 12:26
e915050 to
09342db
Compare
hzcheng
force-pushed
the
agent-pivot/arch-refact
branch
from
August 19, 2026 13:34
09342db to
88553fb
Compare
Owner
Author
|
approve-architecture 88553fb |
Owner
Author
|
approve 88553fb |
added 2 commits
August 19, 2026 23:24
…ics (PR 3/6) Net -846 lines. The trusted kernel has taken over. - Delete scripts/run-guard-mutation-parity.js - Remove guardSemantics from the record schema and delta flags - checkArchitectureChange.js: no record matching; a relaxing or re-partition classification is authorized only by the owner's approve-architecture <full-head-sha> comment (consumption wired in the transition PR #297) - describeArchitectureChange.js: diff reporter only - run-trusted-kernel.js: materialize the head via git ls-tree + git show (no symmetric diff against unfetched objects); consume the owner's approve-architecture comment from the GitHub API (fail-closed) - trustedKernel.js: drop the deleted parity script from protected paths - Remove parity from tests, protected paths, and release packaging
Skill-Harvest: none
hzcheng
force-pushed
the
agent-pivot/arch-refact
branch
from
August 19, 2026 15:26
88553fb to
6af4210
Compare
Owner
Author
|
approve-architecture 6af4210 |
Owner
Author
|
approve 6af4210 |
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
PR 3/6 of the Harness Simplification program (docs/architecture/harness-simplification-decision.md). Deletes the old machine-authorization chain now that the trusted kernel has taken over:
run-guard-mutation-parity.js,guardSemantics, and Architecture Change record consumption. Net -797 lines.This PR is classified relaxing (it deletes guard machinery), so it requires owner architecture approval — see the Owner approvals section.
What changed
scripts/run-guard-mutation-parity.js; remove parity from tests, protected paths, and release packaging.guardSemanticsfrom the record schema and delta flags; Architecture Change records are historical ADRs only.checkArchitectureChange.js: no record matching; a relaxing/re-partition classification is authorized only by the owner'sapprove-architecture <full-head-sha>comment (consumption wired in transition PR feat(architecture): wire approve-architecture consumption (transition to PR 3/6) #297).describeArchitectureChange.js: diff reporter only.run-trusted-kernel.js: materialize the head viagit ls-tree+git show; consume the owner's approve-architecture comment via the GitHub API (fail-closed).trustedKernel.js: drop the deleted parity script from protected paths (kernel evaluator fixes landed separately in fix(architecture): make the trusted kernel evaluate real heads #298).Skill harvest
none — both lessons from this rescue (default-branch check ownership, owner-approvals convention) were already recorded in #297 and #299.
Owner approvals
Copy each line into a separate PR comment below (both bind the exact head SHA and expire when the head moves):
Change impact declaration