Repository navigation
W-112: owner change rule in AGENTS.md - #16
Merged
Merged
Conversation
commit: |
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.
Consequence analysis
This is a contributor-instruction change, not a product/runtime change. Intended behavior: every future code, test, test-oracle, configuration, installation, documentation and diagnostic change starts with explicit edge-case/consequence analysis, uses the smallest sufficient implementation, establishes a defect's cause, preserves behavior and evidence, and obtains review/CI at the exact proposed head before adoption. Existing roller architecture, published-package requirements, verifier gates, no-fake-completion rule, work-item branches, PR titles and squash-only merge authority remain unchanged. Handoffs gain consequence analysis as the opening part.
RCA: roller's root agreement does not yet record the owner's standing 2026-09-30 rule already present in the plugin repository. There is no runtime defect to repair. KISS: copy the applicable authoritative sections rather than inventing another process or abstraction. No added state, dependency, transport, process, permission, installation setting or test oracle.
Runtime empty/populated journals, boundary races, cancellation, partial/uncertain mutation outcomes, duplicate/stale events, shutdown, reconnect, active/idle messaging and recovery are inapplicable to this delta: no executable, manifest, lockfile, config or test changes. The copied rule explicitly requires their analysis whenever a future change affects them. Workflow edge cases are applicable: later pushed corrections invalidate earlier head-specific review; review-bot absence/pending findings cannot become PASS; all summary concerns, inline comments and nitpicks must be addressed or independently refuted; CI alone does not replace review; merges must be pinned to the reviewed head. Existing stronger architect acceptance and squash-only rules still apply.
User-visible roller behavior, installed versions and native evidence are unchanged. Cost is the documented analysis/review obligation for contributors; no functionality is removed. No host-install or profile changes. The architect independently reviewed this analysis before adoption; this PR remains a proposal pending implementation verification, exact-head bot review and CI.
Changes and provenance
AGENTS.md: copy the owner quote, mandatory analysis, KISS, RCA, preservation/proof and scope/coordination/truthful-completion sections verbatim from plugin AGENTS.md at 75f8d65246967f023598eda41f353025ab9945a4.Verification
Compared the copied block byte-for-byte with the pinned plugin source and reconstructed the previous agreement by removing that block and reversing only the handoff-opening edit. Both checks pass.
git diff --checkpasses;git diff --stat:Local
pnpm gate:Both hosted container legs passed 15/15 at head
d1ac85878266390a3dad7287da8a7e92d2db90ab: rc.2 has 280 uniform DSH records, alpha.2 has 291. CI run completed successfully; itshead_shawas independently checked againstgit rev-parse HEAD.The architect confirmed that no review bot is configured on this repository (also none on PR #14); that is a fact, not a review PASS, and no integration is being enabled. The copied bot-head rule applies when a bot exists. For this PR, exact-head evidence is both hosted DSH legs plus the architect's verifier. No merge or release is requested without the architect's acceptance and those head-specific checks.