@abaplint/core 2.119.44 → 2.119.45

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/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.45";
     }
     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

package/build/src/version.js

@@ -1,11 +1,7 @@
 "use strict";
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.defaultRelease = exports.defaultVersion = exports.Release = exports.ReleaseList = exports.LanguageVersion = exports.Version = void 0;
+exports.defaultRelease = exports.Release = exports.ReleaseList = exports.LanguageVersion = exports.Version = void 0;
 exports.releaseAtLeast = releaseAtLeast;
-exports.fromABAP = fromABAP;
-exports.fromKernel = fromKernel;
-exports.fromCloud = fromCloud;
-exports.fromOP = fromOP;
 exports.versionToABAPRelease = versionToABAPRelease;
 exports.getPreviousVersion = getPreviousVersion;
 /** @deprecated Use ABAPRelease and Release instead */
@@ -34,7 +30,6 @@ var Version;
      * Release.Newest + {@link LanguageVersion.Cloud} pair.
      */
     Version["Cloud"] = "Cloud";
-    Version["Newest"] = "Newest";
 })(Version || (exports.Version = Version = {}));
 /**
  * Dialect / flavor axis, orthogonal to the on-prem/cloud release track.
@@ -142,62 +137,6 @@ exports.Release = Object.freeze(Object.assign(Object.assign({}, Object.fromEntri
 function releaseAtLeast(running, required) {
     return running.ordinal >= required.ordinal;
 }
-// Lookup maps for the four constructor functions
-const byAbap = new Map();
-const byKernel = new Map();
-const byOp = new Map();
-const byCloud = new Map();
-for (const r of exports.ReleaseList) {
-    if (r.abap !== null) {
-        byAbap.set(r.abap, r);
-    }
-    if (r.kernel !== null) {
-        byKernel.set(r.kernel, r);
-    }
-    if (r.cloud !== null) {
-        byCloud.set(r.cloud, r);
-    }
-    if (r.op !== null) {
-        // fromOP returns the *last* row at that op (the full on-prem release),
-        // so we overwrite on each iteration — later rows win.
-        byOp.set(r.op, r);
-    }
-}
-/** Look up a release by its ABAP name, e.g. "7.90", "740SP05", "9.16" */
-function fromABAP(abap) {
-    const found = byAbap.get(abap);
-    if (found === undefined) {
-        throw new Error("Unknown ABAP release: " + abap);
-    }
-    return found;
-}
-/** Look up a release by its kernel release number, e.g. 753, 790, 916 */
-function fromKernel(kernel) {
-    const found = byKernel.get(kernel);
-    if (found === undefined) {
-        throw new Error("Unknown kernel release: " + kernel);
-    }
-    return found;
-}
-/** Look up a release by its cloud release number, e.g. 793, 916 */
-function fromCloud(cloud) {
-    const found = byCloud.get(cloud);
-    if (found === undefined) {
-        throw new Error("Unknown cloud release: " + cloud);
-    }
-    return found;
-}
-/**
- * Look up a release by its on-prem release number, e.g. 758, 816.
- * Returns the *last* cloud row at that op number, which is the full on-prem release.
- */
-function fromOP(op) {
-    const found = byOp.get(op);
-    if (found === undefined) {
-        throw new Error("Unknown on-prem release: " + op);
-    }
-    return found;
-}
 // ---------------------------------------------------------------------------
 // Deprecated compatibility layer — keeps existing callers working
 // ---------------------------------------------------------------------------
@@ -217,7 +156,6 @@ const byDeprecated = Object.freeze({
     [Version.v757]: exports.Release["v757"],
     [Version.v758]: exports.Release["v758"],
     [Version.v816]: exports.Release["v816"],
-    [Version.Newest]: exports.Release["Newest"],
     [Version.Cloud]: exports.Release["Newest"], // Cloud → Newest sentinel
     [Version.OpenABAP]: exports.Release["v702"], // OpenABAP shares v702 release identity
 });
@@ -228,8 +166,7 @@ function versionToABAPRelease(v) {
     }
     return found;
 }
-exports.defaultVersion = Version.v758;
-exports.defaultRelease = exports.Release.v758;
+exports.defaultRelease = exports.Release.Newest;
 function getPreviousVersion(v) {
     if (v === Version.OpenABAP) {
         return Version.v702;

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@abaplint/core",
-    "version": "2.119.44",
+    "version": "2.119.45",
     "description": "abaplint - Core API",
     "main": "build/src/index.js",
     "typings": "build/abaplint.d.ts",