Skip to content

Conversation

@OskarStark
Copy link
Contributor

Q A
Bug fix? no
New feature? no
Docs? no
Issues --
License MIT

Tests the Scraper tool's ability to extract title and content from HTML pages using MockHttpClient, and verifies that sources are properly tracked in the SourceMap.

@carsonbot carsonbot changed the title Add test case for Scraper tool Add test case for Scraper tool Nov 24, 2025
@OskarStark OskarStark added the Agent Issues & PRs about the AI Agent component label Nov 24, 2025
@carsonbot carsonbot changed the title Add test case for Scraper tool [Agent] Add test case for Scraper tool Nov 24, 2025
@OskarStark OskarStark changed the title [Agent] Add test case for Scraper tool [Agent][Scraper] Add test case Nov 24, 2025
@OskarStark OskarStark merged commit c75a24b into symfony:main Nov 24, 2025
@OskarStark OskarStark deleted the scraper-test branch November 24, 2025 21:47
OskarStark added a commit that referenced this pull request Nov 24, 2025
…owest (OskarStark)

This PR was merged into the main branch.

Discussion
----------

[Agent][Scraper] Require `masterminds/html5` ^2.8 for CI lowest

| Q             | A
| ------------- | ---
| Bug fix?      | no
| New feature?  | no
| Docs?         | no
| Issues        | Follows #969
| License       | MIT

The Scraper tool tests require a newer version of masterminds/html5 to work correctly in the monorepo CI environment. Version 2.8 handles directory traversal more gracefully.

Commits
-------

1fb5cb8 Require masterminds/html5 ^2.8 for ScraperTest compatibility
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