emrg: evolution_prompt quick-ref — add #558 entry - #560
Merged
Conversation
argszero
commented
Aug 7, 2026
argszero
left a comment
Owner
Author
There was a problem hiding this comment.
✅ LGTM — cycle 20260807-224842 (1st angle: content accuracy + placement)
Verified on head bb32bd7:
- +1 entry line after the #556 line, before section 2.2 — chronological order correct.
- Content cross-checked against merged 5846f7d:
_evolution_count()present in daemon.py (definition + 3 call sites: pong evolution_count, evolution_summary count ×2),total_evolutions()in scheduler.py (aggregates handler evolutions, legacy fallback when scheduler unavailable),test_task_scheduler_total_evolutionsin test_scheduler.py — all anchors grep-verified. - Legacy-bug framing accurate (#95 pre-scheduler design; self.evolutions never appended); count/recent consistency note correct (recent reads disk logs, count reads scheduler).
- 0 conflict markers, 0 Jinja2 delimiters; direct render test passes with the entry surviving substitution.
- No test files touched → doc-count guard unaffected; 573 pass locally; CI green (31189315683).
argszero
commented
Aug 7, 2026
argszero
left a comment
Owner
Author
There was a problem hiding this comment.
✅ LGTM — cycle 20260807-225252 (2nd angle: merge-readiness + format consistency)
Verified on head bb32bd7 (unchanged since 1st review):
- Mergeability: MERGEABLE + CLEAN after #559 merged into master (quick-ref PR touches only evolution_prompt.md — no overlap with #559's scheduler/tests/doc-count files, so no conflict).
- Format consistency: single entry matches the established quick-ref pattern (
> - <Feature> (#NN ...) ✅), chronologically placed after #556, before section 2.2 — consistent with neighboring entries. - 0 conflict markers, 0 Jinja2; direct render verified previously; CI green (31189315683).
- No test files touched → doc-count guard unaffected (master at 575 post-#559, this PR changes none).
Ready for 1 more vote.
argszero
commented
Aug 7, 2026
argszero
left a comment
Owner
Author
There was a problem hiding this comment.
✅ LGTM — cycle 20260807-225703 (3rd angle: final content audit vs master)
Verified on head bb32bd7 (unchanged, CLEAN):
- diff vs master: exactly +2 lines (entry + blank), single file evolution_prompt.md.
- Content anchors re-verified against current master (88e1b9b, incl. #558/#559):
_evolution_count×4 in daemon.py,total_evolutions×2 in scheduler.py — the #558 entry's claims match the merged code exactly. - No drift from the #559 merge (which touched scheduler.py/tests/docs — no overlap with this prompt-only change).
- 0 conflict markers, 0 Jinja2; CI green (31189315683).
3 consecutive ✅ from different cycles (224842, 225252, 225703) with no ❌ between → ready to merge.
argszero
added a commit
that referenced
this pull request
Aug 7, 2026
…unt fixes (#553-#561) (#562) Version bump 0.2.11 → 0.2.12 across all 7 version sources (pyproject.toml / emrg/__init__.py / gui/package.json / gui/package-lock.json / uv.lock / build-runtime.sh / make-installer.sh). Release for rant 发布新版本 (2026-08-07T23:54:46) — ships 9 commits accumulated since v0.2.11: - #553 Windows TUI Unicode input via ReadConsoleInputW - #554 GUI interleaved text/tool message order - #556 rant UX (daemon-authoritative timestamp + GUI textarea + UTF-8 log) - #558 evolution count always 0 fix - #559 exclude aborted evolution cycles from count and idle-halt backoff - #557/#560/#561 quick-ref entries All 575 tests green.
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.
Add quick-ref entry for merged #558 (evolution count aggregation fix, external contribution from pm25coder).