-
Notifications
You must be signed in to change notification settings - Fork 6
Fixes #28
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Open
sergiobayona
wants to merge
27
commits into
567-labs:main
Choose a base branch
from
sergiobayona:fixes
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Fixes #28
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
04f0a8b
fix tool function name return value
sergiobayona f9953be
use the openai client chat function
sergiobayona edebf01
refreshed vcr cassette
sergiobayona 4d201ea
fixes
sergiobayona ceadd29
update easy talk
sergiobayona d26f225
two basic modes
sergiobayona 691ea52
support for structured and tool response
sergiobayona 1d8f4f3
new patch design
sergiobayona 1fba7a1
named spaced responses
sergiobayona 2d61629
adjust per EasyTalk changes
sergiobayona d4d7481
improved response handling
sergiobayona b7796a4
refactored response module
sergiobayona c61496e
updated specs
sergiobayona 8d3bf20
namedspaced mode setting
sergiobayona d64aa7d
patch fixes
sergiobayona 59ec585
use easy_talk 2
sergiobayona e01f295
ignore CLAUDE.md
sergiobayona ca3ab9f
Add unified Instructor::Mode system for OpenAI and Anthropic
sergiobayona 7d6a5ee
updated gem dependencies versions
sergiobayona f7788da
new mode mechanism
sergiobayona af3928b
feature specs name-spaced by provider
sergiobayona 5897167
updated cassettes
sergiobayona 7b09f4c
use sonnet 4
sergiobayona 6250986
anthropic mode specs
sergiobayona 47d78e3
trigger a validation error
sergiobayona 8d9de60
trigger refusal
sergiobayona 6c09306
updated response
sergiobayona File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,23 @@ | ||
# frozen_string_literal: true | ||
|
||
require_relative '../mode' | ||
|
||
module Instructor | ||
# Anthropic-specific mode configuration and management | ||
module Anthropic | ||
# Sets the current mode for Anthropic API interactions | ||
# | ||
# @param mode [Symbol] The mode to use (from Instructor::Mode constants) | ||
# @return [Symbol] The mode that was set | ||
def self.mode=(mode) | ||
@mode = mode | ||
end | ||
|
||
# Gets the current mode for Anthropic API interactions | ||
# | ||
# @return [Symbol] The current mode, defaults to ANTHROPIC_TOOLS | ||
def self.mode | ||
@mode ||= Instructor::Mode::ANTHROPIC_TOOLS | ||
end | ||
end | ||
end |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The method tool_mode? uses mode.to_s.start_with!("anthropic") to decide if the mode is tool‐based. This string-based check is a bit brittle. Consider comparing against a set of known Anthropic mode symbols for more clarity.