Skip to content

feat: support optional unresolved request fields - #552

Open
moonlander101 wants to merge 2 commits into
VoidenHQ:mainfrom
moonlander101:feat/required-request-fields
Open

feat: support optional unresolved request fields#552
moonlander101 wants to merge 2 commits into
VoidenHQ:mainfrom
moonlander101:feat/required-request-fields

Conversation

@moonlander101

@moonlander101 moonlander101 commented Aug 20, 2026

Copy link
Copy Markdown

Addresses #508

Alongside plugin changes

Changes:

  • Adds a Required checkbox to request-table rows alongside Enabled.
  • Omits enabled-but-optional fields when their variables cannot be resolved.
  • Preserves validation errors for enabled and required fields with unresolved variables.
  • Keeps existing rows required by default for backward compatibility.
  • Adds aligned headers for Enabled, Required, Key, Value, and Description.
  • Passes required-field metadata through request construction and secure execution.
  • Adds tests for validation, omission behavior, hybrid requests, and header alignment.
  • Validation: 12 focused tests and the executor TypeScript check passed.

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