fix: disable take_screenshot tool and remove screenshot references from system prompt for lightpanda engine - #150
Conversation
…om system prompt for lightpanda engine
|
@infer can you fix this? use /adl |
|
Model: Todos
Pull Request✅ Infer Result: SuccessEverything is clean and pushed. The PR #150 now has two commits that together provide a complete, ADL-compliant fix for the lightpanda screenshot issue. Model: Tokens: 2,333,214 in · 13,033 out · 2,346,247 total (29 requests) Tool calls: 37 total · 100% success rate TracesStatsTriggered by edenreich · Infer Action · OSS |
…or lightpanda screenshot support
Co-authored-by: Eden Reich <eden.reich@gmail.com>
|
@infer Let's keep it simple just update the system prompt - no need to update each skill - a small instruction in the system prompt is enough, also remove the changes from the main.go let it get generated. use /adl |
|
Model: Todos
Pull Request✅ Infer Result: SuccessEverything is clean and pushed. The PR #150 is up to date with the simplified approach: a small lightpanda note in the system prompt, skill files reverted to origin/main, and Model: Tokens: 1,602,863 in · 8,728 out · 1,611,591 total (30 requests) Tool calls: 35 total · 100% success rate TracesStatsTriggered by edenreich · Infer Action · OSS |
…vert skill files, let ADL generate main.go
## [0.8.1](v0.8.0...v0.8.1) (2026-08-01) ### 🐛 Bug Fixes * disable take_screenshot tool and remove screenshot references from system prompt for lightpanda engine ([#150](#150)) ([51d7c3d](51d7c3d)), closes [#149](#149), references [#5](#5) ### 🔧 Miscellaneous * **deps:** bump ADL CLI to v0.58.1 ([#151](#151)) ([6b94c67](6b94c67))
|
🎉 This PR is included in version 0.8.1 🎉 The release is available on:
Your semantic-release bot 📦🚀 |
Resolves #149
Summary
When the browser engine is configured as
lightpanda, thetake_screenshottool was still registered and available to the agent, but it would always fail at runtime because lightpanda has no graphical rendering engine. This change conditionally disables the tool and removes screenshot-related instructions from the system prompt when lightpanda is detected.Changes
isLightpandacheck inmain.goby comparing the configured engine against theplaywright.Lightpandaconstanttake_screenshottool only when the engine supports screenshots (chromium, firefox, webkit); log a skip message for lightpandaplaywright.go'sTakeScreenshotmethod (which returns a descriptive error for lightpanda) is preserved as a safety net