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

refactor(cfg): standardize harvester API table()->bindingTable() (#2195 U9)

Browse source 
The binding-table accessor was named table() in the C/C++/C#/Go harvests
but bindingTable() in the other 7. Rename the 4 (definitions + their
visitor call sites) to the majority name bindingTable(). Pure rename; the
4 visitor suites stay green and tsc confirms no call site was missed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
Gergo Magyar 2026-06-14 16:20:23 +00:00
parent fdaba53229
commit 7192036183
8 changed files with 13 additions and 13 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
gitnexus/src/core/ingestion/cfg/visitors/c-cpp-harvest.ts
View file

@ -116,7 +116,7 @@ export class CCppHarvester {
}
/** The completed binding table — pass to `CfgBuilder.finish`. */
table(): readonly BindingEntry[] {
bindingTable(): readonly BindingEntry[] {
return this.bindings;
}

4
gitnexus/src/core/ingestion/cfg/visitors/c-cpp.ts
View file

@ -738,11 +738,11 @@ function buildFunctionCfg(
walk.finishGotos();
if (!res) {
builder.edge(builder.entryIndex, builder.exitIndex, 'seq'); // empty body
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
}
builder.edge(builder.entryIndex, res.entry, 'seq');
builder.connect(res.exits, builder.exitIndex, 'seq'); // normal fall-off → EXIT
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
} catch (err) {
// Never throw out of buildFunctionCfg — a malformed AST shape must skip only
// this one function's CFG, never drop the whole file's language group (R4).

2
gitnexus/src/core/ingestion/cfg/visitors/csharp-harvest.ts
View file

@ -112,7 +112,7 @@ export class CsharpHarvester {
}
/** The completed binding table — pass to `CfgBuilder.finish`. */
table(): readonly BindingEntry[] {
bindingTable(): readonly BindingEntry[] {
return this.bindings;
}

6
gitnexus/src/core/ingestion/cfg/visitors/csharp.ts
View file

@ -875,18 +875,18 @@ function buildFunctionCfg(fnNode: SyntaxNode, filePath: string): FunctionCfg | u
);
builder.edge(builder.entryIndex, blk, 'seq');
builder.edge(blk, builder.exitIndex, 'return');
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
}
const walk = new CsharpCfgWalk(builder, harvest);
const res = walk.visitSeq(body.namedChildren.filter((c) => c.type !== 'comment'));
if (!res) {
builder.edge(builder.entryIndex, builder.exitIndex, 'seq'); // empty body
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
}
builder.edge(builder.entryIndex, res.entry, 'seq');
builder.connect(res.exits, builder.exitIndex, 'seq'); // normal fall-off → EXIT
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
} catch (err) {
// Never throw out of buildFunctionCfg — a malformed AST shape must skip only
// this one function's CFG, never drop the whole file's language group (R4).

2
gitnexus/src/core/ingestion/cfg/visitors/go-harvest.ts
View file

@ -133,7 +133,7 @@ export class GoHarvester {
}
/** The completed binding table — pass to `CfgBuilder.finish`. */
table(): readonly BindingEntry[] {
bindingTable(): readonly BindingEntry[] {
return this.bindings;
}

2
gitnexus/src/core/ingestion/cfg/visitors/go.ts
View file

@ -869,7 +869,7 @@ function buildFunctionCfg(fnNode: SyntaxNode, filePath: string): FunctionCfg | u
const afterDefers = walk.finishDefers(normalExits);
builder.connect(afterDefers, builder.exitIndex, 'seq');
walk.flushGotos(builder);
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
} catch (err) {
// Never throw out of buildFunctionCfg — a malformed AST shape must skip only
// this one function's CFG, never drop the whole file's language group (R4).

2
gitnexus/src/core/ingestion/cfg/visitors/java-harvest.ts
View file

@ -112,7 +112,7 @@ export class JavaHarvester {
}
/** The completed binding table — pass to `CfgBuilder.finish`. */
table(): readonly BindingEntry[] {
bindingTable(): readonly BindingEntry[] {
return this.bindings;
}

6
gitnexus/src/core/ingestion/cfg/visitors/java.ts
View file

@ -961,18 +961,18 @@ function buildFunctionCfg(fnNode: SyntaxNode, filePath: string): FunctionCfg | u
);
builder.edge(builder.entryIndex, blk, 'seq');
builder.edge(blk, builder.exitIndex, 'return');
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
}
const walk = new JavaCfgWalk(builder, harvest);
const res = walk.visitSeq(body.namedChildren.filter((c) => !isComment(c)));
if (!res) {
builder.edge(builder.entryIndex, builder.exitIndex, 'seq'); // empty body
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
}
builder.edge(builder.entryIndex, res.entry, 'seq');
builder.connect(res.exits, builder.exitIndex, 'seq'); // normal fall-off → EXIT
return builder.finish(harvest.table());
return builder.finish(harvest.bindingTable());
} catch (err) {
// Never throw out of buildFunctionCfg — a malformed AST shape must skip only
// this one function's CFG, never drop the whole file's language group (R4).