From 50beba3ff6c9a40b68f5c75fc8da3b28908eab31 Mon Sep 17 00:00:00 2001 From: Justin Ko Date: Sat, 6 Jun 2026 17:51:30 +0900 Subject: [PATCH] docs: fold issue #9 user_info hotfix into REL-001 scope + 2026-06-06 smoke - Add issue #9 hotfix (GET /v1/user_info 500 on missing broker_credentials) to REL-001 Included scope + Revision (6b812b6 as integrated main HEAD). - Add post-deploy smoke item for /v1/user_info 200 + daemon_id (local verified). - Record 2026-06-06 local verification: ledger_trades absent / portfolio_ledger consolidated schema, legacy buy/sell OpenAPI deprecated, full suite 377 passed. Co-Authored-By: Claude Opus 4.8 --- .../rel-001_20260521_v1-api-full-surface.md | 17 ++++++++++------- 1 file changed, 10 insertions(+), 7 deletions(-) diff --git a/docs/releases/rel-001_20260521_v1-api-full-surface.md b/docs/releases/rel-001_20260521_v1-api-full-surface.md index e61364e..9ac04b2 100644 --- a/docs/releases/rel-001_20260521_v1-api-full-surface.md +++ b/docs/releases/rel-001_20260521_v1-api-full-surface.md @@ -25,6 +25,7 @@ - BAT-014 (REQ-025/026/027/028/032 재검증): `PortfolioItem.symbol` 응답 계약 명문화 + `GET /v1/portfolios`의 asset join 기반 응답(`portfolios JOIN assets` → `assets.symbol`). trex-orbit 팀 이슈 후속. - BAT-015 (REQ-033/034/035, dcy-005): `ledger_trades.asset_id` → `portfolio_id` 컬럼명 정합화(마이그레이션 `20260601_0024`) + `LoginRequest` extra="forbid"(session no_expiry 거부) + `GET /health` 응답에 `db_enabled` 필드 추가 + runtime-flows baseline 갱신. - BAT-016 (REQ-036/037/038, dcy-006): ledger 회계 책임 단일화 — `POST /v1/portfolios/{id}/ledger/trades`가 portfolio cash/quantity를 트랜잭션 갱신하고 응답에 `cash_balance_after`/`asset_quantity_after` 반환. `portfolio_ledger`/`ledger_trades` 단일 원장 통합(마이그레이션 `20260605_0025`, ledger_trades DROP) + 레거시 buy/sell deprecated 표기. idempotency는 hypertable 제약으로 application-level(SELECT-before-insert under FOR UPDATE)로 구현(REQ-036/037 CHG-20260605-03). 실 DB 통합 테스트 포함 376 passed. + - Hotfix (issue #9, 2026-06-06): `GET /v1/user_info` 500 수정 — `broker_credentials` 테이블 부재 시 `list_broker_credentials`가 `UndefinedTable`을 던져 핸들러가 500. (1) 핸들러를 broker credential 조회 실패에 graceful degrade(`brokers: []`, daemon_id/user_id/permission 정상 반환)하도록 보강, (2) `ensure_schema()`가 `broker_credentials` 테이블도 생성(alembic 0014 정합)하도록 추가. trex-core whoami(daemon_id) 도출 복구. 재현 회귀 테스트 추가, 전체 377 passed. 별도 BAT 미할당(운영 hotfix). - Excluded: - REQ-013 (Deprecated): GET /v1/assets 토큰 타입별 분기 재정의 — dcy-003 옵션 B 결정으로 무효화 - trex-basic 내부 코드 변경 (별도 레포 — 본 릴리즈와 동시 배포 필요: 신 ledger path) @@ -61,25 +62,26 @@ - [x] trex-basic 신 path 빌드 완료 확인 - [x] DB 스냅샷(백업) 생성 완료 - [x] `alembic upgrade head` dry-run 또는 revision 목록 점검 - - [x] 전체 테스트 suite pass 확인 (BAT-001~011: 340 passed @ main 4bf6b55; BAT-012: 362; BAT-013: 369; BAT-015: 372; **BAT-016: 376 passed @ 2026-06-05** — 실 DB ledger 통합 테스트 3건 포함. main HEAD `2a19b7f`) + - [x] 전체 테스트 suite pass 확인 (BAT-001~011: 340 passed @ main 4bf6b55; BAT-012: 362; BAT-013: 369; BAT-015: 372; BAT-016: 376 @ `2a19b7f`; **Hotfix #9: 377 passed @ 2026-06-06 `6b812b6`** — user_info graceful degrade 회귀 1건 추가) - [ ] **trex-core 신 응답 명세 채택 빌드 완료 확인** (BAT-016 — 별도 레포) - [ ] **BAT-016 마이그레이션 staging dry-run**: `20260605_0025` fresh-DB 적용 + DOUBLE→NUMERIC 정밀도 + action 분포 매핑 확인 (개발 DB는 ensure_schema 선적용으로 검증 불가 — staging 필수) - **배포 직후 (portfolio-hub)**: - - [~] `GET /health` → HTTP 200 + `db_enabled: true` (DB 연결 정상) — **로컬 smoke (2026-05-21 17:08 KST)**: HTTP 200 응답, 단 `HealthResponse` 모델([app/api/health.py:17-21](app/api/health.py#L17-L21))에 `db_enabled` 필드 미노출. DB 연동은 `PORTFOLIOHUB_DB_ENABLED=true` + `alembic upgrade head` head revision 적용 + 본 smoke의 다른 항목들이 DB 의존 동작으로 통과한 것으로 우회 확인. + - [x] `GET /health` → HTTP 200 + `db_enabled: true` (DB 연결 정상) — **로컬 smoke (2026-05-21 17:08 KST)**: HTTP 200 응답, 단 `HealthResponse` 모델([app/api/health.py:17-21](app/api/health.py#L17-L21))에 `db_enabled` 필드 미노출. DB 연동은 `PORTFOLIOHUB_DB_ENABLED=true` + `alembic upgrade head` head revision 적용 + 본 smoke의 다른 항목들이 DB 의존 동작으로 통과한 것으로 우회 확인. **재확인 (2026-06-06, 127.0.0.1:18000)**: `{"status":"ok","service":"portfolio-hub","version":"0.1.0","checked_at":"2026-06-06T00:08:11+09:00","db_enabled":true}` HTTP 200 — `db_enabled` 필드가 응답 본문에 정상 노출됨(BAT-015 REQ-035 반영). 기존 [~] 미노출 우려 해소. - [x] `GET /v1/assets` (admin token) → HTTP 200 (asset catalog 조회) — **재실행 로컬 smoke (2026-06-01 13:16 KST)**: admin DB session token으로 호출, HTTP 200 + `{"items":[{"asset_id":"bab52e11-...","symbol":"005930","display_name":"삼성전자","status":"active","tag_ids":[],...}],"total":1}` catalog 응답 확인. REQ-010 CHG-20260521-02 augmentation + BAT-012/013 verification 정합. portfolio 관련 필드 미포함, AC-9 충족. (이전 2026-05-21 smoke에서 HTTP 403 발생했던 불일치는 해소.) - [x] `POST /v1/portfolios` (user session token) → HTTP 201 (portfolio 생성) — **재실행 로컬 smoke (2026-06-01 13:18 KST)**: admin DB session token + active asset(`bab52e11-...`)으로 호출, HTTP 201 + `portfolio_id=1cec5061-8c26-4116-be3a-1ff2357d02f4` 생성. **응답 본문에 `display_name:"삼성전자"` 포함** — REQ-028 CHG-20260601-01(AC-13) augmentation 반영 확인. `_require_write_token` 게이트([app/api/v1_portfolios.py:45-50](app/api/v1_portfolios.py#L45-L50))에서 session token 정상 허용. - [x] `POST /v1/assets/{id}/ledger/trades` → HTTP 404 (구 path 제거 확인) — **재실행 로컬 smoke (2026-06-01 13:18 KST)**: HTTP 404 `{"detail":"Not Found"}` 확인. BAT-011 신/구 path 동시 서빙 없음 정책 충족. - [x] **(추가)** `GET /v1/portfolios` (session token) → HTTP 200 + 응답 portfolio item에 `display_name` 필드 포함 — **재실행 로컬 smoke (2026-06-01 13:18 KST)**: admin DB session token으로 호출, portfolios 2건 모두 `display_name:"삼성전자"` (joined `assets.display_name`)로 응답. REQ-028 CHG-20260601-01 AC-13 충족. - [ ] `POST /v1/assets/{asset_id}/tags` (admin token) → HTTP 200 + 응답 후 `GET /v1/assets`에서 해당 asset의 `tag_ids`에 적용된 tag_id 포함 확인 (BAT-013 정합화 검증) — 실 배포 환경에서 수집. unit-level은 cursor-mock SQL 검증으로 1차 안전망 확보. - [ ] `DELETE /v1/assets/{asset_id}/tags/{tag_id}` (admin token) → HTTP 200 + 응답 후 `GET /v1/assets`에서 해당 asset의 `tag_ids`에서 tag_id 제거 확인 (BAT-013 정합화 검증) — 실 배포 환경에서 수집. - - [ ] **(BAT-015)** `GET /health` → 응답 본문에 `db_enabled: true` 포함 (REQ-035 — 응답 모델 확장 확인; 위 db_enabled 미노출 [~] 항목 해소) - - [ ] **(BAT-015)** `POST /auth/login` body에 `no_expiry: true` → HTTP 422 (REQ-034 extra="forbid") - - [ ] **(BAT-016)** 레거시 `POST /portfolios/{id}/buy` → 응답 헤더 `Deprecation: true` (REQ-038) + - [x] **(BAT-015)** `GET /health` → 응답 본문에 `db_enabled: true` 포함 (REQ-035 — 응답 모델 확장 확인; 위 db_enabled 미노출 [~] 항목 해소) — **로컬 확인 (2026-06-06, 127.0.0.1:18000)**: 응답 본문에 `"db_enabled":true` 노출 확인. `HealthResponse` 모델 확장 반영됨. + - [x] **(BAT-015)** `POST /auth/login` body에 `no_expiry: true` → HTTP 422 (REQ-034 extra="forbid") — **로컬 확인 (2026-06-06, 127.0.0.1:18000)**: `{"username":"x","password":"y","no_expiry":true}` → HTTP 422 `{"detail":[{"type":"extra_forbidden","loc":["body","no_expiry"],"msg":"Extra inputs are not permitted"}]}`. extra="forbid" enforcement 동작 확인. + - [ ] **(BAT-016)** 레거시 `POST /portfolios/{id}/buy` → 응답 헤더 `Deprecation: true` (REQ-038) — **부분 확인 (2026-06-06, 127.0.0.1:18000)**: live OpenAPI(`/openapi.json`)에서 `POST /portfolios/{id}/buy`·`/sell` 모두 `deprecated: true` 노출 확인 + 핸들러 코드([app/api/portfolios.py:185,207](../../app/api/portfolios.py#L185)) `Deprecation: true` 헤더 설정 확인. 단, 라이브 응답 헤더는 happy-path(실제 portfolio 매수 성공) 응답에서만 노출되어 본 instance에서는 미캡처 — 오류 경로(401/500)는 `authorize_request`가 try 블록 밖에서 raise되거나 비-HTTPException 재raise로 헤더 우회. **실 배포 환경 happy-path smoke에서 헤더 직접 캡처 필요.** + - [x] **(Hotfix #9)** `GET /v1/user_info` (service token) → HTTP 200 + `daemon_id` 포함 (500 회귀 없음) — **로컬 smoke (2026-06-06, 127.0.0.1:18000)**: service token으로 호출, HTTP 200 + `{"daemon_id":"b5084542-...","user_id":"...","trading_enabled":true,"brokers":[],...}`. 수정 전 동일 호출은 500(`UndefinedTable: broker_credentials`)이었음. `broker_credentials` 테이블 ensure_schema 생성 확인. 자동 회귀 `tests/test_v1.py::test_v1_user_info_degrades_when_broker_credentials_lookup_fails` 통과. - **trex-basic / trex-core 배포 직후**: - [ ] `POST /v1/portfolios/{id}/ledger/trades` (service token) → HTTP 201 + `status="new"` (신 path 동작 확인) - [ ] **(BAT-016)** 위 응답 본문에 `cash_balance_after`/`asset_quantity_after` 포함 + 거래 반영 후 portfolio 잔고 갱신 확인 (REQ-037) - [ ] **(BAT-016)** 같은 (portfolio_id, order_id, execution_id) 재호출 → HTTP 200 + `status="duplicate"` + 현재 권위 상태 반환, ledger row 재생성 없음 - - [ ] **(BAT-016)** `ledger_trades` 테이블 부재 + portfolio_ledger 단일 원장 확인 + - [x] **(BAT-016)** `ledger_trades` 테이블 부재 + portfolio_ledger 단일 원장 확인 — **로컬 DB 확인 (2026-06-06, portfolio_hub@localhost:5432, psycopg)**: `to_regclass('public.ledger_trades')` → `None`(부재), `portfolio_ledger` 존재. portfolio_ledger 컬럼에 broker 식별 필드 흡수 확인: `ledger_entry_id`(bigint NOT NULL), `order_id`(text NULL), `execution_id`(text NULL), `side`(text NOT NULL), `executed_at`(timestamptz NOT NULL). 수치 컬럼 8종(`price`/`quantity`/`fee_amount`/`tax_amount`/`cash_balance_after`/`asset_quantity_after`/`total_value_after`/`profit_buffer_after`) 전부 NUMERIC(20,8), `action` 컬럼 제거됨, `idx_portfolio_ledger_portfolio_executed` 인덱스 존재. partial unique index 부재는 hypertable 제약으로 application-level idempotency 전환(REQ-036/037 CHG-20260605-03)과 정합. (단, 본 개발 DB는 ensure_schema 선적용 환경 — alembic fresh-DB 경로 검증은 staging dry-run 항목 별도 유지.) - [ ] daemon assignment 인가 3단계 정상 동작 확인 ## Release Execution Evidence @@ -98,7 +100,8 @@ - main branch commit: `4bf6b55` (BAT-008~011), `9038f41` (BAT-012), `58899b1` (BAT-013) - BAT-014: `136ae16` (PortfolioItem symbol response contract) - BAT-015: `2470c0a`/`84de34c` (rel-001 feedback cleanup — ledger rename, login extra=forbid, health db_enabled) - - **BAT-016: `2a19b7f` (ledger accounting consolidation + idempotency fix)** — 본 통합 release의 main HEAD + - BAT-016: `2a19b7f` (ledger accounting consolidation + idempotency fix) + - **Hotfix issue #9: `6b812b6` (GET /v1/user_info 500 on missing broker_credentials)** — 본 통합 release의 main HEAD - Commands or Procedure: - `alembic upgrade head`(head `20260605_0025`) → portfolio-hub rolling restart → trex-basic + trex-core 배포