learnship 2.0.1 → 2.0.2

package/.claude-plugin/plugin.json

@@ -1,7 +1,7 @@
 {
   "name": "learnship",
   "description": "Agentic engineering done right — 49 structured workflows, persistent memory across sessions, integrated learning partner, and impeccable UI design system. Works with Claude Code, Windsurf, Cursor, Gemini CLI, OpenCode, and Codex.",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "author": {
     "name": "Favio Vazquez",
     "email": "favio.vazquezp@gmail.com"

package/.cursor-plugin/plugin.json

@@ -2,7 +2,7 @@
   "name": "learnship",
   "displayName": "learnship",
   "description": "Agentic engineering done right — 49 structured workflows, persistent memory across sessions, integrated learning partner, and impeccable UI design system.",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "logo": "assets/logo.png",
   "author": {
     "name": "Favio Vazquez",

package/gemini-extension.json

@@ -1,6 +1,6 @@
 {
   "name": "learnship",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "Agentic engineering done right — 49 structured workflows, persistent memory across sessions, integrated learning partner, and impeccable UI design system.",
   "author": "Favio Vazquez",
   "homepage": "https://faviovazquez.github.io/learnship/",

package/learnship/workflows/health.md

@@ -96,7 +96,14 @@ git status --short .planning/ 2>/dev/null | head -10
 node -e "
 try{
   const cfg=JSON.parse(require('fs').readFileSync('.planning/config.json','utf8'));
-  const missing=['mode','granularity','model_profile','learning_mode'].filter(k=>!(k in cfg));
+  const top=['mode','granularity','model_profile','learning_mode','parallelization','test_first'].filter(k=>!(k in cfg));
+  const nested=[];
+  if(!cfg.planning||!('commit_mode' in cfg.planning)) nested.push('planning.commit_mode');
+  if(!cfg.workflow||!('review' in cfg.workflow)) nested.push('workflow.review');
+  if(!cfg.workflow||!('solutions_search' in cfg.workflow)) nested.push('workflow.solutions_search');
+  if(!cfg.review||!('auto_after_verify' in cfg.review)) nested.push('review.auto_after_verify');
+  if(!cfg.ship) nested.push('ship.*');
+  const missing=[...top,...nested];
   if(missing.length)console.log('W004: config.json missing fields: '+missing.join(', '));
 }catch(e){}
 "

package/learnship/workflows/new-milestone.md

@@ -94,6 +94,43 @@ If a MILESTONE-CONTEXT.md was consumed, delete it:
 git rm .planning/MILESTONE-CONTEXT.md 2>/dev/null || true
 ```
 
+## Step 6b: Config Review
+
+Check if the project config has all v2 keys:
+
+```bash
+node -e "
+const fs=require('fs');
+try{
+  const cfg=JSON.parse(fs.readFileSync('.planning/config.json','utf8'));
+  const missing=[];
+  if(!('model_profile' in cfg)) missing.push('model_profile');
+  if(!('test_first' in cfg)) missing.push('test_first');
+  if(!('parallelization' in cfg)) missing.push('parallelization');
+  if(!cfg.workflow||!('review' in cfg.workflow)) missing.push('workflow.review');
+  if(!cfg.workflow||!('solutions_search' in cfg.workflow)) missing.push('workflow.solutions_search');
+  if(!cfg.review||!('auto_after_verify' in cfg.review)) missing.push('review.auto_after_verify');
+  if(!cfg.ship) missing.push('ship.*');
+  if(!cfg.planning||!('commit_mode' in cfg.planning)) missing.push('planning.commit_mode');
+  if(missing.length)console.log('MISSING: '+missing.join(', '));
+  else console.log('CONFIG_COMPLETE');
+}catch(e){console.log('NO_CONFIG');}
+"
+```
+
+**If MISSING or NO_CONFIG:** Offer to update config with v2 defaults:
+
+```
+Your config is missing some v2 settings: [list]
+
+These control: multi-persona review, solutions search, ship pipeline, TDD mode.
+Want me to add them with recommended defaults? (yes/no)
+```
+
+If yes: merge missing keys into `.planning/config.json` using the defaults from `@./templates/config.json`. Show the updated config for confirmation.
+
+**If CONFIG_COMPLETE:** Continue silently.
+
 ## Step 7: Research Decision
 
 Read `workflow.research` from `.planning/config.json`.
@@ -183,7 +220,7 @@ git commit -m "docs: update AGENTS.md — milestone [VERSION] started"
 
 **[VERSION] — [Name]** — [N] phases, [X] requirements
 
-▶ Next: discuss-phase 1 → plan-phase 1 → execute-phase 1
+▶ Next: discuss-phase 1 → plan-phase 1 → execute-phase 1 → verify-work 1 → review → ship → compound
 ```
 
 ---

package/learnship/workflows/new-project.md

@@ -99,14 +99,40 @@ Ask: "How should the learning partner (agentic-learning) work during this projec
 - **Auto** (recommended) — I'll offer relevant learning actions at natural checkpoints (after planning, after execution, etc.)
 - **Manual** — I'll only activate when you explicitly invoke `@agentic-learning`
 
-**Group C — Workflow agents (these add quality but cost tokens/time):**
+**Group C — Model profile:**
 
-Ask: "Which workflow agents should be enabled?"
+Ask: "Which model quality tier do you want?"
+- **Quality** — Large-tier models for all decision-making agents (highest cost, best results)
+- **Balanced** (recommended) — Large for planning, medium for execution
+- **Budget** — Medium for code, small for research/verification (lowest cost)
+
+**Group D — Development style:**
+
+Ask: "Test-first (TDD) mode?"
+- **No** (recommended) — Write tests alongside implementation
+- **Yes** — Enforce red-green-refactor: write failing test first, verify red, implement, verify green
+
+**Group E — Workflow agents (these add quality but cost tokens/time):**
+
+Ask: "Which workflow agents and quality steps should be enabled?"
 - **Research** (recommended) — Investigate domain before planning each phase
 - **Plan Check** (recommended) — Verify plans achieve their goals before execution
 - **Verifier** (recommended) — Confirm deliverables match phase goals after execution
+- **Review** (recommended) — Multi-persona code review after verification
+- **Solutions Search** (recommended) — Search prior solutions for reusable patterns during planning
+
+Ask: "Auto-trigger review after verify-work passes?"
+- **No** (recommended) — Run `/review` manually when ready
+- **Yes** — Automatically start review after verification succeeds
 
-**Group D — Parallel execution:**
+**Group F — Ship pipeline defaults:**
+
+Ask: "Ship pipeline preferences?"
+- **Auto-test before shipping** (recommended: yes) — Run tests before every ship
+- **Conventional commits** (recommended: yes) — Use `feat:`, `fix:`, `docs:` commit prefixes
+- **Auto-generate PR description** (recommended: yes) — Create PR body from commit messages
+
+**Group G — Parallel execution:**
 
 <!-- LEARNSHIP_PARALLEL_BLOCK -->
 
@@ -124,30 +150,51 @@ Create `.planning/config.json` with all settings:
 {
   "mode": "yolo|interactive",
   "granularity": "coarse|standard|fine",
-  "commit_docs": true|false,
-  "commit_mode": "auto|manual",
+  "model_profile": "quality|balanced|budget",
   "learning_mode": "auto|manual",
   "parallelization": false|true,
+  "test_first": false|true,
+  "planning": {
+    "commit_docs": true|false,
+    "commit_mode": "auto|manual",
+    "search_gitignored": false
+  },
   "workflow": {
     "research": true|false,
     "plan_check": true|false,
-    "verifier": true|false
+    "verifier": true|false,
+    "validation": true|false,
+    "review": true|false,
+    "solutions_search": true|false
+  },
+  "review": {
+    "auto_after_verify": false|true
+  },
+  "ship": {
+    "auto_test": true|false,
+    "conventional_commits": true|false,
+    "pr_template": true|false
+  },
+  "git": {
+    "branching_strategy": "none|phase|milestone",
+    "phase_branch_template": "phase-{phase}-{slug}",
+    "milestone_branch_template": "{milestone}-{slug}"
   }
 }
 ```
 
-If `commit_docs` is false, add `.planning/` to `.gitignore`:
+If `planning.commit_docs` is false, add `.planning/` to `.gitignore`:
 ```bash
 echo ".planning/" >> .gitignore
 ```
 
-**If `commit_mode` is `auto`:** Stage and commit the initial setup now:
+**If `planning.commit_mode` is `auto`:** Stage and commit the initial setup now:
 ```bash
 git add .gitignore .planning/config.json
 git commit -m "chore: initialize learnship project setup"
 ```
 
-**If `commit_mode` is `manual`:** Show this message and skip all future commit steps:
+**If `planning.commit_mode` is `manual`:** Show this message and skip all future commit steps:
 ```
 → Manual commit mode — I will not run any git commits.
   Stage and commit whenever you are ready.
@@ -401,9 +448,13 @@ Files created:
 
 ▶ Next: `/discuss-phase 1` — **start here, not `/plan-phase`**
 
-The phase loop is: `discuss-phase` → `plan-phase` → `execute-phase` → `verify-work`
+The full phase loop:
+`discuss-phase` → `plan-phase` → `execute-phase` → `verify-work` → `review` → `ship` → `compound`
+
 `discuss-phase` is mandatory before planning — it captures your intent and writes the CONTEXT.md that plan-phase depends on. Skipping it means planning without context.
 
+After verify-work passes: `/review` for multi-persona code review, `/ship` to test+commit+push+PR, `/compound` to capture what you learned.
+
 > **Platform detected:** `[PLATFORM]` — parallelization is `[true/false]`
 ```
 

package/learnship/workflows/settings.md

@@ -22,15 +22,28 @@ cp templates/config.json .planning/config.json 2>/dev/null || cat > .planning/co
   "granularity": "standard",
   "model_profile": "balanced",
   "learning_mode": "auto",
+  "parallelization": false,
+  "test_first": false,
   "planning": {
     "commit_docs": true,
+    "commit_mode": "auto",
     "search_gitignored": false
   },
   "workflow": {
     "research": true,
     "plan_check": true,
     "verifier": true,
-    "validation": true
+    "validation": true,
+    "review": true,
+    "solutions_search": true
+  },
+  "review": {
+    "auto_after_verify": false
+  },
+  "ship": {
+    "auto_test": true,
+    "conventional_commits": true,
+    "pr_template": true
   },
   "git": {
     "branching_strategy": "none",
@@ -64,12 +77,19 @@ Current configuration:
 [2] Granularity:       [current] (coarse | standard | fine)
 [3] Model profile:     [current] (quality | balanced | budget)
 [4] Learning mode:     [current] (auto | manual)
-[5] Research agent:    [on/off]
-[6] Plan check agent:  [on/off]
-[7] Verifier agent:    [on/off]
-[8] Test validation:  [on/off]
-[9] Git branching:     [current] (none | phase | milestone)
-[10] Commit docs:      [on/off]
+[5] Test-first (TDD):  [on/off]
+[6] Research agent:    [on/off]
+[7] Plan check agent:  [on/off]
+[8] Verifier agent:    [on/off]
+[9] Test validation:   [on/off]
+[10] Review workflow:   [on/off]
+[11] Solutions search:  [on/off]
+[12] Auto-review after verify: [on/off]
+[13] Ship: auto-test:   [on/off]
+[14] Ship: conventional commits: [on/off]
+[15] Ship: PR template: [on/off]
+[16] Git branching:     [current] (none | phase | milestone)
+[17] Commit docs:       [on/off]
 
 Enter a number to change a setting, or 'done' to save.
 ```
@@ -118,7 +138,16 @@ Learning mode controls when learning actions are offered:
 Current: [current]. New value?
 ```
 
-**[5-7] Agent toggles (research / plan_check / verifier):**
+**[5] Test-first (TDD):**
+```
+Test-first mode enforces red-green-refactor during execute-phase:
+- on: write failing test → verify red → implement → verify green
+- off: write tests alongside implementation (default)
+
+Current: [current]. New value? (on/off)
+```
+
+**[6-8] Agent toggles (research / plan_check / verifier):**
 ```
 [Research / Plan check / Verifier] agent:
 - on: agent runs (recommended for production work)
@@ -127,7 +156,7 @@ Current: [current]. New value?
 Current: [current]. New value? (on/off)
 ```
 
-**[8] Test validation:**
+**[9] Test validation:**
 ```
 Test validation maps automated test coverage to requirements during plan-phase.
 - on: plans include automated verify commands per task (recommended)
@@ -136,7 +165,43 @@ Test validation maps automated test coverage to requirements during plan-phase.
 Current: [current]. New value? (on/off)
 ```
 
-**[9] Git branching:**
+**[10] Review workflow:**
+```
+Multi-persona code review after verification:
+- on: /review is available and can be auto-triggered (recommended)
+- off: skip review workflow
+
+Current: [current]. New value? (on/off)
+```
+
+**[11] Solutions search:**
+```
+Search .planning/solutions/ for prior art during plan-phase:
+- on: reuse patterns from solved problems (recommended)
+- off: skip solutions search
+
+Current: [current]. New value? (on/off)
+```
+
+**[12] Auto-review after verify:**
+```
+Automatically trigger /review after verify-work passes:
+- on: review starts immediately after successful verification
+- off: run /review manually when ready (default)
+
+Current: [current]. New value? (on/off)
+```
+
+**[13-15] Ship pipeline (auto_test / conventional_commits / pr_template):**
+```
+[Auto-test / Conventional commits / PR template]:
+- on: enabled (recommended)
+- off: disabled
+
+Current: [current]. New value? (on/off)
+```
+
+**[16] Git branching:**
 ```
 Branching strategy:
 - none: no automatic branches (good for solo work)
@@ -146,7 +211,7 @@ Branching strategy:
 Current: [current]. New value?
 ```
 
-**[10] Commit docs:**
+**[17] Commit docs:**
 ```
 Whether .planning/ files are committed to git:
 - on: planning artifacts tracked in git (default)
@@ -166,15 +231,28 @@ cat > .planning/config.json << EOF
   "granularity": "[value]",
   "model_profile": "[value]",
   "learning_mode": "[value]",
+  "parallelization": [true/false],
+  "test_first": [true/false],
   "planning": {
     "commit_docs": [true/false],
+    "commit_mode": "[auto/manual]",
     "search_gitignored": false
   },
   "workflow": {
     "research": [true/false],
     "plan_check": [true/false],
     "verifier": [true/false],
-    "validation": [true/false]
+    "validation": [true/false],
+    "review": [true/false],
+    "solutions_search": [true/false]
+  },
+  "review": {
+    "auto_after_verify": [true/false]
+  },
+  "ship": {
+    "auto_test": [true/false],
+    "conventional_commits": [true/false],
+    "pr_template": [true/false]
   },
   "git": {
     "branching_strategy": "[value]",

package/learnship/workflows/verify-work.md

@@ -181,14 +181,28 @@ Display summary:
 | Skipped | [N]   |
 ```
 
-**If no issues:** 
+**If no issues:**
+
+Read `review.auto_after_verify` from `.planning/config.json` (defaults to `false`).
+
+**If `review.auto_after_verify` is `true`:**
+```
+All tests passed. ✓
+
+Auto-review is enabled — starting multi-persona code review now.
+```
+Immediately run the `review` workflow for this phase's changes.
+
+**If `review.auto_after_verify` is `false`:**
 ```
 All tests passed. ✓
 
-💡 Compound learnings? Run `/compound` to capture any notable solutions
-or patterns from this phase while context is fresh.
+▶ Recommended next steps:
+  `/review`   — multi-persona code review (6 lenses: correctness, testing, security, performance, maintainability, adversarial)
+  `/ship`     — test → lint → commit → push → PR
+  `/compound` — capture notable solutions or patterns while context is fresh
 
-▶ Next: discuss-phase [X+1]
+▶ Or continue: discuss-phase [X+1]
 ```
 
 **If issues found:** Continue to Step 7.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "learnship",
-  "version": "2.0.1",
+  "version": "2.0.2",
   "description": "Learn as you build. Build with intent. — A multi-platform agentic engineering system for Windsurf, Claude Code, Cursor, OpenCode, Gemini CLI, and Codex: spec-driven workflows, integrated learning, and production-grade design.",
   "keywords": [
     "agentic",

package/templates/agents.md

@@ -97,7 +97,7 @@ follow. The map should become territory.
 This project uses **learnship**. Key facts:
 
 - All planning artifacts live in `.planning/` — read STATE.md and ROADMAP.md first when unsure where we are
-- The phase loop: `discuss-phase` → `plan-phase` → `execute-phase` → `verify-work`
+- The phase loop: `discuss-phase` → `plan-phase` → `execute-phase` → `verify-work` → `review` → `ship` → `compound`
 - Current status is always in `.planning/STATE.md`
 - Decisions are tracked in `.planning/DECISIONS.md` — read it before proposing approaches that may conflict
 - Run `/ls` if context is unclear about what phase we're on or what to do next — it shows status and offers to run the next step