fix: ranking API 필수 파라미터 KEYB 추가 및 GUBN 값 수정 (#258) #260

agentson · 2026-02-26T00:20:16+09:00

agentson commented

2026-02-26 00:20:16 +09:00

Summary

fetch_overseas_rankings()의 fluctuation/volume params에 KEYB: '' 추가
KIS 공식 문서 기준 KEYB는 Required=Y 필수 파라미터였으나 누락되어 있어 매 사이클 rt_cd=2, ERROR INPUT FIELD NOT FOUND [KEYB] 발생
GUBN 주석 오류 수정: 0=하락율, 1=상승율 (이전 주석은 0=전체로 잘못 기재)
GUBN 0→1: 상승율 기준 급등 종목 스캔

영향

기존: ranking API 항상 실패 → fallback(거래 이력 기반) → TQQQ 같은 이전 종목만 반복
수정 후: 실시간 변동성/상승률 상위 종목 스캔 정상 작동

Test plan

test_fluctuation_uses_correct_params — KEYB, GUBN=1 검증
test_volume_uses_correct_params — KEYB 검증
실제 API 호출 확인: 나스닥 상승율 상위 5종목 정상 반환

Closes #258

## Summary - `fetch_overseas_rankings()`의 fluctuation/volume params에 `KEYB: ''` 추가 - KIS 공식 문서 기준 KEYB는 Required=Y 필수 파라미터였으나 누락되어 있어 매 사이클 `rt_cd=2, ERROR INPUT FIELD NOT FOUND [KEYB]` 발생 - GUBN 주석 오류 수정: `0=하락율, 1=상승율` (이전 주석은 `0=전체`로 잘못 기재) - GUBN `0→1`: 상승율 기준 급등 종목 스캔 ## 영향 기존: ranking API 항상 실패 → fallback(거래 이력 기반) → TQQQ 같은 이전 종목만 반복 수정 후: 실시간 변동성/상승률 상위 종목 스캔 정상 작동 ## Test plan - [x] `test_fluctuation_uses_correct_params` — KEYB, GUBN=1 검증 - [x] `test_volume_uses_correct_params` — KEYB 검증 - [x] 실제 API 호출 확인: 나스닥 상승율 상위 5종목 정상 반환 Closes #258

agentson added 1 commit 2026-02-26 00:20:16 +09:00

fix: ranking API 필수 파라미터 KEYB 추가 및 GUBN 값 수정 (#258 )

CI / test (pull_request) Has been cancelled

Details

8af5f564c3

KIS 공식 문서(20260221) 기준 KEYB(NEXT KEY BUFF)는 Required=Y이나
누락되어 있어 항상 rt_cd=2 오류 발생, fallback 경로로만 실행됨.

- fluctuation/volume 양쪽 params에 KEYB: '' 추가
- GUBN 주석 수정: 0=하락율, 1=상승율 (문서 기준)
- GUBN 값 0→1 수정: 상승율 기준으로 변동성 급등 종목 스캔

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

jihoson merged commit f87c4dc2f0 into main

2026-02-26 00:20:59 +09:00

jihoson referenced this issue from a commit

2026-02-26 00:21:00 +09:00

Merge pull request 'fix: ranking API 필수 파라미터 KEYB 추가 및 GUBN 값 수정 (#258)' (#260) from feature/issue-258-ranking-api-keyb-param into main

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: jihoson/The-Ouroboros#260