Skip to content

Bug: DocuEngine validation-rule parsing swallows errors and can skip required-field checks #29

Description

@francescobianco

Summary

The DocuEngine parameter validator wraps validation-rule parsing in a broad except Exception: pass. If a backend rule changes or a formatting edge case appears, the required-field validation may silently stop applying.

Reproduction

  1. Follow the validate_params() path in src/openapi_mcp_sdk/apis/docuengine.py.
  2. Notice that validation expression failures are swallowed.
  3. A malformed or unexpected rule can therefore bypass the intended validation branch.

Expected

Validation parser failures should return a clear error or at least produce a warning that the rule could not be enforced.

Actual

Rule parsing errors are ignored and execution continues.

Why it matters

This can turn backend-side validation drift into hard-to-debug user-facing behavior.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions