Skip to content

fix: remove v1 route and update tests for devnet v0.12#439

Merged
geoxel merged 1 commit intomainfrom
geoxel/1257
Apr 10, 2026
Merged

fix: remove v1 route and update tests for devnet v0.12#439
geoxel merged 1 commit intomainfrom
geoxel/1257

Conversation

@geoxel
Copy link
Copy Markdown
Contributor

@geoxel geoxel commented Apr 10, 2026

fix: remove v1 route and update tests for devnet v0.12

Drop support for the v1 relayer API route and migrate all routing logic to v2 only, in preparation for devnet v0.12.

Core changes:

  • relayerUrl.ts: parseRelayerUrl no longer accepts fallbackVersion or relayerRouteVersion. Always returns version 2. v1 Zama URLs (Sepolia/Mainnet) are automatically upgraded to their v2 equivalents.
  • createRelayerProvider.ts: drops defaultRelayerVersion parameter and unconditionally returns a RelayerV2Provider. RelayerV1Provider removed.
  • createRelayerFhevm.ts: simplified, v1 provider wiring removed.

Test cleanup:

  • Deleted RelayerV1Fhevm.test.ts and RelayerV1Provider.test.ts.
  • Pruned index.test.ts, relayerUrl.test.ts, and v2 test files to remove v1 test cases and update URL expectations.
  • Added src/test/scripts/test-devnet.sh for devnet integration testing.
  • Simplified test-testnet.sh to remove v1-specific test paths.

CLI / bin updates:

  • Removed "--version" option from bin/ commands (input-proof.js, public-decrypt.js, user-decrypt.js, etc.) and bin/utils.js.

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 10, 2026

Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @geoxel on file. In order for us to review and merge your code, please sign:

  • For individual contribution: our CLA
  • for Bounty submission, if you are an individual: our T&C
  • for Bounty submission, if you are a company: our T&C
    to get yourself added.

If you already signed one of this document, just wait to be added to the bot config.

@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 10, 2026

Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @geoxel on file. In order for us to review and merge your code, please sign:

  • For individual contribution: our CLA
  • for Bounty submission, if you are an individual: our T&C
  • for Bounty submission, if you are a company: our T&C
    to get yourself added.

If you already signed one of this document, just wait to be added to the bot config.

fix: remove v1 route and update tests for devnet v0.12

Drop support for the v1 relayer API route and migrate all routing logic
to v2 only, in preparation for devnet v0.12.

Core changes:
- relayerUrl.ts: parseRelayerUrl no longer accepts fallbackVersion or
  relayerRouteVersion. Always returns version 2. v1 Zama URLs
  (Sepolia/Mainnet) are automatically upgraded to their v2 equivalents.
- createRelayerProvider.ts: drops defaultRelayerVersion parameter and
  unconditionally returns a RelayerV2Provider. RelayerV1Provider removed.
- createRelayerFhevm.ts: simplified, v1 provider wiring removed.

Test cleanup:
- Deleted RelayerV1Fhevm.test.ts and RelayerV1Provider.test.ts.
- Pruned index.test.ts, relayerUrl.test.ts, and v2 test files to remove
  v1 test cases and update URL expectations.
- Added src/test/scripts/test-devnet.sh for devnet integration testing.
- Simplified test-testnet.sh to remove v1-specific test paths.

CLI / bin updates:
- Removed "--version" option from bin/ commands (input-proof.js,
  public-decrypt.js, user-decrypt.js, etc.) and bin/utils.js.
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 10, 2026

Thank you for your pull request. We require contributors to sign our Contributor License Agreement / Terms and Conditions, and we don't seem to have the users @geoxel on file. In order for us to review and merge your code, please sign:

  • For individual contribution: our CLA
  • for Bounty submission, if you are an individual: our T&C
  • for Bounty submission, if you are a company: our T&C
    to get yourself added.

If you already signed one of this document, just wait to be added to the bot config.

Copy link
Copy Markdown
Contributor

@0xalexbel 0xalexbel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@IceTDrinker
Copy link
Copy Markdown
Member

@cla-bot check

@cla-bot cla-bot bot added the cla-signed label Apr 10, 2026
@cla-bot
Copy link
Copy Markdown

cla-bot bot commented Apr 10, 2026

The cla-bot has been summoned, and re-checked this pull request!

@geoxel geoxel merged commit 124ae0a into main Apr 10, 2026
5 checks passed
@geoxel geoxel deleted the geoxel/1257 branch April 10, 2026 12:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants