msg watch: hint at --tui in the normal-mode banner#140
Merged
Conversation
The line-streaming watcher now prints a one-line hint inside the LIVE banner suggesting the richer full-screen dashboard. Shown only when --tui wasn't passed and stdout is an interactive TTY, so scripted/piped/--plain/ --once paths are unaffected.
Collaborator
Author
🪙 100% AI-authored · 4 files
💬 The ask
📍 What shipped
Tip ✅ Security scan cleanNo credentials leaked in this branch's diff. h5i scanned every added line against the secret rule pack and found nothing to rotate. Note Duplicate-code scan clean — no copy-paste blocks introduced. 🧠 Reasoning by file — 5871 nodes across 5 lanes, latest 24 only%%{init: {"flowchart": {"nodeSpacing": 42, "rankSpacing": 48, "diagramPadding": 14}, "themeVariables": {"fontSize": "18px"}} }%%
flowchart TB
subgraph lane_0["💭 Reasoning · 4 steps"]
direction LR
n_lane_0_0["THINK · The h5i context snapshot is linked to the commit, but h5i refs aren't pu…"]:::t
n_lane_0_1["THINK · I'll add the hint as a row inside the banner box (so it stays aligned), …"]:::t
n_lane_0_2["THINK · Builds clean. Let me confirm the hint fits the banner width (so the box …"]:::t
n_lane_0_3["THINK · Let me confirm the two remaining things: that ”Cautions” (secret/dup row…"]:::t
n_lane_0_0 --> n_lane_0_1
n_lane_0_1 --> n_lane_0_2
n_lane_0_2 --> n_lane_0_3
end
subgraph lane_1["📄 src/pr.rs · 8 ops"]
direction LR
n_lane_1_0["READ × 4"]:::o
n_lane_1_1["EDIT × 4"]:::a
n_lane_1_0 --> n_lane_1_1
end
subgraph lane_2["📄 src/main.rs · 7 ops"]
direction LR
n_lane_2_0["READ × 3"]:::o
n_lane_2_1["EDIT"]:::a
n_lane_2_2["READ × 3"]:::o
n_lane_2_0 --> n_lane_2_1
n_lane_2_1 --> n_lane_2_2
end
subgraph lane_3["📄 src/repository.rs · 4 ops"]
direction LR
n_lane_3_0["READ"]:::o
n_lane_3_1["EDIT"]:::a
n_lane_3_2["READ"]:::o
n_lane_3_3["EDIT"]:::a
n_lane_3_0 --> n_lane_3_1
n_lane_3_1 --> n_lane_3_2
n_lane_3_2 --> n_lane_3_3
end
subgraph lane_4["📄 src/ctx.rs · 1 op"]
direction LR
n_lane_4_0["READ"]:::o
end
classDef o fill:#dbeafe,stroke:#1e3a8a,color:#0b1c4a;
classDef t fill:#fef3c7,stroke:#92400e,color:#3f2d05;
classDef a fill:#dcfce7,stroke:#166534,color:#0a2e16;
classDef n fill:#ede9fe,stroke:#5b21b6,color:#221251;
classDef m fill:#e5e7eb,stroke:#374151,color:#0b0f17;
📜 Per-commit provenance
|
h5i share pr post built every section (Cautions, per-commit, cost/tokens, 'What shipped') from h5i_log(limit) — the last 25 commits reachable from HEAD with no base scoping — so commits already on the base branch leaked into the PR. Two fixes: - Add H5iRepository::h5i_log_since(base, limit): a base..HEAD revwalk (push_head + hide(base)). pr.rs now resolves the PR base (gh baseRefName, else origin/HEAD default, else main/master), merge-bases it with HEAD, and renders only the branch's own commits. Degrades to the full walk when no base resolves. - 'What shipped' derived from the global main.md roadmap (identical on every branch). Source it from the branch's own commit subjects instead. Regression test: h5i_log_since_scopes_to_base_exclusive.
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 line-streaming watcher now prints a one-line hint inside the LIVE banner suggesting the richer full-screen dashboard. Shown only when --tui wasn't passed and stdout is an interactive TTY, so scripted/piped/--plain/ --once paths are unaffected.