Repository navigation
Make the Flow use its space and stop burning a core - #212
Merged
madskronborg merged 8 commits intoOct 26, 2026
Merged
madskronborg merged 8 commits into
madskronborg merged 8 commits into
Conversation
Nuxt Icon compiles only the names it finds in the view layer's .vue files, and flow-actions.ts built its names from a fragment, so a Step's Watch, Note, Log, Evidence and Answer actions drew an empty button. The names are now written whole and the scan also reads desktop/src/shared, which hands the view icons by name elsewhere too (a Machine out of reach, one without Collie). Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Desktop's CEF composites in software, so a glyph rotating every frame repaints at 60 fps: with a record open on a Task with two running Steps, the renderer and GPU process held about 40% and 30% of a core. Twelve steps a turn keeps the motion and costs about 15% and 3%. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
An open record rebuilds its Task's Flow on every write under the state directory, and three quarters of that build was reading every agent's delivery ledger, which only the Reopened check of a finished Run uses. On vm-mk's state a running Task's Flow drops from ~200 ms to ~45 ms. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The map was a capped strip that scrolled sideways: at 1000 px two Steps per Run showed, the current one off screen, ticket names wrapped onto four lines and the sticky lane label covered the first glyph, while half the record sat empty under a one-line hint. Each Run's Steps now wrap in a grid under a one-line header, names are shortened with the whole name kept in the hover and the spoken name, and a wide record shows the inspector beside the map, keeping the selected Step in view. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The inspector printed raw instants ("Started 2026-10-26T10:22:22.731Z")
and put "Started by" at the very bottom of the record, far from the Step
it describes. Times now read "11:22", or "24 Oct, 11:22" on another day,
and the link sits with the Step's facts.
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
A Run that straddled the upgrade to Steps recorded launches that name no Step, and its Flow drew each one beside the replayed Step of the same id. Two nodes with one id break the map's keys and its arrow-key navigation, and the Step itself showed no agent. Such a launch is now read as the entered Step whose key is its operation. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The map is capped under the inspector in a narrow record, and a window resized after a Step was selected left that Step scrolled out of the map. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
At a board column's width the branch took two lines, the Flow strip
cut its label short ("5 min, us…") and the pane's place took two more,
repeating the Machine the footer names and ending on the tab that tells
panes apart. The branch and the pane now take one line each, with the
whole of them on hover, the pane leads with its tab, the strip's label
moves under its glyphs rather than being cut, and the card's padding is
tighter: our own card went from 334 to 231 px tall.
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.
The Flow used its space poorly and was slow to use, so this polishes it after hands-on use in an isolated dev Desktop against the laptop's and vm-mk's real hosts. No control was pressed on a real Run.
taskFlowOfwent from ~200 ms to ~43 ms (median of 7).The before and after screenshots and timings are on mk-laptop: the measurements are in
~/.local/state/herdr/plugins/cego.collie/runs/run-3df1ab2e/handoffs/flow-polish-measurements.mdand the screenshots inhandoffs/screenshots/. The host-side gains (ledger reads, replayed Steps) reach a Machine only once its host runs this release.Verifications: the gate's
checkpassed on2cb95eb5, the commit before this head. On this head (a2c8ad99),card-testsanddesktop-typecheckpassed. Every other recorded verification ran on an earlier tree.Verification this run is held to:
/home/mk/.local/share/mise/installs/bun/latest/bin/bun run check(in/home/mk/.herdr/worktrees/.collie/madskronborg/test-and-polish-the-new-flow-feature-in-3u9a28)The board card commit (
a2c8ad99) came after the review and after the gate, so its own tests checked it, not a reviewer. It changed:BoardCard.vue: tighter padding; the branch and pane are each cut to one line, with a title; the Flow strip wraps.paneLineindesktop/src/shared/terminal-input.tsandusePanes.ts: the pane line reads tab · workspace.test/desktop-card-pane.test.ts, which failed first.docs/using.md.Non-blocking finding left open:
test/desktop-icon-bundle.test.ts:31(minor), "Record a failing baseline for the new icon-bundle test". The record already holds one:icon-bundleran with--expect failone53cd719, before the fix, and failed (exit 1). It names the five missing icons (message-square-plus, terminal, message-circle-question, scroll-text, file-check).Not settled by the run
🤖 Generated with Claude Code