Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: inference-gateway/browser-agent
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: v0.3.2
Choose a base ref
...
head repository: inference-gateway/browser-agent
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: v0.3.3
Choose a head ref
  • 2 commits
  • 20 files changed
  • 2 contributors

Commits on Oct 14, 2025

  1. refactor: Remove write_to_csv skill and related tests (#36) 

    ## Summary

Removes the specialized `write_to_csv` skill in favor of using ADK's
built-in `create_artifact` tool. The `take_screenshot` skill now saves
files locally and returns file paths instead of automatically creating
artifacts. This promotes better separation of concerns - skills handle
browser automation while the agent uses `create_artifact` for making
files downloadable.

## Changes

### Removed
- **write_to_csv skill** and all related tests
(`skills/write_to_csv.go`, `skills/write_to_csv_test.go`)
- Automatic artifact creation logic from `take_screenshot` skill

### Modified
- **take_screenshot skill**: Now saves screenshots to local filesystem
and returns the file path instead of automatically creating artifacts
- **Agent system prompt**: Updated to include instructions for using
`create_artifact` tool with screenshots and data extraction
- **Main server initialization**: 
- Changed from `NewDefaultToolBox()` to `NewToolBox()` +
`WithDefaultToolBox()` pattern
  - Added `ArtifactService` initialization and configuration
- Updated server builder to use `WithArtifactService` instead of
`WithArtifactStorage`
- **Dependencies**: Upgraded ADK from v0.14.0 to v0.15.0

### Benefits
- **Better separation of concerns**: Skills focus on their core
functionality (browser automation), artifact creation is handled by the
agent
- **More flexible**: Users can choose whether to create artifacts or
just work with local files
- **Reduced code duplication**: Removes specialized artifact handling
from individual skills
- **Consistent pattern**: All file creation now follows the same
pattern: skill creates file → agent uses `create_artifact` to make it
downloadable

## Test Plan
- [x] Removed tests for `write_to_csv` skill
- [x] Updated `take_screenshot` tests to validate file path return
instead of artifact creation
- [x] Manual testing: Verify screenshots can be taken and then converted
to artifacts using `create_artifact` tool
- [ ] Manual testing: Verify data extraction → CSV conversion → artifact
creation workflow

## Migration Notes
For users upgrading to this version:
- The `write_to_csv` skill is no longer available
- To create CSV files, use data extraction combined with the
`create_artifact` tool
- Screenshots now return file paths; use `create_artifact` to make them
downloadable

---------

Signed-off-by: Eden Reich <eden.reich@gmail.com>
    @edenreich
    edenreich authored Oct 14, 2025
    Configuration menu
    Copy the full SHA
    144d901 View commit details
    Browse the repository at this point in the history

  2. chore(release): 🔖 0.3.3 [skip ci] 

    ## [0.3.3](v0.3.2...v0.3.3) (2025-10-14)

### ♻️ Improvements

* Remove write_to_csv skill and related tests ([#36](#36)) ([144d901](144d901))
    @semantic-release-bot
    semantic-release-bot committed Oct 14, 2025
    Configuration menu
    Copy the full SHA
    483b035 View commit details
    Browse the repository at this point in the history
Loading