Skip to content

fix: include file_path in smart_tests_generate response#11

Open
mcp-tool-shop wants to merge 1 commit intomainfrom
fix/smart-tests-file-path
Open

fix: include file_path in smart_tests_generate response#11
mcp-tool-shop wants to merge 1 commit intomainfrom
fix/smart-tests-file-path

Conversation

@mcp-tool-shop
Copy link
Member

Summary

  • Add file_path key to the JSON response from smart_tests_generate tool
  • Fixes KeyError: 'file_path' in test_smart_tests_generate_success and test_smart_tests_generate_exception
  • Response now includes both file_path (full resolved path) and file_name (basename)

Test plan

  • test_smart_tests_generate_success passes
  • test_smart_tests_generate_exception passes
  • All 186 tests pass

🤖 Generated with Claude Code

The tool response included file_name but not file_path, causing
test_smart_tests_generate_success and test_smart_tests_generate_exception
to fail with KeyError: 'file_path'.

Co-Authored-By: Claude Opus 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