Refactor imports.

[alex-mcgraw-askSage]

Description

Refactor some interface imports, and fixed a bug in the CLI tooling. File reads weren't working, but now are.

AI Code Review

• Team members only: AI review runs automatically when PR is opened or marked ready for review
• Team members can also trigger a review by commenting @continue-review

Checklist

• [✅] I've read the contributing guide
• [NA] The relevant docs, if any, have been updated or created
• [NA] The relevant tests, if any, have been updated or created

Screen recording or screenshot

Example of file reads in the CLI tool.

Tests

NA

---

Continue Tasks: ▶️ 3 queued — View all

---

Summary by cubic

Fixed CLI file reading and standardized AskSage type definitions across core and openai-adapters to reduce errors and improve compatibility.

• Bug Fixes

  • CLI file reads now work as expected.
  • Streaming responses filter malformed tool calls and default missing arguments to an empty string to prevent crashes.
  • Finish_reason now uses valid tool calls instead of raw ones, avoiding incorrect "tool_calls" endings.

• Refactors

  • Moved AskSage interfaces (Tool, ToolChoice, ToolCall, Response, TokenResponse) to openai-adapters/types and updated imports.
  • Exported AskSage types from openai-adapters index for external use.
  • Replaced generic ToolChoice/TokenResponse with AskSage-specific types in core and adapters.

^{Written for commit 63a8f8c. Summary will update on new commits.}

[cubic-dev-ai]

1 issue found across 4 files

Prompt for AI agents (all issues)

Check if these issues are valid — if so, understand the root cause of each and fix them.


<file name="packages/openai-adapters/src/apis/AskSage.ts">

<violation number="1" location="packages/openai-adapters/src/apis/AskSage.ts:431">
P2: Streaming finish_reason is derived from raw tool call count, so it can report "tool_calls" even when all tool calls were filtered out and none were emitted.</violation>
</file>

_{Reply with feedback, questions, or to request a fix. Tag @cubic-dev-ai to re-run a review.}

[RomneyDa]

Cubic comment seems valid

[RomneyDa]

@alex-mcgraw-askSage noticed the title mentions a fix for CLI file reads, was that intentional?

[alex-mcgraw-askSage]

@alex-mcgraw-askSage noticed the title mentions a fix for CLI file reads, was that intentional?

@RomneyDa Thanks for the catch on the rawToolCalls!

In regards to CLI file reads, yes. When testing the updates, I tried reading files, "tell me about this file @", and i was getting an error. A bug got introduced in my last PR with file reads, so needed that fixed.

edit 1: My bad, I'm looking back at the changes and see everything is related to the extensions and none to core. I realized the problem was something with my debugger config, so no need to worry about that. I'll fix that title now.

[sestinj]

🎉 This PR is included in version 1.9.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀