Merge pull request #731 from Kipper-Lang/721-bug-internal-references-to-class-properties-without-this-are-allowed-but-throw-runtime-error

Fixed internal references to class properties without `this` being allowed and throwing runtime errors

Author: Luna-Klatzer

.github/workflows/cov-badge.yml

@@ -14,7 +14,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x]
+        node-version: [22.19.x]
 
     steps:
       - name: Checkout repository

.github/workflows/nodejs-cli-run.yml

@@ -12,7 +12,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x, 18.x, 20.x, 22.x]
+        node-version: [16.x, 18.x, 20.x, 22.x, 24.x]
 
     steps:
       - uses: actions/checkout@v4
@@ -34,7 +34,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x, 18.x, 20.x, 22.x]
+        node-version: [16.x, 18.x, 20.x, 22.x, 24.x]
 
     steps:
       - uses: actions/checkout@v4
@@ -56,7 +56,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x, 18.x, 20.x, 22.x]
+        node-version: [16.x, 18.x, 20.x, 22.x, 24.x]
 
     steps:
       - uses: actions/checkout@v4

.github/workflows/nodejs-test.yml

@@ -9,7 +9,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x, 18.x, 20.x, 22.x]
+        node-version: [16.x, 18.x, 20.x, 22.19.x, 24.x]
 
     steps:
       - uses: actions/checkout@v4
@@ -22,6 +22,11 @@ jobs:
           version: 8.x.x
           run_install: false
       - run: pnpm install --frozen-lockfile
+      # Restore old config for Node 16, 18 & 20
+      - run: |
+          if [[ $(node -v | grep -E '^v(16|18|20)\.') ]]; then
+            cp .mocharc.old.json .mocharc.json
+          fi
       - run: pnpm test
 
   # Run tests with 'pnpm i' dependencies
@@ -30,7 +35,7 @@ jobs:
 
     strategy:
       matrix:
-        node-version: [16.x, 18.x, 20.x, 22.x]
+        node-version: [16.x, 18.x, 20.x, 22.19.x, 24.x]
 
     steps:
       - uses: actions/checkout@v4
@@ -43,4 +48,9 @@ jobs:
           version: 8.x.x
           run_install: false
       - run: pnpm i
+      # Restore old config for Node 16, 18 & 20
+      - run: |
+          if [[ $(node -v | grep -E '^v(16|18|20)\.') ]]; then
+            cp .mocharc.old.json .mocharc.json
+          fi
       - run: pnpm test

.mocharc.json

@@ -15,5 +15,8 @@
 	],
 	"spec": [
 		"**/*.test.ts"
+	],
+	"node-option": [
+		"no-experimental-strip-types"
 	]
 }

.mocharc.old.json

@@ -0,0 +1,19 @@
+{
+  "extension": ["ts"],
+  "timeout": 15000,
+  "exit": true,
+  "recursive": true,
+	"watch": false,
+	"watch-files": [
+		"./kipper/core/src/",
+		"./kipper/cli/src/",
+		"./node_modules/@kipper/"
+	],
+  "reporter": "spec",
+  "require": [
+		"ts-node/register"
+	],
+	"spec": [
+		"**/*.test.ts"
+	]
+}

.run/kipper compile (DEBUG).run.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="kipper compile (DEBUG)" type="NodeJSConfigurationType" application-parameters="compile -s &quot;&quot;" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
+  <configuration default="false" name="kipper compile (DEBUG)" type="NodeJSConfigurationType" application-parameters="compile -s &quot;&quot;" node-parameters="--no-experimental-strip-types" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
     <envs>
       <env name="DEBUG" value="*" />
     </envs>

.run/kipper compile (js).run.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="kipper compile (js)" type="NodeJSConfigurationType" application-parameters="compile -s &quot;&quot; -t js" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
+  <configuration default="false" name="kipper compile (js)" type="NodeJSConfigurationType" application-parameters="compile -s &quot;&quot; -t js" node-parameters="--no-experimental-strip-types" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
     <method v="2" />
   </configuration>
 </component>

.run/kipper compile (ts).run.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="kipper compile (ts)" type="NodeJSConfigurationType" application-parameters="compile -s &quot;&quot; -t ts" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
+  <configuration default="false" name="kipper compile (ts)" type="NodeJSConfigurationType" application-parameters="compile -s &quot;&quot; -t ts" node-parameters="--no-experimental-strip-types" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
     <method v="2" />
   </configuration>
 </component>

.run/kipper help (DEBUG).run.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="kipper help (DEBUG)" type="NodeJSConfigurationType" application-parameters="help" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
+  <configuration default="false" name="kipper help (DEBUG)" type="NodeJSConfigurationType" application-parameters="help" node-parameters="--no-experimental-strip-types" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
     <envs>
       <env name="DEBUG" value="*" />
     </envs>

.run/kipper help.run.xml

@@ -1,5 +1,5 @@
 <component name="ProjectRunConfigurationManager">
-  <configuration default="false" name="kipper help" type="NodeJSConfigurationType" application-parameters="help" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
+  <configuration default="false" name="kipper help" type="NodeJSConfigurationType" application-parameters="help" node-parameters="--no-experimental-strip-types" path-to-js-file="./kipper/cli/bin/run" working-dir="$PROJECT_DIR$/">
     <method v="2" />
   </configuration>
 </component>