Skip to content

fix(vault): surface operation failures#182

Open
vutuanlinh2k2 wants to merge 1 commit into
mainfrom
fix/vault-operation-errors-181
Open

fix(vault): surface operation failures#182
vutuanlinh2k2 wants to merge 1 commit into
mainfrom
fix/vault-operation-errors-181

Conversation

@vutuanlinh2k2

Copy link
Copy Markdown
Contributor

Summary

  • surface list, save, create, and delete failures through retryable Vault UI states
  • preserve drafts, dialogs, selections, and the last loaded tree across rejected operations
  • handle post-mutation refresh failures without repeating successful creates or deletes
  • expose an optional structured onOperationError callback for host telemetry

Testing

  • pnpm typecheck
  • pnpm vitest run tests/vault/vault-pane.test.ts — 33 tests passed
  • pnpm exec vitest run --exclude src/sandbox/index.test.ts
  • NODE_OPTIONS=--max-old-space-size=8192 pnpm build

The unfiltered suite still has three pre-existing deferred-file/base64 failures in src/sandbox/index.test.ts; the changed Vault suite and all other test files pass.

Closes #181

Preserve recoverable Vault UI state across list and mutation failures, expose structured error reporting, and retry post-mutation refreshes without repeating completed writes.

Closes #181
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.

[BUG] Surface Vault list and mutation failures in VaultPane

1 participant