@imfusion/web-ui 0.10.1-dev.11.g9f5bc06e → 0.10.1-dev.13.g0d178239

package/CHANGELOG.md

@@ -34,6 +34,16 @@
     `defaultOptions`, where it blocked invalidation refetches.
   - `imf-web-ui-conventions` puts a route's own views, forms, hooks, and tests in a `-<file-name>/` folder beside the route
     file, and reserves `partials/` for parts that more than one route uses.
+  - `imf-web-ui-conventions` lifts a value into a custom property on the component's root only when it repeats within that
+    stylesheet or a state or variant changes several descendants. A value that only recurs across files stays a literal in
+    each, and a shared component owns it when those files show one design element.
+  - `imf-web-ui-conventions` exempts TanStack options factories (`queryOptions`, `mutationOptions`) from the explicit return
+    type on exported functions, since spelling it out loses the typed query key.
+  - `imf-web-ui-conventions` keeps the Git hooks in `.vite-hooks/` and installs the Vite+ dispatcher with `vp config` from
+    `git:config`.
+- CI and verification — Nikolaus Brunner <brunner@imfusion.com>
+  - The Git hooks live in `.vite-hooks/` and run through the Vite+ hook dispatcher, which `npm run git:config` installs.
+    Existing clones and worktrees rerun `npm run git:config`; until then their hooks do not run.
 - Components ([WEBSDK-247](https://imfusion.atlassian.net/browse/WEBSDK-247)) — Nikolaus Brunner <brunner@imfusion.com>
   - The AppShell navbar drawer, `Code`, and `ImageDisplayOptions` scroll in a `ScrollArea`, with its overlay thumb and edge
     fades.

package/README.md

@@ -65,8 +65,10 @@ npm install
 npm run git:config
 ```
 
-`git:config` sets the tracked hooks path, enables rebase pulls, and requires fast-forward merges. Every checkout and worktree
-needs its own `node_modules`; the hooks stop with "run npm ci" without one.
+The hooks are tracked in `.vite-hooks/`. `git:config` installs the Vite+ hook dispatcher into the git-ignored
+`.vite-hooks/_/` and points the hooks path at it, enables rebase pulls, and requires fast-forward merges. Every checkout and
+worktree needs its own `node_modules` and dispatcher, so run `npm ci` and `npm run git:config` in each; without the
+dispatcher the hooks do not run at all.
 
 The pre-commit hook runs `vp staged` against the staged state: it lints and format-checks the repository, checks that every
 generated `*.gen.*` file matches its sources (`npm run verify:generated`), and runs the pinned-dependency, Knip, type-check,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@imfusion/web-ui",
-  "version": "0.10.1-dev.11.g9f5bc06e",
+  "version": "0.10.1-dev.13.g0d178239",
   "description": "The official Web UI component library for ImFusion web apps",
   "author": "ImFusion GmbH",
   "homepage": "https://imfusion.com",
@@ -95,7 +95,7 @@
     "skills:eval:compare": "tsx src/llms/evals/runner/compare.ts",
     "skills:eval:affected": "tsx src/llms/evals/runner/affected.ts",
     "skills:eval:report": "tsx src/llms/evals/runner/report.ts",
-    "git:config": "git config core.hooksPath .githooks && git config pull.rebase true && git config merge.ff only"
+    "git:config": "git config core.hooksPath .vite-hooks/_ && vp config --no-agent && git config pull.rebase true && git config merge.ff only"
   },
   "peerDependencies": {
     "@imfusion/sdk": "*",

package/src/llms/skills/imf-web-ui-conventions/topics/git.md

@@ -7,12 +7,15 @@ Use the repository's tracked configuration and keep verification scopes clear.
 Run once in a clone:
 
 ```sh
-git config core.hooksPath .githooks
+git config core.hooksPath .vite-hooks/_
+vp config --no-agent
 git config pull.rebase true
 git config merge.ff only
 ```
 
-The hooks are tracked in `.githooks/`. Check both the Git setting and the directory; a missing hook path fails silently.
+Vite+ manages the hooks. The hook scripts are tracked in `.vite-hooks/`; `vp config` installs the dispatcher into the
+git-ignored `.vite-hooks/_/`. `vp config` leaves a `core.hooksPath` that points elsewhere alone, so set it first. Every clone
+and worktree needs its own dispatcher. Check both with `vp hooks status`; a missing dispatcher skips the hooks silently.
 
 ## Verify scopes
 

package/src/llms/skills/imf-web-ui-conventions/topics/styling.md

@@ -9,7 +9,9 @@ Use native CSS and Web UI's public seams. The library's CSS Modules and token na
 - Nest states, pseudo-elements, and child selectors under the root.
 - Write declarations before nested rules; browsers before Chrome 130, Firefox 132, and Safari 18 move later ones above them.
 - Do not nest under a selector list whose selectors differ in specificity. The nested rule takes the highest one.
-- Lift repeated values into custom properties, especially when a state or variant changes several descendants.
+- Lift a value into a custom property on the component's root when it repeats within that stylesheet, or when a state or
+  variant changes several descendants. A value that only recurs across files stays a literal in each; when those files show
+  one design element, a shared component owns it. Never define component values on `:root`.
 - Keep structurally different rules explicit; not every repeated value needs an abstraction.
 
 ```css

package/src/llms/skills/imf-web-ui-conventions/topics/tooling.md

@@ -49,7 +49,7 @@ from `vite-plus/test`. It needs Node `^22.22.1 || ^24.11 || >=26` and Git 2.32 o
 `vp <command>` is a built-in; `vp run <script>` runs a package script.
 
 Move an existing Vite, Vitest, ESLint, or Prettier project with `vp migrate --no-hooks` (`vp help migrate` lists its
-options), then check the result against this topic. The hook lives in `.githooks/`, as `git.md` describes.
+options), then check the result against this topic. The hook lives in `.vite-hooks/`, as `git.md` describes.
 
 ## Oxfmt
 
@@ -106,7 +106,7 @@ Use strict TypeScript with bundler module resolution, `verbatimModuleSyntax`, un
 
 ## Pre-commit checks
 
-`.githooks/pre-commit` runs `vp staged --hide-unstaged --fail-on-changes --revert --diff-filter=ACMRD` with `set -e`. Those
+`.vite-hooks/pre-commit` runs `vp staged --hide-unstaged --fail-on-changes --revert --diff-filter=ACMRD` with `set -e`. Those
 flags make the checks see only the staged state, include deletion-only commits, fail the commit when a check would change a
 file, and restore the tree afterwards, so the hook never rewrites or stages files. The `staged` block in `vite.config.ts` is
 a function returning path-free commands such as `npm run --silent verify:lint`, so each project-wide check runs once. Use

package/src/llms/skills/imf-web-ui-conventions/topics/typescript.md

@@ -6,6 +6,8 @@ Prefer pure functions, immutable values, and side effects at the edges.
 
 - Do not use `any`. Accept `unknown` at an untrusted boundary and narrow it.
 - Infer local values; give exported functions explicit return types.
+  - TanStack options factories (`queryOptions`, `mutationOptions`, and similar) return their helper's inferred type. Spelling
+    it out loses the typed query key (`DataTag`) or repeats the helper's own generics.
 - Model states directly instead of filling a nullable value later.
 - Derive types from their source. Use `z.infer` for schemas and `React.ComponentProps` for Web UI components.
 - Avoid `as`. Keep an assertion only at an untyped third-party boundary where it is the honest boundary.