Skip to content

Fix workbench market failure status and error details - #633

Merged
yaojin3616 merged 1 commit into
dataelement:mainfrom
cinderzhan:codex/fix-workbench-market-errors
Oct 1, 2026
Merged

yaojin3616 merged 1 commit into
dataelement:mainfrom
cinderzhan:codex/fix-workbench-market-errors

Conversation

@cinderzhan

@cinderzhan cinderzhan commented Sep 30, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • Reconcile workbench market install records with the native plugin inventory. A package removed by recovery is available to install again, and a successful recovery removal clears its market record.
  • Distinguish install, update, and load failures on market cards, with the actual reason available by hover and keyboard focus.
  • Keep failed installs with stale sidebar records visible in Installed so users can remove the residue. Clear the persistent market record first; retain sidebar state if that request fails. A first install attempt with no saved record can clear its failure hint.

Validation

  • Targeted Vitest: 3 files, 154 tests passed against the PR commit snapshot.
  • npm run typecheck and npm run build passed against the PR commit snapshot.
  • git diff --check passed.

The packaged app UI has not been manually verified. A third-party workbench's own activation error remains a separate failure to diagnose from its reported details.

@cinderzhan
cinderzhan force-pushed the codex/fix-workbench-market-errors branch from 1004239 to eb91313 Compare September 30, 2026 08:27
yaojin3616 added a commit that referenced this pull request Oct 1, 2026
Merge #633: workbench market failure status (conflicts resolved)
@yaojin3616
yaojin3616 merged commit ad15c56 into dataelement:main Oct 1, 2026
9 checks 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