v0.2.6-pre.016
This commit is contained in:
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-app-wallet-desk/src/app_state.rs
|
||||
// version: 18
|
||||
// version: 19
|
||||
|
||||
//! Shared backend state owned by the Wallet Desk Tauri application.
|
||||
|
||||
@@ -202,7 +202,7 @@ impl AppState {
|
||||
let note_texts = note.into_iter().collect::<std::vec::Vec<_>>();
|
||||
let metadata = ksp_wallet_lib::WalletCreateMetadataV1::new(alias, note_texts);
|
||||
ksp_logging_lib::debug!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_SESSION, wallet_id = filename.as_str(), view_enabled, "Wallet creation requested under effective Config root");
|
||||
let created = ksp_wallet_lib::create_wallet_file_v1(destination.as_path(), owner_password, view_password, metadata).await;
|
||||
let created = ksp_wallet_lib::create_wallet_file(destination.as_path(), owner_password, view_password, metadata).await;
|
||||
let owner = match created {
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(error) => {
|
||||
@@ -297,7 +297,7 @@ impl AppState {
|
||||
let transfer_format = pending.wallet_format();
|
||||
ksp_logging_lib::debug!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_IMPORT, wallet_id = filename.as_str(), transfer_format = transfer_format.code(), view_enabled, "Wallet transfer import requested under effective Config root");
|
||||
let imported =
|
||||
ksp_wallet_lib::import_wallet_transfer_v1(destination.as_path(), pending.source_bytes(), transfer_format, owner_password, view_password, metadata)
|
||||
ksp_wallet_lib::import_wallet_transfer(destination.as_path(), pending.source_bytes(), transfer_format, owner_password, view_password, metadata)
|
||||
.await;
|
||||
let owner = match imported {
|
||||
std::result::Result::Ok(value) => value,
|
||||
@@ -326,7 +326,7 @@ impl AppState {
|
||||
std::result::Result::Err(error) => return std::result::Result::Err(error),
|
||||
};
|
||||
let password = ksp_wallet_lib::ViewPassword::new(request.password);
|
||||
let opened = ksp_wallet_lib::open_wallet_view_file_v1(context.path.as_path(), password).await;
|
||||
let opened = ksp_wallet_lib::open_wallet_view_file(context.path.as_path(), password).await;
|
||||
let view = match opened {
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(error) => {
|
||||
@@ -346,7 +346,7 @@ impl AppState {
|
||||
std::result::Result::Err(error) => return std::result::Result::Err(error),
|
||||
};
|
||||
let password = ksp_wallet_lib::OwnerPassword::new(request.password);
|
||||
let opened = ksp_wallet_lib::open_wallet_owner_file_v1(context.path.as_path(), password).await;
|
||||
let opened = ksp_wallet_lib::open_wallet_owner_file(context.path.as_path(), password).await;
|
||||
let owner = match opened {
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(error) => {
|
||||
@@ -390,7 +390,7 @@ impl AppState {
|
||||
return std::result::Result::Err(error);
|
||||
},
|
||||
};
|
||||
let opened = ksp_wallet_lib::open_wallet_view_file_v1(context.path.as_path(), ksp_wallet_lib::ViewPassword::new(secret)).await;
|
||||
let opened = ksp_wallet_lib::open_wallet_view_file(context.path.as_path(), ksp_wallet_lib::ViewPassword::new(secret)).await;
|
||||
match opened {
|
||||
std::result::Result::Ok(view) => {
|
||||
ksp_logging_lib::info!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_SECRET, wallet_id = context.wallet_id.as_str(), capability = context.capability.label(), candidate_count, "Configured-secret Wallet unlock succeeded");
|
||||
@@ -439,7 +439,7 @@ impl AppState {
|
||||
return std::result::Result::Err(error);
|
||||
},
|
||||
};
|
||||
let opened = ksp_wallet_lib::open_wallet_owner_file_v1(context.path.as_path(), ksp_wallet_lib::OwnerPassword::new(secret)).await;
|
||||
let opened = ksp_wallet_lib::open_wallet_owner_file(context.path.as_path(), ksp_wallet_lib::OwnerPassword::new(secret)).await;
|
||||
match opened {
|
||||
std::result::Result::Ok(owner) => {
|
||||
ksp_logging_lib::info!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_SECRET, wallet_id = context.wallet_id.as_str(), capability = context.capability.label(), candidate_count, "Configured-secret Wallet unlock succeeded");
|
||||
@@ -516,7 +516,7 @@ impl AppState {
|
||||
(wallet_id, path)
|
||||
},
|
||||
};
|
||||
let locked = ksp_wallet_lib::inspect_locked_wallet_file_v1(path.as_path()).await;
|
||||
let locked = ksp_wallet_lib::inspect_locked_wallet_file(path.as_path()).await;
|
||||
let locked = match locked {
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(error) => return std::result::Result::Err(error),
|
||||
@@ -914,23 +914,23 @@ impl AppState {
|
||||
std::result::Result::Err(_) => return std::result::Result::Err(session_lock_error()),
|
||||
};
|
||||
let previous = std::mem::replace(&mut *session, crate::WalletSession::no_selection());
|
||||
return match previous {
|
||||
match previous {
|
||||
crate::WalletSession::ViewOperation { wallet_id: reserved_wallet_id, path: reserved_path, pubkey: reserved_pubkey }
|
||||
if reserved_wallet_id == wallet_id && reserved_path == path && reserved_pubkey == pubkey =>
|
||||
{
|
||||
*session = crate::WalletSession::View { wallet_id: wallet_id.clone(), path, wallet: view };
|
||||
ksp_logging_lib::info!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_SESSION, wallet_id = wallet_id.as_str(), operation, "Wallet VIEW privileged operation completed");
|
||||
std::result::Result::Ok(dto)
|
||||
return std::result::Result::Ok(dto);
|
||||
},
|
||||
other => {
|
||||
*session = other;
|
||||
drop(view);
|
||||
std::result::Result::Err(ksp_core_lib::Error::new(
|
||||
return std::result::Result::Err(ksp_core_lib::Error::new(
|
||||
crate::ERROR_CODE_WALLET_SESSION_INVALID,
|
||||
"Wallet VIEW operation completion no longer owns the selected session",
|
||||
))
|
||||
));
|
||||
},
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
fn restore_view_after_operation_failure(&self, context: ViewOperationContext) {
|
||||
@@ -958,7 +958,7 @@ impl AppState {
|
||||
async fn recover_view_state_conflict(&self, context: ViewOperationContext) {
|
||||
let ViewOperationContext { operation, path, pubkey, view, wallet_id } = context;
|
||||
drop(view);
|
||||
let inspected = ksp_wallet_lib::inspect_locked_wallet_file_v1(path.as_path()).await;
|
||||
let inspected = ksp_wallet_lib::inspect_locked_wallet_file(path.as_path()).await;
|
||||
let session = self.wallet_session.lock();
|
||||
let mut session = match session {
|
||||
std::result::Result::Ok(value) => value,
|
||||
@@ -1039,7 +1039,7 @@ impl AppState {
|
||||
std::result::Result::Err(_) => return std::result::Result::Err(session_lock_error()),
|
||||
};
|
||||
let previous = std::mem::replace(&mut *session, crate::WalletSession::no_selection());
|
||||
return match previous {
|
||||
match previous {
|
||||
crate::WalletSession::OwnerOperation {
|
||||
wallet_id: reserved_wallet_id,
|
||||
path: reserved_path,
|
||||
@@ -1048,17 +1048,17 @@ impl AppState {
|
||||
} if reserved_wallet_id == wallet_id && reserved_path == path && reserved_pubkey == pubkey && reserved_view_enabled == view_enabled => {
|
||||
*session = crate::WalletSession::Owner { wallet_id: wallet_id.clone(), path, view_enabled: result_view_enabled, wallet: owner };
|
||||
ksp_logging_lib::info!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_SESSION, wallet_id = wallet_id.as_str(), operation, "Wallet OWNER privileged operation completed");
|
||||
std::result::Result::Ok(dto)
|
||||
return std::result::Result::Ok(dto);
|
||||
},
|
||||
other => {
|
||||
*session = other;
|
||||
drop(owner);
|
||||
std::result::Result::Err(ksp_core_lib::Error::new(
|
||||
return std::result::Result::Err(ksp_core_lib::Error::new(
|
||||
crate::ERROR_CODE_WALLET_SESSION_INVALID,
|
||||
"Wallet OWNER operation completion no longer owns the selected session",
|
||||
))
|
||||
));
|
||||
},
|
||||
};
|
||||
}
|
||||
}
|
||||
|
||||
fn restore_owner_after_operation_failure(&self, context: OwnerOperationContext) {
|
||||
@@ -1089,7 +1089,7 @@ impl AppState {
|
||||
async fn recover_owner_state_conflict(&self, context: OwnerOperationContext) {
|
||||
let OwnerOperationContext { operation, owner, path, pubkey, result_view_enabled: _, view_enabled, wallet_id } = context;
|
||||
drop(owner);
|
||||
let inspected = ksp_wallet_lib::inspect_locked_wallet_file_v1(path.as_path()).await;
|
||||
let inspected = ksp_wallet_lib::inspect_locked_wallet_file(path.as_path()).await;
|
||||
let session = self.wallet_session.lock();
|
||||
let mut session = match session {
|
||||
std::result::Result::Ok(value) => value,
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-app-wallet-desk/src/wallet_inventory.rs
|
||||
// version: 3
|
||||
// version: 4
|
||||
|
||||
//! Root-scoped Wallet inventory and locked-file selection for Wallet Desk.
|
||||
|
||||
@@ -113,7 +113,7 @@ pub(crate) async fn list_wallet_inventory(root: &std::path::Path) -> ksp_core_li
|
||||
ksp_logging_lib::trace!(target: crate::TRACING_TARGET, domain = crate::TRACING_DOMAIN_WALLET_INVENTORY, filename = filename.as_str(), is_symlink = metadata.file_type().is_symlink(), "Wallet inventory skipped a non-regular candidate");
|
||||
continue;
|
||||
}
|
||||
let inspected = ksp_wallet_lib::inspect_locked_wallet_file_v1(path.as_path()).await;
|
||||
let inspected = ksp_wallet_lib::inspect_locked_wallet_file(path.as_path()).await;
|
||||
match inspected {
|
||||
std::result::Result::Ok(locked) => entries.push(valid_inventory_entry(filename, locked)),
|
||||
std::result::Result::Err(error) => {
|
||||
@@ -155,7 +155,7 @@ pub(crate) async fn resolve_locked_wallet(
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(error) => return std::result::Result::Err(error),
|
||||
};
|
||||
let locked = ksp_wallet_lib::inspect_locked_wallet_file_v1(path.as_path()).await;
|
||||
let locked = ksp_wallet_lib::inspect_locked_wallet_file(path.as_path()).await;
|
||||
let locked = match locked {
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(error) => return std::result::Result::Err(error),
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-app-wallet-desk/tests/desktop_contract.rs
|
||||
// version: 16
|
||||
// version: 17
|
||||
|
||||
//! Desktop build, shell and Config-status contract audits for Wallet Desk.
|
||||
|
||||
@@ -98,7 +98,7 @@ fn pre_004_inventory_uses_wallet_library_and_exposes_only_locked_safe_fields() {
|
||||
let inventory = read_text(root.join("src/wallet_inventory.rs").as_path());
|
||||
let main = read_text(root.join("frontend/ts/main.ts").as_path());
|
||||
assert!(manifest.contains("ksp-wallet-lib = { path = \"../ksp-wallet-lib\" }"));
|
||||
assert!(inventory.contains("ksp_wallet_lib::inspect_locked_wallet_file_v1"));
|
||||
assert!(inventory.contains("ksp_wallet_lib::inspect_locked_wallet_file"));
|
||||
assert!(inventory.contains("symlink_metadata"));
|
||||
assert!(inventory.contains("WALLET_FILE_SUFFIX"));
|
||||
assert!(tauri.contains("list_wallets"));
|
||||
@@ -132,7 +132,7 @@ fn pre_005_create_and_session_lifecycle_are_backend_owned() {
|
||||
let tauri = read_text(root.join("src/tauri.rs").as_path());
|
||||
let html = read_text(root.join("frontend/main.html").as_path());
|
||||
let main = read_text(root.join("frontend/ts/main.ts").as_path());
|
||||
assert!(state.contains("ksp_wallet_lib::create_wallet_file_v1"));
|
||||
assert!(state.contains("ksp_wallet_lib::create_wallet_file"));
|
||||
assert!(state.contains("WalletSession::Owner"));
|
||||
assert!(state.contains("WalletSession::Locked"));
|
||||
assert!(state.contains("std::mem::replace"));
|
||||
@@ -157,8 +157,8 @@ fn pre_006_unlock_surface_is_explicit_backend_owned_and_supports_view_owner() {
|
||||
let tauri = read_text(root.join("src/tauri.rs").as_path());
|
||||
let html = read_text(root.join("frontend/main.html").as_path());
|
||||
let main = read_text(root.join("frontend/ts/main.ts").as_path());
|
||||
assert!(state.contains("open_wallet_view_file_v1"));
|
||||
assert!(state.contains("open_wallet_owner_file_v1"));
|
||||
assert!(state.contains("open_wallet_view_file"));
|
||||
assert!(state.contains("open_wallet_owner_file"));
|
||||
assert!(state.contains("WalletSession::PrivilegedOperation"));
|
||||
assert!(state.contains("WalletSession::View"));
|
||||
assert!(secrets.contains("environment_report"));
|
||||
@@ -220,7 +220,7 @@ fn pre_008_import_uses_rust_native_picker_and_wallet_transfer_adapters() {
|
||||
assert!(tauri.contains("import_wallet"));
|
||||
assert!(import.contains("ksp_wallet_lib::inspect_wallet_transfer"));
|
||||
assert!(import.contains("zeroize::Zeroizing"));
|
||||
assert!(state.contains("ksp_wallet_lib::import_wallet_transfer_v1"));
|
||||
assert!(state.contains("ksp_wallet_lib::import_wallet_transfer"));
|
||||
assert!(html.contains("id=\"inspectImportSource\""));
|
||||
assert!(html.contains("id=\"importWalletForm\""));
|
||||
assert!(html.contains("id=\"importSourcePubkey\""));
|
||||
@@ -384,3 +384,19 @@ fn pre_014_template_has_normalized_html_headers_light_header_and_kbot_style_spla
|
||||
assert!(splash_ts.contains("add_message"));
|
||||
assert!(!splash_ts.contains("../sass/splash.scss"));
|
||||
}
|
||||
|
||||
#[test]
|
||||
fn pre_016_wallet_desk_uses_version_neutral_wallet_dispatch() {
|
||||
let state = read_source("src/app_state.rs");
|
||||
let inventory = read_source("src/wallet_inventory.rs");
|
||||
assert!(state.contains("ksp_wallet_lib::create_wallet_file"));
|
||||
assert!(state.contains("ksp_wallet_lib::import_wallet_transfer"));
|
||||
assert!(state.contains("ksp_wallet_lib::open_wallet_view_file"));
|
||||
assert!(state.contains("ksp_wallet_lib::open_wallet_owner_file"));
|
||||
assert!(inventory.contains("ksp_wallet_lib::inspect_locked_wallet_file"));
|
||||
assert!(!state.contains("create_wallet_file_v1"));
|
||||
assert!(!state.contains("import_wallet_transfer_v1"));
|
||||
assert!(!state.contains("open_wallet_view_file_v1"));
|
||||
assert!(!state.contains("open_wallet_owner_file_v1"));
|
||||
assert!(!inventory.contains("inspect_locked_wallet_file_v1"));
|
||||
}
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-app-wallet-desk/tests/desktop_security.rs
|
||||
// version: 13
|
||||
// version: 14
|
||||
|
||||
//! Static desktop security contracts for the Wallet Desk pre.002 shell.
|
||||
|
||||
@@ -172,7 +172,7 @@ fn owner_metadata_values_are_request_only_and_state_conflict_forces_reauthorizat
|
||||
assert!(!metadata.contains("derive(Clone"));
|
||||
assert!(!metadata.contains("derive(Debug"));
|
||||
assert!(state.contains("recover_owner_state_conflict"));
|
||||
assert!(state.contains("inspect_locked_wallet_file_v1"));
|
||||
assert!(state.contains("inspect_locked_wallet_file"));
|
||||
assert!(state.contains("ERROR_CODE_STATE_CONFLICT"));
|
||||
assert!(main.contains("isWalletStateConflict"));
|
||||
assert!(main.contains("select_wallet"));
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
// file: crates/ksp-app-wallet-desk/tests/wallet_desk_devnet_smoke.rs
|
||||
// version: 1
|
||||
// version: 2
|
||||
|
||||
//! Opt-in live composition smoke owned by the Wallet Desk orchestration surface.
|
||||
|
||||
@@ -72,7 +72,7 @@ async fn committed_wallet_desk_composite_creates_wallet_and_reaches_devnet_balan
|
||||
std::result::Result::Err(_) => return,
|
||||
};
|
||||
let wallet_path = wallet_root.join(std::format!("wallet-desk-pre013-smoke-{}-{stamp}.kspwallet", std::process::id()));
|
||||
let owner = ksp_wallet_lib::create_wallet_file_v1(
|
||||
let owner = ksp_wallet_lib::create_wallet_file(
|
||||
wallet_path.as_path(),
|
||||
ksp_wallet_lib::OwnerPassword::new("ksp-pre013-devnet-owner".to_owned()),
|
||||
std::option::Option::Some(ksp_wallet_lib::ViewPassword::new("ksp-pre013-devnet-view".to_owned())),
|
||||
@@ -84,13 +84,13 @@ async fn committed_wallet_desk_composite_creates_wallet_and_reaches_devnet_balan
|
||||
std::result::Result::Ok(value) => value,
|
||||
std::result::Result::Err(_) => return,
|
||||
};
|
||||
let locked = ksp_wallet_lib::inspect_locked_wallet_file_v1(wallet_path.as_path()).await;
|
||||
let locked = ksp_wallet_lib::inspect_locked_wallet_file(wallet_path.as_path()).await;
|
||||
assert!(locked.is_ok(), "created smoke wallet should inspect while locked");
|
||||
if let std::result::Result::Ok(locked) = locked {
|
||||
assert_eq!(locked.format_version(), ksp_wallet_lib::KSPWALLET_FORMAT_VERSION_V1);
|
||||
assert_eq!(locked.format_version(), ksp_wallet_lib::KSPWALLET_FORMAT_VERSION_V2);
|
||||
assert!(locked.view_enabled());
|
||||
}
|
||||
let view = ksp_wallet_lib::open_wallet_view_file_v1(wallet_path.as_path(), ksp_wallet_lib::ViewPassword::new("ksp-pre013-devnet-view".to_owned())).await;
|
||||
let view = ksp_wallet_lib::open_wallet_view_file(wallet_path.as_path(), ksp_wallet_lib::ViewPassword::new("ksp-pre013-devnet-view".to_owned())).await;
|
||||
assert!(view.is_ok(), "created smoke wallet should open with VIEW");
|
||||
let view = match view {
|
||||
std::result::Result::Ok(value) => value,
|
||||
|
||||
Reference in New Issue
Block a user