Skip to content

Add psdk-plugin build command#27

Merged
NuriYuri merged 6 commits into
mainfrom
plugin
Jun 28, 2026
Merged

Add psdk-plugin build command#27
NuriYuri merged 6 commits into
mainfrom
plugin

Conversation

@NuriYuri

Copy link
Copy Markdown
Contributor

This PR adds the psdk-plugin build command.

This command works in PSDK project and in standalone plugin folder.

Due to time constraints, it's not fixing any of the issues properly, it's brining MVP (which is building a plugin in standalone folder).

@pw-rey pw-rey self-requested a review June 25, 2026 18:57
@pw-rey

pw-rey commented Jun 25, 2026

Copy link
Copy Markdown
Collaborator

Je viens de faire une passe de test sur la PR. Je note les choses qui fonctionnent parfaitement dans un premier temps, et ce qui n'a pas l'air de fonctionner dans un second temps.
Ce qui fonctionne :

  • les tests run sans problème
  • bundler et l'installation de la gem se font sans problème
  • psdk-cli est automatiquement reconnu dans le terminal
  • les commandes et les arguments chainés fonctionnent (exemple : psdk-cli plugin build MyCoolPlugin --out-dir=scripts
  • le help fonctionne correctement pour toutes les commandes de la branche plugin
  • l'argument list montre correctement la liste des plugins du projet, qu'importe que l'on soit à la racine ou dans ./scripts

Ce qui ne semble pas fonctionner :

  • pour une raison que j'ignore actuellement, tous les plugins compilés à l'aide de la gem ne peuvent pas être installés par PSDK au lancement. Je peux confirmer que cela vient bien de la CLI étant donné que le même plugin compilé par PSDK fonctionne sans aucun problème. Je n'ai pas encore creusé concrètement d'où ça peut venir.

@NuriYuri

Copy link
Copy Markdown
Contributor Author

Nice, je vois que t'as commit pour corriger le problème. Si tout va bien on peut merge :)

@pw-rey pw-rey left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

A toi l'honneur. :d

@NuriYuri NuriYuri merged commit 9bb5f41 into main Jun 28, 2026
1 check passed
@NuriYuri NuriYuri deleted the plugin branch June 28, 2026 15:24
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