feat(framework): add create-notes skill, slim note-authoring content out of heartbeats#123
Draft
BobbyZhouZijian wants to merge 1 commit into
Draft
feat(framework): add create-notes skill, slim note-authoring content out of heartbeats#123BobbyZhouZijian wants to merge 1 commit into
BobbyZhouZijian wants to merge 1 commit into
Conversation
…out of heartbeats The 3 heartbeats that produce notes (reflect / consolidate / pivot) each duplicated substantial note-authoring guidance: section templates, the 'creator: frontmatter is required' rule, filename conventions, and the shell-escaping gotcha. The guidance drifted between files, ~135 lines of consolidated content was hard to keep in sync, and adding a new note variant (e.g. infra notes for grader issues) required editing 3 places. Centralize everything in coral/template/skills/create-notes/SKILL.md as 4 variants (A: experiment, B: infra, C: focus, D: synthesis / connections / open-questions) plus a self-audit checklist, the file-writing gotcha (bash backtick corruption), and a worked before/after example. The prompts are now slim triggers that point to the skill. Net: +461 lines in skill, -101 across the 3 prompts; one place to add a new note variant or update the self-audit. Trigger table (which heartbeat → which variant) lives at the top of the skill so the agent can jump to the right section without reading the whole file. Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Centralizes all note-authoring guidance in a single new skill (
coral/template/skills/create-notes/SKILL.md) and trims the duplicated content out of the 3 heartbeats that produce notes (reflect,consolidate,pivot). The prompts become slim triggers that point to the skill.Why
The 3 note-producing heartbeats each duplicated substantial guidance: section templates, the
creator:frontmatter rule, filename conventions, and the shell-escaping gotcha. The ~135 lines of consolidated content was hard to keep in sync — drift was already visible (e.g. consolidate.md listed one filename convention, reflect.md listed another). Adding a new note variant required editing 3 places.Centralizing fixes both: one place for the canonical content, prompts carry only the trigger + which skill section to load.
What's in the new skill
creator:matters: team-level processes (consolidate team-audit, librarian attribution, migration) silently skip notes without itpython3 -c "..."+ backtick-stripping bug that produced empty-skeleton notesThe 7-section experiment template was extracted from observed failures in
examples/vector_db_bench(island 1): notes had concrete numbers but no record of rejected alternatives, no cross-links, unsourced magic numbers (50 GB/s memory bandwidth appeared in two notes without citation), and quantitative predictions that were never backfilled.What changed in the prompts
coral/hub/prompts/reflect.mdcoral/hub/prompts/consolidate.mdcoral/hub/prompts/pivot.mdcoral/template/agents/librarian.mdmain(the duplicated "Frontmatter discipline" section only exists on the multi-island branch — out of scope here).coral/hub/prompts/lint_wiki.mdcoral/hub/prompts/warmstart_research.mdNet: +461 lines in skill, -101 across the 3 prompts.
Deferred (not in this PR)
descriptionfor triggering accuracy. The current description is already pushy ("Trigger this skill whenever you are about to Write a file under notes/ — even if the prompt didn't say 'write a note'"), but a proper run would let the team validate it.librarian.mdhas its own "Frontmatter discipline" duplication. Out of scope for this PR; will handle when multi-island merges.Test plan
reflectheartbeat, verify it loads the create-notes skill and writes a 7-section note with all 8 self-audit items satisfieduv run ruff check .(no Python changes, should pass clean)Co-Authored-By
🤖 Generated with Claude Code
Co-Authored-By: Claude Opus 4.8 noreply@anthropic.com