Skip to content

Claude/optimistic dubinsky#12

Merged
hnshah merged 10 commits intomainfrom
claude/optimistic-dubinsky
Apr 17, 2026
Merged

Claude/optimistic dubinsky#12
hnshah merged 10 commits intomainfrom
claude/optimistic-dubinsky

Conversation

@hnshah
Copy link
Copy Markdown
Owner

@hnshah hnshah commented Apr 17, 2026

Summary

Skills-first restructure. PageKit now ships as a Claude Code plugin: one
install, twelve skills, two subagents, a chat-paste path for non-Claude-Code
chat windows. Breaking for repo layout; non-breaking for the method itself.

See CHANGELOG.md v0.3.0 for the full delta.

Verification

  • bash scripts/doctor.sh → PASS
  • bash scripts/slop-check.sh → clean
  • bash scripts/run-check.sh examples/vegan-dog-food-verdel → PUBLISHABLE
  • bash scripts/run-check.sh examples/personal-crm-founders → PUBLISHABLE
  • CI green (this PR)
  • Plugin install smoke test in fresh Claude Code session
  • Full fresh orchestrator run on a fictional product
  • Chat-paste path into ChatGPT

Hiten Shah and others added 10 commits April 15, 2026 21:33
Strangler-fig migration step. `.claude/skills/` → `skills/` and
`.claude/agents/` → `agents/` as sibling copies. Old paths unchanged so
doctor.sh, SessionStart hook, pre-commit, and CI remain green.

Subsequent commits bundle per-skill references, add the plugin manifest,
cut dependencies over, then remove the old copies.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Each step skill now owns its framework, template and prompt under
`skills/<name>/references/`. Shared references (anti-slop.md,
run-logging.md, method.md, output-judgment-template.md) live under
`skills/pagekit/references/` and sibling skills point at them with
`../pagekit/references/...`.

`skills/pagekit/references/method.md` is distilled from pagekit.yaml
so the manifest can retire in a later commit without losing the
method-at-a-glance view.

Old frameworks/, templates/, prompts/ still exist unchanged; SKILL.md
files still reference the old paths. Path-cut-over happens in commit 3.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Updated all 12 SKILL.md files under skills/ to point at `./references/`
(own bundle) and `../pagekit/references/` (orchestrator-owned shared
refs like anti-slop.md and run-logging.md).

Both agents under agents/ now reference the bundled paths:
- skills/pagekit-claim-check/references/framework.md
- skills/pagekit/references/anti-slop.md
- skills/pagekit/references/run-logging.md

Old .claude/skills/ and .claude/agents/ copies remain in place so CI,
doctor.sh, and the SessionStart hook keep passing during the migration.

Strangler-fig step 3/10.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Introduces .claude-plugin/ at repo root with two files:
- plugin.json: plugin-level manifest (name, version, author, repo)
- marketplace.json: marketplace catalog so users can add via
  `/plugin marketplace add hnshah/pagekit` and then
  `/plugin install pagekit@pagekit`

Skills (under skills/) and subagents (under agents/) are auto-discovered
from the conventional directories; they are not re-listed in the manifest.

Strangler-fig step 4/10.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
doctor.sh:
- Keeps the legacy .claude/ bundle check (removed in a later commit).
- Adds new sections for the plugin manifest, top-level skills/ layout,
  shared orchestrator references, per-skill references, and subagents.
- first-page-draft intentionally ships with prompt-only references
  (framework/template inherit upstream); the checker accounts for that.

new-run.sh:
- Resolves canonical prompts from skills/pagekit-<step>/references/prompt.md
  with a fallback to the legacy prompts/<file>.md for the migration window.
- Scaffold placeholders now point at the new skills/pagekit-*/references/
  paths so new runs carry the post-restructure references.
- sources/README.md scaffold now points at examples/vegan-dog-food-verdel/.

claim-check.sh:
- Resolves the canonical claim-check prompt from
  skills/pagekit-claim-check/references/prompt.md with a fallback to
  the legacy prompts/07-claim-check.md.

Smoke-tested end-to-end: new-run.sh scaffolds cleanly against the new
resolution, claim-check.sh still expands the same prompt body.

Strangler-fig step 5/10.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…heck

Worked runs now live in examples/:
- examples/vegan-dog-food-verdel/ (canonical)
- examples/personal-crm-founders/ (second)
- examples/README.md explains the frozen-snapshot framing (pre-restructure
  YAML frontmatter in each example's prompts/ reflects the repo state at
  the time the run executed; new runs resolve prompts from the
  skills/pagekit-*/references/ paths via scripts/new-run.sh)

runs/ is now user scratch space only (empty, .gitkeep holds the dir).

prompt-paste/ is a thin chat-app path:
- 7 canonical prompts copied verbatim from prompts/
- README explains how to run the chain manually in ChatGPT, Perplexity,
  Grok, Gemini, or any non-Claude-Code chat window

scripts/slop-check.sh now scans both runs/ and examples/ in its default
target mode so CI catches regression in the canonical examples.

Strangler-fig step 6/10.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
README.md:
- Opens with the skills-first tagline.
- One-line plugin install (/plugin marketplace add, /plugin install).
- Table of the 12 skills and what each one does.
- Links the two examples and the chat-paste path.
- Drops START-HERE, quickstart, guides, agentic, guided-runs entry points
  (those folders get removed in a later commit). Shortened ~140 lines.

AGENTS.md:
- Neutral contract for any agent cold-reading the repo.
- Points Claude Code at skills/ and agents/; points other agents at
  prompt-paste/. Slimmed to ~45 lines.

CLAUDE.md:
- Repo contract for the orchestrator skill (same instructions it gets
  via the project CLAUDE.md hook). ~50 lines.
- Rules now cite skills/pagekit/references/ paths rather than frameworks/.

All three docs pass scripts/slop-check.sh cleanly.

Strangler-fig step 7/10.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
- Delete START-HERE.md, quickstart/, guides/, guided-runs/, agentic/.
  The rewritten README, AGENTS.md, and CLAUDE.md no longer reference any
  of them; the prompt-paste path covers chat-app users; the orchestrator
  skill covers Claude Code.
- Trim scripts/doctor.sh: drop the START-HERE.md line and the three
  legacy sections (Guided runs, Guides and quickstarts, Agentic tier).
- CONTRIBUTING.md: drop the stale START-HERE.md reference from the
  "must agree with" list.

doctor.sh and slop-check.sh still pass.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Remove `.claude/skills/`, `.claude/agents/`, `frameworks/`, `templates/`,
  `prompts/`, `pagekit.yaml`. Their content now lives (or is absorbed
  into) `skills/`, `agents/`, `.claude-plugin/`, and
  `skills/pagekit/references/method.md`.
- Update every surviving live reference:
  - `skills/pagekit-*/references/prompt.md` YAML frontmatter now points
    at `./framework.md` and `./template.md` (or the upstream siblings for
    first-page-draft).
  - `skills/pagekit-*/references/*.md` body mentions of
    `frameworks/anti-slop.md` now use `../../pagekit/references/anti-slop.md`.
  - `skills/pagekit/references/anti-slop.md` and `run-logging.md` updated
    to reference bundled siblings instead of the deleted root files.
  - `skills/pagekit-signal-doc/references/wedge-definition-template.md`
    is the new home for the wedge-definition template.
- Strip YAML frontmatter from `prompt-paste/*.md` (the README promised
  stripped-for-paste prompts; deliver on that). Update body references
  to plain wording so the chat-paste path does not point at deleted
  files.
- Scripts: drop the legacy-path fallback in `new-run.sh` and
  `claim-check.sh`; update `run-check.sh` and `slop-check.sh` comments
  to point at the bundled references; rewrite `scripts/README.md` to
  match.
- `doctor.sh`: drop the Frameworks, Templates, Canonical prompts, and
  legacy `.claude/` bundle sections. Add a check for the new
  wedge-definition-template location.
- Rewrite `CONTRIBUTING.md` against the new layout.
- Update `.github/PULL_REQUEST_TEMPLATE.md` to reference skills/.

doctor.sh, slop-check.sh, run-check.sh on both examples, claim-check.sh,
and new-run.sh all pass after the cut.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Add a v0.3.0 CHANGELOG entry covering the skills-first restructure:
  plugin manifests, top-level skills/ with bundled references/,
  agents/ at repo root, examples/ (two worked runs), prompt-paste/
  for chat windows. Method integrity note: anti-slop rules,
  run-logging tiers, claim-check severities, and the 7 steps are
  unchanged from v0.2.
- run-check.sh: guard the `missing_pub[@]` expansion against the
  unbound-variable warning that fired when PUBLISHABLE had zero gaps.
  Cosmetic; tier output was always correct.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@hnshah hnshah merged commit 57a2070 into main Apr 17, 2026
1 check passed
@hnshah hnshah deleted the claude/optimistic-dubinsky branch April 17, 2026 18: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