Skip to content

Conversation

@tim-hm
Copy link
Collaborator

@tim-hm tim-hm commented Dec 12, 2025

No description provided.

BREAKING CHANGE: Remove nilchain (cosmos) dependencies in favor of ERC-20 burn contract payments on Ethereum.

- Remove Payer client and nilchain transaction handling
- Remove @bufbuild/protobuf, @cosmjs/proto-signing, @cosmjs/stargate deps
- Remove protobuf generation (buf.yaml, buf.gen.yaml)
- Add chain_id to payment payload for network validation
- Use RFC 8785 canonical JSON (canonicalize) for payment digest
- Add chainId parameter to NilauthClient constructor
- Remove deprecated payAndValidate() method
Gi
BREAKING CHANGE: NilauthClient has been moved to @nillion/nilauth-client.

Removes nilauth client and related code to focus the package exclusively on NUC token creation and validation.

Removed:
- NilauthClient, NilauthTypes, BlindModule exports
- Nilauth-specific errors (NilauthUnreachable, NilauthErrorResponse, etc.)
- Dependencies: canonicalize, ky, http-status-codes, docker-compose, dotenv
- Docker test infrastructure (no external services needed)
- Integration tests for nilauth (moved to nilauth-client-ts)

Added:
- Payload.REVOKE_COMMAND to namespace for external consumers

Migration: Install @nillion/nilauth-client alongside @nillion/nuc for nilauth functionality. Import NilauthClient from the new package.
@tim-hm tim-hm changed the title Feat/replace nilchain with erc 20 feat/replace nilchain payments with erc 20 payments Dec 12, 2025
@github-actions
Copy link

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 84.7% 576 / 680
🔵 Statements 84.68% 586 / 692
🔵 Functions 88.59% 132 / 149
🔵 Branches 76.77% 271 / 353
File Coverage
File Stmts Branches Functions Lines Uncovered Lines
Changed Files
src/core/signer.ts 88.46% 84.21% 93.33% 88.46% 49-50, 127-130, 139-142
src/nuc/builder.ts 87.23% 78.84% 88% 87.23% 139-158, 214-215, 383, 468, 588-590, 602, 638-640, 652
src/nuc/payload.ts 100% 100% 100% 100%
Generated in workflow #340 for commit 07e6d35 by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants