fix(e2e-nightly): add signet to BTC client resolveParams

[ws4charlie]

Summary

• Add signet case to resolveParams in zetaclient/chains/bitcoin/client/client.go

Context

PR #4539 added signet support to the e2e config layer (e2e/config/config.go, cmd/zetae2e/config/clients.go) but missed the resolveParams function in the zetaclient BTC client. This causes "unknown chain params "signet"" errors when running BTC e2e tests on testnet (signet).

Before	After
zetae2e bitcoin-address fails with unknown chain params "signet"	resolveParams("signet") returns chains.SigNetParams correctly

This PR is a companion to e2e#142 which switches btc_network_name from testnet3 to signet.

Test plan

• Verify zetae2e bitcoin-address works with signet config
• Run BTC e2e tests on testnet with signet network name

🤖 Generated with Claude Code

---

Note

Low Risk
Low risk: a small, isolated change that only extends resolveParams to recognize signet and map it to chains.SigNetParams.

Overview
Fixes BTC client initialization for Signet configs by teaching resolveParams in zetaclient/chains/bitcoin/client/client.go to accept chains.SigNetParams.Name and return chains.SigNetParams instead of erroring with "unknown chain params".

^{Written by Cursor Bugbot for commit ead2f49. Configure here.}

Summary by CodeRabbit

• New Features
  • Added SigNet support to Bitcoin chain parameter resolution.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 01738739-40ac-4ae7-b1a1-0e2e8ae1850f

📥 Commits

Reviewing files that changed from the base of the PR and between 4342585 and 64c2c2f.

📒 Files selected for processing (2)

• changelog.md
• zetaclient/chains/bitcoin/client/client.go

---

📝 Walkthrough

Walkthrough

This pull request adds SigNet chain parameter support to the Bitcoin client by introducing a new case in the resolveParams function to handle chains.SigNetParams. The change includes corresponding documentation in the changelog.

Changes

Cohort / File(s)	Summary
SigNet Parameter Resolution zetaclient/chains/bitcoin/client/client.go	Adds a new case statement in resolveParams to recognize and resolve chains.SigNetParams.Name, returning the appropriate SigNet parameters alongside existing MainNet, TestNet3, RegressionNet, and SimNet support.
Changelog Entry changelog.md	Documents the addition of SigNet support to the Bitcoin client parameter resolution as a fix for the e2e "unknown chain params" error.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

• fix: add support for 'signet' name in the e2e config #4539: Complements this change by adding SigNet/Signet handling in e2e configuration and client mapping infrastructure.

Suggested reviewers

• kingpinXD

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and specifically describes the main change: adding signet support to the BTC client's resolveParams function to fix e2e test failures.
Description check	✅ Passed	The description provides context, motivation, and concrete test cases, though the testing checklist items are not marked as completed rather than omitted.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch fix/btc-signet-resolve-params

📝 Coding Plan

• Generate coding plan for human review comments

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 OpenGrep (1.16.4)

zetaclient/chains/bitcoin/client/client.go

┌──────────────┐
│ Opengrep CLI │
└──────────────┘

�[32m✔�[39m �[1mOpengrep OSS�[0m
�[32m✔�[39m Basic security coverage for first-party code vulnerabilities.

�[1m Loading rules from local config...�[0m

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[codecov]

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
zetaclient/chains/bitcoin/client/client.go	0.00%	2 Missing ⚠️

📢 Thoughts on this report? Let us know!