Skip to content

fix(l1): retry failed extraction outputs#263

Open
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/l1-filter-failure-retry
Open

fix(l1): retry failed extraction outputs#263
YOMXXX wants to merge 1 commit into
TencentCloud:mainfrom
YOMXXX:fix/l1-filter-failure-retry

Conversation

@YOMXXX

@YOMXXX YOMXXX commented Jun 26, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Treat non-empty, non-JSON L1 extraction responses (for example provider content-filter refusals) as extraction failures instead of empty successful results.
  • Propagate extractL1Memories().success === false through the standard L1 runner so the pipeline does not advance the L1 checkpoint cursor after failed extraction.
  • Add regression coverage for both the extractor classification and checkpoint-preservation path.

Why

When a provider returns a refusal/content-filter message, the current parser returns an empty extraction. The L1 runner then marks the L0 cursor complete, so the affected messages are skipped permanently and later L2 runs have nothing to process. This matches the silent extracted=0, stored=0 failure mode reported in #104.

Tests

  • npm test -- src/core/record/l1-extractor.test.ts src/utils/pipeline-factory.test.ts
  • npm test
  • npm run build

Closes #104

@Maxwell-Code07

Copy link
Copy Markdown
Collaborator

We have received multiple PRs submitted by you. Thank you for your consistent contributions. We will review them one by one and provide feedback accordingly.

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.

[Bug] 主代理触发平台风控后tdai卡在无法进行

2 participants