Label logic driver source + kind in hovers

[evanwporter]

Closes #351

Copied from MikePopoloski/slang#1861

Within SystemVerilog the logic keyword is somewhat unique in that its meaning isn't wholly contained within the declaration ("meaning" isn't quite the right word here but hopefully you get my point :) ).

Consider (FYI: I used chatgpt to generate the following examples):

1. logic assigned by a continuous assignment

module ContinuousLogic (
    input logic a,
    output logic q
);
    assign q = a;
endmodule

2. logic assigned as combinational logic

module CombinationalLogic (
    input logic a,
    input logic b,
    output logic q
);
    always_comb begin
        q = a & b;
    end
endmodule

3. logic assigned as a latch

module LatchLogic (
    input logic en,
    input logic d,
    output logic q
);
    always_latch begin
        if (en)
            q = d;
    end
endmodule

module InferredLatchLogic (
    input logic en,
    input logic d,
    output logic q
);
    always_comb begin
        if (en)
            q = d;
        // q is not assigned when en is false
    end
endmodule

4. logic assigned as a register / flip-flop

module RegisterLogic (
    input logic clk,
    input logic d,
    output logic q
);
    always_ff @(posedge clk) begin
        q <= d;
    end
endmodule

In all 4 examples q is a variable, however the exact way that q is set changes in each example.

To this end this PR attemps to add an additional piece of info to the hovers that distinguishes between each of the different ways q can be set--ie: as a latch, net or register.

Images:

[toddstrader]

This makes more sense to me re: terminology. I'll leave it to @AndrewNolte to opine on the implementation.

[AndrewNolte]

Sorry did a big refactor here for hover logic- #339

[evanwporter]

Nice fix. I like how it’s split up. It looks cleaner now. Shouldn’t be too hard too resolve conflicts though it’s tough to say since I’m on mobile

…

On Mon, Jun 15, 2026 at 12:59 PM Andrew Nolte ***@***.***> wrote: *AndrewNolte* left a comment (hudson-trading/slang-server#388) <#388 (comment)> Sorry did a big refactor here for hover logic- #339 <#339> — Reply to this email directly, view it on GitHub <#388?email_source=notifications&email_token=A3QHV6PVQQPAW7RDMLUYVQL5ABISVA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINZRGE4TCNBXGE2KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KYZTPN52GK4S7MNWGSY3L#issuecomment-4711914714>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/A3QHV6OKB3IPKUTXY4UF7OT5ABISVAVCNFSNUABGKJSXA33TNF2G64TZHMYTANBSGE4DKMBYGQ5US43TOVSTWNBWGMZTMNJWGA4DLILWAI> . Triage notifications, keep track of coding agent tasks and review pull requests on the go with GitHub Mobile for iOS <https://github.com/notifications/mobile/ios/A3QHV6PO4VWKT6XSLPMXAET5ABISVA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINZRGE4TCNBXGE2KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2KUZTPN52GK4S7NFXXG> and Android <https://github.com/notifications/mobile/android/A3QHV6I6MDDEWTBTHS4C4NT5ABISVA5CNFSNUABFM5UWIORPF5TWS5BNNB2WEL2JONZXKZKDN5WW2ZLOOQXTINZRGE4TCNBXGE2KM4TFMFZW63VGMF2XI2DPOKSWK5TFNZ2K4ZTPN52GK4S7MFXGI4TPNFSA>. Download it today! You are receiving this because you authored the thread.Message ID: ***@***.***>

[evanwporter]

it works but the way that the extraDisplayNode and the ShallowAnalysis is passed around could be improved