@patricksardinha/agentkit-cli 0.2.0 → 0.4.0

package/tests/generators/playbookGenerator.test.ts

@@ -10,7 +10,7 @@ const sampleAgents: Agent[] = [
     name: 'Infra & Setup',
     fullName: 'Agent 1 · Infra & Setup',
     slug: 'infra-setup',
-    scope: 'scaffolding du projet',
+    scope: 'project scaffolding',
     outputs: ['package.json', 'tsconfig.json'],
     criterion: 'npm run build',
   },
@@ -19,76 +19,166 @@ const sampleAgents: Agent[] = [
     name: 'Components',
     fullName: 'Agent 2 · Components',
     slug: 'components',
-    scope: 'composants UI réutilisables',
+    scope: 'reusable UI components',
     outputs: ['src/components/'],
     criterion: 'npm test',
   },
 ]
 
-describe('generatePlaybook', () => {
-  it('returns a non-empty string', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(typeof result).toBe('string')
-    expect(result.length).toBeGreaterThan(0)
+// ─── Common behaviour (both modes) ────────────────────────────────────────────
+
+describe('generatePlaybook (common)', () => {
+  for (const hasBlueprint of [true, false]) {
+    const label = hasBlueprint ? 'hasBlueprint: true' : 'hasBlueprint: false'
+
+    it(`[${label}] returns a non-empty string`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(typeof result).toBe('string')
+      expect(result.length).toBeGreaterThan(0)
+    })
+
+    it(`[${label}] contains the project name`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('my-app')
+    })
+
+    it(`[${label}] contains the one-instruction block`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('Read PLAYBOOK.md and execute the procedure')
+    })
+
+    it(`[${label}] contains global execution rules`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('Global Execution Rules')
+      expect(result).toContain('max 3 attempts')
+      expect(result).toContain('human validation')
+      expect(result).toContain('Never move to the next agent')
+    })
+
+    it(`[${label}] always contains Phase 0`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('Phase 0')
+    })
+
+    it(`[${label}] always contains Phase 1`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('Phase 1')
+    })
+
+    it(`[${label}] Phase 0 comes before Phase 1`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result.indexOf('Phase 0')).toBeLessThan(result.indexOf('Phase 1'))
+    })
+
+    it(`[${label}] contains a block for each agent`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      for (const agent of sampleAgents) {
+        expect(result).toContain(agent.fullName)
+        expect(result).toContain(agent.scope)
+        expect(result).toContain(`agents/agent-${agent.number}-${agent.slug}/skills.md`)
+      }
+    })
+
+    it(`[${label}] each agent block contains outputs and criterion`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('package.json')
+      expect(result).toContain('npm run build')
+      expect(result).toContain('src/components/')
+      expect(result).toContain('npm test')
+    })
+
+    it(`[${label}] contains the future iterations section`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('Future Iterations')
+      expect(result).toContain('agentkit add --feature')
+    })
+
+    it(`[${label}] contains the human validation section`, () => {
+      const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint })
+      expect(result).toContain('Human Validation Required')
+      expect(result).toContain('3 consecutive failures')
+      expect(result).toContain('Missing external dependency')
+      expect(result).toContain('End of Phase 0')
+    })
+
+    it(`[${label}] handles an empty agents list gracefully`, () => {
+      const result = generatePlaybook({ agents: [], projectName: 'empty-project', hasBlueprint })
+      expect(typeof result).toBe('string')
+      expect(result).toContain('empty-project')
+    })
+  }
+})
+
+// ─── Phase 0 with blueprint (Decomposition mode) ──────────────────────────────
+
+describe('generatePlaybook Phase 0 — Decomposition (hasBlueprint: true)', () => {
+  it('contains the decomposition title', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: true })
+    expect(result).toContain('Agent Decomposition')
+  })
+
+  it('references PROJECT_BLUEPRINT.md', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: true })
+    expect(result).toContain('PROJECT_BLUEPRINT.md')
+  })
+
+  it('contains decomposition rules', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: true })
+    expect(result).toContain('Maximum 6 agents')
+    expect(result).toContain('AGENT_WORKFLOW.md')
   })
 
-  it('contains the project name', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(result).toContain('my-app')
+  it('contains the validation gate', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: true })
+    expect(result).toContain('Should I proceed')
   })
 
-  it('starts with the unique instruction', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(result).toContain("Donne cette instruction à Claude Code")
-    expect(result).toContain("Lis PLAYBOOK.md et exécute la procédure")
+  it('does NOT contain Project Discovery language', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: true })
+    expect(result).not.toContain('Project Discovery')
+    expect(result).not.toContain('Ask the user these questions')
   })
+})
+
+// ─── Phase 0 without blueprint (Discovery mode) ───────────────────────────────
 
-  it('contains global execution rules', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(result).toContain('Règles d\'exécution globales')
-    expect(result).toContain('3 tentatives')
-    expect(result).toContain('validation humaine')
-    expect(result).toContain('Ne jamais passer')
+describe('generatePlaybook Phase 0 — Discovery (hasBlueprint: false)', () => {
+  it('contains the discovery title', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: false })
+    expect(result).toContain('Project Discovery')
   })
 
-  it('contains a block for each agent', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    for (const agent of sampleAgents) {
-      expect(result).toContain(agent.fullName)
-      expect(result).toContain(agent.scope)
-      expect(result).toContain(`agents/agent-${agent.number}-${agent.slug}/skills.md`)
-    }
+  it('instructs Claude Code to ask the user questions', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: false })
+    expect(result).toContain('Ask the user these questions')
+    expect(result).toContain('What is this project')
+    expect(result).toContain('What are the main features')
   })
 
-  it('each agent block contains outputs and criterion', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(result).toContain('package.json')
-    expect(result).toContain('npm run build')
-    expect(result).toContain('src/components/')
-    expect(result).toContain('npm test')
+  it('still contains decomposition rules', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: false })
+    expect(result).toContain('Maximum 6 agents')
+    expect(result).toContain('AGENT_WORKFLOW.md')
   })
 
-  it('contains the "Itérations futures" section', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(result).toContain('Itérations futures')
-    expect(result).toContain('agentkit add --feature')
+  it('still contains the validation gate', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: false })
+    expect(result).toContain('Should I proceed')
   })
 
-  it('contains the human validation section', () => {
-    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app' })
-    expect(result).toContain('Validation humaine requise')
-    expect(result).toContain('3 échecs consécutifs')
-    expect(result).toContain('Dépendance externe manquante')
-    expect(result).toContain('Conflit')
+  it('does NOT reference PROJECT_BLUEPRINT.md as a file to read', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: false })
+    expect(result).not.toContain('A `PROJECT_BLUEPRINT.md` was provided')
   })
 
-  it('handles an empty agents list gracefully', () => {
-    const result = generatePlaybook({ agents: [], projectName: 'empty-project' })
-    expect(typeof result).toBe('string')
-    expect(result).toContain('empty-project')
+  it('does NOT contain Agent Decomposition title', () => {
+    const result = generatePlaybook({ agents: sampleAgents, projectName: 'my-app', hasBlueprint: false })
+    expect(result).not.toContain('Agent Decomposition (run this first)')
   })
 })
 
+// ─── toSlug ───────────────────────────────────────────────────────────────────
+
 describe('toSlug', () => {
   it('converts name to lowercase hyphenated slug', () => {
     expect(toSlug('Infra & Setup')).toBe('infra-setup')
@@ -104,6 +194,8 @@ describe('toSlug', () => {
   })
 })
 
+// ─── extractAgentsFromWorkflow ────────────────────────────────────────────────
+
 describe('extractAgentsFromWorkflow', () => {
   it('extracts agents from a workflow string matching template format', () => {
     const workflow = `# Agent Workflow — React Project
@@ -149,4 +241,4 @@ Critère   : npm run build
   it('returns empty array for content with no agent blocks', () => {
     expect(extractAgentsFromWorkflow('# Just a title\nNo agents here.')).toEqual([])
   })
-})
+})

package/tests/generators/workflowGenerator.test.ts

@@ -81,4 +81,44 @@ describe('generateWorkflow', () => {
     expect(typeof result).toBe('string')
     expect(result.length).toBeGreaterThan(0)
   })
+
+  describe('blueprint placeholder', () => {
+    const blueprint = `# My Project\n\n## Goal\nBuild something\n\n## Features\n- Auth\n- Dashboard\n`
+
+    it('returns a placeholder when blueprintContent is provided', () => {
+      const result = generateWorkflow(makeStack('react'), blueprint)
+      expect(result).toContain('AGENT_WORKFLOW.md')
+      expect(result).toContain('Phase 0')
+      expect(result).toContain('PROJECT_BLUEPRINT.md')
+      expect(result).toContain('Waiting for Phase 0 decomposition')
+    })
+
+    it('uses projectName in heading when provided', () => {
+      const result = generateWorkflow(makeStack('react'), blueprint, 'my-app')
+      expect(result).toContain('AGENT_WORKFLOW.md — my-app')
+    })
+
+    it('falls back to framework name when projectName is omitted', () => {
+      const result = generateWorkflow(makeStack('react'), blueprint)
+      expect(result).toContain('AGENT_WORKFLOW.md — react')
+    })
+
+    it('does NOT parse blueprint sections as agents', () => {
+      const result = generateWorkflow(makeStack('react'), blueprint, 'my-app')
+      expect(result).not.toContain('Agent · Goal')
+      expect(result).not.toContain('Agent · Features')
+      expect(result).not.toContain('Agent 1')
+    })
+
+    it('returns the placeholder for every framework when blueprint is provided', () => {
+      const frameworks: StackInfo['framework'][] = [
+        'react', 'nextjs', 'tauri', 'fastapi', 'express', 'node', 'unknown',
+      ]
+      for (const framework of frameworks) {
+        const result = generateWorkflow(makeStack(framework), blueprint, 'proj')
+        expect(result).toContain('Phase 0')
+        expect(result).not.toContain('Agent 1')
+      }
+    })
+  })
 })