gitnexus 1.6.6-rc.73 → 1.6.6-rc.75

package/dist/core/group/extractors/http-patterns/index.d.ts

@@ -4,12 +4,17 @@ export type { HttpDetection, HttpLanguagePlugin, HttpRole } from './types.js';
  * Glob for files worth scanning for HTTP routes. Kept alongside the
  * registry so adding a new language widens the glob in one edit.
  *
+ * `.kt`/`.kts` are always present in the glob even when the optional
+ * `tree-sitter-kotlin` grammar isn't installed — `getPluginForFile`
+ * will return `undefined` for those files in that case, so the
+ * orchestrator simply skips them at scan time without erroring.
+ *
  * `.vue` / `.svelte` files are intentionally omitted for the source-scan
  * path — they need their own grammar-aware extraction and the existing
  * regex fallback for them was never very accurate. The graph-assisted
  * Strategy A still handles them via the ingestion pipeline.
  */
-export declare const HTTP_SCAN_GLOB = "**/*.{ts,tsx,js,jsx,java,go,py,php}";
+export declare const HTTP_SCAN_GLOB = "**/*.{ts,tsx,js,jsx,java,kt,kts,go,py,php}";
 /**
  * Return the HTTP plugin registered for the given file's extension,
  * or `undefined` if the extension is not registered.

package/dist/core/group/extractors/http-patterns/index.js

@@ -1,6 +1,7 @@
 import * as path from 'node:path';
 import { isBladeTemplateFilename } from '../../../../_shared/index.js';
 import { JAVA_HTTP_PLUGIN } from './java.js';
+import { KOTLIN_HTTP_PLUGIN } from './kotlin.js';
 import { GO_HTTP_PLUGIN } from './go.js';
 import { PYTHON_HTTP_PLUGIN } from './python.js';
 import { PHP_HTTP_PLUGIN } from './php.js';
@@ -14,6 +15,11 @@ import { JAVASCRIPT_HTTP_PLUGIN, TYPESCRIPT_HTTP_PLUGIN, TSX_HTTP_PLUGIN } from
  * new language, drop a `http-patterns/<lang>.ts` that exports a
  * `HttpLanguagePlugin`, import it here and register the extension(s).
  * No edits to `http-route-extractor.ts` are required.
+ *
+ * Optional grammar plugins (e.g. `kotlin.ts`, which depends on the
+ * optionalDependency `tree-sitter-kotlin`) export `null` when the
+ * native binding is unavailable; we skip registration in that case so
+ * a missing optional grammar never crashes the orchestrator.
  */
 const REGISTRY = {
     '.java': JAVA_HTTP_PLUGIN,
@@ -25,16 +31,25 @@ const REGISTRY = {
     '.ts': TYPESCRIPT_HTTP_PLUGIN,
     '.tsx': TSX_HTTP_PLUGIN,
 };
+if (KOTLIN_HTTP_PLUGIN) {
+    REGISTRY['.kt'] = KOTLIN_HTTP_PLUGIN;
+    REGISTRY['.kts'] = KOTLIN_HTTP_PLUGIN;
+}
 /**
  * Glob for files worth scanning for HTTP routes. Kept alongside the
  * registry so adding a new language widens the glob in one edit.
  *
+ * `.kt`/`.kts` are always present in the glob even when the optional
+ * `tree-sitter-kotlin` grammar isn't installed — `getPluginForFile`
+ * will return `undefined` for those files in that case, so the
+ * orchestrator simply skips them at scan time without erroring.
+ *
  * `.vue` / `.svelte` files are intentionally omitted for the source-scan
  * path — they need their own grammar-aware extraction and the existing
  * regex fallback for them was never very accurate. The graph-assisted
  * Strategy A still handles them via the ingestion pipeline.
  */
-export const HTTP_SCAN_GLOB = '**/*.{ts,tsx,js,jsx,java,go,py,php}';
+export const HTTP_SCAN_GLOB = '**/*.{ts,tsx,js,jsx,java,kt,kts,go,py,php}';
 /**
  * Return the HTTP plugin registered for the given file's extension,
  * or `undefined` if the extension is not registered.

package/dist/core/group/extractors/http-patterns/java.js

@@ -20,6 +20,19 @@ const METHOD_ANNOTATION_TO_HTTP = {
     PatchMapping: 'PATCH',
 };
 // ─── Provider: Spring class-level @RequestMapping prefix ──────────────
+// Two patterns are needed because the AST shape differs depending on
+// whether the annotation uses a positional argument or a named one:
+//   @RequestMapping("/api")          → (annotation_argument_list (string_literal))
+//   @RequestMapping(path = "/api")   → (annotation_argument_list (element_value_pair key:(identifier) value:(string_literal)))
+//   @RequestMapping(value = "/api")  → same as above
+//
+// The named-argument pattern MUST constrain the `key` field to the route
+// member names (`path`/`value`); without it, the query also captures
+// non-route attributes such as `produces`, `consumes`, `headers`, `name`,
+// `params` (their right-hand string literals would be mis-extracted as
+// route prefixes — e.g. `produces = "application/json"` would corrupt
+// every method route under that controller). The sibling
+// `topic-patterns/java.ts` uses the same `key:` constraint approach.
 const SPRING_CLASS_PREFIX_PATTERNS = compilePatterns({
     name: 'java-spring-class-prefix',
     language: Java,
@@ -34,9 +47,26 @@ const SPRING_CLASS_PREFIX_PATTERNS = compilePatterns({
               arguments: (annotation_argument_list (string_literal) @prefix)))) @class
       `,
         },
+        {
+            meta: {},
+            query: `
+        (class_declaration
+          (modifiers
+            (annotation
+              name: (identifier) @ann (#eq? @ann "RequestMapping")
+              arguments: (annotation_argument_list
+                (element_value_pair
+                  key: (identifier) @key (#match? @key "^(path|value)$")
+                  value: (string_literal) @prefix))))) @class
+      `,
+        },
     ],
 });
 // ─── Provider: Spring @(Get|Post|...)Mapping method annotations ───────
+// Same dual-pattern approach: positional vs named argument. The named
+// pattern restricts the annotation member name to `path`/`value` to
+// avoid capturing unrelated string-valued attributes
+// (`produces`, `consumes`, `headers`, `name`, `params`, ...).
 const SPRING_METHOD_ROUTE_PATTERNS = compilePatterns({
     name: 'java-spring-method-route',
     language: Java,
@@ -52,6 +82,20 @@ const SPRING_METHOD_ROUTE_PATTERNS = compilePatterns({
           name: (identifier) @method_name) @method
       `,
         },
+        {
+            meta: {},
+            query: `
+        (method_declaration
+          (modifiers
+            (annotation
+              name: (identifier) @ann (#match? @ann "^(Get|Post|Put|Delete|Patch)Mapping$")
+              arguments: (annotation_argument_list
+                (element_value_pair
+                  key: (identifier) @key (#match? @key "^(path|value)$")
+                  value: (string_literal) @path))))
+          name: (identifier) @method_name) @method
+      `,
+        },
     ],
 });
 // ─── Consumer: Spring RestTemplate (object-named + method-named) ──────

package/dist/core/group/extractors/http-patterns/kotlin.d.ts

@@ -0,0 +1,8 @@
+import type { HttpLanguagePlugin } from './types.js';
+/**
+ * The exported plugin is `null` when tree-sitter-kotlin's native
+ * binding is unavailable. `http-patterns/index.ts` checks for null
+ * before registering `.kt`/`.kts` so missing optional grammars never
+ * crash the orchestrator.
+ */
+export declare const KOTLIN_HTTP_PLUGIN: HttpLanguagePlugin | null;

package/dist/core/group/extractors/http-patterns/kotlin.js

@@ -0,0 +1,227 @@
+import { createRequire } from 'node:module';
+import { compilePatterns, runCompiledPatterns, unquoteLiteral, } from '../tree-sitter-scanner.js';
+/**
+ * Kotlin HTTP plugin (Spring providers).
+ *
+ * Mirrors the Java plugin for Spring `@RequestMapping` class prefixes
+ * and `@(Get|Post|...)Mapping` method annotations on Kotlin Spring
+ * Boot controllers. Both positional shorthand (`@GetMapping("/x")`)
+ * and named annotation arguments (`@GetMapping(value = "/x")` and
+ * `@GetMapping(path = "/x")`) are supported.
+ *
+ * Consumer detection (RestTemplate / WebClient / OkHttp) is intentionally
+ * out of scope for this plugin — Kotlin call-site ASTs are sufficiently
+ * different from Java's `method_invocation` shape that they warrant a
+ * separate, focused follow-up.
+ *
+ * tree-sitter-kotlin (fwcd) AST shapes used here:
+ *   class_declaration
+ *     modifiers
+ *       annotation
+ *         constructor_invocation
+ *           user_type → type_identifier   ← annotation name
+ *           value_arguments
+ *             value_argument
+ *               (simple_identifier  "=")? ← absent for positional, present for named
+ *               string_literal
+ *     type_identifier                     ← class name
+ *
+ * tree-sitter-kotlin is an optional npm dependency — when its native
+ * binding is unavailable the plugin gracefully exports `null` and
+ * `http-patterns/index.ts` skips registration for `.kt`/`.kts` files.
+ */
+const _require = createRequire(import.meta.url);
+/** Loaded lazily; null when the grammar binding isn't installed. */
+let Kotlin = null;
+try {
+    Kotlin = _require('tree-sitter-kotlin');
+}
+catch {
+    Kotlin = null;
+}
+const METHOD_ANNOTATION_TO_HTTP = {
+    GetMapping: 'GET',
+    PostMapping: 'POST',
+    PutMapping: 'PUT',
+    DeleteMapping: 'DELETE',
+    PatchMapping: 'PATCH',
+};
+/**
+ * Build the plugin only if the Kotlin grammar is available. Compiling
+ * the queries against a null grammar would throw at module load time
+ * and abort the whole http-route-extractor module.
+ */
+function buildKotlinPlugin(language) {
+    // ─── Provider: Spring class-level @RequestMapping prefix ──────────────
+    // Two patterns mirror the Java plugin's positional vs named split:
+    //   @RequestMapping("/api")          → value_argument has string_literal as its first named child
+    //   @RequestMapping(path = "/api")   → value_argument has [simple_identifier @key, string_literal]
+    //   @RequestMapping(value = "/api")  → same as above, with key="value"
+    //
+    // Tree-sitter-kotlin grammar (fwcd 0.3.8) does NOT have a separate
+    // node for named arguments — both positional and named forms share
+    // `value_argument`. The positional pattern uses the immediate-child
+    // anchor `.` so it only matches when the string_literal is the FIRST
+    // named child (i.e. no preceding simple_identifier "=" prefix). The
+    // named pattern explicitly captures the simple_identifier and uses
+    // `#match?` to restrict it to `path`/`value`, matching the same
+    // safety bar that the Java plugin enforces (see java.ts and the
+    // sibling topic-patterns/java.ts for the analogous constraint).
+    //
+    // Without the `key:` constraint the named query would also capture
+    // unrelated attributes like `produces`, `consumes`, `headers`,
+    // `name`, `params` — emitting bogus route contracts (a regression
+    // identical to the one Claude flagged on PR #1834 for Java).
+    const SPRING_CLASS_PREFIX_PATTERNS = compilePatterns({
+        name: 'kotlin-spring-class-prefix',
+        language,
+        patterns: [
+            {
+                meta: {},
+                query: `
+          (class_declaration
+            (modifiers
+              (annotation
+                (constructor_invocation
+                  (user_type (type_identifier) @ann (#eq? @ann "RequestMapping"))
+                  (value_arguments
+                    (value_argument . (string_literal) @prefix)))))
+            (type_identifier) @cls) @class
+        `,
+            },
+            {
+                meta: {},
+                query: `
+          (class_declaration
+            (modifiers
+              (annotation
+                (constructor_invocation
+                  (user_type (type_identifier) @ann (#eq? @ann "RequestMapping"))
+                  (value_arguments
+                    (value_argument
+                      (simple_identifier) @key (#match? @key "^(path|value)$")
+                      (string_literal) @prefix)))))
+            (type_identifier) @cls) @class
+        `,
+            },
+        ],
+    });
+    // ─── Provider: Spring @(Get|Post|...)Mapping method annotations ───────
+    // Same dual-pattern positional/named approach. The Kotlin AST puts the
+    // function name (`simple_identifier`) outside the `modifiers` subtree,
+    // so we capture it from `function_declaration` directly.
+    const SPRING_METHOD_ROUTE_PATTERNS = compilePatterns({
+        name: 'kotlin-spring-method-route',
+        language,
+        patterns: [
+            {
+                meta: {},
+                query: `
+          (function_declaration
+            (modifiers
+              (annotation
+                (constructor_invocation
+                  (user_type (type_identifier) @ann (#match? @ann "^(Get|Post|Put|Delete|Patch)Mapping$"))
+                  (value_arguments
+                    (value_argument . (string_literal) @path)))))
+            (simple_identifier) @method_name) @method
+        `,
+            },
+            {
+                meta: {},
+                query: `
+          (function_declaration
+            (modifiers
+              (annotation
+                (constructor_invocation
+                  (user_type (type_identifier) @ann (#match? @ann "^(Get|Post|Put|Delete|Patch)Mapping$"))
+                  (value_arguments
+                    (value_argument
+                      (simple_identifier) @key (#match? @key "^(path|value)$")
+                      (string_literal) @path)))))
+            (simple_identifier) @method_name) @method
+        `,
+            },
+        ],
+    });
+    /**
+     * Find the nearest enclosing class_declaration ancestor for a node, or
+     * null if the node is top-level. Mirrors the Java plugin's helper.
+     */
+    function findEnclosingClass(node) {
+        let cur = node.parent;
+        while (cur) {
+            if (cur.type === 'class_declaration')
+                return cur;
+            cur = cur.parent;
+        }
+        return null;
+    }
+    /**
+     * Join a class-level prefix and a method-level path. Identical
+     * semantics to the Java plugin: strip leading/trailing slashes on
+     * the prefix, strip leading slashes on the method path, ensure a
+     * single slash between them.
+     */
+    function joinPath(prefix, methodPath) {
+        const cleanPrefix = prefix.replace(/^\/+/, '').replace(/\/+$/, '');
+        const cleanSub = methodPath.replace(/^\/+/, '');
+        if (!cleanPrefix)
+            return `/${cleanSub}`;
+        return `/${cleanPrefix}/${cleanSub}`;
+    }
+    return {
+        name: 'kotlin-http',
+        language,
+        scan(tree) {
+            const out = [];
+            // ─── Class prefixes ─────────────────────────────────────────────
+            const prefixByClassId = new Map();
+            for (const match of runCompiledPatterns(SPRING_CLASS_PREFIX_PATTERNS, tree)) {
+                const prefixNode = match.captures.prefix;
+                const classNode = match.captures.class;
+                if (!prefixNode || !classNode)
+                    continue;
+                const prefix = unquoteLiteral(prefixNode.text);
+                if (prefix !== null)
+                    prefixByClassId.set(classNode.id, prefix);
+            }
+            // ─── Method routes ──────────────────────────────────────────────
+            for (const match of runCompiledPatterns(SPRING_METHOD_ROUTE_PATTERNS, tree)) {
+                const annNode = match.captures.ann;
+                const pathNode = match.captures.path;
+                const nameNode = match.captures.method_name;
+                const methodNode = match.captures.method;
+                if (!annNode || !pathNode || !methodNode)
+                    continue;
+                const httpMethod = METHOD_ANNOTATION_TO_HTTP[annNode.text];
+                if (!httpMethod)
+                    continue;
+                const rawPath = unquoteLiteral(pathNode.text);
+                if (rawPath === null)
+                    continue;
+                const enclosingClass = findEnclosingClass(methodNode);
+                const prefix = enclosingClass ? (prefixByClassId.get(enclosingClass.id) ?? '') : '';
+                const fullPath = joinPath(prefix, rawPath);
+                out.push({
+                    role: 'provider',
+                    framework: 'spring',
+                    method: httpMethod,
+                    path: fullPath,
+                    name: nameNode?.text ?? null,
+                    confidence: 0.8,
+                });
+            }
+            return out;
+        },
+    };
+}
+/**
+ * The exported plugin is `null` when tree-sitter-kotlin's native
+ * binding is unavailable. `http-patterns/index.ts` checks for null
+ * before registering `.kt`/`.kts` so missing optional grammars never
+ * crash the orchestrator.
+ */
+export const KOTLIN_HTTP_PLUGIN = Kotlin
+    ? buildKotlinPlugin(Kotlin)
+    : null;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "gitnexus",
-  "version": "1.6.6-rc.73",
+  "version": "1.6.6-rc.75",
   "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",