Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search and Scrape Webpages #23

Merged
merged 4 commits into from
Oct 18, 2024
Merged

Search and Scrape Webpages #23

merged 4 commits into from
Oct 18, 2024

Conversation

toyamarinyon
Copy link
Contributor

@toyamarinyon toyamarinyon commented Oct 18, 2024

Summary

Search web pages using a query derived from the user's request and scrape each page.
Scraped data transform markdown and persistence to Vercel Blob

Private beta users can try out this prototype.
It does not affect other users.

Changes

Add webSearches property into graphv2 column of agents table.

Release flow

  1. Merge this pull request
  2. Migrate database with following script:
bun drizzle-kit migrate
  1. Migrate data with following script:
bun scripts/20241018-db-patch-web-search.ts

@toyamarinyon toyamarinyon requested a review from shige as a code owner October 18, 2024 03:46
Copy link

vercel bot commented Oct 18, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
giselle ✅ Ready (Inspect) Visit Preview 💬 Add feedback Oct 18, 2024 5:13am

@shige
Copy link
Member

shige commented Oct 18, 2024

@toyamarinyon I got the error in preview https://giselle-1rwcfr8c0-r06-edge.vercel.app/p/agnt_gjujxm4uhoc87hqzumrh2qh0 .

Unhandled Rejection: Error: TAVILY_API_KEY is not defined
    at n (/var/task/.next/server/app/(playground)/p/[agentId]/page.js:31:2671)
    at /var/task/.next/server/app/(playground)/p/[agentId]/page.js:31:1586
    at Array.map (<anonymous>)
    at /var/task/.next/server/app/(playground)/p/[agentId]/page.js:31:1572
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
Node.js process exited with exit status: 128. The logs above can help with debugging the issue.

Could you verify the operation in the preview environment?

@shige shige added the enhancement New feature or request label Oct 18, 2024
@shige
Copy link
Member

shige commented Oct 18, 2024

Maybe the environment variables were set after the Vercel deployment? 🤔

@toyamarinyon
Copy link
Contributor Author

@shige I Yes, a redeployment was necessary. Also, a database migration was needed, so I'll make some adjustments.

@shige
Copy link
Member

shige commented Oct 18, 2024

Thanks! I'll check it after CI passed.

Copy link
Member

@shige shige left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@shige shige merged commit 9b60fc9 into main Oct 18, 2024
7 checks passed
@shige shige deleted the websearch branch October 18, 2024 05:17
@shige
Copy link
Member

shige commented Oct 18, 2024

Release flow

  1. Merge this pull request
  2. Migrate database with following script:
bun drizzle-kit migrate
  1. Migrate data with following script:
bun scripts/20241018-db-patch-web-search.ts

All done! 🧑‍💻

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants