3.0.0 Merge pull request #91 from SolidOS/newFace

Structure improvements

Author: bourgeoa

.github/workflows/ci.yml

@@ -9,10 +9,10 @@ permissions:
 on:
   push:
     branches:
-      - "**"
+      - main
   pull_request:
     branches:
-      - "**"  
+      - main
   workflow_dispatch:
 
 jobs:
@@ -23,7 +23,6 @@ jobs:
     strategy:
       matrix:
         node-version: 
-          - 18.x
           - 20.x
           - 22.x
 

eslint.config.mjs

@@ -1,23 +1,39 @@
-import { defineConfig, globalIgnores } from "eslint/config";
-import globals from "globals";
+import globals from 'globals'
 
-export default defineConfig([globalIgnores(["src/**/*.test.js"]), {
-    languageOptions: {
-        globals: {
-            ...globals.browser,
-            ...globals.node,
-            Atomics: "readonly",
-            SharedArrayBuffer: "readonly",
+export default [
+    {
+        ignores: [
+            'node_modules/**',
+            'coverage/**'
+        ],
+    },
+    {
+        files: ['src/**/*.js'],
+        languageOptions: {
+            globals: {
+                ...globals.browser,
+                ...globals.node,
+                Atomics: 'readonly',
+                SharedArrayBuffer: 'readonly',
+            },
         },
 
-        ecmaVersion: 8,
-        sourceType: "commonjs",
-    },
+        rules: {
+        // Code style - match TypeScript settings
+        semi: ['error', 'never'],
+        quotes: ['error', 'single'],
 
-    rules: {
-        "no-unused-vars": ["warn", {
-            argsIgnorePattern: "^_",
-            varsIgnorePattern: "^_",
-        }],
-    },
-}]);
+        // Strict checking - match TypeScript strictness
+        'no-console': 'warn',
+        'no-unused-vars': 'warn', // Match TypeScript noUnusedLocals: true
+        'no-undef': 'error',
+        strict: ['error', 'global'], // Match TypeScript alwaysStrict: true
+
+        // Additional strictness to match TypeScript behavior
+        'no-implicit-globals': 'error',
+        'prefer-const': 'error', // Encourage immutability
+        'no-var': 'error', // Use let/const only
+        'no-redeclare': 'error'
+        }
+    }
+]

jest.config.js

@@ -0,0 +1,15 @@
+export default {
+  collectCoverage: true,
+  coverageDirectory: 'coverage',
+  testEnvironment: 'jsdom',
+  testEnvironmentOptions: {
+      customExportConditions: ['node']
+  },
+  setupFilesAfterEnv: ["./test/helpers/jest.setup.js"],
+  transformIgnorePatterns: ["/node_modules/(?!lit-html).+\\.js"],
+  roots: ['<rootDir>/src', '<rootDir>/test'],
+  moduleNameMapper: {
+    '^SolidLogic$': 'solid-logic',
+    '^\\$rdf$': 'rdflib'
+  },
+}