fix: unify .h reclassification between scan and update, fix stale JSDoc and redundant hash

- Extract hasCppSourceFiles() and effectiveLanguage() to traverser.js
  so both scanner.js and update command share the same .h → cpp logic
- Update query.js JSDoc to reference current field names (fileImports/importedBy)
- Remove redundant hash recomputation in update command

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: invalid

cli/src/commands/update.js

@@ -1,7 +1,7 @@
 import path from 'path';
 import fs from 'fs/promises';
 import { loadGraph, loadMeta, saveGraph, computeFileHash, getGitCommitHash, isEntryPoint } from '../graph.js';
-import { traverseFiles, detectLanguage } from '../traverser.js';
+import { traverseFiles, detectLanguage, hasCppSourceFiles, effectiveLanguage } from '../traverser.js';
 import { initParser, parseFile } from '../parser.js';
 import { detectModuleName } from '../scanner.js';
 import { detectChangedFiles, mergeGraphUpdate } from '../differ.js';
@@ -37,6 +37,7 @@ export function registerUpdateCommand(program) {
 
       // Step 3: Traverse files and compute current hashes
       const files = await traverseFiles(rootDir);
+      const hasCpp = hasCppSourceFiles(files);
       const currentHashes = {};
 
       for (const absPath of files) {
@@ -66,17 +67,20 @@ export function registerUpdateCommand(program) {
 
       for (const relPath of changedPaths) {
         const absPath = path.resolve(rootDir, relPath);
-        const language = detectLanguage(absPath);
+        let language = detectLanguage(absPath);
         if (!language) continue;
 
+        // Reclassify .h files as C++ when the project contains C++ sources
+        language = effectiveLanguage(absPath, language, hasCpp);
+
         let content;
         try {
           content = await fs.readFile(absPath, 'utf-8');
         } catch {
           continue;
         }
 
-        const hash = computeFileHash(content);
+        const hash = currentHashes[relPath];
 
         let parsed;
         try {

cli/src/query.js

@@ -12,7 +12,7 @@
  * @param {string} symbolName - The name to search for (case-sensitive substring match).
  * @param {object} [options] - Optional filters.
  * @param {'function'|'class'|'type'} [options.type] - Restrict search to a specific kind.
- * @returns {Array<{kind: string, name: string, signature?: string, file: string, module: string, lines: {start: number, end: number}, calls: string[], calledBy: string[]}>}
+ * @returns {Array<{kind: string, name: string, signature?: string, file: string, module: string, lines: {start: number, end: number}, fileImports: string[], importedBy: string[]}>}
  */
 export function querySymbol(graph, symbolName, options = {}) {
   const results = [];

cli/src/scanner.js

@@ -1,6 +1,6 @@
 import path from 'path';
 import fs from 'fs/promises';
-import { traverseFiles, detectLanguage } from './traverser.js';
+import { traverseFiles, detectLanguage, hasCppSourceFiles, effectiveLanguage } from './traverser.js';
 import { initParser, parseFile } from './parser.js';
 import { createEmptyGraph, computeFileHash, isEntryPoint } from './graph.js';
 
@@ -54,10 +54,7 @@ export async function scanProject(rootDir, options = {}) {
   const files = await traverseFiles(rootDir, { exclude: options.exclude || [] });
 
   // Detect whether the project contains C++ files, so we can treat .h as C++
-  const hasCppFiles = files.some(f => {
-    const ext = path.extname(f).toLowerCase();
-    return ['.cpp', '.cc', '.cxx', '.hpp', '.hh'].includes(ext);
-  });
+  const hasCppFiles = hasCppSourceFiles(files);
 
   // Index: absolute path (normalised with forward slashes) → parsed data + module
   const fileIndex = new Map();
@@ -72,9 +69,7 @@ export async function scanProject(rootDir, options = {}) {
     if (!language) continue;
 
     // Reclassify .h files as C++ when the project contains C++ sources
-    if (language === 'c' && hasCppFiles && path.extname(absPath).toLowerCase() === '.h') {
-      language = 'cpp';
-    }
+    language = effectiveLanguage(absPath, language, hasCppFiles);
 
     let content;
     try {

cli/src/traverser.js

@@ -48,4 +48,26 @@ export async function traverseFiles(rootDir, options = {}) {
   return files.sort();
 }
 
+/** C++ file extensions (excluding .h which is shared with C). */
+const CPP_EXTENSIONS = new Set(['.cpp', '.cc', '.cxx', '.hpp', '.hh']);
+
+/**
+ * Check whether a list of absolute file paths contains any C++ source files.
+ * Used to decide if `.h` headers should be reclassified as C++.
+ */
+export function hasCppSourceFiles(files) {
+  return files.some(f => CPP_EXTENSIONS.has(path.extname(f).toLowerCase()));
+}
+
+/**
+ * Return the effective language for a file, applying `.h` → `cpp`
+ * reclassification when the project contains C++ sources.
+ */
+export function effectiveLanguage(filePath, baseLanguage, projectHasCpp) {
+  if (baseLanguage === 'c' && projectHasCpp && path.extname(filePath).toLowerCase() === '.h') {
+    return 'cpp';
+  }
+  return baseLanguage;
+}
+
 export { LANGUAGE_EXTENSIONS, ALL_EXTENSIONS };