Skip to content

Lessons: keep one with an owner's note that refines it - #114

Merged
hhff merged 3 commits into
mainfrom
lesson-note
Nov 6, 2026
Merged

hhff merged 3 commits into
mainfrom
lesson-note

Conversation

@hhff

@hhff hhff commented Nov 5, 2026

Copy link
Copy Markdown
Member

Built on #107, which has the Keep-it loading state; the diff shrinks once #107 merges.

What

  • dashboard/index.html: a lesson card has a one-line note box ("Add a note to refine it"). Keep it sends the note with the answer. The note survives redraws like other drafts.
  • scripts/devteam.mjs:
    • knowledge set <k> --note "…" --owner: only the owner may add a note, as one line of at most 600 characters.
    • The dashboard's knowledge action passes the note.
    • knowledgeBlock renders … The owner adds: <note>, reading the note only from the owner's own log lines (ownerNote).
  • schema/log.schema.json: knowledge.owner_note, which only the owner may set (x-authority.set.knowledge.owner).
  • test/kit.test.mjs: a supervisor's --note is refused; a hand-written supervisor line carrying owner_note doesn't reach the prompt; the owner's note does.

Why

The owner wanted to keep a lesson but refine it slightly without rewriting the supervisor's statement.

Authority

The note goes into every agent prompt the lesson applies to, so it's the owner's alone: the CLI refuses anyone else, the schema allows only the owner to set the field, and prompts ignore any non-owner line carrying it.

Migration

None. Restart the dashboard after the sync.

🤖 Generated with Claude Code

hhff and others added 3 commits November 5, 2026 10:23
…e owner's note reaches the prompt

A lesson to keep could only be kept or retired as the supervisor wrote it. The card now has a note
box: Keep it saves the note with the answer, and every prompt shows the lesson with "The owner
adds: …". knowledge set --note is the owner's only (schema: owner_note is set by the owner only),
and prompts read the note from the owner's own lines, so a supervisor's line carrying one counts for
nothing.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@hhff

hhff commented Nov 6, 2026

Copy link
Copy Markdown
Member Author

Merging for v0.20.1 (upstream loop).

@hhff
hhff merged commit 60b4cc4 into main Nov 6, 2026
4 checks passed
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.

1 participant