Skip to content

feat(prune)!: native-first Phase 1 — prune 21 dead commands + exclude dead docs (115→94) - #279

Merged
Data-Wise merged 5 commits into
devfrom
feature/craft-native-first-breakup
Jul 9, 2026
Merged

feat(prune)!: native-first Phase 1 — prune 21 dead commands + exclude dead docs (115→94)#279
Data-Wise merged 5 commits into
devfrom
feature/craft-native-first-breakup

Conversation

@Data-Wise

@Data-Wise Data-Wise commented Jul 9, 2026

Copy link
Copy Markdown
Owner

What this ships (v3.0.0 increment)

The banked, verified slice of the native-first breakup — the part the evidence supported.

  • Phase 1a — prune 21 dead-namespace commands (115 → 94). Removed workflow/ + task/ namespaces entirely, check/, and dead git/+site/ commands with zero external callers (recon-gated: grepped flow-cli, atlas, ~/.claude, tutorials, homebrew). Kept 15 as thin aliases (live callers). −5,533 lines.
  • Phase 1b — exclude dead docs from the built site. docs/specs,plans,archive dropped from published mkdocs (kept in-repo/git as live capture targets); 12 inbound links rewritten, 29 stale nav entries removed. mkdocs build --strict clean.

⏹ What this deliberately does NOT do (and why)

A Stage-2 workflow empirically tested the native-first thinning premise (3 pilots × 4-axis gate + cross-plugin routing test). Result: NO-GO on Phase 2, two independent blockers:

  1. The do/hub router structurally can't dispatch cross-plugin — no path to native (/simplify) or third-party (agent-skills:*) skills.
  2. 3/3 pilots = thin-keep, none thin-route — the "better plugin" replacements aren't ≥ craft on all 4 axes (native is weaker/auto-applies; complete matches are third-party; ci:generate fails 4/4 — craft ships 11 language templates, the skill generates nothing).

The "craft reinvents worse wheels" hypothesis was tested and refuted for these commands. The 4-axis gate did its job: it stopped deletion of working, superior functionality. Phases 2 (cascade thinning) and 3 (folio split) are dropped; full rationale in the SPEC OUTCOME block.

Guarded via ADR-002 rich-body gate

3 "safe" deletions (git:docs:refcard, check:gen-validator, workflow:insights) were caught carrying unique logic and restored — nothing silently lost.

Safety / dependencies

  • ADR-002 rich-body trap caught mid-build (3 held), the exact silent-loss failure this repo has hit before.
  • Rebased onto current dev (a parallel session added branch-guard + planning docs); redundant duplicate-planning commit dropped, zero code conflict.

Test evidence

  • Identical Phase-1 content passed a full green suite (2,676 passed / 0 real failures) pre-rebase.
  • validate-counts green (94/45/8); test_dist_doc_accuracy 604/604; broken_links + mkdocs --strict green.
  • Full suite re-running on the rebased tree (adds only dev's orthogonal branch-guard change).

Docs trail: docs/specs/SPEC-craft-native-first-breakup-2026-07-09.md (OUTCOME block), GRILL-craft-refactor-2026-07-09.md, docs/ideas/craft-refactor-proposal.html.

🤖 Generated with Claude Code

https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG

Data-Wise and others added 5 commits July 9, 2026 17:24
@Data-Wise @claude
feat(prune)!: native-first Phase 1a — remove 21 dead-namespace comman…
4f1fd4e 
…ds (115→94)

v3.0.0 increment. Recon-gated prune of deprecated commands with zero external
callers, per SPEC-craft-native-first-breakup + Phase 0 recon.

Deleted (21): workflow/ (5) + task/ (3) namespaces entirely, check/ (1),
git/{init,git-recap,sync} + git/docs/{safety-rails,undo-guide,learning-guide}
(true shims → skills/dev/git/references/), site/{nav,init,add,audit,consolidate,
theme,create,preview}. Kept 15 as thin aliases (live flow-cli/tutorial/homebrew
callers). site/docs/frameworks.md retained.

HELD via ADR-002 rich-body gate (restored, NOT deleted): git/docs/refcard,
check/gen-validator, workflow/insights — carry unique logic not preserved in
their skills. Caught by test_skill_referenced_commands_exist during build.

Fixes: bump-version --counts-only + swept 20 files 115→94; count-cascade
(plugin.json breakdown, README, exclusions.txt historical README:42 entry);
de-brittled test floors (hub >=100→59); OPT_IN drops deleted spec-review;
removed obsolete test_git_init suite; repointed 3 true-shim refs in git SKILL.md.

Tests: full suite 0 real failures (2676 passed; expected XFAIL/XPASS only);
validate-counts 94/45/8 green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
@Data-Wise @claude
docs(prune): exclude dead docs (specs/plans/archive) from built mkdoc…
70a9e3f 
…s site

Phase 1b (v3.0.0). SPEC R1-B2/R1-B6: dead process-artifact dirs leave the
PUBLISHED site but stay in-repo/git as live capture targets. No retention rule (C2).

- exclude_docs EXTENDED (not overwritten): specs/ added; plans/+archive/ preserved
- 12 inbound links into specs/_archive rewritten to GitHub blob URLs (R1-B6 sweep)
- 29 stale nav entries + 2 parent headers pointing into excluded dirs removed
- mkdocs build --strict exit 0; broken_links + doc-accuracy tests green

Done via bounded Stage-2 workflow (docs-exclude agent), verified before commit.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
@Data-Wise @claude
docs(spec): record native-first refutation — ship Phase 1, drop Phase…
3490e41 
…s 2-3

Stage-2 workflow (3 pilots + 4-axis gate + routing test) resolved the gating
unknown NO-GO: router can't dispatch cross-plugin AND 3/3 pilots = thin-keep.
Native-first thinning premise refuted for these commands; Phase 1 (prune +
docs-exclude) is the banked, verified value.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
@Data-Wise @claude
chore: drop tasks/ execution scaffolding before PR
7218f52 
Working artifacts (agent-skills plan/todo + Phase-0 recon) don't belong on dev
per merge-cleanup convention. The durable recon finding (delete 21 / hold 3 /
native-first refuted) is recorded in the SPEC OUTCOME block.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
@Data-Wise @claude
test(git-shim): drop deleted sync.md + git-recap.md from shim-correct…
d343242 
…ness array

CI's "Validate Plugin Structure" job (branch-guard bash suites) asserted
commands/git/{sync,git-recap}.md exist; Phase 1a deleted them (zero external
callers). Removed from the SHIMS map; the 7 kept git aliases remain. This bash
suite runs separately from pytest — missed in the pre-push pytest-only pass.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
@Data-Wise
Data-Wise merged commit 4fa44fb into dev Jul 9, 2026
6 checks passed
Data-Wise added a commit that referenced this pull request Jul 9, 2026
@Data-Wise @claude
docs: post-merge status + ORCHESTRATE close-out for PR #279 (native-f…
5255c6a 
…irst Phase 1)

- .STATUS: Phase 1 prune MERGED (115→94), native-first thinning refuted,
  Phases 2-3 dropped, verification-gap note (pytest ≠ CI bash suites)
- ORCHESTRATE: marked CLOSED with outcome banner (not a live checklist)

Durable lessons captured to memory: pytest-doesnt-cover-craft-ci-bash-suites,
native-first-thinning-refuted-for-craft-dev-ops, + rich-body-trap refinement.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
@Data-Wise
Data-Wise deleted the feature/craft-native-first-breakup branch July 9, 2026 23:51
Data-Wise added a commit that referenced this pull request Jul 9, 2026
@Data-Wise @claude
chore: .STATUS — worktree + remote branch cleaned up (PR #279)
7712592 
Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
Data-Wise added a commit that referenced this pull request Jul 10, 2026
@Data-Wise @claude
docs: strip 21 deleted-command refs from docs + website (115→94 cleanup)
d6aaa90 
Completes the doc/website audit for the native-first Phase 1 prune (PR #279).
Removes ~264 references to the 21 deleted commands across 40 files; archives
5 whole-doc-about-deleted-command files to docs/archive/deleted-commands-v3/
and drops their mkdocs nav entries.

- Strip deleted-command refs: git/{init,sync,git-recap}, git:docs/{safety-rails,
  undo-guide,learning-guide}, workflow/{stuck,focus,recap,spec-review},
  task/{status,output,cancel}, site/{init,add,audit,consolidate,nav,theme,
  create,preview}
- Archive (git mv) + de-nav: git-init docs set + git/sync.md
- Fix 5 broken links/anchors to archived/removed content (strict build now clean)
- Counts left at 94/45/8 (managed by bump-version.sh); category subtotals are
  known YELLOW drift, not touched here

mkdocs build --strict: clean (0 warnings). validate-counts: green.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
Data-Wise added a commit that referenced this pull request Jul 10, 2026
@Data-Wise @claude
docs(grill): folio split decision ledger — 5 branches locked
ce5c698 
Fresh grill on the docs-authoring extraction (folio), standing on
docs-maintenance merits only (native-first refuted in PR #279). Locks:
B1 folio=tools+shared docs-standards (content per-project); B2 caller-based
cut (docs:update/changelog stay in craft, authoring→folio); B3 drop DOCS
from /do+/hub; B4 duplicate count tooling; B5 history-preserving subtree
split, craft v4.0.0 + folio v1.0.0. Supersedes the dropped Phase 3.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
Data-Wise added a commit that referenced this pull request Jul 10, 2026
@Data-Wise @claude
docs(v4): skills-first surface brainstorm — 69 is an illusion, target…
1f8ee82 
… ~22 cmds

Adversarial-creative review of the post-split surface: classification
shows 27 of 69 are ALREADY deprecated shims (v3 skill-migration debt),
2 more shims, 6 routers, 2 references — only 32 genuinely rich.
Proposal: complete the shim-kill at v4 (ADR-002 salvage gate, ~12%
rich-body base rate from PR #279), kill 2 teaching-residue utils,
demote 1 internal ref, subcommand-consolidate ci (8 to 1), arch (4 to
1), code-audits (5 to 1), orch (3 to 1), plan (2 to 1) via savant's
proven depth-1 pattern. Result: ~22 commands + ~40 skills. Does NOT
re-run refuted native-first (own skills, not third-party). Rider on
the v4.0.0 folio release: one major, one migration, one cascade.
Self-attacked on 6 angles, all survive with gates (skill
slash-invocability check per shim; nested skills excepted).

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01PCmhbNUV6iSQrx1USQjACG
Data-Wise added a commit that referenced this pull request Jul 14, 2026
@Data-Wise @claude
fix(tests): re-baseline 3 stale v115 assertions after site:create prune
1f57b93 
PR #279's native-first prune (2026-07-09) intentionally removed
/craft:site:create and its content from docs/workflows/index.md (Site
Creation Workflow section, 3 click bindings) and docs/PLAYGROUND.md
(old Scenario 4, renumbering 5/6 down by one) -- but never updated
this test file's expectations to match. Not regressions: verified via
git show on each removal commit that the content loss was deliberate.

- test_visual_workflows_page_exists: drop "Site Creation Workflow"
  from expected list, diagram floor 5->4 (matches the page's own
  updated front matter)
- test_interactive_mermaid_diagrams: click-node floor 10->7 (3 clicks
  belonged to the removed diagram)
- test_command_playground_exists: drop "Scenario 6:" (renumbered to
  Scenario 5, already covered)

Full suite: 2733 passed / 1 unrelated pre-existing timing flake
(test_block_decision_under_200ms, 205ms vs 200ms budget under
full-suite system load; confirmed passing in isolation -- same class
as the already-documented test_performance_cached_run flake).

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Data-Wise added a commit that referenced this pull request Jul 16, 2026
@Data-Wise
release: v4.0.0 (folio split + v4 surface rider)
f1fb851 
Version bump from 2.61.2 to 4.0.0 via bump-version.sh (47 commands, 2
agents, 40 skills, 64 specs). Converts CHANGELOG.md/docs/CHANGELOG.md
Unreleased section to 4.0.0, expanding docs/CHANGELOG.md with the full
PR list since 2.61.1 (#279 through #293).

Closes gaps surfaced by pre-release-check.sh's doc-coverage and
docs-staleness gates:
- Missing REFCARD rows (code:demo, git:issue-check) and mkdocs.yml nav
  entries (code:demo, site:deploy); new docs/commands/code/demo.md and
  docs/commands/site/deploy.md written (no prior site page existed for
  either command).
- audit-router skill (T3.6) was never added to docs/skills-agents.md.
- v4 rollback runbook added to mkdocs.yml nav.

pre-release-check.sh 4.0.0: PASSED (remaining items are cross-repo or
pre-existing advisories: homebrew-tap's craft.rb desc count, a
pre-existing internal-doc nav gap, and .STATUS drift notes).

Full pytest suite: 2637 passed, 0 failed. CI bash suites all green
(branch_guard_e2e's sole failure is the documented runner-sensitive
perf test, confirmed via SKIP_PERF).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Data-Wise