toki/proto-socket
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions
proto-socket/agent-task/m-inbound-queue-ordering/10+09_stress_baseline/CODE_REVIEW-cloud-G08.md

7.4 KiB
Raw Blame History

Code Review Reference - TEST

[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 user-only decision, user-owned external environment prerequisite, or scope conflict, fill 사용자 리뷰 요청 with evidence and stop with active files in place; code-review decides whether to write USER_REVIEW.md. Evidence gaps that a follow-up agent can close by rerunning commands or collecting artifacts are normal follow-up issues, not user-review blockers by themselves. 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-06-02 task=m-inbound-queue-ordering/10+09_stress_baseline, plan=0, tag=TEST

Roadmap Targets

  • Milestone: agent-roadmap/milestones/inbound-queue-ordering.md
  • Task ids:
    • rt-bench: 빠른 request-response 왕복 메시징 벤치마크를 추가한다.
    • burst-stress: burst inbound traffic 스트레스 테스트를 추가한다.
    • sustained-load: 지속 부하 테스트 기준선을 추가한다.
    • parallel-clients: 다중 connection 병렬 스트레스 테스트를 추가한다.
    • gateway-bench: 언어별 worker gateway 성능 기준을 검증한다.
  • Completion mode: check-on-pass

구현 항목별 완료 여부

항목 완료 여부
[TEST-1] Stress runner contract [ ]
[TEST-2] Roundtrip/burst/sustained/multi-client axes [ ]
[TEST-3] Gateway on/off baseline [ ]

구현 체크리스트

  • 구현 시작 전 09_verify_ordering의 complete.log가 active 또는 archive에 있는지 확인한다.
  • local stress/benchmark runner entrypoint와 결과 기록 형식을 추가한다.
  • roundtrip, burst, sustained, parallel-clients, gateway on/off 측정 축을 configurable하게 구현한다.
  • 안정성 합격선 mismatch/timeout/FIFO violation/pending leak 0을 runner exit code에 반영한다.
  • focused stress smoke와 전체 local matrix를 실행한다.
  • CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.

코드리뷰 전용 체크리스트

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

  • 코드리뷰 결과에 PASS, WARN, FAIL 중 하나의 판정을 append한다.
  • 판정과 차원별 평가, Required/Suggested/Nit 분류가 서로 일치한다.
  • active CODE_REVIEW-*-G??.md를 code_review_cloud_G08_N.log로 아카이브한다.
  • active PLAN-*-G??.md를 plan_cloud_G08_M.log로 아카이브한다.
  • .gitignore의 Agent-Ops 관리 block이 agent-task/**/*.md와 agent-task/**/*.log를 unignore하여 plan/review/archive 산출물이 추적 가능한지 확인한다.
  • PASS이면 agent-ops/skills/common/code-review/templates/complete-log-template.md 기준으로 complete.log를 작성하고 active .md 파일을 남기지 않는다.
  • PASS이면 active task 디렉터리 agent-task/m-inbound-queue-ordering/10+09_stress_baseline/를 agent-task/archive/YYYY/MM/m-inbound-queue-ordering/10+09_stress_baseline/로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
  • PASS이고 task group이 m-<milestone-slug>이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 update-roadmap 직접 호출을 하지 않는다.
  • PASS split 작업이면 이동 후 빈 active parent agent-task/m-inbound-queue-ordering/를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
  • WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active PLAN-{build_lane}-GNN.md와 CODE_REVIEW-{review_lane}-GNN.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가 사용자 결정으로 완료/PASS 해소되면 USER_REVIEW.md를 해소 상태로 갱신하고 complete.log를 작성한 뒤 task directory를 archive로 이동한다.

계획 대비 변경 사항

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

주요 설계 결정

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

사용자 리뷰 요청

기본값은 없음이다. 구현 중 사용자 결정, 사용자 소유 외부 환경/secret/서비스 준비, 또는 계획 범위 변경 없이는 안전하게 진행할 수 없으면 아래 항목을 실제 내용으로 교체하고, 구현을 중단한 뒤 active 파일을 그대로 둔 채 리뷰를 요청한다. 후속 에이전트가 명령 재실행이나 산출물 수집으로 해소할 수 있는 검증 증거 공백만으로는 사용자 리뷰 요청을 작성하지 않는다.

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

리뷰어를 위한 체크포인트

  • 10+09 dependency가 실제 09_verify_ordering/complete.log로 충족된 뒤 구현됐는지 확인한다.
  • stress runner가 local result record와 exit code로 안정성 위반을 표현하는지 확인한다.
  • 절대 성능 수치를 hard fail로 두지 않고 mismatch/timeout/FIFO/leak 0을 안정성 fail로 두는지 확인한다.

검증 결과

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

필수 규칙:

  • 검증 명령은 고정된 계약이다. 임의로 대체하지 않는다.
  • 대체가 필요하면 계획 대비 변경 사항에 이유와 대체 명령을 기록한다.
  • 검증 결과에는 실제 stdout/stderr를 붙여 넣는다.
  • 사용자 리뷰 요청으로 명령을 끝까지 실행하지 못했다면 사용자 리뷰 요청에 실행한 명령, 실제 출력, 미실행 명령의 사유를 기록한다.

TEST-1 중간 검증

$ bash agent-ops/skills/project/run-proto-socket-test-matrix/scripts/run_stress.sh --quick
(output)

TEST-2 중간 검증

$ bash agent-ops/skills/project/run-proto-socket-test-matrix/scripts/run_stress.sh --quick --profile roundtrip,burst,sustained,parallel
(output)

TEST-3 중간 검증

$ bash agent-ops/skills/project/run-proto-socket-test-matrix/scripts/run_stress.sh --quick --profile gateway
(output)

최종 검증

$ bash agent-ops/skills/project/run-proto-socket-test-matrix/scripts/run_stress.sh --quick
(output)

$ bash agent-ops/skills/project/run-proto-socket-test-matrix/scripts/run_matrix.sh --all
(output)

$ git diff --check
(output)