Conversation
… configs topic_limit_floor had zero runtime consumers — no pruning, matching, or maintenance logic ever read it. Its only effect was a silent hazard: readConfig rewrote any file with floor != limit to floor := limit on EVERY load, clobbering explicit user values; and the schema advertised a knob the runtime never honored. - drop the field from MemorySchema.Config and updateConfig - delete normalizeConfig and the readConfig rewrite branch; decodeConfig now simply decodes (legacy files carrying the field stay valid — Schema Struct decoding tolerates the extra property, dropped on the next write) - defaults (selectConfiguration), admission candidates, and test fixtures no longer carry the field Tests (red-first, test/memory/config.test.ts): legacy divergent-floor file decodes with the field absent; decoded/updated configs never contain it; schema bounds preserved. Memory suite 93/0; lint 4829 -> 4828.
fix(memory): remove dead topic_limit_floor knob that silently rewrote configs
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
1 participant
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.
合并 dev 到 main：
#291 fix(memory): remove dead topic_limit_floor knob that silently rewrote configs
背景：memory.jsonc/dag.jsonc 自动生成逻辑核对（dag 侧正确无需改动）。请 main 全量门禁复核。