Skip to content

[#58] 콘텐츠 재생성 라우터 #68

Open
jeongHwarr wants to merge 4 commits intodevfrom
feat/text-regenerate-router
Open

[#58] 콘텐츠 재생성 라우터 #68
jeongHwarr wants to merge 4 commits intodevfrom
feat/text-regenerate-router

Conversation

@jeongHwarr
Copy link
Copy Markdown
Collaborator

📝 Summary

✅ Checklist

  • 관련 이슈가 명시되어 있습니다.
  • 테스트가 완료되었습니다.
  • 문서 업데이트가 포함되었습니다.
  • 코드 리뷰를 위한 사전 검토를 완료했습니다.

📄 Description

  • 텍스트 체크 결과가 fail일 경우 텍스트 재생성하는 기능 추가

💡 Notice (Optional)

🔗 Related Issue(s)

#58

-->

@jeongHwarr jeongHwarr requested a review from gsgh3016 July 26, 2025 05:12
@gsgh3016
Copy link
Copy Markdown
Contributor

이슈 제목만 컨벤션에 맞게 수정해주세요! 나머지는 괜찮아보입니다 :)

@jeongHwarr jeongHwarr changed the title Add TextRegeneratorRouterNode to workflow and nodes.py. [#58] 콘텐츠 재생성 라우터 Aug 2, 2025
@jeongHwarr jeongHwarr self-assigned this Aug 2, 2025
@jeongHwarr jeongHwarr linked an issue Aug 2, 2025 that may be closed by this pull request
3 tasks
@qjrm1430
Copy link
Copy Markdown
Member

qjrm1430 commented Aug 5, 2025

감찬님 해당 PR 리뷰 한번 진행해서 문제없으면 승인해주시고 merge 부탁드려요~

Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bugbot free trial expires on August 19, 2025
Learn more in the Cursor dashboard.

Comment thread agents/text/workflow.py
"text_regenerate_router",
lambda state: state["next"], # state에서 next 키의 값을 사용
{
"text_generation": "text_content_check", # 실패시 text_generation으로 돌아감
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Bug: Infinite Loop in Text Workflow

The TextWorkflow contains an incorrect conditional edge mapping for text_regenerate_router. When the router returns 'text_generation' (indicating text regeneration is needed after a content check failure), it incorrectly routes to 'text_content_check' instead of 'text_generation'. This creates an infinite loop, preventing actual text regeneration and repeatedly checking the same failed content.

Fix in Cursor Fix in Web

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.

[#Text] 콘텐츠 재생성 라우터

3 participants