Credit Karma caching with variables example#49
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 3 potential issues.
❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.
Reviewed by Cursor Bugbot for commit 3f0cbaa. Configure here.
| browserbase_session_create_params={ | ||
| "project_id": os.getenv("BROWSERBASE_PROJECT_ID"), | ||
| }, | ||
| ) |
There was a problem hiding this comment.
Python example missing caching configuration entirely
High Severity
The StagehandConfig is missing a cache_dir parameter. The TypeScript counterpart correctly sets cacheDir: "credit-karma-cache", and the Python README explicitly advertises caching as a key feature ("10x faster subsequent runs", references credit-karma-cache/ directory). Without cache_dir, no local caching occurs, making this example fail to demonstrate its primary advertised feature.
Reviewed by Cursor Bugbot for commit 3f0cbaa. Configure here.
| await page.act( | ||
| f"in the mortgage calculator form, enter '{USER_CONFIG['cash_out']}' in the cash out amount field" | ||
| ) | ||
| print(f"Entered cash-out amount: ${USER_CONFIG['cash_out']}") |
There was a problem hiding this comment.
Python act() uses f-strings instead of variables
High Severity
All act() calls use Python f-strings to interpolate USER_CONFIG values directly into instruction strings instead of using the %variableName% syntax with a variables parameter. The TypeScript version correctly uses variables: { creditScore: USER_CONFIG.creditScore } with %creditScore% placeholders. The README explicitly documents the %variableName% syntax as a key feature. Without variables, cache keys change with every parameter value, defeating cache reuse across different inputs.
Reviewed by Cursor Bugbot for commit 3f0cbaa. Configure here.
| "dependencies": { | ||
| "@browserbasehq/stagehand": "^3.0.0", | ||
| "boxen": "^8.0.1", | ||
| "chalk": "^5.3.0", |
There was a problem hiding this comment.
Unused boxen and chalk dependencies in package.json
Low Severity
boxen and chalk are listed as dependencies but are never imported or used anywhere in index.ts. These are dead dependencies that add unnecessary install weight for users following the quickstart.
Reviewed by Cursor Bugbot for commit 3f0cbaa. Configure here.


Note
Low Risk
Adds new example templates and documentation without modifying shared library/runtime code; risk is limited to potential dependency/setup issues for template consumers.
Overview
Adds new Stagehand + Browserbase templates for mortgage-rate automation on Credit Karma, including caching and parameterized
act()variables (Python and TypeScript), plus structured rate extraction via Pydantic/Zod schemas.Also introduces a Python human-in-the-loop demo that pauses an automation based on configurable rules and resumes/aborts based on terminal approve/reject input, along with the required env/dependency files and setup docs.
Reviewed by Cursor Bugbot for commit 3f0cbaa. Bugbot is set up for automated code reviews on this repo. Configure here.