@sylphx/flow 1.1.1 → 1.3.0

package/src/core/formatting/bytes.test.ts

@@ -1,115 +0,0 @@
-/**
- * Tests for Byte Formatting Utilities
- */
-
-import { describe, expect, it } from 'bun:test';
-import { formatBytes, formatFileSize } from './bytes.js';
-
-describe('formatBytes', () => {
-  describe('default options (decimals: 2, longUnits)', () => {
-    it('formats 0 bytes', () => {
-      expect(formatBytes(0)).toBe('0 Bytes');
-    });
-
-    it('formats bytes', () => {
-      expect(formatBytes(500)).toBe('500 Bytes');
-    });
-
-    it('formats kilobytes', () => {
-      expect(formatBytes(1024)).toBe('1 KB');
-      expect(formatBytes(1536)).toBe('1.5 KB');
-    });
-
-    it('formats megabytes', () => {
-      expect(formatBytes(1048576)).toBe('1 MB');
-      expect(formatBytes(1572864)).toBe('1.5 MB');
-    });
-
-    it('formats gigabytes', () => {
-      expect(formatBytes(1073741824)).toBe('1 GB');
-      expect(formatBytes(1610612736)).toBe('1.5 GB');
-    });
-
-    it('formats terabytes', () => {
-      expect(formatBytes(1099511627776)).toBe('1 TB');
-    });
-
-    it('rounds to 2 decimal places', () => {
-      expect(formatBytes(1587)).toBe('1.55 KB');
-      expect(formatBytes(1638400)).toBe('1.56 MB');
-    });
-  });
-
-  describe('with decimals option', () => {
-    it('formats with 0 decimals', () => {
-      expect(formatBytes(1536, { decimals: 0 })).toBe('2 KB');
-      expect(formatBytes(1024, { decimals: 0 })).toBe('1 KB');
-    });
-
-    it('formats with 1 decimal', () => {
-      expect(formatBytes(1536, { decimals: 1 })).toBe('1.5 KB');
-      expect(formatBytes(1587, { decimals: 1 })).toBe('1.5 KB');
-    });
-
-    it('formats with 3 decimals', () => {
-      expect(formatBytes(1587, { decimals: 3 })).toBe('1.55 KB'); // toFixed trims to 1.550, then trimmed
-    });
-  });
-
-  describe('with shortUnits option', () => {
-    it('uses short unit for 0 bytes', () => {
-      expect(formatBytes(0, { shortUnits: true })).toBe('0 B');
-    });
-
-    it('uses short unit for bytes', () => {
-      expect(formatBytes(500, { shortUnits: true })).toBe('500 B');
-    });
-
-    it('uses short units for kilobytes', () => {
-      expect(formatBytes(1024, { shortUnits: true })).toBe('1 KB');
-    });
-
-    it('uses short units for megabytes', () => {
-      expect(formatBytes(1048576, { shortUnits: true })).toBe('1 MB');
-    });
-  });
-
-  describe('combined options', () => {
-    it('uses short units with 1 decimal', () => {
-      expect(formatBytes(1536, { decimals: 1, shortUnits: true })).toBe('1.5 KB');
-    });
-
-    it('uses short units with 0 decimals', () => {
-      expect(formatBytes(1536, { decimals: 0, shortUnits: true })).toBe('2 KB');
-    });
-  });
-
-  describe('edge cases', () => {
-    it('handles 1 byte', () => {
-      expect(formatBytes(1)).toBe('1 Bytes');
-    });
-
-    it('handles very large numbers', () => {
-      const result = formatBytes(1099511627776 * 1024); // 1 PB
-      expect(result).toContain('TB'); // Will show in TB since we only go up to TB
-    });
-
-    it('handles fractional KB', () => {
-      expect(formatBytes(1500)).toBe('1.46 KB');
-    });
-  });
-});
-
-describe('formatFileSize', () => {
-  it('formats with 1 decimal and short units', () => {
-    expect(formatFileSize(0)).toBe('0 B');
-    expect(formatFileSize(1024)).toBe('1 KB'); // toFixed(1) gives "1.0", trimmed to "1"
-    expect(formatFileSize(1536)).toBe('1.5 KB');
-    expect(formatFileSize(1048576)).toBe('1 MB'); // toFixed(1) gives "1.0", trimmed to "1"
-  });
-
-  it('is an alias for formatBytes with specific options', () => {
-    const bytes = 1572864;
-    expect(formatFileSize(bytes)).toBe(formatBytes(bytes, { decimals: 1, shortUnits: true }));
-  });
-});

package/src/core/validation/limit.test.ts

@@ -1,155 +0,0 @@
-/**
- * Tests for Limit Validation Utilities
- */
-
-import { describe, expect, it } from 'bun:test';
-import { validateLimit } from './limit.js';
-
-describe('validateLimit', () => {
-  describe('with default parameters (defaultLimit=50, maxLimit=1000)', () => {
-    it('returns default limit when undefined', () => {
-      const result = validateLimit(undefined);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(50);
-    });
-
-    it('accepts valid number', () => {
-      const result = validateLimit(100);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(100);
-    });
-
-    it('accepts valid string number', () => {
-      const result = validateLimit('500');
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(500);
-    });
-
-    it('accepts limit at max', () => {
-      const result = validateLimit(1000);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(1000);
-    });
-
-    it('rejects limit above max', () => {
-      const result = validateLimit(1001);
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe('Limit cannot exceed 1000');
-    });
-
-    it('rejects zero', () => {
-      const result = validateLimit(0);
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe(
-        'Limit must be a positive number'
-      );
-    });
-
-    it('rejects negative number', () => {
-      const result = validateLimit(-10);
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe(
-        'Limit must be a positive number'
-      );
-    });
-
-    it('rejects invalid string', () => {
-      const result = validateLimit('invalid');
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe(
-        'Limit must be a positive number'
-      );
-    });
-
-    it('rejects empty string', () => {
-      const result = validateLimit('');
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe(
-        'Limit must be a positive number'
-      );
-    });
-  });
-
-  describe('with custom default (defaultLimit=10)', () => {
-    it('returns custom default when undefined', () => {
-      const result = validateLimit(undefined, 10);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(10);
-    });
-  });
-
-  describe('with custom max (maxLimit=100)', () => {
-    it('accepts limit at custom max', () => {
-      const result = validateLimit(100, 10, 100);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(100);
-    });
-
-    it('rejects limit above custom max', () => {
-      const result = validateLimit(101, 10, 100);
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe('Limit cannot exceed 100');
-    });
-  });
-
-  describe('knowledge feature defaults (defaultLimit=10, maxLimit=100)', () => {
-    it('returns 10 when undefined', () => {
-      const result = validateLimit(undefined, 10, 100);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(10);
-    });
-
-    it('accepts 50', () => {
-      const result = validateLimit(50, 10, 100);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(50);
-    });
-
-    it('rejects 101', () => {
-      const result = validateLimit(101, 10, 100);
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe('Limit cannot exceed 100');
-    });
-  });
-
-  describe('memory feature defaults (defaultLimit=50, maxLimit=1000)', () => {
-    it('returns 50 when undefined', () => {
-      const result = validateLimit(undefined, 50, 1000);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(50);
-    });
-
-    it('accepts 500', () => {
-      const result = validateLimit(500, 50, 1000);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(500);
-    });
-
-    it('rejects 1001', () => {
-      const result = validateLimit(1001, 50, 1000);
-      expect(result._tag).toBe('Failure');
-      expect(result._tag === 'Failure' && result.error.message).toBe('Limit cannot exceed 1000');
-    });
-  });
-
-  describe('edge cases', () => {
-    it('accepts 1 (minimum valid)', () => {
-      const result = validateLimit(1);
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(1);
-    });
-
-    it('handles string with whitespace', () => {
-      const result = validateLimit('  100  ');
-      expect(result._tag).toBe('Success');
-      expect(result._tag === 'Success' && result.value).toBe(100);
-    });
-
-    it('rejects decimal numbers', () => {
-      const result = validateLimit('10.5');
-      expect(result._tag).toBe('Success');
-      // parseInt truncates to 10
-      expect(result._tag === 'Success' && result.value).toBe(10);
-    });
-  });
-});

package/src/core/validation/query.test.ts

@@ -1,44 +0,0 @@
-/**
- * Tests for Query Validation Utilities
- */
-
-import { describe, expect, it } from 'bun:test';
-import { normalizeQuery } from './query.js';
-
-describe('normalizeQuery', () => {
-  it('trims leading whitespace', () => {
-    expect(normalizeQuery('  hello')).toBe('hello');
-  });
-
-  it('trims trailing whitespace', () => {
-    expect(normalizeQuery('hello  ')).toBe('hello');
-  });
-
-  it('trims both leading and trailing whitespace', () => {
-    expect(normalizeQuery('  hello  ')).toBe('hello');
-  });
-
-  it('preserves internal whitespace', () => {
-    expect(normalizeQuery('hello world')).toBe('hello world');
-  });
-
-  it('handles empty string', () => {
-    expect(normalizeQuery('')).toBe('');
-  });
-
-  it('handles string with only whitespace', () => {
-    expect(normalizeQuery('   ')).toBe('');
-  });
-
-  it('handles tabs and newlines', () => {
-    expect(normalizeQuery('\t\nhello\t\n')).toBe('hello');
-  });
-
-  it('handles already trimmed string', () => {
-    expect(normalizeQuery('hello')).toBe('hello');
-  });
-
-  it('handles multi-line queries', () => {
-    expect(normalizeQuery('  line1\nline2  ')).toBe('line1\nline2');
-  });
-});

package/src/domains/codebase/index.ts

@@ -1,5 +0,0 @@
-/**
- * Codebase domain - Code analysis and search
- */
-
-export { registerCodebaseSearchTool, registerCodebaseTools } from './tools.js';

package/src/domains/codebase/tools.ts

@@ -1,139 +0,0 @@
-/**
- * Codebase tools
- * All tools for working with project source code and files
- */
-
-import type { McpServer } from '@modelcontextprotocol/sdk/server/mcp.js';
-import { z } from 'zod';
-import { getSearchService } from '../../services/search/unified-search-service.js';
-
-/**
- * Register codebase search tool
- */
-export function registerCodebaseSearchTool(server: McpServer): void {
-  server.registerTool(
-    'codebase_search',
-    {
-      description: `Search project source files, documentation, and code. Use this to find implementations, functions, classes, or any code-related content.
-
-**IMPORTANT: Use this tool PROACTIVELY before starting work, not reactively when stuck.**
-
-This tool searches across all codebase files and returns the most relevant matches with content snippets.
-
-When to use this tool (BEFORE starting work):
-- **Before implementation**: Find existing patterns, similar functions, or reusable components
-- **Before refactoring**: Understand current implementation and dependencies
-- **Before adding features**: Check for existing similar functionality or conflicting code
-- **Before debugging**: Search for error messages, function names, or related code
-- **Before writing tests**: Find existing test patterns and test utilities
-
-The search includes:
-- Source code files (.ts, .js, .tsx, .jsx, etc.)
-- Configuration files (.json, .yaml, .toml, etc.)
-- Documentation files (.md, .txt, etc.)
-- Build and deployment files
-
-**Best Practice**: Search the codebase BEFORE writing new code to avoid duplication and follow existing patterns.`,
-      inputSchema: {
-        query: z
-          .string()
-          .describe('Search query - use natural language, function names, or technical terms'),
-        limit: z
-          .number()
-          .default(10)
-          .optional()
-          .describe('Maximum number of results to return (default: 10)'),
-        include_content: z
-          .boolean()
-          .default(true)
-          .optional()
-          .describe('Include file content snippets in results (default: true)'),
-        file_extensions: z
-          .array(z.string())
-          .optional()
-          .describe('Filter by file extensions (e.g., [".ts", ".tsx", ".js"])'),
-        path_filter: z
-          .string()
-          .optional()
-          .describe('Filter by path pattern (e.g., "src/components", "tests", "docs")'),
-        exclude_paths: z
-          .array(z.string())
-          .optional()
-          .describe(
-            'Exclude paths containing these patterns (e.g., ["node_modules", ".git", "dist"])'
-          ),
-      },
-    },
-    async ({
-      query,
-      limit = 10,
-      include_content = true,
-      file_extensions,
-      path_filter,
-      exclude_paths,
-    }) => {
-      try {
-        // Use UnifiedSearchService - same logic as CLI
-        const searchService = getSearchService();
-        await searchService.initialize();
-
-        // Check codebase status
-        const status = await searchService.getStatus();
-
-        // If indexing in progress, show progress
-        if (status.codebase.isIndexing) {
-          const progressBar =
-            '█'.repeat(Math.floor(status.codebase.progress / 5)) +
-            '░'.repeat(20 - Math.floor(status.codebase.progress / 5));
-          return {
-            content: [
-              {
-                type: 'text',
-                text: `⏳ **Codebase Indexing In Progress**\n\nThe codebase is currently being indexed. Please wait...\n\n**Progress:** ${status.codebase.progress}%\n\`${progressBar}\`\n\n**Status:**\n- Files indexed: ${status.codebase.progress > 0 ? Math.floor((status.codebase.fileCount * status.codebase.progress) / 100) : 0}/${status.codebase.fileCount}\n${status.codebase.currentFile ? `- Current file: \`${status.codebase.currentFile}\`` : ''}\n\n**Estimated time:** ${status.codebase.progress > 0 ? 'Less than 1 minute' : 'Starting...'}\n\n💡 **Tip:** Try your search again in a few seconds.`,
-              },
-            ],
-          };
-        }
-
-        if (!status.codebase.indexed) {
-          return {
-            content: [
-              {
-                type: 'text',
-                text: `📭 **Codebase Not Indexed**\n\nThe codebase has not been indexed yet.\n\n**To fix:**\n- Run: \`sylphx codebase reindex\` from the command line\n- This will create a search index for all source files\n\n**Why this is needed:**\nThe first time you use codebase search, you need to build an index of all files. This only needs to be done once (or when files change significantly).`,
-              },
-            ],
-          };
-        }
-
-        // Perform search using unified service
-        const result = await searchService.searchCodebase(query, {
-          limit,
-          include_content,
-          file_extensions,
-          path_filter,
-          exclude_paths,
-        });
-
-        // Return MCP-formatted results
-        return searchService.formatResultsForMCP(result.results, query, result.totalIndexed);
-      } catch (error) {
-        return {
-          content: [
-            {
-              type: 'text',
-              text: `✗ Codebase search error: ${(error as Error).message}`,
-            },
-          ],
-        };
-      }
-    }
-  );
-}
-
-/**
- * Register all codebase tools
- */
-export function registerCodebaseTools(server: McpServer): void {
-  registerCodebaseSearchTool(server);
-}

package/src/domains/knowledge/index.ts

@@ -1,10 +0,0 @@
-/**
- * Knowledge domain - Documentation and guides
- */
-
-export { getKnowledgeContent } from './resources.js';
-export {
-  registerKnowledgeGetTool,
-  registerKnowledgeSearchTool,
-  registerKnowledgeTools,
-} from './tools.js';