Skip to content

Strengthen replan fallback and task tracking#43

Merged
braedonsaunders merged 1 commit intomainfrom
codex/fix-release-asset-collisions
Feb 25, 2026
Merged

Strengthen replan fallback and task tracking#43
braedonsaunders merged 1 commit intomainfrom
codex/fix-release-asset-collisions

Conversation

@braedonsaunders
Copy link
Owner

Summary

  • tighten the supervisor replan logic so it never accepts a no-op decision without falling back, records more context (parentTaskId, provider action metadata), and keeps sub-orchestrator child lists in sync
  • polish the UI (phase bar, deploy header, orchestrate agent cards) and README to improve usability, visibility, and onboarding guidance
  • add a regression test covering resume-from-provider-blocked snapshots to ensure the master orchestrator reopens paused planning tasks

Testing

  • Not run (not requested)

@braedonsaunders braedonsaunders force-pushed the codex/fix-release-asset-collisions branch from 5dee4af to 254b05f Compare February 25, 2026 01:31
@braedonsaunders braedonsaunders merged commit 9aec821 into main Feb 25, 2026
1 check passed
@braedonsaunders braedonsaunders deleted the codex/fix-release-asset-collisions branch February 25, 2026 01:34
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 254b05f070

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment on lines +2174 to +2179
if (!modelDecision) {
emitSupervisorThought(run.id, {
reasoning: "Replan decision unavailable from provider — accepting current state.",
action: "accept",
})
return {}

Choose a reason for hiding this comment

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

P2 Badge Run deterministic fallback when replan decision is missing

The new if (!modelDecision) branch exits with action: "accept" before invoking buildDeterministicFallbackDecision(), so any failure path where buildReplanDecision() throws (for example provider/model lookup errors before the structured fallback object is produced) now leaves failed/blocked tasks untouched. That bypasses the retry/skip recovery this commit added for other no-op decisions, and can leave runs stuck with unrecovered failures instead of progressing through deterministic fallback handling.

Useful? React with 👍 / 👎.

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