Skip to content

chore: gut agent#2

Merged
grantdfoster merged 21 commits intomasterfrom
feat--web-scraping
Nov 5, 2025
Merged

chore: gut agent#2
grantdfoster merged 21 commits intomasterfrom
feat--web-scraping

Conversation

@grantdfoster
Copy link
Collaborator

@grantdfoster grantdfoster commented Nov 4, 2025

  • guts the agent from the client in favor of implementing directly into go trader

Copilot AI review requested due to automatic review settings November 4, 2025 18:43
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR refactors the agent system from topic-based sentiment analysis to asset-based sentiment tracking. The changes replace a string-based sentiment model with a numeric scoring system (1-100) and expand tool capabilities to include web scraping alongside Twitter search.

Key changes:

  • Refactored data model from TopicSummary to AssetSummary with numeric sentiment scoring (1-100 scale)
  • Added WebSearch tool for scraping cryptocurrency market websites
  • Updated TwitterSearch tool interface from Run() to Execute() method and improved error handling
  • Introduced configurable query options pattern for flexible agent behavior
  • Updated dependencies including cogito upgrade from v0.3.3 to v0.5.0

Reviewed Changes

Copilot reviewed 10 out of 11 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
types/agent.go Changed data model from TopicSummary to AssetSummary with numeric sentiment
agent/agent.go Added QueryOptions pattern, switched from ContentReview to ExecuteTools, improved extraction error handling
agent/twitter.go Renamed Run() to Execute(), added timeout error handling, updated default date range
agent/web.go New file implementing WebSearch tool for website scraping
agent/errors.go New utility for detecting timeout errors
agent/assets.go New file defining default crypto assets list
agent/websites.go New file defining default market data websites
agent/kols.go New file defining Key Opinion Leaders list for Twitter monitoring
agent/agent_test.go Updated test to verify asset sentiment analysis with new query pattern
go.mod Updated cogito dependency and added local replace directive
go.sum Updated dependency checksums for various packages

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@grantdfoster grantdfoster changed the title feat: web scraping chore: gut agent Nov 5, 2025
@grantdfoster grantdfoster merged commit 302f2b0 into master Nov 5, 2025
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants