Skip to content

fix(framework): use node validators in hooks#295

Merged
alexsoyes merged 2 commits into
mainfrom
pr/node-validation
Jun 19, 2026
Merged

fix(framework): use node validators in hooks#295
alexsoyes merged 2 commits into
mainfrom
pr/node-validation

Conversation

@alexsoyes

@alexsoyes alexsoyes commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Replace jq/pipx/Python hook logic with local Node validators.
  • Add repository-specific JSON checks for Claude plugin manifests, marketplace metadata, and Claude settings.
  • Add js-yaml for YAML syntax validation.
  • Remove CI pipx setup from the validate workflow.

Verification

  • node scripts/validate-json.mjs <all tracked JSON> passed for 17 files.
  • node scripts/validate-yaml.mjs <tracked YAML excluding plugin skill assets> passed for 12 files.
  • GitHub checks passed on the pushed branch.

@alexsoyes

Copy link
Copy Markdown
Contributor Author

@blafourcade chiant de dépendre de python pour si peu, on keep node only?

@alexsoyes alexsoyes enabled auto-merge (squash) June 18, 2026 06:19
@blafourcade

blafourcade commented Jun 18, 2026

Copy link
Copy Markdown
Contributor

@blafourcade chiant de dépendre de python pour si peu, on keep node only?

@alexsoyes Oui j'avais pas forcément challenge cet aspect côté repo. Uniquement côté plugins (hooks, scripts)

Comment thread scripts/validate-json.mjs Outdated
@blafourcade blafourcade assigned alexsoyes and unassigned blafourcade Jun 18, 2026
@alexsoyes

Copy link
Copy Markdown
Contributor Author

J'ai corrigé ! @blafourcade

@alexsoyes alexsoyes assigned blafourcade and unassigned alexsoyes Jun 19, 2026
@blafourcade

Copy link
Copy Markdown
Contributor

J'ai corrigé ! @blafourcade

@alexsoyes on merge en l'état mais on va peut être mettre en place un "plugin" "marketplace" etc à terme avec un fichier de config à côté pour plus propre et configurable

@alexsoyes alexsoyes merged commit e6b8355 into main Jun 19, 2026
9 checks passed
@alexsoyes alexsoyes deleted the pr/node-validation branch June 19, 2026 08:52
@aidd-bot aidd-bot Bot mentioned this pull request Jun 19, 2026
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