Structural improvements

.github/workflows/CI.yml

@@ -22,7 +22,6 @@ jobs:
     strategy:
       matrix:
         node-version: 
-          - 18.x
           - 20.x
           - 22.x
 

babel.config.js

@@ -1,13 +1,14 @@
-module.exports = {
+export default {
   presets: [
     [
-      "@babel/preset-env",
+      '@babel/preset-env',
       {
         targets: {
-          node: "current",
+          node: 'current',
         },
+        modules: 'cjs',
       },
     ],
-    "@babel/preset-typescript",
+    '@babel/preset-typescript',
   ],
-};
+}

dev/context.ts

@@ -1,6 +1,6 @@
-import { DataBrowserContext, PaneRegistry } from "pane-registry";
-import { solidLogicSingleton, store } from "solid-logic";
-import { longChatPane } from "../src/longChatPane";
+import { DataBrowserContext, PaneRegistry } from 'pane-registry'
+import { solidLogicSingleton, store } from 'solid-logic'
+import { longChatPane } from '../src/longChatPane'
 import { LiveStore } from 'rdflib'
 
 export const context: DataBrowserContext = {

dev/index.js

@@ -106,7 +106,7 @@ async function appendChatPane (dom, uri) {
   dom.body.appendChild(paneDiv)
 }
 
-const webIdToShow = 'https://solidos.solidcommunity.net/Team/SolidOs%20team%20chat/index.ttl#this'
+const webIdToShow = 'https://timea.solidcommunity.net/New%20chat%20example/index.ttl#this'
 
 fetcher.load(webIdToShow).then(() => {
   appendChatPane(document, webIdToShow)

eslint.config.mjs

@@ -1,23 +1,84 @@
-import globals from "globals";
+import globals from 'globals'
+import tseslintPlugin from '@typescript-eslint/eslint-plugin'
+import tsParser from '@typescript-eslint/parser'
+import neostandard from 'neostandard'
 
 export default [
+  ...neostandard(),
   {
+    ignores: [
+      '**/*.html',
+      '**/*.md',
+      '**/*.json',
+      'node_modules/**',
+      'dist/**',
+      'lib/**'
+    ],
+  },
+  {
+    files: ['src/**/*.js', 'src/**/*.cjs', 'src/**/*.mjs'],
     languageOptions: {
       globals: {
         ...globals.browser,
         ...globals.node,
-        Atomics: "readonly",
-        SharedArrayBuffer: "readonly",
+        Atomics: 'readonly',
+        SharedArrayBuffer: 'readonly',
       },
     },
     rules: {
-      "no-unused-vars": [
-        "warn",
-        {
-          argsIgnorePattern: "^_",
-          varsIgnorePattern: "^_",
-        },
-      ],
+      semi: ['error', 'never'],
+      quotes: ['error', 'single'],
+      'no-console': 'error',
+      'no-unused-vars': 'error',
+      'no-undef': 'error'
     },
   },
-];
+  {
+    files: ['src/**/*.ts'],
+    plugins: {
+      '@typescript-eslint': tseslintPlugin,
+    },
+
+    languageOptions: {
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+        Atomics: 'readonly',
+        SharedArrayBuffer: 'readonly',
+      },
+      parser: tsParser,
+      parserOptions: {
+        project: ['./tsconfig.json']
+      },
+    },
+    rules: {
+      semi: ['error', 'never'],
+      quotes: ['error', 'single'],
+      'no-console': 'error',
+      'no-unused-vars': 'off',
+      '@typescript-eslint/no-unused-vars': ['warn', {
+        argsIgnorePattern: '^_',
+        varsIgnorePattern: '^_',
+      }],
+      // '@typescript-eslint/no-explicit-any': 'warn', - codebase not ready for this
+    },
+  },
+  {
+    files: ['test/**/*.ts'],
+    plugins: {
+      '@typescript-eslint': tseslintPlugin,
+    },
+    languageOptions: {
+      parser: tsParser,
+      parserOptions: {
+        project: ['./tsconfig.test.json'],
+      },
+    },
+    rules: {
+      semi: ['error', 'never'],
+      quotes: ['error', 'single'],
+      'no-console': 'off', // Allow console in tests
+      'no-undef': 'off', // Tests may define globals
+    }
+  }
+]

jest.config.js

@@ -1,8 +1,8 @@
 module.exports = {
   testEnvironment: 'jsdom',
-  setupFilesAfterEnv: ["./jest.setup.ts"],
-  transformIgnorePatterns: ["/node_modules/(?!lit-html).+\\.js"],
+  setupFilesAfterEnv: ['./jest.setup.ts'],
+  transformIgnorePatterns: ['/node_modules/(?!lit-html).+\\.js'],
   testEnvironmentOptions: {
-      customExportConditions: ['node']
+    customExportConditions: ['node']
   }
-};
+}

jest.setup.ts

@@ -1,8 +1,8 @@
-import "@testing-library/jest-dom";
-import fetchMock from "jest-fetch-mock";
-const { TextEncoder, TextDecoder } = require('util')
+import '@testing-library/jest-dom'
+import fetchMock from 'jest-fetch-mock'
+import { TextEncoder, TextDecoder } from 'util'
 
 global.TextEncoder = TextEncoder;
 global.TextDecoder = TextDecoder
 
-fetchMock.enableMocks();
+fetchMock.enableMocks()