From 9147f0a1f5dc7bbee943e1dd9997b85107821c15 Mon Sep 17 00:00:00 2001
From: xdBronch <51252236+xdBronch@users.noreply.github.com>
Date: Thu, 24 Oct 2024 20:16:45 -0400
Subject: [PATCH] wrap number hover in code block on zed

---
 src/Server.zig         | 10 +++++++++-
 src/features/hover.zig | 15 +++++++++++----
 2 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/src/Server.zig b/src/Server.zig
index 2f55701f5..dd790a614 100644
--- a/src/Server.zig
+++ b/src/Server.zig
@@ -1558,7 +1558,15 @@ fn hoverHandler(server: *Server, arena: std.mem.Allocator, request: types.HoverP
     var analyser = server.initAnalyser(handle);
     defer analyser.deinit();
 
-    return hover_handler.hover(&analyser, arena, handle, source_index, markup_kind, server.offset_encoding);
+    return hover_handler.hover(
+        &analyser,
+        arena,
+        handle,
+        source_index,
+        markup_kind,
+        server.offset_encoding,
+        server.client_capabilities.client_name,
+    );
 }
 
 fn documentSymbolsHandler(server: *Server, arena: std.mem.Allocator, request: types.DocumentSymbolParams) Error!lsp.ResultType("textDocument/documentSymbol") {
diff --git a/src/features/hover.zig b/src/features/hover.zig
index 79ee84876..8e6f1ae49 100644
--- a/src/features/hover.zig
+++ b/src/features/hover.zig
@@ -370,6 +370,7 @@ fn hoverNumberLiteral(
     token_index: Ast.TokenIndex,
     arena: std.mem.Allocator,
     markup_kind: types.MarkupKind,
+    client_name: ?[]const u8,
 ) error{OutOfMemory}!?[]const u8 {
     const tree = handle.tree;
     // number literals get tokenized separately from their minus sign
@@ -388,15 +389,19 @@ fn hoverNumberLiteral(
         }
     };
 
+    // Zed currently doesn't render markdown unless wrapped in code blocks
+    // Remove this when this issue is closed https://github.com/zed-industries/zed/issues/5386
+    const is_zed = if (client_name) |name| std.mem.startsWith(u8, name, "Zed") else false;
     switch (markup_kind) {
         .markdown => return try std.fmt.allocPrint(arena,
-            \\| Base | {[value]s:<[count]} |
+            \\{[md_ticks]s}| Base | {[value]s:<[count]} |
             \\| ---- | {[dash]s:-<[count]} |
             \\| BIN  | {[sign]s}0b{[number]b:<[len]} |
             \\| OCT  | {[sign]s}0o{[number]o:<[len]} |
             \\| DEC  | {[sign]s}{[number]d:<[len]}   |
-            \\| HEX  | {[sign]s}0x{[number]X:<[len]} |
+            \\| HEX  | {[sign]s}0x{[number]X:<[len]} |{[md_ticks]s}
         , .{
+            .md_ticks = if (is_zed) "\n```" else "",
             .sign = if (is_negative) "-" else "",
             .dash = "-",
             .value = "Value",
@@ -422,6 +427,7 @@ fn hoverDefinitionNumberLiteral(
     source_index: usize,
     markup_kind: types.MarkupKind,
     offset_encoding: offsets.Encoding,
+    client_name: ?[]const u8,
 ) !?types.Hover {
     const tracy_zone = tracy.trace(@src());
     defer tracy_zone.end();
@@ -429,7 +435,7 @@ fn hoverDefinitionNumberLiteral(
     const tree = handle.tree;
     const token_index = offsets.sourceIndexToTokenIndex(tree, source_index);
     const num_loc = offsets.tokenToLoc(tree, token_index);
-    const hover_text = (try hoverNumberLiteral(handle, token_index, arena, markup_kind)) orelse return null;
+    const hover_text = (try hoverNumberLiteral(handle, token_index, arena, markup_kind, client_name)) orelse return null;
 
     return .{
         .contents = .{ .MarkupContent = .{
@@ -447,6 +453,7 @@ pub fn hover(
     source_index: usize,
     markup_kind: types.MarkupKind,
     offset_encoding: offsets.Encoding,
+    client_name: ?[]const u8,
 ) !?types.Hover {
     const pos_context = try Analyser.getPositionContext(arena, handle.tree, source_index, true);
 
@@ -456,7 +463,7 @@ pub fn hover(
         .field_access => |loc| try hoverDefinitionFieldAccess(analyser, arena, handle, source_index, loc, markup_kind, offset_encoding),
         .label_access, .label_decl => |loc| try hoverDefinitionLabel(analyser, arena, handle, source_index, loc, markup_kind, offset_encoding),
         .enum_literal => try hoverDefinitionEnumLiteral(analyser, arena, handle, source_index, markup_kind, offset_encoding),
-        .number_literal, .char_literal => try hoverDefinitionNumberLiteral(arena, handle, source_index, markup_kind, offset_encoding),
+        .number_literal, .char_literal => try hoverDefinitionNumberLiteral(arena, handle, source_index, markup_kind, offset_encoding, client_name),
         else => null,
     };