@lightningjs/blits 1.30.0 → 1.30.2

package/.github/workflows/deploy-docs.yml

@@ -1,27 +1,27 @@
-name: Trigger Website Docs Update
-
-on:
-  push:
-    branches: [ master ]
-    paths:
-      - 'docs/**'
-
-jobs:
-  trigger:
-    runs-on: ubuntu-latest
-    steps:
-      - name: Trigger Deployment
-        timeout-minutes: 5
-        run: |
-          response=$(curl -X POST -d {} -H "Content-Type:application/json" -w "%{http_code}" -s -o /dev/null \
-            --max-time 30 \
-            --connect-timeout 10 \
-            --retry 2 \
-            --retry-delay 5 \
-            "${{ secrets.WEBSITE_DEPLOY_URL }}")
-          if [ "$response" -ge 200 ] && [ "$response" -lt 300 ]; then
-            echo "Deployment triggered successfully"
-          else
-            echo "Failed to trigger deployment. Status code: $response"
-            exit 1
-          fi
+name: Trigger Website Docs Update
+
+on:
+  push:
+    branches: [ master ]
+    paths:
+      - 'docs/**'
+
+jobs:
+  trigger:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Trigger Deployment
+        timeout-minutes: 5
+        run: |
+          response=$(curl -X POST -d {} -H "Content-Type:application/json" -w "%{http_code}" -s -o /dev/null \
+            --max-time 30 \
+            --connect-timeout 10 \
+            --retry 2 \
+            --retry-delay 5 \
+            "${{ secrets.WEBSITE_DEPLOY_URL }}")
+          if [ "$response" -ge 200 ] && [ "$response" -lt 300 ]; then
+            echo "Deployment triggered successfully"
+          else
+            echo "Failed to trigger deployment. Status code: $response"
+            exit 1
+          fi

package/.github/workflows/run-blits-tests.yml

@@ -1,142 +1,142 @@
-name: Run Blits Tests
-
-on:
-  pull_request:
-    branches:
-      - dev
-      - master
-
-jobs:
-  run-blits-tests:
-    runs-on: ubuntu-latest
-    outputs:
-      tests_failed: ${{ steps.analyze-results.outputs.result && fromJSON(steps.analyze-results.outputs.result).testsFailed }}
-
-    steps:
-    - name: Checkout Code
-      uses: actions/checkout@v3
-
-    - name: Set up Node.js
-      uses: actions/setup-node@v3
-      with:
-        node-version: 20.16.0
-
-    - name: Save PR Information
-      run: |
-        echo '${{ github.event.pull_request.number }}' > ./pr_number.txt
-        echo '${{ github.event.pull_request.head.repo.full_name }}' > ./pr_repo.txt
-        echo '${{ github.event.pull_request.head.ref }}' > ./pr_branch.txt
-
-    - name: Install Dependencies
-      run: npm install
-
-    - name: Run Tests
-      id: run-tests
-      run: npm run test > test-report.txt 2>&1 || echo "Command failed with exit code $?" >> test-report.txt
-
-    - name: Analyze Test Results
-      id: analyze-results
-      uses: actions/github-script@v6
-      with:
-        result-encoding: json
-        script: |
-          const fs = require('fs');
-          const timestamp = new Date().toISOString();
-
-          const results = {
-            timestamp,
-            summary: '',
-            testsFailed: false,
-            error: null
-          };
-
-          // Check if test report exists and has content
-          if (!fs.existsSync('test-report.txt') || !fs.statSync('test-report.txt').size) {
-            results.summary = 'Test execution error: No test output generated.';
-            results.testsFailed = true;
-            return outputResults(results);
-          }
-
-          // Read test report
-          const testOutput = fs.readFileSync('test-report.txt', 'utf8');
-
-          // Check for errors and test results
-          const hasError = testOutput.includes('Error');
-          const testSummaryMatch = testOutput.match(/passed:\s*(\d+)\s*failed:\s*(\d+)\s*of\s*(\d+)\s*tests/);
-
-          if (hasError && (!testSummaryMatch || testSummaryMatch[3] === '0')) {
-            // Error with no tests or zero tests
-            results.summary = 'Test execution encountered errors. No tests were run.';
-            results.testsFailed = true;
-            results.error = testOutput;
-          } else if (testSummaryMatch) {
-            // We have a test summary
-            const passed = parseInt(testSummaryMatch[1]);
-            const failed = parseInt(testSummaryMatch[2]);
-            const total = parseInt(testSummaryMatch[3]);
-
-            results.summary = testSummaryMatch[0];
-
-            if (failed > 0 || total === 0) {
-              results.testsFailed = true;
-
-              if (total === 0) {
-                results.summary = 'No tests were found to run. This is considered a failure.';
-              }
-            }
-          } else {
-            // No test summary found
-            results.summary = 'Test execution produced no valid test summary';
-            results.testsFailed = true;
-            results.error = testOutput;
-          }
-
-          return outputResults(results);
-
-          /**
-           * Outputs results and saves result files
-           */
-          function outputResults(results) {
-            try {
-              fs.writeFileSync('./timestamp.txt', results.timestamp);
-              fs.writeFileSync('./summary.txt', results.summary);
-              fs.writeFileSync('./failed.txt', results.testsFailed.toString());
-
-              // Save error output if it exists
-              if (results.error) {
-                fs.writeFileSync('./error.txt', results.error);
-              }
-            } catch (err) {
-              console.error('Error saving test results:', err);
-              core.warning(`Error saving test results: ${err.message}`);
-            }
-
-            return results;
-          }
-
-    - name: Upload Test Report Artifact
-      if: always()
-      uses: actions/upload-artifact@v4
-      with:
-        name: test-report
-        path: test-report.txt
-
-    - name: Upload Test Results
-      if: always()
-      uses: actions/upload-artifact@v4
-      with:
-        name: test-results
-        path: |
-
-          ./timestamp.txt
-          ./summary.txt
-          ./failed.txt
-          ./error.txt
-          ./pr_number.txt
-          ./pr_repo.txt
-          ./pr_branch.txt
-
-    # Fail the workflow if tests failed
-    - name: Check Test Status
-      if: ${{ steps.analyze-results.outputs.result && fromJSON(steps.analyze-results.outputs.result).testsFailed }}
+name: Run Blits Tests
+
+on:
+  pull_request:
+    branches:
+      - dev
+      - master
+
+jobs:
+  run-blits-tests:
+    runs-on: ubuntu-latest
+    outputs:
+      tests_failed: ${{ steps.analyze-results.outputs.result && fromJSON(steps.analyze-results.outputs.result).testsFailed }}
+
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v3
+
+    - name: Set up Node.js
+      uses: actions/setup-node@v3
+      with:
+        node-version: 20.16.0
+
+    - name: Save PR Information
+      run: |
+        echo '${{ github.event.pull_request.number }}' > ./pr_number.txt
+        echo '${{ github.event.pull_request.head.repo.full_name }}' > ./pr_repo.txt
+        echo '${{ github.event.pull_request.head.ref }}' > ./pr_branch.txt
+
+    - name: Install Dependencies
+      run: npm install
+
+    - name: Run Tests
+      id: run-tests
+      run: npm run test > test-report.txt 2>&1 || echo "Command failed with exit code $?" >> test-report.txt
+
+    - name: Analyze Test Results
+      id: analyze-results
+      uses: actions/github-script@v6
+      with:
+        result-encoding: json
+        script: |
+          const fs = require('fs');
+          const timestamp = new Date().toISOString();
+
+          const results = {
+            timestamp,
+            summary: '',
+            testsFailed: false,
+            error: null
+          };
+
+          // Check if test report exists and has content
+          if (!fs.existsSync('test-report.txt') || !fs.statSync('test-report.txt').size) {
+            results.summary = 'Test execution error: No test output generated.';
+            results.testsFailed = true;
+            return outputResults(results);
+          }
+
+          // Read test report
+          const testOutput = fs.readFileSync('test-report.txt', 'utf8');
+
+          // Check for errors and test results
+          const hasError = testOutput.includes('Error');
+          const testSummaryMatch = testOutput.match(/passed:\s*(\d+)\s*failed:\s*(\d+)\s*of\s*(\d+)\s*tests/);
+
+          if (hasError && (!testSummaryMatch || testSummaryMatch[3] === '0')) {
+            // Error with no tests or zero tests
+            results.summary = 'Test execution encountered errors. No tests were run.';
+            results.testsFailed = true;
+            results.error = testOutput;
+          } else if (testSummaryMatch) {
+            // We have a test summary
+            const passed = parseInt(testSummaryMatch[1]);
+            const failed = parseInt(testSummaryMatch[2]);
+            const total = parseInt(testSummaryMatch[3]);
+
+            results.summary = testSummaryMatch[0];
+
+            if (failed > 0 || total === 0) {
+              results.testsFailed = true;
+
+              if (total === 0) {
+                results.summary = 'No tests were found to run. This is considered a failure.';
+              }
+            }
+          } else {
+            // No test summary found
+            results.summary = 'Test execution produced no valid test summary';
+            results.testsFailed = true;
+            results.error = testOutput;
+          }
+
+          return outputResults(results);
+
+          /**
+           * Outputs results and saves result files
+           */
+          function outputResults(results) {
+            try {
+              fs.writeFileSync('./timestamp.txt', results.timestamp);
+              fs.writeFileSync('./summary.txt', results.summary);
+              fs.writeFileSync('./failed.txt', results.testsFailed.toString());
+
+              // Save error output if it exists
+              if (results.error) {
+                fs.writeFileSync('./error.txt', results.error);
+              }
+            } catch (err) {
+              console.error('Error saving test results:', err);
+              core.warning(`Error saving test results: ${err.message}`);
+            }
+
+            return results;
+          }
+
+    - name: Upload Test Report Artifact
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: test-report
+        path: test-report.txt
+
+    - name: Upload Test Results
+      if: always()
+      uses: actions/upload-artifact@v4
+      with:
+        name: test-results
+        path: |
+
+          ./timestamp.txt
+          ./summary.txt
+          ./failed.txt
+          ./error.txt
+          ./pr_number.txt
+          ./pr_repo.txt
+          ./pr_branch.txt
+
+    # Fail the workflow if tests failed
+    - name: Check Test Status
+      if: ${{ steps.analyze-results.outputs.result && fromJSON(steps.analyze-results.outputs.result).testsFailed }}
       run: exit 1

package/.github/workflows/run-lint.yml

@@ -1,128 +1,128 @@
-name: Run ESLint
-
-on:
-  pull_request:
-    branches:
-      - dev
-      - master
-
-jobs:
-  run-lint:
-    runs-on: ubuntu-latest
-    permissions:
-      contents: read
-
-    steps:
-    - name: Checkout Code
-      uses: actions/checkout@v3
-      with:
-        # Needed to compare changes against base branch
-        fetch-depth: 0
-
-    - name: Save PR Information
-      run: |
-        echo '${{ github.event.pull_request.number }}' > ./pr_number.txt
-        echo '${{ github.event.pull_request.head.repo.full_name }}' > ./pr_repo.txt
-        echo '${{ github.event.pull_request.head.ref }}' > ./pr_branch.txt
-
-    - name: Set up Node.js
-      uses: actions/setup-node@v3
-      with:
-        node-version: 20.16.0
-
-    - name: Install Dependencies
-      run: npm install
-
-    - name: Get list of changed JS files
-      id: changed-files
-      run: |
-        # Get the list of changed .js files compared to the base branch
-        git diff --name-only --diff-filter=ACMRT origin/${{ github.base_ref }} HEAD | grep '\.js$' > js-changed-files.txt || true
-
-        if [ -s js-changed-files.txt ]; then
-          echo "has_js_changes=true" >> $GITHUB_OUTPUT
-          cat js-changed-files.txt | tr '\n' ' ' > js-changed-files-spaces.txt
-          echo "files=$(cat js-changed-files-spaces.txt)" >> $GITHUB_OUTPUT
-        else
-          echo "has_js_changes=false" >> $GITHUB_OUTPUT
-        fi
-
-    - name: Run ESLint on changed files
-      id: eslint
-      if: steps.changed-files.outputs.has_js_changes == 'true'
-      run: |
-        # Create directory for output
-        mkdir -p ./lint-results
-
-        # Run ESLint only on changed files with the project's config
-        # Filtering out warnings - only keep errors
-        npx eslint --config .eslintrc.cjs ${{ steps.changed-files.outputs.files }} -f json > ./lint-results/eslint-output.json || true
-
-        # Check if there are any errors in the ESLint output
-        if [ -s ./lint-results/eslint-output.json ]; then
-          error_count=$(node -e "
-            const results = require('./lint-results/eslint-output.json');
-            let errorCount = 0;
-            for (const file of results) {
-              errorCount += file.errorCount || 0;
-            }
-            console.log(errorCount);
-          ")
-
-          echo "errors_found=$error_count" >> $GITHUB_OUTPUT
-
-          if [ "$error_count" -gt 0 ]; then
-            # Create the formatted error report
-            echo '❌ **Linting errors found**' > ./lint-results/errors.md
-            echo '' >> ./lint-results/errors.md
-            echo 'These linting errors must be fixed before this PR can be merged:' >> ./lint-results/errors.md
-            echo '' >> ./lint-results/errors.md
-            echo '```' >> ./lint-results/errors.md
-
-            # Extract and format errors from the JSON
-            error_count=$(node -e "
-              const results = require('./lint-results/eslint-output.json');
-              let errorSummary = '';
-
-              for (const file of results) {
-                if (file.errorCount > 0) {
-                  const filePath = file.filePath.replace(process.env.GITHUB_WORKSPACE + '/', '');
-                  errorSummary += filePath + '\\n';
-
-                  for (const msg of file.messages) {
-                    if (msg.severity === 2) {
-                      errorSummary += '  Line ' + msg.line + ':' + msg.column + ': ' +
-                                     msg.message + ' (' + msg.ruleId + ')\\n';
-                    }
-                  }
-                  errorSummary += '\\n';
-                }
-              }
-
-              console.log(errorSummary.trim());
-            ")
-
-            echo "$error_count" >> ./lint-results/errors.md
-            echo '```' >> ./lint-results/errors.md
-            echo '' >> ./lint-results/errors.md
-            echo '_Run `npm run lint:fix` to automatically fix some of these issues._' >> ./lint-results/errors.md
-          fi
-        else
-          echo "errors_found=0" >> $GITHUB_OUTPUT
-        fi
-
-    - name: Upload Lint Results
-      if: steps.changed-files.outputs.has_js_changes == 'true'
-      uses: actions/upload-artifact@v4
-      with:
-        name: lint-results
-        path: |
-          ./lint-results/
-          ./pr_number.txt
-          ./pr_repo.txt
-          ./pr_branch.txt
-
-    # Fail the workflow if linting errors found
-    - name: Check Lint Status
-      if: steps.changed-files.outputs.has_js_changes == 'true' && steps.eslint.outputs.errors_found != '0'
+name: Run ESLint
+
+on:
+  pull_request:
+    branches:
+      - dev
+      - master
+
+jobs:
+  run-lint:
+    runs-on: ubuntu-latest
+    permissions:
+      contents: read
+
+    steps:
+    - name: Checkout Code
+      uses: actions/checkout@v3
+      with:
+        # Needed to compare changes against base branch
+        fetch-depth: 0
+
+    - name: Save PR Information
+      run: |
+        echo '${{ github.event.pull_request.number }}' > ./pr_number.txt
+        echo '${{ github.event.pull_request.head.repo.full_name }}' > ./pr_repo.txt
+        echo '${{ github.event.pull_request.head.ref }}' > ./pr_branch.txt
+
+    - name: Set up Node.js
+      uses: actions/setup-node@v3
+      with:
+        node-version: 20.16.0
+
+    - name: Install Dependencies
+      run: npm install
+
+    - name: Get list of changed JS files
+      id: changed-files
+      run: |
+        # Get the list of changed .js files compared to the base branch
+        git diff --name-only --diff-filter=ACMRT origin/${{ github.base_ref }} HEAD | grep '\.js$' > js-changed-files.txt || true
+
+        if [ -s js-changed-files.txt ]; then
+          echo "has_js_changes=true" >> $GITHUB_OUTPUT
+          cat js-changed-files.txt | tr '\n' ' ' > js-changed-files-spaces.txt
+          echo "files=$(cat js-changed-files-spaces.txt)" >> $GITHUB_OUTPUT
+        else
+          echo "has_js_changes=false" >> $GITHUB_OUTPUT
+        fi
+
+    - name: Run ESLint on changed files
+      id: eslint
+      if: steps.changed-files.outputs.has_js_changes == 'true'
+      run: |
+        # Create directory for output
+        mkdir -p ./lint-results
+
+        # Run ESLint only on changed files with the project's config
+        # Filtering out warnings - only keep errors
+        npx eslint --config .eslintrc.cjs ${{ steps.changed-files.outputs.files }} -f json > ./lint-results/eslint-output.json || true
+
+        # Check if there are any errors in the ESLint output
+        if [ -s ./lint-results/eslint-output.json ]; then
+          error_count=$(node -e "
+            const results = require('./lint-results/eslint-output.json');
+            let errorCount = 0;
+            for (const file of results) {
+              errorCount += file.errorCount || 0;
+            }
+            console.log(errorCount);
+          ")
+
+          echo "errors_found=$error_count" >> $GITHUB_OUTPUT
+
+          if [ "$error_count" -gt 0 ]; then
+            # Create the formatted error report
+            echo '❌ **Linting errors found**' > ./lint-results/errors.md
+            echo '' >> ./lint-results/errors.md
+            echo 'These linting errors must be fixed before this PR can be merged:' >> ./lint-results/errors.md
+            echo '' >> ./lint-results/errors.md
+            echo '```' >> ./lint-results/errors.md
+
+            # Extract and format errors from the JSON
+            error_count=$(node -e "
+              const results = require('./lint-results/eslint-output.json');
+              let errorSummary = '';
+
+              for (const file of results) {
+                if (file.errorCount > 0) {
+                  const filePath = file.filePath.replace(process.env.GITHUB_WORKSPACE + '/', '');
+                  errorSummary += filePath + '\\n';
+
+                  for (const msg of file.messages) {
+                    if (msg.severity === 2) {
+                      errorSummary += '  Line ' + msg.line + ':' + msg.column + ': ' +
+                                     msg.message + ' (' + msg.ruleId + ')\\n';
+                    }
+                  }
+                  errorSummary += '\\n';
+                }
+              }
+
+              console.log(errorSummary.trim());
+            ")
+
+            echo "$error_count" >> ./lint-results/errors.md
+            echo '```' >> ./lint-results/errors.md
+            echo '' >> ./lint-results/errors.md
+            echo '_Run `npm run lint:fix` to automatically fix some of these issues._' >> ./lint-results/errors.md
+          fi
+        else
+          echo "errors_found=0" >> $GITHUB_OUTPUT
+        fi
+
+    - name: Upload Lint Results
+      if: steps.changed-files.outputs.has_js_changes == 'true'
+      uses: actions/upload-artifact@v4
+      with:
+        name: lint-results
+        path: |
+          ./lint-results/
+          ./pr_number.txt
+          ./pr_repo.txt
+          ./pr_branch.txt
+
+    # Fail the workflow if linting errors found
+    - name: Check Lint Status
+      if: steps.changed-files.outputs.has_js_changes == 'true' && steps.eslint.outputs.errors_found != '0'
       run: exit 1

package/.husky/pre-commit

@@ -1 +1 @@
-npx lint-staged
+npx lint-staged

package/.vscode/extensions.json

@@ -1,7 +1,7 @@
-{
-	"recommendations": [
-		"editorconfig.editorconfig",
-		"dbaeumer.vscode-eslint",
-    "LightningJS.lightning-blits"
-	]
-}
+{
+	"recommendations": [
+		"editorconfig.editorconfig",
+		"dbaeumer.vscode-eslint",
+    "LightningJS.lightning-blits"
+	]
+}

package/.vscode/settings.json

@@ -1,9 +1,9 @@
-{
-    "editor.formatOnSave": false,
-    "eslint.validate": [
-      "javascript"
-    ],
-    "editor.codeActionsOnSave": {
-      "source.fixAll.eslint": "explicit"
-    }
-  }
+{
+    "editor.formatOnSave": false,
+    "eslint.validate": [
+      "javascript"
+    ],
+    "editor.codeActionsOnSave": {
+      "source.fixAll.eslint": "explicit"
+    }
+  }