fix: close v1.8 post-merge integration gaps

Addresses 5 live code gaps left after PRs #39 and #40 merged:

1. Delete orphaned get-component-usage source file (already removed from
   dispatcher in PR #39, but file remained on disk causing confusion)

2. Remove 'get_component_usage' from INDEX_CONSUMING_TOOL_NAMES allowlist
   in src/index.ts. The dispatcher no longer routes to this tool, so the
   allowlist entry created a phantom entry that would be validated during
   test runs despite not being available.

3. Remove get_component_usage branch from args dispatch ternary in
   index-versioning-migration test. Now that the tool is removed, this
   branch is dead code and should not be validated.

4. Replace guidance strings in search-quality.ts that still directed
   agents to the removed get_component_usage tool. Both instances now
   point to get_symbol_references, which covers the same use case.

5. Add fallback decision card when intelligence.json is absent in
   search-codebase.ts. Previously, preflight would silently skip when
   intelligence was missing, leaving users without guidance. Now
   returns ready=false with actionable nextAction message.

All changes verified:
- Zero references to get_component_usage in src/ and tests/
- Index-versioning-migration test passes (5/5)
- Full test suite: 231/234 passing (3 pre-existing timeouts)

Author: PatrickSys

CHANGELOG.md

@@ -62,6 +62,7 @@
 
 - Null-pointer crash in GenericAnalyzer when chunk content is undefined.
 - Tree-sitter symbol extraction now treats node offsets as UTF-8 byte ranges and evicts cached parsers on failures/timeouts.
+- **Post-merge integration gaps** (v1.8 audit): Removed orphaned `get_component_usage` source file, deleted phantom allowlist entry, removed dead guidance strings referencing the deleted tool. Added fallback decision card when `intelligence.json` is absent during edit-intent searches, now returns `ready: false` with actionable guidance instead of silently skipping.
 
 ## [1.6.2] - 2026-02-17
 

internal-docs

@@ -30,7 +30,7 @@ export function assessSearchQuality(
       nextSteps: [
         'Try a narrower query with one concrete symbol, route, or file hint.',
         'Apply search filters (framework/language/componentType/layer).',
-        'Use get_component_usage for dependency or wiring lookups.'
+        'Use get_symbol_references to find where a specific symbol is used across the codebase.'
       ]
     };
   }
@@ -76,7 +76,7 @@ export function assessSearchQuality(
       nextSteps: [
         'Add one or two concrete symbols, routes, or file hints to the query.',
         'Apply filters (framework/language/componentType/layer) to narrow candidates.',
-        'Use get_component_usage when the question is about wiring or usages.'
+        'Use get_symbol_references when the question is about where a symbol or function is used.'
       ]
     })
   };

src/core/search-quality.ts

@@ -84,7 +84,6 @@ const LEGACY_PATHS = {
 export const INDEX_CONSUMING_TOOL_NAMES = [
   'search_codebase',
   'get_symbol_references',
-  'get_component_usage',
   'detect_circular_dependencies',
   'get_team_patterns',
   'get_codebase_metadata'

src/index.ts

@@ -439,12 +439,18 @@ export async function handle(
   // Compose preflight card for edit/refactor/migrate intents
   let preflight: any = undefined;
   const preflightIntents = ['edit', 'refactor', 'migrate'];
-  if (intent && preflightIntents.includes(intent) && intelligence) {
-    try {
-      // --- Avoid / Prefer patterns ---
-      const avoidPatternsList: any[] = [];
-      const preferredPatternsList: any[] = [];
-      const patterns = intelligence.patterns || {};
+  if (intent && preflightIntents.includes(intent)) {
+    if (!intelligence) {
+      preflight = {
+        ready: false,
+        nextAction: 'Run a full index rebuild to generate pattern intelligence before editing.'
+      };
+    } else {
+      try {
+        // --- Avoid / Prefer patterns ---
+        const avoidPatternsList: any[] = [];
+        const preferredPatternsList: any[] = [];
+        const patterns = intelligence.patterns || {};
       for (const [category, data] of Object.entries<any>(patterns)) {
         // Primary pattern = preferred if Rising or Stable
         if (data.primary) {
@@ -658,8 +664,9 @@ export async function handle(
       }
 
       preflight = decisionCard;
-    } catch {
-      // Preflight construction failed — skip preflight, don't fail the search
+      } catch {
+        // Preflight construction failed — skip preflight, don't fail the search
+      }
     }
   }
 

src/tools/get-component-usage.ts

@@ -461,13 +461,11 @@ describe('index-consuming allowlist enforcement', () => {
               ? { query: 'test' }
               : toolName === 'get_symbol_references'
                 ? { symbol: 'alpha' }
-                : toolName === 'get_component_usage'
-                  ? { name: 'x' }
-                  : toolName === 'detect_circular_dependencies'
+                : toolName === 'detect_circular_dependencies'
+                  ? {}
+                  : toolName === 'get_team_patterns'
                     ? {}
-                    : toolName === 'get_team_patterns'
-                      ? {}
-                      : {}
+                    : {}
         }
       });