Merge pull request #973 from tradingstrategy-ai/svelte-upgrades

Upgrades (node, svelte, sveltekit, etc.)

Author: kenkunz

.env.development

@@ -16,6 +16,11 @@ TS_PRIVATE_ADMIN_PW=youbetcha
 # Clear geo-blocks for development
 TS_PUBLIC_GEO_BLOCK="{}"
 
+# Use dummy Cloudflare Turnstile keys for dev
+# See: https://developers.cloudflare.com/turnstile/troubleshooting/testing/
+TS_PUBLIC_TURNSTILE_SITE_KEY="1x00000000000000000000AA"
+TS_PRIVATE_TURNSTILE_SECRET_KEY="1x0000000000000000000000000000000AA"
+
 # Uncomment to test chain maintenance error
 # TS_PUBLIC_CHAINS_UNDER_MAINTENANCE=`{ "binance": "BNB Chain" }`
 

.github/workflows/javascript.yml

@@ -21,7 +21,7 @@ jobs:
       - uses: actions/checkout@v4
       - uses: actions/setup-node@v4
         with:
-          node-version: 20.11
+          node-version: 20.19
           cache: npm
       - name: Install dependencies
         run: npm ci
@@ -40,7 +40,7 @@ jobs:
           submodules: 'recursive'
       - uses: actions/setup-node@v4
         with:
-          node-version: 20.11
+          node-version: 20.19
           cache: npm
       - name: Install dependencies
         run: npm ci

.nvmrc

@@ -1 +1 @@
-20.11
+20.19

Dockerfile

@@ -1,7 +1,7 @@
 #######################################
 # Build stage
 #######################################
-FROM node:20.11 as builder
+FROM node:20.19 as builder
 
 WORKDIR /app
 
@@ -22,7 +22,7 @@ RUN npm run build
 #######################################
 # Serve stage
 #######################################
-FROM node:20.11-slim
+FROM node:20.19-slim
 
 WORKDIR /app
 

README.md

@@ -6,7 +6,7 @@ A frontend for [Trading Strategy protocol](https://tradingstrategy.ai).
 
 ## Install NPM dependencies
 
-Node >=20.11 required.
+Node >=20.19 required.
 
 ```shell
 npm install

eslint.config.js

@@ -1,33 +1,39 @@
 import prettier from 'eslint-config-prettier';
-import js from '@eslint/js';
 import { includeIgnoreFile } from '@eslint/compat';
+import js from '@eslint/js';
 import svelte from 'eslint-plugin-svelte';
 import globals from 'globals';
 import { fileURLToPath } from 'node:url';
 import ts from 'typescript-eslint';
+import svelteConfig from './svelte.config.js';
+
 const gitignorePath = fileURLToPath(new URL('./.gitignore', import.meta.url));
 
 export default ts.config(
 	includeIgnoreFile(gitignorePath),
 	js.configs.recommended,
 	...ts.configs.recommended,
-	...svelte.configs['flat/recommended'],
+	...svelte.configs.recommended,
 	prettier,
-	...svelte.configs['flat/prettier'],
+	...svelte.configs.prettier,
 	{
 		languageOptions: {
-			globals: {
-				...globals.browser,
-				...globals.node
-			}
+			globals: { ...globals.browser, ...globals.node }
+		},
+		rules: {
+			// typescript-eslint strongly recommend that you do not use the no-undef lint rule on TypeScript projects.
+			// see: https://typescript-eslint.io/troubleshooting/faqs/eslint/#i-get-errors-from-the-no-undef-rule-about-global-variables-not-being-defined-even-though-there-are-no-typescript-errors
+			'no-undef': 'off'
 		}
 	},
 	{
-		files: ['**/*.svelte'],
-
+		files: ['**/*.svelte', '**/*.svelte.ts', '**/*.svelte.js'],
 		languageOptions: {
 			parserOptions: {
-				parser: ts.parser
+				projectService: true,
+				extraFileExtensions: ['.svelte'],
+				parser: ts.parser,
+				svelteConfig
 			}
 		}
 	}