Allow triggering npm publish indendently

Author: sinelaw

.github/workflows/npm-publish.yml

@@ -4,10 +4,19 @@
 name: npm-publish
 
 on:
+  workflow_dispatch:
+    inputs:
+      version:
+        description: 'Version to publish (e.g., 0.1.32)'
+        required: true
+        type: string
   workflow_call:
     inputs:
       plan:
-        required: true
+        required: false
+        type: string
+      version:
+        required: false
         type: string
 
 jobs:
@@ -19,14 +28,23 @@ jobs:
     env:
       GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
       PLAN: ${{ inputs.plan }}
-    if: ${{ !fromJson(inputs.plan).announcement_is_prerelease || fromJson(inputs.plan).publish_prereleases }}
+      VERSION: ${{ inputs.version }}
+    if: ${{ inputs.version || !fromJson(inputs.plan).announcement_is_prerelease || fromJson(inputs.plan).publish_prereleases }}
     steps:
-      - name: Fetch npm packages
+      # Download from artifacts when called from release workflow
+      - name: Fetch npm packages from artifacts
+        if: ${{ inputs.plan }}
         uses: actions/download-artifact@v4
         with:
           pattern: artifacts-*
           path: npm/
           merge-multiple: true
+      # Download from GitHub release when triggered manually
+      - name: Download npm package from release
+        if: ${{ !inputs.plan && inputs.version }}
+        run: |
+          mkdir -p npm
+          gh release download "v${VERSION}" --repo "${{ github.repository }}" --pattern "*-npm-package.tar.gz" --dir npm/
       - uses: actions/setup-node@v4
         with:
           node-version: '24.x'
@@ -37,7 +55,15 @@ jobs:
         env:
           NPM_CONFIG_PROVENANCE: true
         run: |
-          for release in $(echo "$PLAN" | jq --compact-output '.releases[] | select([.artifacts[] | endswith("-npm-package.tar.gz")] | any)'); do
-            pkg=$(echo "$release" | jq '.artifacts[] | select(endswith("-npm-package.tar.gz"))' --raw-output)
-            npm publish --access public --provenance "./npm/${pkg}"
-          done
+          if [ -n "$PLAN" ]; then
+            # Called from release workflow - use plan to find packages
+            for release in $(echo "$PLAN" | jq --compact-output '.releases[] | select([.artifacts[] | endswith("-npm-package.tar.gz")] | any)'); do
+              pkg=$(echo "$release" | jq '.artifacts[] | select(endswith("-npm-package.tar.gz"))' --raw-output)
+              npm publish --access public --provenance "./npm/${pkg}"
+            done
+          else
+            # Manual trigger - publish all npm packages found
+            for pkg in npm/*-npm-package.tar.gz; do
+              npm publish --access public --provenance "$pkg"
+            done
+          fi

src/app/async_messages.rs

@@ -549,9 +549,10 @@ impl Editor {
 
     /// Handle file explorer expanded to path
     pub(super) fn handle_file_explorer_expanded_to_path(&mut self, mut view: FileTreeView) {
-        tracing::debug!("File explorer expanded to active file path");
+        tracing::trace!("handle_file_explorer_expanded_to_path: restoring file_explorer after async expand");
         view.update_scroll_for_selection();
         self.file_explorer = Some(view);
+        self.file_explorer_sync_in_progress = false;
     }
 }
 

src/app/file_explorer.rs

@@ -76,17 +76,28 @@ impl Editor {
             return;
         }
 
+        // Don't start a new sync if one is already in progress
+        if self.file_explorer_sync_in_progress {
+            return;
+        }
+
         if let Some(metadata) = self.buffer_metadata.get(&self.active_buffer()) {
             if let Some(file_path) = metadata.file_path() {
                 let target_path = file_path.clone();
                 let working_dir = self.working_dir.clone();
 
                 if target_path.starts_with(&working_dir) {
                     if let Some(mut view) = self.file_explorer.take() {
+                        tracing::trace!(
+                            "sync_file_explorer_to_active_file: taking file_explorer for async expand to {:?}",
+                            target_path
+                        );
                         if let (Some(runtime), Some(bridge)) =
                             (&self.tokio_runtime, &self.async_bridge)
                         {
                             let sender = bridge.sender();
+                            // Mark sync as in progress so render knows to keep the layout
+                            self.file_explorer_sync_in_progress = true;
 
                             runtime.spawn(async move {
                                 let _success = view.expand_and_select_file(&target_path).await;

src/app/mod.rs

@@ -188,6 +188,10 @@ pub struct Editor {
     /// Whether file explorer is visible
     file_explorer_visible: bool,
 
+    /// Whether file explorer is being synced to active file (async operation in progress)
+    /// When true, we still render the file explorer area even if file_explorer is temporarily None
+    file_explorer_sync_in_progress: bool,
+
     /// File explorer width as percentage (0.0 to 1.0)
     /// This is the runtime value that can be modified by dragging the border
     file_explorer_width_percent: f32,
@@ -693,6 +697,7 @@ impl Editor {
             file_explorer: None,
             fs_manager,
             file_explorer_visible: false,
+            file_explorer_sync_in_progress: false,
             file_explorer_width_percent: file_explorer_width,
             mouse_enabled: true,
             mouse_cursor_position: None,
@@ -2567,8 +2572,8 @@ impl Editor {
         // Use effective_tabs_width() to account for file explorer taking 30% of width
         self.ensure_active_tab_visible(active_split, buffer_id, self.effective_tabs_width());
 
-        // Sync file explorer to the new active file (if visible and applicable)
-        self.sync_file_explorer_to_active_file();
+        // Note: We don't sync file explorer here to avoid flicker during tab switches.
+        // File explorer syncs when explicitly focused via focus_file_explorer().
 
         // Emit buffer_activated hook for plugins
         self.plugin_manager.run_hook(
@@ -2622,7 +2627,8 @@ impl Editor {
                     view_state.add_buffer(buffer_id);
                     view_state.previous_buffer = Some(previous_buffer);
                 }
-                self.sync_file_explorer_to_active_file();
+                // Note: We don't sync file explorer here to avoid flicker during split focus changes.
+                // File explorer syncs when explicitly focused via focus_file_explorer().
             }
         } else {
             // Same split, different buffer (tab switch) - use set_active_buffer for terminal resume

src/app/render.rs

@@ -81,10 +81,18 @@ impl Editor {
         let prompt_line_idx = 4;
 
         // Split main content area based on file explorer visibility
+        // Also keep the layout split if a sync is in progress (to avoid flicker)
         let editor_content_area;
+        let file_explorer_should_show =
+            self.file_explorer_visible && (self.file_explorer.is_some() || self.file_explorer_sync_in_progress);
 
-        if self.file_explorer_visible && self.file_explorer.is_some() {
+        if file_explorer_should_show {
             // Split horizontally: [file_explorer | editor]
+            tracing::trace!(
+                "render: file explorer layout active (present={}, sync_in_progress={})",
+                self.file_explorer.is_some(),
+                self.file_explorer_sync_in_progress
+            );
             // Convert f32 percentage (0.0-1.0) to u16 percentage (0-100)
             let explorer_percent = (self.file_explorer_width_percent * 100.0) as u16;
             let editor_percent = 100 - explorer_percent;
@@ -99,7 +107,7 @@ impl Editor {
             self.cached_layout.file_explorer_area = Some(horizontal_chunks[0]);
             editor_content_area = horizontal_chunks[1];
 
-            // Render file explorer
+            // Render file explorer (only if we have it - during sync we just keep the area reserved)
             if let Some(ref mut explorer) = self.file_explorer {
                 let is_focused = self.key_context == KeyContext::FileExplorer;
 
@@ -131,6 +139,8 @@ impl Editor {
                     close_button_hovered,
                 );
             }
+            // Note: if file_explorer is None but sync_in_progress is true,
+            // we just leave the area blank (or could render a placeholder)
         } else {
             // No file explorer: use entire main content area for editor
             self.cached_layout.file_explorer_area = None;