chore(deps): bump knip (fixes SvelteKit config detection) (#342)

* chore(deps): bump knip (fixes SvelteKit config detection)

knip 6.17.1 -> 6.29.0 via the dev catalog.

The SK3 fix landed in knip 6.19.0 (webpro-nl/knip#1810, "support new
optional sveltekit config pattern via vite config"), written against
sveltekit@3.0.0-next.4 -- our exact case. The plugin's config list is now
['svelte.config.js', ...viteConfig], so it finds the Kit options that
SvelteKit 3 keeps inside vite.config.ts.

Removed as a result:
- workspaces["apps/grove"].sveltekit.config -- auto-detected now; the
  workspace key held nothing else and is gone.
- workspaces["."].entry ["vite.config.ts"] -- knip 6.21.0 detects vite
  configs on its own and now reports this as a redundant-entry config
  hint, which is fatal under treatConfigHintsAsErrors.
- console's hand-mapped $app/env + $app/server paths, replaced by one
  ignoreUnresolved that mirrors what the plugin contributes.

apps/console cannot use the auto-detection. resolveFromAST only takes the
vite-config branch when it statically sees `sveltekit` imported from
@sveltejs/kit/vite, and console configures Kit through
svelte-plugin-composer's kit() wrapper. Without help the plugin
contributes nothing there -- the bare bump reported 125 unused files, 8
unused deps, 16 unresolved $lib imports and 68 unused exports/types. So
console restates the route/hook production entries and the $lib alias
explicitly, with a comment. Rewriting console's real build config or
adding a decoy svelte.config.js to satisfy the linter were both rejected.

Also drops 5 dead type re-exports the newer knip detects: four in
data/cost.ts and TerminalTarget in server/api/console-api.ts. Every
consumer imports those from the originating module, not the barrel.

Verified: lint:knip, lint:knip:prod, check, test (264), manypkg,
typesync:check and a frozen-lockfile install all pass. Canary unused
files/exports in both console and grove are still reported, confirming
the clean run reflects a live graph rather than an empty one.

Co-Authored-By: Opus <noreply@anthropic.com>

* chore(deps): update knip to 6.31

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Jwe5UwSrJX5HQkjSskfWYv

* chore: drop decisions file + dedupe lockfile

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Jwe5UwSrJX5HQkjSskfWYv

* refactor(console): drop svelte-plugin-composer for native SvelteKit wiring

Configure the console app’s Vite plugins directly instead of through
svelte-plugin-composer: spread effect() (svelte-effect-runtime) and ts(true)
(svelte-global-typescript) ahead of a directly-called sveltekit(), which now
carries adapter/kit/compilerOptions inline (this SvelteKit build no longer reads
svelte.config.js). knip’s SvelteKit plugin can now statically see the sveltekit()
call, so the console workspace drops the manual $lib/$app/route-hook stand-ins
and mirrors apps/grove.

Verified: console build green with identical SER transform output (18 .remote.js
server chunks, zero ServerOnlyImportError, effect_remote_ client wrapper),
svelte-check at parity (31 pre-existing errors, unchanged), knip + knip --strict
green, lockfile deduped.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Jwe5UwSrJX5HQkjSskfWYv

---------

Co-authored-by: Janet <janet@petalcat.dev>
Co-authored-by: Opus <noreply@anthropic.com>
Co-authored-by: returnsvoidjanet <returnsvoidjanet@users.noreply.github.com>

Author: 3 people

apps/console/package.json

@@ -56,7 +56,6 @@
 		"fontless": "catalog:prod",
 		"svelte-check": "catalog:dev",
 		"svelte-global-typescript": "catalog:",
-		"svelte-plugin-composer": "catalog:",
 		"typescript": "catalog:dev",
 		"vite": "catalog:dev",
 		"vitest": "catalog:dev"

apps/console/src/lib/data/cost.ts

@@ -6,15 +6,7 @@ import type {
 	CostDimension,
 } from "@petalnet/types";
 
-export type {
-	CostComparisonMetric,
-	CostComparisonMetricKey,
-	CostComparisonReceipt,
-	CostComparisonRequest,
-	CostComparisonResult,
-	CostComparisonSide,
-	CostDimension,
-} from "@petalnet/types";
+export type { CostComparisonMetric, CostComparisonResult, CostDimension } from "@petalnet/types";
 
 export interface DailyCost {
 	day: string;

apps/console/src/lib/server/api/console-api.ts

@@ -98,7 +98,7 @@ import {
 } from "../domain/terminal/service.ts";
 import { readUpdateApprovals } from "../domain/updates/approvals.ts";
 
-export type { TerminalAdapter, TerminalTarget } from "../domain/terminal/service.ts";
+export type { TerminalAdapter } from "../domain/terminal/service.ts";
 
 const askRequestSchema = Schema.Struct({
 	question: Schema.String.check(Schema.isMinLength(1), Schema.isMaxLength(2_000)),

apps/console/vite.config.ts

@@ -1,11 +1,11 @@
 import adapter from "@petalnet/svelte-ws";
 import { websocket } from "@petalnet/svelte-ws/vite";
 import { sentrySvelteKit } from "@sentry/sveltekit";
+import { sveltekit } from "@sveltejs/kit/vite";
 import tailwindcss from "@tailwindcss/vite";
 import { fontless } from "fontless";
 import { effect } from "svelte-effect-runtime";
 import { ts } from "svelte-global-typescript";
-import { compose, kit } from "svelte-plugin-composer";
 import { defineConfig } from "vitest/config";
 
 export default defineConfig({
@@ -28,19 +28,19 @@ export default defineConfig({
 				{ name: "Geist Mono", provider: "fontsource", weights: [400, 500] },
 			],
 		}),
-		...compose([
-			effect(),
-			ts(true),
-			kit({
-				adapter: adapter(),
-				compilerOptions: { experimental: { async: true } },
-				kit: {
-					tracing: { server: true },
-					experimental: {
-						remoteFunctions: true,
-					},
-				},
-			}),
-		]),
+		// SER's svelte-transform is a normal-order Vite transform, so it must precede the
+		// SvelteKit plugin to lower `<script effect>` / yield* syntax before Svelte compiles.
+		// `ts(true)` (svelte-global-typescript) likewise runs before sveltekit() for the build;
+		// its matching preprocess is passed to sveltekit() below so editor tooling parses plain
+		// <script> blocks as TypeScript (this SvelteKit build no longer reads svelte.config.js —
+		// all config is passed directly to the sveltekit() plugin).
+		...effect(),
+		ts(true),
+		sveltekit({
+			adapter: adapter(),
+			compilerOptions: { experimental: { async: true } },
+			tracing: { server: true },
+			experimental: { remoteFunctions: true },
+		}),
 	],
 });

knip.config.ts

@@ -6,9 +6,6 @@ export default {
 	ignoreExportsUsedInFile: { type: true, interface: true },
 	treatConfigHintsAsErrors: true,
 	workspaces: {
-		".": {
-			entry: ["vite.config.ts"],
-		},
 		"apps/collegemap": {
 			drizzle: {
 				config: [],
@@ -17,13 +14,6 @@ export default {
 		},
 		"apps/console": {
 			ignoreDependencies: ["crossws!"],
-			sveltekit: {
-				config: ["vite.config.ts"],
-			},
-			paths: {
-				"$app/env": ["node_modules/@sveltejs/kit/types/index.d.ts"],
-				"$app/server": ["node_modules/@sveltejs/kit/types/index.d.ts"],
-			},
 			// Scripts are deploy/ops entrypoints (seed, bridge daemon, token mint, capability
 			// install) — production surface, hence the `!` markers.
 			entry: [
@@ -36,9 +26,6 @@ export default {
 			],
 		},
 		"apps/grove": {
-			sveltekit: {
-				config: ["vite.config.ts"],
-			},
 			entry: ["effectdb.config.ts!", "src/env.ts!"],
 		},
 		"apps/storybook": {