Skip to content

fix(agent): support /goal in Codex cloud runs#3467

Merged
trunk-io[bot] merged 8 commits into
mainfrom
posthog-code/fix-codex-cloud-goal-command
Jul 15, 2026
Merged

fix(agent): support /goal in Codex cloud runs#3467
trunk-io[bot] merged 8 commits into
mainfrom
posthog-code/fix-codex-cloud-goal-command

Conversation

@tatoalo

@tatoalo tatoalo commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Problem

Codex cloud runs pass /goal through as ordinary model input because they bypass Codex's interactive TUI command dispatcher.

Why: Cloud tasks should support the same long-running goal controls regardless of whether they use the Claude or Codex runtime adapter.

Changes

  • translate Codex /goal commands to the native app-server goal RPCs
  • advertise /goal alongside available Codex skills
  • cover reading, setting, clearing, pausing, and resuming goals without starting a model turn

@trunk-io

trunk-io Bot commented Jul 15, 2026

Copy link
Copy Markdown

😎 Merged successfully - details.

@github-actions

github-actions Bot commented Jul 15, 2026

Copy link
Copy Markdown

React Doctor found no issues in the changed files. 🎉

Reviewed by React Doctor for commit c6fb600.

@tatoalo tatoalo self-assigned this Jul 15, 2026
Translate Codex /goal commands to native app-server goal RPCs and advertise the command alongside skills.

Add regression coverage for reading, setting, clearing, pausing, and resuming goals without starting a model turn.

Generated-By: PostHog Code
Task-Id: f8939190-58a1-493b-904c-457c7859b3d6
@tatoalo tatoalo force-pushed the posthog-code/fix-codex-cloud-goal-command branch from 9e8e486 to 93c4fdb Compare July 15, 2026 11:34
tatoalo and others added 3 commits July 15, 2026 12:37
Generated-By: PostHog Code
Task-Id: f8939190-58a1-493b-904c-457c7859b3d6
Generated-By: PostHog Code
Task-Id: f8939190-58a1-493b-904c-457c7859b3d6
@tatoalo tatoalo marked this pull request as ready for review July 15, 2026 13:47
@greptile-apps

greptile-apps Bot commented Jul 15, 2026

Copy link
Copy Markdown
Contributor

Reviews (1): Last reviewed commit: "fix(agent): preserve Codex goals across ..." | Re-trigger Greptile

@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 15, 2026
Comment thread packages/agent/src/sagas/resume-saga.ts Outdated
stamphog[bot]

This comment was marked as outdated.

@stamphog stamphog Bot removed the Stamphog This will request an autostamp by stamphog on small changes label Jul 15, 2026
tatoalo and others added 4 commits July 15, 2026 13:53
Generated-By: PostHog Code
Task-Id: f8939190-58a1-493b-904c-457c7859b3d6
Generated-By: PostHog Code
Task-Id: f8939190-58a1-493b-904c-457c7859b3d6
@tatoalo tatoalo added the Stamphog This will request an autostamp by stamphog on small changes label Jul 15, 2026
@tatoalo tatoalo requested a review from a team July 15, 2026 14:38

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Both concrete bugs flagged by bot reviews on older commits are fixed in the current diff: the interrupt-failure race preserves the cancellation flag until the RPC succeeds, and the goal-scan loop uses continue on malformed entries instead of an early return. The stamphog review was on an older commit; all greptile inline comments are resolved with matching fixes visible in the diff. No showstoppers in this additive goal-command feature.

@trunk-io trunk-io Bot merged commit 73b695c into main Jul 15, 2026
36 checks passed
@trunk-io trunk-io Bot deleted the posthog-code/fix-codex-cloud-goal-command branch July 15, 2026 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Stamphog This will request an autostamp by stamphog on small changes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant