Skip to content

Fix init command migration#135

Merged
thelovekesh merged 2 commits intomainfrom
fix/init-command-migration
Jan 6, 2026
Merged

Fix init command migration#135
thelovekesh merged 2 commits intomainfrom
fix/init-command-migration

Conversation

@thelovekesh
Copy link
Collaborator

  • Add a fix to remove duplicate team and tags items.
  • Fix base64 encoding of readme before adding to manifest.

@thelovekesh thelovekesh added the command any changes to the wpm cli command label Jan 6, 2026
@thelovekesh thelovekesh requested a review from Copilot January 6, 2026 14:35
@github-actions
Copy link

github-actions bot commented Jan 6, 2026

📦 wpm cli builds 📦

Copy link
Contributor

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 fixes issues in the init command migration by addressing duplicate handling and readme encoding. The changes ensure proper deduplication of team members and tags, validate plugin dependencies, and correct the readme content format.

  • Added validation and deduplication logic for tags and team members to prevent duplicates
  • Fixed readme content to use plain string instead of base64 encoding
  • Added validation for RequiresPlugins to filter out invalid plugin names

Reviewed changes

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

File Description
cli/command/publish/publish.go Removed base64 encoding from readme content, returning plain string instead
cli/command/init/init.go Added RequiresPlugins validation, implemented deduplication for tags and team using sorted slices with slices.Compact

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@thelovekesh thelovekesh merged commit c73ff83 into main Jan 6, 2026
21 checks passed
@thelovekesh thelovekesh deleted the fix/init-command-migration branch January 6, 2026 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

command any changes to the wpm cli command

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant