Skip to content

fix(upgrade): improve failure diagnostics#273

Merged
RussellLuo merged 1 commit into
OpenCSGs:mainfrom
GatewayJ:fix/issue-2761-codex-agent-actions
Jul 2, 2026
Merged

fix(upgrade): improve failure diagnostics#273
RussellLuo merged 1 commit into
OpenCSGs:mainfrom
GatewayJ:fix/issue-2761-codex-agent-actions

Conversation

@GatewayJ

@GatewayJ GatewayJ commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • classify common upgrade failures such as network, HTTP status, disk space, permission, missing path, and invalid archive errors
  • persist structured upgrade failure metadata and log path separately from the raw error text
  • render localized upgrade failure guidance in the modal and sidebar, including long log paths in light and dark themes
  • add regression coverage for backend classification, API propagation, CLI output, and frontend formatting

Root cause

The upgrade helper previously surfaced the raw download/write error directly, so stream/network failures and local filesystem problems were hard to distinguish. The log path was appended into the raw message instead of being exposed as structured status.

Validation

  • go test ./...
  • pnpm --dir web/app test -- upgradeStatus useUpgradeController UpgradeModal SidebarUserButton
  • pnpm --dir web/app typecheck
  • pnpm --dir web/app format:check
  • pnpm --dir web/app lint
  • git diff --check

@GatewayJ GatewayJ requested a review from RussellLuo July 2, 2026 05:22
@GatewayJ GatewayJ marked this pull request as ready for review July 2, 2026 05:23
@GatewayJ GatewayJ changed the title Improve upgrade failure diagnostics fix(upgrade): improve failure diagnostics Jul 2, 2026
@GatewayJ GatewayJ force-pushed the fix/issue-2761-codex-agent-actions branch from 2b15012 to 11aaa5e Compare July 2, 2026 05:32
@GatewayJ GatewayJ force-pushed the fix/issue-2761-codex-agent-actions branch from 11aaa5e to 5643213 Compare July 2, 2026 05:46
@RussellLuo RussellLuo merged commit 71d2b1b into OpenCSGs:main Jul 2, 2026
1 check passed
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.

2 participants