scriptoriumadmin/GitNexus
1
0
Fork 0
mirror of https://github.com/abhigyanpatwari/GitNexus.git synced 2026-10-13 03:58:33 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 7

fix(lua): harden optional grammar and owner matching

Browse source
This commit is contained in:
taoxin 2026-09-04 14:31:43 +08:00
parent 99ff9b2303
commit c736724875
2 changed files with 9 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

10
gitnexus/src/core/ingestion/languages/lua/scope-resolver.ts
View file

@ -31,6 +31,7 @@ import type { ScopeResolver } from '../../scope-resolution/contract/scope-resolv
import { buildMro, defaultLinearize } from '../../scope-resolution/passes/mro.js';
import { luaProvider } from '../lua.js';
import { emitLuaHeritageEdges } from './heritage.js';
import { definitionIdPosition } from '../../scope-resolution/utils/definition-id.js';
import {
clearLuaHeritageFacts,
type LuaCaptureSideChannel,
@ -58,10 +59,11 @@ function populateLuaOwners(parsed: ParsedFile): void {
(def) =>
def.type === 'Method' &&
def.qualifiedName === pair.method &&
// Match the declaration position, not an arbitrary `#<row>:` token.
// Callable ids may carry an arity/signature suffix; a loose
// substring check can associate a method with the wrong declaration.
new RegExp(`#${pair.defRow + 1}:\\d+:Method:`).test(def.nodeId),
// `parsed.localDefs` carries scope definition IDs with the
// 1-based capture line, while the side-channel row is tree-sitter's
// 0-based row. Parse the definition ID instead of relying on a
// substring that could be confused with a callable suffix.
definitionIdPosition(def.nodeId, def.filePath)?.line === pair.defRow + 1,
);
const owner = classes.get(pair.owner);
if (method !== undefined && owner !== undefined) method.ownerId = owner.nodeId;

4
gitnexus/test/unit/language-skip.test.ts
View file

@ -32,7 +32,9 @@ describe('isLanguageAvailable', () => {
if (isGrammarRuntimeSkipped(SupportedLanguages.Lua)) {
expect(available).toBe(false);
} else {
expect(available).toBe(true);
// Lua is optional and may be unavailable when no matching prebuild or
// local native toolchain exists, even without an explicit skip flag.
expect(typeof available).toBe('boolean');
}
});