Skip to content

Backfill required MCP form and field operations - #23

Merged
S4Mi merged 1 commit into
devfrom
feat/mcp-required-operation-backfill-dev
Aug 10, 2026
Merged

Backfill required MCP form and field operations#23
S4Mi merged 1 commit into
devfrom
feat/mcp-required-operation-backfill-dev

Conversation

@S4Mi

@S4Mi S4Mi commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

Summary

  • backfill MCP-required form and field operations that are missing from the mcp-1.0 source bundle using the v3.0 formz bundle during MCP artifact generation
  • add MCP metadata/examples for form delete, single-field read/create/delete, and keep joint form-and-fields endpoints as the primary form authoring path
  • extend MCP spec validation so these core operations cannot silently disappear from openapi-v3.0.mcp.yaml

Why

The MCP server policy requires these operation IDs to exist in the MCP OpenAPI artifact. The hosted mcp-1.0 source currently omits some of them, so FormalooMCP had to treat the local spec as safer than the hosted MCP artifact.

This PR fixes the generated docs side while preserving the intended API order: agents should start with composed MCP form tools or the joint form-and-fields endpoints (formFieldsRetrieve / formFieldsPartialUpdate) for normal form authoring. Single-field endpoints are still valid for targeted one-field reads, adds, or deletes, but they are not documented as the first/default route for multi-field form construction or coordinated updates.

Validation

  • ./generate.sh
  • npm run validate:mcp-spec
  • verified generated openapi-v3.0.mcp.yaml contains formsDestroy, fieldsRetrieve, fieldsCreate, and fieldsDestroy
  • verified the generated single-field descriptions steer form authoring, ordering, logic-aware edits, and coordinated updates to the joint form-and-fields flow first
  • cross-checked generated MCP operation IDs against FormalooMCP operation-policy.json: no missing policy operations

@S4Mi
S4Mi force-pushed the feat/mcp-required-operation-backfill-dev branch from d17f8ae to 9592441 Compare August 10, 2026 17:24
@S4Mi
S4Mi force-pushed the feat/mcp-required-operation-backfill-dev branch from 9592441 to 8a9e0d7 Compare August 10, 2026 17:29
@S4Mi
S4Mi marked this pull request as ready for review August 10, 2026 18:15
@S4Mi
S4Mi merged commit 6dd718f into dev Aug 10, 2026
1 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