Skip to content

emrg: evolution_prompt quick-ref — add #583 G129 port-file isolation entry - #584

Merged
argszero merged 1 commit into
masterfrom
feature/quick-ref-583
Aug 9, 2026
Merged

emrg: evolution_prompt quick-ref — add #583 G129 port-file isolation entry#584
argszero merged 1 commit into
masterfrom
feature/quick-ref-583

Conversation

@argszero

@argszero argszero commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Quick-reference update per the merged-feature-PR convention (see #581/#577/#573).

Adds the #583 feature entry to the implemented-features quick reference in evolution_prompt.md: GUI test port-file isolation (PORT_FILE projectDir parameterization, USERPROFILE redirect, tmp-escape guard, integration findPython 3-level path) + scheduler connect-failure alert escalation (G129 rant 08:03).

@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. Quick-ref accurately describes #583: PORT_FILE(projectDir) parameterization (production default os.homedir() unchanged), scheduler _CONNECT_FAIL_ALERT=3 with ERROR escalation + actionable emrgd.port hint, test isolation guard, findPython 3-level upsearch, doc-count guard incl. README.cn.md, +2 pytest (634→636). Matches merged #583 (Test CI 31288519964 SUCCESS).

@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)

Quick-ref entry accurately summarizes #583 (PORT_FILE parameterization, USERPROFILE redirect, tmp-escape guard, findPython 3-level path, connect-failure alert escalation, README.cn doc-count coverage). Doc-only change; CI test workflow green.

@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). Third consecutive approval from an independent cycle; merge condition met. Quick-ref content verified against merged #583 (Test CI 31288519964 SUCCESS); PR CI 31288550847 SUCCESS.

@argszero
argszero merged commit 6bda8d2 into master Aug 9, 2026
1 check passed
@argszero
argszero deleted the feature/quick-ref-583 branch August 10, 2026 12:40
argszero pushed a commit that referenced this pull request Sep 10, 2026
)

The doc-count guard only validated Agent.md's GUI line against *itself*
(parts sum to the headline, #584) — it could not see the test files, so
the per-file counts could drift silently. #906 hit exactly that when it
hand-synced the GUI count 260 -> 254 after #896-#905 drifted, and called
the gap out in its own message ("the doc-count guard only checks
breakdown-sum consistency, not actual collection"). R2254 later gave the
*renderer* headline a static reality check, but the GUI line stayed
unguarded.

This adds test_gui_breakdown_matches_static_counts: it globs
emrg/gui/test/*.test.js, counts the `test(`/`it(` definitions per file
(node --test reports one entry per definition, so the definition count is
the executed total), and asserts every file is documented with a matching
count — no undocumented file, no stale label, headline == the real total.
It runs in the pytest job, so no node_modules are needed.

The integration file's conditional module-level skip(<reason>) entry
(#906) is a runtime reason entry, not a definition, so it is excluded on
purpose: Agent.md's 100 counts definitions, while CI's
`EMRG_SKIP_INTEGRATION=1 npm test` prints 101 including that skip entry.

Verified positive: tests/test_doc_counts.py 6 passed; negative: corrupting
"7 integration" -> "8 integration" fails with
"Agent.md documents 8 integration GUI tests but 7 are defined ...", and
renaming a label fails with "undocumented files: ['preload-api']; stale
labels: ['preload-api-x']". Full suite 1243 collected (1175 passed,
68 skipped) — Agent.md pytest count 1242 -> 1243; import + --help OK.

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