update esling config with svelte support and unused-vars warning, fix linter warnings

Author: sjsarsa

.eslintrc.json

@@ -1,10 +1,32 @@
 import { defineConfig, globalIgnores } from 'eslint/config'
 import typescriptEslint from '@typescript-eslint/eslint-plugin'
 import tsParser from '@typescript-eslint/parser'
+import svelteEslintParser from 'svelte-eslint-parser'
+import eslintConfigPrettier from 'eslint-config-prettier/flat'
 
 export default defineConfig([
   globalIgnores(['**/out', '**/dist', '**/*.d.ts']),
   {
+    rules: {
+      '@/semi': ['warn', 'never'],
+      curly: 'warn',
+      'no-unused-vars': [
+        'warn',
+        {
+          vars: 'all',
+          argsIgnorePattern: '^_',
+          varsIgnorePattern: '^_',
+          caughtErrorsIgnorePattern: '^_',
+        },
+      ],
+      eqeqeq: 'warn',
+      'no-throw-literal': 'warn',
+      semi: ['warn', 'never'],
+      quotes: ['warn', 'single'],
+    },
+  },
+  {
+    files: ['**/*.{js,jsx,mjs,cjs,ts,tsx,json}'],
     plugins: {
       '@typescript-eslint': typescriptEslint,
     },
@@ -23,13 +45,16 @@ export default defineConfig([
           format: ['camelCase', 'PascalCase'],
         },
       ],
-
-      '@/semi': ['warn', 'never'],
-      curly: 'warn',
-      eqeqeq: 'warn',
-      'no-throw-literal': 'warn',
-      semi: ['warn', 'never'],
-      quotes: ['warn', 'single'],
     },
   },
+  {
+    files: ['**/*.svelte'],
+    languageOptions: {
+      parser: svelteEslintParser,
+      parserOptions: {
+        parser: tsParser,
+      },
+    },
+  },
+  eslintConfigPrettier,
 ])

eslint.config.mjs

@@ -252,7 +252,8 @@
     "svelte-watch": "rollup -c -w --bundleConfigAsCjs",
     "svelte-compile": "rollup -c --bundleConfigAsCjs",
     "pretest": "npm run compile && npm run lint",
-    "lint": "eslint src --ext ts",
+    "lint": "eslint src webviews --ext ts,svelte",
+    "lint-fix": "npm run lint -- --fix",
     "format-all": "prettier --write \"./**/*.{js,jsx,mjs,cjs,ts,tsx,json,svelte}\"",
     "test": "vscode-test",
     "test:svelte": "npm run svelte-compile && vitest --config vitest.config.mjs --environment jsdom --run"
@@ -275,7 +276,11 @@
     "@vscode/test-cli": "^0.0.10",
     "@vscode/test-electron": "^2.5.2",
     "eslint": "^9.25.1",
+    "eslint-config-prettier": "^10.1.8",
+    "eslint-plugin-svelte": "^3.11.0",
     "jsdom": "^26.1.0",
+    "prettier": "^3.6.2",
+    "prettier-plugin-svelte": "^3.4.0",
     "rollup": "^4.40.1",
     "rollup-plugin-postcss": "^4.0.2",
     "rollup-plugin-svelte": "^7.2.2",
@@ -284,8 +289,6 @@
     "svelte-preprocess": "^6.0.3",
     "tslib": "^2.8.1",
     "typescript": "^5.8.3",
-    "vitest": "^3.1.3",
-    "prettier": "^3.6.2",
-    "prettier-plugin-svelte": "^3.4.0"
+    "vitest": "^3.1.3"
   }
 }

package-lock.json

@@ -320,7 +320,6 @@ export default class ExtensionStateManager {
                 Formatting.normalizeSeparator(parsedTaskDir + path.sep + taskFile.file_name),
               )
             } else {
-              const fileNameToOsPath = Formatting.normalizePath(taskFile.file_name)
               const pathParts = timData.path.split('/')
               const demo = pathParts.at(-1)
               if (demo) {

package.json

@@ -14,12 +14,10 @@ import {
   Course,
   LoginData,
   Task,
-  TaskCreationFeedback,
   TaskPoints,
   TimData,
   UserData,
 } from '../common/types'
-import Formatting from '../common/formatting'
 import { parseCoursesFromJson } from '../utilities/parsers'
 import ExtensionStateManager from './ExtensionStateManager'
 import path from 'path'
@@ -266,7 +264,7 @@ export default class Tide {
    * @param ideTaskId ide_task_id of the task that the points are fetched for
    * @param callback unused at the moment. TODO: Remove?
    */
-  public static async getTaskPoints(taskSetPath: string, ideTaskId: string, callback: any) {
+  public static async getTaskPoints(taskSetPath: string, ideTaskId: string, _callback: any) {
     try {
       vscode.commands.executeCommand('tide.setPointsUpdating')
       await this.runAndHandle(

src/api/ExtensionStateManager.ts

@@ -134,7 +134,7 @@ export function registerCommands(ctx: vscode.ExtensionContext) {
               return
             }
             Tide.submitTask(taskPath, callback)
-          } catch (error) {
+          } catch (_error) {
             vscode.window.showErrorMessage('Error occurred during the submit: ${error}')
           }
         }

src/api/tide.ts

@@ -10,7 +10,7 @@
 import * as vscode from 'vscode'
 import Logger from '../utilities/logger'
 
-export function registerEventListeners(ctx: vscode.ExtensionContext) {
+export function registerEventListeners(_ctx: vscode.ExtensionContext) {
   /**
    * Listens to changes in configuration.
    */

src/init/commands.ts

@@ -1,8 +1,6 @@
 import * as vscode from 'vscode'
-import { Course, TimData } from '../common/types'
-import path from 'path'
+import { TimData } from '../common/types'
 import ExtensionStateManager from '../api/ExtensionStateManager'
-import Formatting from '../common/formatting'
 
 /**
  * Class for Status Bar item displaying answer limit related information