Skip to content

fix(roadmap): run_assess_task가 notify_be_roadmap을 호출하도록 연결#89

Merged
Youcu merged 1 commit into
developfrom
fix/roadmap-webhook-wire
Jun 4, 2026
Merged

fix(roadmap): run_assess_task가 notify_be_roadmap을 호출하도록 연결#89
Youcu merged 1 commit into
developfrom
fix/roadmap-webhook-wire

Conversation

@Youcu

@Youcu Youcu commented Jun 4, 2026

Copy link
Copy Markdown
Contributor

문제

webhook.notify_be_roadmap(result를 싣는 /roadmap/complete 콜백)은 develop에 이미 추가돼 있으나, run_assess_task가 여전히 공용 notify_be(/jobs/complete, result 없음·PDF 없어 status 항상 ERROR)를 호출 → 로드맵 평가 결과가 BE로 전달되지 않음(함수만 있고 연결 누락).

변경 (연결 1곳)

  • import notify_benotify_be_roadmap
  • finally: notify_be_roadmap(ai_job_id, result=results, error_message) 호출 (성공 시 result, 실패 시 None→ERROR 자동).
  • 공용 notify_be(pdf_pipeline 사용)는 무변경.

검증

  • compileall app/ 통과. 변경은 roadmap_assess.py의 import + finally 한 곳뿐.

비고

🤖 Generated with Claude Code

webhook.notify_be_roadmap(/roadmap/complete, result 포함)은 이미 추가돼 있었으나
run_assess_task가 여전히 공용 notify_be(/jobs/complete, result 없음·항상 ERROR)를
호출하고 있어 로드맵 결과가 BE로 전달되지 않았다. 연결만 수정:
- import notify_be → notify_be_roadmap
- finally: notify_be_roadmap(ai_job_id, result=results, error_message) 호출
  (성공 시 result, 실패 시 None→ERROR)

공용 notify_be(pdf_pipeline)는 무변경.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@Youcu Youcu merged commit d45798a into develop Jun 4, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant