Fixup linting

.prettierrc.json

@@ -11,6 +11,7 @@
   "importOrder": [
      "^@passlock/shared/(.*)$", 
      "^@passlock/client/(.*)$", 
+     "^@passlock/sveltekit/(.*)$", 
      "^[./]"
   ],
   "importOrderSeparation": true,

package.json

@@ -6,8 +6,13 @@
   "type": "module",
   "scripts": {
     "clean": "pnpm -r run clean",
-    "typecheck": "pnpm -r run typecheck",
     "build": "pnpm -r run build",
+    "build:clean": "pnpm -r run build:clean",
+    "build:readme": "pnpm -r run build:readme",
+    "build:production": "pnpm -r run build:production",
+    "format": "pnpm -r run format",
+    "lint": "pnpm -r run lint",
+    "lint:fix": "pnpm -r run lint:fix",
     "upgrade:deps": "ncu --peer -x effect -x @effect/* -x @passlock/* -u && pnpm -r run upgrade:deps",
     "upgrade:pnpm": "corepack use pnpm@latest"
   },

packages/client/README.md

@@ -1,4 +1,7 @@
-<!-- The pnpm script build:readme replaces the tokens in this file -->
+<!-- 
+The pnpm script build:readme replaces tokens 
+in README.template.md and outputs to README.md 
+-->
 <div align="center">
   <a href="https://github.com/passlock-dev/passlock">
     <img src="https://github.com/passlock-dev/passkeys-frontend/assets/208345/53ee00d3-8e6c-49ea-b43c-3f901450c73b" alt="Passlock logo" width="80" height="80">

packages/client/README.template.md

@@ -1,4 +1,7 @@
-<!-- The pnpm script build:readme replaces the tokens in this file -->
+<!-- 
+The pnpm script build:readme replaces tokens 
+in README.template.md and outputs to README.md 
+-->
 <div align="center">
   <a href="#{GITHUB_REPO}#">
     <img src="#{PASSLOCK_LOGO}#" alt="Passlock logo" width="80" height="80">

packages/client/eslint.config.mjs

@@ -0,0 +1,34 @@
+import eslint from '@eslint/js'
+import eslintPluginPrettierRecommended from 'eslint-plugin-prettier/recommended'
+import globals from 'globals'
+import tseslint from 'typescript-eslint'
+
+export default tseslint.config(
+  eslint.configs.recommended,
+  ...tseslint.configs.strict,
+  eslintPluginPrettierRecommended,
+  {
+    languageOptions: {
+      ecmaVersion: 2021,
+      sourceType: 'module',
+      globals: {
+        ...globals.browser,
+      },
+    },
+    files: ['**/*.ts', '**/*.js'],
+    ignores: ['/*', '!/src'],
+    rules: {
+      '@typescript-eslint/no-unused-vars': [
+        'error',
+        {
+          ignoreRestSiblings: true,
+          argsIgnorePattern: '^_',
+          varsIgnorePattern: '^_',
+          caughtErrorsIgnorePattern: '^_',
+        },
+      ],
+      '@typescript-eslint/consistent-type-definitions': ['warn', 'type'],
+      '@typescript-eslint/consistent-type-imports': ['error'],
+    },
+  },
+)

packages/client/package.json

@@ -55,9 +55,9 @@
     "build:clean": "pnpm run clean && pnpm run build",
     "build:readme": "LATEST=${npm_package_version} tsx scripts/replace-readme-tokens.ts",
     "replaceTokens": "LATEST=${npm_package_version} tsx scripts/replace-tokens.ts",
-    "build:production": "pnpm run replaceTokens && echo '' && publint",
-    "format": "prettier --write \"src/**/*.+(js|ts|json)\"",
-    "lint": "eslint --ext .ts src",
+    "build:production": "pnpm run build:clean && pnpm run replaceTokens && echo '' && publint",
+    "format": "prettier --write \"(src|scripts)/**/*.+(js|ts|json)\"",
+    "lint": "eslint ./src",
     "lint:fix": "pnpm run lint --fix",
     "prepublishOnly": "pnpm run build:production",
     "upgrade:deps": "ncu --peer -x effect -x @effect/* -u"
@@ -80,6 +80,7 @@
     "@vitest/ui": "^2.0.5",
     "eslint": "^9.9.0",
     "eslint-config-prettier": "^9.1.0",
+    "globals": "^15.9.0",
     "jsdom": "^24.1.1",
     "prettier": "^3.3.3",
     "publint": "^0.2.9",

packages/client/scripts/replace-readme-tokens.ts

@@ -1,8 +1,9 @@
 import { fileURLToPath } from 'node:url'
+
 import { replaceReadmeTokens } from '@passlock/shared/scripts/replace-readme-tokens.js'
 
 // see https://stackoverflow.com/a/60309682
 if (process.argv[1] === fileURLToPath(import.meta.url)) {
-	console.log('Replacing packages/client tokens')
-	await replaceReadmeTokens(import.meta.url)
-}
+  console.log('Replacing packages/client tokens')
+  await replaceReadmeTokens(import.meta.url)
+}

packages/client/scripts/replace-tokens.ts

@@ -1,10 +1,11 @@
 import { fileURLToPath } from 'node:url'
-import { replaceReadmeTokens } from '@passlock/shared/scripts/replace-readme-tokens.js'
+
 import { replaceCodeTokens } from '@passlock/shared/scripts/replace-code-tokens.js'
+import { replaceReadmeTokens } from '@passlock/shared/scripts/replace-readme-tokens.js'
 
 // see https://stackoverflow.com/a/60309682
 if (process.argv[1] === fileURLToPath(import.meta.url)) {
-	console.log('Replacing packages/client tokens')
-	await replaceReadmeTokens(import.meta.url)
-	await replaceCodeTokens(import.meta.url)
-}
+  console.log('Replacing packages/client tokens')
+  await replaceReadmeTokens(import.meta.url)
+  await replaceCodeTokens(import.meta.url)
+}

packages/client/src/authentication/authenticate.fixture.ts

@@ -1,13 +1,15 @@
+import { Effect as E, Layer as L, Option as O } from 'effect'
+
 import {
   OptionsRes,
   VerificationReq,
   VerificationRes,
 } from '@passlock/shared/dist/rpc/authentication.js'
 import { IsExistingUserRes, VerifyEmailRes } from '@passlock/shared/dist/rpc/user.js'
 import type { AuthenticationCredential } from '@passlock/shared/dist/schema/passkey.js'
-import { Effect as E, Layer as L, Option as O } from 'effect'
-import { AuthenticationClient } from '../rpc/authentication.js'
+
 import * as Fixtures from '../test/fixtures.js'
+import { AuthenticationClient } from '../rpc/authentication.js'
 import { type AuthenticationRequest, GetCredential } from './authenticate.js'
 
 export const session = 'session'

packages/client/src/authentication/authenticate.test.ts

@@ -1,9 +1,10 @@
 import { Effect as E, Layer as L, Layer, LogLevel, Logger, Option as O, pipe } from 'effect'
 import { describe, expect, test, vi } from 'vitest'
 import { mock } from 'vitest-mock-extended'
+
+import * as Fixture from './authenticate.fixture.js'
 import { AuthenticationClient } from '../rpc/authentication.js'
 import { StorageService } from '../storage/storage.js'
-import * as Fixture from './authenticate.fixture.js'
 import { AuthenticateServiceLive, AuthenticationService, GetCredential } from './authenticate.js'
 
 describe('authenticate should', () => {

packages/client/src/authentication/authenticate.ts

@@ -5,6 +5,8 @@ import {
   type CredentialRequestOptionsJSON,
   parseRequestOptionsFromJSON,
 } from '@github/webauthn-json/browser-ponyfill'
+import { Context, Effect as E, Layer, flow, pipe } from 'effect'
+
 import { InternalBrowserError, type NotSupported } from '@passlock/shared/dist/error/error.js'
 import {
   type OptionsErrors,
@@ -14,7 +16,7 @@ import {
 } from '@passlock/shared/dist/rpc/authentication.js'
 import type { AuthenticationCredential } from '@passlock/shared/dist/schema/passkey.js'
 import type { Principal } from '@passlock/shared/dist/schema/principal.js'
-import { Context, Effect as E, Layer, flow, pipe } from 'effect'
+
 import { Capabilities } from '../capabilities/capabilities.js'
 import { AuthenticationClient } from '../rpc/authentication.js'
 import { StorageService } from '../storage/storage.js'

packages/client/src/capabilities/capabilities.ts

@@ -1,9 +1,10 @@
 /**
  * Test if the browser supports passkeys, conditional UI etc
  */
-import { NotSupported } from '@passlock/shared/dist/error/error.js'
 import { Context, Effect as E, Layer, identity, pipe } from 'effect'
 
+import { NotSupported } from '@passlock/shared/dist/error/error.js'
+
 /* Service */
 
 export class Capabilities extends Context.Tag('@services/Capabilities')<

packages/client/src/connection/connection.fixture.ts

@@ -1,5 +1,7 @@
-import { ConnectRes } from '@passlock/shared/dist/rpc/connection.js'
 import { Effect as E, Layer as L } from 'effect'
+
+import { ConnectRes } from '@passlock/shared/dist/rpc/connection.js'
+
 import { ConnectionClient } from '../rpc/connection.js'
 
 export const preConnectRes = new ConnectRes({ warmed: true })

packages/client/src/connection/connection.test.ts

@@ -1,10 +1,11 @@
 import { Effect as E, Layer as L, Layer, LogLevel, Logger, pipe } from 'effect'
 import { describe, expect, test } from 'vitest'
 import { mock } from 'vitest-mock-extended'
+
+import * as Fixture from './connection.fixture.js'
 import { Dispatcher } from '../rpc/client.js'
 import { RpcConfig } from '../rpc/config.js'
 import { ConnectionClient } from '../rpc/connection.js'
-import * as Fixture from './connection.fixture.js'
 import { ConnectionService, ConnectionServiceLive } from './connection.js'
 
 describe('preConnect should', () => {

packages/client/src/connection/connection.ts

@@ -2,6 +2,7 @@
  * Hits the rpc endpoint to warm up a lambda
  */
 import { Context, Effect as E, Layer, flow, pipe } from 'effect'
+
 import { Dispatcher } from '../rpc/client.js'
 import type { RpcConfig } from '../rpc/config.js'
 import { ConnectionClient } from '../rpc/connection.js'