Repository navigation
Flow journal: MP-88, MP-89, MP-93, MP-90, MP-92, MP-91, MP-94 - #83
Merged
Merged
Conversation
…rom the do reason
The project-note rule appeared in howto twice and again in the project role, and the task role repeated howto's line about what the engine journals. The copies cost about 210 characters in every warmup and pushed real project and global warmups past the 9500-character hook budget. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A task warmup kept its card whole and, once journal entries and memory facts were gone, printed above the limit; a long card pushed the Claude hook past 10,000 characters. The card's end is now cut last, with a line naming the context_read call that reads it in full. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The organization chat does a project chat's work for its projects but never reads the project role, so it had no rule for the project-scope note. The rule now sits in both roles, and PIPELINE.md names these two chats; howto stays without it, since every role, the main chat included, pays for it in the warmup budget. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
ARCHITECTURE.md still listed the task card as never cut. A clip inside a fenced block left it open, so the clip line and the sections after it read as code; the clip now closes it. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Counting fence lines missed ~~~ blocks, longer fences and indented code; the clip now closes the block with its own opener, found by the same rule demote uses. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Flow
journal: the implementation subtasks of the epic MP-84 (journal research, https://draft.yokeloop.com/artifacts/mp-84-mypi-wvsfnkmf), merged one at a time intoflow/journalfromorigin/maine1596c2. Assembled branch: build, cargo (30), fast (187) and boundary (65) tests green.project remove)<status> (unchanged). <reason>note)docs/PIPELINE.md«Journal», skills, prompts, AGENTS pointer)## Журналsections, storestats/last)mp-task boardreason/reasonAt)Engineer's answers taken on the way:
clone_preparedwritten inproject-checkout.tsinside the registration transaction for cloned/initialized/adopted; per-type scope table (session_opened/message_sent in task, project, org; clone_prepared project only; task facts task only).(unchanged)./dostage-1in_progressreason is the plan link in both modes (worktree path dropped).web/src/shared/lib/journal.tswith a deepEqual test againstEVENT_TYPES; web colours per type (indigo/violet/grape, teal/yellow).message_sentrows stay visible in the task warmup as journal facts (README/AGENTS wording adjusted).status_changedor the(unchanged)note, one project-wide journal read; shown only forblocked.Lead's edits of plans: none (all plans as written by plan-writer; line references re-resolved by re-scouts after MP-88 and MP-89).
Deviations on record: MP-88 re-exports
EVENT_TYPESthroughsrc/app/commands.ts(dependency-cruiser forbids cli/http/mcp → module public.ts); MP-91 changed one boundary assertion intest/boundary/cli.test.ts(a parent message now appears in the task warmup as amessage_sentfact line; inbox assertions unchanged).Operator steps after merge and engine update:
mypi db init(schema 15); reload Claude Code / Pi sessions.Live acceptance (2026-11-07)
Disposable stand on f569f21, rechecked on 88e8479. Claude Code 2.1.283 (claude-haiku-4-5-20251001), Pi 1.0.4 (openai-codex/gpt-6-sol), herdr 0.9.5, node v24.21.0, schema 15; 14 model turns.
(unchanged)notes, milestone text = journal text), HTTP?type=filter, MCPjournal/task_update, live/open(session_opened started/focused/resumed, Claude and Pi),clone_preparedon/openof a project without a clone,/boardblocked reason from the journal, web filter and badge colours, TUI filter cycle and colours,mp-task boardon the installed engine (read-only), warmup timing (107 → 71 ms), skill and doc texts.warmup --max-chars 9500gave MP 9888, global 9763, MP-91 10002, MP-84 10363 characters. The fix is on the MP-91 branch and merged in 88e8479: the chats' note rule appears once (project and organization roles), and as its last budget step a task warmup clips the end of its card, with acontext_readpointer and an open fence closed. Review round 2 approved. After the fix: MP 9467, global 9459, tasks 9495–9496./planrework and/do(they publish to derive); covered by the engine checks and skill texts.🤖 Generated with Claude Code