P2E: reconcile parity docs with real verifiers + dogfood wiki (wj-1zm)#9
Merged
Conversation
PARITY.md's "Verified by" column cited aspirational phase numbers
("golden run (Phase 2)", "Phase-3 scenario") left over from the
pre-vendoring plan. Point each row at what actually verifies it now:
tests/parity-crossvalidate.test.ts, the verbatim vendored
update-noop.test.ts, the tests/build-disciplines.test.ts drift-lock,
and tests/check-format.test.ts.
parity-watch.yml watched only three of the five spec-bearing files
wijzer vendors (missing src/constants.ts and src/agent/types.ts) and
its remediation text described a hand-port workflow that no longer
matches the vendor -> regenerate -> npm test flow. Widen SPEC_FILES
and rewrite the remediation steps to match.
README.md now states Node is a dev/CI-only dependency (running tests
and regenerating the derived reference docs) — the plugin itself
needs no node at user runtime.
tests/plugin-structure.test.ts drops two assertions that just
restated OpenWiki's own numbers back at wijzer's docs (parity-first,
wj-1s6); that coverage already lives in build-disciplines.test.ts
(derivation-time) and check-format.test.ts (format-gate enforcement).
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Runs the prompt-driven /wijzer:init skill against wijzer's own repo, producing openwiki/ (quickstart, architecture/overview, parity/overview, skills/overview, .last-update.json) and passing scripts/check-format.sh. This is the golden-run leg of P2E's validation — the CLAUDE.md commit in this branch adds the "## OpenWiki" pointer this wiki's quickstart is linked from. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
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
Final bead of the OPT2+ parity arc — reconciles wijzer's documentation and CI
with the reality of the vendored-real-code approach, and validates the
rewritten
/wijzer:initby dogfooding it on wijzer itself.PARITY.md's "Verified by" column pointed ataspirational plan artifacts ("golden run (Phase 2)", "Phase-3 scenario").
Every row now cites the actual verifier —
tests/parity-crossvalidate.test.ts,the verbatim vendored
update-noop.test.ts, thetests/build-disciplines.test.tsdrift-lock,tests/check-format.test.ts..github/workflows/parity-watch.ymlnow watches the full vendored specset (adds
src/constants.ts+src/agent/types.ts), drops the staletests/noop.test.tsmapping, and its remediation text now drives the realre-vendor → regenerate →
npm testflow instead of a hand-port workflowthat no longer applies.
README.mdstates Node is a dev/CI-onlydependency — the plugin needs no
nodeat user runtime.tests/plugin-structure.test.tsdrops two parity-restatementassertions (parity-first, wj-1s6) — that coverage now lives where the
values are actually derived/enforced:
tests/build-disciplines.test.ts(derivation-time) and
tests/check-format.test.ts(format-gateenforcement).
openwiki/(quickstart, architecture/overview,parity/overview, skills/overview,
.last-update.json) generated by runningthe rewritten
/wijzer:initagainst wijzer's own repo, plus a## OpenWikipointer appended to
CLAUDE.mdlinking to it.Validation
itself (this PR's
openwiki/) and separately against an unfamiliarSwift repo — both runs passed
scripts/check-format.shwithok:true.code-qualityagent (91 tests green, shellcheckclean, Fable-5 review) and this
commitagent's independent re-run ofnpm test(91/91) andnpm run lint(clean).Commits
c0dda6edocs: reconcile parity claims with real verifiers (P2E, wj-1zm)ad9788edocs: dogfood the rewritten /wijzer:init on wijzer itself (P2E, wj-1zm)Merge policy
Per this repo's Agent Config: squash + delete branch, auto-merge on green
CI (no human approval gate). This PR is not being merged by the commit
agent — leaving that to
pr-monitor/CI per policy.Test plan
npm test— 91/91 passednpm run lint— shellcheck cleancheck-format.sh— ok:true on wijzer and an unfamiliar Swift repo🤖 Generated with Claude Code