Skip to content

fix(heartbeat): ignore untouched default template#1617

Merged
afjcjsbx merged 1 commit intosipeed:mainfrom
yzxlr:codex/fix-1561-heartbeat-template-idle
Mar 21, 2026
Merged

fix(heartbeat): ignore untouched default template#1617
afjcjsbx merged 1 commit intosipeed:mainfrom
yzxlr:codex/fix-1561-heartbeat-template-idle

Conversation

@yzxlr
Copy link
Copy Markdown
Contributor

@yzxlr yzxlr commented Mar 15, 2026

Summary

  • treat the generated HEARTBEAT.md template as idle until the user adds tasks below the marker
  • stop sending the example/instructions block to the LLM as a real heartbeat prompt
  • add regression tests for the untouched-template and user-task cases

Closes #1561

Testing

  • go test ./pkg/heartbeat ./pkg/config

@CLAassistant
Copy link
Copy Markdown

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

@afjcjsbx
Copy link
Copy Markdown
Collaborator

LGTM

@sipeed-bot sipeed-bot bot added type: bug Something isn't working go Pull requests that update go code labels Mar 16, 2026
@afjcjsbx afjcjsbx merged commit c0bb8d6 into sipeed:main Mar 21, 2026
3 of 4 checks passed
@sipeed-bot
Copy link
Copy Markdown

sipeed-bot bot commented Mar 25, 2026

@yzxlr Smart fix! Treating the generated HEARTBEAT.md template as idle until the user actually adds tasks makes a lot of sense. No more sending boilerplate instructions to the LLM as if they were real prompts.

We are running a PicoClaw Dev Group on Discord for contributors. If you want to join, send an email to [email protected] with the subject [Join PicoClaw Dev Group] + Your GitHub account and we will send you the invite!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

go Pull requests that update go code type: bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] 心跳消耗token问题

3 participants