hana-linter 1.0.2 → 1.1.0

package/CHANGELOG.md

@@ -4,6 +4,20 @@ All notable changes to this project will be documented in this file.
 
 ---
 
+### 1.1.0
+
+- Console output is now grouped by file, then by failed rule, reducing noise when multiple violations exist in the same file
+- Added line numbers to content-lint violations (fields, input/output parameters) so users can jump directly to the offending identifier in their editor
+- All four Chevrotain visitors (`hdbtable`, `hdbview`, `hdbprocedure`, `hdbfunction`) now capture `token.startLine` from the parsed token and propagate it through `ExtractedSubject` and `LintIssue`
+
+---
+
+### 1.0.3
+
+- Fixed build: `src/assets/.hana-linter.json` is now correctly copied to `dist/assets/` during `pnpm build` via `copyfiles`, ensuring the default config template is bundled in the published package
+
+---
+
 ### 1.0.2
 
 - Added Chevrotain-based `.hdbfunction` parser; content-linting of function input parameters is now fully reliable

package/dist/assets/.hana-linter.json

@@ -0,0 +1,224 @@
+{
+    "rootDir": "db",
+    "ignoredDirectories": ["node_modules", ".git", "gen"],
+    "extensionRuleSets": [
+        {
+            "extension": "hdb*",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Upper snake case only",
+                        "pattern": "^[A-Z0-9]+(?:_[A-Z0-9]+)*$"
+                    },
+                    {
+                        "description": "Max length 30",
+                        "pattern": "^.{1,30}$",
+                        "flags": "u"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbtable",
+            "folderName": "tables",
+            "groups": {
+                "any": [
+                    {
+                        "description": "Prefix T_",
+                        "pattern": "^T_.+"
+                    },
+                    {
+                        "description": "Prefix TX_",
+                        "pattern": "^TX_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbview",
+            "folderName": "sqlviews",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix V_",
+                        "pattern": "^V_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbfunction",
+            "folderName": "functions",
+            "groups": {
+                "any": [
+                    {
+                        "description": "Prefix TF_",
+                        "pattern": "^TF_.+"
+                    },
+                    {
+                        "description": "Prefix SF_",
+                        "pattern": "^SF_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbprocedure",
+            "folderName": "procedures",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix PR_",
+                        "pattern": "^PR_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbcalculationview",
+            "folderName": "models",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix CV_",
+                        "pattern": "^CV_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbanalyticalprivilege",
+            "folderName": "authorizations",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix AP_",
+                        "pattern": "^AP_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbrole",
+            "folderName": "roles",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix R_",
+                        "pattern": "^R_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbsequence",
+            "folderName": "sequences",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix S_",
+                        "pattern": "^S_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbtabletype",
+            "folderName": "tabletypes",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix TT_",
+                        "pattern": "^TT_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbconstraint",
+            "folderName": "constraints",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix C_",
+                        "pattern": "^C_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbschedulerjob",
+            "folderName": "jobs",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix J_",
+                        "pattern": "^J_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbindex",
+            "folderName": "indexes",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix IDX_",
+                        "pattern": "^IDX_.+"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbtrigger",
+            "folderName": "triggers",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Prefix TR_",
+                        "pattern": "^TR_.+"
+                    }
+                ]
+            }
+        }
+    ],
+    "contentRuleSets": [
+        {
+            "extension": ".hdbtable",
+            "target": "field",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Field names in uppercase snake case",
+                        "pattern": "^[A-Z0-9]+(?:_[A-Z0-9]+)*$"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbprocedure",
+            "target": "inputParameter",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Input parameters prefixed with IP_",
+                        "pattern": "^IP_[A-Z0-9_]+$"
+                    }
+                ]
+            }
+        },
+        {
+            "extension": ".hdbprocedure",
+            "target": "outputParameter",
+            "groups": {
+                "all": [
+                    {
+                        "description": "Output parameters prefixed with OP_",
+                        "pattern": "^OP_[A-Z0-9_]+$"
+                    }
+                ]
+            }
+        }
+    ]
+}

package/dist/content-lint.js

@@ -70,6 +70,7 @@ function evaluateAllRules(filePath, extension, subject, rules) {
                 extension,
                 subjectType: subject.type,
                 subjectName: subject.name,
+                lineNumber: subject.lineNumber,
                 failedRuleDescription: rule.description,
                 failedPattern: toRegexLiteral(rule)
             });
@@ -89,6 +90,7 @@ function evaluateAnyRules(filePath, extension, subject, rules) {
             extension,
             subjectType: subject.type,
             subjectName: subject.name,
+            lineNumber: subject.lineNumber,
             failedRuleDescription: 'At least one OR-group rule must match: ' + rules.map((rule) => rule.description).join(' | '),
             failedPattern: rules.map(toRegexLiteral).join(' OR ')
         }

package/dist/parsers/hdbfunction/__tests__/extractFunctionParameters.test.js

@@ -22,8 +22,8 @@ function inputs(ddl) {
             ) RETURNS NVARCHAR(100) AS BEGIN END
         `;
         (0, vitest_1.expect)((0, index_1.extractFunctionParameters)(ddl)).toEqual([
-            { type: 'inputParameter', name: 'IV_CUSTOMER_ID' },
-            { type: 'inputParameter', name: 'IV_DATE' }
+            { type: 'inputParameter', name: 'IV_CUSTOMER_ID', lineNumber: 3 },
+            { type: 'inputParameter', name: 'IV_DATE', lineNumber: 4 }
         ]);
     });
     (0, vitest_1.it)('does not produce outputParameter entries for any parameter', () => {
@@ -46,7 +46,7 @@ function inputs(ddl) {
                 IN IV_STATUS NVARCHAR(1)
             ) RETURNS TABLE (ID INTEGER, NAME NVARCHAR(100)) AS BEGIN END
         `;
-        (0, vitest_1.expect)((0, index_1.extractFunctionParameters)(ddl)).toEqual([{ type: 'inputParameter', name: 'IV_STATUS' }]);
+        (0, vitest_1.expect)((0, index_1.extractFunctionParameters)(ddl)).toEqual([{ type: 'inputParameter', name: 'IV_STATUS', lineNumber: 3 }]);
     });
     (0, vitest_1.it)('does not produce outputParameter entries for a table function', () => {
         const ddl = `
@@ -113,7 +113,7 @@ function inputs(ddl) {
             ) RETURNS INTEGER AS BEGIN END
         `;
         const result = (0, index_1.extractFunctionParameters)(ddl);
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT' });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT', lineNumber: 3 });
         (0, vitest_1.expect)(result.map((s) => s.name)).not.toContain('COL1');
         (0, vitest_1.expect)(result.map((s) => s.name)).not.toContain('COL2');
     });
@@ -168,8 +168,8 @@ function inputs(ddl) {
         `;
         const result = (0, index_1.extractFunctionParameters)(ddl);
         (0, vitest_1.expect)(result).toEqual([
-            { type: 'inputParameter', name: 'IV_STATUS' },
-            { type: 'inputParameter', name: 'IV_ID' }
+            { type: 'inputParameter', name: 'IV_STATUS', lineNumber: 2 },
+            { type: 'inputParameter', name: 'IV_ID', lineNumber: 2 }
         ]);
     });
 });
@@ -224,7 +224,8 @@ function inputs(ddl) {
         const ddl = `FUNCTION F (IN "IV_CUSTOMER_ID" NVARCHAR(10)) RETURNS INTEGER AS BEGIN END`;
         (0, vitest_1.expect)((0, index_1.extractFunctionParameters)(ddl)).toContainEqual({
             type: 'inputParameter',
-            name: 'IV_CUSTOMER_ID'
+            name: 'IV_CUSTOMER_ID',
+            lineNumber: 1
         });
     });
     (0, vitest_1.it)('handles mixed quoted and unquoted parameters', () => {

package/dist/parsers/hdbfunction/visitor.js

@@ -45,10 +45,10 @@ class HdbFunctionParameterVisitor extends BaseCstVisitorWithDefaults {
         const nameNodes = ctx['parameterName'];
         if (!nameNodes?.length || !nameNodes[0])
             return;
-        const name = this.extractName(nameNodes[0]);
-        if (!name)
+        const extracted = this.extractName(nameNodes[0]);
+        if (!extracted)
             return;
-        this.parameters.push({ type: 'inputParameter', name });
+        this.parameters.push({ type: 'inputParameter', name: extracted.name, lineNumber: extracted.lineNumber });
     }
     // -----------------------------------------------------------------------
     // tableColumnDefinition — no-op.
@@ -99,7 +99,8 @@ class HdbFunctionParameterVisitor extends BaseCstVisitorWithDefaults {
         if (!token)
             return undefined;
         const raw = token.image;
-        return raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        return { name, lineNumber: token.startLine };
     }
 }
 exports.HdbFunctionParameterVisitor = HdbFunctionParameterVisitor;

package/dist/parsers/hdbprocedure/__tests__/extractProcedureParameters.test.js

@@ -27,8 +27,8 @@ function outputs(ddl) {
             ) AS BEGIN END
         `;
         (0, vitest_1.expect)((0, index_1.extractProcedureParameters)(ddl)).toEqual([
-            { type: 'inputParameter', name: 'IV_CUSTOMER_ID' },
-            { type: 'inputParameter', name: 'IV_DATE' }
+            { type: 'inputParameter', name: 'IV_CUSTOMER_ID', lineNumber: 3 },
+            { type: 'inputParameter', name: 'IV_DATE', lineNumber: 4 }
         ]);
     });
     (0, vitest_1.it)('does not produce outputParameter entries for IN parameters', () => {
@@ -52,8 +52,8 @@ function outputs(ddl) {
             ) AS BEGIN END
         `;
         (0, vitest_1.expect)((0, index_1.extractProcedureParameters)(ddl)).toEqual([
-            { type: 'outputParameter', name: 'EV_COUNT' },
-            { type: 'outputParameter', name: 'EV_STATUS' }
+            { type: 'outputParameter', name: 'EV_COUNT', lineNumber: 3 },
+            { type: 'outputParameter', name: 'EV_STATUS', lineNumber: 4 }
         ]);
     });
     (0, vitest_1.it)('does not produce inputParameter entries for OUT parameters', () => {
@@ -68,15 +68,15 @@ function outputs(ddl) {
     (0, vitest_1.it)('produces both types for an INOUT scalar parameter', () => {
         const ddl = `PROCEDURE P (INOUT CV_STATUS NVARCHAR(1)) AS BEGIN END`;
         (0, vitest_1.expect)((0, index_1.extractProcedureParameters)(ddl)).toEqual([
-            { type: 'inputParameter', name: 'CV_STATUS' },
-            { type: 'outputParameter', name: 'CV_STATUS' }
+            { type: 'inputParameter', name: 'CV_STATUS', lineNumber: 1 },
+            { type: 'outputParameter', name: 'CV_STATUS', lineNumber: 1 }
         ]);
     });
     (0, vitest_1.it)('produces both types for an INOUT TABLE parameter', () => {
         const ddl = `PROCEDURE P (INOUT TV_RESULT TABLE (ID INTEGER)) AS BEGIN END`;
         const result = (0, index_1.extractProcedureParameters)(ddl);
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'TV_RESULT' });
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'outputParameter', name: 'TV_RESULT' });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'TV_RESULT', lineNumber: 1 });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'outputParameter', name: 'TV_RESULT', lineNumber: 1 });
     });
     (0, vitest_1.it)('yields exactly two entries for a single INOUT parameter', () => {
         const ddl = `PROCEDURE P (INOUT CV_X INTEGER) AS BEGIN END`;
@@ -94,7 +94,7 @@ function outputs(ddl) {
             ) AS BEGIN END
         `;
         const result = (0, index_1.extractProcedureParameters)(ddl);
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT' });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT', lineNumber: 3 });
         (0, vitest_1.expect)(result.map((s) => s.name)).not.toContain('COL1');
         (0, vitest_1.expect)(result.map((s) => s.name)).not.toContain('COL2');
     });
@@ -226,14 +226,16 @@ function outputs(ddl) {
         const ddl = `PROCEDURE P (IN "IV_CUSTOMER_ID" NVARCHAR(10)) AS BEGIN END`;
         (0, vitest_1.expect)((0, index_1.extractProcedureParameters)(ddl)).toContainEqual({
             type: 'inputParameter',
-            name: 'IV_CUSTOMER_ID'
+            name: 'IV_CUSTOMER_ID',
+            lineNumber: 1
         });
     });
     (0, vitest_1.it)('strips double-quotes from a quoted OUT parameter name', () => {
         const ddl = `PROCEDURE P (OUT "EV_RESULT" INTEGER) AS BEGIN END`;
         (0, vitest_1.expect)((0, index_1.extractProcedureParameters)(ddl)).toContainEqual({
             type: 'outputParameter',
-            name: 'EV_RESULT'
+            name: 'EV_RESULT',
+            lineNumber: 1
         });
     });
 });
@@ -355,10 +357,10 @@ function outputs(ddl) {
             ) AS BEGIN END
         `;
         const result = (0, index_1.extractProcedureParameters)(ddl);
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'IV_ID' });
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'outputParameter', name: 'EV_COUNT' });
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'CV_STATUS' });
-        (0, vitest_1.expect)(result).toContainEqual({ type: 'outputParameter', name: 'CV_STATUS' });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'IV_ID', lineNumber: 3 });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'outputParameter', name: 'EV_COUNT', lineNumber: 4 });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'inputParameter', name: 'CV_STATUS', lineNumber: 5 });
+        (0, vitest_1.expect)(result).toContainEqual({ type: 'outputParameter', name: 'CV_STATUS', lineNumber: 5 });
         (0, vitest_1.expect)(result).toHaveLength(4);
     });
     (0, vitest_1.it)('handles mixed scalar and TABLE parameters', () => {

package/dist/parsers/hdbprocedure/visitor.js

@@ -53,14 +53,14 @@ class HdbProcedureParameterVisitor extends BaseCstVisitorWithDefaults {
         const nameNodes = ctx['parameterName'];
         if (!nameNodes?.length || !nameNodes[0])
             return;
-        const name = this.extractName(nameNodes[0]);
-        if (!name)
+        const extracted = this.extractName(nameNodes[0]);
+        if (!extracted)
             return;
         if (isIn || isInout) {
-            this.parameters.push({ type: 'inputParameter', name });
+            this.parameters.push({ type: 'inputParameter', name: extracted.name, lineNumber: extracted.lineNumber });
         }
         if (isOut || isInout) {
-            this.parameters.push({ type: 'outputParameter', name });
+            this.parameters.push({ type: 'outputParameter', name: extracted.name, lineNumber: extracted.lineNumber });
         }
     }
     // -----------------------------------------------------------------------
@@ -101,7 +101,8 @@ class HdbProcedureParameterVisitor extends BaseCstVisitorWithDefaults {
         if (!token)
             return undefined;
         const raw = token.image;
-        return raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        return { name, lineNumber: token.startLine };
     }
 }
 exports.HdbProcedureParameterVisitor = HdbProcedureParameterVisitor;

package/dist/parsers/hdbtable/__tests__/extractTableColumns.test.js

@@ -22,9 +22,9 @@ function names(ddl) {
             )
         `;
         (0, vitest_1.expect)((0, index_1.extractTableColumns)(ddl)).toEqual([
-            { type: 'field', name: 'ID' },
-            { type: 'field', name: 'NAME' },
-            { type: 'field', name: 'CREATED_AT' }
+            { type: 'field', name: 'ID', lineNumber: 3 },
+            { type: 'field', name: 'NAME', lineNumber: 4 },
+            { type: 'field', name: 'CREATED_AT', lineNumber: 5 }
         ]);
     });
     (0, vitest_1.it)('does not include the constraint name in the result', () => {
@@ -108,7 +108,7 @@ function names(ddl) {
 (0, vitest_1.describe)('AC-4: quoted identifier normalisation', () => {
     (0, vitest_1.it)('strips surrounding double-quotes from a quoted column identifier', () => {
         const ddl = `COLUMN TABLE "MY_TABLE" ("MY_COLUMN" NVARCHAR(100))`;
-        (0, vitest_1.expect)((0, index_1.extractTableColumns)(ddl)).toContainEqual({ type: 'field', name: 'MY_COLUMN' });
+        (0, vitest_1.expect)((0, index_1.extractTableColumns)(ddl)).toContainEqual({ type: 'field', name: 'MY_COLUMN', lineNumber: 1 });
     });
     (0, vitest_1.it)('strips quotes even when the table name is also quoted', () => {
         const ddl = `COLUMN TABLE "SCHEMA"."T" ("COL" INTEGER)`;
@@ -269,8 +269,8 @@ function names(ddl) {
     (0, vitest_1.it)('handles a mix in the same table', () => {
         const ddl = `COLUMN TABLE T ("QUOTED" INTEGER, UNQUOTED NVARCHAR(10))`;
         (0, vitest_1.expect)((0, index_1.extractTableColumns)(ddl)).toEqual([
-            { type: 'field', name: 'QUOTED' },
-            { type: 'field', name: 'UNQUOTED' }
+            { type: 'field', name: 'QUOTED', lineNumber: 1 },
+            { type: 'field', name: 'UNQUOTED', lineNumber: 1 }
         ]);
     });
 });

package/dist/parsers/hdbtable/visitor.js

@@ -43,10 +43,11 @@ class HdbTableColumnVisitor extends BaseCstVisitorWithDefaults {
         if (!tokenElement) {
             return;
         }
-        const raw = tokenElement.image;
+        const token = tokenElement;
+        const raw = token.image;
         // Strip surrounding double-quotes from quoted identifiers ("MY_COL" → MY_COL).
         const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
-        this.columns.push({ type: 'field', name });
+        this.columns.push({ type: 'field', name, lineNumber: token.startLine });
     }
 }
 exports.HdbTableColumnVisitor = HdbTableColumnVisitor;

package/dist/parsers/hdbview/__tests__/extractViewColumns.test.js

@@ -41,8 +41,8 @@ function names(ddl) {
     (0, vitest_1.it)('extracts AS aliases from the top-level SELECT clause', () => {
         const ddl = `VIEW V_BAR AS SELECT T."CUST_ID" AS "ID", T."CUST_NAME" AS "NAME" FROM T`;
         (0, vitest_1.expect)((0, index_1.extractViewColumns)(ddl)).toEqual([
-            { type: 'field', name: 'ID' },
-            { type: 'field', name: 'NAME' }
+            { type: 'field', name: 'ID', lineNumber: 1 },
+            { type: 'field', name: 'NAME', lineNumber: 1 }
         ]);
     });
     (0, vitest_1.it)('handles unquoted alias names', () => {
@@ -158,11 +158,11 @@ function names(ddl) {
 (0, vitest_1.describe)('AC-6: quoted identifier normalisation', () => {
     (0, vitest_1.it)('strips double-quotes from quoted aliases', () => {
         const ddl = `VIEW V AS SELECT T.X AS "MY_ALIAS" FROM T`;
-        (0, vitest_1.expect)((0, index_1.extractViewColumns)(ddl)).toContainEqual({ type: 'field', name: 'MY_ALIAS' });
+        (0, vitest_1.expect)((0, index_1.extractViewColumns)(ddl)).toContainEqual({ type: 'field', name: 'MY_ALIAS', lineNumber: 1 });
     });
     (0, vitest_1.it)('strips double-quotes from quoted names in explicit column list', () => {
         const ddl = `VIEW V ("MY_COL") AS SELECT T.X FROM T`;
-        (0, vitest_1.expect)((0, index_1.extractViewColumns)(ddl)).toContainEqual({ type: 'field', name: 'MY_COL' });
+        (0, vitest_1.expect)((0, index_1.extractViewColumns)(ddl)).toContainEqual({ type: 'field', name: 'MY_COL', lineNumber: 1 });
     });
 });
 // ---------------------------------------------------------------------------

package/dist/parsers/hdbview/visitor.js

@@ -113,7 +113,7 @@ class HdbViewColumnVisitor extends BaseCstVisitorWithDefaults {
         const raw = token.image;
         // Strip surrounding double-quotes: "MY_COL" → MY_COL.
         const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
-        this.columns.push({ type: 'field', name });
+        this.columns.push({ type: 'field', name, lineNumber: token.startLine });
     }
 }
 exports.HdbViewColumnVisitor = HdbViewColumnVisitor;

package/dist/report.js

@@ -3,6 +3,7 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.printReport = printReport;
 /**
  * Print lint result report to stdout/stderr.
+ * Issues are grouped by file, then by failed rule.
  *
  * @param issues - Found naming violations.
  * @param filesToValidate - Number of candidate files processed.
@@ -14,15 +15,45 @@ function printReport(issues, filesToValidate) {
     }
     console.error(`❌ HANA naming lint failed. Violations: ${issues.length}`);
     console.error('');
+    // Group issues by file path.
+    const byFile = new Map();
     for (const issue of issues) {
-        console.error(`- File: ${issue.filePath}`);
-        console.error(`  Artifact: ${issue.artifactName}`);
-        if (issue.subjectType && issue.subjectName) {
-            console.error(`  Subject: ${issue.subjectType} (${issue.subjectName})`);
+        const bucket = byFile.get(issue.filePath);
+        if (bucket) {
+            bucket.push(issue);
+        }
+        else {
+            byFile.set(issue.filePath, [issue]);
+        }
+    }
+    for (const [filePath, fileIssues] of byFile) {
+        const count = fileIssues.length;
+        console.error(`File: ${filePath}  (${count} violation${count === 1 ? '' : 's'})`);
+        // Group file issues by failed rule description.
+        const byRule = new Map();
+        for (const issue of fileIssues) {
+            const key = `${issue.failedRuleDescription}||${issue.failedPattern}`;
+            const bucket = byRule.get(key);
+            if (bucket) {
+                bucket.push(issue);
+            }
+            else {
+                byRule.set(key, [issue]);
+            }
+        }
+        for (const ruleIssues of byRule.values()) {
+            const { failedRuleDescription, failedPattern } = ruleIssues[0];
+            console.error(`  Rule: "${failedRuleDescription}" (pattern: ${failedPattern})`);
+            for (const issue of ruleIssues) {
+                const location = issue.lineNumber !== undefined ? ` at line ${issue.lineNumber}` : '';
+                if (issue.subjectType && issue.subjectName) {
+                    console.error(`    - ${issue.subjectType} "${issue.subjectName}"${location}`);
+                }
+                else {
+                    console.error(`    - artifact "${issue.artifactName}"${location}`);
+                }
+            }
         }
-        console.error(`  Type: ${issue.extension}`);
-        console.error(`  Failed rule: ${issue.failedRuleDescription}`);
-        console.error(`  Expected regex: ${issue.failedPattern}`);
         console.error('');
     }
 }

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "hana-linter",
-    "version": "1.0.2",
+    "version": "1.1.0",
     "description": "Linter for SAP HANA artifacts in an SAP CAP project",
     "main": "./dist/index.js",
     "bin": {
@@ -17,7 +17,7 @@
         "url": "git+https://github.com/qualiture/hana-linter.git"
     },
     "scripts": {
-        "build": "tsc",
+        "build": "tsc && copyfiles -u 1 \"src/assets/.hana-linter.json\" dist",
         "test": "vitest run",
         "test:coverage": "vitest run --coverage"
     },
@@ -26,6 +26,7 @@
     "devDependencies": {
         "@types/node": "^25.9.3",
         "@vitest/coverage-v8": "^4.1.9",
+        "copyfiles": "^2.4.1",
         "vitest": "^4.1.9"
     },
     "dependencies": {

package/src/content-lint.ts

@@ -83,6 +83,7 @@ function evaluateAllRules(filePath: string, extension: string, subject: Extracte
                 extension,
                 subjectType: subject.type,
                 subjectName: subject.name,
+                lineNumber: subject.lineNumber,
                 failedRuleDescription: rule.description,
                 failedPattern: toRegexLiteral(rule)
             });
@@ -106,6 +107,7 @@ function evaluateAnyRules(filePath: string, extension: string, subject: Extracte
             extension,
             subjectType: subject.type,
             subjectName: subject.name,
+            lineNumber: subject.lineNumber,
             failedRuleDescription: 'At least one OR-group rule must match: ' + rules.map((rule) => rule.description).join(' | '),
             failedPattern: rules.map(toRegexLiteral).join(' OR ')
         }

package/src/parsers/hdbfunction/__tests__/extractFunctionParameters.test.ts

@@ -24,8 +24,8 @@ describe('AC-1: IN parameter extraction (scalar function)', () => {
             ) RETURNS NVARCHAR(100) AS BEGIN END
         `;
         expect(extractFunctionParameters(ddl)).toEqual([
-            { type: 'inputParameter', name: 'IV_CUSTOMER_ID' },
-            { type: 'inputParameter', name: 'IV_DATE' }
+            { type: 'inputParameter', name: 'IV_CUSTOMER_ID', lineNumber: 3 },
+            { type: 'inputParameter', name: 'IV_DATE', lineNumber: 4 }
         ]);
     });
 
@@ -52,7 +52,7 @@ describe('AC-2: IN parameter extraction (table function)', () => {
                 IN IV_STATUS NVARCHAR(1)
             ) RETURNS TABLE (ID INTEGER, NAME NVARCHAR(100)) AS BEGIN END
         `;
-        expect(extractFunctionParameters(ddl)).toEqual([{ type: 'inputParameter', name: 'IV_STATUS' }]);
+        expect(extractFunctionParameters(ddl)).toEqual([{ type: 'inputParameter', name: 'IV_STATUS', lineNumber: 3 }]);
     });
 
     it('does not produce outputParameter entries for a table function', () => {
@@ -128,7 +128,7 @@ describe('AC-5: TABLE-type IN parameter columns not extracted', () => {
             ) RETURNS INTEGER AS BEGIN END
         `;
         const result = extractFunctionParameters(ddl);
-        expect(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT' });
+        expect(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT', lineNumber: 3 });
         expect(result.map((s) => s.name)).not.toContain('COL1');
         expect(result.map((s) => s.name)).not.toContain('COL2');
     });
@@ -188,8 +188,8 @@ describe('AC-6: function body SQL does not pollute extraction', () => {
         `;
         const result = extractFunctionParameters(ddl);
         expect(result).toEqual([
-            { type: 'inputParameter', name: 'IV_STATUS' },
-            { type: 'inputParameter', name: 'IV_ID' }
+            { type: 'inputParameter', name: 'IV_STATUS', lineNumber: 2 },
+            { type: 'inputParameter', name: 'IV_ID', lineNumber: 2 }
         ]);
     });
 });
@@ -251,7 +251,8 @@ describe('AC-9: quoted identifier normalisation', () => {
         const ddl = `FUNCTION F (IN "IV_CUSTOMER_ID" NVARCHAR(10)) RETURNS INTEGER AS BEGIN END`;
         expect(extractFunctionParameters(ddl)).toContainEqual({
             type: 'inputParameter',
-            name: 'IV_CUSTOMER_ID'
+            name: 'IV_CUSTOMER_ID',
+            lineNumber: 1
         });
     });
 

package/src/parsers/hdbfunction/visitor.ts

@@ -49,10 +49,10 @@ export class HdbFunctionParameterVisitor extends BaseCstVisitorWithDefaults {
         const nameNodes = ctx['parameterName'] as CstNode[] | undefined;
         if (!nameNodes?.length || !nameNodes[0]) return;
 
-        const name = this.extractName(nameNodes[0]);
-        if (!name) return;
+        const extracted = this.extractName(nameNodes[0]);
+        if (!extracted) return;
 
-        this.parameters.push({ type: 'inputParameter', name });
+        this.parameters.push({ type: 'inputParameter', name: extracted.name, lineNumber: extracted.lineNumber });
     }
 
     // -----------------------------------------------------------------------
@@ -97,7 +97,7 @@ export class HdbFunctionParameterVisitor extends BaseCstVisitorWithDefaults {
     // Strips surrounding double-quotes from quoted identifiers.
     // -----------------------------------------------------------------------
 
-    private extractName(node: CstNode): string | undefined {
+    private extractName(node: CstNode): { name: string; lineNumber?: number } | undefined {
         if (!node.children) return undefined;
 
         // parameterName → identifier → Identifier | QuotedIdentifier
@@ -113,6 +113,7 @@ export class HdbFunctionParameterVisitor extends BaseCstVisitorWithDefaults {
         if (!token) return undefined;
 
         const raw = token.image;
-        return raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        return { name, lineNumber: token.startLine };
     }
 }

package/src/parsers/hdbprocedure/__tests__/extractProcedureParameters.test.ts

@@ -30,8 +30,8 @@ describe('AC-1: IN parameter extraction', () => {
             ) AS BEGIN END
         `;
         expect(extractProcedureParameters(ddl)).toEqual([
-            { type: 'inputParameter', name: 'IV_CUSTOMER_ID' },
-            { type: 'inputParameter', name: 'IV_DATE' }
+            { type: 'inputParameter', name: 'IV_CUSTOMER_ID', lineNumber: 3 },
+            { type: 'inputParameter', name: 'IV_DATE', lineNumber: 4 }
         ]);
     });
 
@@ -59,8 +59,8 @@ describe('AC-2: OUT parameter extraction', () => {
             ) AS BEGIN END
         `;
         expect(extractProcedureParameters(ddl)).toEqual([
-            { type: 'outputParameter', name: 'EV_COUNT' },
-            { type: 'outputParameter', name: 'EV_STATUS' }
+            { type: 'outputParameter', name: 'EV_COUNT', lineNumber: 3 },
+            { type: 'outputParameter', name: 'EV_STATUS', lineNumber: 4 }
         ]);
     });
 
@@ -78,16 +78,16 @@ describe('AC-3: INOUT parameter yields inputParameter and outputParameter', () =
     it('produces both types for an INOUT scalar parameter', () => {
         const ddl = `PROCEDURE P (INOUT CV_STATUS NVARCHAR(1)) AS BEGIN END`;
         expect(extractProcedureParameters(ddl)).toEqual([
-            { type: 'inputParameter', name: 'CV_STATUS' },
-            { type: 'outputParameter', name: 'CV_STATUS' }
+            { type: 'inputParameter', name: 'CV_STATUS', lineNumber: 1 },
+            { type: 'outputParameter', name: 'CV_STATUS', lineNumber: 1 }
         ]);
     });
 
     it('produces both types for an INOUT TABLE parameter', () => {
         const ddl = `PROCEDURE P (INOUT TV_RESULT TABLE (ID INTEGER)) AS BEGIN END`;
         const result = extractProcedureParameters(ddl);
-        expect(result).toContainEqual({ type: 'inputParameter', name: 'TV_RESULT' });
-        expect(result).toContainEqual({ type: 'outputParameter', name: 'TV_RESULT' });
+        expect(result).toContainEqual({ type: 'inputParameter', name: 'TV_RESULT', lineNumber: 1 });
+        expect(result).toContainEqual({ type: 'outputParameter', name: 'TV_RESULT', lineNumber: 1 });
     });
 
     it('yields exactly two entries for a single INOUT parameter', () => {
@@ -108,7 +108,7 @@ describe('AC-4: TABLE-type parameter columns not extracted', () => {
             ) AS BEGIN END
         `;
         const result = extractProcedureParameters(ddl);
-        expect(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT' });
+        expect(result).toContainEqual({ type: 'inputParameter', name: 'TV_INPUT', lineNumber: 3 });
         expect(result.map((s) => s.name)).not.toContain('COL1');
         expect(result.map((s) => s.name)).not.toContain('COL2');
     });
@@ -253,7 +253,8 @@ describe('AC-8: quoted identifier normalisation', () => {
         const ddl = `PROCEDURE P (IN "IV_CUSTOMER_ID" NVARCHAR(10)) AS BEGIN END`;
         expect(extractProcedureParameters(ddl)).toContainEqual({
             type: 'inputParameter',
-            name: 'IV_CUSTOMER_ID'
+            name: 'IV_CUSTOMER_ID',
+            lineNumber: 1
         });
     });
 
@@ -261,7 +262,8 @@ describe('AC-8: quoted identifier normalisation', () => {
         const ddl = `PROCEDURE P (OUT "EV_RESULT" INTEGER) AS BEGIN END`;
         expect(extractProcedureParameters(ddl)).toContainEqual({
             type: 'outputParameter',
-            name: 'EV_RESULT'
+            name: 'EV_RESULT',
+            lineNumber: 1
         });
     });
 });
@@ -401,10 +403,10 @@ describe('mixed parameter modes', () => {
             ) AS BEGIN END
         `;
         const result = extractProcedureParameters(ddl);
-        expect(result).toContainEqual({ type: 'inputParameter', name: 'IV_ID' });
-        expect(result).toContainEqual({ type: 'outputParameter', name: 'EV_COUNT' });
-        expect(result).toContainEqual({ type: 'inputParameter', name: 'CV_STATUS' });
-        expect(result).toContainEqual({ type: 'outputParameter', name: 'CV_STATUS' });
+        expect(result).toContainEqual({ type: 'inputParameter', name: 'IV_ID', lineNumber: 3 });
+        expect(result).toContainEqual({ type: 'outputParameter', name: 'EV_COUNT', lineNumber: 4 });
+        expect(result).toContainEqual({ type: 'inputParameter', name: 'CV_STATUS', lineNumber: 5 });
+        expect(result).toContainEqual({ type: 'outputParameter', name: 'CV_STATUS', lineNumber: 5 });
         expect(result).toHaveLength(4);
     });
 

package/src/parsers/hdbprocedure/visitor.ts

@@ -58,14 +58,14 @@ export class HdbProcedureParameterVisitor extends BaseCstVisitorWithDefaults {
         const nameNodes = ctx['parameterName'] as CstNode[] | undefined;
         if (!nameNodes?.length || !nameNodes[0]) return;
 
-        const name = this.extractName(nameNodes[0]);
-        if (!name) return;
+        const extracted = this.extractName(nameNodes[0]);
+        if (!extracted) return;
 
         if (isIn || isInout) {
-            this.parameters.push({ type: 'inputParameter', name });
+            this.parameters.push({ type: 'inputParameter', name: extracted.name, lineNumber: extracted.lineNumber });
         }
         if (isOut || isInout) {
-            this.parameters.push({ type: 'outputParameter', name });
+            this.parameters.push({ type: 'outputParameter', name: extracted.name, lineNumber: extracted.lineNumber });
         }
     }
 
@@ -98,7 +98,7 @@ export class HdbProcedureParameterVisitor extends BaseCstVisitorWithDefaults {
     // Strips surrounding double-quotes from quoted identifiers.
     // -----------------------------------------------------------------------
 
-    private extractName(node: CstNode): string | undefined {
+    private extractName(node: CstNode): { name: string; lineNumber?: number } | undefined {
         if (!node.children) return undefined;
 
         // parameterName → identifier → Identifier | QuotedIdentifier
@@ -114,6 +114,7 @@ export class HdbProcedureParameterVisitor extends BaseCstVisitorWithDefaults {
         if (!token) return undefined;
 
         const raw = token.image;
-        return raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
+        return { name, lineNumber: token.startLine };
     }
 }

package/src/parsers/hdbtable/__tests__/extractTableColumns.test.ts

@@ -24,9 +24,9 @@ describe('AC-1: standard column extraction', () => {
             )
         `;
         expect(extractTableColumns(ddl)).toEqual([
-            { type: 'field', name: 'ID' },
-            { type: 'field', name: 'NAME' },
-            { type: 'field', name: 'CREATED_AT' }
+            { type: 'field', name: 'ID', lineNumber: 3 },
+            { type: 'field', name: 'NAME', lineNumber: 4 },
+            { type: 'field', name: 'CREATED_AT', lineNumber: 5 }
         ]);
     });
 
@@ -121,7 +121,7 @@ describe('AC-3: line comment exclusion', () => {
 describe('AC-4: quoted identifier normalisation', () => {
     it('strips surrounding double-quotes from a quoted column identifier', () => {
         const ddl = `COLUMN TABLE "MY_TABLE" ("MY_COLUMN" NVARCHAR(100))`;
-        expect(extractTableColumns(ddl)).toContainEqual({ type: 'field', name: 'MY_COLUMN' });
+        expect(extractTableColumns(ddl)).toContainEqual({ type: 'field', name: 'MY_COLUMN', lineNumber: 1 });
     });
 
     it('strips quotes even when the table name is also quoted', () => {
@@ -309,8 +309,8 @@ describe('mixed quoted and unquoted identifiers', () => {
     it('handles a mix in the same table', () => {
         const ddl = `COLUMN TABLE T ("QUOTED" INTEGER, UNQUOTED NVARCHAR(10))`;
         expect(extractTableColumns(ddl)).toEqual([
-            { type: 'field', name: 'QUOTED' },
-            { type: 'field', name: 'UNQUOTED' }
+            { type: 'field', name: 'QUOTED', lineNumber: 1 },
+            { type: 'field', name: 'UNQUOTED', lineNumber: 1 }
         ]);
     });
 });

package/src/parsers/hdbtable/visitor.ts

@@ -50,10 +50,11 @@ export class HdbTableColumnVisitor extends BaseCstVisitorWithDefaults {
             return;
         }
 
-        const raw = (tokenElement as IToken).image;
+        const token = tokenElement as IToken;
+        const raw = token.image;
         // Strip surrounding double-quotes from quoted identifiers ("MY_COL" → MY_COL).
         const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
 
-        this.columns.push({ type: 'field', name });
+        this.columns.push({ type: 'field', name, lineNumber: token.startLine });
     }
 }

package/src/parsers/hdbview/__tests__/extractViewColumns.test.ts

@@ -49,8 +49,8 @@ describe('AC-2: SELECT alias extraction (no explicit column list)', () => {
     it('extracts AS aliases from the top-level SELECT clause', () => {
         const ddl = `VIEW V_BAR AS SELECT T."CUST_ID" AS "ID", T."CUST_NAME" AS "NAME" FROM T`;
         expect(extractViewColumns(ddl)).toEqual([
-            { type: 'field', name: 'ID' },
-            { type: 'field', name: 'NAME' }
+            { type: 'field', name: 'ID', lineNumber: 1 },
+            { type: 'field', name: 'NAME', lineNumber: 1 }
         ]);
     });
 
@@ -182,12 +182,12 @@ describe('AC-5: line comment exclusion', () => {
 describe('AC-6: quoted identifier normalisation', () => {
     it('strips double-quotes from quoted aliases', () => {
         const ddl = `VIEW V AS SELECT T.X AS "MY_ALIAS" FROM T`;
-        expect(extractViewColumns(ddl)).toContainEqual({ type: 'field', name: 'MY_ALIAS' });
+        expect(extractViewColumns(ddl)).toContainEqual({ type: 'field', name: 'MY_ALIAS', lineNumber: 1 });
     });
 
     it('strips double-quotes from quoted names in explicit column list', () => {
         const ddl = `VIEW V ("MY_COL") AS SELECT T.X FROM T`;
-        expect(extractViewColumns(ddl)).toContainEqual({ type: 'field', name: 'MY_COL' });
+        expect(extractViewColumns(ddl)).toContainEqual({ type: 'field', name: 'MY_COL', lineNumber: 1 });
     });
 });
 

package/src/parsers/hdbview/visitor.ts

@@ -125,6 +125,6 @@ export class HdbViewColumnVisitor extends BaseCstVisitorWithDefaults {
         const raw = token.image;
         // Strip surrounding double-quotes: "MY_COL" → MY_COL.
         const name = raw.startsWith('"') ? raw.slice(1, -1) : raw;
-        this.columns.push({ type: 'field', name });
+        this.columns.push({ type: 'field', name, lineNumber: token.startLine });
     }
 }

package/src/report.ts

@@ -2,6 +2,7 @@ import { LintIssue } from './types/issues';
 
 /**
  * Print lint result report to stdout/stderr.
+ * Issues are grouped by file, then by failed rule.
  *
  * @param issues - Found naming violations.
  * @param filesToValidate - Number of candidate files processed.
@@ -15,15 +16,47 @@ export function printReport(issues: readonly LintIssue[], filesToValidate: reado
     console.error(`❌ HANA naming lint failed. Violations: ${issues.length}`);
     console.error('');
 
+    // Group issues by file path.
+    const byFile = new Map<string, LintIssue[]>();
     for (const issue of issues) {
-        console.error(`- File: ${issue.filePath}`);
-        console.error(`  Artifact: ${issue.artifactName}`);
-        if (issue.subjectType && issue.subjectName) {
-            console.error(`  Subject: ${issue.subjectType} (${issue.subjectName})`);
+        const bucket = byFile.get(issue.filePath);
+        if (bucket) {
+            bucket.push(issue);
+        } else {
+            byFile.set(issue.filePath, [issue]);
         }
-        console.error(`  Type: ${issue.extension}`);
-        console.error(`  Failed rule: ${issue.failedRuleDescription}`);
-        console.error(`  Expected regex: ${issue.failedPattern}`);
+    }
+
+    for (const [filePath, fileIssues] of byFile) {
+        const count = fileIssues.length;
+        console.error(`File: ${filePath}  (${count} violation${count === 1 ? '' : 's'})`);
+
+        // Group file issues by failed rule description.
+        const byRule = new Map<string, LintIssue[]>();
+        for (const issue of fileIssues) {
+            const key = `${issue.failedRuleDescription}||${issue.failedPattern}`;
+            const bucket = byRule.get(key);
+            if (bucket) {
+                bucket.push(issue);
+            } else {
+                byRule.set(key, [issue]);
+            }
+        }
+
+        for (const ruleIssues of byRule.values()) {
+            const { failedRuleDescription, failedPattern } = ruleIssues[0]!;
+            console.error(`  Rule: "${failedRuleDescription}" (pattern: ${failedPattern})`);
+
+            for (const issue of ruleIssues) {
+                const location = issue.lineNumber !== undefined ? ` at line ${issue.lineNumber}` : '';
+                if (issue.subjectType && issue.subjectName) {
+                    console.error(`    - ${issue.subjectType} "${issue.subjectName}"${location}`);
+                } else {
+                    console.error(`    - artifact "${issue.artifactName}"${location}`);
+                }
+            }
+        }
+
         console.error('');
     }
 }

package/src/types/issues.ts

@@ -3,6 +3,7 @@ import type { ContentTarget } from './rules';
 export type ExtractedSubject = {
     readonly type: ContentTarget;
     readonly name: string;
+    readonly lineNumber?: number;
 };
 
 export type LintIssue = {
@@ -11,6 +12,7 @@ export type LintIssue = {
     readonly extension: string;
     readonly subjectType?: 'artifact' | 'field' | 'inputParameter' | 'outputParameter';
     readonly subjectName?: string;
+    readonly lineNumber?: number;
     readonly failedRuleDescription: string;
     readonly failedPattern: string;
 };