fix(web): get deno task check passing and add it to CI

.github/workflows/web.yml

@@ -40,13 +40,13 @@ jobs:
       # - name: Lint
       # run: deno task lint
 
-      # - name: Typecheck
-      # run: deno task check
-
       # package.json based projects need to run deno install step
       - name: Install Dependencies
         run: deno install --allow-scripts
 
+      - name: Typecheck
+        run: deno task check
+
       - name: Build
         run: deno task build
 

packages/core/src/lib/file_processor.ts

@@ -649,7 +649,12 @@ class FileProcessor {
   }
 
   async * #readlines(stream: ReadableStream<Uint8Array>) {
-    const reader = stream.pipeThrough(new TextDecoderStream()).getReader()
+    // TextDecoderStream types its writable side as WritableStream<BufferSource>, which
+    // TypeScript's DOM lib does not treat as assignable to the WritableStream<Uint8Array>
+    // that pipeThrough infers. The pairing is valid at runtime (a Uint8Array is a
+    // BufferSource), so bridge the lib-level mismatch explicitly.
+    const decoder = new TextDecoderStream() as ReadableWritablePair<string, Uint8Array>
+    const reader = stream.pipeThrough(decoder).getReader()
     let buffer = ''
     let read_result = await reader.read()
     while (!read_result.done) {

packages/web/package.json

@@ -8,19 +8,29 @@
 		"build": "vite build",
 		"build:local": "LOCAL_BUILD=true vite build",
 		"prepare": "svelte-kit sync || echo ''",
-		"check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json",
-		"check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.json --watch"
+		"check": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.svelte.json",
+		"check:watch": "svelte-kit sync && svelte-check --tsconfig ./tsconfig.svelte.json --watch"
 	},
 	"dependencies": {
 		"@andykais/ts-rpc": "npm:@jsr/andykais__ts-rpc@^0.2.4",
 		"@opentelemetry/api": "1.9.0",
-		"@torm/sqlite": "npm:@jsr/torm__sqlite@^1.9.7"
+		"@std/datetime": "npm:@jsr/std__datetime@^0.225.3",
+		"@std/fmt": "npm:@jsr/std__fmt@0.225.6",
+		"@std/fs": "npm:@jsr/std__fs@0.229.3",
+		"@std/log": "npm:@jsr/std__log@^0.224.6",
+		"@std/media-types": "npm:@jsr/std__media-types@0.224.0",
+		"@std/path": "npm:@jsr/std__path@1.0.3",
+		"@std/yaml": "npm:@jsr/std__yaml@^1.0.5",
+		"@torm/sqlite": "npm:@jsr/torm__sqlite@^1.9.7",
+		"ts-pattern": "npm:ts-pattern@5.2.0",
+		"zod": "npm:zod@4.1.11"
 	},
 	"devDependencies": {
 		"@deno/vite-plugin": "1.0.6",
 		"@sveltejs/kit": "2.43.7",
 		"@sveltejs/vite-plugin-svelte": "6.2.1",
 		"@tailwindcss/vite": "4.1.14",
+		"@types/deno": "^2.5.0",
 		"svelte": "5.39.8",
 		"svelte-check": "4.3.2",
 		"tailwindcss": "4.1.14",

packages/web/src/app.d.ts

@@ -1,9 +1,15 @@
 // See https://svelte.dev/docs/kit/types#app.d.ts
 // for information about these interfaces
+import type { Forager } from '@forager/core'
+import type { Config } from '$lib/server/config.ts'
+
 declare global {
 	namespace App {
 		// interface Error {}
-		// interface Locals {}
+		interface Locals {
+			forager: Forager
+			config: Config
+		}
 		// interface PageData {}
 		// interface PageState {}
 		// interface Platform {}

packages/web/src/hooks.server.ts

@@ -29,8 +29,10 @@ if (!building) {
     forager.init()
   } else {
     if (env.FORAGER_INSTANCE) {
-      forager = env.FORAGER_INSTANCE
-      config = env.FORAGER_CONFIG
+      // In production the CLI injects a live Forager instance and parsed config
+      // through these env slots, so they are not the plain strings $env types them as.
+      forager = env.FORAGER_INSTANCE as unknown as Forager
+      config = env.FORAGER_CONFIG as unknown as Config
     } else {
       throw new Error(`FORAGER_INSTANCE must be passed to sveltekit hooks`)
     }

packages/web/src/lib/actions/focusable.ts

@@ -1,11 +1,15 @@
-export function focusable(node: HTMLElement) {
+export function focusable(node: HTMLElement, focused: boolean) {
+  const apply = (should_focus: boolean) => {
+    if (should_focus) {
+      node.focus()
+    } else {
+      node.blur()
+    }
+  }
+  apply(focused)
   return {
-    update(focused: boolean) {
-      if (focused) {
-        node.focus()
-      } else {
-        node.blur()
-      }
+    update(should_focus: boolean) {
+      apply(should_focus)
     }
   }
 

packages/web/src/lib/actions/scrollable.ts

@@ -1,4 +1,4 @@
-export function scrollable(node: HTMLElement) {
+export function scrollable(node: HTMLElement, _focused: boolean) {
   let is_visible = false
   const observer = new IntersectionObserver((entries) => {
     is_visible = entries[0].isIntersecting

packages/web/src/lib/api.ts

@@ -18,11 +18,37 @@ class ForagerTagApi extends rpc.ApiController<Context> {
   parent_delete = this.context.forager.tag.parent_delete
 }
 
+class ForagerMediaApi extends rpc.ApiController<Context> {
+  create = this.context.forager.media.create
+  update = this.context.forager.media.update
+  upsert = this.context.forager.media.upsert
+  delete = this.context.forager.media.delete
+  search = this.context.forager.media.search
+  group = this.context.forager.media.group
+  get = this.context.forager.media.get
+  thumbnail = this.context.forager.media.thumbnail
+  reload = this.context.forager.media.reload
+}
+
+class ForagerSeriesApi extends rpc.ApiController<Context> {
+  create = this.context.forager.series.create
+  update = this.context.forager.series.update
+  upsert = this.context.forager.series.upsert
+  add = this.context.forager.series.add
+  get = this.context.forager.series.get
+  search = this.context.forager.series.search
+}
+
+class ForagerViewsApi extends rpc.ApiController<Context> {
+  start = this.context.forager.views.start
+  update = this.context.forager.views.update
+}
+
 class ForagerApi extends rpc.ApiController<Context> {
-  media = this.context.forager.media
-  series = this.context.forager.series
+  media = this.module(ForagerMediaApi)
+  series = this.module(ForagerSeriesApi)
   tag = this.module(ForagerTagApi)
-  views = this.context.forager.views
+  views = this.module(ForagerViewsApi)
 
 }
 

packages/web/src/lib/base_controller.ts

@@ -26,8 +26,6 @@ abstract class BaseController {
     if (this.#config) return this.#config
     else throw new Error(`Controller::config not initialized`)
   }
-
-  abstract handlers: {}
 }
 
 export { BaseController }

packages/web/src/lib/components/Header.svelte

@@ -8,6 +8,7 @@
     ...props
   }: {
     title: string
+    height?: number
     children: SvelteHTMLElements['div']['children']
   } = $props()
 

packages/web/src/lib/components/SelectInput.svelte

@@ -4,10 +4,10 @@
     value: string
   }
   interface Props {
-    label: string
-    options: SelectOption
+    label?: string
+    options: SelectOption[]
     value: string
-    onchange: () => void | undefined
+    onchange?: (e: Event) => void
   }
   let {
     label,