Skip to content

fix: validate controlled state updates - #47

Open
ddebowczyk wants to merge 16 commits into
Jeshwin:mainfrom
ddebowczyk:fix/validate-controlled-state
Open

ddebowczyk wants to merge 16 commits into
Jeshwin:mainfrom
ddebowczyk:fix/validate-controlled-state

Conversation

@ddebowczyk

Copy link
Copy Markdown

Summary

  • validate each controlled useLaymanController state update before accepting it
  • retain the previous valid state when validation fails
  • cover the invariant and document the public contract

Verification

  • npm test
  • npm run lint
  • npm run build:lib
  • npm run typecheck:consumer
  • npm run typecheck:tauri-example
  • npm run check:package
  • npm pack --dry-run
  • markdownlint README.md "docs/**/*.md"
  • yq . .github/workflows/ci.yml
  • git diff --check
@ddebowczyk
ddebowczyk force-pushed the fix/validate-controlled-state branch 2 times, most recently from 6c32365 to 4a1ec6c Compare August 12, 2026 13:42

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant