Skip to content

Conversation

@OskarStark
Copy link
Contributor

@OskarStark OskarStark commented Sep 9, 2025

Q A
Bug fix? no
New feature? yes
Docs? no
Issues --
License MIT
CleanShot 2025-09-09 at 09 51 05@2x

@OskarStark OskarStark self-assigned this Sep 9, 2025
@carsonbot carsonbot added Agent Issues & PRs about the AI Agent component Feature New feature Status: Needs Review labels Sep 9, 2025
@OskarStark OskarStark force-pushed the feature/add-mapbox-geocoding-tool branch from 40105f6 to 0e1c4d7 Compare September 9, 2025 06:36
@OskarStark OskarStark changed the title [Agent] Add Mapbox.com geocoding tool for address-to-coordinates conversion [Agent][Toolbox] Add Mapbox.com geocoding tool for address-to-coordinates conversion Sep 9, 2025
$processor = new AgentProcessor($toolbox);
$agent = new Agent($platform, $model, [$processor], [$processor], logger());

$messages = new MessageBag(Message::ofUser('What address is at coordinates longitude -73.985131, latitude 40.758895?'));
Copy link
Member

Choose a reason for hiding this comment

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

those example coordinates result in an "couldn't retrieve" response for me.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Same for me, but I can't find a coordinate which brings up a real address. shall we remove the reverse thing?

Copy link
Member

Choose a reason for hiding this comment

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

No, it's actually with your example as well that it brings back the right address in New York, but the array handling of the response data in Symfony\AI\Agent\Toolbox\Tool\Mapbox::reverseGeocode is wrong.

@OskarStark OskarStark force-pushed the feature/add-mapbox-geocoding-tool branch 3 times, most recently from 171941d to 95021b6 Compare September 9, 2025 16:43
@OskarStark
Copy link
Contributor Author

Fixed
CleanShot 2025-09-10 at 10 51 46@2x

@OskarStark OskarStark force-pushed the feature/add-mapbox-geocoding-tool branch from 4487035 to 80b83fd Compare September 10, 2025 08:52
@OskarStark OskarStark merged commit 4aeca02 into symfony:main Sep 10, 2025
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Agent Issues & PRs about the AI Agent component Feature New feature Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants