Skip to content

test: add regression tests for issues #287 and #341#380

Merged
robert-j-y merged 2 commits intomainfrom
devin/1769568904-issue-regression-tests
Jan 28, 2026
Merged

test: add regression tests for issues #287 and #341#380
robert-j-y merged 2 commits intomainfrom
devin/1769568904-issue-regression-tests

Conversation

@robert-j-y
Copy link
Contributor

@robert-j-y robert-j-y commented Jan 28, 2026

Description

Adds e2e regression tests for two fixed issues:

Issue #287 - Tool calls with missing arguments field

  • Tests that tool calls work correctly when upstream providers omit the arguments field for tools with no parameters
  • Previously caused AI_TypeValidationError because the schema required arguments to be a string
  • Uses anthropic/claude-3.5-haiku (mentioned in original issue context as potentially omitting the arguments field)

Issue #341 - Cache control only applies to last text part

  • Tests that message-level cache_control works correctly with multi-part messages on Anthropic models
  • Previously applied cache_control to ALL parts, which could exceed provider cache segment limits
  • Uses anthropic/claude-3.5-haiku which supports cache control

Updates since last revision

Human Review Checklist

Checklist

  • I have run pnpm stylecheck and pnpm typecheck
  • I have run pnpm test and all tests pass
  • I have added tests for my changes (if applicable)
  • I have updated documentation (if applicable) - N/A

Changeset

  • I have run pnpm changeset to create a changeset file

Note: Used pnpm changeset --empty since this is a test-only change that doesn't need a release.


Link to Devin run: https://app.devin.ai/sessions/ad14476ac9334b8a962f771f56e48975
Requested by: Robert Yeakel (@robert-j-y)

devin-ai-integration bot and others added 2 commits January 28, 2026 03:06
- Issue #287: Tool calls with missing arguments field
  Tests that tool calls work correctly when the arguments field is omitted
  by upstream providers for tools with no parameters

- Issue #341: Cache control only applies to last text part
  Tests that message-level cache_control works correctly with multi-part
  messages on Anthropic models

Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>
Updated to use anthropic/claude-3.5-haiku as mentioned in the original
issue context (Anthropic Haiku was noted as potentially omitting the
arguments field for tools with no parameters).

Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>
@robert-j-y robert-j-y merged commit 2dabe2c into main Jan 28, 2026
2 checks passed
@robert-j-y robert-j-y deleted the devin/1769568904-issue-regression-tests branch January 28, 2026 03:19
kesavan-byte pushed a commit to osm-API/ai-sdk-provider that referenced this pull request Feb 13, 2026
…erTeam#341 (OpenRouterTeam#380)

* test: add regression tests for issues OpenRouterTeam#287 and OpenRouterTeam#341

- Issue OpenRouterTeam#287: Tool calls with missing arguments field
  Tests that tool calls work correctly when the arguments field is omitted
  by upstream providers for tools with no parameters

- Issue OpenRouterTeam#341: Cache control only applies to last text part
  Tests that message-level cache_control works correctly with multi-part
  messages on Anthropic models

Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>

* fix: use Anthropic Haiku model for issue OpenRouterTeam#287 test

Updated to use anthropic/claude-3.5-haiku as mentioned in the original
issue context (Anthropic Haiku was noted as potentially omitting the
arguments field for tools with no parameters).

Co-Authored-By: Robert Yeakel <robert.yeakel@openrouter.ai>

---------

Co-authored-by: Devin AI <158243242+devin-ai-integration[bot]@users.noreply.github.com>
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