fix(checkpoint): remove stray diff marker in RecalibrateSources inter…#393
Open
zzzbbbxxx123 wants to merge 1 commit into
Open
fix(checkpoint): remove stray diff marker in RecalibrateSources inter…#393zzzbbbxxx123 wants to merge 1 commit into
zzzbbbxxx123 wants to merge 1 commit into
Conversation
…face The stray '+' character at the start of line 154 (a leftover git diff marker) broke the RecalibrateSources interface definition, preventing vectorStore from being recognized as a valid property. This caused the recalibrate() method to fail type-checking and prevented the checkpoint recalibration logic from accessing the vector store as source of truth. Removing the '+' restores the interface to its intended definition, allowing the recalibrate() method (already implemented) to correctly recount L1 records and L0 conversations from actual on-disk data, fixing the counter drift issue after cleanup operations. Closes TencentCloud#157 Signed-off-by: zzz <zzz@users.noreply.github.com> Signed-off-by: zzz <zzz_h7@foxmail.com>
Collaborator
|
Thank you for submitting this PR and participating in Tencent Rhino-bird Open-source Training Program! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description | 描述
修复
src/utils/checkpoint.ts中RecalibrateSources接口的语法错误：第 154 行残留了一个 git diff 标记符+，导致vectorStore属性无法被 TypeScript 识别。这使得已实现的recalibrate()方法在类型检查时报错，无法正确从 VectorStore 读取 L1 记录数作为校准基准。修改内容：
+残留字符，恢复接口正确定义checkpoint.recalibrate()实现启动时自动校准修复后，
recalibrate()在 Gateway 启动时自动执行，从实际 JSONL 数据/VectorStore 重新统计 L1 记录数和 L0 对话轮次，修正total_memories_extracted、l0_conversations_count和memories_since_last_persona的漂移值，确保状态上报和 Persona 触发阈值基于真实数据。Related Issue | 关联 Issue
Fix #157
Change Type | 修改类型
Self-test Checklist | 自测清单
Additional Notes | 其他说明
全部 72 个测试用例通过（含新增 5 个 checkpoint 测试）：
Test Files 5 passed (5)
Tests 72 passed (72)