Applies reviewer suggestion from PR #10071 review #7500 (comment #249234):
replace chained .parent calls with a named PROJECT_ROOT constant for
improved readability and maintainability.
ISSUES CLOSED: #9824
Bug #4198 (PlanContextInheritance prioritises fragments near the child focus) is NOT yet fixed. The previous commit incorrectly removed the @tdd_expected_fail tag, causing the unit_tests CI gate to fail with "Expected 2 skeleton fragments, got 1".
Re-add @tdd_expected_fail to the scenario so CI correctly inverts the failing assertion back to a pass, per the TDD bug-fix workflow.
ISSUES CLOSED: #9824
Collapse multi-line assert into single line to satisfy ruff format check.
The CI lint job runs both ruff check and ruff format --check; the format
check was failing because the assert statement used unnecessary parentheses
across three lines.
ISSUES CLOSED: #9824
Workers were systematically omitting CHANGELOG.md, CONTRIBUTORS.md, and
commit footer (ISSUES CLOSED: #N), causing all PRs to be blocked from merge.
Added a mandatory 8-item PR Compliance Checklist to the worker prompt body
in implementation-supervisor.md that supervisors must pass to every worker.
The checklist covers:
1. CHANGELOG.md update under [Unreleased]
2. CONTRIBUTORS.md update
3. Commit footer with ISSUES CLOSED: #N
4. CI verification (all quality gates green)
5. BDD/Behave test coverage
6. Epic reference in PR description
7. Labels applied via forgejo-label-manager
8. Milestone assignment
Also removed @tdd_expected_fail tag from PlanContextInheritance test in
depth_breadth_projection.feature (bug #4198 is fixed).
Added BDD tests in features/pr_compliance_checklist.feature with 10 scenarios
covering all 8 checklist items.
ISSUES CLOSED: #9824
The PR branch was stale (behind master). Fast-forwarded to master HEAD
to trigger a fresh CI run. All PR changes were already merged into master.
ISSUES CLOSED: #9824
HAL9000