toki/iop
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions
iop/agent-task/m-provider-resource-admission-ownership-alignment/06+05_refresh_consistency/CODE_REVIEW-local-G07.md

16 KiB
Raw Blame History

Code Review Reference - PROVIDER_POLICY_REFRESH

[IMPLEMENTING AGENT — READ FIRST] Filling in this file is the mandatory final step of implementation. The task is NOT complete until every implementation-owned section below is filled in. Complete the 구현 체크리스트; the final checklist item is mandatory before saving. Fill implementation-owned sections, then stop with active files in place and report ready for review. If implementation is blocked by a selected Milestone 구현 잠금 > 결정 필요 item, fill 사용자 리뷰 요청 with linked evidence and stop with active files in place; code-review decides whether to write USER_REVIEW.md. Environment/secret/service blockers, generic scope changes, repeated failures, and evidence gaps that a follow-up agent can close are normal follow-up issues, not user-review blockers by themselves. Do not ask the user directly, present choices in chat, or call request_user_input during implementation; record only Milestone lock decisions in 사용자 리뷰 요청 and stop for code-review. Finalization (코드리뷰 결과, log rename, complete.log, archive moves, 코드리뷰 전용 체크리스트) is review-agent-only, even after compaction/resume. Follow the ownership table at the bottom of this file for which sections you own.

개요

date=2026-07-19 task=m-provider-resource-admission-ownership-alignment/06+05_refresh_consistency, plan=0, tag=PROVIDER_POLICY_REFRESH

Roadmap Targets

  • Milestone: agent-roadmap/phase/operational-observability-provider-management/milestones/provider-resource-admission-ownership-alignment.md
  • Milestone link: Milestone 문서
  • Task ids:
    • refresh-consistency: provider resource와 공통 queue policy live refresh 정합성
  • Completion mode: check-on-pass

이 파일을 읽는 리뷰 에이전트에게

[REVIEW AGENT ONLY] 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다.

각 항목의 구현을 실제 소스 파일과 대조하고, 검증 결과 섹션의 출력이 코드와 일치하는지 확인하세요. 리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다.

  1. 판정을 append한다.
  2. CODE_REVIEW-local-G07.md → code_review_local_G07_N.log, PLAN-local-G07.md → plan_local_G07_M.log로 아카이브한다.
  3. PASS이면 complete.log 작성 후 active task 디렉터리를 agent-task/archive/YYYY/MM/m-provider-resource-admission-ownership-alignment/06+05_refresh_consistency/로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 USER_REVIEW.md를 작성한다. USER_REVIEW.md가 연결된 Milestone 결정으로 완료/PASS 해소되면 code-review가 USER_REVIEW.md를 해소 상태로 갱신하고 complete.log 작성 후 archive 이동한다.
  4. PASS이고 task group이 m-provider-resource-admission-ownership-alignment이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 update-roadmap 호출은 런타임 책임이다.
  5. 적용 가능한 코드리뷰 전용 체크리스트 항목을 최종 .log 위치에서 체크한 뒤 보고한다.

구현 항목별 완료 여부

항목 완료 여부
[PROVIDER_POLICY_REFRESH-1] Existing waiter policy rebase [ ]
[PROVIDER_POLICY_REFRESH-2] Resource attribute refresh와 over-capacity recovery [ ]
[PROVIDER_POLICY_REFRESH-3] Concurrent refresh/admission/release race [ ]

구현 체크리스트

  • 기존 pending item이 enqueue 시각을 보존하고 common timeout refresh의 단축·연장·0(no-timeout)을 즉시 반영하며 stale timer 발화 뒤 현재 deadline을 재확인해 spurious timeout을 막되 max_queue shrink가 기존 waiter를 강제 제거하지 않도록 구현한다.
  • capacity/long capacity shrink가 기존 lease를 보존하고 over-capacity 동안 신규 lease를 막으며 increase 후 전역 waiter를 즉시 pump하도록 deterministic tests로 고정한다.
  • priority와 enabled refresh가 선행 03의 live candidate resolver를 통해 모든 관련 waiter 선택에 즉시 반영되고 고립/first-candidate fallback이 없도록 검증한다.
  • atomic runtime apply와 concurrent refresh/admission/release race에서 duplicate lease·capacity peak·pending/resource/lease final-zero 불변식을 검증한다.
  • CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

코드리뷰 전용 체크리스트

[REVIEW AGENT ONLY] 이 체크리스트는 코드리뷰 에이전트만 사용한다. 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다.

  • 코드리뷰 결과에 PASS, WARN, FAIL 중 하나의 판정을 append한다.
  • 판정과 차원별 평가, Required/Suggested/Nit 분류가 서로 일치한다.
  • active CODE_REVIEW-*-G??.md를 code_review_local_G07_N.log로 아카이브한다.
  • active PLAN-*-G??.md를 plan_local_G07_M.log로 아카이브한다.
  • .gitignore의 Agent-Ops 관리 block이 agent-task/**/*.md와 agent-task/**/*.log를 unignore하고 agent-roadmap/current.md를 ignore하는지 확인한다.
  • PASS이면 agent-ops/skills/common/code-review/templates/complete-log-template.md 기준으로 complete.log를 작성하고 active .md 파일을 남기지 않는다.
  • PASS이면 active task 디렉터리 agent-task/m-provider-resource-admission-ownership-alignment/06+05_refresh_consistency/를 agent-task/archive/YYYY/MM/m-provider-resource-admission-ownership-alignment/06+05_refresh_consistency/로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
  • PASS이고 task group이 m-provider-resource-admission-ownership-alignment이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 update-roadmap 직접 호출을 하지 않는다.
  • PASS split 작업이면 이동 후 빈 active parent agent-task/m-provider-resource-admission-ownership-alignment/를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
  • WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active PLAN-local-G07.md와 CODE_REVIEW-local-G07.md를 작성하고 complete.log를 작성하지 않는다.
  • USER_REVIEW이면 agent-ops/skills/common/code-review/templates/user-review-template.md 기준으로 USER_REVIEW.md를 작성하고 active PLAN-*.md, CODE_REVIEW-*.md, complete.log를 남기지 않는다.
  • USER_REVIEW가 연결된 Milestone 결정으로 완료/PASS 해소되면 USER_REVIEW.md를 해소 상태로 갱신하고 complete.log를 작성한 뒤 task directory를 archive로 이동한다.

계획 대비 변경 사항

구현 에이전트가 계획과 다르게 구현한 부분을 이유와 함께 기록한다.

주요 설계 결정

구현 에이전트가 주요 설계 결정 사항을 기록한다.

사용자 리뷰 요청

기본값은 없음이다. 구현 중 새 결정이 필요해 보여도 직접 질문하거나 선택지를 제시하거나 request_user_input을 호출하지 않는다. 이 섹션은 선택된 Milestone 구현 잠금 > 결정 필요 항목이 실구현을 차단할 때만 채운다. 외부 환경/secret/서비스 준비, 검증 증거 공백, 반복 실패, 일반 범위 조정은 사용자 리뷰 요청이 아니며 검증 결과, 계획 대비 변경 사항, 또는 code-review의 일반 follow-up plan으로 처리한다.

  • 상태: 없음
  • 사유 유형: 없음
  • 연결 대상: 없음
  • 결정 필요: 없음
  • 차단 근거: 없음
  • 실행한 검증/명령: 없음
  • 자동 후속 불가 이유: 없음
  • 재개 조건: 없음

리뷰어를 위한 체크포인트

  • timeout refresh가 original enqueue time을 기준으로 old timer를 stop/drain/reset하고, old timer와 연장/0 notification이 경쟁해 timer branch가 먼저 선택되어도 manager lock에서 current deadline/membership을 재검사해 spurious timeout이 없는지 확인한다.
  • max shrink가 existing waiter를 evict하지 않고 신규 enqueue만 차단하는지 확인한다.
  • actual leases가 capacity/long shrink에서 유지되며 over-capacity 동안 counter가 더 증가하지 않고 release/raise 후 회복하는지 확인한다.
  • priority/enabled refresh가 선행 03 live candidate universe를 사용하고 모든 group waiter를 pump하는지 확인한다.
  • race test가 단순 data-race 부재뿐 아니라 peak/final pending/resource/lease invariants를 assertion하는지 확인한다.

검증 결과

구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다.

필수 규칙:

  • 검증 명령은 고정된 계약이다. 임의로 대체하지 않는다.
  • 대체가 필요하면 계획 대비 변경 사항에 이유와 대체 명령을 기록한다.
  • 검증 결과에는 실제 stdout/stderr를 붙여 넣는다.
  • 사용자 리뷰 요청으로 명령을 끝까지 실행하지 못했다면 사용자 리뷰 요청에 실행한 명령, 실제 출력, 미실행 명령의 사유를 기록한다.
  • mobile/UI hang, timeout, 또는 2분 무진행은 blind retry를 중단하고 focused rerun 명령과 screenshot/window/UI-tree evidence path를 남기며, 불가능하면 정확한 사유를 남긴다.

PROVIDER_POLICY_REFRESH-1 중간 검증

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/internal/service -run 'TestProviderPool(PolicyRefreshShortensExistingWaiterTimeout|PolicyRefreshExtendsExistingWaiterTimeout|PolicyRefreshDisablesExistingWaiterTimeout|PolicyRefreshExtensionIgnoresStaleTimer|MaxQueueShrinkPreservesExistingWaiters)$'
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -race -count=5 ./apps/edge/internal/service -run 'TestProviderPoolPolicyRefresh'
(output)

PROVIDER_POLICY_REFRESH-2 중간 검증

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/internal/service -run 'TestRefresh(CapacityShrinkPreservesLeaseAndBlocksUntilRecovery|LongCapacityShrinkPreservesLeaseAndBlocksLongAdmission|PriorityChangesQueuedProviderSelection|DisableReenableReevaluatesAllWaiters)$'
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/internal/bootstrap -run 'TestRefreshRuntimeSnapshotPreservesLeaseAndAppliesPolicyAtomically'
(output)

PROVIDER_POLICY_REFRESH-3 중간 검증

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -race -count=5 ./apps/edge/internal/service -run 'TestRefreshConcurrentAdmissionReleasePreservesProviderPolicyInvariants|TestProviderPoolPolicyRefresh'
(output)

최종 검증

$ gofmt -d apps/edge/internal/service/model_queue_types.go apps/edge/internal/service/model_queue_admission.go apps/edge/internal/service/service.go apps/edge/internal/service/provider_resolution.go apps/edge/internal/service/model_queue_test_support_test.go apps/edge/internal/service/provider_scheduling_test.go apps/edge/internal/service/service_internal_test.go apps/edge/internal/service/model_queue_admission_test.go apps/edge/internal/bootstrap/runtime_refresh_test.go
(output)

$ git diff --check
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/internal/service -run 'TestProviderPool(PolicyRefreshShortensExistingWaiterTimeout|PolicyRefreshExtendsExistingWaiterTimeout|PolicyRefreshDisablesExistingWaiterTimeout|PolicyRefreshExtensionIgnoresStaleTimer|MaxQueueShrinkPreservesExistingWaiters)$'
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/internal/service -run 'TestRefresh(CapacityShrinkPreservesLeaseAndBlocksUntilRecovery|LongCapacityShrinkPreservesLeaseAndBlocksLongAdmission|PriorityChangesQueuedProviderSelection|DisableReenableReevaluatesAllWaiters)$'
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -race -count=5 ./apps/edge/internal/service -run 'TestRefreshConcurrentAdmissionReleasePreservesProviderPolicyInvariants|TestProviderPoolPolicyRefresh'
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/internal/bootstrap -run 'TestRefreshRuntimeSnapshotPreservesLeaseAndAppliesPolicyAtomically'
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh go test -count=1 ./apps/edge/...
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh scripts/e2e-long-context-admission-smoke.sh --preflight --out-dir /tmp/iop-long-admission-smoke-provider-policy-refresh
(output)

$ GOCACHE=/tmp/iop-go-cache-provider-policy-refresh scripts/e2e-long-context-admission-smoke.sh --preflight --scenario mixed --out-dir /tmp/iop-long-admission-smoke-provider-policy-refresh-mixed
(run only after preflight OK; otherwise record blocker)

[IMPLEMENTING AGENT — BEFORE SAVING] Have you filled in every implementation-owned section: completion table, implementation checklist, changes from plan, design decisions, and verification output? If anything is blank, go back and fill it in before saving this file. Leave review-agent-only sections unchanged.

섹션 소유권

Section Owner Note
Header comment, 개요, 리뷰 에이전트 지시 Fixed at stub creation Implementing agent must not modify or execute these (archive, complete.log, and task-directory archive move are review-agent only)
Roadmap Targets Fixed at stub creation from plan when present Implementing agent must not modify; code-review copies it into complete.log as Roadmap Completion only on PASS
Archive Evidence Snapshot Fixed at stub creation from plan when present Implementing agent uses it as default prior-loop context; read only the specific archive files cited there when more detail is required
Agent UI Completion Mixed Present only for plan-required agent-ui code work; implementing agent fills actual evidence, review agent applies 구현됨 status/evidence update on PASS and copies the section into complete.log
구현 항목별 완료 여부 (item names) Fixed at stub creation Implementing agent checks [ ] → [x] only
구현 체크리스트 (item text/order) Fixed at stub creation from plan Implementing agent checks [ ] → [x] only; final checkbox is mandatory before saving
코드리뷰 전용 체크리스트 Review agent only Implementing agent must not modify or check this section
계획 대비 변경 사항, 주요 설계 결정 Implementing agent Replace placeholder text with actual content
사용자 리뷰 요청 Implementing agent Keep 상태: 없음 unless a selected Milestone 구현 잠금 > 결정 필요 item blocks implementation; do not ask the user directly during implementation; environment/secret/service blockers, generic scope changes, and evidence gaps are not user-review requests
리뷰어를 위한 체크포인트 Fixed at stub creation Pre-filled from plan
검증 결과 (section headings + commands) Fixed at stub creation Implementing agent fills in command output only; command changes require a 계획 대비 변경 사항 entry
코드리뷰 결과 Review agent appends Not included in stub