Skip to content

fix(ultragoal): bound paused Stop guidance#3237

Merged
Yeachan-Heo merged 1 commit into
devfrom
fix/issue-3236-ultragoal-paused-stop-loop
Jul 20, 2026
Merged

fix(ultragoal): bound paused Stop guidance#3237
Yeachan-Heo merged 1 commit into
devfrom
fix/issue-3236-ultragoal-paused-stop-loop

Conversation

@Yeachan-Heo

Copy link
Copy Markdown
Owner

Summary

  • recognize paused Ultragoal review and user-decision states
  • emit one pause-specific Stop message without invalid completion reconciliation
  • allow the duplicate Stop replay to finish

Verification

  • npm run build
  • npx biome lint src/scripts/codex-native-hook.ts src/scripts/__tests__/codex-native-hook.test.ts
  • focused paused Stop regression
  • focused Ultragoal Stop regressions

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@Yeachan-Heo

Yeachan-Heo commented Jul 20, 2026

Copy link
Copy Markdown
Owner Author

Terminal review verdict: MERGE_READY

Reviewed serially against origin/dev at exact head 556e2b5d130c726d920431ad3f81e2fbb34b6cfb.

  • Red-team diff review found no blocking correctness, loop-control, or regression issue. Paused review_blocked / needs_user_decision paths emit bounded guidance and suppress only the active Stop-hook replay; active reconciliation remains unchanged.
  • Focused verification passed (4/4), changed-file Biome lint passed, and git diff --check passed.
  • Required hosted CI was green at the reviewed head. The PR was clean and mergeable into dev.
  • Squash merge verified as b72fd1ea5e7cf446fc611cbccece2254c2afba54; post-merge dev CI run 29728636158 passed.


[repo owner's gaebal-gajae (clawdbot) 🦞]

@Yeachan-Heo
Yeachan-Heo merged commit b72fd1e into dev Jul 20, 2026
20 checks passed
@i24hour

i24hour commented Jul 20, 2026

Copy link
Copy Markdown

hi

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants