Skip to content

emrg: evolution_prompt quick-ref — add #543 GUI message display fixes entry - #544

Merged
argszero merged 1 commit into
masterfrom
feature/quickref-543
Aug 7, 2026
Merged

emrg: evolution_prompt quick-ref — add #543 GUI message display fixes entry#544
argszero merged 1 commit into
masterfrom
feature/quickref-543

Conversation

@argszero

@argszero argszero commented Aug 7, 2026

Copy link
Copy Markdown
Owner

Follow-up to merged #543 (a06af27): adds the implemented-features quick-ref entry for the GUI message display fixes, following the established post-merge pattern (#537/#538/#542).

Entry content (verified against the merged diff):

  • Bug A: updateEmptyState was only called on session switch — Chat.append()/clear() now sync App.updateEmptyState?.()
  • Bug B: G122 16ms delta buffer vs pass-through done race — main.js flushDeltaBuf before terminal events (webContents.send order-preserving) + chat.js doneRids drops stale deltas (UUID never reused, 500 cap) + cancelled/stream-error clearTyping (daemon cancelled carries no request_id; mid-round cancel emits no done frame)
  • +3 regression tests, npm test 88→91 with docs synced

Single-line addition to evolution_prompt.md; no code or test changes.

@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 (independent verification on 9b302c5)

Single-line quick-ref entry checked point-by-point against merged master (a06af27):

  • Entry uniqueness: grep count = 1 (no duplicate)
  • Code anchors all present: flushDeltaBuf in main.js (3 refs), doneRids in chat.js (5 refs), 500-entry cap (1 ref), clearTyping wired in app.js (2 call sites: cancelled + stream-error), App.updateEmptyState?.() in chat.js (2 call sites: append + clear)
  • Behavioral claims accurate: terminal-event flush before done/error/cancelled, UUID-never-reused rationale, daemon cancelled carries no request_id / mid-round cancel emits no done frame
  • Doc counts consistent: README.md and Agent.md both say 91 (88 + 3 new tests)
  • Placement correct: directly after the #541 entry, chronological by PR number
  • CI 31155019475 green; no code or test changes in this PR

@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 (independent re-verification, different angle: format/mergeability)

Branch unchanged (9b302c5). This pass checked integration concerns rather than content (content verified in the previous cycle):

  1. Mergeability: GitHub reports MERGEABLE / CLEAN against current master (a06af27) — no conflict with the merged #543 quick-ref neighborhood.
  2. Format consistency: the entry follows the exact neighbor pattern (> - <title> (# ) ✅), placed chronologically after #541.
  3. No conflict markers anywhere in evolution_prompt.md.
  4. Template integrity: the {{ var }} inventory on the branch is byte-identical to master (owner/repo x28, source_dir x15, etc.) — the single-line addition touched no template variables, so prompt rendering is unaffected.
  5. CI 31155019475 remains green; docs-only change.

Second independent cycle approval; one more needed for the merge gate.

@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 3 (independent re-verification, third angle: rendering safety + immutability)

Final pass before the merge gate:

  1. Head SHA immutability: 9b302c5 unchanged since both prior cycle reviews — the approved content is exactly what will merge.
  2. Diff is exactly +1/-0 in evolution_prompt.md; GitHub reports MERGEABLE.
  3. Rendering safety: the template is consumed via Jinja2 (scheduler._build_evolution_prompt); the new line contains zero brace characters, so no {{ }}, {% %}, or {# #} delimiter can be introduced — prompt rendering is provably unaffected.
  4. Full suite on the branch: Python 508 passed, import OK; CI 31155019475 green.

Three independent cycle reviews complete (content anchors / format+mergeability / rendering safety) — merge gate satisfied.

@argszero
argszero merged commit b522165 into master Aug 7, 2026
1 check passed
@argszero
argszero deleted the feature/quickref-543 branch August 7, 2026 06:55
argszero added a commit that referenced this pull request Aug 7, 2026
…p tolerance (#552)

Version bump 0.2.10 → 0.2.11 across all 6 version sources
(pyproject.toml / emrg/__init__.py / gui/package.json / uv.lock /
make-installer.sh / build-runtime.sh). Release for Windows verification:

- #541 LLM gzip body tolerance
- #543 GUI message display fixes (#544 quick-ref)
- #545 Windows GCM silent-fail Stage 1
- #546 Windows TUI CJK input + legacy arrow keys + /rant visibility
- #548/#549/#550 GitHub auth in GUI (PAT + device flow + banner, Stage 2)
- #551 quick-ref

All 548 tests green.

Co-authored-by: EMRG Evolution <emrg@argszero.dev>
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