Use Fetch API for retrieval-only templates - #107
Open
shrey150 wants to merge 3 commits into
Open
Conversation
|
Review the following changes in direct dependencies. Learn more about Socket for GitHub.
|
shrey150
commented
Aug 24, 2026
shrey150
commented
Aug 24, 2026
Addresses feedback from shrey150
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes using default effort and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Want higher recall? High effort reviews run extra passes and find more bugs. A team admin can switch effort levels in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 27a1001. Configure here.
Addresses feedback from cursor[bot]
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.

Summary
Scope
website-link-testerbusiness-lookupcompany-value-prop-generatorcouncil-eventssec-filing-researchThis intentionally does not rewrite interactive or rendered-content workflows.
cerebras-docs-checkerstill demonstrates rendered accessibility content,company-address-finderis a Search API candidate for a separate change, and interactive templates remain browser/Stagehand based.The link tester includes bounded retries for transport errors and 5xx responses after a real Fetch request transiently returned 503 during verification.
Before/after semantic comparison
The exact pre-Fetch parent commit and this branch were both rerun with live credentials. All ten pre-Fetch language entrypoints completed successfully. The new implementations preserve the workflow outputs with these deliberate differences:
website-link-testerbusiness-lookupcompany-value-prop-generatorcouncil-events?Mode=2026to request the full-year HTMLsec-filing-researchE2E Test Matrix
All results below were produced from the latest branch with a real
BROWSERBASE_API_KEY; no historical runs were reused.website-link-tester:MAX_LINKS=5 pnpm startwebsite-link-tester:MAX_LINKS=5 uv run python main.pybusiness-lookup:pnpm startJalebi Streetrecord with ownership, account, location, address, and classification fieldsbusiness-lookup:uv run python main.pycompany-value-prop-generator:pnpm startYour...openercompany-value-prop-generator:uv run python main.pyYour...openercouncil-events:pnpm startCalendar.aspx?Mode=2026council-events:uv run python main.pyCalendar.aspx?Mode=2026sec-filing-research:pnpm start0000320193; 5 official recent filings returnedsec-filing-research:uv run python main.py0000320193; 5 official recent filings returnedStatic verification
pnpm checktsc --noEmitfor the changed TypeScript entrypointscompileallfor the changed Python entrypointsgit diff --checkPolicy audit
Note
Low Risk
Changes are limited to example templates and documentation, with simpler deterministic code paths and no auth or production service logic.
Overview
This PR replaces browser automation and outer-agent stacks with Browserbase Fetch API across five paired TypeScript/Python templates:
business-lookup,company-value-prop-generator,council-events,sec-filing-research, andwebsite-link-tester.Setup and docs now only require
BROWSERBASE_API_KEY(plus optional env likeBUSINESS_NAME,TARGET_URL,SEARCH_QUERY). Stagehand, Vercel AI SDK / Deep Agents, andAI_GATEWAY_API_KEYare removed where they were only used for HTTP retrieval. Dependencies shift to@browserbasehq/sdk/browserbase, with cheerio or BeautifulSoup for HTML link parsing in the link tester.Behavior changes by template: SF business lookup fetches the Open Data JSON endpoint and picks an exact DBA match in code; SEC research resolves CIK via
company_tickers.jsonand readssubmissions/CIK*.jsoninstead of EDGAR UI navigation; council events and value-prop flows use schema-constrained Fetch JSON instead ofact/extract; the link tester reports HTTP status, content type, and titles (with retries on 5xx) and drops AI “content matches link text” checks. The root README table descriptions are updated to match.Interactive Stagehand-based templates are unchanged by design.
Reviewed by Cursor Bugbot for commit 34194a2. Bugbot is set up for automated code reviews on this repo. Configure here.