docs: add v0.8.5 release notes and update changeset

- Add v0.8.5 changelog entry to website
- Update latest-version.ts with new release info
- Update tools index with v0.8.5 marker
- Update changeset to use patch bumps for all packages
- Fix all v0.9.0 references to v0.8.5
- Document pattern analysis features and performance improvements

Author: prosdev

.changeset/dev-explore-to-dev-inspect.md

@@ -1,44 +1,87 @@
 ---
+"@lytics/dev-agent-core": patch
 "@lytics/dev-agent-mcp": patch
 "@lytics/dev-agent": patch
 "@lytics/dev-agent-cli": patch
 ---
 
-Refactor: Rename dev_explore → dev_inspect with focused actions
+feat(mcp): refactor dev_inspect and optimize pattern analysis
 
-**BREAKING CHANGES:**
+**API Simplification:**
 
-- `dev_explore` renamed to `dev_inspect`
-- Actions changed from `['pattern', 'similar', 'relationships']` to `['compare', 'validate']`
-- Removed `action: "pattern"` → Use `dev_search` instead
-- Removed `action: "relationships"` → Use `dev_refs` instead  
-- Renamed `action: "similar"` → `action: "compare"`
+- `dev_inspect` simplified to single-purpose tool (action parameter streamlined)
+- Previously: `dev_inspect({ action: "compare", query: "file.ts" })`
+- Now: `dev_inspect({ query: "file.ts" })`
+- Existing usage continues to work with dynamic MCP schema discovery
 
-**What's New:**
+**Major Features:**
 
-- `dev_inspect` with `action: "compare"` finds similar code implementations
-- `dev_inspect` with `action: "validate"` checks pattern consistency (placeholder for future)
-- Clearer tool boundaries: search vs. inspect vs. refs
-- File-focused analysis (always takes file path, not search query)
+- Created `PatternAnalysisService` with 5 pattern extractors:
+  - Import style (ESM, CJS, mixed, unknown)
+  - Error handling (throw, result, callback, unknown)
+  - Type coverage (full, partial, none)
+  - Testing (co-located test files)
+  - File size (lines vs similar files)
+- Batch scanning optimization (5-10x faster: 500-1000ms vs 2-3 seconds)
+- Embedding-based similarity search (no more false matches)
+- Extension filtering (`.ts` only compares with `.ts`)
+- Comprehensive pattern analysis (finds similar files + analyzes patterns)
+
+**Performance:**
+
+- One ts-morph initialization vs 6 separate scans
+- Batch scan all files in one pass
+- `searchByDocumentId()` for embedding-based similarity
+- Pattern analysis: 500-1000ms (down from 2-3 seconds)
+
+**Bug Fixes:**
+
+- Fixed `findSimilar` to use document embeddings instead of file paths
+- Fixed `--force` flag to properly clear old vector data
+- Fixed race condition in LanceDB table creation
+- Removed `outputSchema` from all 9 MCP adapters (Cursor/Claude compatibility)
+
+**New Features:**
+
+- Test utilities in `@lytics/dev-agent-core/utils`:
+  - `isTestFile()` — Check if file is a test file
+  - `findTestFile()` — Find co-located test files
+- Vector store `clear()` method
+- Vector store `searchByDocumentId()` method
+- Comprehensive pattern comparison with statistical analysis
 
 **Migration Guide:**
 
 ```typescript
-// Before
-dev_explore { action: "similar", query: "src/auth.ts" }
-dev_explore { action: "pattern", query: "error handling" }
-dev_explore { action: "relationships", query: "src/auth.ts" }
-
-// After
-dev_inspect { action: "compare", query: "src/auth.ts" }
-dev_search { query: "error handling" }
-dev_refs { name: "authenticateUser" }
+// Before (v0.8.4)
+dev_inspect({ action: "compare", query: "src/auth.ts" })
+dev_inspect({ action: "validate", query: "src/auth.ts" })
+
+// After (v0.8.5) - Streamlined!
+dev_inspect({ query: "src/auth.ts" })
 ```
 
-**Why:**
+The tool now automatically finds similar files AND performs pattern analysis. No migration needed - MCP tools discover the new schema dynamically.
+
+**Re-index Recommended:**
+
+```bash
+dev index . --force
+```
+
+This clears old data and rebuilds with improved embedding-based search.
+
+**Documentation:**
+
+- Complete rewrite of dev-inspect.mdx
+- Updated README.md with pattern categories
+- Updated CLAUDE.md with new descriptions
+- Added v0.8.5 changelog entry to website
+- Migration guide from dev_explore
 
-- Eliminate tool duplication (`pattern` duplicated `dev_search`)
-- Clear single responsibility (file analysis only)
-- Better naming (`inspect` = deep file examination)
-- Reserve `dev_explore` for future external context (standards, examples, docs)
+**Tests:**
 
+- All 1100+ tests passing
+- Added 10 new test-utils tests
+- Pattern analysis service fully tested
+- Integration tests for InspectAdapter

packages/cli/tsconfig.json

@@ -15,11 +15,5 @@
     { "path": "../logger" }
   ],
   "include": ["src/**/*"],
-  "exclude": [
-    "node_modules",
-    "dist",
-    "**/*.test.ts",
-    "**/*.spec.ts",
-    "**/__tests__/**"
-  ]
+  "exclude": ["node_modules", "dist", "**/*.test.ts", "**/*.spec.ts", "**/__tests__/**"]
 }

packages/core/src/services/__fixtures__/modern-typescript.ts

@@ -8,7 +8,9 @@
  * - Explicit return types
  */
 
-import { v4 as uuidv4 } from 'uuid';
+// Mock uuid for fixture purposes (not a real dependency)
+const uuidv4 = () => '00000000-0000-0000-0000-000000000000';
+
 import type { User, ValidationError } from './types';
 
 export type Result<T, E = Error> = { ok: true; value: T } | { ok: false; error: E };

packages/core/src/services/__tests__/search-service.test.ts

@@ -139,10 +139,8 @@ describe('SearchService', () => {
 
       const mockIndexer: RepositoryIndexer = {
         initialize: vi.fn().mockResolvedValue(undefined),
-        search: vi
-          .fn()
-          .mockResolvedValueOnce([targetFile]) // First call to find the file
-          .mockResolvedValueOnce(similarResults), // Second call to find similar
+        getAll: vi.fn().mockResolvedValue([targetFile, ...similarResults]),
+        searchByDocumentId: vi.fn().mockResolvedValue([targetFile, ...similarResults]),
         close: vi.fn().mockResolvedValue(undefined),
       } as unknown as RepositoryIndexer;
 
@@ -154,15 +152,16 @@ describe('SearchService', () => {
         threshold: 0.8,
       });
 
-      expect(mockIndexer.search).toHaveBeenCalledTimes(2);
+      expect(mockIndexer.getAll).toHaveBeenCalledOnce();
+      expect(mockIndexer.searchByDocumentId).toHaveBeenCalledOnce();
       expect(results).toHaveLength(2); // Should exclude the original file
       expect(results.find((r) => r.metadata.path === 'src/payments/process.ts')).toBeUndefined();
     });
 
     it('should return empty array when file not found', async () => {
       const mockIndexer: RepositoryIndexer = {
         initialize: vi.fn().mockResolvedValue(undefined),
-        search: vi.fn().mockResolvedValue([]), // File not found
+        getAll: vi.fn().mockResolvedValue([]), // File not found
         close: vi.fn().mockResolvedValue(undefined),
       } as unknown as RepositoryIndexer;
 

packages/core/tsconfig.json

@@ -17,8 +17,5 @@
     "**/__tests__/**",
     "**/__fixtures__/**"
   ],
-  "references": [
-    { "path": "../logger" },
-    { "path": "../types" }
-  ]
+  "references": [{ "path": "../logger" }, { "path": "../types" }]
 }

packages/mcp-server/src/adapters/__tests__/github-adapter.test.ts

@@ -5,7 +5,6 @@
 import type { GitHubService } from '@lytics/dev-agent-core';
 import type { GitHubDocument, GitHubSearchResult } from '@lytics/dev-agent-subagents';
 import { beforeEach, describe, expect, it, vi } from 'vitest';
-import type { GitHubOutput } from '../../schemas/index.js';
 import { GitHubAdapter } from '../built-in/github-adapter';
 import type { ToolExecutionContext } from '../types';
 
@@ -186,9 +185,9 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      expect((result.data as GitHubOutput)?.content).toContain('GitHub Search Results');
-      expect((result.data as GitHubOutput)?.content).toContain('#1');
-      expect((result.data as GitHubOutput)?.content).toContain('Test Issue');
+      expect(result.data).toContain('GitHub Search Results');
+      expect(result.data).toContain('#1');
+      expect(result.data).toContain('Test Issue');
     });
 
     it('should search with filters', async () => {
@@ -236,7 +235,7 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      expect((result.data as GitHubOutput)?.content).toContain('No matching issues or PRs found');
+      expect(result.data).toContain('No matching issues or PRs found');
     });
 
     it('should include token footer in search results', async () => {
@@ -260,7 +259,7 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      const content = (result.data as GitHubOutput)?.content;
+      const content = result.data;
       expect(content).toBeDefined();
       // Token info is now in metadata, not content
       expect(result.metadata).toHaveProperty('tokens');
@@ -283,9 +282,9 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      expect((result.data as GitHubOutput)?.content).toContain('Issue #1');
-      expect((result.data as GitHubOutput)?.content).toContain('Test Issue');
-      expect((result.data as GitHubOutput)?.content).toContain('testuser');
+      expect(result.data).toContain('Issue #1');
+      expect(result.data).toContain('Test Issue');
+      expect(result.data).toContain('testuser');
     });
 
     it('should get issue context in verbose format', async () => {
@@ -302,10 +301,10 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      expect((result.data as GitHubOutput)?.content).toContain('**Related Issues:** #2, #3');
-      expect((result.data as GitHubOutput)?.content).toContain('**Related PRs:** #10');
-      expect((result.data as GitHubOutput)?.content).toContain('**Linked Files:** `src/test.ts`');
-      expect((result.data as GitHubOutput)?.content).toContain('**Mentions:** @developer1');
+      expect(result.data).toContain('**Related Issues:** #2, #3');
+      expect(result.data).toContain('**Related PRs:** #10');
+      expect(result.data).toContain('**Linked Files:** `src/test.ts`');
+      expect(result.data).toContain('**Mentions:** @developer1');
     });
 
     it('should handle issue not found', async () => {
@@ -357,9 +356,9 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      expect((result.data as GitHubOutput)?.content).toContain('Related Issues/PRs');
-      expect((result.data as GitHubOutput)?.content).toContain('#2');
-      expect((result.data as GitHubOutput)?.content).toContain('Related Issue');
+      expect(result.data).toContain('Related Issues/PRs');
+      expect(result.data).toContain('#2');
+      expect(result.data).toContain('Related Issue');
     });
 
     it('should handle no related items', async () => {
@@ -378,7 +377,7 @@ describe('GitHubAdapter', () => {
       );
 
       expect(result.success).toBe(true);
-      expect((result.data as GitHubOutput)?.content).toContain('No related issues or PRs found');
+      expect(result.data).toContain('No related issues or PRs found');
     });
   });
 
@@ -411,12 +410,11 @@ describe('GitHubAdapter', () => {
 
       expect(result.success).toBe(true);
       if (result.success) {
-        const output = result.data as GitHubOutput;
-        expect(output.content).toContain('Related Issue 1');
-        expect(output.content).toContain('Related Issue 2');
-        expect(output.content).toContain('90% similar'); // Score shown as percentage
-        expect(output.resultsTotal).toBe(2);
-        expect(output.resultsReturned).toBe(2);
+        expect(result.data).toContain('Related Issue 1');
+        expect(result.data).toContain('Related Issue 2');
+        expect(result.data).toContain('90% similar'); // Score shown as percentage
+        expect(result.metadata?.results_total).toBe(2);
+        expect(result.metadata?.results_returned).toBe(2);
       }
 
       expect(mockGitHubService.getContext).toHaveBeenCalledWith(1);
@@ -437,8 +435,7 @@ describe('GitHubAdapter', () => {
 
       expect(result.success).toBe(true);
       if (result.success) {
-        const output = result.data as GitHubOutput;
-        expect(output.content).toContain('No related issues or PRs found');
+        expect(result.data).toContain('No related issues or PRs found');
       }
     });
   });