[nvjsr]

Problem

The Starknet deploy account RPC wire format test in sdk-coin-starknet calls https://api.cartridge.gg/x/starknet/sepolia, which has been permanently discontinued (HTTP 410 Gone):

$ curl -X POST https://api.cartridge.gg/x/starknet/sepolia
starknet sepolia is discontinued and no longer served
HTTP status: 410

This broke unit tests for every BitGoJS PR that touches sdk-core or its dependents, because lerna's unit-test-changed runs the starknet suite as a dependent of sdk-core. Evidence:

• PR feat(lightning): support custodial user auth key resets #10099 (Oct 29, 17:56 UTC) — passed (API was still alive)
• PR feat(express): add default domain-scoped CA trust for BitGo connections #10073 (pranishnepal, Oct 30, 03:06 UTC) — failed on starknet (API dead)
• PR fix(sdk-core): skip auto-sign for message-sign approvals #10102 (DEFI-961, Oct 30, 06:06 UTC) — failed on starknet (API dead)
• No public Sepolia RPC alternative is available (tested 10+ endpoints — all dead, DNS-resolving, or require API keys)

Fix

Replace the live RPC call with local wire-format validation. The test previously verified that the broadcast format reaches the node's parameter validation layer (not a JSON parse error) by POSTing to the live RPC and checking the error message. The new version verifies the same property locally:

1. Parse tx.toBroadcastFormat() as JSON — a parse failure would mean the node rejects before validation
2. Verify the JSON-RPC envelope serializes
3. Assert the required Starknet node fields exist: type, version, nonce, class_hash, contract_address_salt, constructor_calldata

This tests the same property (the wire format is valid enough to reach node validation) without the external dependency.

Testing

Before: 110 passing, 1 failing (SyntaxError from dead API)
After:  111 passing, 0 failing (635ms, fully local)

Impact

• Unblocks all PRs that currently fail on the starknet test
• Removes a flaky external dependency from the unit test suite
• Test runs 5x faster (no network call)

Related

• PR fix(sdk-core): skip auto-sign for message-sign approvals #10102 (DEFI-961) — the message-sign PA fix that was blocked by this test
• PR feat(express): add default domain-scoped CA trust for BitGo connections #10073 — an unrelated PR also blocked by this test

[claude]

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

_{Tip: disable this comment in your organization's Code Review settings.}

[github-actions]

⚠️ Compatibility checks are failing on Node.js 27-nightly (experimental, non-blocking). This does not block merge, but it signals an incompatibility to fix before this release becomes supported.

View run

[linear-code]

COINS-2204