Skip to content

fix: simplify notifications and extend runtime timeout - #515

Merged
Pigbibi merged 2 commits into
mainfrom
fix/concise-notifications-timeout-20260926
Sep 26, 2026
Merged

Pigbibi merged 2 commits into
mainfrom
fix/concise-notifications-timeout-20260926

Conversation

@Pigbibi

@Pigbibi Pigbibi commented Sep 26, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • keep runtime notifications focused on total equity, nonzero holdings, supplemental content, and trade or cycle result
  • add a shared compact-section adapter with explicit localized supplemental lines
  • raise Cloud Run request timeout to 600 seconds and Gunicorn timeout to 570 seconds so the service worker stays below the platform deadline

Verification

  • 109 tests and 11 subtests passed
  • runtime workflow shell tests passed
  • Ruff passed
  • git diff --check passed

Production note

The current live revision still uses the 300-second timeout. Deployment and a later natural scheduled cycle are required to verify recovery.

Pigbibi and others added 2 commits September 26, 2026 22:48
Co-Authored-By: Codex <noreply@openai.com>
Co-Authored-By: Codex <noreply@openai.com>
@Pigbibi
Pigbibi merged commit 5630850 into main Sep 26, 2026
1 check passed
@Pigbibi
Pigbibi deleted the fix/concise-notifications-timeout-20260926 branch September 26, 2026 14:56
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.

1 participant