gitnexus 1.6.4-rc.6 → 1.6.4-rc.8

package/dist/core/ingestion/languages/python/captures.js

@@ -22,6 +22,7 @@ import { synthesizeReceiverTypeBinding } from './receiver-binding.js';
 import { computePythonArityMetadata } from './arity-metadata.js';
 import { recordCacheHit, recordCacheMiss } from './cache-stats.js';
 import { getTreeSitterBufferSize } from '../../constants.js';
+import { pythonFunctionDefinitionLabel } from './simple-hooks.js';
 export function emitPythonScopeCaptures(sourceText, _filePath, cachedTree) {
     // Skip the parse when the caller (parse phase's ASTCache) already
     // produced a Tree for this source. Cache miss = re-parse, same as
@@ -86,6 +87,10 @@ export function emitPythonScopeCaptures(sourceText, _filePath, cachedTree) {
             const anchorCap = grouped['@declaration.function'];
             const fnNode = findNodeAtRange(tree.rootNode, anchorCap.range, 'function_definition');
             if (fnNode !== null) {
+                if (pythonFunctionDefinitionLabel(fnNode, 'Function') === 'Method') {
+                    delete grouped['@declaration.function'];
+                    grouped['@declaration.method'] = { ...anchorCap, name: '@declaration.method' };
+                }
                 const arity = computePythonArityMetadata(fnNode);
                 if (arity.parameterCount !== undefined) {
                     grouped['@declaration.parameter-count'] = syntheticCapture('@declaration.parameter-count', fnNode, String(arity.parameterCount));

package/dist/core/ingestion/languages/python/index.d.ts

@@ -77,4 +77,4 @@ export { interpretPythonImport, interpretPythonTypeBinding } from './interpret.j
 export { pythonMergeBindings } from './merge-bindings.js';
 export { pythonArityCompatibility } from './arity.js';
 export { resolvePythonImportTarget, type PythonResolveContext } from './import-target.js';
-export { pythonBindingScopeFor, pythonImportOwningScope, pythonReceiverBinding, } from './simple-hooks.js';
+export { pythonBindingScopeFor, pythonFunctionDefinitionLabel, pythonImportOwningScope, pythonReceiverBinding, } from './simple-hooks.js';

package/dist/core/ingestion/languages/python/index.js

@@ -77,4 +77,4 @@ export { interpretPythonImport, interpretPythonTypeBinding } from './interpret.j
 export { pythonMergeBindings } from './merge-bindings.js';
 export { pythonArityCompatibility } from './arity.js';
 export { resolvePythonImportTarget } from './import-target.js';
-export { pythonBindingScopeFor, pythonImportOwningScope, pythonReceiverBinding, } from './simple-hooks.js';
+export { pythonBindingScopeFor, pythonFunctionDefinitionLabel, pythonImportOwningScope, pythonReceiverBinding, } from './simple-hooks.js';

package/dist/core/ingestion/languages/python/simple-hooks.d.ts

@@ -5,7 +5,9 @@
  * "absence == default") so reviewers don't have to re-derive the
  * analysis.
  */
-import type { CaptureMatch, ParsedImport, Scope, ScopeId, ScopeTree, TypeRef } from '../../../../_shared/index.js';
+import type { CaptureMatch, NodeLabel, ParsedImport, Scope, ScopeId, ScopeTree, TypeRef } from '../../../../_shared/index.js';
+import type { SyntaxNode } from 'tree-sitter';
+export declare function pythonFunctionDefinitionLabel(functionNode: SyntaxNode, defaultLabel: NodeLabel): NodeLabel;
 /** Python has no block scope, so the central extractor's "innermost
  *  enclosing scope" default is already correct: `for x in …` creates
  *  `x` in the enclosing function/module scope (because we never emit a

package/dist/core/ingestion/languages/python/simple-hooks.js

@@ -5,6 +5,14 @@
  * "absence == default") so reviewers don't have to re-derive the
  * analysis.
  */
+import { findAncestorBeforeBoundary, FUNCTION_NODE_TYPES } from '../../utils/ast-helpers.js';
+const PYTHON_METHOD_CONTAINER_TYPES = new Set(['class_definition']);
+export function pythonFunctionDefinitionLabel(functionNode, defaultLabel) {
+    if (defaultLabel !== 'Function')
+        return defaultLabel;
+    const ancestor = findAncestorBeforeBoundary(functionNode, PYTHON_METHOD_CONTAINER_TYPES, FUNCTION_NODE_TYPES);
+    return ancestor === null ? 'Function' : 'Method';
+}
 // ─── bindingScopeFor ──────────────────────────────────────────────────────
 /** Python has no block scope, so the central extractor's "innermost
  *  enclosing scope" default is already correct: `for x in …` creates

package/dist/core/ingestion/languages/python.js

@@ -28,7 +28,7 @@ import { pythonVariableConfig } from '../variable-extractors/configs/python.js';
 import { createCallExtractor } from '../call-extractors/generic.js';
 import { pythonCallConfig } from '../call-extractors/configs/python.js';
 import { createHeritageExtractor } from '../heritage-extractors/generic.js';
-import { emitPythonScopeCaptures, interpretPythonImport, interpretPythonTypeBinding, pythonArityCompatibility, pythonBindingScopeFor, pythonImportOwningScope, pythonMergeBindings, pythonReceiverBinding, resolvePythonImportTarget, } from './python/index.js';
+import { emitPythonScopeCaptures, pythonFunctionDefinitionLabel, interpretPythonImport, interpretPythonTypeBinding, pythonArityCompatibility, pythonBindingScopeFor, pythonImportOwningScope, pythonMergeBindings, pythonReceiverBinding, resolvePythonImportTarget, } from './python/index.js';
 const BUILT_INS = new Set([
     'print',
     'len',
@@ -75,6 +75,7 @@ export const pythonProvider = defineLanguage({
     classExtractor: createClassExtractor(pythonClassConfig),
     heritageExtractor: createHeritageExtractor(SupportedLanguages.Python),
     builtInNames: BUILT_INS,
+    labelOverride: pythonFunctionDefinitionLabel,
     // ── RFC #909 Ring 3: scope-based resolution hooks (RFC §5) ──────────
     // Python is the first migration. See ./python/index.ts for the
     // full per-hook rationale and the canonical capture vocabulary in

package/dist/core/ingestion/scope-extractor-bridge.d.ts

@@ -13,8 +13,11 @@
  *      `emitScopeCaptures`. Returns `undefined`; zero work done. This is
  *      the state of every language today — `ParsedFile` production stays
  *      dormant until a language migrates.
- *   2. Invokes the hook + feeds its output to `ScopeExtractor.extract`.
- *   3. **Swallows exceptions from either side.** A failure here returns
+ *   2. Short-circuits empty / whitespace-only files. There is no scope
+ *      content to extract, and some tree-sitter queries do not match an
+ *      otherwise valid empty root node.
+ *   3. Invokes the hook + feeds its output to `ScopeExtractor.extract`.
+ *   4. **Swallows exceptions from either side.** A failure here returns
  *      `undefined` and emits a warning via `onWarn`; legacy parsing on
  *      the same file continues unaffected by the scope-extraction miss.
  *      Scope-based resolution is the new path under construction — it

package/dist/core/ingestion/scope-extractor-bridge.js

@@ -13,8 +13,11 @@
  *      `emitScopeCaptures`. Returns `undefined`; zero work done. This is
  *      the state of every language today — `ParsedFile` production stays
  *      dormant until a language migrates.
- *   2. Invokes the hook + feeds its output to `ScopeExtractor.extract`.
- *   3. **Swallows exceptions from either side.** A failure here returns
+ *   2. Short-circuits empty / whitespace-only files. There is no scope
+ *      content to extract, and some tree-sitter queries do not match an
+ *      otherwise valid empty root node.
+ *   3. Invokes the hook + feeds its output to `ScopeExtractor.extract`.
+ *   4. **Swallows exceptions from either side.** A failure here returns
  *      `undefined` and emits a warning via `onWarn`; legacy parsing on
  *      the same file continues unaffected by the scope-extraction miss.
  *      Scope-based resolution is the new path under construction — it
@@ -29,6 +32,8 @@ import { extract as extractScope } from './scope-extractor.js';
 export function extractParsedFile(provider, sourceText, filePath, onWarn, cachedTree) {
     if (provider.emitScopeCaptures === undefined)
         return undefined;
+    if (sourceText.trim().length === 0)
+        return undefined;
     try {
         const captures = provider.emitScopeCaptures(sourceText, filePath, cachedTree);
         return extractScope(captures, filePath, provider);

package/dist/core/ingestion/utils/ast-helpers.d.ts

@@ -31,6 +31,8 @@ export declare const FUNCTION_NODE_TYPES: Set<string>;
  */
 export declare const CLASS_CONTAINER_TYPES: Set<string>;
 export declare const CONTAINER_TYPE_TO_LABEL: Record<string, string>;
+/** Return the first matching ancestor unless a boundary ancestor is reached first. */
+export declare function findAncestorBeforeBoundary(node: SyntaxNode, targetTypes: ReadonlySet<string>, boundaryTypes: ReadonlySet<string>): SyntaxNode | null;
 /**
  * Determine the graph node label from a tree-sitter capture map.
  * Handles language-specific reclassification via the provider's labelOverride hook

package/dist/core/ingestion/utils/ast-helpers.js

@@ -153,6 +153,18 @@ export const CONTAINER_TYPE_TO_LABEL = {
     object_declaration: 'Class',
     companion_object: 'Class',
 };
+/** Return the first matching ancestor unless a boundary ancestor is reached first. */
+export function findAncestorBeforeBoundary(node, targetTypes, boundaryTypes) {
+    let current = node.parent;
+    while (current !== null) {
+        if (boundaryTypes.has(current.type))
+            return null;
+        if (targetTypes.has(current.type))
+            return current;
+        current = current.parent;
+    }
+    return null;
+}
 /**
  * Determine the graph node label from a tree-sitter capture map.
  * Handles language-specific reclassification via the provider's labelOverride hook

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitnexus",
-  "version": "1.6.4-rc.6",
+  "version": "1.6.4-rc.8",
   "description": "Graph-powered code intelligence for AI agents. Index any codebase, query via MCP or CLI.",
   "author": "Abhigyan Patwari",
   "license": "PolyForm-Noncommercial-1.0.0",