octocode-cli 1.2.7 → 1.2.9

package/skills/octocode-research/src/types/guards.ts

@@ -5,34 +5,6 @@
  * @module types/guards
  */
 
-/**
- * Check if value is a non-empty string
- */
-export function isNonEmptyString(value: unknown): value is string {
-  return typeof value === 'string' && value.length > 0;
-}
-
-/**
- * Check if value is a positive finite number
- */
-export function isPositiveNumber(value: unknown): value is number {
-  return typeof value === 'number' && value > 0 && Number.isFinite(value);
-}
-
-/**
- * Check if value is a non-negative finite number
- */
-export function isNonNegativeNumber(value: unknown): value is number {
-  return typeof value === 'number' && value >= 0 && Number.isFinite(value);
-}
-
-/**
- * Check if value is an array of non-empty strings
- */
-export function isStringArray(value: unknown): value is string[] {
-  return Array.isArray(value) && value.every(isNonEmptyString);
-}
-
 /**
  * Check if value is an array (of any type)
  */

package/skills/octocode-research/src/types/mcp.ts

@@ -1,18 +1,8 @@
 /**
- * MCP Protocol Types for /tools/list and /prompts/list endpoints
- * Based on: https://modelcontextprotocol.io/docs/concepts/tools
+ * MCP Protocol utilities for /tools endpoints
  */
 
-// JSON Schema type for tool input schemas
-export interface JsonSchema {
-  $schema?: string;
-  type: 'object';
-  properties: Record<string, JsonSchemaProperty>;
-  required?: string[];
-  additionalProperties?: boolean;
-}
-
-export interface JsonSchemaProperty {
+interface JsonSchemaProperty {
   type: string;
   description?: string;
   items?: JsonSchemaProperty;
@@ -28,60 +18,17 @@ export interface JsonSchemaProperty {
   default?: unknown;
 }
 
-/**
- * MCP Tool representation for /tools/list
- */
-interface McpTool {
-  name: string;
-  description: string;
-  inputSchema: JsonSchema;
-}
-
-/**
- * MCP Prompt argument for /prompts/list
- */
-export interface McpPromptArgument {
-  name: string;
-  description: string;
-  required?: boolean;
-}
-
-/**
- * MCP Prompt representation for /prompts/list
- */
-export interface McpPrompt {
-  name: string;
-  description: string;
-  arguments?: McpPromptArgument[];
-}
-
-/**
- * Response metadata
- */
-export interface ListResponseMeta {
-  totalCount: number;
-  version: string;
-}
-
-/**
- * Response type for GET /tools/list
- */
-export interface ListToolsResponse {
-  tools: McpTool[];
-  _meta?: ListResponseMeta;
-}
-
-/**
- * Response type for GET /prompts/list
- */
-export interface ListPromptsResponse {
-  prompts: McpPrompt[];
-  _meta?: ListResponseMeta;
+interface JsonSchema {
+  $schema?: string;
+  type: 'object';
+  properties: Record<string, JsonSchemaProperty>;
+  required?: string[];
+  additionalProperties?: boolean;
 }
 
 /**
- * Transform description-based schema to JSON Schema format
- * This is a fallback transformer when full JSON schemas are not available
+ * Transform description-based schema to JSON Schema format.
+ * Fallback transformer when full JSON schemas are not available.
  */
 export function transformToJsonSchema(
   schemaDescriptions: Record<string, string>,
@@ -89,15 +36,13 @@ export function transformToJsonSchema(
 ): JsonSchema {
   const properties: Record<string, JsonSchemaProperty> = {};
   
-  // Create properties from descriptions
   for (const [key, description] of Object.entries(schemaDescriptions)) {
     properties[key] = {
-      type: 'string', // Default type - could be enhanced with type inference
+      type: 'string',
       description,
     };
   }
 
-  // Wrap in queries array structure (standard MCP bulk query pattern)
   return {
     $schema: 'http://json-schema.org/draft-07/schema#',
     type: 'object',

package/skills/octocode-research/src/types/responses.ts

@@ -1,6 +1,5 @@
 /**
  * Response types for MCP tool results.
- * Use these instead of Record<string, unknown> for type-safe data handling.
  *
  * @module types/responses
  */
@@ -11,20 +10,6 @@ import { hasProperty, hasArrayProperty } from './guards.js';
 // File and Search Types
 // =============================================================================
 
-/**
- * A match within a file
- */
-export interface MatchLocation {
-  line: number;
-  column?: number;
-  value?: string;
-  byteOffset?: number;
-  charOffset?: number;
-}
-
-/**
- * A file with matches from search results
- */
 export interface FileMatch {
   path: string;
   line?: number;
@@ -33,12 +18,15 @@ export interface FileMatch {
   matchCount?: number;
   contextBefore?: string[];
   contextAfter?: string[];
-  allMatches?: MatchLocation[];
+  allMatches?: Array<{
+    line: number;
+    column?: number;
+    value?: string;
+    byteOffset?: number;
+    charOffset?: number;
+  }>;
 }
 
-/**
- * Pagination info for paginated results
- */
 export interface PaginationInfo {
   page?: number;
   currentPage?: number;
@@ -49,83 +37,10 @@ export interface PaginationInfo {
   nextCursor?: string;
 }
 
-/**
- * Search result from localSearchCode or githubSearchCode
- */
-interface SearchResult {
-  files?: FileMatch[];
-  totalMatches?: number;
-  totalFiles?: number;
-  pagination?: PaginationInfo;
-}
-
-// =============================================================================
-// LSP Types
-// =============================================================================
-
-/**
- * LSP position
- */
-interface LspPosition {
-  line: number;
-  character: number;
-}
-
-/**
- * LSP range
- */
-interface LspRange {
-  start: LspPosition;
-  end: LspPosition;
-}
-
-/**
- * LSP definition location
- */
-interface LspDefinition {
-  uri: string;
-  range: LspRange;
-}
-
-/**
- * LSP reference location
- */
-export interface LspReference extends LspDefinition {
-  context?: string;
-}
-
-/**
- * LSP call hierarchy item
- */
-export interface LspCallHierarchyItem {
-  name: string;
-  kind?: number;
-  uri: string;
-  range?: LspRange;
-  selectionRange?: LspRange;
-}
-
-// =============================================================================
-// Repository Structure Types
-// =============================================================================
-
-/**
- * Repository structure result
- */
-export interface RepoStructure {
-  files?: string[];
-  folders?: string[];
-  totalFiles?: number;
-  totalDirectories?: number;
-}
-
 // =============================================================================
-// Type Guards
+// Internal helpers
 // =============================================================================
 
-/**
- * Check if an object is a FileMatch
- */
 function isFileMatch(obj: unknown): obj is FileMatch {
   return (
     typeof obj === 'object' &&
@@ -135,59 +50,26 @@ function isFileMatch(obj: unknown): obj is FileMatch {
   );
 }
 
-/**
- * Check if an object has valid pagination info
- */
 function hasValidPagination(obj: unknown): obj is { pagination: PaginationInfo } {
   if (!hasProperty(obj, 'pagination')) return false;
   const p = obj.pagination;
   return typeof p === 'object' && p !== null;
 }
 
-/**
- * Check if an object is a SearchResult
- */
-export function isSearchResult(obj: unknown): obj is SearchResult {
-  if (typeof obj !== 'object' || obj === null) return false;
-  if (hasArrayProperty(obj, 'files')) {
-    const files = (obj as { files: unknown[] }).files;
-    return files.every(isFileMatch);
-  }
-  return hasProperty(obj, 'totalMatches') || hasProperty(obj, 'totalFiles');
-}
-
-/**
- * Check if an object is an LspDefinition
- */
-export function isLspDefinition(obj: unknown): obj is LspDefinition {
-  return (
-    typeof obj === 'object' &&
-    obj !== null &&
-    'uri' in obj &&
-    'range' in obj &&
-    typeof (obj as LspDefinition).uri === 'string'
-  );
-}
+// =============================================================================
+// Public extractors
+// =============================================================================
 
-/**
- * Extract files array from data safely
- */
 export function extractFiles(data: unknown): FileMatch[] {
   if (!hasArrayProperty(data, 'files')) return [];
   return (data as { files: unknown[] }).files.filter(isFileMatch);
 }
 
-/**
- * Extract pagination from data safely
- */
 export function extractPagination(data: unknown): PaginationInfo | undefined {
   if (!hasValidPagination(data)) return undefined;
   return data.pagination;
 }
 
-/**
- * Extract total matches from data safely
- */
 export function extractTotalMatches(data: unknown): number {
   if (hasProperty(data, 'totalMatches') && typeof data.totalMatches === 'number') {
     return data.totalMatches;

package/skills/octocode-research/src/utils/circuitBreaker.ts

@@ -458,21 +458,6 @@ startPeriodicCleanup();
 // Manual Cleanup Functions
 // =============================================================================
 
-/**
- * Clear a specific circuit breaker.
- * Use for testing or when a service is decommissioned.
- * @param name - Circuit breaker name to clear
- * @returns true if circuit existed and was cleared
- */
-export function clearCircuit(name: string): boolean {
-  const existed = circuits.delete(name);
-  configs.delete(name);
-  if (existed) {
-    console.log(agentLog(`🧹 Circuit ${name} cleared`));
-  }
-  return existed;
-}
-
 /**
  * Clear all circuit breakers.
  * Use for testing cleanup or server shutdown.
@@ -484,9 +469,3 @@ export function clearAllCircuits(): void {
   console.log(agentLog(`🧹 Cleared ${count} circuit(s)`));
 }
 
-/**
- * Get the number of active circuits (for monitoring).
- */
-export function getCircuitCount(): number {
-  return circuits.size;
-}

package/skills/octocode-research/src/utils/logger.ts

@@ -259,7 +259,7 @@ export function logWarn(message: string, data?: unknown): void {
 // Public API - Tools Logger
 // ============================================================================
 
-export interface ToolLogEntry {
+interface ToolLogEntry {
   tool: string;
   route: string;
   method: string;
@@ -271,58 +271,11 @@ export interface ToolLogEntry {
   requestId?: string;
 }
 
-/**
- * Log a tool invocation (async, non-blocking).
- */
 export function logToolCall(entry: ToolLogEntry): void {
   const logEntry = formatLogEntry('TOOL', `${entry.method} ${entry.route}`, entry);
   writeLogAsync(TOOLS_LOG, logEntry);
 }
 
-/**
- * Log a successful tool result.
- */
-export function logToolSuccess(
-  tool: string,
-  route: string,
-  method: string,
-  params: Record<string, unknown>,
-  duration: number,
-  resultSize: number
-): void {
-  logToolCall({
-    tool,
-    route,
-    method,
-    params,
-    duration,
-    success: true,
-    resultSize,
-  });
-}
-
-/**
- * Log a failed tool invocation.
- */
-export function logToolError(
-  tool: string,
-  route: string,
-  method: string,
-  params: Record<string, unknown>,
-  duration: number,
-  error: string
-): void {
-  logToolCall({
-    tool,
-    route,
-    method,
-    params,
-    duration,
-    success: false,
-    error,
-  });
-}
-
 // ============================================================================
 // Utility Functions
 // ============================================================================
@@ -334,20 +287,6 @@ export function getLogsPath(): string {
   return LOGS_DIR;
 }
 
-/**
- * Get the path to the errors log file.
- */
-export function getErrorLogPath(): string {
-  return ERROR_LOG;
-}
-
-/**
- * Get the path to the tools log file.
- */
-export function getToolsLogPath(): string {
-  return TOOLS_LOG;
-}
-
 /**
  * Initialize logger synchronously (call at startup).
  * After startup, all operations are async.
@@ -356,41 +295,6 @@ export function initializeLogger(): void {
   ensureLogsDirSync();
 }
 
-// ============================================================================
-// Express Middleware Integration
-// ============================================================================
-
-/**
- * Create a logging middleware that logs tool invocations.
- * Use this to wrap route handlers for automatic logging.
- */
-export function createToolLogger(toolName: string) {
-  return (
-    req: { method: string; path: string; query: unknown },
-    res: { statusCode: number; on: (event: string, cb: () => void) => void },
-    next: () => void
-  ): void => {
-    const start = Date.now();
-
-    res.on('finish', () => {
-      const duration = Date.now() - start;
-      const success = res.statusCode < 400;
-
-      logToolCall({
-        tool: toolName,
-        route: req.path,
-        method: req.method,
-        params: req.query as Record<string, unknown>,
-        duration,
-        success,
-        error: success ? undefined : `HTTP ${res.statusCode}`,
-      });
-    });
-
-    next();
-  };
-}
-
 // ============================================================================
 // Security Utilities
 // ============================================================================

package/skills/octocode-research/src/utils/resilience.ts

@@ -4,8 +4,8 @@
  * @module utils/resilience
  */
 
-import { withCircuitBreaker, type CircuitBreakerConfig } from './circuitBreaker.js';
-import { withRetry, type RetryConfig, RETRY_CONFIGS } from './retry.js';
+import { withCircuitBreaker } from './circuitBreaker.js';
+import { withRetry, RETRY_CONFIGS } from './retry.js';
 import { withTimeout } from './asyncTimeout.js';
 
 /** Default timeout for MCP tool calls (30 seconds) */
@@ -52,16 +52,6 @@ const TOOL_CIRCUIT_MAP: Record<string, string> = {
   packageSearch: 'package',
 };
 
-/**
- * Combined resilience configuration
- */
-export interface ResilienceConfig {
-  circuitBreaker?: Partial<CircuitBreakerConfig>;
-  retry?: RetryConfig;
-  /** Fallback value when circuit is open */
-  fallback?: () => unknown;
-}
-
 /**
  * Pre-configured resilience strategies
  */

package/skills/octocode-research/src/utils/responseFactory.ts

@@ -5,38 +5,8 @@
  * @module utils/responseFactory
  */
 
-import type { FileMatch, PaginationInfo } from '../types/responses.js';
-import { extractFiles, extractPagination, extractTotalMatches } from '../types/responses.js';
 import { isObject, hasProperty, isArray, hasStringProperty, hasNumberProperty } from '../types/guards.js';
 
-// =============================================================================
-// Common Extractors (Type-Safe)
-// =============================================================================
-
-/**
- * Extract file matches from search results with proper typing
- */
-export function extractFileMatches(data: unknown): FileMatch[] {
-  return extractFiles(data);
-}
-
-/**
- * Extract pagination info with proper typing
- */
-export function extractPaginationInfo(data: unknown): PaginationInfo | undefined {
-  return extractPagination(data);
-}
-
-/**
- * Extract total match count with proper typing
- */
-export function extractMatchCount(data: unknown): number {
-  return extractTotalMatches(data);
-}
-
-/**
- * Safely extract string property
- */
 export function safeString(obj: unknown, key: string, fallback = ''): string {
   if (hasStringProperty(obj, key)) {
     return obj[key];
@@ -44,9 +14,6 @@ export function safeString(obj: unknown, key: string, fallback = ''): string {
   return fallback;
 }
 
-/**
- * Safely extract number property
- */
 export function safeNumber(obj: unknown, key: string, fallback = 0): number {
   if (hasNumberProperty(obj, key)) {
     return obj[key];
@@ -54,9 +21,6 @@ export function safeNumber(obj: unknown, key: string, fallback = 0): number {
   return fallback;
 }
 
-/**
- * Safely extract array property
- */
 export function safeArray<T>(obj: unknown, key: string): T[] {
   if (isObject(obj) && hasProperty(obj, key) && isArray(obj[key])) {
     return obj[key] as T[];
@@ -64,9 +28,6 @@ export function safeArray<T>(obj: unknown, key: string): T[] {
   return [];
 }
 
-/**
- * Extract match locations from a file result
- */
 export function extractMatchLocations(matches: unknown[]): Array<{
   line: number;
   column?: number;
@@ -86,9 +47,6 @@ export function extractMatchLocations(matches: unknown[]): Array<{
   });
 }
 
-/**
- * Transform pagination from MCP format to skill format
- */
 export function transformPagination(pagination: unknown): { page: number; total: number; hasMore: boolean } | undefined {
   if (!isObject(pagination)) return undefined;
   

package/skills/octocode-research/src/utils/responseParser.ts

@@ -24,19 +24,13 @@ interface McpToolResponse {
   isError?: boolean;
 }
 
-/**
- * Research context from the MCP response
- */
-export interface ResearchContext {
+interface ResearchContext {
   mainResearchGoal?: string;
   researchGoal?: string;
   reasoning?: string;
 }
 
-/**
- * Single result item from bulk response
- */
-export interface BulkResultItem {
+interface BulkResultItem {
   id: number;
   status: 'hasResults' | 'empty' | 'error';
   data: Record<string, unknown>;
@@ -57,10 +51,7 @@ export interface ParsedResponse {
   status: 'hasResults' | 'empty' | 'error' | 'unknown';
 }
 
-/**
- * Parsed bulk response with all results
- */
-export interface ParsedBulkResponse {
+interface ParsedBulkResponse {
   /** All results from bulk query */
   results: BulkResultItem[];
   /** Categorized hints by status */
@@ -172,19 +163,6 @@ export function parseToolResponse(response: McpToolResponse): ParsedResponse {
   return emptyResult;
 }
 
-/**
- * Convenience function to get data field with type safety
- */
-export function getDataField<T>(
-  response: McpToolResponse,
-  field: string,
-  defaultValue: T
-): T {
-  const { data } = parseToolResponse(response);
-  const value = data[field];
-  return value !== undefined ? (value as T) : defaultValue;
-}
-
 /**
  * Parse ALL results from a bulk MCP tool response.
  * Use this when handling multiple queries to get all results.