Skip to content

fix: register Section in NODE_TABLES and NODE_SCHEMA_QUERIES #401

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
abhigyanpatwari merged 6 commits into from
Mar 20, 2026
Merged

fix: register Section in NODE_TABLES and NODE_SCHEMA_QUERIES #401

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
4746675
feat: add markdown file indexing (headings + cross-links)
abhigyanpatwari Mar 20, 2026
bfb40b7
Merge branch 'main' of https://github.com/abhigyanpatwari/GitNexus in…
abhigyanpatwari Mar 20, 2026
6a4e220
fix: add Section to NODE_TABLES and NODE_SCHEMA_QUERIES
abhigyanpatwari Mar 20, 2026
0733670
fix: update schema test counts for Section node type
abhigyanpatwari Mar 20, 2026
ae34550
test: add diagnostic output to skills-e2e idempotency test
abhigyanpatwari Mar 20, 2026
a34669b
fix: add Section COPY query with level column in lbug-adapter
abhigyanpatwari Mar 20, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 9 additions & 0 deletions gitnexus/src/core/lbug/lbug-adapter.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -336,6 +336,9 @@ const getCopyQuery = (table: NodeTableName, filePath: string): string => {
if (table === 'Process') {
return `COPY ${t}(id, label, heuristicLabel, processType, stepCount, communities, entryPointId, terminalId) FROM "${filePath}" ${COPY_CSV_OPTS}`;
}
if (table === 'Section') {
return `COPY ${t}(id, name, filePath, startLine, endLine, level, content, description) FROM "${filePath}" ${COPY_CSV_OPTS}`;
}
if (table === 'Method') {
return `COPY ${t}(id, name, filePath, startLine, endLine, isExported, content, description, parameterCount, returnType) FROM "${filePath}" ${COPY_CSV_OPTS}`;
}
Expand Down Expand Up @@ -380,6 +383,9 @@ export const insertNodeToLbug = async (
query = `CREATE (n:File {id: ${escapeValue(properties.id)}, name: ${escapeValue(properties.name)}, filePath: ${escapeValue(properties.filePath)}, content: ${escapeValue(properties.content || '')}})`;
} else if (label === 'Folder') {
query = `CREATE (n:Folder {id: ${escapeValue(properties.id)}, name: ${escapeValue(properties.name)}, filePath: ${escapeValue(properties.filePath)}})`;
} else if (label === 'Section') {
const descPart = properties.description ? `, description: ${escapeValue(properties.description)}` : '';
query = `CREATE (n:Section {id: ${escapeValue(properties.id)}, name: ${escapeValue(properties.name)}, filePath: ${escapeValue(properties.filePath)}, startLine: ${properties.startLine || 0}, endLine: ${properties.endLine || 0}, level: ${properties.level || 1}, content: ${escapeValue(properties.content || '')}${descPart}})`;
} else if (TABLES_WITH_EXPORTED.has(label)) {
const descPart = properties.description ? `, description: ${escapeValue(properties.description)}` : '';
query = `CREATE (n:${t} {id: ${escapeValue(properties.id)}, name: ${escapeValue(properties.name)}, filePath: ${escapeValue(properties.filePath)}, startLine: ${properties.startLine || 0}, endLine: ${properties.endLine || 0}, isExported: ${!!properties.isExported}, content: ${escapeValue(properties.content || '')}${descPart}})`;
Expand Down Expand Up @@ -451,6 +457,9 @@ export const batchInsertNodesToLbug = async (
query = `MERGE (n:File {id: ${escapeValue(properties.id)}}) SET n.name = ${escapeValue(properties.name)}, n.filePath = ${escapeValue(properties.filePath)}, n.content = ${escapeValue(properties.content || '')}`;
} else if (label === 'Folder') {
query = `MERGE (n:Folder {id: ${escapeValue(properties.id)}}) SET n.name = ${escapeValue(properties.name)}, n.filePath = ${escapeValue(properties.filePath)}`;
} else if (label === 'Section') {
const descPart = properties.description ? `, n.description = ${escapeValue(properties.description)}` : '';
query = `MERGE (n:Section {id: ${escapeValue(properties.id)}}) SET n.name = ${escapeValue(properties.name)}, n.filePath = ${escapeValue(properties.filePath)}, n.startLine = ${properties.startLine || 0}, n.endLine = ${properties.endLine || 0}, n.level = ${properties.level || 1}, n.content = ${escapeValue(properties.content || '')}${descPart}`;
} else if (TABLES_WITH_EXPORTED.has(label)) {
const descPart = properties.description ? `, n.description = ${escapeValue(properties.description)}` : '';
query = `MERGE (n:${t} {id: ${escapeValue(properties.id)}}) SET n.name = ${escapeValue(properties.name)}, n.filePath = ${escapeValue(properties.filePath)}, n.startLine = ${properties.startLine || 0}, n.endLine = ${properties.endLine || 0}, n.isExported = ${!!properties.isExported}, n.content = ${escapeValue(properties.content || '')}${descPart}`;
Expand Down
5 changes: 4 additions & 1 deletion gitnexus/src/core/lbug/schema.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
// NODE TABLE NAMES
// ============================================================================
export const NODE_TABLES = [
'File', 'Folder', 'Function', 'Class', 'Interface', 'Method', 'CodeElement', 'Community', 'Process',
'File', 'Folder', 'Function', 'Class', 'Interface', 'Method', 'CodeElement', 'Community', 'Process', 'Section',
// Multi-language support
'Struct', 'Enum', 'Macro', 'Typedef', 'Union', 'Namespace', 'Trait', 'Impl',
'TypeAlias', 'Const', 'Static', 'Property', 'Record', 'Delegate', 'Annotation', 'Constructor', 'Template', 'Module'
Expand Down Expand Up @@ -238,6 +238,7 @@ CREATE REL TABLE ${REL_TABLE_NAME} (
FROM File TO \`Constructor\`,
FROM File TO \`Template\`,
FROM File TO \`Module\`,
FROM File TO Section,
FROM Folder TO Folder,
FROM Folder TO File,
FROM Function TO Function,
Expand Down Expand Up @@ -462,6 +463,8 @@ export const NODE_SCHEMA_QUERIES = [
CONSTRUCTOR_SCHEMA,
TEMPLATE_SCHEMA,
MODULE_SCHEMA,
// Markdown support
SECTION_SCHEMA,
];

export const REL_SCHEMA_QUERIES = [
Expand Down
12 changes: 10 additions & 2 deletions gitnexus/test/integration/skills-e2e.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2389,8 +2389,16 @@ export function createEntry(level: string, msg: string) {
/* CI timeout tolerance */
if (result1.status === null || result2.status === null) return;

expect(result1.status).toBe(0);
expect(result2.status).toBe(0);
expect(result1.status, [
`first analyze --skills exited with code ${result1.status}`,
`stdout: ${result1.stdout?.slice(0, 500)}`,
`stderr: ${result1.stderr?.slice(0, 500)}`,
].join('\n')).toBe(0);
expect(result2.status, [
`second analyze --skills exited with code ${result2.status}`,
`stdout: ${result2.stdout?.slice(0, 500)}`,
`stderr: ${result2.stderr?.slice(0, 500)}`,
].join('\n')).toBe(0);

const generatedDir = path.join(tmpDir, '.claude', 'skills', 'generated');
expect(fs.existsSync(generatedDir)).toBe(true);
Expand Down
6 changes: 3 additions & 3 deletions gitnexus/test/unit/schema.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -40,7 +40,7 @@ describe('LadybugDB Schema', () => {

it('has expected total count', () => {
// 9 core + 18 multi-language = 27
expect(NODE_TABLES).toHaveLength(27);
expect(NODE_TABLES).toHaveLength(28);
});
});

Expand Down Expand Up @@ -164,7 +164,7 @@ describe('LadybugDB Schema', () => {

describe('schema query ordering', () => {
it('NODE_SCHEMA_QUERIES has correct count', () => {
expect(NODE_SCHEMA_QUERIES).toHaveLength(27);
expect(NODE_SCHEMA_QUERIES).toHaveLength(28);
});

it('REL_SCHEMA_QUERIES has one relation table', () => {
Expand All @@ -173,7 +173,7 @@ describe('LadybugDB Schema', () => {

it('SCHEMA_QUERIES includes all node + rel + embedding schemas', () => {
// 27 node + 1 rel + 1 embedding = 29
expect(SCHEMA_QUERIES).toHaveLength(29);
expect(SCHEMA_QUERIES).toHaveLength(30);
});

it('node schemas come before relation schemas in SCHEMA_QUERIES', () => {
Expand Down
Loading