Skip to content

fix: exclude provekit deps from mobile builds via feature flags#386

Closed
agentotto[bot] wants to merge 1 commit into
mainfrom
otto/feature-flag-provekit
Closed

fix: exclude provekit deps from mobile builds via feature flags#386
agentotto[bot] wants to merge 1 commit into
mainfrom
otto/feature-flag-provekit

Conversation

@agentotto
Copy link
Copy Markdown
Contributor

@agentotto agentotto Bot commented Apr 30, 2026

Summary

  • depend on the protocol feature-flag branch from feat: feature-flag provekit dependencies behind provekit feature world-id-protocol#708
  • keep WalletKit default/mobile builds from enabling provekit, zk-ownership-prove, or zk-ownership-verify
  • add an explicit provekit feature to walletkit-core and walletkit for platforms that want WIP-103 ownership proofs
  • cfg-gate WalletKit ownership proof wrapper/API and integration-test ownership proof section behind provekit

Validation

  • cargo tree -p walletkit --no-default-features --features storage,issuers --quiet | rg -c 'provekit[-_]|provekit_common' returns no actual ProveKit crate matches
  • cargo tree -p walletkit --no-default-features --features storage,issuers,provekit --quiet | rg -c 'provekit[-_]|provekit_common' returns ProveKit crates when opted in
  • cargo check -p walletkit-core --no-default-features --features storage,issuers
  • cargo check -p walletkit-core --no-default-features --features storage,issuers,provekit

Related

@paolodamico
Copy link
Copy Markdown
Contributor

this seems outdated now

@paolodamico paolodamico deleted the otto/feature-flag-provekit branch May 20, 2026 17:07
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.

1 participant