Skip to content

chore(generated-clients): BLO-135 remove StarkExTransactionEvaluationRequest from MR generated client#2876

Merged
naveen-imtb merged 2 commits into
mainfrom
naveenpalaniswamy/blo-135-remove-starkextransactionevaluationrequest-from-multi-rollup
May 25, 2026
Merged

chore(generated-clients): BLO-135 remove StarkExTransactionEvaluationRequest from MR generated client#2876
naveen-imtb merged 2 commits into
mainfrom
naveenpalaniswamy/blo-135-remove-starkextransactionevaluationrequest-from-multi-rollup

Conversation

@naveen-imtb
Copy link
Copy Markdown
Contributor

Summary

  • Refreshes vendored mr-openapi.json from S3. The upstream spec had StarkExTransactionEvaluationRequest removed when guardian BSB-6 PR #23081 merged; the local snapshot was never updated (net diff: +11,844/−6,071 lines of accumulated spec drift)
  • Deletes stark-ex-transaction-evaluation-request.ts — the generated model for the now-absent type
  • Collapses TransactionEvaluationRequest union to just ZkEvmTransactionEvaluationRequest
  • Removes the barrel export from models/index.ts

No non-generated consumer remains — starkEx/imxGuardianClient.ts was removed in #2808.

Note: make generate-mr-api-client (full Docker regeneration) was intentionally skipped to avoid pulling in ~11k lines of unrelated spec drift into the TypeScript client files. The three TS file edits are surgical.

Closes BLO-135

Test plan

  • pnpm build passes in packages/internal/generated-clients
  • pnpm build passes in packages/wallet
  • grep -r StarkExTransactionEvaluationRequest packages/internal/generated-clients/src returns no results

@naveen-imtb naveen-imtb requested a review from a team as a code owner May 25, 2026 04:25
@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented May 25, 2026

View your CI Pipeline Execution ↗ for commit 6c075a6

Command Status Duration Result
nx affected -t build,lint,test ✅ Succeeded 3m 18s View ↗
nx run-many -p @imtbl/sdk,@imtbl/checkout-widge... ✅ Succeeded 1m 32s View ↗

☁️ Nx Cloud last updated this comment at 2026-05-25 04:29:54 UTC

@naveen-imtb naveen-imtb enabled auto-merge May 25, 2026 04:32
@naveen-imtb naveen-imtb added this pull request to the merge queue May 25, 2026
Merged via the queue into main with commit 93e64b4 May 25, 2026
8 checks passed
@naveen-imtb naveen-imtb deleted the naveenpalaniswamy/blo-135-remove-starkextransactionevaluationrequest-from-multi-rollup branch May 25, 2026 04:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants