Skip to content

desktop(react): own Job / Launch Review#437

Merged
Whiteks1 merged 1 commit intomainfrom
codex/issue-436-job-launch-review-plan
Apr 20, 2026
Merged

desktop(react): own Job / Launch Review#437
Whiteks1 merged 1 commit intomainfrom
codex/issue-436-job-launch-review-plan

Conversation

@Whiteks1
Copy link
Copy Markdown
Owner

Summary

  • add React-owned Job / Launch Review tabs for launch job status, stdout, stderr, linked run, artifacts, and deterministic failure context
  • wire React context actions for job tabs while preserving /api/launch-control as the transitional Launch data boundary
  • connect existing System/Paper Ops launch-review actions to the React-owned job surface
  • update docs so Job / Launch Review is no longer a legacy-renderer blocker for desktop(cleanup): post-v1 legacy shell retirement boundary #412

Scope

This is the narrow #436 implementation.

Out of scope:

Validation

  • npm ci
  • npm run -s typecheck
  • npm run -s build
  • npm run -s smoke:fallback
  • npm run -s smoke:react:fallback
  • node --test tests/snapshot-status.test.mjs
  • git diff --check

Attempted but not used as a local blocker:

  • npm run -s smoke:real-path failed locally because research_ui did not become reachable on Windows (serverReady=false, apiReady=false), while Desktop happy-path checks were true. Remote desktop-real-path should be authoritative.

Closes #436

Copy link
Copy Markdown

@sourcery-ai sourcery-ai Bot left a comment

Choose a reason for hiding this comment

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

Sorry @Whiteks1, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

@Whiteks1 Whiteks1 merged commit 4f1079a into main Apr 20, 2026
3 checks passed
@Whiteks1 Whiteks1 deleted the codex/issue-436-job-launch-review-plan branch April 20, 2026 21:51
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.

desktop(react): own Job / Launch Review and define Launch API boundary

1 participant