Skip to content

feat: Support all pydantic model's field types#213

Open
dsowinski2 wants to merge 1 commit into
mainfrom
feat/ent-157-add-possibility-to-create-list-type-agentplugin-parameter
Open

feat: Support all pydantic model's field types#213
dsowinski2 wants to merge 1 commit into
mainfrom
feat/ent-157-add-possibility-to-create-list-type-agentplugin-parameter

Conversation

@dsowinski2
Copy link
Copy Markdown
Contributor

@dsowinski2 dsowinski2 commented Oct 29, 2025

This PR changes a way of handling agent, sources and integration arguments with json schema.

Now it can handle all types of Pydantic model parameters (Lists, Enums, Optionals etc.)

Enthusiast Common:

  1. Added check for Union types - No more than 2 types and one of them must be None. This will guarantee that form receive strict field type.
  2. Each Optional Field received default None

BE:

  1. Agents, sources and Integrations types endpoints are simplified and now returns Pydantic's model_json_schema()

UI:

  1. Added RJSF lib to handle forms based on json schema
  2. RSJF templates customization
  3. RjsfForm component for handling all configuration args

@dsowinski2 dsowinski2 self-assigned this Oct 29, 2025
@dsowinski2 dsowinski2 force-pushed the feat/ent-157-add-possibility-to-create-list-type-agentplugin-parameter branch from d3d6617 to 3c5fb99 Compare October 29, 2025 11:06
@dsowinski2 dsowinski2 marked this pull request as draft January 29, 2026 11:46
@dsowinski2 dsowinski2 force-pushed the feat/ent-157-add-possibility-to-create-list-type-agentplugin-parameter branch 7 times, most recently from 73e48d0 to 44532ae Compare February 3, 2026 12:16
@dsowinski2 dsowinski2 changed the title feat: Handle list type parameters in configuration feat: Support all pydantic model's field types Feb 3, 2026
@dsowinski2 dsowinski2 marked this pull request as ready for review February 3, 2026 12:25
@dsowinski2 dsowinski2 requested a review from mateus-po February 10, 2026 06:58
@dsowinski2 dsowinski2 force-pushed the feat/ent-157-add-possibility-to-create-list-type-agentplugin-parameter branch from 44532ae to e95bdb9 Compare February 11, 2026 09:10
Copy link
Copy Markdown
Contributor

@mateus-po mateus-po left a comment

Choose a reason for hiding this comment

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

if properly tested with different argument types - LGTM

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