Skip to content

Fix local stability issues in startup, reports, chat, and prompts#366

Open
veil-chow-fyaic wants to merge 4 commits into
volcengine:mainfrom
veil-chow-fyaic:fix/local-stability-bugs
Open

Fix local stability issues in startup, reports, chat, and prompts#366
veil-chow-fyaic wants to merge 4 commits into
volcengine:mainfrom
veil-chow-fyaic:fix/local-stability-bugs

Conversation

@veil-chow-fyaic

Copy link
Copy Markdown

Summary

  • reuse an already healthy backend instead of treating port 1733 as a fatal startup error, and avoid stopping a backend this Electron process did not spawn
  • make daily reports deterministic by using the report start date, normalizing fenced Markdown, and updating an existing daily report for that date
  • isolate chat agents per request to avoid shared event queue/session state across conversations
  • harden prompt persistence and prompt lookup by saving user prompts under the writable user settings directory and falling back from screenshot_analyze to the legacy screenshot_contextual_batch prompt
  • coerce LLM-returned extracted fields into stable shapes so list/dict summaries do not fail ExtractedData validation
  • reduce noisy development update checks and improve notification content for generated tips

Related issues

Refs #365, #350, #342, #338, #318, #313, #253, #246, #231, #221, #174, #166.

Tests

  • /Users/fuyo-aic/Projects/MineContext-fork-fixes/.venv/bin/python -m unittest discover -s tests -v
  • python3 -m py_compile opencontext/context_consumption/generation/generation_report.py
  • pnpm exec tsc --noEmit -p tsconfig.node.json --composite false
  • pnpm exec tsc --noEmit -p tsconfig.web.json --composite false

Notes

This PR keeps the fixes focused on stability and backward-compatible fallbacks. It does not attempt to solve platform-specific capture limitations such as minimized-window capture on Windows.

@codeCraft-Ritik codeCraft-Ritik left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Solid maintenance-focused PR. There are several independent fixes here, but they all seem aligned around improving reliability and reducing unexpected behavior. The daily report determinism changes are particularly nice for consistency and debugging.

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