Skip to content

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

@francescobianco

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.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions