Skip to content

feat(agent): verify complete_step steps against todos#2569

Merged
esengine merged 1 commit into
esengine:main-v2from
GTC2080:GTC/todo-step-receipts
Jun 1, 2026
Merged

feat(agent): verify complete_step steps against todos#2569
esengine merged 1 commit into
esengine:main-v2from
GTC2080:GTC/todo-step-receipts

Conversation

@GTC2080

@GTC2080 GTC2080 commented Jun 1, 2026

Copy link
Copy Markdown
Contributor

Related to #2537

Summary

  • Extend the runtime-only evidence ledger to capture successful todo_write receipts for the current turn.
  • Require complete_step.step to match the latest successful todo list when one exists in the same turn.
  • Accept matches by todo content, activeForm, or 1-based item number; reject pending or missing todo items.
  • Preserve existing behavior when there is no successful todo_write receipt in the current turn.

Conflict check

  • Scoped to runtime receipt validation and focused tests.
  • No UI changes.
  • No performance claims.
  • No prompt or tool schema changes.
  • Avoids auto-plan, multi-agent, goal, cache, MCP, and agent hook work areas.

Review evidence

  • No UI changes; screenshots not applicable.
  • No performance claims; cache hit rate, token consumption, and runtime metrics are not claimed for this correctness change.
  • No prompt or tool schema changes.

Verification

  • D:\Go\go\bin\go.exe test ./internal/evidence ./internal/tool/builtin ./internal/agent: PASS
  • D:\Go\go\bin\go.exe test ./internal/cli -run TestRunStatuslineCmd -count=1 -v: PASS
  • D:\Go\go\bin\go.exe test ./internal/...: PASS
  • git diff --check: PASS

Notes

  • The local legacy npm git hook still expects package.json on this Go branch. The commit was made with --no-verify after the Go validation above passed.

@github-actions github-actions Bot added the v2 Go rewrite (1.x) — main-v2 branch, active development label Jun 1, 2026
@esengine esengine merged commit 22d5f00 into esengine:main-v2 Jun 1, 2026
3 checks passed
@GTC2080 GTC2080 deleted the GTC/todo-step-receipts branch June 1, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

v2 Go rewrite (1.x) — main-v2 branch, active development

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants