Skip to content

chore: Addressing blitz feedback#115

Draft
jsonbailey wants to merge 2 commits intomainfrom
jb/aic-1664/blitz-feedback
Draft

chore: Addressing blitz feedback#115
jsonbailey wants to merge 2 commits intomainfrom
jb/aic-1664/blitz-feedback

Conversation

@jsonbailey
Copy link
Copy Markdown
Contributor

No description provided.

jsonbailey and others added 2 commits March 30, 2026 11:23
Raw Python callables passed to bind_tools() lack the schema metadata
required to generate a valid OpenAI tools[].function spec, causing a
400 "Missing required parameter: 'tools[0].function'" error.

Wrapping each callable with StructuredTool.from_function(name=config_key)
also ensures the model response carries the config key as the tool name,
fixing a separate bug where function.__name__ was being tracked instead
of the LaunchDarkly config key.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
_to_openai_name converted hyphens to underscores based on an assumed
restriction that doesn't exist — the OpenAI API allows hyphens in tool
and function names (^[a-zA-Z0-9_-]{1,64}$). Removing it simplifies the
tool name lookup and agent naming.

Also adds model=model.name to the Agent constructor so each node runs
with its configured model rather than the SDK default.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.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