Skip to content

test: gate permix/src on 95% Vitest coverage - #69

Closed
martijn00 wants to merge 39 commits into
letstri:mainfrom
martijn00:test/coverage-95
Closed

test: gate permix/src on 95% Vitest coverage#69
martijn00 wants to merge 39 commits into
letstri:mainfrom
martijn00:test/coverage-95

Conversation

@martijn00

Copy link
Copy Markdown
Contributor

Summary

  • Adds Vitest V8 coverage for permix/src with a 95% gate on statements, lines, functions, and branches, plus test.yml CI and a pre-publish coverage check.
  • Fills remaining unit-test holes in core, extractor, Next withPermix, providers, and adapters so the union of open production PRs (feat: add provider authorization adapters #66 providers and perf: enforce bundle budgets and narrow React checks #67 catalog/adapters/budgets) stays above the threshold.
  • Stacked on the merge of feat/bundle-size-budgets and feat/provider-adapters. No new production behavior beyond the existing CLI v8 ignore around the process entry.

Testing

  • pnpm --filter permix test:coverage (98.83% statements, 95.05% branches, 99.09% functions, 98.95% lines)
  • pnpm lint
  • pnpm --filter permix run check-types
  • pnpm format:check
  • pnpm verify

Checklist

  • added or updated tests where it materially reduces regression risk
  • docs updated where needed (docs/content/docs/, README.md, CONTRIBUTING.md)
  • permix/skills/ aligned when public API or integration patterns changed

Made with Cursor

martijn00 and others added 30 commits August 27, 2026 19:31
Develop the library on TypeScript 7 while keeping the consumer peer at 5.9–7, and pin shared toolchain versions through a strict pnpm catalog so CI and local installs stay aligned.

Co-authored-by: Cursor <cursoragent@cursor.com>
Give each factory its own provider/hook/Check bindings so nested policies do not share state, and hydrate dehydrated rules on the first client render without mutating during render.

Co-authored-by: Cursor <cursoragent@cursor.com>
Cache one initialized instance per RSC request so concurrent server callers share setup, and keep layouts synchronous with permission work behind Suspense for Cache Components.

Co-authored-by: Cursor <cursoragent@cursor.com>
Subscribe to setup/ready during render instead of after an effect so dehydrated booleans and already-initialized rules are visible on the first paint.

Co-authored-by: Cursor <cursoragent@cursor.com>
Toolchain skills from skills.sh fail format check and are not project source.

Co-authored-by: Cursor <cursoragent@cursor.com>
Oxfmt rejected the barrel and type-test files, and pinning React 18 in the catalog conflicts with docs/fumadocs React 19 peers.

Co-authored-by: Cursor <cursoragent@cursor.com>
Oxfmt requires a single trailing newline after the merge from the React factory branch.

Co-authored-by: Cursor <cursoragent@cursor.com>
Reconstruct published npm history, show it on the docs site, and wire conventional-commit releases.

Co-authored-by: Cursor <cursoragent@cursor.com>
Upgrade Fumadocs to 16.15 with the Base UI alias.

Ignore .agents and .claude so CI format check skips third-party skill files.

Co-authored-by: Cursor <cursoragent@cursor.com>
pnpm 11 rejects packages newer than one day in frozen CI installs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Wire Permix into Nest via APP_GUARD so each request gets its own instance, then enforce @check on handlers without tying the API to Express middleware. Closes letstri#11.

Co-authored-by: Cursor <cursoragent@cursor.com>
Store one Permix instance per Nitro event so Nuxt server routes and Vue SSR share rules without leaking state across requests.

Co-authored-by: Cursor <cursoragent@cursor.com>
Store a per-request Permix instance on Astro locals so middleware, endpoints, and pages can share rules without tying UI islands to a new client adapter.

Co-authored-by: Cursor <cursoragent@cursor.com>
React Router 7 (including Remix) needs a per-request instance on middleware context so loaders can dehydrate into permix/react.

Co-authored-by: Cursor <cursoragent@cursor.com>
Reuse existing Zod, Valibot, or ArkType schemas on action specs and via
permix/standard-schema so ReBAC callbacks stay typed without duplicate
interfaces. check() still does not parse data at runtime.

Co-authored-by: Cursor <cursoragent@cursor.com>
Parse check() data with the entity schema when validate is deny or throw, and lock Zod, Valibot, ArkType, and Effect Schema behind regression tests so those integrations stay working.

Co-authored-by: Cursor <cursoragent@cursor.com>
…talog-extraction

Co-authored-by: Cursor <cursoragent@cursor.com>
…talog-extraction

Co-authored-by: Cursor <cursoragent@cursor.com>
…talog-extraction

Co-authored-by: Cursor <cursoragent@cursor.com>
…talog-extraction

Co-authored-by: Cursor <cursoragent@cursor.com>
…talog-extraction

Co-authored-by: Cursor <cursoragent@cursor.com>
…talog-extraction

Co-authored-by: Cursor <cursoragent@cursor.com>
Avoid passing undefined React Router params. Narrow the generated Svelte
component at its factory boundary so TypeScript 7 checks remain valid.

Co-authored-by: Cursor <cursoragent@cursor.com>
Derive typed definitions and metadata from static permission markers so application usage becomes
the catalog source of truth. Include CLI, watch and check workflows, Next.js integration, typed
payload overlays, provider coverage validation, and consumer guidance.

Co-authored-by: Cursor <cursoragent@cursor.com>
Centralize per-invocation authentication, rule resolution, typed decisions,
dehydration, and catalog coverage for isolated provider integrations.

Co-authored-by: Cursor <cursoragent@cursor.com>
Provide an isolated Fetch handler, deterministic OpenAPI contract, and typed
client for caller-scoped and trusted service authorization checks.

Co-authored-by: Cursor <cursoragent@cursor.com>
martijn00 and others added 9 commits August 28, 2026 18:23
Add verified claims and user adapters, generated Database type inference,
typed policy manifests, and secure RLS recipes for Supabase applications.

Co-authored-by: Cursor <cursoragent@cursor.com>
Provide isolated server and client plugins, typed request helpers, and optional
access-control inference without shared mutable rule state.

Co-authored-by: Cursor <cursoragent@cursor.com>
Provide isolated Auth and request resolution, explicit permission mappings,
bearer-token permissions transport, and a thin Next.js convenience layer.

Co-authored-by: Cursor <cursoragent@cursor.com>
Resolve authenticated identities and isolated rules before Convex handlers,
while preserving function validators and generated data-model inference.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add a credential-free acceptance example.
Align consumer skills with the stable adapter APIs.

Co-authored-by: Cursor <cursoragent@cursor.com>
Verify the installed tarball and execute RLS boundaries against local Supabase.

Co-authored-by: Cursor <cursoragent@cursor.com>
Measure every public entry in CI and preserve tree-shaking boundaries.
Avoid rerendering Check when its permission result is unchanged.

Co-authored-by: Cursor <cursoragent@cursor.com>
Bring provider runtime onto the catalog tree and register bundle-size
fixtures so the union of open PRs has a complete public export surface.

Co-authored-by: Cursor <cursoragent@cursor.com>
Add V8 thresholds and a CI job so library coverage cannot regress.

Co-authored-by: Cursor <cursoragent@cursor.com>
@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.

@letstri

letstri commented Sep 6, 2026

Copy link
Copy Markdown
Owner

Closing: toolchain/CI churn that we don't want to take on, and it's stacked on PRs that are being closed.

@letstri letstri closed this Sep 6, 2026
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