Skip to content

feat: validateStakeCred#64

Merged
vladimirvolek merged 1 commit intomasterfrom
feat/parse-stake-cred
Oct 20, 2025
Merged

feat: validateStakeCred#64
vladimirvolek merged 1 commit intomasterfrom
feat/parse-stake-cred

Conversation

@slowbackspace
Copy link
Copy Markdown
Contributor

No description provided.

Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds a new validateStakeCred function that validates and extracts stake credentials from various Cardano address formats. The function supports multiple input formats including stake addresses, stake keys, stake key hashes, and script addresses, returning normalized stake credential information.

Key changes:

  • Implements validateStakeCred function with support for multiple bech32 address formats
  • Adds comprehensive test fixtures covering different stake credential types and networks
  • Updates constant naming from Prefixes to PREFIXES for consistency

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
src/validation.ts Implements the main validateStakeCred function and helper functions, updates prefix constants
test/fixtures/validation.ts Adds comprehensive test fixtures for various stake credential formats
test/tests/validation.test.ts Adds test cases for the new validateStakeCred function
CHANGELOG.md Documents the new feature addition

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@slowbackspace slowbackspace marked this pull request as ready for review October 17, 2025 11:12
@vladimirvolek vladimirvolek merged commit 812a335 into master Oct 20, 2025
2 checks passed
@vladimirvolek vladimirvolek deleted the feat/parse-stake-cred branch October 20, 2025 12:23
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.

3 participants