Skip to content

feat(auths-sdk,auths-core,auths-cli): stabilize core architecture & error boundaries#37

Merged
bordumb merged 4 commits intomainfrom
fn-33
Mar 6, 2026
Merged

feat(auths-sdk,auths-core,auths-cli): stabilize core architecture & error boundaries#37
bordumb merged 4 commits intomainfrom
fn-33

Conversation

@bordumb
Copy link
Contributor

@bordumb bordumb commented Mar 6, 2026

Summary

  • fn-33.1: Replace SdkStorageError::OperationFailed(String) with 5 typed #[from] variants (Identity, Init, AgentProvisioning, Driver, Attestation) — eliminates stringly-typed error wrapping in setup.rs and device.rs
  • fn-33.2: Implement AuthsErrorInfo for TrustError, SetupError, DeviceError; register SDK errors in CLI renderer downcast chain for structured JSON/text error output
  • fn-33.3: Audit all core/SDK crates for unwrap() in production code — none found; promote unwrap_used and expect_used workspace lints from warn to deny; add #![allow(...)] to auths-test-utils
  • fn-33.4: Add --json output to auths device list via DeviceEntry struct and JsonResponse<T> pattern

Test plan

  • cargo nextest run --workspace — 1567/1567 passed
  • cargo test --all --doc — 3 passed, 0 failed
  • All pre-push hooks passed (fmt, clippy, deny, wasm, cross-check)

@vercel
Copy link

vercel bot commented Mar 6, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
auths Ready Ready Preview, Comment Mar 6, 2026 8:01pm

@bordumb bordumb merged commit 247d18e into main Mar 6, 2026
8 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.

1 participant