Skip to content

fix(plugins): validate plugin command messages - #179

Open
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/plugin-message-validation
Open

fix(plugins): validate plugin command messages#179
aiirvizionz wants to merge 1 commit into
profullstack:masterfrom
aiirvizionz:clientkit/plugin-message-validation

Conversation

@aiirvizionz

Copy link
Copy Markdown
Contributor

Summary

  • reject missing, blank, or non-string plugin command messages before plugin initialization
  • trim valid command strings before dispatching to the plugin manager
  • add focused route coverage for malformed JSON, non-string messages, and trimmed valid commands

Validation

  • git diff --check
  • node node_modules\vitest\vitest.mjs run --config $env:TEMP\qryptchat-vitest-minimal.config.mjs "src/app/api/plugins/execute/route.test.js" (3 tests)

Billing

  • uGig billable PR candidate: expected invoice $0.50 if accepted/merged
  • Not counted as revenue until paid/settled

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