Skip to content

fix: rehydrate Svelte on state change and run UI checks through the instance - #71

Open
martijn00 wants to merge 1 commit into
letstri:mainfrom
martijn00:fix/ui-check-path-and-hydrate
Open

fix: rehydrate Svelte on state change and run UI checks through the instance#71
martijn00 wants to merge 1 commit into
letstri:mainfrom
martijn00:fix/ui-check-path-and-hydrate

Conversation

@martijn00

Copy link
Copy Markdown
Contributor

Summary

  • stacks on #70; review the last commit(s) once fix: reject prototype paths, fail closed on deny, and run unit tests in CI聽#70 merges
  • UI check() uses overlay rules only while the instance is not ready, then goes through instance.check() so hooks and Standard Schema validate: 'deny' run
  • Svelte PermixHydrate rehydrates when state changes; Vue Check caches a computed boolean
  • README, getting started, and CONTRIBUTING document that core setup() is not request-safe, and that pnpm verify:full includes Playwright
  • overlapping setup() last-write-wins is characterized until immutable instances land

Test plan

  • pnpm exec vitest run src/core src/react src/vue src/solid src/svelte src/standard-schema
  • pnpm test
  • pnpm run check-types
  • pnpm run lint
  • pnpm --filter permix size:compare

Checklist

  • docs updated (docs/content/docs/index.mdx, README.md, CONTRIBUTING.md)
  • permix/skills/ aligned for the request-safety note
  • added tests for Svelte rehydrate, UI check hooks, Standard Schema deny hooks, and concurrent setup() last-write-wins

Made with Cursor

@vercel

vercel Bot commented Aug 28, 2026

Copy link
Copy Markdown

@martijn00 is attempting to deploy a commit to the letstri Team on Vercel.

A member of the Team first needs to authorize it.

@martijn00
martijn00 force-pushed the fix/ui-check-path-and-hydrate branch 2 times, most recently from 394f494 to 2a1ac73 Compare August 28, 2026 23:56
PermixHydrate only hydrated once at init, so a new `state` prop (for
example after navigation) was ignored. React and Vue already re-hydrate
on prop change; Svelte now does the same via $effect.pre.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@letstri
letstri force-pushed the fix/ui-check-path-and-hydrate branch from 2a1ac73 to 267637e Compare September 6, 2026 07:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants