fix: steady omp state, real omp stats, and an edge between stacked panes - #78
Merged
Conversation
Three fixes from using the board with an oh-my-pi session: - omp animates its spinner in an unterminated tail and can go a whole turn without completing a line. The idle timeout that reverts a stalled session to READY is fed only by complete lines, so a working session looked idle after 2s and the next spinner frame flipped it back to THINKING — visible flapping for the length of the turn. A changed tail now counts as output; an unchanged one (the reader re-sends it once a second) still does not, so a session whose screen has genuinely stopped moving idles out as before. - omp had no stats watcher, only screen scraping. It records per-turn usage in ~/.omp/agent/sessions, and /new rolls a new transcript file there — which is what wiped a session's token tracking mid-run. omp_log.rs tails that transcript, follows the roll, and carries the previous file's totals forward. Context follows /new down, being a snapshot of the live prompt rather than a total. - Vertically split panes butted one pane's last row of output against the next pane's title with nothing in between. Each pane's title row now doubles as its top border, with a rule from the end of the title to the right margin, costing no rows of output. Co-Authored-By: Claude <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
1 participant
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.
Three fixes from using the board with an oh-my-pi session:
omp animates its spinner in an unterminated tail and can go a whole turn without completing a line. The idle timeout that reverts a stalled session to READY is fed only by complete lines, so a working session looked idle after 2s and the next spinner frame flipped it back to THINKING — visible flapping for the length of the turn. A changed tail now counts as output; an unchanged one (the reader re-sends it once a second) still does not, so a session whose screen has genuinely stopped moving idles out as before.
omp had no stats watcher, only screen scraping. It records per-turn usage in ~/.omp/agent/sessions, and /new rolls a new transcript file there — which is what wiped a session's token tracking mid-run. omp_log.rs tails that transcript, follows the roll, and carries the previous file's totals forward. Context follows /new down, being a snapshot of the live prompt rather than a total.
Vertically split panes butted one pane's last row of output against the next pane's title with nothing in between. Each pane's title row now doubles as its top border, with a rule from the end of the title to the right margin, costing no rows of output.