0xMiden / 0xMiden/wallet

Re-enable guardian E2E on web-sdk 0.16 once OZ ships a protocol-0.16 multisig/guardian client

未關閉 適合新手
#522 0 則留言 0 個 reaction 已指派 0 人 在 GitHub 檢視
主要語言
TypeScript
星號
5
分支
28
平均合併
9 小時 50 分鐘
30 天內合併 PR
123

描述

## Context

The `next` branch is migrating to web-sdk 0.16 (PR #521). Guardian account **creation** fails on SDK 0.16 with a MASM link error:

```
Error creating Guardian account: script builder: failed to link module
openzeppelin::auth::guardian_ecdsa … undefined constant 'AUTH_UNAUTHORIZED_EVENT'
```

### Root cause
`@openzeppelin/miden-multisig-client` has **no release built against miden-protocol 0.16**. The latest (`0.16.1`) still depends on `@miden-sdk/miden-sdk@^0.15.8`; its `guardian_ecdsa` MASM references `AUTH_UNAUTHORIZED_EVENT`, a 0.15-era auth constant removed/renamed in protocol 0.16.0-alpha.4. OZ's `0.16.x` version tracks their own client, not miden-protocol 0.16.

This breaks guardian in the **product** on 0.16, not only in E2E. A client-only MASM patch is not viable (co-sign ABI must match across client + guardian server + on-chain verification).

### Quarantined on #521 (re-enable when OZ ships a protocol-0.16 client)
- [ ] `pr-e2e-bridge-guardian.yml` — remove the job-level `if: false` guard
- [ ] `pr-e2e-guardian-lifecycle.yml` — remove the job-level `if: false` guard
- [ ] `playwright.swap.config.ts` — remove the `testIgnore` for `swap-guardian.spec.ts`
- [ ] Revert both guardian gates to require `success` only (drop the `skipped` tolerance)
- [ ] Bump `@openzeppelin/miden-multisig-client` + `@openzeppelin/guardian-client` to the protocol-0.16 release; bump `GUARDIAN_IMAGE_TAG` to the matching server image

Non-guardian E2E (local, earn, swap non-guardian) is green on 0.16.

貢獻指南

開啟貢獻指南

研究方向

The issue is blocked on an external dependency update from OpenZeppelin. To prepare, locate the CI configuration files mentioned: `pr-e2e-bridge-guardian.yml`, `pr-e2e-guardian-lifecycle.yml`, and `playwright.swap.config.ts`. Find the `testIgnore` for `swap-guardian.spec.ts` and the job-level `if: false` guards. Once OpenZeppelin releases a protocol-0.16 compatible client, update the package versions and image tag as listed, then remove the guards to re-enable the tests.

由索引模型根據 Issue 內容生成。

評估

技術堆疊
javascript, nodejs, typescript
領域
backend, devops, testing
Issue 類型
缺陷
難度
2/5
預估耗時
1-3 小時
活躍度
冷清
描述清晰度
描述清楚
新手友好度
65/100

把新 issue 寄到你的電子郵件信箱

精選適合新手參與的 GitHub issue 摘要。