Skip to content

Comments

refactor: enhance version parsing with protocol support and validation#30

Merged
9romise merged 1 commit intomainfrom
refactor/version
Feb 5, 2026
Merged

refactor: enhance version parsing with protocol support and validation#30
9romise merged 1 commit intomainfrom
refactor/version

Conversation

@9romise
Copy link
Member

@9romise 9romise commented Feb 5, 2026

No description provided.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Feb 5, 2026

📝 Walkthrough

Walkthrough

This pull request extends version parsing to support multiple protocols (workspace, catalog, npm, and jsr) in addition to the existing npm protocol. A new isSupportedProtocol function is introduced to validate parsed protocols. The parseVersion utility is updated to parse optional protocol prefixes and extract the version string accordingly. Multiple provider modules are modified to use the new isSupportedProtocol check, returning early if the parsed protocol is not supported. Test coverage is added for the newly supported protocols.

Possibly related PRs

🚥 Pre-merge checks | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Description check ⚠️ Warning The pull request has no description provided by the author, making it impossible to assess relevance or intent. Add a pull request description explaining the changes, their purpose, and any relevant context for reviewers.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch refactor/version

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

@9romise 9romise merged commit 329c2a0 into main Feb 5, 2026
6 checks passed
@9romise 9romise deleted the refactor/version branch February 5, 2026 15:09
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.

1 participant