Skip to content

fix(quick-add): scale automation timeouts - #38

Merged
milldr merged 3 commits into
mainfrom
tembo/fix-quick-add-timeouts
Sep 2, 2026
Merged

milldr merged 3 commits into
mainfrom
tembo/fix-quick-add-timeouts

Conversation

@milldr

@milldr milldr commented Sep 2, 2026

Copy link
Copy Markdown
Owner

Summary

  • scale quick-add automation timeouts with setup, macro count, and date navigation
  • give automatic login enough time to complete on slow Cronometer sessions
  • extend the remote browser lifetime to cover the longest supported quick-add flow
  • add regression coverage for single- and multi-macro timeout budgets

Why

Cronometer logs each quick-add macro as a separate food-dialog operation. The previous shared 60-second deadline could save an early macro and abort later ones, leaving a partial diary entry that was unsafe to retry. Slow login sessions could hit the same fixed deadline before the write began.

Verification

  • npm test — 229 tests passed
  • npm run build
  • npm run lint
  • npm run format:check
  • git diff --check

The patched local build was also exercised against a fat-only correction. It reached the login flow but Cronometer did not complete login before the expanded deadline; a read-only export confirmed no additional diary entry was created.


Want tembo to make any changes? Add a comment with @tembo and i'll get back to work!

View on Tembo  Review in Tembo  View Agent Settings

tembo Bot and others added 2 commits September 2, 2026 16:59
Co-authored-by: Dan <miller0daniel@gmail.com>
Co-authored-by: Dan <miller0daniel@gmail.com>
@tembo tembo Bot added tembo Pull request created by Tembo labels Sep 2, 2026
Co-authored-by: Dan <miller0daniel@gmail.com>
@milldr
milldr merged commit eeb6c6a into main Sep 2, 2026
3 checks passed
@milldr
milldr deleted the tembo/fix-quick-add-timeouts branch September 2, 2026 17:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tembo Pull request created by Tembo

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant