feat: add turnkey support

[garyghayrat]

No description provided.

[netlify]

✅ Deploy Preview for jolly-shaw-20fe62 ready!

Name	Link
🔨 Latest commit	1eddafb
🔍 Latest deploy log	https://app.netlify.com/projects/jolly-shaw-20fe62/deploys/6a0b341f8ee8e3000801de04
😎 Deploy Preview	https://deploy-preview-715--jolly-shaw-20fe62.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify project configuration.

[marcomariscal]

I think this can race. Expanding the row starts one fee request, then starting the withdrawal starts another. If the first one finishes last, it can overwrite the fee for the active withdrawal. Could we keep the fee for this withdrawal as its own value and pass that through confirmation/signing? A test with two out-of-order fee responses would catch it.

[garyghayrat]

1eddafb

[github-actions]

Coverage after merging feat/replace-defender-with-turnkey into master will be

83.99%

Coverage Report

File	Stmts	Branches	Funcs	Lines	Uncovered Lines
contracts-periphery/src
UmbraBatchSend.sol	100%	100%	100%	100%
UniswapWithdrawHook.sol	100%	100%	100%	100%
umbra-js/src
ethers.ts	74.58%	100%	58.33%	100%
types.ts	50%	100%	0%	100%
umbra-js/src/classes
KeyPair.ts	98.54%	97.62%	100%	98.75%	248, 250
RandomNumber.ts	100%	100%	100%	100%
StealthKeyRegistry.ts	100%	100%	100%	100%
TxHistoryProvider.ts	85.19%	65%	100%	96.55%	12, 17, 20, 23, 29, 43, 48, 8
Umbra.ts	82.47%	74.25%	85.37%	87%	144–145, 166, 239–240, 308, 326–329, 365–366, 397, 412, 415, 422–424, 426, 433, 436–437, 462, 462, 462, 464, 464, 464, 466, 466, 466, 466–467, 467, 467–469, 484, 484, 484–485, 489, 492, 512, 530–532, 573, 628, 632, 678, 691–692, 702–703, 717, 725–726, 756–757, 832–833, 836, 839–840, 846–848, 855, 855, 855–858, 893, 895–896, 896–897
umbra-js/src/utils
cns.ts	50%	33.33%	50%	55%	33–36, 40, 40, 40, 40, 40–41, 44–46
constants.ts	100%	100%	100%	100%
ens.ts	36.11%	25%	33.33%	40%	25–27, 35, 56–57, 57, 57–58, 63–64, 66, 70, 70, 70, 70, 70–71, 75–77
sharedSecret.ts	83.05%	72.73%	100%	87.88%	24, 27–28, 42–43, 47–48, 62, 62–63
utils.ts	71.02%	63.98%	80%	74.82%	102, 102, 1026–1027, 1039–1040, 118–120, 122–124, 130, 152–153, 179–181, 181, 181–182, 182, 182, 203, 268, 271, 276, 280, 284, 294, 306–311, 313–320, 320, 320, 320, 320, 320, 320–321, 323, 333, 347–348, 348, 348–349, 353–354, 364, 364, 364–366, 368, 384–386, 414, 437, 440, 444, 444–445, 445, 445–446, 446, 448, 452–453, 456–457, 480, 483, 483, 483–485, 515, 517–518, 527, 527, 527–528, 530, 533, 542–543, 584, 601, 601, 601, 630, 652–654, 657, 667, 679, 679, 68, 680, 685, 687–688, 69, 699–703, 716, 718, 769–770, 770, 770–772, 772, 772, 783, 94
umbra-js/test
testPrivateKeys.ts	100%	100%	100%	100%
utils.ts	100%	100%	100%	100%

[marcomariscal]

Looks good. The fee estimate race is addressed with the isolated withdrawal fee and out-of-order response test.

[apbendi]

@garyghayrat Tested myself with Optimism and Mainnet. Funded the wallet across all accounts. Let's coordinate on Monday to get it rolled out to prod.