Skip to content

P2D: pivot skills to prompt-driven, drop value-add scripts (wj-o87)#8

Merged
gpxl merged 2 commits into
mainfrom
claude/p2d-prompt-driven-skills
Jul 8, 2026
Merged

P2D: pivot skills to prompt-driven, drop value-add scripts (wj-o87)#8
gpxl merged 2 commits into
mainfrom
claude/p2d-prompt-driven-skills

Conversation

@gpxl

@gpxl gpxl commented Jul 8, 2026

Copy link
Copy Markdown
Owner

Summary

  • Parity-first pivot: removes three bash scripts (inventory.sh,
    inject-pointer.sh, diff-summary.sh) that had no OpenWiki counterpart —
    wijzer's doctrine is to never carry scripted behavior OpenWiki itself
    doesn't have. Their tests are removed alongside them.
  • Prompt-driven skills: skills/init/SKILL.md and skills/update/SKILL.md
    are rewritten as thin wrappers. Discovery, git inspection, and the
    AGENTS.md/CLAUDE.md pointer block are now prompt-driven (following the
    generated disciplines) instead of scripted, calling only the
    exact-semantics scripts (check-noop, snapshot, write-state,
    check-format). update scopes its git inspection from
    openwiki/.last-update.json using OpenWiki's
    gitHead -> updatedAt -> recent-history fallback chain. skills/ask is
    unchanged.
  • Generator reversal: scripts/build-disciplines.mjs now derives
    OpenWiki's verbatim ## OpenWiki pointer block instead of substituting a
    scripted alternative — this deliberately reverses P2C's
    inject-pointer.sh adaptation
    . Fenced code blocks are exempted from
    vocabulary translation and the residual-vocab guard via a new
    withFencesProtected helper (with an even-fence-count assertion), so the
    literal block survives generation unmodified. references/disciplines.md
    and references/wiki-format.md are regenerated from the reversed
    generator.
  • Docs updated to match: PARITY.md, references/state-schema.md,
    README.md, CLAUDE.md, CHANGELOG.md (new P2D entry), plus a stale
    tests/noop.test.ts reference fix. agents/wiki-scout.md aligned to the
    subagent discipline.
  • Tests: tests/plugin-structure.test.ts gains P2D structural guards;
    tests/build-disciplines.test.ts gains verbatim-block + fence-strip
    assertions.

Both independent quality reviews passed: the code-quality agent
(82 tests green, shellcheck clean, no coverage/quality gaps) and an
independent Fable-5 correctness/intent review, after two rounds that
fixed real blockers.

Bead: wj-o87 (P2D)

Merge policy

Squash + delete branch per this repo's pr_merge_strategy. Not merging
here — leaving that to the merge/CI step.

Test plan

  • npm test — 82 tests passed (8 files)
  • npm run lint — shellcheck clean
  • Build — n/a (build_cmd: (none) for this repo)
  • Code-quality agent: PASS
  • Independent Fable-5 review: PASS (after fixes)
  • Manual verification

🤖 Generated with Claude Code

gpxl and others added 2 commits July 8, 2026 16:37
… wj-o87)

wijzer's parity-first doctrine means it should never carry scripted
behavior that OpenWiki itself doesn't have. inventory.sh,
inject-pointer.sh, and diff-summary.sh were all value-adds with no
OpenWiki counterpart, so remove them (and their tests) and move that
responsibility into the skills themselves.

skills/init and skills/update are rewritten as thin wrappers: discovery,
git inspection, and the AGENTS.md/CLAUDE.md pointer are now
prompt-driven, following the generated disciplines, and call only the
exact-semantics scripts (check-noop, snapshot, write-state,
check-format). update scopes its git inspection from
openwiki/.last-update.json using OpenWiki's gitHead -> updatedAt ->
recent-history fallback chain.

This deliberately reverses P2C's inject-pointer.sh adaptation:
build-disciplines.mjs now derives OpenWiki's verbatim `## OpenWiki`
pointer block instead of substituting a scripted alternative. Fenced
code blocks are exempted from vocabulary translation and the residual-
vocab guard via a new withFencesProtected helper (with an even-fence-
count assertion), so the literal block survives generation unmodified.

references/disciplines.md and references/wiki-format.md are
regenerated from the reversed generator. Docs (PARITY.md,
references/state-schema.md, README.md, CLAUDE.md, CHANGELOG.md) and
agents/wiki-scout.md are updated to match, and
tests/plugin-structure.test.ts / tests/build-disciplines.test.ts gain
guards for the new verbatim-block and fence-protection behavior.

Verified: 82 tests green, shellcheck clean, independent Fable-5
correctness/intent review passed after two rounds of fixes.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
…ven-skills

# Conflicts:
#	scripts/build-disciplines.mjs
@gpxl
gpxl merged commit 1510ee5 into main Jul 8, 2026
4 checks passed
@gpxl
gpxl deleted the claude/p2d-prompt-driven-skills branch July 8, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant