Skip to content

Fix: missing autoRecallMinRepeated config parsing (follow-up to #40)#52

Closed
kctony wants to merge 1 commit intowin4r:mainfrom
kctony:fix/autorecall-minrepeated
Closed

Fix: missing autoRecallMinRepeated config parsing (follow-up to #40)#52
kctony wants to merge 1 commit intowin4r:mainfrom
kctony:fix/autorecall-minrepeated

Conversation

@kctony
Copy link
Contributor

@kctony kctony commented Mar 4, 2026

問題說明

PR #40 新增了 autoRecall 去重機制,但漏掉了兩個關鍵設定:

  1. PluginConfig 介面缺少 autoRecallMinRepeated? 定義
  2. parsePluginConfig 函數缺少 autoRecallMinRepeated 解析邏輯

導致 openclaw.json 中的 autoRecallMinRepeated 配置無法正確讀入,功能失效。

修正內容

  • 第 42 行:在 PluginConfig 介面新增 autoRecallMinRepeated?: number;
  • 第 991 行:在 parsePluginConfig 新增解析邏輯 autoRecallMinRepeated: parsePositiveInt(cfg.autoRecallMinRepeated),

影響範圍

  • 不影響現有功能
  • 僅補上配置解析邏輯,讓去重機制能正常運作

@rwmjhb
Copy link
Collaborator

rwmjhb commented Mar 5, 2026

Hi @kctony, thanks for the follow-up fix for #40!

Just a heads-up — this change (adding autoRecallMinRepeated to PluginConfig + parsePluginConfig) is already included in PR #50 (2nd commit, submitted at nearly the same time).

Merging both PRs as-is will cause a conflict on index.ts. Could you decide which PR should carry this fix?

Let me know which way you'd like to go!

@rwmjhb
Copy link
Collaborator

rwmjhb commented Mar 6, 2026

感謝你的修復!這個改動已經包含在 #50 的第二個 commit 中,所以這裡就不重複合併了。再次感謝 🙏

@rwmjhb rwmjhb closed this Mar 6, 2026
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