Skip to content

Conversation

@edenreich
Copy link
Contributor

@edenreich edenreich commented Oct 18, 2025

Summary

This PR adds headless browser configuration support and updates project dependencies to their latest versions.

Key Changes

Browser Configuration:

  • ✅ Add BROWSER_HEADLESS environment variable support (default: true)
  • ✅ Fix hardcoded headless value in NewBrowserConfigFromConfig function (internal/playwright/playwright.go:92)
  • ✅ Allow users to disable headless mode for debugging and development by setting BROWSER_HEADLESS=false
  • ✅ Update README with BROWSER_HEADLESS configuration documentation

Dependency Updates:

  • ✅ Upgrade ADL CLI from v0.23.7 to v0.23.8 in generated workflow and configuration files
  • ✅ Upgrade ADK dependency from v0.15.1 to v0.15.2 in go.mod
  • ✅ Upgrade Claude Code from 2.0.11 to 2.0.15 in Flox manifest

Refactoring:

  • ✅ Streamline artifact server creation in main.go

Configuration Usage

Users can now control headless mode via environment variable:

# Run in headless mode (default)
BROWSER_HEADLESS=true go run .

# Run with visible browser for debugging
BROWSER_HEADLESS=false go run .

Testing

  • ✅ All unit tests pass
  • ✅ Linter checks pass (0 issues)
  • ✅ Headless configuration properly reads from environment

Files Changed

  • agent.yaml - Added headless configuration
  • config/config.go - Added Headless field to BrowserConfig struct
  • internal/playwright/playwright.go - Fixed hardcoded headless value to use config
  • README.md - Documented BROWSER_HEADLESS variable
  • main.go - Refactored artifact server creation
  • .flox/env/manifest.toml - Updated Claude Code version
  • .github/workflows/ci.yml - Updated ADL CLI version
  • go.mod / go.sum - Updated ADK dependency

🤖 Generated with Claude Code

edenreich and others added 3 commits October 18, 2025 19:28
- Bump claude-code version from 2.0.11 to 2.0.15 in manifest.toml
- Update ADL CLI version from 0.23.7 to 0.23.8 in generated workflow and configuration files
- Add headless configuration option to agent.yaml and BrowserConfig struct
- Update Go module dependency for inference-gateway/adk from v0.15.1 to v0.15.2
- Add BROWSER_HEADLESS variable to README for browser configuration
- Refactor main.go to streamline artifact server creation

Signed-off-by: Eden Reich <eden.reich@gmail.com>
- Fix hardcoded headless value in NewBrowserConfigFromConfig function
- Now properly reads BROWSER_HEADLESS environment variable
- Allows users to disable headless mode for debugging and development
- All tests pass successfully

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@edenreich edenreich merged commit c08b4fb into main Oct 18, 2025
1 check passed
@edenreich edenreich deleted the feature/add-headless-configuration branch October 18, 2025 17:36
ig-semantic-release-bot bot pushed a commit that referenced this pull request Oct 18, 2025
## [0.4.0](v0.3.4...v0.4.0) (2025-10-18)

### ✨ Features

* Add headless configuration ([#37](#37)) ([c08b4fb](c08b4fb))
@ig-semantic-release-bot
Copy link

🎉 This PR is included in version 0.4.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants