Skip to content

[codex] 이슈 코멘트 자기 깨움 루프 차단#4186

Closed
seunghwanly wants to merge 2 commits intopaperclipai:masterfrom
seunghwanly:ppb-22-self-comment-wake-loop
Closed

[codex] 이슈 코멘트 자기 깨움 루프 차단#4186
seunghwanly wants to merge 2 commits intopaperclipai:masterfrom
seunghwanly:ppb-22-self-comment-wake-loop

Conversation

@seunghwanly
Copy link
Copy Markdown

Thinking Path

  • Paperclip는 에이전트 회사의 작업을 코멘트와 wakeup으로 오케스트레이션하는 컨트롤 플레인입니다.
  • 이번 변경은 server/src/routes/issues.ts의 이슈 코멘트 wakeup 경로에 걸려 있습니다.
  • 로컬 implicit 실행이 자신의 이슈에 체크포인트 코멘트를 남길 때 assignee wake가 다시 큐잉되어 self-wake 루프가 생길 수 있었습니다.
  • 특히 PATCH /api/issues/:idPOST /api/issues/:id/comments는 같은 런이 남긴 코멘트인지 구분해야 안전합니다.
  • 이 PR은 동일 runId가 남긴 self-comment를 assignee wake 대상에서 제외하는 로직을 추가하고, 그 회귀를 막는 테스트를 보강합니다.
  • 결과적으로 반복 코멘트로 같은 이슈가 즉시 다시 깨는 현상을 줄여 recurring intake/task heartbeat가 안정적으로 동작합니다.

What Changed

  • 동일 runId의 self-comment를 assignee comment wake에서 제외하는 로직을 server/src/routes/issues.ts에 추가했습니다.
  • PATCH /api/issues/:id 경로에서 status 변경으로 execution lock이 비워져도 이전 executionRunId까지 확인하도록 처리했습니다.
  • POST /api/issues/:id/comments 경로에도 같은 self-comment suppression 규칙을 맞췄습니다.
  • server/src/__tests__/issue-update-comment-wakeup-routes.test.ts에 local implicit run이 자기 이슈에 코멘트할 때 self-wake가 발생하지 않는 회귀 테스트 2개를 추가했습니다.

Verification

  • pnpm exec vitest run server/src/__tests__/issue-update-comment-wakeup-routes.test.ts

Risks

  • 중간 정도의 리스크입니다. self-comment suppression 조건이 너무 넓으면 실제로 필요한 assignee wake까지 막을 수 있으므로 runId와 현재/이전 executionRunId가 일치하는 경우로만 제한했습니다.

ROADMAP.md를 확인했고, 이번 변경은 계획된 신규 코어 기능이 아니라 좁은 범위의 버그 수정 및 회귀 테스트 추가라 중복되지 않습니다.

Model Used

  • OpenAI Codex GPT-5.4 (gpt-5.4), tool-enabled coding session with shell execution and targeted test runs.

Checklist

  • I have included a thinking path that traces from project context to this change
  • I have specified the model used (with version and capability details)
  • I have checked ROADMAP.md and confirmed this PR does not duplicate planned core work
  • I have run tests locally and they pass
  • I have added or updated tests where applicable
  • If this change affects the UI, I have included before/after screenshots
  • I have updated relevant documentation to reflect my changes
  • I have considered and documented any risks above
  • I will address all Greptile and reviewer comments before requesting merge

seunghwanly and others added 2 commits April 21, 2026 12:31
Co-authored-by: OpenAI Codex <codex@openai.com>
Co-authored-by: OpenAI Codex <codex@openai.com>
@seunghwanly seunghwanly deleted the ppb-22-self-comment-wake-loop branch April 21, 2026 04:26
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