hone-ai 0.5.0 → 0.10.0

package/src/prompt.test.ts

@@ -1,41 +1,41 @@
-import { describe, expect, test, beforeEach, afterEach } from 'bun:test';
-import { mkdirSync, writeFileSync, rmSync, existsSync } from 'fs';
-import { join } from 'path';
-import { constructPrompt, type PromptPhase } from './prompt';
-import type { HoneConfig } from './config';
+import { describe, expect, test, beforeEach, afterEach } from 'bun:test'
+import { mkdirSync, writeFileSync, rmSync, existsSync } from 'fs'
+import { join } from 'path'
+import { constructPrompt, type PromptPhase } from './prompt'
+import type { HoneConfig } from './config'
 
-const TEST_WORKSPACE = join(process.cwd(), '.test-prompt-workspace');
-const TEST_PLANS_DIR = join(TEST_WORKSPACE, '.plans');
+const TEST_WORKSPACE = join(process.cwd(), '.test-prompt-workspace')
+const TEST_PLANS_DIR = join(TEST_WORKSPACE, '.plans')
 
 beforeEach(() => {
   // Create isolated test workspace
   if (existsSync(TEST_WORKSPACE)) {
-    rmSync(TEST_WORKSPACE, { recursive: true, force: true });
+    rmSync(TEST_WORKSPACE, { recursive: true, force: true })
   }
-  mkdirSync(TEST_PLANS_DIR, { recursive: true });
-  
+  mkdirSync(TEST_PLANS_DIR, { recursive: true })
+
   // Change to test workspace
-  process.chdir(TEST_WORKSPACE);
-});
+  process.chdir(TEST_WORKSPACE)
+})
 
 afterEach(() => {
   // Clean up and return to original directory
-  const originalDir = join(TEST_WORKSPACE, '..');
-  process.chdir(originalDir);
+  const originalDir = join(TEST_WORKSPACE, '..')
+  process.chdir(originalDir)
   if (existsSync(TEST_WORKSPACE)) {
-    rmSync(TEST_WORKSPACE, { recursive: true, force: true });
+    rmSync(TEST_WORKSPACE, { recursive: true, force: true })
   }
-});
+})
 
 const mockConfig: HoneConfig = {
   defaultAgent: 'claude',
   models: {
     opencode: 'claude-sonnet-4-20250514',
-    claude: 'claude-sonnet-4-20250514'
+    claude: 'claude-sonnet-4-20250514',
   },
   feedbackInstructions: 'test: bun test',
-  lintCommand: 'bun run lint'
-};
+  lintCommand: 'bun run lint',
+}
 
 describe('constructPrompt', () => {
   test('includes phase header', () => {
@@ -43,239 +43,238 @@ describe('constructPrompt', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('# HONE: IMPLEMENT PHASE');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('# HONE: IMPLEMENT PHASE')
+  })
+
   test('includes AGENTS.md reference if exists', () => {
-    writeFileSync(join(TEST_WORKSPACE, 'AGENTS.md'), '# Learning 1\nSome pattern');
+    writeFileSync(join(TEST_WORKSPACE, 'AGENTS.md'), '# Learning 1\nSome pattern')
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('# CONTEXT FILES');
-    expect(prompt).toContain('@AGENTS.md');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('# CONTEXT FILES')
+    expect(prompt).toContain('@AGENTS.md')
+  })
+
   test('includes task file reference', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    title: Test Task\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('# CONTEXT FILES');
-    expect(prompt).toContain('@.plans/tasks-test.yml');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('# CONTEXT FILES')
+    expect(prompt).toContain('@.plans/tasks-test.yml')
+  })
+
   test('includes progress file reference if exists', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    writeFileSync(
-      join(TEST_PLANS_DIR, 'progress-test.txt'),
-      'Previous iteration completed'
-    );
-    
+    )
+    writeFileSync(join(TEST_PLANS_DIR, 'progress-test.txt'), 'Previous iteration completed')
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('# CONTEXT FILES');
-    expect(prompt).toContain('@.plans/progress-test.txt');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('# CONTEXT FILES')
+    expect(prompt).toContain('@.plans/progress-test.txt')
+  })
+
   test('implement phase includes task selection instructions', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('# TASK SELECTION');
-    expect(prompt).toContain('bun test');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('# TASK SELECTION')
+    expect(prompt).toContain('bun test')
+  })
+
   test('implement phase includes lint command if configured', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('bun run lint');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('bun run lint')
+  })
+
   test('review phase includes review checklist', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'review',
       featureName: 'test',
       config: mockConfig,
-      taskId: 'task-001'
-    });
-    
-    expect(prompt).toContain('# HONE: REVIEW PHASE');
-    expect(prompt).toContain('# REVIEW CHECKLIST');
-    expect(prompt).toContain('Correctness');
-    expect(prompt).toContain('task-001');
-  });
-  
+      taskId: 'task-001',
+    })
+
+    expect(prompt).toContain('# HONE: REVIEW PHASE')
+    expect(prompt).toContain('# REVIEW CHECKLIST')
+    expect(prompt).toContain('Correctness')
+    expect(prompt).toContain('task-001')
+  })
+
   test('finalize phase includes review feedback', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
-    const feedback = '**Issue**: Missing error handling\n**Suggestion**: Add try-catch';
+    )
+
+    const feedback = '**Issue**: Missing error handling\n**Suggestion**: Add try-catch'
     const prompt = constructPrompt({
       phase: 'finalize',
       featureName: 'test',
       config: mockConfig,
       taskId: 'task-001',
-      reviewFeedback: feedback
-    });
-    
-    expect(prompt).toContain('# HONE: FINALIZE PHASE');
-    expect(prompt).toContain('# REVIEW FEEDBACK');
-    expect(prompt).toContain('Missing error handling');
-  });
-  
+      reviewFeedback: feedback,
+    })
+
+    expect(prompt).toContain('# HONE: FINALIZE PHASE')
+    expect(prompt).toContain('# REVIEW FEEDBACK')
+    expect(prompt).toContain('Missing error handling')
+  })
+
   test('finalize phase without review feedback shows default message', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'finalize',
       featureName: 'test',
       config: mockConfig,
-      taskId: 'task-001'
-    });
-    
-    expect(prompt).toContain('No review feedback provided');
-  });
-  
+      taskId: 'task-001',
+    })
+
+    expect(prompt).toContain('No review feedback provided')
+  })
+
   test('handles missing AGENTS.md gracefully', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).not.toContain('@AGENTS.md');
-    expect(prompt).toContain('# TASK SELECTION');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).not.toContain('@AGENTS.md')
+    expect(prompt).toContain('# TASK SELECTION')
+  })
+
   test('handles missing progress file gracefully', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).not.toContain('progress-test.txt');
-    expect(prompt).toContain('# TASK SELECTION');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).not.toContain('progress-test.txt')
+    expect(prompt).toContain('# TASK SELECTION')
+  })
+
   test('uses default feedback instructions if not configured', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const configWithoutFeedback: HoneConfig = {
       ...mockConfig,
-      feedbackInstructions: undefined
-    };
-    
+      feedbackInstructions: undefined,
+    }
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: configWithoutFeedback
-    });
-    
-    expect(prompt).toContain('test: bun test');
-  });
-  
+      config: configWithoutFeedback,
+    })
+
+    expect(prompt).toContain('test: bun test')
+  })
+
   test('implement phase includes failure handling instructions', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('If tests fail or there are errors you cannot fix');
-    expect(prompt).toContain('DO NOT output TASK_COMPLETED');
-    expect(prompt).toContain('task will remain pending');
-    expect(prompt).toContain('Only output this marker if the task is fully complete');
-  });
-  
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain('If tests fail or there are errors you cannot fix')
+    expect(prompt).toContain('DO NOT output TASK_COMPLETED')
+    expect(prompt).toContain('task will remain pending')
+    expect(prompt).toContain('Only output this marker if the task is fully complete')
+  })
+
   test('implement phase enforces task file isolation', () => {
     writeFileSync(
       join(TEST_PLANS_DIR, 'tasks-test.yml'),
       'tasks:\n  - id: task-001\n    status: pending\n'
-    );
-    
+    )
+
     const prompt = constructPrompt({
       phase: 'implement',
       featureName: 'test',
-      config: mockConfig
-    });
-    
-    expect(prompt).toContain('CRITICAL: You MUST only choose tasks from the task file referenced in CONTEXT FILES');
-    expect(prompt).toContain('Do NOT select tasks from any other task file');
-  });
-});
+      config: mockConfig,
+    })
+
+    expect(prompt).toContain(
+      'CRITICAL: You MUST only choose tasks from the task file referenced in CONTEXT FILES'
+    )
+    expect(prompt).toContain('Do NOT select tasks from any other task file')
+  })
+})

package/src/prompt.ts

@@ -1,15 +1,15 @@
-import { existsSync } from 'fs';
-import { join, relative } from 'path';
-import { getPlansDir, type HoneConfig } from './config';
+import { existsSync } from 'fs'
+import { join, relative } from 'path'
+import { getPlansDir, type HoneConfig } from './config'
 
-export type PromptPhase = 'implement' | 'review' | 'finalize';
+export type PromptPhase = 'implement' | 'review' | 'finalize'
 
 export interface PromptOptions {
-  phase: PromptPhase;
-  featureName: string;
-  config: HoneConfig;
-  taskId?: string;
-  reviewFeedback?: string;
+  phase: PromptPhase
+  featureName: string
+  config: HoneConfig
+  taskId?: string
+  reviewFeedback?: string
 }
 
 /**
@@ -17,49 +17,49 @@ export interface PromptOptions {
  * References files using @<path> syntax instead of embedding content.
  */
 export function constructPrompt(options: PromptOptions): string {
-  const { phase, featureName, config, taskId, reviewFeedback } = options;
-  
-  const parts: string[] = [];
-  
+  const { phase, featureName, config, taskId, reviewFeedback } = options
+
+  const parts: string[] = []
+
   // Build list of file references (relative to project root)
-  const fileRefs: string[] = [];
-  const cwd = process.cwd();
-  
+  const fileRefs: string[] = []
+  const cwd = process.cwd()
+
   // Add task file reference (required)
-  const taskPath = join(getPlansDir(), `tasks-${featureName}.yml`);
+  const taskPath = join(getPlansDir(), `tasks-${featureName}.yml`)
   if (existsSync(taskPath)) {
-    const relPath = relative(cwd, taskPath);
-    fileRefs.push(`@${relPath}`);
+    const relPath = relative(cwd, taskPath)
+    fileRefs.push(`@${relPath}`)
   }
-  
+
   // Add progress file reference if exists
-  const progressPath = join(getPlansDir(), `progress-${featureName}.txt`);
+  const progressPath = join(getPlansDir(), `progress-${featureName}.txt`)
   if (existsSync(progressPath)) {
-    const relPath = relative(cwd, progressPath);
-    fileRefs.push(`@${relPath}`);
+    const relPath = relative(cwd, progressPath)
+    fileRefs.push(`@${relPath}`)
   }
-  
+
   // Add AGENTS.md reference if exists
-  const agentsPath = join(cwd, 'AGENTS.md');
+  const agentsPath = join(cwd, 'AGENTS.md')
   if (existsSync(agentsPath)) {
-    fileRefs.push(`@AGENTS.md`);
+    fileRefs.push(`@AGENTS.md`)
   }
-  
+
   // Add phase-specific header with file references
-  parts.push(getPhaseHeader(phase));
-  parts.push('');
-  
+  parts.push(getPhaseHeader(phase))
+  parts.push('')
+
   if (fileRefs.length > 0) {
-    parts.push('# CONTEXT FILES');
-    parts.push('');
-    parts.push(fileRefs.join(' '));
-    parts.push('');
+    parts.push('# CONTEXT FILES')
+    parts.push('')
+    parts.push(fileRefs.join(' '))
+    parts.push('')
   }
-  
+
   // Add phase-specific instructions
-  parts.push(getPhaseInstructions(phase, config, taskId, reviewFeedback));
-  
-  return parts.join('\n');
+  parts.push(getPhaseInstructions(phase, config, taskId, reviewFeedback))
+
+  return parts.join('\n')
 }
 
 /**
@@ -68,11 +68,11 @@ export function constructPrompt(options: PromptOptions): string {
 function getPhaseHeader(phase: PromptPhase): string {
   switch (phase) {
     case 'implement':
-      return '# HONE: IMPLEMENT PHASE';
+      return '# HONE: IMPLEMENT PHASE'
     case 'review':
-      return '# HONE: REVIEW PHASE';
+      return '# HONE: REVIEW PHASE'
     case 'finalize':
-      return '# HONE: FINALIZE PHASE';
+      return '# HONE: FINALIZE PHASE'
   }
 }
 
@@ -85,16 +85,16 @@ function getPhaseInstructions(
   taskId?: string,
   reviewFeedback?: string
 ): string {
-  const feedbackInstructions = config.feedbackInstructions || 'test: bun test';
-  const lintCommand = config.lintCommand;
-  
+  const feedbackInstructions = config.feedbackInstructions || 'test: bun test'
+  const lintCommand = config.lintCommand
+
   switch (phase) {
     case 'implement':
-      return getImplementInstructions(feedbackInstructions, lintCommand);
+      return getImplementInstructions(feedbackInstructions, lintCommand)
     case 'review':
-      return getReviewInstructions(taskId);
+      return getReviewInstructions(taskId)
     case 'finalize':
-      return getFinalizeInstructions(feedbackInstructions, lintCommand, reviewFeedback);
+      return getFinalizeInstructions(feedbackInstructions, lintCommand, reviewFeedback)
   }
 }
 
@@ -137,12 +137,12 @@ Only run feedback loops AFTER you have fully completed implementing the task.
 
 Run feedback loops ONLY when the task implementation is complete:
 
-${feedbackInstructions}`;
+${feedbackInstructions}`
 
   if (lintCommand) {
-    instructions += `\n${lintCommand}`;
+    instructions += `\n${lintCommand}`
   }
-  
+
   instructions += `
 
 If CLI or script, run them and verify output.
@@ -161,9 +161,9 @@ At the end, output on a single line:
 TASK_COMPLETED: <task-id>
 
 This allows hone to track which task you completed.
-Only output this marker if the task is fully complete and all feedback loops pass.`;
+Only output this marker if the task is fully complete and all feedback loops pass.`
 
-  return instructions;
+  return instructions
 }
 
 /**
@@ -198,7 +198,7 @@ Provide specific, actionable feedback. If everything looks good, say "LGTM" (Loo
 Structure your feedback as:
 - **Issue**: Description of the problem
 - **Suggestion**: How to fix it
-- **Priority**: critical | high | medium | low`;
+- **Priority**: critical | high | medium | low`
 }
 
 /**
@@ -213,7 +213,7 @@ function getFinalizeInstructions(
 
 Finalize the task by applying review feedback and updating all tracking files.
 
-IMPORTANT: This is the ONLY phase where git commits are allowed. If you already committed in 
+IMPORTANT: This is the ONLY phase where git commits are allowed. If you already committed in
 the implement phase, you did it wrong - but continue with finalization anyway.
 
 # REVIEW FEEDBACK
@@ -227,10 +227,10 @@ ${reviewFeedback || 'No review feedback provided (review was skipped or approved
 
 2. **Run Final Feedback Loops** (if needed)
    - If you applied feedback or made any changes:
-   ${feedbackInstructions}`;
+   ${feedbackInstructions}`
 
   if (lintCommand) {
-    instructions += `\n   ${lintCommand}`;
+    instructions += `\n   ${lintCommand}`
   }
 
   instructions += `
@@ -247,23 +247,23 @@ ${reviewFeedback || 'No review feedback provided (review was skipped or approved
      TASK-XXX: <task-title>
      Date: <ISO-8601-datetime>
      ================================================================================
-     
+
      Summary:
      <concise summary of what was done>
-     
+
      Files Changed:
      - file1.ts (created/modified/deleted with brief description)
      - file2.ts (...)
-     
+
      Key Decisions:
      - Decision 1
      - Decision 2
-     
+
      Next Task: <next-task-id> or "All tasks complete"
      \`\`\`
 
 5. **Update AGENTS.md** (if learnings exist)
-   - Add useful learnings and patterns under appropriate heading
+   - Add useful learnings and gotchas under appropriate heading
    - Be terse - only add truly useful info that future agents need
    - Don't duplicate existing info
 
@@ -287,9 +287,7 @@ If you cannot commit (e.g., no changes to commit), that indicates a problem - in
 At the end, output on a single line:
 FINALIZED: <task-id>
 
-Only output this marker AFTER you have successfully created the git commit.`;
+Only output this marker AFTER you have successfully created the git commit.`
 
-  return instructions;
+  return instructions
 }
-
-