docs(configuration): consolidate the four disagreeing sections into one child-boundary section

[moncher-dev]

Phase 5 of epic #847. 승인된 계획과 선행 조건 변경은 #847의 2026-09-05 코멘트 둘에 있습니다.

무엇을 하는가

docs/configuration.md에서 같은 경계를 서로 다르게 설명하는 절들을 자식 경계 한 절로 통합합니다.

왜

"명시적이지 않고 조용히 누락된다"는 문제의 문서 쪽 절반입니다. 이 문서는 사람만 읽는 것이 아니라 워커가 읽고 작업합니다. 같은 경계에 대해 네 곳이 다른 말을 하면 워커가 어느 것을 믿을지가 운에 달립니다.

대상 — docs/configuration.md 683행

현재 절	위치	처리
Environment Loading Order	:373	통합
Credential Brokers And Git Access	:434	P4‑2c 이후 삭제
Agent Runtime Credentials	:459	통합
Custom Runtime Environment Contract	:487	통합
Worker Context Variables	:568	통합
Agent-visible Symphony context	:603	통합
Hook Variables	:647	통합

#871(PR #886)이 :397-414에 정확한 자식 경계 표를 이미 추가했습니다. 그것이 통합의 씨앗입니다. 이 이슈는 그 위에 나머지를 접고 낡은 것을 지웁니다.

선행 조건이 P4‑6이 아닌 이유

원래 계획은 P4‑6(doctor 축소) 뒤였는데 바꿨습니다. doctor 축소는 이 문서와 아무 관련이 없고, "문서는 맨 뒤"라는 임의 직렬화였습니다. 실제 의존은 둘입니다.

• test(e2e): assert the agent-child environment contract as a black-box check #872 — Phase 3 종료. 자식 경계가 블랙박스로 검증되면 더 움직이지 않습니다. 완료됨.
• refactor(runtimes): remove the agent provider credential broker and residual plumbing #904 (P4‑2c) — 마지막 브로커 제거. :434 절이 지워질 대상으로 확정됩니다.

그래서 이 이슈는 #904만 기다립니다.

완료 조건

• 자식 경계가 한 절에서 한 번만 설명된다. 위 표의 절들이 그 절로 접히거나 삭제된다.
• 남은 서술이 머지된 코드와 일치한다. 특히 packages/core/src/runtime/agent-child-env.ts, custom-child-env.ts, 그리고 #872가 남긴 블랙박스 스펙이 실제로 단언하는 내용과 맞아야 한다.
• :442의 GITHUB_TOKEN_BROKER_SECRET 행이 이미 #879에서 고쳐졌다. 브로커가 사라진 뒤의 사실에 맞게 다시 정리한다.
• 문서 내부 앵커 링크가 깨지지 않는다. #agent-visible-symphony-context 등 다른 문서가 참조하는 앵커를 확인한다.
• 코드 변경 없음. changeset 불필요.

범위 밖

docs/architecture.md(P5‑1), 그리고 문서가 기술하는 동작 자체의 변경. 이 이슈는 기술만 합니다.

[moncher-dev]

Workpad — #910 — Cycle 1

Type: fresh pickup
Branch: symphony/hojinzs-github-symphony-910
Draft PR: #926
Cycle opened: 2026-09-05T21:50:28Z · Trigger: Ready issue with prerequisite #904 completed by merged PR #924

Status Transitions

• 2026-09-05T21:51:16Z · Ready → In progress · Scope is actionable and prerequisite refactor(runtimes): remove the agent provider credential broker and residual plumbing #904 is merged; documentation consolidation is starting (cycle 1 open)

Plan

• 1. Consolidate the child environment boundary in docs/configuration.md, preserve referenced anchors, validate formatting and links, and commit the documentation slice
• 2. Publish via /push and open Draft PR via /gh-pr-writeup
• 3. Wrap-up: re-verify the original issue · pass the docs-only Completion Bar · changeset (if needed) · PR ready · transition to In review

Rework / PR Feedback

Not applicable for this fresh cycle.

Completion Bar

• In-scope requirements implemented
• pnpm lint
• pnpm test (N/A under docs-only scope-proportional validation)
• pnpm typecheck (N/A under docs-only scope-proportional validation)
• pnpm build (N/A under docs-only scope-proportional validation)
• Docker E2E (N/A; no integration behavior changed)
• Tests for new functionality (N/A; documentation only)
• Code conventions (CLAUDE.md)
• Review threads answered and resolved (N/A; fresh cycle)

Validation

• pnpm format — fail: repository-wide pre-existing formatting drift in 69 files, including the changed document before targeted formatting
• pnpm exec prettier --write docs/configuration.md && pnpm exec prettier --check docs/configuration.md — pass; isolated rerun formatted and verified the only changed file
• pnpm lint — pass
• git diff --check — pass
• Anchor reference audit with rg — pass; explicit #environment-loading-order and #agent-visible-symphony-context anchors preserve all inbound references
• Original-issue coverage audit — pass; all listed sections are folded under one top-level boundary, broker configuration rows are removed, and wording matches agent-child-env.ts, custom-child-env.ts, their unit tests, and upstream SPEC §15.3
• Changeset: N/A (documentation-only change without a release label)
• Docker E2E evidence: N/A (no integration behavior changes)
• Merge commit (Land cycle only): N/A

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

• None

Progress Log

• 2026-09-05T21:50:28Z: Confirmed the assigned issue is Ready, found no prior workpad or linked PR, and verified prerequisite refactor(runtimes): remove the agent provider credential broker and residual plumbing #904 merged via PR refactor(runtimes): remove agent credential broker #924.
• 2026-09-05T21:52:58Z: Consolidated the environment descriptions under one worker/agent-child boundary, removed retired broker configuration rows, preserved inbound anchors, and committed a4eceec.
• 2026-09-05T21:53:58Z: Published the assigned branch, opened Draft PR docs(configuration): consolidate child boundary #926, re-verified every issue requirement and upstream-spec conformance, and completed the docs-only validation bar. No repository-local divergence was introduced; the existing custom-runtime divergence remains explicitly documented.
• 2026-09-05T21:53:58Z: Handoff prepared because PR docs(configuration): consolidate child boundary #926 describes published HEAD a4eceec, all applicable checks pass, and no changeset is required.

Intended close transition:

🔁 Status: In progress → In review

Reason: PR #926 is ready; the documentation requirements and docs-only Completion Bar are satisfied
Cycle: 1 close

Blockers

None

[moncher-dev]

🔁 Status: Ready → In progress

Reason: Scope is actionable and prerequisite #904 is merged; documentation consolidation is starting
Cycle: 1 open

[moncher-dev]

🔁 Status: In progress → In review

Reason: PR #926 is ready; the documentation requirements and docs-only Completion Bar are satisfied
Cycle: 1 close

[moncher-dev]

재작업 라운드 1 — PR #926, 차단은 P2 둘뿐입니다

hojinzs가 a4eceec에서 CHANGES_REQUESTED를 남겼습니다. 전문과 인라인은 PR #926에 있습니다.

다시 검증하지 말 것

리뷰어가 문서를 읽고 정합성을 판단하는 대신 자식 환경 빌더를 실제로 실행해서 대조했습니다. 살아남은 서술 대부분이 맞습니다 — custom 기본 자식의 portable 변수 집합, GIT_TERMINAL_PROMPT=0, "Codex와 Claude만 private DOCKER_CONFIG를 받는다", 트래커 토큰 미도달, agent-visible 허용목록 9개가 AGENT_VISIBLE_SYMPHONY_CONTEXT_ENVIRONMENT_NAMES와 정확히 일치하는 것까지 전부 실행으로 확인됐습니다.

앵커 감사도 끝났습니다. 이 파일로 들어오는 링크가 저장소 전체에 셋뿐이고 전부 보존됩니다. #environment-loading-order는 제목이 바뀌었지만 :393의 명시적 <a id>로 해결됐고, #auth-and-api-endpoints는 제목 문자열이 그대로이며, #agent-visible-symphony-context는 :634의 명시적 앵커로 살아 있습니다. 앵커가 바뀐 나머지 넷은 참조하는 곳이 없습니다.

GITHUB_TOKEN_BROKER_SECRET 정리도 옳습니다. 세 행을 지우고 denylist 사실 문장으로 대체했는데, 코드상 GITHUB_TOKEN_BROKER_URL/CACHE_PATH는 AGENT_CHILD_CREDENTIAL_ENVIRONMENT_NAMES에만, ..._SECRET은 CUSTOM_RUNTIME_RESERVED_AUTH_ENVIRONMENT_NAMES에만 살아 있으므로 "not configuration inputs"가 동작상 정확합니다.

과설계도 임의 결정도 없다고 판정됐습니다. Title Case 통일은 파일의 기존 관례이고 <a id>는 앵커 보존의 최소 수단입니다.

P2 하나 — LC_* 서술이 Codex에만 참입니다 (차단)

:413의 "Portable runtime variables" 행에 이렇게 쓰여 있습니다.

LC_* locale variables are also preserved.

Codex에서만 참입니다. runtime-codex/src/runtime.ts:561이 SAFE_RUNTIME_ENV_KEYS.has(entry[0]) || entry[0].startsWith("LC_")로 통과시키지만, Claude와 custom 경로에서는 실제로 탈락합니다. 리뷰어가 빌더를 실행해 LC_ALL이 제거되는 것을 확인했습니다.

이 절이 "이 경계에 대한 단일 진실"을 자처한다는 점이 핵심입니다. 그 안에 남은 유일한 사실 오류이므로 통합 PR에서 함께 고쳐야 합니다.

P2 둘 — 무관한 두 절이 환경 경계 아래로 들어갔습니다 (차단)

## Environment Loading Order를 ## Worker And Agent-Child Environment Boundary로 개명하고 뒤따르는 ##들을 ###으로 강등하면서, 이 ## 절이 :395부터 :695까지 늘어났습니다. 그 결과 GitHub Tracker Transition Extension(:659)과 Workspace-Key Migration(:669)이 환경 경계 절의 하위 항목이 됐습니다. 둘 다 환경 변수와 무관합니다 — 하나는 GitHub Project 상태 쓰기 소유권(§11.5 divergence)이고 하나는 워크스페이스 디렉터리 명명 규칙입니다.

P3와 nit — 판단에 맡깁니다

• :521 — custom 절이 strip 목록을 세 번째로 재서술합니다. :416의 "Credentials and host configuration" 행과 같은 사실을 다른 낱말로 반복합니다. 지금은 코드와 일치하지만, 중복 서술이 어긋나기 시작하는 게 이 이슈가 존재하는 이유입니다.
• :494 — "they are not part of the general child environment allowlist"가 CODEX_HOME에 대해서는 문자 그대로 틀립니다. runtime.ts:132의 SAFE_RUNTIME_ENV_KEYS에 들어 있어 allowlist를 통과한 뒤 워크스페이스 내부 경로로 덮어써집니다. 같은 표의 각주가 이미 정확하니 총괄 문장만 좁히면 됩니다.

리뷰어가 명시했습니다 — 미반영 사유를 스레드에 남기면 그대로 수용하겠다고요.

범위 밖 후속

packages/runtime-codex/src/launcher.ts:41-43이 브로커 변수 셋을 여전히 읽어 runtime.ts:75-77 필드에 넣습니다. 소비자가 테스트 픽스처뿐이라 동작은 없지만, 이 문서가 "설정 입력이 아니다"라고 단언한 이상 죽은 읽기는 정리 대상입니다. 별도 이슈로 다룹니다. 이 PR에서 건드리지 마세요.

[moncher-dev]

Workpad — #910 — Cycle 2

Type: rework cycle (PR #926)
Branch: symphony/hojinzs-github-symphony-910
Draft PR: #926
Cycle opened: 2026-09-05T22:49:56Z · Trigger: Human review returned PR #926 with requested documentation corrections

Status Transitions

• 2026-09-05T22:51:00Z · Ready → In progress · PR docs(configuration): consolidate child boundary #926 has requested documentation corrections that require a rework cycle (cycle 2 open)

Plan

• 1. Correct the consolidated child-boundary section for runtime-specific locale behavior, compatibility mode, section hierarchy, and remaining duplicated or overbroad wording; validate and commit
• 2. Reply to and resolve every review thread, publish the rework, refresh PR docs(configuration): consolidate child boundary #926, request re-review, and transition to In review

Rework / PR Feedback

• Major merge blockers from review:
  • P2: qualify compatibility mode because runtime.isolation.inherit_environment: true inherits non-credential worker variables and does not install a private DOCKER_CONFIG.
  • P2: state that only Codex preserves LC_*; Claude and default custom runtimes preserve LANG through fixed allowlists but drop other locale variables.
  • P2: promote GitHub Tracker Transition Extension and Workspace-Key Migration out of the child-boundary hierarchy.
  • P3: remove the custom-runtime restatement of the shared credential stripping contract and retain only custom-specific differences.
  • Nit: narrow the general allowlist claim so it does not incorrectly include CODEX_HOME.

Completion Bar

• In-scope requirements implemented
• pnpm lint
• pnpm test (N/A under docs-only scope-proportional validation)
• pnpm typecheck (N/A under docs-only scope-proportional validation)
• pnpm build (N/A under docs-only scope-proportional validation)
• Docker E2E (N/A; no integration behavior changed)
• Tests for new functionality (N/A; documentation only)
• Code conventions (CLAUDE.md)
• Review threads answered and resolved

Validation

• pnpm format — fail: pre-existing repository-wide formatting drift in 68 unrelated files
• pnpm exec prettier --write docs/configuration.md && pnpm exec prettier --check docs/configuration.md — pass
• pnpm lint — pass
• git diff --check — pass
• Anchor reference audit with rg — pass; the explicit #environment-loading-order and #agent-visible-symphony-context anchors and the unchanged #auth-and-api-endpoints heading preserve all inbound links
• Runtime/spec coverage audit — pass; wording matches Codex, Claude, default custom, and compatibility-mode child construction. No new upstream-spec divergence was introduced; the existing custom compatibility-mode divergence remains explicit.
• PR review audit — pass; all five threads received resolution replies and are resolved
• Published branch verification — pass; PR docs(configuration): consolidate child boundary #926 and remote branch point to 46bec3e54f51d3960d87e96cfc24ff42f8964ed6
• Changeset: N/A (documentation-only change without a release label)
• Docker E2E evidence: N/A (no integration behavior changes)
• Merge commit (Land cycle only): N/A

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

• None

Progress Log

• 2026-09-05T22:49:56Z: Opened rework Cycle 2 after inspecting PR docs(configuration): consolidate child boundary #926 reviews, all unresolved threads, top-level comments, and the existing delivery commit.
• 2026-09-05T22:51:00Z: Confirmed the orchestrator transition to In progress and recorded the five review findings as the rework plan.
• 2026-09-05T22:52:55Z: Corrected all review findings, committed 46bec3e, published the assigned branch, replied to and resolved all five threads, refreshed PR docs(configuration): consolidate child boundary #926, and requested re-review from hojinzs.
• 2026-09-05T22:53:30Z: Re-verified issue coverage, upstream-spec conformance, remote HEAD, clean worktree, and the docs-only Completion Bar. Handoff is ready because every requested correction is published and reviewable.

Intended close transition:

🔁 Status: In progress → In review

Reason: PR #926 rework is published at 46bec3e; all review threads are answered and the docs-only Completion Bar is satisfied
Cycle: 2 close

Blockers

None