@abaplint/core 2.119.44 → 2.119.46

package/build/src/abap/2_statements/statements/rollback.js

@@ -3,9 +3,10 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.Rollback = void 0;
 const combi_1 = require("../combi");
 const expressions_1 = require("../expressions");
+const version_1 = require("../../../version");
 class Rollback {
     getMatcher() {
-        return (0, combi_1.seq)("ROLLBACK", (0, combi_1.altPrio)("WORK", expressions_1.DatabaseConnection));
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("ROLLBACK", (0, combi_1.altPrio)("WORK", expressions_1.DatabaseConnection)));
     }
 }
 exports.Rollback = Rollback;

package/build/src/abap/2_statements/statements/rollback_entities.js

@@ -6,7 +6,7 @@ const version_1 = require("../../../version");
 class RollbackEntities {
     getMatcher() {
         const s = "ROLLBACK ENTITIES";
-        return (0, combi_1.ver)(version_1.Release.v754, s);
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.ver)(version_1.Release.v754, s));
     }
 }
 exports.RollbackEntities = RollbackEntities;

package/build/src/abap/2_statements/statements/set_handler.js

@@ -3,12 +3,13 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.SetHandler = void 0;
 const combi_1 = require("../combi");
 const expressions_1 = require("../expressions");
+const version_1 = require("../../../version");
 class SetHandler {
     getMatcher() {
         const activation = (0, combi_1.seq)("ACTIVATION", expressions_1.Source);
         const fo = (0, combi_1.seq)("FOR", (0, combi_1.altPrio)("ALL INSTANCES", expressions_1.Source));
         const ret = (0, combi_1.seq)("SET HANDLER", (0, combi_1.plus)(expressions_1.MethodSource), (0, combi_1.optPrio)(fo), (0, combi_1.optPrio)(activation));
-        return ret;
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, ret);
     }
 }
 exports.SetHandler = SetHandler;

package/build/src/abap/2_statements/statements/set_locks.js

@@ -7,7 +7,7 @@ const version_1 = require("../../../version");
 class SetLocks {
     getMatcher() {
         const s = (0, combi_1.seq)("SET LOCKS OF", expressions_1.SimpleName, "ENTITY", expressions_1.SimpleName, "FROM", expressions_1.Source, "FAILED", expressions_1.Target, "REPORTED", expressions_1.Target);
-        return (0, combi_1.ver)(version_1.Release.v754, s);
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.ver)(version_1.Release.v754, s));
     }
 }
 exports.SetLocks = SetLocks;

package/build/src/abap/2_statements/statements/static.js

@@ -36,12 +36,13 @@ Object.defineProperty(exports, "__esModule", { value: true });
 exports.Static = void 0;
 const combi_1 = require("../combi");
 const Expressions = __importStar(require("../expressions"));
+const version_1 = require("../../../version");
 class Static {
     getMatcher() {
         const p = (0, combi_1.opt)((0, combi_1.per)(Expressions.Type, Expressions.Value, Expressions.Length, Expressions.Decimals));
         const type = (0, combi_1.seq)((0, combi_1.opt)(Expressions.ConstantFieldLength), p);
-        const ret = (0, combi_1.seq)((0, combi_1.alt)("STATIC", "STATICS"), Expressions.DefinitionName, (0, combi_1.optPrio)(Expressions.ConstantFieldLength), (0, combi_1.alt)(type, Expressions.TypeTable));
-        return ret;
+        const body = (0, combi_1.seq)(Expressions.DefinitionName, (0, combi_1.optPrio)(Expressions.ConstantFieldLength), (0, combi_1.alt)(type, Expressions.TypeTable));
+        return (0, combi_1.altPrio)((0, combi_1.seq)("STATIC", body), (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("STATICS", body)));
     }
 }
 exports.Static = Static;

package/build/src/abap/2_statements/statements/subtract.js

@@ -1,12 +1,13 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Subtract = void 0;
+const version_1 = require("../../../version");
 const combi_1 = require("../combi");
 const expressions_1 = require("../expressions");
 class Subtract {
     getMatcher() {
         const ret = (0, combi_1.seq)("SUBTRACT", expressions_1.Source, "FROM", expressions_1.Target);
-        return ret;
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, ret);
     }
 }
 exports.Subtract = Subtract;

package/build/src/abap/2_statements/statements/test_injection.js

@@ -1,11 +1,12 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.TestInjection = void 0;
+const version_1 = require("../../../version");
 const combi_1 = require("../combi");
 const expressions_1 = require("../expressions");
 class TestInjection {
     getMatcher() {
-        return (0, combi_1.seq)("TEST-INJECTION", expressions_1.TestSeamName);
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.seq)("TEST-INJECTION", expressions_1.TestSeamName));
     }
 }
 exports.TestInjection = TestInjection;

package/build/src/abap/2_statements/statements/test_seam.js

@@ -6,7 +6,7 @@ const expressions_1 = require("../expressions");
 const version_1 = require("../../../version");
 class TestSeam {
     getMatcher() {
-        return (0, combi_1.ver)(version_1.Release.v750, (0, combi_1.seq)("TEST-SEAM", expressions_1.TestSeamName), { also: combi_1.AlsoIn.OpenABAP });
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.ver)(version_1.Release.v750, (0, combi_1.seq)("TEST-SEAM", expressions_1.TestSeamName), { also: combi_1.AlsoIn.OpenABAP }));
     }
 }
 exports.TestSeam = TestSeam;

package/build/src/abap/2_statements/statements/update_database.js

@@ -1,17 +1,18 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.UpdateDatabase = void 0;
+const version_1 = require("../../../version");
 const combi_1 = require("../combi");
 const expressions_1 = require("../expressions");
-const version_1 = require("../../../version");
+const version_2 = require("../../../version");
 class UpdateDatabase {
     getMatcher() {
         const parameters = (0, combi_1.seq)(expressions_1.SQLFieldAndValue, (0, combi_1.star)((0, combi_1.seq)((0, combi_1.opt)(","), expressions_1.SQLFieldAndValue)));
         const set = (0, combi_1.seq)("SET", (0, combi_1.alt)(parameters, expressions_1.Dynamic), (0, combi_1.opt)((0, combi_1.seq)("WHERE", expressions_1.SQLCond)));
         const indicators = (0, combi_1.seq)("INDICATORS SET STRUCTURE", expressions_1.ComponentName);
-        const fromTable = (0, combi_1.seq)("FROM", (0, combi_1.opt)("TABLE"), expressions_1.SQLSource, (0, combi_1.opt)((0, combi_1.ver)(version_1.Release.v755, indicators)));
+        const fromTable = (0, combi_1.seq)("FROM", (0, combi_1.opt)("TABLE"), expressions_1.SQLSource, (0, combi_1.opt)((0, combi_1.ver)(version_2.Release.v755, indicators)));
         const ret = (0, combi_1.seq)("UPDATE", expressions_1.DatabaseTable, (0, combi_1.opt)(expressions_1.SQLClient), (0, combi_1.opt)(expressions_1.DatabaseConnection), (0, combi_1.opt)((0, combi_1.alt)(fromTable, set)));
-        return ret;
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, ret);
     }
 }
 exports.UpdateDatabase = UpdateDatabase;

package/build/src/abap/2_statements/statements/wait.js

@@ -1,19 +1,20 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.Wait = void 0;
-const combi_1 = require("../combi");
 const version_1 = require("../../../version");
+const combi_1 = require("../combi");
+const version_2 = require("../../../version");
 const expressions_1 = require("../expressions");
 class Wait {
     getMatcher() {
         const up = (0, combi_1.seq)("UP TO", expressions_1.Source, "SECONDS");
         const channels = "MESSAGING CHANNELS";
-        const push = (0, combi_1.ver)(version_1.Release.v750, "PUSH CHANNELS", { also: combi_1.AlsoIn.OpenABAP });
+        const push = (0, combi_1.ver)(version_2.Release.v750, "PUSH CHANNELS", { also: combi_1.AlsoIn.OpenABAP });
         const tasks = "ASYNCHRONOUS TASKS";
         const type = (0, combi_1.seq)("FOR", (0, combi_1.per)(channels, push, tasks));
         const until = (0, combi_1.seq)((0, combi_1.opt)(type), "UNTIL", expressions_1.Cond, (0, combi_1.opt)(up));
         const ret = (0, combi_1.seq)("WAIT", (0, combi_1.alt)(until, up));
-        return ret;
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, ret);
     }
 }
 exports.Wait = Wait;

package/build/src/abap/2_statements/statements/with.js

@@ -9,7 +9,7 @@ class With {
     getMatcher() {
         const exposing = (0, combi_1.optPrio)((0, combi_1.seq)("WITH", (0, combi_1.altPrio)(new expressions_1.SQLCTEHierarchy(), expressions_1.SQLCTEAssociations)));
         const cte = (0, combi_1.seq)(expressions_1.WithName, "AS", (0, combi_1.tok)(tokens_1.WParenLeftW), expressions_1.SelectCTE, (0, combi_1.tok)(tokens_1.WParenRightW), exposing);
-        return (0, combi_1.ver)(version_1.Release.v751, (0, combi_1.seq)("WITH", cte, (0, combi_1.star)((0, combi_1.seq)(",", cte)), expressions_1.Select));
+        return (0, combi_1.verNotLang)(version_1.LanguageVersion.KeyUser, (0, combi_1.ver)(version_1.Release.v751, (0, combi_1.seq)("WITH", cte, (0, combi_1.star)((0, combi_1.seq)(",", cte)), expressions_1.Select)));
     }
 }
 exports.With = With;

package/build/src/abap/5_syntax/_current_scope.js

@@ -1,7 +1,6 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
 exports.CurrentScope = void 0;
-const version_1 = require("../../version");
 const _builtin_1 = require("./_builtin");
 const ddic_1 = require("../../ddic");
 const position_1 = require("../../position");
@@ -37,7 +36,7 @@ class CurrentScope {
     }
     ///////////////////////////
     getRelease() {
-        return (0, version_1.versionToABAPRelease)(this.reg.getConfig().getVersion());
+        return this.reg.getConfig().getRelease();
     }
     getVersion() {
         return this.reg.getConfig().getVersion();

package/build/src/abap/5_syntax/expressions/select.js

@@ -50,7 +50,7 @@ const _syntax_input_1 = require("../_syntax_input");
 const version_1 = require("../../../version");
 const isSimple = /^\w+$/;
 class Select {
-    static runSyntax(node, input, skipImplicitInto = false) {
+    static runSyntax(node, input, skipImplicitInto = false, selectLoop = false) {
         var _a;
         const token = node.getFirstToken();
         let from = node.findDirectExpression(Expressions.SQLFrom);
@@ -109,6 +109,14 @@ class Select {
             sql_source_1.SQLSource.runSyntax(s, input);
         }
         for (const up of node.findDirectExpressions(Expressions.SQLUpTo)) {
+            if (intoExpression
+                && this.isStrictMode(node, input)
+                && (selectLoop === false || input.scope.getLanguageVersion() === version_1.LanguageVersion.Cloud)
+                && from.getFirstToken().getStart().isBefore(up.getFirstToken().getStart())
+                && up.getFirstToken().getStart().isBefore(intoExpression.getFirstToken().getStart())) {
+                const message = `The addition "UP TO n ROWS" must only be placed after the INTO/APPENDING clause.`;
+                input.issues.push((0, _syntax_input_1.syntaxIssue)(input, up.getFirstToken(), message));
+            }
             for (const s of up.findDirectExpressions(Expressions.SQLSource)) {
                 sql_source_1.SQLSource.runSyntax(s, input);
             }
@@ -134,7 +142,7 @@ class Select {
                 input.issues.push((0, _syntax_input_1.syntaxIssue)(input, orderBy.getFirstToken(), message));
             }
         }
-        if (this.isStrictMode(node)) {
+        if (this.isStrictMode(node, input)) {
             this.strictModeChecks(node, input);
         }
         if (input.scope.getType() === _scope_type_1.ScopeType.OpenSQL) {
@@ -142,7 +150,11 @@ class Select {
         }
     }
     // there are multiple rules, but gotta start somewhere
-    static isStrictMode(node) {
+    static isStrictMode(node, input) {
+        // strict OpenSQL is always required in the cloud language version
+        if (input.scope.getLanguageVersion() === version_1.LanguageVersion.Cloud) {
+            return true;
+        }
         const into = node.findDirectExpressionsMulti([Expressions.SQLIntoList, Expressions.SQLIntoStructure, Expressions.SQLIntoTable])[0];
         const where = node.findDirectExpression(Expressions.SQLCond);
         // INTO is after WHERE

package/build/src/abap/5_syntax/statements/delete_internal.js

@@ -54,8 +54,9 @@ class DeleteInternal {
             const localVariable = input.scope.findVariable(target.concatTokens());
             if (localVariable === undefined && node.getChildren().length === 5 && node.getChildren()[2].concatTokens().toUpperCase() === "FROM") {
                 // it might be a database table
-                tabl = (_a = input.scope.getDDIC()) === null || _a === void 0 ? void 0 : _a.lookupTableOrView(target.concatTokens());
-                if (tabl) {
+                const found = (_a = input.scope.getDDIC()) === null || _a === void 0 ? void 0 : _a.lookupTableOrView(target.concatTokens());
+                if ((found === null || found === void 0 ? void 0 : found.object) !== undefined) {
+                    tabl = found;
                     input.scope.getDDICReferences().addUsing(input.scope.getParentObj(), { object: tabl.object });
                 }
             }
@@ -63,7 +64,7 @@ class DeleteInternal {
                 targetType = target_1.Target.runSyntax(target, input);
                 if (node.findDirectTokenByText("TABLE") === undefined
                     && node.findDirectTokenByText("ADJACENT") === undefined
-                    && node.findDirectTokenByText("FROM")
+                    && (node.findDirectTokenByText("FROM") || node.findDirectTokenByText("INDEX"))
                     && targetType instanceof basic_1.TableType
                     && targetType.getAccessType() === basic_1.TableAccessType.hashed) {
                     const message = "Implicit or explicit index operation on hashed table is not possible";

package/build/src/abap/5_syntax/statements/loop.js

@@ -99,6 +99,14 @@ class Loop {
         }
         const targetConcat = loopTarget === null || loopTarget === void 0 ? void 0 : loopTarget.concatTokens().toUpperCase();
         const topType = sourceType; // todo: refactor topType vs sourceType vs rowType
+        if (topType instanceof basic_1.TableType
+            && topType.getAccessType() === basic_1.TableAccessType.hashed
+            && node.findDirectTokenByText("USING") === undefined
+            && (node.findDirectTokenByText("FROM") || node.findDirectTokenByText("TO"))) {
+            const message = "Implicit or explicit index operation on hashed table is not possible";
+            input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+            return;
+        }
         if (sourceType instanceof basic_1.TableType) {
             rowType = sourceType.getRowType();
             sourceType = rowType;

package/build/src/abap/5_syntax/statements/modify_internal.js

@@ -58,6 +58,13 @@ class ModifyInternal {
                 // ok
             }
             else if (targetType instanceof basic_1.TableType) {
+                if (node.findDirectTokenByText("TABLE") === undefined
+                    && node.findDirectTokenByText("INDEX")
+                    && targetType.getAccessType() === basic_1.TableAccessType.hashed) {
+                    const message = "Implicit or explicit index operation on hashed table is not possible";
+                    input.issues.push((0, _syntax_input_1.syntaxIssue)(input, node.getFirstToken(), message));
+                    return;
+                }
                 if (node.findDirectTokenByText("TABLE")
                     && node.findDirectTokenByText("INDEX")
                     && targetType.isWithHeader() === false) {

package/build/src/abap/5_syntax/statements/select_loop.js

@@ -40,7 +40,7 @@ class SelectLoop {
     runSyntax(node, input) {
         const s = node.findDirectExpression(Expressions.Select);
         if (s) {
-            select_1.Select.runSyntax(s, input);
+            select_1.Select.runSyntax(s, input, false, true);
         }
     }
 }

package/build/src/cds/cds_parser.js

@@ -48,27 +48,27 @@ class CDSParser {
         }
         let tokens = cds_lexer_1.CDSLexer.run(file);
         tokens = tokens.filter(t => !(t instanceof tokens_1.Comment));
-        let res = combi_1.Combi.run(new Expressions.CDSDefineView(), tokens, version_1.defaultVersion);
+        let res = combi_1.Combi.run(new Expressions.CDSDefineView(), tokens, version_1.defaultRelease);
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSDefineAbstract(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSDefineAbstract(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSDefineProjection(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSDefineProjection(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSAnnotate(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSAnnotate(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSDefineCustom(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSDefineCustom(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSDefineTableFunction(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSDefineTableFunction(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSExtendView(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSExtendView(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
-            res = combi_1.Combi.run(new Expressions.CDSDefineHierarchy(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.CDSDefineHierarchy(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             return undefined;

package/build/src/config.js

@@ -47,10 +47,7 @@ class Config {
         for (const rule of sorted) {
             rules[rule.getMetadata().key] = rule.getConfig();
         }
-        let version = version_1.defaultVersion;
-        if (ver) {
-            version = ver;
-        }
+        const version = ver;
         // defaults: dont skip anything, report everything. The user can decide to skip stuff
         // its difficult to debug errors not being reported
         const config = {
@@ -158,42 +155,54 @@ class Config {
         return this.config.syntax;
     }
     getRelease() {
-        return (0, version_1.versionToABAPRelease)(this.getVersion());
+        const v = this.config.syntax.version;
+        if (v !== undefined && typeof v !== "string") {
+            return version_1.Release[v.release];
+        }
+        if (this.config.global === undefined || v === undefined) {
+            return version_1.defaultRelease;
+        }
+        return (0, version_1.versionToABAPRelease)(v);
     }
     getOpenABAP() {
         return this.config.syntax.openABAP === true;
     }
     getVersion() {
-        if (this.config.global === undefined || this.config.syntax.version === undefined) {
-            return version_1.defaultVersion;
+        const v = this.config.syntax.version;
+        if (this.config.global === undefined || v === undefined || typeof v !== "string") {
+            return version_1.Version.v816;
         }
-        return this.config.syntax.version;
+        return v;
     }
     getLanguageVersion() {
         var _a;
+        const v = this.config.syntax.version;
+        if (v !== undefined && typeof v !== "string") {
+            return v.language;
+        }
         return (_a = this.config.syntax.languageVersion) !== null && _a !== void 0 ? _a : version_1.LanguageVersion.Normal;
     }
     checkVersion() {
-        if (this.config.syntax.version === undefined) {
-            return; // handled in getVersion
+        const version = this.config.syntax.version;
+        if (version === undefined || typeof version !== "string") {
+            return; // undefined handled in getVersion, object form is already explicit
         }
         let match = false;
         const vers = version_1.Version;
         for (const v in version_1.Version) {
-            if (vers[v] === this.config.syntax.version) {
+            if (vers[v] === version) {
                 match = true;
                 break;
             }
         }
         if (match === false) {
-            this.config.syntax.version = version_1.defaultVersion;
+            this.config.syntax.version = undefined;
             return;
         }
-        if (this.config.syntax.version === version_1.Version.Cloud) {
-            this.config.syntax.version = version_1.Version.Newest;
+        if (version === version_1.Version.Cloud) {
             this.config.syntax.languageVersion = version_1.LanguageVersion.Cloud;
         }
-        else if (this.config.syntax.version === version_1.Version.OpenABAP) {
+        else if (version === version_1.Version.OpenABAP) {
             this.config.syntax.version = version_1.Version.v702;
             this.config.syntax.openABAP = true;
         }

package/build/src/ddl/ddl_parser.js

@@ -47,9 +47,9 @@ var DDLKind;
 class DDLParser {
     parse(file) {
         const tokens = ddl_lexer_1.DDLLexer.run(file);
-        let res = combi_1.Combi.run(new Expressions.DDLStructure(), tokens, version_1.defaultVersion);
+        let res = combi_1.Combi.run(new Expressions.DDLStructure(), tokens, version_1.defaultRelease);
         if (res === undefined) {
-            res = combi_1.Combi.run(new Expressions.DDLTable(), tokens, version_1.defaultVersion);
+            res = combi_1.Combi.run(new Expressions.DDLTable(), tokens, version_1.defaultRelease);
         }
         if (res === undefined || !(res[0] instanceof nodes_1.ExpressionNode)) {
             return undefined;

package/build/src/registry.js

@@ -74,7 +74,7 @@ class Registry {
     }
     static abaplintVersion() {
         // magic, see build script "version.js"
-        return "2.119.44";
+        return "2.119.46";
     }
     getDDICReferences() {
         return this.ddicReferences;

package/build/src/rules/index.js

@@ -172,6 +172,7 @@ __exportStar(require("./strict_sql"), exports);
 __exportStar(require("./superclass_final"), exports);
 __exportStar(require("./superfluous_value"), exports);
 __exportStar(require("./sy_modification"), exports);
+__exportStar(require("./sy_read_restriction"), exports);
 __exportStar(require("./tabl_enhancement_category"), exports);
 __exportStar(require("./tables_declared_locally"), exports);
 __exportStar(require("./try_without_catch"), exports);

package/build/src/rules/sy_read_restriction.js

@@ -0,0 +1,152 @@
+"use strict";
+var __createBinding = (this && this.__createBinding) || (Object.create ? (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    var desc = Object.getOwnPropertyDescriptor(m, k);
+    if (!desc || ("get" in desc ? !m.__esModule : desc.writable || desc.configurable)) {
+      desc = { enumerable: true, get: function() { return m[k]; } };
+    }
+    Object.defineProperty(o, k2, desc);
+}) : (function(o, m, k, k2) {
+    if (k2 === undefined) k2 = k;
+    o[k2] = m[k];
+}));
+var __setModuleDefault = (this && this.__setModuleDefault) || (Object.create ? (function(o, v) {
+    Object.defineProperty(o, "default", { enumerable: true, value: v });
+}) : function(o, v) {
+    o["default"] = v;
+});
+var __importStar = (this && this.__importStar) || (function () {
+    var ownKeys = function(o) {
+        ownKeys = Object.getOwnPropertyNames || function (o) {
+            var ar = [];
+            for (var k in o) if (Object.prototype.hasOwnProperty.call(o, k)) ar[ar.length] = k;
+            return ar;
+        };
+        return ownKeys(o);
+    };
+    return function (mod) {
+        if (mod && mod.__esModule) return mod;
+        var result = {};
+        if (mod != null) for (var k = ownKeys(mod), i = 0; i < k.length; i++) if (k[i] !== "default") __createBinding(result, mod, k[i]);
+        __setModuleDefault(result, mod);
+        return result;
+    };
+})();
+Object.defineProperty(exports, "__esModule", { value: true });
+exports.SyReadRestriction = exports.SyReadRestrictionConf = void 0;
+const issue_1 = require("../issue");
+const Expressions = __importStar(require("../abap/2_statements/expressions"));
+const _abap_rule_1 = require("./_abap_rule");
+const _basic_rule_config_1 = require("./_basic_rule_config");
+const _irule_1 = require("./_irule");
+const version_1 = require("../version");
+const severity_1 = require("../severity");
+// ABAP Cloud: 17 system fields allowed for reading
+// https://help.sap.com/doc/abapdocu_cp_index_htm/CLOUD/en-US/ABENSYSTEM_FIELDS.html
+const CLOUD_ALLOWED = new Set([
+    "SY-BATCH",
+    "SY-DBCNT",
+    "SY-FDPOS",
+    "SY-INDEX",
+    "SY-LANGU",
+    "SY-MANDT",
+    "SY-MSGID",
+    "SY-MSGNO",
+    "SY-MSGTY",
+    "SY-MSGV1",
+    "SY-MSGV2",
+    "SY-MSGV3",
+    "SY-MSGV4",
+    "SY-SUBRC",
+    "SY-SYSID",
+    "SY-TABIX",
+    "SY-UNAME",
+]);
+// ABAP for Key Users: 5 system fields allowed for reading
+const KEYUSER_ALLOWED = new Set([
+    "SY-DBCNT",
+    "SY-FDPOS",
+    "SY-INDEX",
+    "SY-SUBRC",
+    "SY-TABIX",
+]);
+class SyReadRestrictionConf extends _basic_rule_config_1.BasicRuleConfig {
+}
+exports.SyReadRestrictionConf = SyReadRestrictionConf;
+class SyReadRestriction extends _abap_rule_1.ABAPRule {
+    constructor() {
+        super(...arguments);
+        this.conf = new SyReadRestrictionConf();
+    }
+    getMetadata() {
+        return {
+            key: "sy_read_restriction",
+            title: "SY field read restriction",
+            shortDescription: `Finds reads of SY fields not allowed in the current language version`,
+            extendedInformation: `In ABAP Cloud, only 17 system fields may be read; reads of other fields produce a warning.\n` +
+                `In ABAP for Key Users, only 5 system fields may be read:\n` +
+                `SY-DBCNT, SY-FDPOS, SY-INDEX, SY-SUBRC, SY-TABIX.\n` +
+                `https://help.sap.com/doc/abapdocu_cp_index_htm/CLOUD/en-US/ABENSYSTEM_FIELDS.html`,
+            tags: [_irule_1.RuleTag.SingleFile],
+            badExample: `lv = sy-uname.`,
+            goodExample: `IF sy-subrc = 0. ENDIF.`,
+        };
+    }
+    getConfig() {
+        return this.conf;
+    }
+    setConfig(conf) {
+        this.conf = conf;
+    }
+    runParsed(file, _obj) {
+        var _a;
+        const issues = [];
+        const langVer = this.reg.getConfig().getLanguageVersion();
+        if (langVer === version_1.LanguageVersion.Normal) {
+            return [];
+        }
+        const allowed = langVer === version_1.LanguageVersion.KeyUser ? KEYUSER_ALLOWED : CLOUD_ALLOWED;
+        const severity = langVer === version_1.LanguageVersion.KeyUser ? this.conf.severity : severity_1.Severity.Warning;
+        // Collect tokens that appear in writable Target positions — they're modifications,
+        // handled by the sy_modification rule, not this one.
+        const writeTokens = new Set();
+        for (const t of ((_a = file.getStructure()) === null || _a === void 0 ? void 0 : _a.findAllExpressionsRecursive(Expressions.Target)) || []) {
+            const concat = t.concatTokens().toUpperCase();
+            if (!concat.startsWith("SY-")) {
+                continue;
+            }
+            writeTokens.add(t.getFirstToken());
+        }
+        // Walk all tokens looking for the sequence  SY  -  <fieldname>
+        const tokens = file.getTokens();
+        for (let i = 0; i + 2 < tokens.length; i++) {
+            const sy = tokens[i];
+            const dash = tokens[i + 1];
+            const fld = tokens[i + 2];
+            if (sy.getStr().toUpperCase() !== "SY") {
+                continue;
+            }
+            if (dash.getStr() !== "-") {
+                continue;
+            }
+            // Adjacent: no whitespace between the three tokens, on the same row
+            if (sy.getEnd().getCol() !== dash.getStart().getCol()
+                || dash.getEnd().getCol() !== fld.getStart().getCol()
+                || sy.getRow() !== dash.getRow()
+                || dash.getRow() !== fld.getRow()) {
+                continue;
+            }
+            if (writeTokens.has(sy)) {
+                continue;
+            }
+            const fieldName = "SY-" + fld.getStr().toUpperCase();
+            if (!allowed.has(fieldName)) {
+                const message = `SY field "${fieldName}" cannot be read in language version ${langVer}`;
+                issues.push(issue_1.Issue.atToken(file, sy, message, this.getMetadata().key, severity));
+            }
+        }
+        return issues;
+    }
+}
+exports.SyReadRestriction = SyReadRestriction;
+//# sourceMappingURL=sy_read_restriction.js.map