@abaplint/cli 2.102.60 → 2.102.62

package/build/cli.js

@@ -1620,6 +1620,9 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.Token = void 0;
 const position_1 = __webpack_require__(/*! ../../../position */ "./node_modules/@abaplint/core/build/src/position.js");
 class Token {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.str}`;
+    }
     constructor(start, str) {
         this.start = start;
         this.str = str;
@@ -26313,12 +26316,16 @@ exports.SQLCompare = void 0;
 const Expressions = __webpack_require__(/*! ../../2_statements/expressions */ "./node_modules/@abaplint/core/build/src/abap/2_statements/expressions/index.js");
 const nodes_1 = __webpack_require__(/*! ../../nodes */ "./node_modules/@abaplint/core/build/src/abap/nodes/index.js");
 const basic_1 = __webpack_require__(/*! ../../types/basic */ "./node_modules/@abaplint/core/build/src/abap/types/basic/index.js");
+const source_1 = __webpack_require__(/*! ./source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/source.js");
 const sql_source_1 = __webpack_require__(/*! ./sql_source */ "./node_modules/@abaplint/core/build/src/abap/5_syntax/expressions/sql_source.js");
 class SQLCompare {
     runSyntax(node, scope, filename, tables) {
         var _a;
         let sourceType;
         let token;
+        for (const s of node.findAllExpressions(Expressions.SimpleSource3)) {
+            new source_1.Source().runSyntax(s, scope, filename);
+        }
         for (const s of node.findAllExpressions(Expressions.SQLSource)) {
             for (const child of s.getChildren()) {
                 if (child instanceof nodes_1.ExpressionNode) {
@@ -27123,6 +27130,9 @@ class ScopeData {
     }
 }
 class SpaghettiScopeNode extends ScopeData {
+    [Symbol.for("debug.description")]() {
+        return `SpaghettiSN ${this.identifier.sname} ${this.identifier.stype}`;
+    }
     constructor(identifier, parent) {
         super();
         this.identifier = identifier;
@@ -34707,6 +34717,9 @@ Object.defineProperty(exports, "__esModule", ({ value: true }));
 exports.TypedIdentifier = void 0;
 const _identifier_1 = __webpack_require__(/*! ../4_file_information/_identifier */ "./node_modules/@abaplint/core/build/src/abap/4_file_information/_identifier.js");
 class TypedIdentifier extends _identifier_1.Identifier {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.getName()}:${this.getType().constructor.name}`;
+    }
     static from(id, type, meta) {
         return new TypedIdentifier(id.getToken(), id.getFilename(), type, meta);
     }
@@ -42583,6 +42596,9 @@ const _abstract_object_1 = __webpack_require__(/*! ./_abstract_object */ "./node
 const xml_utils_1 = __webpack_require__(/*! ../xml_utils */ "./node_modules/@abaplint/core/build/src/xml_utils.js");
 const abap_parser_1 = __webpack_require__(/*! ../abap/abap_parser */ "./node_modules/@abaplint/core/build/src/abap/abap_parser.js");
 class ABAPObject extends _abstract_object_1.AbstractObject {
+    [Symbol.for("debug.description")]() {
+        return `${this.constructor.name} ${this.getName()}`;
+    }
     constructor(name) {
         super(name);
         this.parsed = [];
@@ -50095,7 +50111,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.sh"
-        return "2.102.60";
+        return "2.102.62";
     }
     getDDICReferences() {
         return this.ddicReferences;
@@ -52659,7 +52675,7 @@ class CheckSubrc extends _abap_rule_1.ABAPRule {
 
 If sy-dbcnt is checked after database statements, it is considered okay.
 
-"SELECT SINGLE @abap_true FROM " is considered as an existence check
+"SELECT SINGLE @abap_true FROM " is considered as an existence check, also "SELECT COUNT( * )" is considered okay
 
 If IS ASSIGNED is checked after assigning, it is considered okay.
 
@@ -52710,6 +52726,8 @@ FIND statement with MATCH COUNT is considered okay if subrc is not checked`,
             else if (config.selectTable === true
                 && statement.get() instanceof Statements.Select
                 && statement.concatTokens().toUpperCase().startsWith("SELECT SINGLE ") === false
+                && statement.concatTokens().toUpperCase().startsWith("SELECT COUNT( * ) ") === false
+                && statement.concatTokens().toUpperCase().startsWith("SELECT COUNT(*) ") === false
                 && this.isChecked(i, statements) === false
                 && this.checksDbcnt(i, statements) === false) {
                 issues.push(issue_1.Issue.atStatement(file, statement, message, this.getMetadata().key, this.conf.severity));

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/cli",
-    "version": "2.102.60",
+    "version": "2.102.62",
     "description": "abaplint - Command Line Interface",
     "funding": "https://github.com/sponsors/larshp",
     "bin": {
@@ -38,16 +38,16 @@
     },
     "homepage": "https://abaplint.org",
     "devDependencies": {
-        "@abaplint/core": "^2.102.60",
+        "@abaplint/core": "^2.102.62",
         "@types/chai": "^4.3.9",
         "@types/glob": "^7.2.0",
         "@types/minimist": "^1.2.4",
         "@types/mocha": "^10.0.3",
-        "@types/node": "^20.8.7",
+        "@types/node": "^20.8.8",
         "@types/progress": "^2.0.6",
         "chai": "^4.3.10",
         "chalk": "^5.3.0",
-        "eslint": "^8.51.0",
+        "eslint": "^8.52.0",
         "glob": "^7.2.3",
         "json5": "^2.2.3",
         "memfs": "^4.6.0",