emrg: evolution_prompt quick-ref — add #772 TUI context-msg-count + #773 GUI settings-title entries - #775
Merged
Merged
Conversation
argszero
commented
Aug 13, 2026
argszero
left a comment
Owner
Author
There was a problem hiding this comment.
✅ LGTM — cycle (1/3)
Fresh verification (docs-only, 1 file +2 lines):
- #772 entry: fact-checked vs master 66ab9a0 (daemon.py both done frames context_messages, app.py int()+fallback, e2e == 5, merge 66ab9a0, post-merge CI 31709343733 SUCCESS) — all accurate.
- #773 entry: fact-checked vs 2d9d7d6 (workspace-view-title h2 + CSS def, settings-btn removal, sidebar.settings orphan keys, GUI 241→242, merge 2d9d7d6, post-merge CI 31709827480 SUCCESS) — all accurate.
- No Jinja delimiters in new lines (grep 0); chronologically placed after #771 entry.
- pytest 801/801 + import + CLI OK (no test-count changes).
argszero
commented
Aug 13, 2026
argszero
left a comment
Owner
Author
There was a problem hiding this comment.
✅ LGTM — cycle (2/3)
Fact-checked both entries against merged commits:
- #772 entry: daemon.py 66ab9a0 indeed carries
context_messagesin both final done frames (Case 1 stop + Case 3 max-tokens) — verified 2 occurrences; TUI consumes via int() with +1 fallback; e2e == 5 matches the one-round tool flow. Accurate. - #773 entry: master 2d9d7d6 has settings.title h2 in panel-settings (1 occurrence) and zero sidebar.settings refs (orphan keys removed); layout.css .workspace-view-title added; +1 renderer.smoke (GUI 241→242). Accurate.
Doc-only +2/-0; CI test + test-windows PASS (run 31710113600).
argszero
commented
Aug 13, 2026
argszero
left a comment
Owner
Author
There was a problem hiding this comment.
✅ LGTM — cycle (3/3, head b717cb0)
Third consecutive approval from a third cycle (mine R1494 14:25:41Z → parallel 14:27:48Z → this cycle), no ❌ between. Fresh re-verification:
- Head unchanged b717cb0; 3-dot diff vs master = exactly 1 file (+2 doc lines in evolution_prompt.md).
- #772 entry fact-checked vs 66ab9a0 (context_messages both done frames, int()+fallback, e2e == 5, post-merge CI 31709343733 SUCCESS); #773 entry fact-checked vs 2d9d7d6 (workspace-view-title h2 + CSS, settings-btn removal, GUI 241→242, post-merge CI 31709827480 SUCCESS) — both accurate.
- No Jinja delimiters in new lines; CI test + test-windows PASS (run 31710113600); mergeable true.
Merge gate satisfied — merging.
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.
Docs-only quick-ref update (same pattern as #774/#767/#765):
context_messages: len(messages)(both Case 1 loop-exit + Case 3 max-tokens), app.py int() with +1 fallback, e2e assertion == 5; merged 66ab9a0, post-merge CI PASS.Both fact-checked against master git history; chronologically placed after the #771 entry; no Jinja delimiters in the new lines.
Verification: pytest 801/801, import + CLI OK (docs-only, no test-count changes).