octocode-cli 1.2.8 → 1.2.9

package/skills/octocode-engineer/src/types/core.ts

@@ -0,0 +1,203 @@
+export interface Location {
+  file: string;
+  lineStart: number;
+  lineEnd: number;
+  columnStart: number;
+  columnEnd: number;
+}
+
+export interface Metrics {
+  complexity: number;
+  maxBranchDepth: number;
+  maxLoopDepth: number;
+  returns: number;
+  awaits: number;
+  calls: number;
+  loops: number;
+}
+
+export interface HalsteadMetrics {
+  operators: number;
+  operands: number;
+  distinctOperators: number;
+  distinctOperands: number;
+  vocabulary: number;
+  length: number;
+  volume: number;
+  difficulty: number;
+  effort: number;
+  time: number;
+  estimatedBugs: number;
+}
+
+export interface CodeLocation {
+  file: string;
+  lineStart: number;
+  lineEnd: number;
+}
+
+export interface MagicNumberEntry extends CodeLocation {
+  value: number;
+}
+
+export interface ConsoleLogEntry {
+  method: string;
+  lineStart: number;
+  lineEnd: number;
+  hasSensitiveArg: boolean;
+  argSnippet?: string;
+}
+
+export interface MessageChainEntry {
+  chain: string;
+  depth: number;
+  lineStart: number;
+  lineEnd: number;
+}
+
+export interface MagicStringEntry extends CodeLocation {
+  value: string;
+}
+
+export type CatchRethrowEntry = CodeLocation;
+
+export interface BooleanParamCluster {
+  name: string;
+  booleanCount: number;
+  totalParams: number;
+  lineStart: number;
+  lineEnd: number;
+}
+
+export interface PromiseAllUnhandledEntry extends CodeLocation {
+  kind: 'Promise.all' | 'Promise.allSettled' | 'Promise.race' | 'Promise.any';
+}
+
+export interface TreeSitterMetrics extends Metrics {
+  statements: number;
+}
+
+export interface FunctionEntry {
+  kind: string;
+  name: string;
+  nameHint: string;
+  file: string;
+  lineStart: number;
+  lineEnd: number;
+  columnStart: number;
+  columnEnd: number;
+  statementCount: number;
+  complexity: number;
+  maxBranchDepth: number;
+  maxLoopDepth: number;
+  returns: number;
+  awaits: number;
+  calls: number;
+  loops: number;
+  lengthLines: number;
+  cognitiveComplexity: number;
+  halstead?: HalsteadMetrics;
+  maintainabilityIndex?: number;
+  declared?: boolean;
+  params?: number;
+  source?: string;
+}
+
+export interface FlowEntry {
+  kind: string;
+  file: string;
+  lineStart: number;
+  lineEnd: number;
+  columnStart: number;
+  columnEnd: number;
+  statementCount: number;
+}
+
+export interface FlowMapEntry extends FunctionEntry {
+  hash: string;
+  metrics: Metrics;
+}
+
+export interface ControlMapEntry extends FlowEntry {
+  hash: string;
+}
+
+export interface Thresholds {
+  // Architecture
+  couplingThreshold: number;
+  fanInThreshold: number;
+  fanOutThreshold: number;
+  godModuleStatements: number;
+  godModuleExports: number;
+  barrelSymbolThreshold: number;
+  sdpMinDelta: number;
+  sdpMaxSourceInstability: number;
+  layerOrder: string[];
+
+  // Code quality
+  minFunctionStatements: number;
+  minFlowStatements: number;
+  criticalComplexityThreshold: number;
+  godFunctionStatements: number;
+  godFunctionMiThreshold: number;
+  cognitiveComplexityThreshold: number;
+  parameterThreshold: number;
+  halsteadEffortThreshold: number;
+  maintainabilityIndexThreshold: number;
+  anyThreshold: number;
+  flowDupThreshold: number;
+  similarityThreshold: number;
+  deepNestingThreshold: number;
+  multipleReturnThreshold: number;
+  magicStringMinOccurrences: number;
+  booleanParamThreshold: number;
+
+  // Semantic
+  overrideChainThreshold: number;
+  shotgunThreshold: number;
+
+  // Security
+  secretEntropyThreshold: number;
+  secretMinLength: number;
+
+  // Test quality
+  mockThreshold: number;
+}
+
+export type ReporterFormat = 'default' | 'compact' | 'github-actions';
+
+export interface AnalysisOptions {
+  root: string;
+  out: string | null;
+  json: boolean;
+  packageRoot: string;
+  parser: 'auto' | 'typescript' | 'tree-sitter';
+  includeTests: boolean;
+  emitTree: boolean;
+  treeDepth: number;
+  noCache: boolean;
+  clearCache: boolean;
+  semantic: boolean;
+  graph: boolean;
+  graphAdvanced: boolean;
+  flow: boolean;
+  scope: string[] | null;
+  scopeSymbols: Map<string, string[]> | null;
+  features: Set<string> | null;
+  ignoreDirs: Set<string>;
+  findingsLimit: number;
+  noDiversify: boolean;
+  maxRecsPerCategory: number;
+  deepLinkTopN: number;
+  thresholds: Thresholds;
+
+  affected: string | null;
+  saveBaseline: boolean;
+  ignoreKnown: string | null;
+  reporter: ReporterFormat;
+  focus: string | null;
+  focusDepth: number;
+  collapse: number | null;
+  atLeast: number | null;
+  configFile: string | null;
+}

package/skills/octocode-engineer/src/types/dependency.ts

@@ -0,0 +1,215 @@
+import type {
+  ControlMapEntry,
+  FlowMapEntry,
+} from './core.js';
+
+export interface ExportSymbol {
+  name: string;
+  kind: 'value' | 'type' | 'unknown';
+  isDefault?: boolean;
+  lineStart?: number;
+  lineEnd?: number;
+}
+
+export interface ImportedSymbolRef {
+  sourceModule: string;
+  resolvedModule?: string;
+  importedName: string;
+  localName: string;
+  isTypeOnly: boolean;
+  lineStart?: number;
+  lineEnd?: number;
+}
+
+export interface ReExportRef {
+  sourceModule: string;
+  resolvedModule?: string;
+  exportedAs: string;
+  importedName: string;
+  isStar: boolean;
+  isTypeOnly: boolean;
+  lineStart?: number;
+  lineEnd?: number;
+}
+
+export interface DependencyProfile {
+  internalDependencies: string[];
+  externalDependencies: string[];
+  unresolvedDependencies: string[];
+  declaredExports: ExportSymbol[];
+  importedSymbols: ImportedSymbolRef[];
+  reExports: ReExportRef[];
+  package?: string;
+  file?: string;
+}
+
+export interface DependencyState {
+  files: Set<string>;
+  outgoing: Map<string, Set<string>>;
+  incoming: Map<string, Set<string>>;
+  incomingFromTests: Map<string, Set<string>>;
+  incomingFromProduction: Map<string, Set<string>>;
+  externalCounts: Map<string, Set<string>>;
+  unresolvedCounts: Map<string, Set<string>>;
+  declaredExportsByFile: Map<string, ExportSymbol[]>;
+  importedSymbolsByFile: Map<string, ImportedSymbolRef[]>;
+  reExportsByFile: Map<string, ReExportRef[]>;
+}
+
+export interface DependencyRecord {
+  file: string;
+  outboundCount: number;
+  inboundCount: number;
+  inboundFromProduction: number;
+  inboundFromTests: number;
+  externalDependencyCount: number;
+  unresolvedDependencyCount: number;
+}
+
+export interface FileCriticality {
+  file: string;
+  complexityRisk: number;
+  highComplexityFunctions: number;
+  functionCount: number;
+  flows: number;
+  score: number;
+  complexitySum?: number;
+}
+
+export interface DuplicateGroup {
+  hash: string;
+  signature: string;
+  kind: string;
+  occurrences: number;
+  filesCount: number;
+  locations: FlowMapEntry[];
+}
+
+export interface RedundantFlowGroup {
+  kind: string;
+  occurrences: number;
+  filesCount: number;
+  locations: ControlMapEntry[];
+}
+
+export interface Cycle {
+  path: string[];
+  nodeCount: number;
+}
+
+export interface CriticalPath {
+  start: string;
+  path: string[];
+  score: number;
+  length: number;
+  containsCycle: boolean;
+}
+
+export interface SccCluster {
+  id: string;
+  files: string[];
+  nodeCount: number;
+  edgeCount: number;
+  entryEdges: number;
+  exitEdges: number;
+  hubFiles: string[];
+}
+
+export interface Chokepoint {
+  file: string;
+  score: number;
+  reasons: string[];
+  fanIn: number;
+  fanOut: number;
+  articulation: boolean;
+  bridgeCount: number;
+  cycleClusterCount: number;
+  onCriticalPath: boolean;
+}
+
+export interface PackageGraphNode {
+  package: string;
+  inbound: number;
+  outbound: number;
+  internalFiles: number;
+}
+
+export interface PackageHotspot {
+  from: string;
+  to: string;
+  edges: number;
+  [key: string]: unknown;
+}
+
+export interface PackageGraphSummary {
+  packageCount: number;
+  edgeCount: number;
+  packages: PackageGraphNode[];
+  hotspots: PackageHotspot[];
+}
+
+export interface CriticalModule extends DependencyRecord {
+  score: number;
+  riskBand: string;
+  [key: string]: unknown;
+}
+
+export interface TestOnlyModule extends DependencyRecord {
+  lineStart?: number;
+  lineEnd?: number;
+}
+
+export interface ModuleCount {
+  file: string;
+  count: number;
+  score: number;
+}
+
+export interface DependencySummary {
+  totalModules: number;
+  totalEdges: number;
+  unresolvedEdgeCount: number;
+  externalDependencyFiles: number;
+  rootsCount: number;
+  leavesCount: number;
+  roots: string[];
+  leaves: string[];
+  criticalModules: CriticalModule[];
+  testOnlyModules: TestOnlyModule[];
+  unresolvedSample: string[];
+  outgoingTop: ModuleCount[];
+  inboundTop: ModuleCount[];
+  cycles: Cycle[];
+  criticalPaths: CriticalPath[];
+}
+
+export interface HotFile {
+  file: string;
+  riskScore: number;
+  fanIn: number;
+  fanOut: number;
+  complexityScore: number;
+  exportCount: number;
+  inCycle: boolean;
+  onCriticalPath: boolean;
+}
+
+export interface WalkResult {
+  path: string[];
+  score: number;
+  containsCycle: boolean;
+}
+
+export interface DuplicateFlowHint {
+  type: string;
+  message: string;
+  file: string | undefined;
+  lineStart: number | undefined;
+  lineEnd: number | undefined;
+  details: string;
+}
+
+export interface FlowMaps {
+  flowMap: Map<string, FlowMapEntry[]>;
+  controlMap: Map<string, ControlMapEntry[]>;
+}

package/skills/octocode-engineer/src/types/file-entry.ts

@@ -0,0 +1,108 @@
+import type {
+  BoundaryRoleHint,
+  CfgFlags,
+  EffectProfile,
+  InputSourceInfo,
+  SuspiciousString,
+  SymbolUsageSummary,
+  TestProfile,
+  TimerCall,
+  TopLevelEffect,
+} from './collectors.js';
+import type {
+  BooleanParamCluster,
+  CatchRethrowEntry,
+  CodeLocation,
+  ConsoleLogEntry,
+  FlowEntry,
+  FunctionEntry,
+  MagicNumberEntry,
+  MagicStringEntry,
+  MessageChainEntry,
+  PromiseAllUnhandledEntry,
+} from './core.js';
+import type { DependencyProfile } from './dependency.js';
+
+export interface FileEntry {
+  package: string;
+  file: string;
+  parseEngine: string;
+  nodeCount: number;
+  kindCounts: Record<string, number>;
+  functions: FunctionEntry[];
+  flows: FlowEntry[];
+  dependencyProfile: DependencyProfile;
+  emptyCatches?: CodeLocation[];
+  switchesWithoutDefault?: CodeLocation[];
+  anyCount?: number;
+  magicNumbers?: MagicNumberEntry[];
+  typeAssertionEscapes?: {
+    asAny: CodeLocation[];
+    doubleAssertion: CodeLocation[];
+    nonNull: CodeLocation[];
+  };
+  asyncWithoutAwait?: Array<{
+    name: string;
+    lineStart: number;
+    lineEnd: number;
+  }>;
+  unprotectedAsync?: Array<{
+    name: string;
+    awaitCount: number;
+    lineStart: number;
+    lineEnd: number;
+  }>;
+  evalUsages?: CodeLocation[];
+  unsafeHtmlAssignments?: CodeLocation[];
+  suspiciousStrings?: SuspiciousString[];
+  regexLiterals?: Array<{
+    lineStart: number;
+    lineEnd: number;
+    pattern: string;
+  }>;
+  awaitInLoopLocations?: CodeLocation[];
+  syncIoCalls?: Array<{ name: string; lineStart: number; lineEnd: number }>;
+  timerCalls?: TimerCall[];
+  listenerRegistrations?: CodeLocation[];
+  listenerRemovals?: CodeLocation[];
+  testProfile?: TestProfile;
+  inputSources?: InputSourceInfo[];
+  treeSitterNodeCount?: number;
+  treeSitterError?: string;
+  parserFallback?: string;
+  topLevelEffects?: TopLevelEffect[];
+  prototypePollutionSites?: Array<{
+    kind: string;
+    detail: string;
+    lineStart: number;
+    lineEnd: number;
+    guarded: boolean;
+  }>;
+  effectProfile?: EffectProfile;
+  symbolUsageSummary?: SymbolUsageSummary;
+  boundaryRoleHints?: BoundaryRoleHint[];
+  cfgFlags?: CfgFlags;
+  consoleLogs?: ConsoleLogEntry[];
+  messageChains?: MessageChainEntry[];
+  magicStrings?: MagicStringEntry[];
+  catchRethrows?: CatchRethrowEntry[];
+  booleanParamClusters?: BooleanParamCluster[];
+  promiseAllUnhandled?: PromiseAllUnhandledEntry[];
+  issueIds?: string[];
+}
+
+export interface PackageFileSummary {
+  fileCount: number;
+  nodeCount: number;
+  functionCount: number;
+  flowCount: number;
+  kindCounts: Record<string, number>;
+  functions: FunctionEntry[];
+  flows: FlowEntry[];
+}
+
+export interface PackageInfo {
+  name: string;
+  dir: string;
+  folder: string;
+}

package/skills/octocode-engineer/src/types/findings.ts

@@ -0,0 +1,105 @@
+import type { AnalysisSignal, RecommendedValidation } from './analysis.js';
+
+export type { AnalysisLens, AnalysisSignal, RecommendedValidation, FlowTraceStep } from './analysis.js';
+
+export interface PillarFindingStats {
+  totalFindings: number;
+  severityBreakdown: Record<string, number>;
+}
+
+export interface FindingStats {
+  overall?: PillarFindingStats;
+  pillars?: Record<string, PillarFindingStats>;
+}
+
+interface SuggestedFix {
+  strategy: string;
+  steps: string[];
+}
+
+interface LspHint {
+  tool: 'lspFindReferences' | 'lspCallHierarchy' | 'lspGotoDefinition';
+  symbolName: string;
+  lineHint: number;
+  file: string;
+  expectedResult: string;
+}
+
+export interface Finding {
+  id: string;
+  severity: 'critical' | 'high' | 'medium' | 'low' | 'info';
+  category: string;
+  file: string;
+  lineStart: number;
+  lineEnd: number;
+  title: string;
+  reason: string;
+  files: string[];
+  suggestedFix: SuggestedFix;
+  impact?: string;
+  tags?: string[];
+  columnStart?: number;
+  columnEnd?: number;
+  lspHints?: LspHint[];
+  ruleId?: string;
+  analysisLens?: import('./analysis.js').AnalysisLens;
+  confidence?: 'high' | 'medium' | 'low';
+  evidence?: Record<string, unknown>;
+  correlatedSignals?: string[];
+  recommendedValidation?: RecommendedValidation;
+  flowTrace?: import('./analysis.js').FlowTraceStep[];
+}
+
+export interface TopRecommendation {
+  id: string;
+  file: string;
+  severity: string;
+  category: string;
+  title: string;
+  reason: string;
+  suggestedFix?: Finding['suggestedFix'];
+}
+
+export interface ScanSummaryData {
+  totalPackages?: number;
+  totalFiles?: number;
+  totalNodes?: number;
+  totalFunctions?: number;
+  totalFlows?: number;
+  totalDependencyFiles?: number;
+  byPackage?: Record<
+    string,
+    {
+      files: number;
+      nodes: number;
+      functions: number;
+      flows: number;
+      topKinds: [string, number][];
+      rootPath: string;
+    }
+  >;
+  [key: string]: unknown;
+}
+
+export interface AgentOutputData {
+  totalFindings?: number;
+  totalBeforeTruncation?: number;
+  droppedCategories?: string[];
+  findingStats?: FindingStats | null;
+  analysisSummary?: {
+    strongestGraphSignal?: AnalysisSignal | null;
+    strongestAstSignal?: AnalysisSignal | null;
+    combinedSignals?: AnalysisSignal[];
+    recommendedValidation?: RecommendedValidation | null;
+  };
+  highPriority?: number;
+  mediumPriority?: number;
+  lowPriority?: number;
+  topRecommendations?: TopRecommendation[];
+  filesWithIssues?: Array<{
+    file: string;
+    issueCount: number;
+    issueIds: string[];
+  }>;
+  [key: string]: unknown;
+}