Improve error message around failing to install dev extensions (#30711)

Ben Kunkle created 7 months ago

Closes #ISSUE

Release Notes:

- N/A *or* Added/Fixed/Improved ...

Change summary

Cargo.lock                                | 1 +
crates/extensions_ui/Cargo.toml           | 3 ++-
crates/extensions_ui/src/extensions_ui.rs | 5 ++++-
3 files changed, 7 insertions(+), 2 deletions(-)

Detailed changes

Cargo.lock

@@ -5161,6 +5161,7 @@ dependencies = [
  "fuzzy",
  "gpui",
  "language",
+ "log",
  "num-format",
  "picker",
  "project",

crates/extensions_ui/Cargo.toml

@@ -23,6 +23,7 @@ fs.workspace = true
 fuzzy.workspace = true
 gpui.workspace = true
 language.workspace = true
+log.workspace = true
 num-format.workspace = true
 picker.workspace = true
 project.workspace = true
@@ -37,9 +38,9 @@ theme.workspace = true
 ui.workspace = true
 util.workspace = true
 vim_mode_setting.workspace = true
+workspace-hack.workspace = true
 workspace.workspace = true
 zed_actions.workspace = true
-workspace-hack.workspace = true
 
 [dev-dependencies]
 editor = { workspace = true, features = ["test-support"] }

crates/extensions_ui/src/extensions_ui.rs

@@ -132,10 +132,13 @@ pub fn init(cx: &mut App) {
                         match install_task.await {
                             Ok(_) => {}
                             Err(err) => {
+                                log::error!("Failed to install dev extension: {:?}", err);
                                 workspace_handle
                                     .update(cx, |workspace, cx| {
                                         workspace.show_error(
-                                            &err.context("failed to install dev extension"),
+                                            // NOTE: using `anyhow::context` here ends up not printing
+                                            // the error
+                                            &format!("Failed to install dev extension: {}", err),
                                             cx,
                                         );
                                     })