emrg: open_source_prompt requires full-code study before contributing (rant 2026-08-14T15:53:39)

[argszero]

Summary

Host rant 2026-08-14T15:53:39 (emrg): the open-source task prompt must require the contributor to:

1. Read the complete codebase — not just the target file
2. Always re-read the latest code before each contribution (no relying on memory/stale versions; 0.3 Source sync guarantees a fresh pull)
3. Understand the design intent from the repository author's perspective — why the code is designed this way (design rationale, git history/blame, tests as documentation)
4. Only then consider contributing — the contribution must align with the existing design, not reinvent it

Previously B.2 Study project conventions only read CONTRIBUTING.md / PR template / .github listing — it never required reading the full source, so contributors could jump to implementation without understanding the codebase.

Changes

• emrg/server/open_source_prompt.md: new B.2b Read the full codebase (MUST before contributing) section inserted between B.2 and B.3 — full-code read requirements, author-perspective design-intent analysis, and the "understand the design before contributing" gate
• tests/test_scheduler.py: +1 regression test test_open_source_template_full_code_study_b2b asserting all four requirements render (section presence / full-code read / latest-code re-read / author-perspective / contribute-only-after-understanding)
• Agent.md: pytest count 805 → 806 (doc-count guard)

Verification

• uv run pytest tests/ -v: 806 passed (805 + 1 new)
• Import check + CLI help: green

[argszero]

✅ LGTM — cycle 703

Verified:

• New B.2b section covers all 4 rant requirements: full-code read / always re-read latest code / author-perspective design intent / contribute only after understanding
• Regression test test_open_source_template_full_code_study_b2b passes (positive discrimination: asserts section + all 4 requirement phrases render)
• Local: uv run pytest tests/ -v 806 passed; import + CLI checks green
• CI: test + test-windows both SUCCESS (run 31781964559)

[argszero]

✅ LGTM — cycle

Verified independently:

• B.2b section inserted between B.2 and B.3 covers all 4 rant requirements: full-code read / always re-read latest code / author-perspective design intent / contribute only after understanding
• Regression test test_open_source_template_full_code_study_b2b passes locally (1/1)
• Agent.md pytest count 805 → 806 synced (doc-count guard)

[argszero]

✅ LGTM — cycle 704

Independent re-verification:

• Diff clean: 3 files 47+/1- (open_source_prompt.md B.2b section, test_scheduler.py regression test, Agent.md count 805→806)
• B.2b covers all 4 rant requirements: full-code read / always re-read latest / author-perspective design intent / contribute only after understanding
• Test test_open_source_template_full_code_study_b2b passes locally; pytest 806 all green; import + CLI green
• CI test + test-windows both SUCCESS — 3 consecutive ✅ (cycles 703, parallel, 704)

[argszero]

✅ LGTM — cycle

Third consecutive approval (different cycles: 703, R1712, R1713). Re-verified fresh this cycle:

• PR head 65ba0ae unchanged, mergeable
• uv run pytest tests/test_scheduler.py::test_open_source_template_full_code_study_b2b → 1 passed
• No ❌ in history; merge condition satisfied