Repository navigation
Conversation
Define assessment-only states, evidence limits and the remaining qualification work without introducing a reporter, installer or authority/recovery engine. Preserve the historical ADR and defer implementation until human design review. Refs #158
Require explicit selected command paths for diagnostic attribution, scope current digest comparisons without inheriting historical pass results, and clarify human decisions for open qualification debt. Keep every F1–F6 follow-up outside this checklist-only proposal. Refs #158
5 tasks
Exclude only the unapproved checklist proposal with a nested npm ignore rule and use its stable PR URL in the installed ADR. A dry-run file-list comparison confirms no other package selection changes. The reviewed checklist contract text is unchanged. Refs #158
Owner
Author
|
Superseded by implementation PR #160 after the owner explicitly selected a repository-only read-only static reporter on 2026-10-21. The new PR includes this reviewed design history and the bounded extension. Closing this design-only Draft to avoid parallel merge paths; no commits, branches or worktrees are deleted and no implementation is merged. #158 and #23 remain open; #155 remains deferred. |
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
Refs #158 and #23. Propose a checklist-only remote qualification contract before replacement implementation. Keep #155 deferred; do not port its journal, authority, receipt, lock, caller or recovery engine.
Scope decision
Checklist only for this stage. No demonstrated gap justifies a reporter now. No runtime code, CLI flags, machine schema, installer, authority/recovery logic, metadata/profile/version/pin changes, native/SSH/model experiments, release or operations changes.
This Draft is a design review artifact, not design freeze or implementation approval. #158 and #23 remain open and unassigned to a release. The deferred #155 branch and other worktrees are preserved. Human review must decide the minimum contract before any replacement implementation.
Independent work and review
12f98f2793b78f647107368c06e83286c5026e05without initially reading that draft. Codex integrated the results and checked source claims.claude-opus-5-5, using the existing user-approved account.b586639d010ffc388ea09d299b441d3be836808995437f1737b0f96201b8092c, proposal9587833. Exit0, success, one turn, input/source hashes unchanged. Two mandatory findings concerned selected-installation attribution and baseline comparison semantics; Codex checked and corrected them infe441a7.fe441a7d66d99ff1cef9fa3c92fe0c3cb8bb877f(input SHA-256312eb10ecae0319a76f6c0b42d8d7611bb91f3fc714f07467b3f4fa6071278de). Exit0/success, actual modelclaude-opus-5-5, one turn, input/source unchanged; no mandatory findings. It reviewed the changed contract with the complete proposal and primary source context, not a runtime implementation.Validation
node scripts/check-repository.mjs: pass.git diff --check: pass.No merge, tag, release, npm publication, global installation or operational action is requested by this design PR.
Advisory questions for human design review
The follow-up review recommends further precision around intended-version evidence versus relative wire banner equality; the distinction between CLI/wrapper selection and existing SSH/shell/interpreter PATH behavior; recording exact SSH/agent/home context and invalid-path outcomes; and the disposition of historical guide/recovery debt. These are documented design questions, not permission to implement a reporter or revive #155.
Two requests for primary evidence were checked directly by Codex: the ADR header still explicitly says Proposed, and VALIDATION.md's 2026-10-02 integrity/results show the published 0.2.1 artifact on the destination as well as the operator. Those historical records are preserved, not promoted to current qualification.
Packaging-only follow-up after contract review
npm pack --dry-run --json --ignore-scriptsshowed that the new review-only design file would otherwise enter the npm archive and break its fixed allowlist. A nesteddocs/design/.npmignorenow excludes only that file; before/after listing confirmed every other selected file is unchanged. The existing installed ADR points to this stable PR rather than a missing package-relative file. No release allowlist, runtime code or test expectation was changed.The checklist contract document is byte-identical to Claude-reviewed
fe441a7. This final delta changes only the packaging exclusion and cross-link; Codex checked it separately, without a third model review. Current PR contains two Markdown files and one narrowly scoped ignore file.Final current-head verification
Head
353686d78ea3961de3718b7fa4592c0e4a091f79passes all12checks, including macOS/Ubuntu Node22/24, Windows-native Node22/24, repository policy, release gate, CodeQL and GitGuardian. CLI run39524939366. The earlier contract-only CI run was cancelled as superseded; no failed-job rerun or weakened test expectation was used. Normal package checks passed with the proposal excluded from the archive.PR remains Draft, auto-merge is not configured, and no issue is closed. These CI/review results are not human design acceptance or implementation authorization.