Skip to content

Commit d28460c

Browse files
authored
fix: close v1.8 post-merge integration gaps (#44)
* 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) * Fix formatting
1 parent 22bccc1 commit d28460c

File tree

7 files changed

+212
-315
lines changed

7 files changed

+212
-315
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -62,6 +62,7 @@
6262

6363
- Null-pointer crash in GenericAnalyzer when chunk content is undefined.
6464
- Tree-sitter symbol extraction now treats node offsets as UTF-8 byte ranges and evicts cached parsers on failures/timeouts.
65+
- **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.
6566

6667
## [1.6.2] - 2026-02-17
6768

internal-docs

src/core/search-quality.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@ export function assessSearchQuality(
3030
nextSteps: [
3131
'Try a narrower query with one concrete symbol, route, or file hint.',
3232
'Apply search filters (framework/language/componentType/layer).',
33-
'Use get_component_usage for dependency or wiring lookups.'
33+
'Use get_symbol_references to find where a specific symbol is used across the codebase.'
3434
]
3535
};
3636
}
@@ -76,7 +76,7 @@ export function assessSearchQuality(
7676
nextSteps: [
7777
'Add one or two concrete symbols, routes, or file hints to the query.',
7878
'Apply filters (framework/language/componentType/layer) to narrow candidates.',
79-
'Use get_component_usage when the question is about wiring or usages.'
79+
'Use get_symbol_references when the question is about where a symbol or function is used.'
8080
]
8181
})
8282
};

src/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -84,7 +84,6 @@ const LEGACY_PATHS = {
8484
export const INDEX_CONSUMING_TOOL_NAMES = [
8585
'search_codebase',
8686
'get_symbol_references',
87-
'get_component_usage',
8887
'detect_circular_dependencies',
8988
'get_team_patterns',
9089
'get_codebase_metadata'

src/tools/get-component-usage.ts

Lines changed: 0 additions & 108 deletions
This file was deleted.

0 commit comments

Comments
 (0)