Skip to content

Sonny: memory overrides the role files' defaults, and saves what worked - #410

Merged
lydakis merged 1 commit into
mainfrom
claude/project-thread-bhz8kn
Oct 30, 2026
Merged

lydakis merged 1 commit into
mainfrom
claude/project-thread-bhz8kn

Conversation

@lydakis

@lydakis lydakis commented Oct 30, 2026 •

Copy link
Copy Markdown
Owner

Requested by George · project thread

This is the first half of the memory-first prompt audit (write-up). George's principle: what agents can learn into memory beats behavior written into role files and skills. He approved all three recommendations. This PR changes prompt text only and ships in 0.7.0.

What changes

Not in this PR

PR B comes after #403 merges and the coach thread finishes its two-day eval of #403, so each eval measures one change:

  • Reactions reach the nightly said, each with the reply it answered. Today lookout::said_by_person drops them.
  • The habit step also saves approaches the person confirmed, and patterns of ask picks that go against the recommendation.
  • Each feedback fact gets confirm and push-back counts, shown in the index line (ExpeL / ACE style).

Checks

  • cargo test -p sonny on main 1ef4fb7: 516 passed. a_task_is_told_who_it_works_for now expects task.md's new last sentence.
  • cargo fmt --check and cargo clippy -p sonny --all-targets -D warnings pass.
  • No daemon, protocol or schema change.

🤖 Generated with Claude Code

https://claude.ai/code/session_013jkdGGVwTdQTtcHeKUqRa6

@lydakis lydakis self-assigned this Oct 30, 2026
@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Oct 30, 2026 •

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review ✅ Completed 2026-10-30T04:22:44.485213Z 9ab64d4 New commits
🔒 Security Review ✅ Completed 2026-10-30T04:26:15.348662Z 9ab64d4 New commits
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

lydakis pushed a commit that referenced this pull request Oct 30, 2026
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013jkdGGVwTdQTtcHeKUqRa6
Memory beats codified behavior: lead.md and task.md now say that what
the person's memory says about how they want work done overrides their
defaults, and their latest words override both. The scattered "save X to
memory" sentences in lead.md fold into one paragraph, which also asks to
save what the person welcomed, not only what they corrected. The memory
skill and tool say the same: memory that holds only corrections steers
agents away from what already works; save the approach, never that the
person liked being agreed with.

Pinned picks are left to the model and to memory: "Needs you:" as a
literal reply label, "at most three" open decisions, the react tool's
emoji meanings and "React rarely", and "draw it in Mermaid". Drops the
stale "Triggers need Sonny on a Mac" (Linux since #344).

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_013jkdGGVwTdQTtcHeKUqRa6
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