Skip to content

Conversation

@hannesrudolph
Copy link
Collaborator

@hannesrudolph hannesrudolph commented Feb 11, 2026

Summary

Deletes 4 orphaned source files and 4 orphaned test files from src/api/transform/caching/. These addCacheBreakpoints() functions have zero provider consumers — no provider file imports them.

Cache breakpoint placement has moved to inline providerOptions in each provider's createMessage() method:

  • Anthropic/Anthropic Vertex/Minimax: { anthropic: { cacheControl: { type: "ephemeral" } } } via applyCacheControlToAiSdkMessages()
  • Bedrock: { bedrock: { cachePoint: { type: "default" } } } via applyCacheControlToAiSdkMessages()
  • Gemini/Vertex: No breakpoints set (server-side auto-caching)

Files Deleted

File Lines Former Strategy
caching/anthropic.ts 41 system + last 2 user (OpenAI-format input)
caching/gemini.ts 47 system + every Nth user (frequency-based)
caching/vertex.ts 49 last 2 user (Anthropic-format input)
caching/vercel-ai-gateway.ts 30 system + last 2 user
caching/__tests__/anthropic.spec.ts Tests for above
caching/__tests__/gemini.spec.ts Tests for above
caching/__tests__/vertex.spec.ts Tests for above
caching/__tests__/vercel-ai-gateway.spec.ts Tests for above

Total: 1,025 lines deleted.

Verification

  • Confirmed zero imports of addCacheBreakpoints or from.*caching/ across all provider files
  • All 5,499 tests pass
  • Lint and type checks clean

Important

Removed the entire caching module including all provider-specific implementations and their test files for the addCacheBreakpoints functionality.

  • Deleted caching implementations:
    • Removed anthropic.ts, gemini.ts, vercel-ai-gateway.ts, and vertex.ts files containing provider-specific addCacheBreakpoints functions.
  • Deleted test files:
    • Removed anthropic.spec.ts, gemini.spec.ts, vercel-ai-gateway.spec.ts, and vertex.spec.ts test files covering the caching functionality across all providers.

This description was created by Ellipsis for 2c4f10a. You can customize this summary. It will automatically update as commits are pushed.

Delete 4 source files and 4 test files from src/api/transform/caching/:
- anthropic.ts + anthropic.spec.ts
- gemini.ts + gemini.spec.ts
- vertex.ts + vertex.spec.ts
- vercel-ai-gateway.ts + vercel-ai-gateway.spec.ts

These addCacheBreakpoints() functions are orphaned dead code — zero
provider files import them. Cache breakpoint placement has moved to
inline providerOptions in each provider's createMessage() method
(e.g. { anthropic: { cacheControl: { type: 'ephemeral' } } }).
@dosubot dosubot bot added the size:XXL This PR changes 1000+ lines, ignoring generated files. label Feb 11, 2026
@roomote
Copy link
Contributor

roomote bot commented Feb 11, 2026

Rooviewer Clock   See task

No issues found. The 8 deleted files are confirmed orphaned -- zero imports of addCacheBreakpoints or any caching/ module exist across the codebase. The replacement (applyCacheControlToAiSdkMessages) is already in place in the relevant providers.

Mention @roomote in a comment to request specific changes to this pull request or fix all unresolved issues.

@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Feb 11, 2026
@hannesrudolph hannesrudolph merged commit 5800363 into main Feb 11, 2026
23 of 25 checks passed
@hannesrudolph hannesrudolph deleted the refactor/delete-orphaned-caching-transforms branch February 11, 2026 00:39
@github-project-automation github-project-automation bot moved this from New to Done in Roo Code Roadmap Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:XXL This PR changes 1000+ lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants