Skip to content

Conversation

@CamilleLetavernier
Copy link
Contributor

fixes #16777

What it does

  • Clarify the description of the "addFiles" parameter in "context_addFile" tool, to make it consistent with the tool description and the actual tool implementation
    • Only paths relative to the workspace root are expected (e.g. packages/ai-ide/src/browser/context-functions.ts rather than /packages/ai-ide/src/browser/context-functions.ts)

How to test

  • Hard to test, as LLMs are non-deterministic

Follow-ups

Breaking changes

  • This PR introduces breaking changes and requires careful review. If yes, the breaking changes section in the changelog has been updated.

Attribution

Review checklist

Reminder for reviewers

@github-project-automation github-project-automation bot moved this to Waiting on reviewers in PR Backlog Dec 19, 2025
Copy link
Member

@ndoschek ndoschek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM 👍

@github-project-automation github-project-automation bot moved this from Waiting on reviewers to Needs merge in PR Backlog Dec 19, 2025
@CamilleLetavernier CamilleLetavernier merged commit c8b1cff into master Dec 19, 2025
16 of 17 checks passed
@CamilleLetavernier CamilleLetavernier deleted the issues/16777 branch December 19, 2025 14:27
@github-project-automation github-project-automation bot moved this from Needs merge to Done in PR Backlog Dec 19, 2025
@github-actions github-actions bot added this to the 1.68.0 milestone Dec 19, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

Add files to context: misleading tool description confuses LLMs

3 participants