@fresh-editor/fresh-editor 0.2.11 → 0.2.13

package/plugins/git_gutter.ts

@@ -317,7 +317,7 @@ async function updateGitGutter(bufferId: number): Promise<void> {
 /**
  * Handle after file open - initialize git state and update indicators
  */
-globalThis.onGitGutterAfterFileOpen = function (args: {
+function onGitGutterAfterFileOpen(args: {
   buffer_id: number;
   path: string;
 }): boolean {
@@ -339,12 +339,13 @@ globalThis.onGitGutterAfterFileOpen = function (args: {
   updateGitGutter(bufferId);
 
   return true;
-};
+}
+registerHandler("onGitGutterAfterFileOpen", onGitGutterAfterFileOpen);
 
 /**
  * Handle buffer activation - update if we have state but indicators might be stale
  */
-globalThis.onGitGutterBufferActivated = function (args: {
+function onGitGutterBufferActivated(args: {
   buffer_id: number;
 }): boolean {
   const bufferId = args.buffer_id;
@@ -365,12 +366,13 @@ globalThis.onGitGutterBufferActivated = function (args: {
   // (they update on file open and save)
 
   return true;
-};
+}
+registerHandler("onGitGutterBufferActivated", onGitGutterBufferActivated);
 
 /**
  * Handle after file save - refresh indicators
  */
-globalThis.onGitGutterAfterSave = function (args: {
+function onGitGutterAfterSave(args: {
   buffer_id: number;
   path: string;
 }): boolean {
@@ -392,7 +394,8 @@ globalThis.onGitGutterAfterSave = function (args: {
   updateGitGutter(bufferId);
 
   return true;
-};
+}
+registerHandler("onGitGutterAfterSave", onGitGutterAfterSave);
 
 // Note: Git diff compares the file on disk, not the in-memory buffer.
 // Line indicators automatically track position changes via byte-position markers.
@@ -401,12 +404,13 @@ globalThis.onGitGutterAfterSave = function (args: {
 /**
  * Handle buffer closed - cleanup state
  */
-globalThis.onGitGutterBufferClosed = function (args: {
+function onGitGutterBufferClosed(args: {
   buffer_id: number;
 }): boolean {
   bufferStates.delete(args.buffer_id);
   return true;
-};
+}
+registerHandler("onGitGutterBufferClosed", onGitGutterBufferClosed);
 
 // =============================================================================
 // Commands
@@ -415,7 +419,7 @@ globalThis.onGitGutterBufferClosed = function (args: {
 /**
  * Manually refresh git gutter for the current buffer
  */
-globalThis.git_gutter_refresh = function (): void {
+function git_gutter_refresh() : void {
   const bufferId = editor.getActiveBufferId();
   const filePath = editor.getBufferPath(bufferId);
 
@@ -439,7 +443,8 @@ globalThis.git_gutter_refresh = function (): void {
     const count = state?.hunks.length || 0;
     editor.setStatus(editor.t("status.changes", { count: String(count) }));
   });
-};
+}
+registerHandler("git_gutter_refresh", git_gutter_refresh);
 
 // =============================================================================
 // Registration

package/plugins/git_log.ts

@@ -628,7 +628,7 @@ function applyCommitDetailHighlighting(): void {
 // Public Commands - Git Log
 // =============================================================================
 
-globalThis.show_git_log = async function(): Promise<void> {
+async function show_git_log() : Promise<void> {
   if (gitLogState.isOpen) {
     editor.setStatus(editor.t("status.already_open"));
     return;
@@ -678,9 +678,10 @@ globalThis.show_git_log = async function(): Promise<void> {
     gitLogState.splitId = null;
     editor.setStatus(editor.t("status.failed_open"));
   }
-};
+}
+registerHandler("show_git_log", show_git_log);
 
-globalThis.git_log_close = function(): void {
+function git_log_close() : void {
   if (!gitLogState.isOpen) {
     return;
   }
@@ -701,10 +702,11 @@ globalThis.git_log_close = function(): void {
   gitLogState.sourceBufferId = null;
   gitLogState.commits = [];
   editor.setStatus(editor.t("status.closed"));
-};
+}
+registerHandler("git_log_close", git_log_close);
 
 // Cursor moved handler for git log - update highlighting and status
-globalThis.on_git_log_cursor_moved = function(data: {
+function on_git_log_cursor_moved(data: {
   buffer_id: number;
   cursor_id: number;
   old_position: number;
@@ -726,19 +728,21 @@ globalThis.on_git_log_cursor_moved = function(data: {
   if (commitIndex >= 0 && commitIndex < gitLogState.commits.length) {
     editor.setStatus(editor.t("status.commit_position", { current: String(commitIndex + 1), total: String(gitLogState.commits.length) }));
   }
-};
+}
+registerHandler("on_git_log_cursor_moved", on_git_log_cursor_moved);
 
 // Register cursor movement handler
 editor.on("cursor_moved", "on_git_log_cursor_moved");
 
-globalThis.git_log_refresh = async function(): Promise<void> {
+async function git_log_refresh() : Promise<void> {
   if (!gitLogState.isOpen) return;
 
   editor.setStatus(editor.t("status.refreshing"));
   gitLogState.commits = await fetchGitLog();
   updateGitLogView();
   editor.setStatus(editor.t("status.refreshed", { count: String(gitLogState.commits.length) }));
-};
+}
+registerHandler("git_log_refresh", git_log_refresh);
 
 // Helper function to get commit at current cursor position
 function getCommitAtCursor(): GitCommit | null {
@@ -766,7 +770,7 @@ function getCommitAtCursor(): GitCommit | null {
   return null;
 }
 
-globalThis.git_log_show_commit = async function(): Promise<void> {
+async function git_log_show_commit() : Promise<void> {
   if (!gitLogState.isOpen || gitLogState.commits.length === 0) return;
   if (gitLogState.splitId === null) return;
 
@@ -812,9 +816,10 @@ globalThis.git_log_show_commit = async function(): Promise<void> {
   } else {
     editor.setStatus(editor.t("status.failed_open_details"));
   }
-};
+}
+registerHandler("git_log_show_commit", git_log_show_commit);
 
-globalThis.git_log_copy_hash = function(): void {
+function git_log_copy_hash() : void {
   if (!gitLogState.isOpen || gitLogState.commits.length === 0) return;
 
   const commit = getCommitAtCursor();
@@ -826,13 +831,14 @@ globalThis.git_log_copy_hash = function(): void {
   // Copy hash to clipboard
   editor.copyToClipboard(commit.hash);
   editor.setStatus(editor.t("status.hash_copied", { short: commit.shortHash, full: commit.hash }));
-};
+}
+registerHandler("git_log_copy_hash", git_log_copy_hash);
 
 // =============================================================================
 // Public Commands - Commit Detail
 // =============================================================================
 
-globalThis.git_commit_detail_close = function(): void {
+function git_commit_detail_close() : void {
   if (!commitDetailState.isOpen) {
     return;
   }
@@ -855,10 +861,11 @@ globalThis.git_commit_detail_close = function(): void {
   commitDetailState.commit = null;
 
   editor.setStatus(editor.t("status.log_ready", { count: String(gitLogState.commits.length) }));
-};
+}
+registerHandler("git_commit_detail_close", git_commit_detail_close);
 
 // Close file view and go back to commit detail
-globalThis.git_file_view_close = function(): void {
+function git_file_view_close() : void {
   if (!fileViewState.isOpen) {
     return;
   }
@@ -884,7 +891,8 @@ globalThis.git_file_view_close = function(): void {
   if (commitDetailState.commit) {
     editor.setStatus(editor.t("status.commit_ready", { hash: commitDetailState.commit.shortHash }));
   }
-};
+}
+registerHandler("git_file_view_close", git_file_view_close);
 
 // Fetch file content at a specific commit
 async function fetchFileAtCommit(commitHash: string, filePath: string): Promise<string | null> {
@@ -1074,7 +1082,7 @@ function applyFileViewHighlighting(bufferId: number, content: string, filePath:
 }
 
 // Open file at the current diff line position - shows file as it was at that commit
-globalThis.git_commit_detail_open_file = async function(): Promise<void> {
+async function git_commit_detail_open_file() : Promise<void> {
   if (!commitDetailState.isOpen || commitDetailState.bufferId === null) {
     return;
   }
@@ -1148,7 +1156,8 @@ globalThis.git_commit_detail_open_file = async function(): Promise<void> {
   } else {
     editor.setStatus(editor.t("status.move_to_diff"));
   }
-};
+}
+registerHandler("git_commit_detail_open_file", git_commit_detail_open_file);
 
 // =============================================================================
 // Command Registration

package/plugins/go-lsp.ts

@@ -46,7 +46,7 @@ let goLspError: { serverCommand: string; message: string } | null = null;
 /**
  * Handle LSP server errors for Go
  */
-globalThis.on_go_lsp_server_error = function (data: LspServerErrorData): void {
+function on_go_lsp_server_error(data: LspServerErrorData) : void {
   // Only handle Go language errors
   if (data.language !== "go") {
     return;
@@ -68,7 +68,8 @@ globalThis.on_go_lsp_server_error = function (data: LspServerErrorData): void {
   } else {
     editor.setStatus(`Go LSP error: ${data.message}`);
   }
-};
+}
+registerHandler("on_go_lsp_server_error", on_go_lsp_server_error);
 
 // Register hook for LSP server errors
 editor.on("lsp_server_error", "on_go_lsp_server_error");
@@ -76,7 +77,7 @@ editor.on("lsp_server_error", "on_go_lsp_server_error");
 /**
  * Handle status bar click when there's a Go LSP error
  */
-globalThis.on_go_lsp_status_clicked = function (
+function on_go_lsp_status_clicked(
   data: LspStatusClickedData
 ): void {
   // Only handle Go language clicks when there's an error
@@ -97,7 +98,8 @@ globalThis.on_go_lsp_status_clicked = function (
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-};
+}
+registerHandler("on_go_lsp_status_clicked", on_go_lsp_status_clicked);
 
 // Register hook for status bar clicks
 editor.on("lsp_status_clicked", "on_go_lsp_status_clicked");
@@ -105,7 +107,7 @@ editor.on("lsp_status_clicked", "on_go_lsp_status_clicked");
 /**
  * Handle action popup results for Go LSP help
  */
-globalThis.on_go_lsp_action_result = function (
+function on_go_lsp_action_result(
   data: ActionPopupResultData
 ): void {
   // Only handle our popup
@@ -135,7 +137,8 @@ globalThis.on_go_lsp_action_result = function (
     default:
       editor.debug(`go-lsp: Unknown action: ${data.action_id}`);
   }
-};
+}
+registerHandler("on_go_lsp_action_result", on_go_lsp_action_result);
 
 // Register hook for action popup results
 editor.on("action_popup_result", "on_go_lsp_action_result");

package/plugins/html-lsp.ts

@@ -46,7 +46,7 @@ let htmlLspError: { serverCommand: string; message: string } | null = null;
 /**
  * Handle LSP server errors for HTML
  */
-globalThis.on_html_lsp_server_error = function (
+function on_html_lsp_server_error(
   data: LspServerErrorData
 ): void {
   // Only handle HTML language errors
@@ -70,7 +70,8 @@ globalThis.on_html_lsp_server_error = function (
   } else {
     editor.setStatus(`HTML LSP error: ${data.message}`);
   }
-};
+}
+registerHandler("on_html_lsp_server_error", on_html_lsp_server_error);
 
 // Register hook for LSP server errors
 editor.on("lsp_server_error", "on_html_lsp_server_error");
@@ -78,7 +79,7 @@ editor.on("lsp_server_error", "on_html_lsp_server_error");
 /**
  * Handle status bar click when there's an HTML LSP error
  */
-globalThis.on_html_lsp_status_clicked = function (
+function on_html_lsp_status_clicked(
   data: LspStatusClickedData
 ): void {
   // Only handle HTML language clicks when there's an error
@@ -99,7 +100,8 @@ globalThis.on_html_lsp_status_clicked = function (
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-};
+}
+registerHandler("on_html_lsp_status_clicked", on_html_lsp_status_clicked);
 
 // Register hook for status bar clicks
 editor.on("lsp_status_clicked", "on_html_lsp_status_clicked");
@@ -107,7 +109,7 @@ editor.on("lsp_status_clicked", "on_html_lsp_status_clicked");
 /**
  * Handle action popup results for HTML LSP help
  */
-globalThis.on_html_lsp_action_result = function (
+function on_html_lsp_action_result(
   data: ActionPopupResultData
 ): void {
   // Only handle our popup
@@ -137,7 +139,8 @@ globalThis.on_html_lsp_action_result = function (
     default:
       editor.debug(`html-lsp: Unknown action: ${data.action_id}`);
   }
-};
+}
+registerHandler("on_html_lsp_action_result", on_html_lsp_action_result);
 
 // Register hook for action popup results
 editor.on("action_popup_result", "on_html_lsp_action_result");

package/plugins/java-lsp.ts

@@ -22,7 +22,7 @@ interface ActionPopupResultData {
 const INSTALL_URL = "https://github.com/eclipse-jdtls/eclipse.jdt.ls#installation";
 let javaLspError: { serverCommand: string; message: string } | null = null;
 
-globalThis.on_java_lsp_server_error = function (data: LspServerErrorData): void {
+function on_java_lsp_server_error(data: LspServerErrorData) : void {
   if (data.language !== "java") return;
   javaLspError = { serverCommand: data.server_command, message: data.message };
   if (data.error_type === "not_found") {
@@ -30,10 +30,11 @@ globalThis.on_java_lsp_server_error = function (data: LspServerErrorData): void
   } else {
     editor.setStatus(`Java LSP error: ${data.message}`);
   }
-};
+}
+registerHandler("on_java_lsp_server_error", on_java_lsp_server_error);
 editor.on("lsp_server_error", "on_java_lsp_server_error");
 
-globalThis.on_java_lsp_status_clicked = function (data: LspStatusClickedData): void {
+function on_java_lsp_status_clicked(data: LspStatusClickedData) : void {
   if (data.language !== "java" || !javaLspError) return;
   editor.showActionPopup({
     id: "java-lsp-help",
@@ -45,10 +46,11 @@ globalThis.on_java_lsp_status_clicked = function (data: LspStatusClickedData): v
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-};
+}
+registerHandler("on_java_lsp_status_clicked", on_java_lsp_status_clicked);
 editor.on("lsp_status_clicked", "on_java_lsp_status_clicked");
 
-globalThis.on_java_lsp_action_result = function (data: ActionPopupResultData): void {
+function on_java_lsp_action_result(data: ActionPopupResultData) : void {
   if (data.popup_id !== "java-lsp-help") return;
   switch (data.action_id) {
     case "copy_url":
@@ -61,5 +63,6 @@ globalThis.on_java_lsp_action_result = function (data: ActionPopupResultData): v
       javaLspError = null;
       break;
   }
-};
+}
+registerHandler("on_java_lsp_action_result", on_java_lsp_action_result);
 editor.on("action_popup_result", "on_java_lsp_action_result");

package/plugins/json-lsp.ts

@@ -46,7 +46,7 @@ let jsonLspError: { serverCommand: string; message: string } | null = null;
 /**
  * Handle LSP server errors for JSON
  */
-globalThis.on_json_lsp_server_error = function (
+function on_json_lsp_server_error(
   data: LspServerErrorData
 ): void {
   // Only handle JSON language errors
@@ -70,7 +70,8 @@ globalThis.on_json_lsp_server_error = function (
   } else {
     editor.setStatus(`JSON LSP error: ${data.message}`);
   }
-};
+}
+registerHandler("on_json_lsp_server_error", on_json_lsp_server_error);
 
 // Register hook for LSP server errors
 editor.on("lsp_server_error", "on_json_lsp_server_error");
@@ -78,7 +79,7 @@ editor.on("lsp_server_error", "on_json_lsp_server_error");
 /**
  * Handle status bar click when there's a JSON LSP error
  */
-globalThis.on_json_lsp_status_clicked = function (
+function on_json_lsp_status_clicked(
   data: LspStatusClickedData
 ): void {
   // Only handle JSON language clicks when there's an error
@@ -99,7 +100,8 @@ globalThis.on_json_lsp_status_clicked = function (
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-};
+}
+registerHandler("on_json_lsp_status_clicked", on_json_lsp_status_clicked);
 
 // Register hook for status bar clicks
 editor.on("lsp_status_clicked", "on_json_lsp_status_clicked");
@@ -107,7 +109,7 @@ editor.on("lsp_status_clicked", "on_json_lsp_status_clicked");
 /**
  * Handle action popup results for JSON LSP help
  */
-globalThis.on_json_lsp_action_result = function (
+function on_json_lsp_action_result(
   data: ActionPopupResultData
 ): void {
   // Only handle our popup
@@ -137,7 +139,8 @@ globalThis.on_json_lsp_action_result = function (
     default:
       editor.debug(`json-lsp: Unknown action: ${data.action_id}`);
   }
-};
+}
+registerHandler("on_json_lsp_action_result", on_json_lsp_action_result);
 
 // Register hook for action popup results
 editor.on("action_popup_result", "on_json_lsp_action_result");

package/plugins/latex-lsp.ts

@@ -22,7 +22,7 @@ interface ActionPopupResultData {
 const INSTALL_URL = "https://github.com/latex-lsp/texlab#installation";
 let latexLspError: { serverCommand: string; message: string } | null = null;
 
-globalThis.on_latex_lsp_server_error = function (data: LspServerErrorData): void {
+function on_latex_lsp_server_error(data: LspServerErrorData) : void {
   if (data.language !== "latex") return;
   latexLspError = { serverCommand: data.server_command, message: data.message };
   if (data.error_type === "not_found") {
@@ -30,10 +30,11 @@ globalThis.on_latex_lsp_server_error = function (data: LspServerErrorData): void
   } else {
     editor.setStatus(`LaTeX LSP error: ${data.message}`);
   }
-};
+}
+registerHandler("on_latex_lsp_server_error", on_latex_lsp_server_error);
 editor.on("lsp_server_error", "on_latex_lsp_server_error");
 
-globalThis.on_latex_lsp_status_clicked = function (data: LspStatusClickedData): void {
+function on_latex_lsp_status_clicked(data: LspStatusClickedData) : void {
   if (data.language !== "latex" || !latexLspError) return;
   editor.showActionPopup({
     id: "latex-lsp-help",
@@ -45,10 +46,11 @@ globalThis.on_latex_lsp_status_clicked = function (data: LspStatusClickedData):
       { id: "dismiss", label: "Dismiss (ESC)" },
     ],
   });
-};
+}
+registerHandler("on_latex_lsp_status_clicked", on_latex_lsp_status_clicked);
 editor.on("lsp_status_clicked", "on_latex_lsp_status_clicked");
 
-globalThis.on_latex_lsp_action_result = function (data: ActionPopupResultData): void {
+function on_latex_lsp_action_result(data: ActionPopupResultData) : void {
   if (data.popup_id !== "latex-lsp-help") return;
   switch (data.action_id) {
     case "copy_url":
@@ -61,5 +63,6 @@ globalThis.on_latex_lsp_action_result = function (data: ActionPopupResultData):
       latexLspError = null;
       break;
   }
-};
+}
+registerHandler("on_latex_lsp_action_result", on_latex_lsp_action_result);
 editor.on("action_popup_result", "on_latex_lsp_action_result");

package/plugins/lib/finder.ts

@@ -883,6 +883,7 @@ export class Finder<T> {
 
     try {
       const content = await this.editor.readFile(entry.location.file);
+      if (!content) return;
       const lines = content.split("\n");
 
       const contextLines = this.getContextLines();