Dev by h3xxit · Pull Request #94 · universal-tool-calling-protocol/python-utcp · GitHub
Skip to content

Dev#94

Merged
h3xxit merged 2 commits into
mainfrom
dev
Jun 24, 2026
Merged

Dev#94
h3xxit merged 2 commits into
mainfrom
dev

Conversation

@h3xxit

@h3xxit h3xxit commented Jun 24, 2026

Copy link
Copy Markdown
Member

Summary by cubic

Fix OpenAPI conversion so unsupported HTTP methods aren’t silently dropped; they now reach validation and emit a skip warning. Bumps utcp-http to 1.1.9.

  • Bug Fixes
    • The conversion loop now recognizes all OpenAPI operation methods and ignores only non-operation path-item keys.
    • Method validation stays in the per-operation builder, which warns and skips unsupported methods.
    • Test updated to assert the warning is printed for options/head/trace.

Written for commit bb1ee06. Summary will update on new commits.

Review in cubic

h3xxit and others added 2 commits June 24, 2026 22:48
The convert() loop pre-filtered operations to SUPPORTED_HTTP_METHODS, so the
validation guard in _create_tool could never run and its warning never fired —
dead code. Split the method sets: the loop now filters on the full set of
OpenAPI operation methods (so it still skips non-operation path-item keys like
parameters/summary/$ref), and _create_tool remains the single place that
validates against SUPPORTED_HTTP_METHODS, warning and skipping options/head/
trace operations.

Strengthens the skip test to assert the warning is emitted. Bumps utcp-http
1.1.8 -> 1.1.9.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
…pi-unsupported-method-warning

fix(openapi): make unsupported-method warning reachable

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

No issues found across 3 files

Re-trigger cubic

@h3xxit h3xxit merged commit b10d882 into main Jun 24, 2026
19 checks 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