Journal destination: the notebook holds routed questions and observations (#161) - #166
Merged
Conversation
…ith their reports The Journal destination lands (docs/desk.md, D2; closes with PR against issue #161). /journal grows its notebook section at /journal/notebook: every route = journal Thread files in as an entry — the walker's own words, the full Enrichment report readable in place, links to the Thread and its published Artifact page. Un-routing at the desk removes the entry and nothing else; the Thread keeps its history. Draft-worthiness rides the Enrichment the way every other judgment does: a DRAFT header parsed beside TITLE/KIND, stored on the Enrichment (draft_worthy on the Neon row), and read back as the notebook's post-candidate flag — the queue the tweet/article pipeline pulls from, listed together at /journal/notebook?drafts=1. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WeJuNWLgJNWoi2DCGLg92T
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Session-start skill sync (skills-lock.json, mattpocock/skills) pulled a new upstream reference file beside the tracked SKILL.md. Committed separately from the #161 work so it reads as the vendored sync it is — drop this commit if the sync should land another way. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WeJuNWLgJNWoi2DCGLg92T
3 tasks
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
2 participants
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
The Journal destination lands (
docs/desk.md, D2). The/journalroute grows its notebook section at/journal/notebook: everyroute = journalThread files in as an entry — the walker's own words, the full Enrichment report readable in place (EnrichmentReport, with sources, mentions, and the research trace), and links to the Thread and its published Artifact page. Un-routing at the desk removes the entry and nothing else; the Thread keeps its Captures and Enrichments.Draft-worthiness rides the Enrichment the way every other judgment does: a
DRAFT:header parsed besideTITLE:/KIND:(lib/enrichment/system-instruction.ts), stored on the Enrichment (draft_worthyon the Neon row, plumbed through the fake and real gateway clients,process.ts, and both repositories), and read back as the notebook's post-candidate flag. Draft candidates can be listed together at/journal/notebook?drafts=1— the queue the tweet/article pipeline pulls from. Selection logic is pure (lib/journal/notebook.ts) so the same code serves the device's local Threads and the repositories under test. The Map Journal topbar links to the Notebook and back.One rider commit (
d525d56), separate from the ticket work: the session-start skill sync (skills-lock.json) pulled the ask-matt skill's new upstreamPHASE-BOUNDARIES.mdreference file beside its trackedSKILL.md. It's committed on its own so it reads as the vendored sync it is — drop that commit if the sync should land another way.Closes #161
Test plan
tests/journal-notebook.spec.ts(memory repositories): routing to Journal produces an entry with words + report; journal-only selection ordered newest-settled first;DRAFT:header round-trips end-to-end through the enrichment pipeline into the draft-candidate queue; entry carries its Thread and Artifact page; un-routing removes the entry while Captures and Enrichments stay whole.tests/journal-notebook-ui.spec.ts(public browser seam): route a Thread from the desk, read the entry at/journal/notebook— words, report, draft flag, Thread + Artifact links — list draft candidates together, then un-route and see the entry gone with the Thread's history untouched.pnpm exec tsc --noEmitclean; eslint clean on all changed files; new specs plus the neighboring thread-filing / enrichment / artifact / sync-hydrate suites green locally (136 passed).🤖 Generated with Claude Code
https://claude.ai/code/session_01WeJuNWLgJNWoi2DCGLg92T