chore: update domain types and archive backtest task files
This commit is contained in:
parent
0cccb47163
commit
353584c8f9
9 changed files with 1024 additions and 94 deletions
|
|
@ -50,16 +50,16 @@ task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix, plan=0, ta
|
|||
> **[REVIEW AGENT ONLY]** 이 체크리스트는 코드리뷰 에이전트만 사용한다.
|
||||
> 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다.
|
||||
|
||||
- [ ] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
|
||||
- [ ] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
|
||||
- [ ] active `CODE_REVIEW-*-G??.md`를 `code_review_local_G05_N.log`로 아카이브한다.
|
||||
- [ ] active `PLAN-*-G??.md`를 `plan_local_G05_M.log`로 아카이브한다.
|
||||
- [ ] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다.
|
||||
- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
|
||||
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
|
||||
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_local_G05_N.log`로 아카이브한다.
|
||||
- [x] active `PLAN-*-G??.md`를 `plan_local_G05_M.log`로 아카이브한다.
|
||||
- [x] `.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-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`를 `agent-task/archive/YYYY/MM/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
|
||||
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
|
||||
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-backtest-multi-timeframe-coverage/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
|
||||
- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-local-G05.md`와 `CODE_REVIEW-local-G05.md`를 작성하고 `complete.log`를 작성하지 않는다.
|
||||
- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-local-G05.md`와 `CODE_REVIEW-local-G05.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가 연결된 SDD/Milestone 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다.
|
||||
|
||||
|
|
@ -107,27 +107,30 @@ _구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후
|
|||
```bash
|
||||
$ go test ./packages/domain/...
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market 0.003s
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market (cached)
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/trading (cached)
|
||||
```
|
||||
|
||||
### API-2 중간 검증
|
||||
```bash
|
||||
$ go test ./services/worker/...
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker 0.012s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-migrate 0.007s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/backtest 0.008s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-data-check [no test files]
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-migrate (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/config (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/contracts (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/jobs 0.006s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/livetrading 0.015s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/datacheck 0.009s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/importer 0.009s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/papertrading 0.009s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/providers/kis 0.025s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/jobs (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/livetrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/datacheck (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/importer (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/papertrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/providers/kis (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/rediskeys (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/socket 0.067s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres 0.155s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/socket (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres/sqlc [no test files]
|
||||
```
|
||||
|
||||
### 최종 검증
|
||||
|
|
@ -154,3 +157,18 @@ $ go test ./packages/domain/... ./services/worker/...
|
|||
| 사용자 리뷰 요청 | Implementing agent | Keep `상태: 없음` unless selected SDD/Milestone lock decision blocks |
|
||||
| 리뷰어를 위한 체크포인트 | Fixed at stub creation | Review focus |
|
||||
| 검증 결과 | Implementing agent | Paste actual command output |
|
||||
|
||||
## 코드리뷰 결과
|
||||
|
||||
- 종합 판정: FAIL
|
||||
- 차원별 평가:
|
||||
- correctness: Fail
|
||||
- completeness: Fail
|
||||
- test coverage: Fail
|
||||
- API contract: Fail
|
||||
- code quality: Pass
|
||||
- plan deviation: Fail
|
||||
- verification trust: Pass
|
||||
- 발견된 문제:
|
||||
- Required: `packages/domain/market/types.go:156`에서 `CheckBars`가 matrix `Rules`를 평가하기 전에 deprecated 호환 필드인 `c.Venues`를 필수 precondition으로 사용합니다. 그 결과 새 API 사용자가 `ProviderCapability{Provider: ..., Rules: ...}`처럼 matrix rule만 채운 capability를 만들면, rule이 해당 venue/timeframe을 허용해도 항상 `provider "..." does not support venue "..."`로 거절됩니다. 현재 `TestProviderCapabilityMatrix`는 `Venues`/`Timeframes`까지 함께 채워 이 결함을 가리고 있으므로, plan의 "provider/market/venue/asset type/timeframe matrix request와 accepted/rejected decision" foundation을 완전히 만족하지 못합니다. `CheckBars`는 venue 지원 여부를 `Rules`에서 도출하거나, legacy `Venues` map이 비어 있지 않을 때만 compatibility prefilter로 사용하도록 고치고, `Rules`만 채운 capability가 KR/US daily accepted 및 monthly/minute rejected를 반환하는 회귀 테스트를 추가하세요.
|
||||
- 다음 단계: WARN/FAIL 후속 plan/review를 작성한다.
|
||||
|
|
@ -0,0 +1,184 @@
|
|||
<!-- task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix plan=1 tag=REVIEW_API -->
|
||||
|
||||
# Code Review Reference - REVIEW_API
|
||||
|
||||
> **[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 SDD decision or 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 SDD/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-06-18
|
||||
task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix, plan=1, tag=REVIEW_API
|
||||
|
||||
## Archive Evidence Snapshot
|
||||
|
||||
- Archived plan: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_0.log`
|
||||
- Archived review: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_0.log`
|
||||
- Verdict: FAIL
|
||||
- Required summary:
|
||||
- `packages/domain/market/types.go:156`에서 `CheckBars`가 `Rules`를 평가하기 전에 deprecated `c.Venues`를 필수 precondition으로 사용해, `Rules`만 채운 capability를 항상 unsupported venue로 거절한다.
|
||||
- `packages/domain/market/types_test.go:168`의 matrix test는 `Venues`/`Timeframes`도 함께 채워 이 결함을 가린다.
|
||||
- Affected files:
|
||||
- `packages/domain/market/types.go`
|
||||
- `packages/domain/market/types_test.go`
|
||||
- Verification evidence:
|
||||
- 리뷰 중 `go test ./packages/domain/... ./services/worker/...`는 exit code 0으로 통과했으나, 현 테스트는 rules-only capability path를 검증하지 않는다.
|
||||
- Narrow reread allowed if needed:
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_0.log`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_0.log`
|
||||
|
||||
## 이 파일을 읽는 리뷰 에이전트에게
|
||||
|
||||
> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다.
|
||||
|
||||
각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
|
||||
리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다.
|
||||
|
||||
1. 판정을 append한다.
|
||||
2. `CODE_REVIEW-local-G05.md` -> `code_review_local_G05_N.log`, `PLAN-local-G05.md` -> `plan_local_G05_M.log`로 아카이브한다.
|
||||
3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 연결된 SDD/Milestone 결정으로 완료/PASS 해소되면 code-review가 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다.
|
||||
4. PASS이고 task group이 `m-<milestone-slug>`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다.
|
||||
5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다.
|
||||
|
||||
---
|
||||
|
||||
## 구현 항목별 완료 여부
|
||||
|
||||
| 항목 | 완료 여부 |
|
||||
|------|---------|
|
||||
| [REVIEW_API-1] Make matrix capability self-contained | [x] |
|
||||
|
||||
## 구현 체크리스트
|
||||
|
||||
- [x] `CheckBars`가 `Rules`만 채운 `ProviderCapability`에서도 provider/market/venue/asset type/timeframe 판정을 수행하도록 legacy `Venues` precondition을 수정한다.
|
||||
- [x] domain tests에 rules-only capability 회귀 케이스를 추가해 KR/US daily accepted와 monthly/minute rejected reason을 검증한다.
|
||||
- [x] `go test -count=1 ./packages/domain/market`와 `go test ./packages/domain/... ./services/worker/...`를 실행해 domain behavior와 기존 worker call-site compile을 확인한다.
|
||||
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.
|
||||
|
||||
## 코드리뷰 전용 체크리스트
|
||||
|
||||
> **[REVIEW AGENT ONLY]** 이 체크리스트는 코드리뷰 에이전트만 사용한다.
|
||||
> 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다.
|
||||
|
||||
- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
|
||||
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
|
||||
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_local_G05_N.log`로 아카이브한다.
|
||||
- [x] active `PLAN-*-G??.md`를 `plan_local_G05_M.log`로 아카이브한다.
|
||||
- [x] `.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-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`를 `agent-task/archive/YYYY/MM/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
|
||||
- [ ] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
|
||||
- [ ] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-backtest-multi-timeframe-coverage/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
|
||||
- [x] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-local-G05.md`와 `CODE_REVIEW-local-G05.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가 연결된 SDD/Milestone 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다.
|
||||
|
||||
## 계획 대비 변경 사항
|
||||
|
||||
PLAN이 제안한 After 방향(`len(c.Venues) > 0 && !c.Venues[req.Venue]` — Venues가 있을 때만 prefilter)과 달리, `len(c.Rules) == 0 && !c.Venues[req.Venue]` (`Rules`가 없을 때만 fallback)로 구현했다. 기능적 결과는 동일: `Rules`가 채워지면 Venues를 무시하고 rule matcher만 사용한다. PLAN의 "rules matching 결과가 없을 때의 legacy fallback" 의도와도 일치하며, 조건 역전으로 기존 Venues-capable capability의 behavior도 변경되지 않는다. `getVenueMetadata`는 unknown venue를 이미 `!ok`로 거두므로 unknown venue 예외 처리는 market validation 단계에서 자연스럽게 처리됨.
|
||||
|
||||
## 주요 설계 결정
|
||||
|
||||
- `len(c.Rules) == 0` 조건부 legacy fallback: `Rules`가 비어있을 때만 `Venues`를 백호환을 위해 체크한다. `Rules`가 채워진 경우엔 rule matcher가 venue/asset type/timeframe 판정을 모두 담당하므로 `Venues`를 precondition으로 요구하지 않는다.
|
||||
- 회귀 테스트는 `Venues: map[Venue]bool{}`로 비우고 `Rules`만 채운 `rulesOnlyKIS` capability를 만들어, KR KRX equity daily는 accepted, monthly/minute는 rejected reason으로 거절되는지를 검증한다.
|
||||
|
||||
## 사용자 리뷰 요청
|
||||
|
||||
_기본값은 `없음`이다. 구현 중 새 결정이 필요해 보여도 직접 질문하거나 선택지를 제시하거나 `request_user_input`을 호출하지 않는다. 이 섹션은 선택된 SDD 결정 또는 선택된 Milestone `구현 잠금 > 결정 필요` 항목이 실구현을 차단할 때만 채운다. 외부 환경/secret/서비스 준비, 검증 증거 공백, 반복 실패, 일반 범위 조정은 사용자 리뷰 요청이 아니며 `검증 결과`, `계획 대비 변경 사항`, 또는 code-review의 일반 follow-up plan으로 처리한다._
|
||||
|
||||
- 상태: 없음
|
||||
- 사유 유형: 없음
|
||||
- 연결 대상: 없음
|
||||
- 결정 필요: 없음
|
||||
- 차단 근거: 없음
|
||||
- 실행한 검증/명령: 없음
|
||||
- 자동 후속 불가 이유: 없음
|
||||
- 재개 조건: 없음
|
||||
|
||||
## 리뷰어를 위한 체크포인트
|
||||
|
||||
- `CheckBars`가 `ProviderCapability.Rules`만 채운 capability에서도 accepted/rejected decision을 반환하는지 확인한다.
|
||||
- legacy `Venues`/`Timeframes` 필드가 기존 `RequireDailyBars` call site compile compatibility를 깨지 않는지 확인한다.
|
||||
- rules-only regression test가 daily accepted와 monthly/minute rejected reason을 실제로 검증하는지 확인한다.
|
||||
|
||||
## 검증 결과
|
||||
|
||||
필수 규칙:
|
||||
- 검증 명령은 고정된 계약이다. 임의로 대체하지 않는다.
|
||||
- 대체가 필요하면 `계획 대비 변경 사항`에 이유와 대체 명령을 기록한다.
|
||||
- `검증 결과`에는 실제 stdout/stderr를 붙여 넣는다.
|
||||
- 사용자 리뷰 요청으로 명령을 끝까지 실행하지 못했다면 `사용자 리뷰 요청`에 실행한 명령, 실제 출력, 미실행 명령의 사유를 기록한다.
|
||||
- mobile/UI hang, timeout, 또는 2분 무진행은 blind retry를 중단하고 focused rerun 명령과 screenshot/window/UI-tree evidence path를 남기며, 불가능하면 정확한 사유를 남긴다.
|
||||
|
||||
### REVIEW_API-1 중간 검증
|
||||
```bash
|
||||
$ go test -count=1 ./packages/domain/market
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market 0.002s
|
||||
```
|
||||
|
||||
### 최종 검증
|
||||
```bash
|
||||
$ go test ./packages/domain/... ./services/worker/...
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market 0.011s
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/trading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker 0.008s
|
||||
? git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-data-check [no test files]
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-migrate (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/config (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/contracts (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/jobs 0.007s
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/livetrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/datacheck (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/importer (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/papertrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/providers/kis (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/rediskeys (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/socket (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/internal/storage [no test files]
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres/sqlc [no test files]
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
> **[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.
|
||||
|
||||
## Ownership
|
||||
|
||||
| Section | Owner | Note |
|
||||
|---------|-------|------|
|
||||
| Header comment, 개요, 리뷰 에이전트 지시 | Fixed at stub creation | Implementing agent must not modify or execute these |
|
||||
| Archive Evidence Snapshot | Fixed at stub creation from plan | Implementing agent uses it as default prior-loop context |
|
||||
| 구현 항목별 완료 여부 | Implementing agent | Check boxes only |
|
||||
| 구현 체크리스트 | Implementing agent | Check boxes only |
|
||||
| 코드리뷰 전용 체크리스트 | Review agent only | Implementing agent must not modify |
|
||||
| 계획 대비 변경 사항, 주요 설계 결정 | Implementing agent | Replace placeholder text |
|
||||
| 사용자 리뷰 요청 | Implementing agent | Keep `상태: 없음` unless selected SDD/Milestone lock decision blocks |
|
||||
| 리뷰어를 위한 체크포인트 | Fixed at stub creation | Review focus |
|
||||
| 검증 결과 | Implementing agent | Paste actual command output |
|
||||
|
||||
## 코드리뷰 결과
|
||||
|
||||
- 종합 판정: FAIL
|
||||
- 차원별 평가:
|
||||
- correctness: Pass
|
||||
- completeness: Fail
|
||||
- test coverage: Fail
|
||||
- API contract: Pass
|
||||
- code quality: Pass
|
||||
- plan deviation: Fail
|
||||
- verification trust: Pass
|
||||
- 발견된 문제:
|
||||
- Required: `packages/domain/market/types_test.go:217`의 `rulesOnlyKIS` fixture가 `MarketKR`/`VenueKRX` rule만 포함하고, `packages/domain/market/types_test.go:305` 이후 rules-only accepted case도 KR daily만 검증합니다. `PLAN-local-G05.md`의 구현 체크리스트와 테스트 작성 조건은 rules-only capability에서 KR/US daily accepted를 모두 검증하라고 요구하므로, `Rules`만 채운 capability에 `MarketUS`/`VenueNASDAQ` daily rule과 US/NASDAQ ETF daily accepted case를 추가해야 합니다.
|
||||
- 다음 단계:
|
||||
- FAIL 후속: active PLAN/CODE_REVIEW를 log로 아카이브하고, rules-only US/NASDAQ daily accepted 회귀 테스트를 추가하는 좁은 후속 `PLAN-local-G05.md`와 `CODE_REVIEW-local-G05.md`를 작성한다.
|
||||
|
|
@ -0,0 +1,188 @@
|
|||
<!-- task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix plan=2 tag=REVIEW_REVIEW_API -->
|
||||
|
||||
# Code Review Reference - REVIEW_REVIEW_API
|
||||
|
||||
> **[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 SDD decision or 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 SDD/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-06-18
|
||||
task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix, plan=2, tag=REVIEW_REVIEW_API
|
||||
|
||||
## Archive Evidence Snapshot
|
||||
|
||||
- Archived plan: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_1.log`
|
||||
- Archived review: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_1.log`
|
||||
- Verdict: FAIL
|
||||
- Required summary:
|
||||
- `packages/domain/market/types_test.go:217`의 `rulesOnlyKIS` fixture가 KR/KRX rule만 포함한다.
|
||||
- `packages/domain/market/types_test.go:305` 이후 rules-only accepted case도 KR daily만 검증해, 계획상 필수였던 US/NASDAQ daily accepted 회귀 검증이 빠졌다.
|
||||
- Affected files:
|
||||
- `packages/domain/market/types_test.go`
|
||||
- Verification evidence:
|
||||
- 리뷰 중 `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`가 처음에는 `packages/domain/market/types_test.go`를 출력했고, 리뷰어가 `gofmt -w packages/domain/market/types_test.go`로 포맷 drift를 직접 정리했다.
|
||||
- 포맷 정리 후 `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`는 출력 없음.
|
||||
- `go test -count=1 ./packages/domain/market` exit code 0.
|
||||
- `go test ./packages/domain/... ./services/worker/...` exit code 0.
|
||||
- Narrow reread allowed if needed:
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_1.log`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_1.log`
|
||||
|
||||
## 이 파일을 읽는 리뷰 에이전트에게
|
||||
|
||||
> **[REVIEW AGENT ONLY]** 아래 종결 절차는 코드리뷰 에이전트 전용이다. 구현 에이전트는 이 섹션을 실행하지 않는다.
|
||||
|
||||
각 항목의 구현을 실제 소스 파일과 대조하고, `검증 결과` 섹션의 출력이 코드와 일치하는지 확인하세요.
|
||||
리뷰 완료는 아래 순서까지 끝난 상태를 의미합니다.
|
||||
|
||||
1. 판정을 append한다.
|
||||
2. `CODE_REVIEW-local-G05.md` -> `code_review_local_G05_N.log`, `PLAN-local-G05.md` -> `plan_local_G05_M.log`로 아카이브한다.
|
||||
3. PASS이면 `complete.log` 작성 후 active task 디렉터리를 `agent-task/archive/YYYY/MM/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`로 이동한다. WARN/FAIL이면 user-review gate를 확인한 뒤 다음 active plan/review 파일 또는 `USER_REVIEW.md`를 작성한다. `USER_REVIEW.md`가 연결된 SDD/Milestone 결정으로 완료/PASS 해소되면 code-review가 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log` 작성 후 archive 이동한다.
|
||||
4. PASS이고 task group이 `m-<milestone-slug>`이면 완료 이벤트 메타데이터를 보고한다. roadmap 상태 체크와 `update-roadmap` 호출은 런타임 책임이다.
|
||||
5. 적용 가능한 `코드리뷰 전용 체크리스트` 항목을 최종 `.log` 위치에서 체크한 뒤 보고한다.
|
||||
|
||||
---
|
||||
|
||||
## 구현 항목별 완료 여부
|
||||
|
||||
| 항목 | 완료 여부 |
|
||||
|------|---------|
|
||||
| [REVIEW_REVIEW_API-1] Cover rules-only US daily accepted | [x] |
|
||||
|
||||
## 구현 체크리스트
|
||||
|
||||
- [x] `rulesOnlyKIS` fixture에 `MarketUS`/`VenueNASDAQ` daily equity/ETF rule을 추가한다.
|
||||
- [x] rules-only test cases에 US/NASDAQ ETF daily accepted case를 추가해 `Venues` 없이 US daily path가 accepted 되는지 검증한다.
|
||||
- [x] `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`, `go test -count=1 ./packages/domain/market`, `go test ./packages/domain/... ./services/worker/...`를 실행해 formatting, domain regression, worker compatibility를 확인한다.
|
||||
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.
|
||||
|
||||
## 코드리뷰 전용 체크리스트
|
||||
|
||||
> **[REVIEW AGENT ONLY]** 이 체크리스트는 코드리뷰 에이전트만 사용한다.
|
||||
> 구현 에이전트는 이 섹션을 수정하거나 체크하지 않는다.
|
||||
|
||||
- [x] `코드리뷰 결과`에 `PASS`, `WARN`, `FAIL` 중 하나의 판정을 append한다.
|
||||
- [x] 판정과 `차원별 평가`, Required/Suggested/Nit 분류가 서로 일치한다.
|
||||
- [x] active `CODE_REVIEW-*-G??.md`를 `code_review_local_G05_N.log`로 아카이브한다.
|
||||
- [x] active `PLAN-*-G??.md`를 `plan_local_G05_M.log`로 아카이브한다.
|
||||
- [x] `.gitignore`의 Agent-Ops 관리 block이 `agent-task/**/*.md`와 `agent-task/**/*.log`를 unignore하고 `agent-roadmap/current.md`를 ignore하는지 확인한다.
|
||||
- [x] PASS이면 `agent-ops/skills/common/code-review/templates/complete-log-template.md` 기준으로 `complete.log`를 작성하고 active `.md` 파일을 남기지 않는다.
|
||||
- [x] PASS이면 active task 디렉터리 `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`를 `agent-task/archive/YYYY/MM/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`로 이동하고 최종 archive 경로에서 이 체크리스트를 갱신한다.
|
||||
- [x] PASS이고 task group이 `m-<milestone-slug>`이면 런타임이 읽을 완료 이벤트 메타데이터를 보고하고, roadmap 수정이나 `update-roadmap` 직접 호출을 하지 않는다.
|
||||
- [x] PASS split 작업이면 이동 후 빈 active parent `agent-task/m-backtest-multi-timeframe-coverage/`를 제거하거나, 남은 sibling/file이 있어 유지했다고 확인한다.
|
||||
- [ ] WARN/FAIL이고 user-review gate가 트리거되지 않았으면 다음 active `PLAN-local-G05.md`와 `CODE_REVIEW-local-G05.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가 연결된 SDD/Milestone 결정으로 완료/PASS 해소되면 `USER_REVIEW.md`를 해소 상태로 갱신하고 `complete.log`를 작성한 뒤 task directory를 archive로 이동한다.
|
||||
|
||||
## 계획 대비 변경 사항
|
||||
|
||||
계획에 정확히 따랐습니다. 별도의 변경 사항 없음.
|
||||
|
||||
## 주요 설계 결정
|
||||
|
||||
변경 없음. 테스트 fixture와 case 보강만 수행했습니다.
|
||||
|
||||
## 사용자 리뷰 요청
|
||||
|
||||
_기본값은 `없음`이다. 구현 중 새 결정이 필요해 보여도 직접 질문하거나 선택지를 제시하거나 `request_user_input`을 호출하지 않는다. 이 섹션은 선택된 SDD 결정 또는 선택된 Milestone `구현 잠금 > 결정 필요` 항목이 실구현을 차단할 때만 채운다. 외부 환경/secret/서비스 준비, 검증 증거 공백, 반복 실패, 일반 범위 조정은 사용자 리뷰 요청이 아니며 `검증 결과`, `계획 대비 변경 사항`, 또는 code-review의 일반 follow-up plan으로 처리한다._
|
||||
|
||||
- 상태: 없음
|
||||
- 사유 유형: 없음
|
||||
- 연결 대상: 없음
|
||||
- 결정 필요: 없음
|
||||
- 차단 근거: 없음
|
||||
- 실행한 검증/명령: 없음
|
||||
- 자동 후속 불가 이유: 없음
|
||||
- 재개 조건: 없음
|
||||
|
||||
## 리뷰어를 위한 체크포인트
|
||||
|
||||
- `rulesOnlyKIS`가 `Venues` 없이 KR/KRX와 US/NASDAQ daily rule을 모두 포함하는지 확인한다.
|
||||
- rules-only US/NASDAQ ETF daily accepted case가 실제로 `rulesOnlyKIS.CheckBars`를 호출하는지 확인한다.
|
||||
- 기존 monthly/minute rejected reason과 worker compatibility 검증이 유지되는지 확인한다.
|
||||
|
||||
## 검증 결과
|
||||
|
||||
_구현 에이전트가 각 중간 검증 및 최종 검증 명령 실행 후 출력을 여기에 붙여 넣는다._
|
||||
|
||||
### REVIEW_REVIEW_API-1 중간 검증
|
||||
```bash
|
||||
$ gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go
|
||||
# (no output - formatting OK)
|
||||
|
||||
$ go test -count=1 ./packages/domain/market
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market 0.002s
|
||||
```
|
||||
|
||||
### 최종 검증
|
||||
```bash
|
||||
$ gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go
|
||||
# (no output - formatting OK)
|
||||
|
||||
$ go test -count=1 ./packages/domain/market
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market 0.002s
|
||||
|
||||
$ go test ./packages/domain/... ./services/worker/...
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market 0.008s
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/trading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-data-check [no test files]
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-migrate (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/config (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/contracts (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/jobs (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/livetrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/datacheck (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/importer (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/papertrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/providers/kis (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/rediskeys (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/socket (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/internal/storage [no test files]
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres/sqlc [no test files]
|
||||
```
|
||||
|
||||
---
|
||||
|
||||
> **[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.
|
||||
|
||||
## Ownership
|
||||
|
||||
| Section | Owner | Note |
|
||||
|---------|-------|------|
|
||||
| Header comment, 개요, 리뷰 에이전트 지시 | Fixed at stub creation | Implementing agent must not modify or execute these |
|
||||
| Archive Evidence Snapshot | Fixed at stub creation from plan | Implementing agent uses it as default prior-loop context |
|
||||
| 구현 항목별 완료 여부 | Implementing agent | Check boxes only |
|
||||
| 구현 체크리스트 | Implementing agent | Check boxes only |
|
||||
| 코드리뷰 전용 체크리스트 | Review agent only | Implementing agent must not modify |
|
||||
| 계획 대비 변경 사항, 주요 설계 결정 | Implementing agent | Replace placeholder text |
|
||||
| 사용자 리뷰 요청 | Implementing agent | Keep `상태: 없음` unless selected SDD/Milestone lock decision blocks |
|
||||
| 리뷰어를 위한 체크포인트 | Fixed at stub creation | Review focus |
|
||||
| 검증 결과 | Implementing agent | Paste actual command output |
|
||||
|
||||
## 코드리뷰 결과
|
||||
|
||||
- 종합 판정: PASS
|
||||
- 차원별 평가:
|
||||
- correctness: Pass
|
||||
- completeness: Pass
|
||||
- test coverage: Pass
|
||||
- API contract: Pass
|
||||
- code quality: Pass
|
||||
- plan deviation: Pass
|
||||
- verification trust: Pass
|
||||
- 발견된 문제: 없음
|
||||
- 다음 단계:
|
||||
- PASS 종결: active PLAN/CODE_REVIEW를 log로 아카이브하고 `complete.log`를 작성한 뒤 task directory를 `agent-task/archive/2026/06/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/`로 이동한다.
|
||||
|
|
@ -0,0 +1,37 @@
|
|||
# Complete - m-backtest-multi-timeframe-coverage/01_domain_capability_matrix
|
||||
|
||||
## 완료 일시
|
||||
|
||||
2026-06-18
|
||||
|
||||
## 요약
|
||||
|
||||
ProviderCapability matrix API foundation과 rules-only KR/US daily 회귀 테스트를 3회 리뷰 루프 끝에 PASS로 종결했다.
|
||||
|
||||
## 루프 이력
|
||||
|
||||
| Plan | Review | Verdict | 메모 |
|
||||
|------|--------|---------|------|
|
||||
| `plan_local_G05_0.log` | `code_review_local_G05_0.log` | FAIL | `CheckBars`가 `Rules` 평가 전에 legacy `Venues` map을 필수 precondition으로 요구해 rules-only capability를 거절했다. |
|
||||
| `plan_local_G05_1.log` | `code_review_local_G05_1.log` | FAIL | rules-only KR daily coverage는 보강됐지만 계획상 필수였던 US/NASDAQ daily accepted 회귀 테스트가 빠졌다. |
|
||||
| `plan_local_G05_2.log` | `code_review_local_G05_2.log` | PASS | `rulesOnlyKIS`에 US/NASDAQ rule과 accepted/rejected cases가 추가되어 계획 범위가 충족됐다. |
|
||||
|
||||
## 구현/정리 내용
|
||||
|
||||
- `ProviderCapability.CheckBars`가 `Rules`만 채운 capability에서도 provider/market/venue/asset type/timeframe matrix를 평가하도록 legacy `Venues` precondition을 fallback으로 제한했다.
|
||||
- `TestProviderCapabilityMatrix`에 rules-only KR/KRX와 US/NASDAQ daily accepted, monthly/minute rejected reason 회귀 coverage를 추가했다.
|
||||
- `packages/domain/market/types_test.go` formatting drift를 gofmt로 정리했다.
|
||||
|
||||
## 최종 검증
|
||||
|
||||
- `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go` - PASS; 출력 없음.
|
||||
- `go test -count=1 ./packages/domain/market` - PASS; `ok git.toki-labs.com/toki/alt/packages/domain/market 0.003s`.
|
||||
- `go test ./packages/domain/... ./services/worker/...` - PASS; domain/worker packages passed, no-test packages reported `[no test files]`, cached output 허용.
|
||||
|
||||
## 잔여 Nit
|
||||
|
||||
- 없음
|
||||
|
||||
## 후속 작업
|
||||
|
||||
- 없음
|
||||
|
|
@ -86,103 +86,90 @@
|
|||
|
||||
## 구현 체크리스트
|
||||
|
||||
- [ ] `ProviderCapability`를 provider/market/venue/asset type/timeframe matrix request와 accepted/rejected decision을 표현하도록 확장하되, 후속 rollout 전 기존 daily call site가 컴파일될 수 있는 호환 wrapper를 유지한다.
|
||||
- [ ] domain tests에 KR/US, equity/ETF, monthly/daily/minute_1/minute_5 accepted/rejected와 rejection reason case를 추가한다.
|
||||
- [ ] `go test ./packages/domain/...`와 `go test ./services/worker/...`를 실행해 domain behavior와 기존 worker call-site compile을 확인한다.
|
||||
- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.
|
||||
- [x] `ProviderCapability`를 provider/market/venue/asset type/timeframe matrix request와 accepted/rejected decision을 표현하도록 확장하되, 후속 rollout 전 기존 daily call site가 컴파일될 수 있는 호환 wrapper를 유지한다.
|
||||
- [x] domain tests에 KR/US, equity/ETF, monthly/daily/minute_1/minute_5 accepted/rejected와 rejection reason case를 추가한다.
|
||||
- [x] `go test ./packages/domain/...`와 `go test ./services/worker/...`를 실행해 domain behavior와 기존 worker call-site compile을 확인한다.
|
||||
- [x] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.
|
||||
|
||||
### [API-1] Domain capability matrix API
|
||||
### [API-1] Domain capability matrix API — 구현 완료 확인
|
||||
|
||||
문제:
|
||||
`packages/domain/market/types.go:103`의 `ProviderCapability`는 venue/timeframe map만 갖고, `packages/domain/market/types.go:123`의 `RequireDailyBars`는 daily 전용이라 asset type과 arbitrary timeframe rejection reason을 표현할 수 없다.
|
||||
현황: `packages/domain/market/types.go`에 이미 구현됨.
|
||||
|
||||
해결 방법:
|
||||
`ProviderCapabilityRequest`, `ProviderCapabilityDecision`, `ProviderCapabilityRule`를 추가한다. `ProviderCapability.CheckBars(req)` 또는 동등한 이름의 pure function이 provider mismatch, missing venue, unknown venue, market/venue mismatch, asset type mismatch, unsupported timeframe을 stable reason으로 반환하게 한다. 기존 `RequireDailyBars(provider, selector)`는 `CheckBars`에 daily + empty asset type request를 위임하는 wrapper로 남겨 후속 rollout 전 컴파일을 유지한다.
|
||||
확인된 구현 사항:
|
||||
- `ProviderCapabilityRule` struct (market/venue/asset type/timeframe matrix rule): types.go:123
|
||||
- `ProviderCapabilityRequest` struct (call-site request): types.go:132
|
||||
- `ProviderCapabilityDecision` struct (result): types.go:141
|
||||
- `CheckBars(req)` method (full matrix evaluation): types.go:149
|
||||
- 기존 `Venues`/`Timeframes` 필드 및 `SupportsDailyBars` 메서드는 deprecated 호 compat으로 유지됨 (types.go:108-118)
|
||||
- `RequireDailyBars`는 기존 call site 호환을 위해 deprecated 메서드로 유지 (types.go:218)
|
||||
|
||||
Before:
|
||||
테스트 확인: `packages/domain/market/types_test.go`에 `TestProviderCapabilityMatrix` 존재 (types_test.go:168)
|
||||
- KR/KRX equity/ETF daily accepted
|
||||
- US/NASDAQ equity/ETF daily accepted
|
||||
- monthly/minute_1/minute_5 rejected with stable reason
|
||||
- provider mismatch, unknown venue, market/venue mismatch, no matching rule rejected
|
||||
|
||||
```go
|
||||
// packages/domain/market/types.go:107
|
||||
type ProviderCapability struct {
|
||||
Provider Provider
|
||||
Venues map[Venue]bool
|
||||
Timeframes map[Timeframe]bool
|
||||
}
|
||||
수정 파일 체크리스트:
|
||||
|
||||
// packages/domain/market/types.go:123
|
||||
func (c ProviderCapability) RequireDailyBars(provider Provider, selector UniverseSelector) error {
|
||||
```
|
||||
- [x] `packages/domain/market/types.go`: matrix request/decision/rule 타입 추가
|
||||
- [x] `packages/domain/market/types.go`: `CheckBars` 또는 동등한 판정 함수 추가
|
||||
- [x] `packages/domain/market/types.go`: 기존 `SupportsDailyBars`/`RequireDailyBars` 호환 유지
|
||||
- [x] `packages/domain/market/types.go`: rejection reason 문자열은 테스트에서 검증 가능한 stable text
|
||||
|
||||
After:
|
||||
테스트 작성 체크리스트:
|
||||
|
||||
```go
|
||||
type ProviderCapability struct {
|
||||
Provider Provider
|
||||
Rules []ProviderCapabilityRule
|
||||
}
|
||||
|
||||
type ProviderCapabilityRule struct {
|
||||
Market Market
|
||||
Venue Venue
|
||||
AssetTypes map[AssetType]bool
|
||||
Timeframes map[Timeframe]bool
|
||||
}
|
||||
|
||||
type ProviderCapabilityRequest struct {
|
||||
Provider Provider
|
||||
Market Market
|
||||
Venue Venue
|
||||
AssetType AssetType
|
||||
Timeframe Timeframe
|
||||
}
|
||||
|
||||
type ProviderCapabilityDecision struct {
|
||||
Accepted bool
|
||||
Reason string
|
||||
}
|
||||
```
|
||||
|
||||
수정 파일 및 체크리스트:
|
||||
|
||||
- [ ] `packages/domain/market/types.go`: matrix request/decision/rule 타입 추가
|
||||
- [ ] `packages/domain/market/types.go`: `CheckBars` 또는 동등한 판정 함수 추가
|
||||
- [ ] `packages/domain/market/types.go`: 기존 `SupportsDailyBars`/`RequireDailyBars`를 새 판정 함수 wrapper로 유지하거나 명시적으로 deprecate comment를 남긴다
|
||||
- [ ] `packages/domain/market/types.go`: rejection reason 문자열은 테스트에서 검증 가능한 stable text로 둔다
|
||||
|
||||
테스트 작성:
|
||||
`packages/domain/market/types_test.go`에 `TestProviderCapabilityMatrix`를 추가한다. KIS-like matrix로 KR/KRX equity daily, US/NASDAQ equity daily, US/NASDAQ ETF daily는 accepted, monthly/minute_1/minute_5는 rejected reason 포함으로 검증한다. provider mismatch, unknown venue, market/venue mismatch, unsupported asset type도 포함한다.
|
||||
- [x] `packages/domain/market/types_test.go`: `TestProviderCapabilityMatrix` 추가 및 검증
|
||||
|
||||
중간 검증:
|
||||
|
||||
```bash
|
||||
go test ./packages/domain/...
|
||||
$ go test ./packages/domain/...
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/market (cached)
|
||||
ok git.toki-labs.com/toki/alt/packages/domain/trading (cached)
|
||||
```
|
||||
|
||||
기대 결과: exit code 0.
|
||||
기대 결과: exit code 0. 실제 결과와 일치.
|
||||
|
||||
### [API-2] Compatibility compile guard
|
||||
### [API-2] Compatibility compile guard — 구현 완료 확인
|
||||
|
||||
문제:
|
||||
`services/worker/internal/jobs/marketdata_jobs.go:113`은 `capability.RequireDailyBars(...)`를 호출하고, `services/worker/internal/providers/kis/daily_itemchartprice.go:32`는 기존 struct literal을 반환한다. domain foundation 변경이 worker rollout보다 먼저 merge되면 worker compile이 깨질 수 있다.
|
||||
현황: 기존 `Venues`/`Timeframes` 필드와 `SupportsDailyBars` 메서드가 deprecated 호환으로 유지됨.
|
||||
|
||||
해결 방법:
|
||||
domain API에 compatibility wrapper와 old-style construction migration path를 함께 둔다. old `Venues`/`Timeframes` 필드를 제거하지 않고 deprecated 호환 필드로 유지하거나, same-plan에서 worker helper test constructors까지 최소 수정한다. 후속 task가 KIS/job gate를 새 matrix API로 전환한다.
|
||||
확인된 call site:
|
||||
- `services/worker/internal/jobs/marketdata_jobs.go:113`: `capability.RequireDailyBars(...)` 호출 — 컴파일됨
|
||||
- `services/worker/internal/providers/kis/daily_itemchartprice.go:32`: 기존 struct literal 반환 — 컴파일됨
|
||||
- `services/worker/internal/jobs/marketdata_jobs_test.go:28-37`: `kisCapability()` 기존 struct literal — 컴파일됨
|
||||
- `services/worker/cmd/alt-worker/main.go:84`: `kis.Capability()` 사용 — 컴파일됨
|
||||
|
||||
수정 파일 및 체크리스트:
|
||||
수정 파일 체크리스트:
|
||||
|
||||
- [ ] `packages/domain/market/types.go`: old call site compatibility 전략 적용
|
||||
- [ ] `packages/domain/market/types_test.go`: old `RequireDailyBars` wrapper가 daily accepted/rejected를 유지하는 회귀 테스트 보존
|
||||
|
||||
테스트 작성:
|
||||
기존 `TestProviderCapabilityRequireDailyBars`를 유지하거나 matrix API 기반으로 갱신하되, daily wrapper의 기존 KR/US accepted와 provider/venue rejection이 유지되는 assertion을 둔다.
|
||||
- [x] `packages/domain/market/types.go`: old call site compatibility 전략 적용 (Venues/Timeframes 필드 병행 유지)
|
||||
- [x] `packages/domain/market/types_test.go`: old `RequireDailyBars` wrapper 회귀 테스트 보존 (TestProviderCapabilityRequireDailyBars:99)
|
||||
|
||||
중간 검증:
|
||||
|
||||
```bash
|
||||
go test ./services/worker/...
|
||||
$ go test ./services/worker/...
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-data-check [no test files]
|
||||
ok git.toki-labs.com/toki/alt/services/worker/cmd/alt-worker-migrate (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/backtest (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/config (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/contracts (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/jobs (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/livetrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/datacheck (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/marketdata/importer (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/papertrading (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/providers/kis (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/rediskeys (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/socket (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage (cached)
|
||||
ok git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres (cached)
|
||||
? git.toki-labs.com/toki/alt/services/worker/internal/storage/postgres/sqlc [no test files]
|
||||
```
|
||||
|
||||
기대 결과: exit code 0.
|
||||
기대 결과: exit code 0. 실제 결과와 일치.
|
||||
|
||||
## 수정 파일 요약
|
||||
|
||||
|
|
@ -0,0 +1,204 @@
|
|||
<!-- task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix plan=1 tag=REVIEW_API -->
|
||||
|
||||
# Plan - REVIEW_API
|
||||
|
||||
## 이 파일을 읽는 구현 에이전트에게
|
||||
|
||||
`CODE_REVIEW-local-G05.md`의 구현 에이전트 소유 섹션을 채우는 것이 구현의 마지막 단계다. 코드를 바꾸고 검증을 실행한 뒤 실제 구현 내용과 stdout/stderr를 리뷰 stub에 남기고, active 파일은 그대로 둔 채 리뷰 준비를 보고한다. 선택된 SDD 결정 또는 Milestone `구현 잠금 > 결정 필요` 항목이 막으면 리뷰 stub의 `사용자 리뷰 요청` 섹션에 정확한 연결 대상, 근거, 명령 출력, 재개 조건을 기록하고 멈춘다. 구현 중 사용자에게 직접 질문하거나 선택지를 제시하거나 `request_user_input`, `USER_REVIEW.md` 작성, log archive, `complete.log` 작성을 하지 않는다. 환경/secret/service 차단과 검증 증거 공백은 사용자 리뷰가 아니라 `검증 결과` 또는 후속 plan 대상으로 남긴다.
|
||||
|
||||
## 배경
|
||||
|
||||
첫 리뷰에서 `CheckBars`가 새 matrix `Rules`만으로 동작하지 않고 deprecated `Venues` map까지 요구하는 결함이 확인됐다. 이 작업은 matrix API foundation을 self-contained하게 만들고, 같은 결함이 다시 통과하지 않도록 rules-only 회귀 테스트를 추가한다. worker/KIS rollout은 sibling `02+01_kis_job_capability` 범위로 남긴다.
|
||||
|
||||
## 사용자 리뷰 요청 흐름
|
||||
|
||||
사용자 리뷰 요청은 선택된 SDD 결정 또는 선택된 Milestone lock 결정이 실구현을 차단할 때만 active `CODE_REVIEW-local-G05.md`의 `사용자 리뷰 요청` 섹션에 기록한다. 직접 chat 질문은 금지이며, code-review가 검증 후 실제 `USER_REVIEW.md` 작성 여부를 판단한다.
|
||||
|
||||
## Archive Evidence Snapshot
|
||||
|
||||
- Archived plan: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_0.log`
|
||||
- Archived review: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_0.log`
|
||||
- Verdict: FAIL
|
||||
- Required summary:
|
||||
- `packages/domain/market/types.go:156`에서 `CheckBars`가 `Rules`를 평가하기 전에 deprecated `c.Venues`를 필수 precondition으로 사용해, `Rules`만 채운 capability를 항상 unsupported venue로 거절한다.
|
||||
- `packages/domain/market/types_test.go:168`의 matrix test는 `Venues`/`Timeframes`도 함께 채워 이 결함을 가린다.
|
||||
- Affected files:
|
||||
- `packages/domain/market/types.go`
|
||||
- `packages/domain/market/types_test.go`
|
||||
- Verification evidence:
|
||||
- 리뷰 중 `go test ./packages/domain/... ./services/worker/...`는 exit code 0으로 통과했으나, 현 테스트는 rules-only capability path를 검증하지 않는다.
|
||||
- Narrow reread allowed if needed:
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_0.log`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_0.log`
|
||||
|
||||
## 분석 결과
|
||||
|
||||
### 읽은 파일
|
||||
|
||||
- `agent-ops/rules/project/rules.md`
|
||||
- `agent-ops/rules/private/rules.md`
|
||||
- `agent-ops/rules/common/rules-roadmap.md`
|
||||
- `agent-roadmap/current.md`
|
||||
- `agent-ops/skills/common/router.md`
|
||||
- `agent-ops/skills/common/code-review/SKILL.md`
|
||||
- `agent-ops/skills/common/plan/SKILL.md`
|
||||
- `agent-ops/skills/common/_templates/implementation-user-review-request-section.md`
|
||||
- `agent-ops/rules/project/domain/domain-model/rules.md`
|
||||
- `agent-ops/rules/project/domain/worker/rules.md`
|
||||
- `agent-test/local/rules.md`
|
||||
- `agent-test/local/domain-model-smoke.md`
|
||||
- `agent-test/local/worker-smoke.md`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_0.log`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_0.log`
|
||||
- `packages/domain/market/types.go`
|
||||
- `packages/domain/market/types_test.go`
|
||||
- `services/worker/internal/jobs/marketdata_jobs.go`
|
||||
- `services/worker/internal/jobs/marketdata_jobs_test.go`
|
||||
- `services/worker/internal/providers/kis/daily_itemchartprice.go`
|
||||
- `services/worker/cmd/alt-worker/main.go`
|
||||
|
||||
### 테스트 환경 규칙
|
||||
|
||||
- test_env: `local`
|
||||
- env rules: `agent-test/local/rules.md` 존재 및 정독 완료
|
||||
- matched profiles: `agent-test/local/domain-model-smoke.md`, `agent-test/local/worker-smoke.md`
|
||||
- 적용 명령:
|
||||
- bug fix 회귀 확인: `go test -count=1 ./packages/domain/market`
|
||||
- domain/worker compatibility 확인: `go test ./packages/domain/... ./services/worker/...`
|
||||
- Go test cache 정책: 첫 번째 회귀 명령은 fresh 실행을 위해 `-count=1`을 사용한다. 두 번째 compatibility 명령은 cache output 허용.
|
||||
- fallback: 없음
|
||||
|
||||
### 테스트 커버리지 공백
|
||||
|
||||
- 기존 `TestProviderCapabilityMatrix`는 `ProviderCapability.Rules`와 함께 legacy `Venues`/`Timeframes`도 채워서, 새 matrix API가 rules-only capability에서도 작동하는지 검증하지 않는다.
|
||||
- 후속 작업은 rules-only capability daily accepted 및 monthly/minute rejected reason을 반드시 검증한다.
|
||||
|
||||
### 심볼 참조
|
||||
|
||||
- 변경 대상 심볼: `market.ProviderCapability.CheckBars`
|
||||
- renamed/removed symbols: 없음
|
||||
- 관련 call sites:
|
||||
- `packages/domain/market/types_test.go`
|
||||
- `services/worker/internal/jobs/marketdata_jobs.go`는 아직 `RequireDailyBars` 호환 wrapper를 사용한다.
|
||||
- `services/worker/internal/providers/kis/daily_itemchartprice.go`는 현재 legacy `Venues`/`Timeframes` struct literal을 반환하며 worker rollout은 이 작업 범위 밖이다.
|
||||
|
||||
### 분할 판단
|
||||
|
||||
- split decision policy를 후속 plan 작성 전에 적용했다.
|
||||
- 기존 split task group `agent-task/m-backtest-multi-timeframe-coverage/`와 subtask `01_domain_capability_matrix`를 유지한다.
|
||||
- 이 후속은 선행 subtask 내부의 Required 리뷰 이슈만 고치는 작업이며, API foundation과 worker rollout을 다시 섞지 않는다.
|
||||
- 수정 범위가 `types.go`와 `types_test.go`로 제한되어 추가 split은 필요하지 않다.
|
||||
|
||||
### 범위 결정 근거
|
||||
|
||||
- 포함: `packages/domain/market/types.go`, `packages/domain/market/types_test.go`
|
||||
- 제외: `services/worker/**`, `services/worker/internal/providers/kis/**`, `.proto`, API socket, CLI runner, 실제 KIS capability table rollout. worker/KIS 적용은 sibling `02+01_kis_job_capability`가 담당한다.
|
||||
|
||||
### 빌드 등급
|
||||
|
||||
- build: `local-G05`
|
||||
- review: `local-G05`
|
||||
- 근거: 변경은 두 domain 파일로 좁지만 public domain API의 self-contained behavior와 회귀 테스트를 함께 보정해야 한다.
|
||||
|
||||
## 구현 체크리스트
|
||||
|
||||
- [ ] `CheckBars`가 `Rules`만 채운 `ProviderCapability`에서도 provider/market/venue/asset type/timeframe 판정을 수행하도록 legacy `Venues` precondition을 수정한다.
|
||||
- [ ] domain tests에 rules-only capability 회귀 케이스를 추가해 KR/US daily accepted와 monthly/minute rejected reason을 검증한다.
|
||||
- [ ] `go test -count=1 ./packages/domain/market`와 `go test ./packages/domain/... ./services/worker/...`를 실행해 domain behavior와 기존 worker call-site compile을 확인한다.
|
||||
- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.
|
||||
|
||||
### [REVIEW_API-1] Make matrix capability self-contained
|
||||
|
||||
문제:
|
||||
- `packages/domain/market/types.go:156`에서 `CheckBars`가 `Rules` 평가 전에 `c.Venues[req.Venue]`를 검사한다. 이 때문에 새 matrix API 사용자가 legacy compatibility map을 채우지 않으면 rule이 있어도 거절된다.
|
||||
- `packages/domain/market/types_test.go:168`의 현재 test fixture는 legacy map까지 함께 채워 이 결함을 잡지 못한다.
|
||||
|
||||
해결 방법:
|
||||
- `CheckBars`의 venue support 판단을 `Rules` 기반으로 옮긴다.
|
||||
- legacy `Venues` map을 유지해야 하면 `len(c.Venues) > 0`일 때만 compatibility prefilter로 사용하거나, rules matching 결과가 없을 때의 legacy fallback으로만 사용한다.
|
||||
- provider mismatch, unknown venue, market/venue mismatch reason은 기존 stable text를 유지한다.
|
||||
|
||||
Before (`packages/domain/market/types.go:149`):
|
||||
|
||||
```go
|
||||
func (c ProviderCapability) CheckBars(req ProviderCapabilityRequest) ProviderCapabilityDecision {
|
||||
if req.Provider != c.Provider {
|
||||
return ProviderCapabilityDecision{
|
||||
Accepted: false,
|
||||
Reason: fmt.Sprintf("provider %q does not match capability provider %q", req.Provider, c.Provider),
|
||||
}
|
||||
}
|
||||
if !c.Venues[req.Venue] {
|
||||
return ProviderCapabilityDecision{
|
||||
Accepted: false,
|
||||
Reason: fmt.Sprintf("provider %q does not support venue %q", req.Provider, req.Venue),
|
||||
}
|
||||
}
|
||||
```
|
||||
|
||||
After 방향:
|
||||
|
||||
```go
|
||||
func (c ProviderCapability) CheckBars(req ProviderCapabilityRequest) ProviderCapabilityDecision {
|
||||
if req.Provider != c.Provider {
|
||||
return ProviderCapabilityDecision{
|
||||
Accepted: false,
|
||||
Reason: fmt.Sprintf("provider %q does not match capability provider %q", req.Provider, c.Provider),
|
||||
}
|
||||
}
|
||||
meta, ok := GetVenueMetadata(req.Venue)
|
||||
if !ok {
|
||||
return ProviderCapabilityDecision{
|
||||
Accepted: false,
|
||||
Reason: fmt.Sprintf("provider %q does not support venue %q", req.Provider, req.Venue),
|
||||
}
|
||||
}
|
||||
if len(c.Venues) > 0 && !c.Venues[req.Venue] {
|
||||
return ProviderCapabilityDecision{
|
||||
Accepted: false,
|
||||
Reason: fmt.Sprintf("provider %q does not support venue %q", req.Provider, req.Venue),
|
||||
}
|
||||
}
|
||||
// Continue with market validation and rule matching.
|
||||
}
|
||||
```
|
||||
|
||||
수정 파일 및 체크리스트:
|
||||
- [ ] `packages/domain/market/types.go`: `CheckBars`가 rules-only capability를 허용하도록 precondition 순서와 조건을 수정한다.
|
||||
- [ ] `packages/domain/market/types.go`: 기존 stable rejection reason을 유지한다.
|
||||
- [ ] `packages/domain/market/types_test.go`: rules-only capability fixture를 사용한 accepted/rejected regression을 추가한다.
|
||||
|
||||
테스트 작성:
|
||||
- 작성: `packages/domain/market/types_test.go`
|
||||
- test name: `TestProviderCapabilityMatrixRulesOnly` 또는 기존 `TestProviderCapabilityMatrix` 내 명확한 subcases
|
||||
- assertion goal:
|
||||
- `ProviderCapability{Provider: ProviderKIS, Rules: ...}`만으로 KR/KRX equity daily accepted
|
||||
- US/NASDAQ ETF daily accepted
|
||||
- monthly, `TimeframeMin1`, `TimeframeMin5` rejected reason 유지
|
||||
- unknown venue와 no matching rule reason 유지
|
||||
|
||||
중간 검증:
|
||||
|
||||
```bash
|
||||
go test -count=1 ./packages/domain/market
|
||||
```
|
||||
|
||||
기대 결과: exit code 0.
|
||||
|
||||
## 수정 파일 요약
|
||||
|
||||
| 파일 | 항목 |
|
||||
|------|------|
|
||||
| `packages/domain/market/types.go` | REVIEW_API-1 |
|
||||
| `packages/domain/market/types_test.go` | REVIEW_API-1 |
|
||||
|
||||
## 최종 검증
|
||||
|
||||
```bash
|
||||
go test -count=1 ./packages/domain/market
|
||||
go test ./packages/domain/... ./services/worker/...
|
||||
```
|
||||
|
||||
기대 결과: 두 명령 모두 exit code 0. 첫 번째 명령은 fresh 실행이어야 하며, 두 번째 명령의 Go test cache output은 허용한다.
|
||||
|
||||
모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다.
|
||||
|
|
@ -0,0 +1,206 @@
|
|||
<!-- task=m-backtest-multi-timeframe-coverage/01_domain_capability_matrix plan=2 tag=REVIEW_REVIEW_API -->
|
||||
|
||||
# Plan - REVIEW_REVIEW_API
|
||||
|
||||
## 이 파일을 읽는 구현 에이전트에게
|
||||
|
||||
`CODE_REVIEW-local-G05.md`의 구현 에이전트 소유 섹션을 채우는 것이 구현의 마지막 단계다. 코드를 바꾸고 검증을 실행한 뒤 실제 구현 내용과 stdout/stderr를 리뷰 stub에 남기고, active 파일은 그대로 둔 채 리뷰 준비를 보고한다. 선택된 SDD 결정 또는 Milestone `구현 잠금 > 결정 필요` 항목이 막으면 리뷰 stub의 `사용자 리뷰 요청` 섹션에 정확한 연결 대상, 근거, 명령 출력, 재개 조건을 기록하고 멈춘다. 구현 중 사용자에게 직접 질문하거나 선택지를 제시하거나 `request_user_input`, `USER_REVIEW.md` 작성, log archive, `complete.log` 작성을 하지 않는다. 환경/secret/service 차단과 검증 증거 공백은 사용자 리뷰가 아니라 `검증 결과` 또는 후속 plan 대상으로 남긴다.
|
||||
|
||||
## 배경
|
||||
|
||||
두 번째 리뷰에서 `CheckBars`의 rules-only 동작 자체는 고쳐졌지만, 계획이 요구한 KR/US daily accepted rules-only 회귀 테스트 중 US/NASDAQ accepted 케이스가 빠졌다. 이 후속은 테스트 fixture와 case만 보강해 rules-only capability가 KIS의 다중 시장 coverage를 검증하도록 만든다.
|
||||
|
||||
## 사용자 리뷰 요청 흐름
|
||||
|
||||
사용자 리뷰 요청은 선택된 SDD 결정 또는 선택된 Milestone lock 결정이 실구현을 차단할 때만 active `CODE_REVIEW-local-G05.md`의 `사용자 리뷰 요청` 섹션에 기록한다. 직접 chat 질문은 금지이며, code-review가 검증 후 실제 `USER_REVIEW.md` 작성 여부를 판단한다.
|
||||
|
||||
## Archive Evidence Snapshot
|
||||
|
||||
- Archived plan: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_1.log`
|
||||
- Archived review: `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_1.log`
|
||||
- Verdict: FAIL
|
||||
- Required summary:
|
||||
- `packages/domain/market/types_test.go:217`의 `rulesOnlyKIS` fixture가 KR/KRX rule만 포함한다.
|
||||
- `packages/domain/market/types_test.go:305` 이후 rules-only accepted case도 KR daily만 검증해, 계획상 필수였던 US/NASDAQ daily accepted 회귀 검증이 빠졌다.
|
||||
- Affected files:
|
||||
- `packages/domain/market/types_test.go`
|
||||
- Verification evidence:
|
||||
- 리뷰 중 `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`가 처음에는 `packages/domain/market/types_test.go`를 출력했고, 리뷰어가 `gofmt -w packages/domain/market/types_test.go`로 포맷 drift를 직접 정리했다.
|
||||
- 포맷 정리 후 `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`는 출력 없음.
|
||||
- `go test -count=1 ./packages/domain/market` exit code 0.
|
||||
- `go test ./packages/domain/... ./services/worker/...` exit code 0.
|
||||
- Narrow reread allowed if needed:
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/code_review_local_G05_1.log`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/plan_local_G05_1.log`
|
||||
|
||||
## 분석 결과
|
||||
|
||||
### 읽은 파일
|
||||
|
||||
- `AGENTS.md`
|
||||
- `agent-ops/rules/project/rules.md`
|
||||
- `agent-ops/rules/private/rules.md`
|
||||
- `agent-ops/rules/common/rules-roadmap.md`
|
||||
- `agent-roadmap/current.md`
|
||||
- `agent-ops/skills/common/router.md`
|
||||
- `agent-ops/skills/common/code-review/SKILL.md`
|
||||
- `agent-ops/skills/common/plan/SKILL.md`
|
||||
- `agent-ops/skills/common/_templates/implementation-user-review-request-section.md`
|
||||
- `agent-ops/rules/project/domain/domain-model/rules.md`
|
||||
- `agent-ops/rules/project/domain/worker/rules.md`
|
||||
- `agent-test/local/rules.md`
|
||||
- `agent-test/local/domain-model-smoke.md`
|
||||
- `agent-test/local/worker-smoke.md`
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/PLAN-local-G05.md` (archived to `plan_local_G05_1.log`)
|
||||
- `agent-task/m-backtest-multi-timeframe-coverage/01_domain_capability_matrix/CODE_REVIEW-local-G05.md` (archived to `code_review_local_G05_1.log`)
|
||||
- `packages/domain/market/types.go`
|
||||
- `packages/domain/market/types_test.go`
|
||||
- `services/worker/internal/providers/kis/daily_itemchartprice.go`
|
||||
- `services/worker/internal/jobs/marketdata_jobs.go`
|
||||
|
||||
### 테스트 환경 규칙
|
||||
|
||||
- test_env: `local`
|
||||
- env rules: `agent-test/local/rules.md` 존재 및 정독 완료
|
||||
- matched profiles: `agent-test/local/domain-model-smoke.md`; worker compatibility 검증 때문에 `agent-test/local/worker-smoke.md`도 읽음
|
||||
- 적용 명령:
|
||||
- test-only regression 확인: `go test -count=1 ./packages/domain/market`
|
||||
- compatibility 확인: `go test ./packages/domain/... ./services/worker/...`
|
||||
- formatting 확인: `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`
|
||||
- Go test cache 정책: 첫 번째 회귀 명령은 fresh 실행을 위해 `-count=1`을 사용한다. 두 번째 compatibility 명령의 cache output은 허용한다.
|
||||
- fallback: 없음
|
||||
|
||||
### 테스트 커버리지 공백
|
||||
|
||||
- `CheckBars` rules-only KR/KRX daily accepted와 monthly/minute rejected reason은 현재 테스트가 검증한다.
|
||||
- `CheckBars` rules-only US/NASDAQ daily accepted는 현재 테스트가 검증하지 않는다. 이번 후속에서 `rulesOnlyKIS`에 US/NASDAQ rule을 추가하고 US/NASDAQ ETF daily accepted case를 추가한다.
|
||||
|
||||
### 심볼 참조
|
||||
|
||||
- renamed/removed symbols: 없음
|
||||
- 관련 심볼: `market.ProviderCapability.CheckBars`
|
||||
- 관련 call sites:
|
||||
- `packages/domain/market/types_test.go`
|
||||
- `services/worker/internal/jobs/marketdata_jobs.go`는 아직 `RequireDailyBars` compatibility wrapper를 사용한다.
|
||||
- `services/worker/internal/providers/kis/daily_itemchartprice.go`는 legacy `Venues`/`Timeframes` capability를 반환하며 이번 후속 범위 밖이다.
|
||||
|
||||
### 분할 판단
|
||||
|
||||
- split decision policy를 후속 plan 작성 전에 적용했다.
|
||||
- 기존 split task group `agent-task/m-backtest-multi-timeframe-coverage/`와 subtask `01_domain_capability_matrix`를 유지한다.
|
||||
- 변경은 `packages/domain/market/types_test.go`의 fixture와 case 보강으로 제한되어 별도 subtask로 쪼개지 않는다.
|
||||
|
||||
### 범위 결정 근거
|
||||
|
||||
- 포함: `packages/domain/market/types_test.go`
|
||||
- 제외: `packages/domain/market/types.go`, `services/worker/**`, `services/worker/internal/providers/kis/**`, `.proto`, API socket, CLI runner, 실제 KIS capability table rollout. 이번 후속은 누락된 회귀 테스트만 보강한다.
|
||||
|
||||
### 빌드 등급
|
||||
|
||||
- build: `local-G05`
|
||||
- review: `local-G05`
|
||||
- 근거: 변경은 테스트 한 파일로 좁지만 public domain capability matrix의 필수 회귀 조건을 닫는 작업이다.
|
||||
|
||||
## 구현 체크리스트
|
||||
|
||||
- [ ] `rulesOnlyKIS` fixture에 `MarketUS`/`VenueNASDAQ` daily equity/ETF rule을 추가한다.
|
||||
- [ ] rules-only test cases에 US/NASDAQ ETF daily accepted case를 추가해 `Venues` 없이 US daily path가 accepted 되는지 검증한다.
|
||||
- [ ] `gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go`, `go test -count=1 ./packages/domain/market`, `go test ./packages/domain/... ./services/worker/...`를 실행해 formatting, domain regression, worker compatibility를 확인한다.
|
||||
- [ ] CODE_REVIEW-*-G??.md의 구현 에이전트 소유 섹션을 실제 구현 내용과 검증 출력으로 채운다. 이 항목이 완료되기 전에는 구현이 완료된 것이 아니다.
|
||||
|
||||
### [REVIEW_REVIEW_API-1] Cover rules-only US daily accepted
|
||||
|
||||
문제:
|
||||
- `packages/domain/market/types_test.go:217`의 `rulesOnlyKIS` fixture는 KR/KRX rule만 포함한다.
|
||||
- `packages/domain/market/types_test.go:305` 이후 rules-only accepted case는 KR/KRX equity/ETF daily만 검증한다.
|
||||
- 이전 계획은 rules-only capability에서 KR/US daily accepted를 모두 요구했으므로 US/NASDAQ daily accepted 회귀 coverage가 아직 닫히지 않았다.
|
||||
|
||||
해결 방법:
|
||||
- `rulesOnlyKIS.Rules`에 `MarketUS`/`VenueNASDAQ` rule을 추가한다.
|
||||
- 해당 rule의 `AssetTypes`는 `AssetTypeEquity`, `AssetTypeETF`를 true로 두고, `Timeframes`는 daily true, monthly/minute false로 기존 KIS matrix와 맞춘다.
|
||||
- `rulesOnlyCases`에 US/NASDAQ ETF daily accepted case를 추가한다.
|
||||
|
||||
Before (`packages/domain/market/types_test.go:220`):
|
||||
|
||||
```go
|
||||
Rules: []ProviderCapabilityRule{
|
||||
{
|
||||
Market: MarketKR,
|
||||
Venue: VenueKRX,
|
||||
AssetTypes: map[AssetType]bool{
|
||||
AssetTypeEquity: true,
|
||||
AssetTypeETF: true,
|
||||
},
|
||||
Timeframes: map[Timeframe]bool{
|
||||
TimeframeDaily: true,
|
||||
TimeframeMonthly: false,
|
||||
TimeframeMin1: false,
|
||||
TimeframeMin5: false,
|
||||
},
|
||||
},
|
||||
},
|
||||
```
|
||||
|
||||
After 방향:
|
||||
|
||||
```go
|
||||
Rules: []ProviderCapabilityRule{
|
||||
{
|
||||
Market: MarketKR,
|
||||
Venue: VenueKRX,
|
||||
// unchanged
|
||||
},
|
||||
{
|
||||
Market: MarketUS,
|
||||
Venue: VenueNASDAQ,
|
||||
AssetTypes: map[AssetType]bool{
|
||||
AssetTypeEquity: true,
|
||||
AssetTypeETF: true,
|
||||
},
|
||||
Timeframes: map[Timeframe]bool{
|
||||
TimeframeDaily: true,
|
||||
TimeframeMonthly: false,
|
||||
TimeframeMin1: false,
|
||||
TimeframeMin5: false,
|
||||
},
|
||||
},
|
||||
}
|
||||
```
|
||||
|
||||
수정 파일 및 체크리스트:
|
||||
- [ ] `packages/domain/market/types_test.go`: `rulesOnlyKIS`에 US/NASDAQ rule을 추가한다.
|
||||
- [ ] `packages/domain/market/types_test.go`: `rulesOnlyCases`에 US/NASDAQ ETF daily accepted case를 추가한다.
|
||||
|
||||
테스트 작성:
|
||||
- 작성: `packages/domain/market/types_test.go`
|
||||
- test name: 기존 `TestProviderCapabilityMatrix`의 `rules-only/...` subcase
|
||||
- assertion goal:
|
||||
- `ProviderCapability{Provider: ProviderKIS, Venues: empty, Rules: ...}`만으로 US/NASDAQ ETF daily accepted
|
||||
- 기존 KR accepted와 monthly/minute rejected reason은 계속 통과
|
||||
|
||||
중간 검증:
|
||||
|
||||
```bash
|
||||
gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go
|
||||
go test -count=1 ./packages/domain/market
|
||||
```
|
||||
|
||||
기대 결과: `gofmt -l` 출력 없음, `go test` exit code 0.
|
||||
|
||||
## 수정 파일 요약
|
||||
|
||||
| 파일 | 항목 |
|
||||
|------|------|
|
||||
| `packages/domain/market/types_test.go` | REVIEW_REVIEW_API-1 |
|
||||
|
||||
## 최종 검증
|
||||
|
||||
```bash
|
||||
gofmt -l packages/domain/market/types.go packages/domain/market/types_test.go
|
||||
go test -count=1 ./packages/domain/market
|
||||
go test ./packages/domain/... ./services/worker/...
|
||||
```
|
||||
|
||||
기대 결과: `gofmt -l` 출력 없음. 두 Go test 명령 모두 exit code 0. 첫 번째 Go test는 fresh 실행이어야 하며, 두 번째 compatibility 명령의 Go test cache output은 허용한다.
|
||||
|
||||
모든 코드 변경 완료 후 반드시 `CODE_REVIEW-*-G??.md`의 구현 에이전트 소유 섹션을 채운다. 이 파일 작성이 구현의 마지막 단계다.
|
||||
|
|
@ -153,7 +153,10 @@ func (c ProviderCapability) CheckBars(req ProviderCapabilityRequest) ProviderCap
|
|||
Reason: fmt.Sprintf("provider %q does not match capability provider %q", req.Provider, c.Provider),
|
||||
}
|
||||
}
|
||||
if !c.Venues[req.Venue] {
|
||||
// Legacy Venues fallback: if Rules is empty, check Venues for backward
|
||||
// compatibility. When Rules is populated, venue presence is validated by
|
||||
// the rule matcher below, so the legacy Venues map is no longer required.
|
||||
if len(c.Rules) == 0 && !c.Venues[req.Venue] {
|
||||
return ProviderCapabilityDecision{
|
||||
Accepted: false,
|
||||
Reason: fmt.Sprintf("provider %q does not support venue %q", req.Provider, req.Venue),
|
||||
|
|
|
|||
|
|
@ -212,11 +212,48 @@ func TestProviderCapabilityMatrix(t *testing.T) {
|
|||
},
|
||||
}
|
||||
|
||||
// rulesOnlyKIS is a capability with NO Venues map — only Rules.
|
||||
// This is the core regression test for REVIEW_API-1.
|
||||
rulesOnlyKIS := ProviderCapability{
|
||||
Provider: ProviderKIS,
|
||||
Venues: map[Venue]bool{},
|
||||
Rules: []ProviderCapabilityRule{
|
||||
{
|
||||
Market: MarketKR,
|
||||
Venue: VenueKRX,
|
||||
AssetTypes: map[AssetType]bool{
|
||||
AssetTypeEquity: true,
|
||||
AssetTypeETF: true,
|
||||
},
|
||||
Timeframes: map[Timeframe]bool{
|
||||
TimeframeDaily: true,
|
||||
TimeframeMonthly: false,
|
||||
TimeframeMin1: false,
|
||||
TimeframeMin5: false,
|
||||
},
|
||||
},
|
||||
{
|
||||
Market: MarketUS,
|
||||
Venue: VenueNASDAQ,
|
||||
AssetTypes: map[AssetType]bool{
|
||||
AssetTypeEquity: true,
|
||||
AssetTypeETF: true,
|
||||
},
|
||||
Timeframes: map[Timeframe]bool{
|
||||
TimeframeDaily: true,
|
||||
TimeframeMonthly: false,
|
||||
TimeframeMin1: false,
|
||||
TimeframeMin5: false,
|
||||
},
|
||||
},
|
||||
},
|
||||
}
|
||||
|
||||
cases := []struct {
|
||||
name string
|
||||
req ProviderCapabilityRequest
|
||||
name string
|
||||
req ProviderCapabilityRequest
|
||||
wantAccepted bool
|
||||
wantReason string
|
||||
wantReason string
|
||||
}{
|
||||
// Accepted cases
|
||||
{"KIS KR KRX equity daily", ProviderCapabilityRequest{
|
||||
|
|
@ -272,6 +309,59 @@ func TestProviderCapabilityMatrix(t *testing.T) {
|
|||
}, false, "no capability rule matches market \"US\" venue \"NYSE\" asset type \"equity\" timeframe \"1d\""},
|
||||
}
|
||||
|
||||
// rules-only regression test: REVIEW_API-1 — Venues must not be required.
|
||||
rulesOnlyCases := []struct {
|
||||
name string
|
||||
req ProviderCapabilityRequest
|
||||
wantAccepted bool
|
||||
wantReason string
|
||||
}{
|
||||
// KR accepted
|
||||
{"rules-only KR KRX equity daily accepted", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketKR, Venue: VenueKRX,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeDaily,
|
||||
}, true, ""},
|
||||
{"rules-only KR KRX etf daily accepted", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketKR, Venue: VenueKRX,
|
||||
AssetType: AssetTypeETF, Timeframe: TimeframeDaily,
|
||||
}, true, ""},
|
||||
// KR rejected
|
||||
{"rules-only KR KRX equity monthly rejected", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketKR, Venue: VenueKRX,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeMonthly,
|
||||
}, false, "provider \"kis\" does not support asset type \"equity\" timeframe \"1mo\" on venue \"KRX\""},
|
||||
{"rules-only KR KRX equity minute_1 rejected", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketKR, Venue: VenueKRX,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeMin1,
|
||||
}, false, "provider \"kis\" does not support asset type \"equity\" timeframe \"1m\" on venue \"KRX\""},
|
||||
{"rules-only KR KRX equity minute_5 rejected", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketKR, Venue: VenueKRX,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeMin5,
|
||||
}, false, "provider \"kis\" does not support asset type \"equity\" timeframe \"5m\" on venue \"KRX\""},
|
||||
// US/NASDAQ accepted (REVIEW_REVIEW_API - US daily coverage)
|
||||
{"rules-only US NASDAQ equity daily accepted", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketUS, Venue: VenueNASDAQ,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeDaily,
|
||||
}, true, ""},
|
||||
{"rules-only US NASDAQ etf daily accepted", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketUS, Venue: VenueNASDAQ,
|
||||
AssetType: AssetTypeETF, Timeframe: TimeframeDaily,
|
||||
}, true, ""},
|
||||
// US/NASDAQ rejected
|
||||
{"rules-only US NASDAQ equity monthly rejected", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketUS, Venue: VenueNASDAQ,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeMonthly,
|
||||
}, false, "provider \"kis\" does not support asset type \"equity\" timeframe \"1mo\" on venue \"NASDAQ\""},
|
||||
{"rules-only US NASDAQ equity minute_1 rejected", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketUS, Venue: VenueNASDAQ,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeMin1,
|
||||
}, false, "provider \"kis\" does not support asset type \"equity\" timeframe \"1m\" on venue \"NASDAQ\""},
|
||||
{"rules-only US NASDAQ equity minute_5 rejected", ProviderCapabilityRequest{
|
||||
Provider: ProviderKIS, Market: MarketUS, Venue: VenueNASDAQ,
|
||||
AssetType: AssetTypeEquity, Timeframe: TimeframeMin5,
|
||||
}, false, "provider \"kis\" does not support asset type \"equity\" timeframe \"5m\" on venue \"NASDAQ\""},
|
||||
}
|
||||
|
||||
for _, c := range cases {
|
||||
t.Run(c.name, func(t *testing.T) {
|
||||
dec := kis.CheckBars(c.req)
|
||||
|
|
@ -283,4 +373,17 @@ func TestProviderCapabilityMatrix(t *testing.T) {
|
|||
}
|
||||
})
|
||||
}
|
||||
|
||||
// rules-only regression test
|
||||
for _, c := range rulesOnlyCases {
|
||||
t.Run("rules-only/"+c.name, func(t *testing.T) {
|
||||
dec := rulesOnlyKIS.CheckBars(c.req)
|
||||
if dec.Accepted != c.wantAccepted {
|
||||
t.Errorf("CheckBars(%q) accepted: got %v, want %v", c.name, dec.Accepted, c.wantAccepted)
|
||||
}
|
||||
if c.wantReason != "" && dec.Reason != c.wantReason {
|
||||
t.Errorf("CheckBars(%q) reason: got %q, want %q", c.name, dec.Reason, c.wantReason)
|
||||
}
|
||||
})
|
||||
}
|
||||
}
|
||||
|
|
|
|||
Loading…
Reference in a new issue