chore: extract agentic-dev-workflow pack — prune superseded skills/docs (#380)

[thomasluizon]

Summary

Extracts Orbit's dev-workflow skill suite into a new standalone public repo, then prunes the dead docs left behind. Orbit's own skills are not repointed or replaced — this repo only loses three dead docs plus one minimal surgical fix to prime.

New pack repo

https://github.com/thomasluizon/agentic-dev-workflow — a generic, config-driven, tool-agnostic copy of the pipeline (prime → grill → plan → implement → validate → ship, plus review/audit/intake/research skills). Skill logic lives once in core/; three thin adapters (Claude Code, OpenCode, Codex) point at it. Every project-specific value lives in one workflow.config.yaml; a CI genericity gate keeps core/ free of project strings. MIT-licensed.

Changes in this repo (prune only)

• Delete .claude/research/paywall-redesign.md (stale one-off design doc).
• Delete docs/motion-audit.md (superseded audit artifact).
• Delete .claude/rules/context7.md (obsolete rule; Context7 not in use).
• Inline the git worktree add mechanics into prime's multi-issue Step 2, replacing the delegation to the global worktree-work skill. That global skill (and its four worktree-* siblings) was deleted machine-side as part of this extraction, so this is a minimal surgical fix that keeps prime self-contained rather than dangling on a removed dependency (R1 resolution).

No Orbit skill is repointed to consume the pack; the pack is a standalone copy for reuse on other projects.

Global (machine-level) deletions performed

~/.claude/skills/worktree-{work,plan,execute,work-finish,combine} — removed (superseded by inlined worktree mechanics). Keepers grill-me, ship, clean left intact.

Validation

• New repo: genericity CI gate green on first push; install/sync dry-run + real install verified for all three tool adapters; config schema cross-checked against every consuming skill.
• This repo: diff is docs/markdown only; git grep finds no dangling references to the deleted files.

Closes #380

🤖 Generated with Claude Code

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment

Project	Deployment	Actions	Updated (UTC)
orbit-ui-mobile-web	Ignored		Jul 4, 2026 12:34am

[thomasluizon]

Paired backend prune PR: thomasluizon/orbit-api#282

[claude]

Code Review: PR #388

Scope: PR #388 in thomasluizon/orbit-ui-mobile — "chore: extract agentic-dev-workflow pack — prune superseded skills/docs (#380)"
Recommendation: APPROVE

Summary

Docs/config-only prune: deletes three dead artifacts (.claude/research/paywall-redesign.md, docs/motion-audit.md, .claude/rules/context7.md) and inlines the git worktree add mechanics into prime's multi-issue Step 2, replacing a delegation to a global worktree-work skill that was deleted machine-side. No apps/, packages/, or orbit-api source files are touched. Verified no dangling references to any deleted file/skill remain anywhere in the repo (checked WORKFLOW.md, all of .claude/skills, and .github/), and the final prime/SKILL.md content is internally consistent — the new bash block and the pre-existing "Worktree paths" bullets immediately below it agree on both branch name and path.

Findings

Critical

None

High

None

Medium

None

Low / Info

None

Subagents

Agent	Verdict
parity-checker	N/A — no apps/web/** or apps/mobile/** file changed
i18n-syncer	N/A — no user-facing strings or i18n locale files changed
contract-aligner	N/A — no packages/shared/src/types/* or endpoints.ts changed, and only one repo touched
security-reviewer	N/A — orbit-api not touched

Validation

Check	Result
Lint	N/A — diff contains no lintable source (.ts/.tsx/.cs); only Markdown deleted/edited
Type check	N/A — no TS/C# files changed
Tests	N/A — no test-covered code changed
Build (api)	N/A — orbit-api not touched

Deferred — N/A dimensions & files not verdicted

• DESIGN.md / AI-slop (#8): N/A — no apps/* UI files changed.
• Parity (#9): N/A — no apps/web/** or apps/mobile/** file changed.
• i18n (#10): N/A — no locale/user-facing string changes.
• Contract drift + backward-compat (#11): N/A — no Zod schema or DTO changes; only one repo touched.
• Security (#12) / Backend hard rules (#13): N/A — orbit-api not in scope.
• Type safety (#6) / no console.log (#7): N/A — no application source files changed.
• Comment policy (#4): N/A for lint purposes — local/no-comments is scoped (via each workspace's eslint.config) to apps/web, apps/mobile, packages/shared JS/TS. The # only if... shell comment inside the new fenced bash block in SKILL.md is documentation, not lint-covered app code, and matches the file's existing convention of noting conditional steps.

All four changed files got a verdict — nothing else deferred:

• .claude/research/paywall-redesign.md (deleted) — confirmed stale, zero repo-wide references before or after deletion.
• .claude/rules/context7.md (deleted) — confirmed stale (referenced an unused Context7 MCP integration), zero references.
• docs/motion-audit.md (deleted) — confirmed superseded, zero references, not linked from any other doc or CI workflow.
• .claude/skills/prime/SKILL.md (modified) — the new inlined git worktree add commands for both repos correctly mirror the "Worktree paths" bullets immediately below them (same .claude/worktrees/issue-<N> path, same issue-<N> branch name, same backend/both-label conditional on the second command). No remaining reference to the deleted worktree-work skill anywhere in the file or the rest of the repo.

What's good

• Root-cause discipline (Code Standard #1): rather than leaving prime dangling on a deleted global skill, the fix inlines the mechanics directly, so the skill is self-contained again — no silent breakage waiting for the next multi-issue /prime invocation.
• Clean deletions: all three removed docs were verified genuinely dead (no inbound references) rather than assumed dead.
• Correctly scoped: this PR does not repoint any Orbit skill at the newly-extracted agentic-dev-workflow pack repo, keeping the extraction effort and this repo's cleanup cleanly separated, as the PR description states.

Recommendation

Safe to merge as-is. No cross-platform parity, i18n, contract, or security surface is touched, and the one behavioral change (inlined worktree mechanics in prime) is self-consistent with no dangling references.

Severity	Count
Critical (incl. ⚠️ old-client breaks)	0
High	0
Medium	0
Low / Info	0

[claude]

Code Review: PR #388

Scope: PR #388 in thomasluizon/orbit-ui-mobile — "chore: extract agentic-dev-workflow pack — prune superseded skills/docs (#380)"
Recommendation: APPROVE

Summary

Docs/config-only prune: deletes three dead artifacts (.claude/research/paywall-redesign.md, docs/motion-audit.md, .claude/rules/context7.md) and inlines the git worktree add mechanics into prime's multi-issue Step 2, replacing a delegation to a global worktree-work skill that was deleted machine-side. No apps/, packages/, or orbit-api source files are touched. No new commits since the previous review at this same head commit (b252b8ef) — re-verified the diff is unchanged and no unresolved review threads raise any substantive concern.

Findings

Critical

None

High

None

Medium

None

Low / Info

None

Subagents

Agent	Verdict
parity-checker	N/A — no apps/web/** or apps/mobile/** file changed
i18n-syncer	N/A — no user-facing strings or i18n locale files changed
contract-aligner	N/A — no packages/shared/src/types/* or endpoints.ts changed, and orbit-api is not checked out in this job (not verifiable here)
security-reviewer	N/A — orbit-api not touched in this repo's diff

Validation

Check	Result
Lint	N/A — diff contains no lintable source (.ts/.tsx/.cs); only Markdown deleted/edited
Type check	N/A — no TS/C# files changed
Build / Unit Tests / SonarCloud (CI)	SonarCloud Quality Gate passed, 0 new issues, 0 new security hotspots

Deferred — N/A dimensions

• DESIGN.md / AI-slop: N/A — no apps/* UI files changed.
• Parity: N/A — no apps/web/** or apps/mobile/** file changed.
• i18n: N/A — no locale/user-facing string changes.
• Contract drift + backward-compat: N/A — no Zod schema or DTO changes; orbit-api sibling repo not checked out in this CI job, not verifiable here (paired backend prune tracked separately at thomasluizon/orbit-api#282).
• Security / Backend hard rules: N/A — orbit-api not in scope for this repo's diff.

All four changed files got a verdict:

• .claude/research/paywall-redesign.md (deleted) — confirmed stale, zero repo-wide references.
• .claude/rules/context7.md (deleted) — confirmed stale (unused Context7 MCP integration), zero references.
• docs/motion-audit.md (deleted) — confirmed superseded, zero references, not linked from any other doc or CI workflow.
• .claude/skills/prime/SKILL.md (modified) — the inlined git worktree add commands for both repos correctly mirror the "Worktree paths" bullets immediately below them (same path, same branch name, same backend/both-label conditional). No remaining reference to the deleted worktree-work skill anywhere in the file or repo.

What's good

• Root-cause discipline: rather than leaving prime dangling on a deleted global skill, the fix inlines the mechanics directly, so the skill is self-contained again.
• Clean deletions: all three removed docs were verified genuinely dead rather than assumed dead.
• Correctly scoped: no Orbit skill is repointed at the newly-extracted pack repo, keeping this cleanup separate from the extraction effort.

Recommendation

Safe to merge as-is. No cross-platform parity, i18n, contract, or security surface is touched, and the one behavioral change (inlined worktree mechanics in prime) is self-consistent with no dangling references.

Severity	Count
Critical (incl. ⚠️ old-client breaks)	0
High	0
Medium	0
Low / Info	0

[sonarqubecloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud