Skip to content

Credit Karma caching with variables example#49

Open
sophie2chance2 wants to merge 4 commits into
devfrom
sc/credit-karma
Open

Credit Karma caching with variables example#49
sophie2chance2 wants to merge 4 commits into
devfrom
sc/credit-karma

Conversation

@sophie2chance2
Copy link
Copy Markdown
Member

@sophie2chance2 sophie2chance2 commented Jan 30, 2026

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.

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 3 potential issues.

Fix All in Cursor

❌ 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"),
},
)
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

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']}")
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 3f0cbaa. Configure here.

"dependencies": {
"@browserbasehq/stagehand": "^3.0.0",
"boxen": "^8.0.1",
"chalk": "^5.3.0",
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

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.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 3f0cbaa. Configure here.

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.

2 participants