Skip to content

emrg: TUI status bar shows current version (rant 2026-08-13T14:11:03) - #736

Merged
argszero merged 1 commit into
masterfrom
feature/tui-status-version
Aug 13, 2026
Merged

emrg: TUI status bar shows current version (rant 2026-08-13T14:11:03)#736
argszero merged 1 commit into
masterfrom
feature/tui-status-version

Conversation

@argszero

Copy link
Copy Markdown
Owner

Add the current EMRG version (e.g. v0.2.30) as the leftmost segment of the TUI status bar, so the running version is always visible without invoking /version.

  • emrg/client/app.py: extract _format_status_left (module-level, unit-testable) that prepends v<__version__> to the session/model segments; nested _status_left delegates to it. Version imported inside the function to avoid module-level circular imports.
  • tests/test_app_status_left.py: 5 new tests (version prefix, title+short-sid+model layout, sid fallback, model omission, 8-char sid truncation).
  • Agent.md: pytest count 764 → 769 (doc-count guard emrg: sync test counts to 484 + guard test against doc drift (recurs #426/#430/#510) #511).

Host rant 2026-08-13T14:11:03 (TUI 状态栏左段增加当前版本号).

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle. CI test + test-windows both pass (run 31673268492); 769 pytest green locally; _format_status_left unit tests cover the 5 layout states; Agent.md doc-count synced.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle (2/3)

Verified locally on branch feature/tui-status-version (ea37059):

  • _format_status_left extracted module-level, adds v${__version__} prefix (0.2.30 → v0.2.30 main (s_260727) [deepseek-v4-flash]); _status_left closure delegates to it — no behavior regression.
  • All 5 new unit tests pass (version prefix, title/sid/model layout, no-title→sid, no-model→no brackets, 8-char sid truncation).
  • Full suite: 769 passed (12.89s); Agent.md count 764→769 consistent.
  • CI test + test-windows both PASS (run 31673268492). MERGEABLE, CLEAN.

@argszero argszero left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ LGTM — cycle (3rd consecutive ✅)

Re-verified this cycle: branch head ea37059 unchanged since prior reviews; diff identical (Agent.md 764→769, app.py _format_status_left extraction + version prefix, 5 new tests); full suite 769 passed locally (R1365); CI test + test-windows both PASS; mergeStateStatus CLEAN. Mergeable.

@argszero
argszero merged commit e419875 into master Aug 13, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant