Update deep research agent template dashboard#90
Conversation
|
bugbot run verbose=true |
|
Bugbot request id: serverGenReqId_99a1e2b6-44c5-4517-8468-61436b1da0dc |
fba2b10 to
c7da4fe
Compare
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 7f339a5. Configure here.
| } catch (error) { | ||
| notice.innerHTML = `<div class="notice">${escapeHtml(error.message || String(error))}</div>`; | ||
| setStatus("Error", "error"); | ||
| setPipeline("failed", Math.max(0, (loadingStepIndex - 1) % pipelineSteps.length)); |
There was a problem hiding this comment.
Failed pipeline step highlights wrong stage after cycling
Low Severity
The loadingStepIndex increments every 3.6 seconds and wraps via % pipelineSteps.length (6). After ~21.6 seconds the index cycles past all steps. On failure, the expression (loadingStepIndex - 1) % pipelineSteps.length wraps to a low value — e.g., when loadingStepIndex is 7, the result is 0, marking "Planning" as failed instead of the later stage where the run actually was. Since the estimated run time is 240 seconds, most real failures would highlight the wrong pipeline step. Using Math.min(pipelineSteps.length - 1, loadingStepIndex - 1) instead of the modulo would cap at the last step for long runs.
Additional Locations (1)
Reviewed by Cursor Bugbot for commit 7f339a5. Configure here.
Co-authored-by: shriyatheunicorn <shriavj@gmail.com>
There was a problem hiding this comment.
This file is quite long and would be very hard for a beginner to follow or understand what's going on. Is there any way we can split this up into separate files?


Summary
file://opens pointed at local dev API candidates instead of an external deploymentindex.tsVercel function config so the staticpublic/UI plus/api/researchdeploy cleanly in the Browserbase Vercel projectVerification
npm run checknode scripts/check-readme-template-index.mjs/,/api/health, and/browserbase-logo.svgNote
Medium Risk
Large new template adds a full research pipeline plus serverless/local HTTP endpoints that call external Browserbase/Stagehand services, so failures or misconfiguration can affect runtime cost and reliability. Changes are mostly additive and isolated to the new
typescript/deep-research-agentdirectory plus a README index entry.Overview
Adds a new
typescript/deep-research-agenttemplate that runs an iterative search → fetch → browser fallback → synthesis → verification research loop, persisting run workspaces (plan.md,strategy.md, traces, reports) and returning a structured JSON/Markdown brief with source IDs, claim maps, and verifier output.Includes a static dashboard UI (
public/index.html+ logo) with local server support (src/local-web.ts) and Vercel serverless handlers (api/health.ts,api/research.ts,vercel.json) that expose/api/researchand health checks, plus a benchmark mode that loads TSV/JSON/JSONL tasks and writes aggregate results.Updates the root
README.mdtemplate table to listdeep-research-agent, and adds template-level project setup (package.json,.env.example,.gitignore) and response utilities insrc/api-utils.ts.Reviewed by Cursor Bugbot for commit 7f339a5. Bugbot is set up for automated code reviews on this repo. Configure here.