Skip to content

Conversation

@OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? no
Issues Leftover from #948
License MIT

@OskarStark OskarStark added the Agent Issues & PRs about the AI Agent component label Nov 24, 2025
@carsonbot carsonbot changed the title Remove duplicate OpenMeteo tool and move to Bridge structure Remove duplicate OpenMeteo tool and move to Bridge structure Nov 24, 2025
@carsonbot carsonbot changed the title Remove duplicate OpenMeteo tool and move to Bridge structure [Agent] Remove duplicate OpenMeteo tool and move to Bridge structure Nov 24, 2025
@OskarStark OskarStark changed the title [Agent] Remove duplicate OpenMeteo tool and move to Bridge structure [Agent][OpenMeteo] Remove duplicate OpenMeteo tool and move to Bridge structure Nov 24, 2025
@OskarStark OskarStark force-pushed the remove-duplicate-openmeteo branch 2 times, most recently from 36d2007 to c8a6db8 Compare November 24, 2025 09:19
@OskarStark OskarStark changed the title [Agent][OpenMeteo] Remove duplicate OpenMeteo tool and move to Bridge structure [Agent][OpenMeteo] Remove duplicate OpenMeteo tool Nov 24, 2025
@OskarStark OskarStark force-pushed the remove-duplicate-openmeteo branch from c8a6db8 to 35bff4e Compare November 24, 2025 09:21
The OpenMeteo tool was duplicated in both Toolbox/Tool and Bridge/OpenMeteo
directories. This commit removes the duplicate and ensures everything uses
the Bridge structure.

Changes:
- Remove src/agent/src/Toolbox/Tool/OpenMeteo.php
- Move tests from tests/Toolbox/Tool to tests/Bridge/OpenMeteo
- Move fixtures from fixtures/Tool to fixtures/Bridge/OpenMeteo
- Rename fixture files from openmeteo-*.json to *.json
- Update test to use Bridge namespace and new fixture paths
@OskarStark OskarStark force-pushed the remove-duplicate-openmeteo branch from 35bff4e to 0895bae Compare November 24, 2025 09:24
@OskarStark OskarStark merged commit b707e18 into symfony:main Nov 24, 2025
16 checks passed
@OskarStark OskarStark deleted the remove-duplicate-openmeteo branch November 24, 2025 09:28
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 Status: Needs Review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants