Skip to content

fix(record): harden L1 JSON parsing and embedding timeout#250

Open
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/l1-json-parser-embedding-timeout
Open

fix(record): harden L1 JSON parsing and embedding timeout#250
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/l1-json-parser-embedding-timeout

Conversation

@YOMXXX

@YOMXXX YOMXXX commented Jun 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • parse the first valid L1 scene payload instead of greedily matching from the first [ to the last ]
  • tolerate non-JSON bracketed text and object-wrapped scene arrays with trailing text in model output
  • pass embeddingTimeoutMs through L1 memory writes so slow CPU embedding providers can use the configured capture-path timeout
  • add focused regression tests for L1 extraction parsing and vector dual-write embedding timeout propagation

Verification

  • npm test
  • npm run build

Fixes #110
Related: #181

Signed-off-by: 李冠辰 <liguanchen@xiaomi.com>
@Maxwell-Code07

Copy link
Copy Markdown
Collaborator

@YOMXXX 4 PRs in one day tackling L1 extraction robustness — JSON parsing tolerance, dedup hardening, failure diagnostics persistence, and auto-retry. Great coverage! We'll review them one by one 🚀

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.

L1 JSON parse failure + background embedding timeout abort — memory pipeline stalls

2 participants