hone-ai 0.2.0 → 0.9.0

package/src/prd-generator.integration.test.ts

@@ -1,91 +1,91 @@
-import { describe, test, expect, beforeAll, afterAll } from 'bun:test';
-import { existsSync, rmSync, mkdirSync } from 'fs';
-import { readFile } from 'fs/promises';
-import { join } from 'path';
-import { slugify } from './prd-generator';
+import { describe, test, expect, beforeAll, afterAll } from 'bun:test'
+import { existsSync, rmSync, mkdirSync } from 'fs'
+import { readFile } from 'fs/promises'
+import { join } from 'path'
+import { slugify } from './prd-generator'
 
 describe('PRD Generator Integration', () => {
-  const testPlansDir = join(process.cwd(), '.plans-test');
-  const originalCwd = process.cwd();
+  const testPlansDir = join(process.cwd(), '.plans-test')
+  const originalCwd = process.cwd()
 
   beforeAll(() => {
     // Create test .plans directory
     if (existsSync(testPlansDir)) {
-      rmSync(testPlansDir, { recursive: true });
+      rmSync(testPlansDir, { recursive: true })
     }
-    mkdirSync(testPlansDir, { recursive: true });
-  });
+    mkdirSync(testPlansDir, { recursive: true })
+  })
 
   afterAll(() => {
     // Cleanup
     if (existsSync(testPlansDir)) {
-      rmSync(testPlansDir, { recursive: true });
+      rmSync(testPlansDir, { recursive: true })
     }
-  });
+  })
 
   describe('slugify', () => {
     test('converts to lowercase', () => {
-      expect(slugify('Email Validation')).toBe('email-validation');
-    });
+      expect(slugify('Email Validation')).toBe('email-validation')
+    })
 
     test('replaces spaces with hyphens', () => {
-      expect(slugify('user profile page')).toBe('user-profile-page');
-    });
+      expect(slugify('user profile page')).toBe('user-profile-page')
+    })
 
     test('removes special characters', () => {
-      expect(slugify('user@email.com validation!')).toBe('useremailcom-validation');
-    });
+      expect(slugify('user@email.com validation!')).toBe('useremailcom-validation')
+    })
 
     test('trims leading/trailing spaces', () => {
-      expect(slugify('  feature name  ')).toBe('feature-name');
-    });
+      expect(slugify('  feature name  ')).toBe('feature-name')
+    })
 
     test('removes leading/trailing hyphens', () => {
-      expect(slugify('---feature-name---')).toBe('feature-name');
-    });
+      expect(slugify('---feature-name---')).toBe('feature-name')
+    })
 
     test('collapses multiple hyphens', () => {
-      expect(slugify('feature--name')).toBe('feature-name');
-      expect(slugify('feature---name')).toBe('feature-name');
-    });
+      expect(slugify('feature--name')).toBe('feature-name')
+      expect(slugify('feature---name')).toBe('feature-name')
+    })
 
     test('truncates to 50 characters', () => {
-      const longName = 'this-is-a-very-long-feature-name-that-exceeds-fifty-characters';
-      const result = slugify(longName);
-      expect(result.length).toBeLessThanOrEqual(50);
-      expect(result.length).toBe(50);
-      expect(result).toBe('this-is-a-very-long-feature-name-that-exceeds-fift');
-    });
+      const longName = 'this-is-a-very-long-feature-name-that-exceeds-fifty-characters'
+      const result = slugify(longName)
+      expect(result.length).toBeLessThanOrEqual(50)
+      expect(result.length).toBe(50)
+      expect(result).toBe('this-is-a-very-long-feature-name-that-exceeds-fift')
+    })
 
     test('handles empty string', () => {
-      expect(slugify('')).toBe('');
-    });
+      expect(slugify('')).toBe('')
+    })
 
     test('handles string with only special characters', () => {
-      expect(slugify('!@#$%^&*()')).toBe('');
-    });
+      expect(slugify('!@#$%^&*()')).toBe('')
+    })
 
     test('handles mixed case with numbers', () => {
-      expect(slugify('Feature123Test')).toBe('feature123test');
-    });
+      expect(slugify('Feature123Test')).toBe('feature123test')
+    })
 
     test('preserves existing hyphens', () => {
-      expect(slugify('my-feature-name')).toBe('my-feature-name');
-    });
+      expect(slugify('my-feature-name')).toBe('my-feature-name')
+    })
 
     test('handles unicode characters', () => {
-      expect(slugify('feature café')).toBe('feature-caf');
-    });
-  });
+      expect(slugify('feature café')).toBe('feature-caf')
+    })
+  })
 
   describe('Model configuration', () => {
     test('uses correct model name format', async () => {
-      const { loadConfig } = await import('./config');
-      const config = await loadConfig();
-      
+      const { loadConfig } = await import('./config')
+      const config = await loadConfig()
+
       // Model name should match Anthropic API format
-      expect(config.models.claude).toMatch(/^claude-sonnet-4-\d{8}$/);
-      expect(config.models.opencode).toMatch(/^claude-sonnet-4-\d{8}$/);
-    });
-  });
-});
+      expect(config.models.claude).toMatch(/^claude-sonnet-4-\d{8}$/)
+      expect(config.models.opencode).toMatch(/^claude-sonnet-4-\d{8}$/)
+    })
+  })
+})

package/src/prd-generator.test.ts

@@ -1,34 +1,75 @@
-import { describe, it, expect } from 'bun:test';
-import { slugify } from './prd-generator';
+import { describe, it, expect } from 'bun:test'
+import { slugify } from './prd-generator'
+import { readFile } from 'fs/promises'
+import { join } from 'path'
 
 describe('slugify', () => {
   it('should convert to lowercase', () => {
-    expect(slugify('My Feature')).toBe('my-feature');
-  });
-  
+    expect(slugify('My Feature')).toBe('my-feature')
+  })
+
   it('should replace spaces with hyphens', () => {
-    expect(slugify('user authentication system')).toBe('user-authentication-system');
-  });
-  
+    expect(slugify('user authentication system')).toBe('user-authentication-system')
+  })
+
   it('should remove special characters', () => {
-    expect(slugify('Feature (with) special! chars@')).toBe('feature-with-special-chars');
-  });
-  
+    expect(slugify('Feature (with) special! chars@')).toBe('feature-with-special-chars')
+  })
+
   it('should collapse multiple hyphens', () => {
-    expect(slugify('feature   with   spaces')).toBe('feature-with-spaces');
-  });
-  
+    expect(slugify('feature   with   spaces')).toBe('feature-with-spaces')
+  })
+
   it('should trim to 50 characters', () => {
-    const longString = 'this is a very long feature name that exceeds fifty characters';
-    const result = slugify(longString);
-    expect(result.length).toBeLessThanOrEqual(50);
-  });
-  
+    const longString = 'this is a very long feature name that exceeds fifty characters'
+    const result = slugify(longString)
+    expect(result.length).toBeLessThanOrEqual(50)
+  })
+
   it('should handle empty string', () => {
-    expect(slugify('')).toBe('');
-  });
-  
+    expect(slugify('')).toBe('')
+  })
+
   it('should trim leading and trailing hyphens', () => {
-    expect(slugify('  feature  ')).toBe('feature');
-  });
-});
+    expect(slugify('  feature  ')).toBe('feature')
+  })
+})
+
+describe('PRD system prompts', () => {
+  it('should include file and URL processing instructions', async () => {
+    const sourceFile = await readFile(join(__dirname, 'prd-generator.ts'), 'utf-8')
+
+    // Verify clarifying questions prompt includes file/URL instructions
+    expect(sourceFile).toContain('FILE AND URL PROCESSING:')
+    expect(sourceFile).toContain(
+      'automatically read and incorporate their content using the Read tool'
+    )
+    expect(sourceFile).toContain(
+      'automatically fetch and incorporate their content using the WebFetch tool'
+    )
+
+    // Verify both prompts contain the instructions
+    const fileUrlInstructions = sourceFile.match(
+      /FILE AND URL PROCESSING:(.*?)(?=\n\n|\nRules:|\nGenerate)/gs
+    )
+    expect(fileUrlInstructions).toBeTruthy()
+    expect(fileUrlInstructions?.length).toBe(2) // One in clarifying questions, one in PRD generation
+  })
+
+  it('should include reference access failure handling instructions in clarifying questions', async () => {
+    const sourceFile = await readFile(join(__dirname, 'prd-generator.ts'), 'utf-8')
+
+    // Verify specific reference failure handling instructions
+    expect(sourceFile).toContain('REFERENCE ACCESS FAILURES: When file reads or URL fetches fail:')
+    expect(sourceFile).toContain('Continue with question generation using available context')
+    expect(sourceFile).toContain(
+      "Ask clarifying questions about the inaccessible reference's intended purpose or relevance"
+    )
+    expect(sourceFile).toContain(
+      "I couldn't access [file/URL]. What was its relevance to this feature?"
+    )
+    expect(sourceFile).toContain(
+      'Prioritize these clarification questions early in the Q&A session when references are critical'
+    )
+  })
+})