test: cover spec §17 conformance rows not owned by another sub-issue

[moncher-dev]

Epic: #651
리포트 항목: 리포트 §4.4 — docs/reports/2026-08-28-upstream-spec-drift-research.md (PR #650)

범위

다른 하위 이슈가 자연스럽게 덮지 않는 §17 행만 모은다. 코드 변경이 필요하면 해당 이슈로 넘긴다.

• §17.3 listIssuesByStates([]) / fetchIssueStatesByIds([]) → provider 호출 없음 (GitHub, Linear, file 각각 fetch spy)
• §17.3 ID refresh에서 malformed 요청 레코드 → 실패(GitHub requireProjectItemState, Linear)
• §17.2 워크스페이스 경로에 일반 파일이 있을 때 안전 처리(fail or replace 정책 명시)
• §17.2 after_create가 재사용 디렉터리에서 재실행되지 않음 (S3와 중복 시 그쪽으로)
• §17.4 running 이슈 없을 때 reconcile no-op
• §17.5 stderr가 프로토콜 스트림과 분리됨; 미지원 dynamic tool 거부(S21 결정 후)
• §17.7/§13.7 start.ts port/host/bind 동작 (start.test.ts에 port|host|bind 0건)
• docs/architecture.md의 "Test matrix" 슬라이스 또는 core-conformance.test.ts 헤더에 §17 행 ↔ 테스트 매핑 주석

완료 기준

• 위 체크리스트가 pnpm test에 포함되고 AGENT_TEST.md 시나리오 표에 필요한 E2E가 등록됨

[hojinzs]

Backlog → Ready (프로젝트 상태 관리) — 블로커 없음, In review 2(<5), WIP 캡(8) 내 픽업. 테스트에서 고정 포트(4680 등)·실시간 타이머 의존 금지(#758 참조).

[moncher-dev]

Workpad — #678 — Cycle 1

Type: fresh pickup
Branch: feat/678-spec17-conformance-tests
Draft PR: #765
Cycle opened: 2026-08-29T10:10:18Z · Trigger: Ready pickup for scoped §17 conformance coverage

Status Transitions

• 2026-08-29T10:36:00Z · In progress → In review · all scoped §17 conformance coverage and validation gates passed (cycle 1 close)

Plan

• 1. Prove empty listIssuesByStates([]) and fetchIssueStatesByIds([]) avoid provider calls for GitHub, Linear, and file adapters.
• 2. Map the remaining in-scope §17 rows to existing implementations and add only missing focused tests.
• 3. Add the §17-to-test mapping and required E2E scenario documentation.
• 4. Wrap-up: re-verify the original issue · pass the Completion Bar · changeset (if needed) · PR ready · transition to In review

Rework / PR Feedback

N/A — fresh pickup.

Completion Bar

• In-scope requirements implemented
• pnpm lint
• pnpm test
• pnpm typecheck
• pnpm build
• Docker E2E (if integration behavior changed)
• Tests for new functionality (or justified N/A)
• Code conventions (CLAUDE.md)
• Inline review comments answered (rework only; N/A)

Validation

• pnpm lint — pass.
• pnpm test — pass.
• pnpm typecheck — pass.
• pnpm build — pass.
• Changeset: N/A (test/documentation-only change; no release label)
• Docker E2E evidence: ./e2e/run-e2e.sh happy 60 — pass (worker dispatched, retry observed, final health idle).

Delegation (out-of-scope / human / post-merge)

• None

Progress Log

• 2026-08-29T10:10:18Z: Classified scope as Execution/Observability conformance coverage with Coordination/Integration test boundaries; Cycle 1 opened.

Blockers

None

Transition Intent

• Exact request body prepared: Ready → In progress; reason: scoped §17 conformance work is triaged and the feature branch is published.

• Exact request body prepared: In progress → In review; reason: all scoped §17 conformance coverage and validation gates passed; Cycle: 1 close.

• 2026-08-29T10:10:46Z: Orchestrator confirmed Ready → In progress; beginning focused §17 test mapping.

• 2026-08-29T10:13:20Z: Added and pushed test(trackers): cover empty state lookups (8bf5c0c6); Draft PR test: cover remaining §17 conformance rows #765 created. Focused tracker suites passed: file 21, Linear 40, GitHub 130 tests.

• 2026-08-29T10:23:42Z: Added and pushed test(trackers): reject malformed Linear refresh (de2b9443); pnpm --filter @gh-symphony/tracker-linear test -- tracker-linear.test.ts passed (41 tests). The final §17 audit remains open for workspace-file and no-running-reconciliation rows.

• 2026-08-29T10:29:00Z: Added and pushed test(orchestrator): cover safe workspace reconciliation (11461ffa) for the no-active-run reconciliation no-op and safe rejection of a regular file at the workspace path; pnpm --filter @gh-symphony/orchestrator test -- service.test.ts passed.

• 2026-08-29T10:32:00Z: Added and pushed docs(testing): map section 17 coverage (615a3588). docs/architecture.md now maps each scoped §17 row to its authoritative test, and AGENT_TEST.md records the corresponding Docker lifecycle and dynamic-tool confirmations.

• 2026-08-29T10:35:00Z: Re-verified every issue checklist row against the focused adapter, orchestrator, worker, CLI, and documentation tests. Full lint, test, typecheck, build, and Docker E2E passed. Handoff intent: mark PR test: cover remaining §17 conformance rows #765 ready and transition In progress → In review.

• 2026-08-29T10:35:00Z: Refreshed PR test: cover remaining §17 conformance rows #765 with the final conformance mapping, validation evidence, risks, and human-validation checklist.

[moncher-dev]

🔁 Status: Ready → In progress

Reason: scoped §17 conformance work is triaged and the feature branch is published
Cycle: 1 open

[moncher-dev]

🔁 Status: In progress → In review

Reason: all scoped §17 conformance coverage and validation gates passed
Cycle: 1 close