Skip to content

fix: Intent와 Workflow ID 응답 구분 - #19

Merged
EHWIYA merged 1 commit into
developfrom
fix/17-intent-workflow-id
Aug 6, 2026
Merged

fix: Intent와 Workflow ID 응답 구분#19
EHWIYA merged 1 commit into
developfrom
fix/17-intent-workflow-id

Conversation

@hywznn

@hywznn hywznn commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

왜 필요한가요?

분석 결과에서 detectedIntentworkflowId가 모두 EXPIRY_RENEWAL로 반환되고 있었습니다.

  • detectedIntent: 사용자의 요청 종류
  • workflowId: 실제 실행할 Knowledge Workflow의 고유 ID

두 값의 역할이 다르므로 Server가 올바른 Workflow를 선택할 수 있도록 분리합니다.

무엇이 바뀌나요?

  • AnalysisPipeline이 Intent로 Workflow ID를 덮어쓰던 로직을 제거했습니다.
  • 후보의 workflowId에 Intent Agent가 선택한 canonical ID를 그대로 반환합니다.
  • Agent/API 테스트 예상값을 WF-STY-001로 수정했습니다.
  • 응답 예시와 분석 계약 문서를 같은 기준으로 맞췄습니다.

변경 전

{
  "detectedIntent": "EXPIRY_RENEWAL",
  "workflowId": "EXPIRY_RENEWAL"
}

변경 후

{
  "detectedIntent": "EXPIRY_RENEWAL",
  "workflowId": "WF-STY-001"
}

검증

  • 관련 테스트: 12개 통과
  • 이번 변경 파일 Ruff: 통과
  • 전체 테스트: 557개 통과, 1개 환경 의존 실패
    • macOS에 PowerShell이 없어 Windows 전용 OCR smoke test가 실패했습니다.
  • 전체 Ruff의 기존 오류는 이번 변경과 무관하여 수정하지 않았습니다.

병합 전 확인

  • Server가 detectedIntent=EXPIRY_RENEWAL, workflowId=WF-STY-001 응답을 정상 처리하는지 smoke test
  • AI 담당자 계약 검토

브랜치 이름 변경으로 자동 종료된 #18을 대체합니다.

Refs #17

@hywznn hywznn added the bug Something isn't working label Aug 6, 2026
@hywznn
hywznn requested review from EHWIYA and taejung3852 and removed request for taejung3852 August 6, 2026 04:27
@EHWIYA
EHWIYA marked this pull request as ready for review August 6, 2026 05:05
@EHWIYA
EHWIYA merged commit 5cf55c9 into develop Aug 6, 2026
@taejung3852
taejung3852 deleted the fix/17-intent-workflow-id branch August 6, 2026 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants