@fresh-editor/fresh-editor 0.3.0 → 0.3.1

package/plugins/cmake-lsp.ts

@@ -49,7 +49,8 @@ let cmakeLspError: { serverCommand: string; message: string } | null = null;
 /**
  * Handle LSP server errors for CMake
  */
-function on_cmake_lsp_server_error(data: LspServerErrorData): void {
+
+editor.on("lsp_server_error", (data) => {
   if (data.language !== "cmake") {
     return;
   }
@@ -68,14 +69,13 @@ function on_cmake_lsp_server_error(data: LspServerErrorData): void {
   } else {
     editor.setStatus(`CMake LSP error: ${data.message}`);
   }
-}
-registerHandler("on_cmake_lsp_server_error", on_cmake_lsp_server_error);
-editor.on("lsp_server_error", "on_cmake_lsp_server_error");
+});
 
 /**
  * Handle status bar click when there's a CMake LSP error
  */
-function on_cmake_lsp_status_clicked(data: LspStatusClickedData): void {
+
+editor.on("lsp_status_clicked", (data) => {
   if (data.language !== "cmake" || !cmakeLspError) {
     return;
   }
@@ -93,14 +93,13 @@ function on_cmake_lsp_status_clicked(data: LspStatusClickedData): void {
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-}
-registerHandler("on_cmake_lsp_status_clicked", on_cmake_lsp_status_clicked);
-editor.on("lsp_status_clicked", "on_cmake_lsp_status_clicked");
+});
 
 /**
  * Handle action popup results for CMake LSP help
  */
-function on_cmake_lsp_action_result(data: ActionPopupResultData): void {
+
+editor.on("action_popup_result", (data) => {
   if (data.popup_id !== "cmake-lsp-help") {
     return;
   }
@@ -131,8 +130,6 @@ function on_cmake_lsp_action_result(data: ActionPopupResultData): void {
     default:
       editor.debug(`cmake-lsp: Unknown action: ${data.action_id}`);
   }
-}
-registerHandler("on_cmake_lsp_action_result", on_cmake_lsp_action_result);
-editor.on("action_popup_result", "on_cmake_lsp_action_result");
+});
 
 editor.debug("cmake-lsp: Plugin loaded");

package/plugins/code-tour.ts

@@ -123,22 +123,7 @@ interface ActionPopupResultData {
   action_id: string;
 }
 
-function tour_on_action_popup_result(data: ActionPopupResultData) : void {
-  if (data.popup_id !== TOUR_POPUP_ID) return;
 
-  switch (data.action_id) {
-    case "next":
-      nextStep();
-      break;
-    case "prev":
-      prevStep();
-      break;
-    case "exit":
-      exitTour();
-      break;
-  }
-}
-registerHandler("tour_on_action_popup_result", tour_on_action_popup_result);
 
 // ============================================================================
 // Overlay Rendering
@@ -397,6 +382,20 @@ editor.registerCommand(
 );
 
 // Subscribe to action popup results for navigation buttons
-editor.on("action_popup_result", "tour_on_action_popup_result");
+editor.on("action_popup_result", (data) => {
+  if (data.popup_id !== TOUR_POPUP_ID) return;
+
+  switch (data.action_id) {
+    case "next":
+      nextStep();
+      break;
+    case "prev":
+      prevStep();
+      break;
+    case "exit":
+      exitTour();
+      break;
+  }
+});
 
 editor.debug("Code Tour plugin loaded");

package/plugins/config-schema.json

@@ -30,6 +30,7 @@
       "description": "Editor behavior settings (indentation, line numbers, wrapping, etc.)",
       "$ref": "#/$defs/EditorConfig",
       "default": {
+        "animations": true,
         "line_numbers": true,
         "relative_line_numbers": false,
         "highlight_current_line": true,
@@ -45,6 +46,7 @@
         "show_status_bar": true,
         "status_bar": {
           "left": [
+            "{remote}",
             "{filename}",
             "{cursor}",
             "{diagnostics}",
@@ -268,6 +270,12 @@
       "description": "Editor behavior configuration",
       "type": "object",
       "properties": {
+        "animations": {
+          "description": "Enable frame-buffer animations (tab-switch slides, dashboard\nbringup, plugin-driven effects). When `false`, every animation\ncall is a no-op: the UI is fully static and each render lands\nthe final frame immediately. Useful on slow terminals, over\nSSH, or for users who prefer no motion.",
+          "type": "boolean",
+          "default": true,
+          "x-section": "Display"
+        },
         "line_numbers": {
           "description": "Show line numbers in the gutter (default for new buffers)",
           "type": "boolean",
@@ -361,6 +369,7 @@
           "$ref": "#/$defs/StatusBarConfig",
           "default": {
             "left": [
+              "{remote}",
               "{filename}",
               "{cursor}",
               "{diagnostics}",
@@ -751,6 +760,7 @@
             "$ref": "#/$defs/StatusBarElement"
           },
           "default": [
+            "{remote}",
             "{filename}",
             "{cursor}",
             "{diagnostics}",

package/plugins/csharp_support.ts

@@ -169,7 +169,10 @@ async function restoreProject(projectPath: string): Promise<void> {
 /**
  * Handle file open - set project root and restore packages
  */
-async function on_csharp_file_open(data: AfterFileOpenData) : Promise<void> {
+
+
+// Register hook for file open
+editor.on("after_file_open", async (data) => {
   // Only handle .cs files
   if (!data.path.endsWith(".cs")) {
     return;
@@ -206,16 +209,15 @@ async function on_csharp_file_open(data: AfterFileOpenData) : Promise<void> {
       await restoreProject(dir);
     }
   }
-}
-registerHandler("on_csharp_file_open", on_csharp_file_open);
-
-// Register hook for file open
-editor.on("after_file_open", "on_csharp_file_open");
+});
 
 /**
  * Handle LSP server requests from C# language servers (Roslyn-based)
  */
-function on_csharp_lsp_server_request(data: LspServerRequestData) : void {
+
+
+// Register hook for LSP server requests
+editor.on("lsp_server_request", (data) => {
   // Only handle requests from C# language servers
   if (data.server_command !== "csharp-ls" && data.server_command !== "csharp-language-server") {
     return;
@@ -243,16 +245,15 @@ function on_csharp_lsp_server_request(data: LspServerRequestData) : void {
       // Log unhandled requests for debugging
       editor.debug(`csharp_support: Unhandled LSP request: ${data.method}`);
   }
-}
-registerHandler("on_csharp_lsp_server_request", on_csharp_lsp_server_request);
-
-// Register hook for LSP server requests
-editor.on("lsp_server_request", "on_csharp_lsp_server_request");
+});
 
 /**
  * Handle LSP server errors for C#
  */
-function on_csharp_lsp_server_error(data: LspServerErrorData) : void {
+
+
+// Register hook for LSP server errors
+editor.on("lsp_server_error", (data) => {
   // Only handle C# language errors
   if (data.language !== "csharp") {
     return;
@@ -274,16 +275,15 @@ function on_csharp_lsp_server_error(data: LspServerErrorData) : void {
   } else {
     editor.setStatus(`C# LSP error: ${data.message}`);
   }
-}
-registerHandler("on_csharp_lsp_server_error", on_csharp_lsp_server_error);
-
-// Register hook for LSP server errors
-editor.on("lsp_server_error", "on_csharp_lsp_server_error");
+});
 
 /**
  * Handle status bar click when there's a C# LSP error
  */
-function on_csharp_lsp_status_clicked(data: LspStatusClickedData) : void {
+
+
+// Register hook for status bar clicks
+editor.on("lsp_status_clicked", (data) => {
   // Only handle C# language clicks when there's an error
   if (data.language !== "csharp" || !csharpLspError) {
     return;
@@ -302,16 +302,15 @@ function on_csharp_lsp_status_clicked(data: LspStatusClickedData) : void {
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-}
-registerHandler("on_csharp_lsp_status_clicked", on_csharp_lsp_status_clicked);
-
-// Register hook for status bar clicks
-editor.on("lsp_status_clicked", "on_csharp_lsp_status_clicked");
+});
 
 /**
  * Handle action popup results for C# LSP help
  */
-function on_csharp_lsp_action_result(data: ActionPopupResultData) : void {
+
+
+// Register hook for action popup results
+editor.on("action_popup_result", (data) => {
   // Only handle our popup
   if (data.popup_id !== "csharp-lsp-help") {
     return;
@@ -339,10 +338,6 @@ function on_csharp_lsp_action_result(data: ActionPopupResultData) : void {
     default:
       editor.debug(`csharp_support: Unknown action: ${data.action_id}`);
   }
-}
-registerHandler("on_csharp_lsp_action_result", on_csharp_lsp_action_result);
-
-// Register hook for action popup results
-editor.on("action_popup_result", "on_csharp_lsp_action_result");
+});
 
 editor.debug("csharp_support: Plugin loaded");

package/plugins/css-lsp.ts

@@ -46,7 +46,10 @@ let cssLspError: { serverCommand: string; message: string } | null = null;
 /**
  * Handle LSP server errors for CSS
  */
-function on_css_lsp_server_error(data: LspServerErrorData) : void {
+
+
+// Register hook for LSP server errors
+editor.on("lsp_server_error", (data) => {
   // Only handle CSS language errors
   if (data.language !== "css") {
     return;
@@ -68,18 +71,15 @@ function on_css_lsp_server_error(data: LspServerErrorData) : void {
   } else {
     editor.setStatus(`CSS LSP error: ${data.message}`);
   }
-}
-registerHandler("on_css_lsp_server_error", on_css_lsp_server_error);
-
-// Register hook for LSP server errors
-editor.on("lsp_server_error", "on_css_lsp_server_error");
+});
 
 /**
  * Handle status bar click when there's a CSS LSP error
  */
-function on_css_lsp_status_clicked(
-  data: LspStatusClickedData
-): void {
+
+
+// Register hook for status bar clicks
+editor.on("lsp_status_clicked", (data) => {
   // Only handle CSS language clicks when there's an error
   if (data.language !== "css" || !cssLspError) {
     return;
@@ -98,18 +98,15 @@ function on_css_lsp_status_clicked(
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-}
-registerHandler("on_css_lsp_status_clicked", on_css_lsp_status_clicked);
-
-// Register hook for status bar clicks
-editor.on("lsp_status_clicked", "on_css_lsp_status_clicked");
+});
 
 /**
  * Handle action popup results for CSS LSP help
  */
-function on_css_lsp_action_result(
-  data: ActionPopupResultData
-): void {
+
+
+// Register hook for action popup results
+editor.on("action_popup_result", (data) => {
   // Only handle our popup
   if (data.popup_id !== "css-lsp-help") {
     return;
@@ -137,10 +134,6 @@ function on_css_lsp_action_result(
     default:
       editor.debug(`css-lsp: Unknown action: ${data.action_id}`);
   }
-}
-registerHandler("on_css_lsp_action_result", on_css_lsp_action_result);
-
-// Register hook for action popup results
-editor.on("action_popup_result", "on_css_lsp_action_result");
+});
 
 editor.debug("css-lsp: Plugin loaded");

package/plugins/dart-lsp.ts

@@ -49,7 +49,8 @@ let dartLspError: { serverCommand: string; message: string } | null = null;
 /**
  * Handle LSP server errors for Dart
  */
-function on_dart_lsp_server_error(data: LspServerErrorData): void {
+
+editor.on("lsp_server_error", (data) => {
   if (data.language !== "dart") {
     return;
   }
@@ -68,14 +69,13 @@ function on_dart_lsp_server_error(data: LspServerErrorData): void {
   } else {
     editor.setStatus(`Dart LSP error: ${data.message}`);
   }
-}
-registerHandler("on_dart_lsp_server_error", on_dart_lsp_server_error);
-editor.on("lsp_server_error", "on_dart_lsp_server_error");
+});
 
 /**
  * Handle status bar click when there's a Dart LSP error
  */
-function on_dart_lsp_status_clicked(data: LspStatusClickedData): void {
+
+editor.on("lsp_status_clicked", (data) => {
   if (data.language !== "dart" || !dartLspError) {
     return;
   }
@@ -94,14 +94,13 @@ function on_dart_lsp_status_clicked(data: LspStatusClickedData): void {
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-}
-registerHandler("on_dart_lsp_status_clicked", on_dart_lsp_status_clicked);
-editor.on("lsp_status_clicked", "on_dart_lsp_status_clicked");
+});
 
 /**
  * Handle action popup results for Dart LSP help
  */
-function on_dart_lsp_action_result(data: ActionPopupResultData): void {
+
+editor.on("action_popup_result", (data) => {
   if (data.popup_id !== "dart-lsp-help") {
     return;
   }
@@ -137,8 +136,6 @@ function on_dart_lsp_action_result(data: ActionPopupResultData): void {
     default:
       editor.debug(`dart-lsp: Unknown action: ${data.action_id}`);
   }
-}
-registerHandler("on_dart_lsp_action_result", on_dart_lsp_action_result);
-editor.on("action_popup_result", "on_dart_lsp_action_result");
+});
 
 editor.debug("dart-lsp: Plugin loaded");

package/plugins/dashboard.ts

@@ -201,6 +201,55 @@ type RegisteredSection = {
 let dashboardBufferId: number | null = null;
 let fetchToken = 0; // bumped each open; late fetches from a prior open no-op.
 
+// Id of the in-flight slide-in, so we can cancel it when starting a
+// new one (on content change) or when the dashboard is closed
+// mid-slide. Null once the animation settles or is cleared.
+let activeAnimationId: number | null = null;
+
+// Hash of all entries at the last paint (post-focus-highlight too —
+// it's what ultimately lands in the virtual buffer). Used to decide
+// whether setVirtualBufferContent needs to run at all: identical
+// full hash AND identical focus → nothing changed, skip the VB
+// round-trip entirely.
+let lastPaintedFullKey: string | null = null;
+
+// Hash of the entries with the clock stamp stripped. Animations only
+// fire when THIS hash changes, so the 1 Hz clock tick on the top
+// frame updates in place without re-sliding the whole dashboard.
+// Keyboard focus changes don't move this hash either (the hash is
+// taken before the focus overlay is laid on top), so Tab/Shift-Tab
+// pan the highlight without re-animating.
+let lastPaintedStructuralKey: string | null = null;
+
+// focusedIndex the last successful setVirtualBufferContent ran with.
+// Paired with the keys above so we can tell "focus moved but section
+// data is the same" (update VB for the highlight, no animation).
+let lastPaintedFocusedIndex = -1;
+
+// Matches an HH:MM:SS clock stamp. Anything shaped like that is
+// stripped from the structural hash so clock ticks don't animate.
+// The frame renderer is the only dashboard author that emits such a
+// string; if a third-party section happens to show a value in the
+// same shape, the worst case is "we don't re-animate when that
+// value changes" — acceptable noise floor.
+const CLOCK_RE = /\d\d:\d\d:\d\d/g;
+
+// Edge the slide-in enters from. Maps 1:1 to the plugin API's `from`
+// field and is resolved from config (plugins.dashboard.slide_from) on
+// each paint() so hot-reload of the setting Just Works. Defaults to
+// "right" (new content pushes in from the right, old exits left).
+type SlideFrom = "top" | "bottom" | "left" | "right";
+function resolveSlideFrom(): SlideFrom {
+    const config = editor.getConfig() as Record<string, unknown> | null;
+    const plugins = config?.plugins as Record<string, unknown> | undefined;
+    const dashCfg = plugins?.dashboard as Record<string, unknown> | undefined;
+    const raw = dashCfg?.slide_from;
+    if (raw === "top" || raw === "bottom" || raw === "left" || raw === "right") {
+        return raw;
+    }
+    return "right";
+}
+
 // Registered sections, in render order. Built-ins are registered at
 // plugin load (see the bottom of this file); third-party plugins
 // append via the exported `registerSection` API.
@@ -726,6 +775,36 @@ function paint(dims?: { width: number; height: number }) {
             ((focusedIndex % targets.length) + targets.length) % targets.length;
     }
 
+    // Two hashes, taken BEFORE the focus highlight goes on top:
+    //   fullKey      — everything including the clock. Drives the
+    //                  setVirtualBufferContent skip check, so the
+    //                  clock still redraws in place every second.
+    //   structuralKey — clock stamps stripped. Drives the animation.
+    //                  A clock tick alone does not flip this, so it
+    //                  updates silently; a real section data change
+    //                  does, and the slide fires.
+    const fullKey = JSON.stringify(entries);
+    const structuralKey = fullKey.replace(CLOCK_RE, "##:##:##");
+    const fullChanged = fullKey !== lastPaintedFullKey;
+    const structuralChanged = structuralKey !== lastPaintedStructuralKey;
+    const focusChanged = focusedIndex !== lastPaintedFocusedIndex;
+    // A resize / viewport-shape change reshapes frame padding, dash
+    // runs, and centering, which flips the structural hash even when
+    // section data is unchanged. We repaint so the new layout takes
+    // effect but skip the slide — nothing NEW showed up, the user is
+    // just resizing a window. openDashboard clears lastPaintedW/H to
+    // -1 so the first paint after open doesn't trip this guard.
+    const dimsChanged =
+        lastPaintedW !== -1 &&
+        lastPaintedH !== -1 &&
+        (width !== lastPaintedW || height !== lastPaintedH);
+
+    // Identical render → short-circuit. Nothing to push to the
+    // buffer, nothing to animate.
+    if (!fullChanged && !focusChanged) {
+        return;
+    }
+
     // Paint the focus highlight by mutating the entry for the focused
     // row: translate its visual col range into a byte range and push an
     // inline overlay on top of whatever foreground/underline spans the
@@ -758,6 +837,26 @@ function paint(dims?: { width: number; height: number }) {
     editor.setVirtualBufferContent(bufferId, entries);
     lastPaintedW = width;
     lastPaintedH = height;
+    lastPaintedFullKey = fullKey;
+    lastPaintedStructuralKey = structuralKey;
+    lastPaintedFocusedIndex = focusedIndex;
+
+    // Structural-change-driven re-animation: fire only when the
+    // section payload actually differs AND the dashboard isn't just
+    // reshaping in place (clock tick, focus move, and resize all
+    // land here without animating). Cancel any in-flight slide
+    // first so the new one snapshots the fresh content.
+    if (structuralChanged && !dimsChanged) {
+        if (activeAnimationId !== null) {
+            editor.cancelAnimation(activeAnimationId);
+        }
+        activeAnimationId = editor.animateVirtualBuffer(bufferId, {
+            kind: "slideIn",
+            from: resolveSlideFrom(),
+            durationMs: 520,
+            delayMs: 0,
+        });
+    }
 }
 
 // Open a URL in the user's browser via the platform's "open" helper.
@@ -1513,6 +1612,17 @@ async function openDashboard() {
         }
     }
 
+    // Clear the content/focus keys and dims so the first paint after
+    // open is treated as a content change and the slide-in fires.
+    // Dim reset is needed because open is the one case where we DO
+    // want the animation despite "dims changed" (there was no prior
+    // dimension, so the change is really "buffer just appeared").
+    lastPaintedFullKey = null;
+    lastPaintedStructuralKey = null;
+    lastPaintedFocusedIndex = -1;
+    lastPaintedW = -1;
+    lastPaintedH = -1;
+
     bootstrapDashboard(dashboardBufferId);
 }
 
@@ -1584,6 +1694,10 @@ registerHandler(
         // If the dashboard itself was closed, clear our handle so we'll
         // re-open on the next "last tab closed" event.
         if (dashboardBufferId !== null && e.buffer_id === dashboardBufferId) {
+            if (activeAnimationId !== null) {
+                editor.cancelAnimation(activeAnimationId);
+                activeAnimationId = null;
+            }
             dashboardBufferId = null;
             return;
         }
@@ -1603,6 +1717,10 @@ registerHandler(
     "dashboardOnAfterFileOpen",
     (_e: { buffer_id: number; path: string }) => {
         if (dashboardBufferId === null) return;
+        if (activeAnimationId !== null) {
+            editor.cancelAnimation(activeAnimationId);
+            activeAnimationId = null;
+        }
         editor.closeBuffer(dashboardBufferId);
         dashboardBufferId = null;
     },