docs: Markdown Javadoc links in remaining files (Sonar S7474)

Convert the remaining {@link X} to [X] and {@code x} to `x` across the
rest of the codebase (older code outside the prior new-code sweep).
Javadoc resolves every reference (build enforces failOnError + failOnWarnings).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>

Author: dfa1

cli/src/main/java/io/github/dfa1/vortex/cli/CliHandles.java

@@ -16,7 +16,7 @@
 /// Shared handle plumbing for the interactive subcommands (`view`, `tui`).
 ///
 /// A [VortexReader] uses a confined [java.lang.foreign.Arena], so the file must be
-/// opened and closed on the same {@link IoWorker} thread the TUI later dispatches I/O to. These
+/// opened and closed on the same [IoWorker] thread the TUI later dispatches I/O to. These
 /// helpers centralise that worker round-trip plus the local-file / http(s) target resolution.
 @SuppressWarnings("java:S106") // CLI tool: user-facing diagnostics go to System.err by design.
 final class CliHandles {

cli/src/main/java/io/github/dfa1/vortex/cli/tui/IoWorker.java

@@ -7,7 +7,7 @@
 
 /// Single-threaded I/O executor that owns one [io.github.dfa1.vortex.reader.VortexHandle].
 ///
-/// Vortex readers use a confined {@link java.lang.foreign.Arena}, so every
+/// Vortex readers use a confined [java.lang.foreign.Arena], so every
 /// `slice()` / `scan()` call must happen on the same thread that
 /// opened the file. The TUI dispatches all such calls to this worker so the
 /// render loop on the main thread never crosses the arena's owning thread.

cli/src/main/java/io/github/dfa1/vortex/cli/tui/VortexInspectorTui.java

@@ -62,7 +62,7 @@ public static void show(VortexHandle handle, InspectorTree.Progress progress) th
 
     /// Variant that dispatches every `handle` I/O call onto the supplied
     /// [IoWorker]. Required when the handle was opened on a different
-    /// thread (Vortex readers use a confined {@link java.lang.foreign.Arena},
+    /// thread (Vortex readers use a confined [java.lang.foreign.Arena],
     /// so cross-thread access throws `WrongThreadException`).
     ///
     /// Passing `null` for `worker` falls back to synchronous I/O

cli/src/test/java/io/github/dfa1/vortex/cli/tui/term/PosixTerminalSmokeTest.java

@@ -17,7 +17,7 @@
 /// into the alternate screen / raw mode. Class load alone is the safe, meaningful
 /// assertion: it forces every `Linker.downcallHandle` in the static initializer
 /// (tcgetattr, tcsetattr, cfmakeraw, ioctl) to resolve its libc symbol, throwing
-/// {@link UnsatisfiedLinkError} during `<clinit>` if an entry point is missing.
+/// [UnsatisfiedLinkError] during `<clinit>` if an entry point is missing.
 class PosixTerminalSmokeTest {
 
     @Test

core/src/main/java/io/github/dfa1/vortex/core/DType.java

@@ -28,7 +28,7 @@ public sealed interface DType
     /// [#withNullable(boolean)] so call sites read as a fluent adjective:
     /// `DType.i64().asNullable()`.
     ///
-    /// @return a new {@link DType} identical to this one but with `nullable = true`
+    /// @return a new [DType] identical to this one but with `nullable = true`
     default DType asNullable() {
         return withNullable(true);
     }
@@ -158,13 +158,13 @@ static Decimal decimal(int precision, int scale) {
     ///     .build();
     /// ```
     ///
-    /// @return a new {@link StructBuilder}
+    /// @return a new [StructBuilder]
     static StructBuilder structBuilder() {
         return new StructBuilder();
     }
 
     /// Fluent builder for [Struct] dtypes. Use [#structBuilder()] to obtain one.
-    /// Preserves insertion order and rejects duplicate field names at {@link #build()}.
+    /// Preserves insertion order and rejects duplicate field names at [#build()].
     final class StructBuilder {
         private final LinkedHashMap<String, DType> fields = new LinkedHashMap<>();
         private boolean nullable;
@@ -195,7 +195,7 @@ public StructBuilder asNullable() {
 
         /// Builds the [Struct] dtype.
         ///
-        /// @return a new {@link Struct} reflecting every field added so far
+        /// @return a new [Struct] reflecting every field added so far
         public Struct build() {
             return new Struct(
                     java.util.List.copyOf(fields.keySet()),

core/src/main/java/io/github/dfa1/vortex/core/PType.java

@@ -60,7 +60,7 @@ public boolean isSigned() {
     /// uses to identify a physical type.
     ///
     /// Unlike `PType.values()[ordinal]`, this method validates the ordinal against the
-    /// declared range and throws {@link VortexException} for crafted out-of-range values rather
+    /// declared range and throws [VortexException] for crafted out-of-range values rather
     /// than the JDK's {@link ArrayIndexOutOfBoundsException}. Use this at every decode site that
     /// reads a ptype from untrusted metadata.
     ///

core/src/main/java/io/github/dfa1/vortex/proto/ProtoReader.java

@@ -6,7 +6,7 @@
 import java.nio.charset.StandardCharsets;
 
 /// Stateful cursor over a [MemorySegment] that decodes proto3 wire-format primitives.
-/// Generated record types call into this from their {@code decode(...)} static factories.
+/// Generated record types call into this from their `decode(...)` static factories.
 /// Package-private — generated code lives in the same package.
 final class ProtoReader {
 
@@ -108,7 +108,7 @@ String readString() throws IOException {
     }
 
     /// Reads a length-delimited byte sequence into a fresh `byte[]`.
-    /// Use {@link #readLenDelimSegment()} for zero-copy access when the consumer can hold a segment slice.
+    /// Use [#readLenDelimSegment()] for zero-copy access when the consumer can hold a segment slice.
     byte[] readBytes() throws IOException {
         int len = readVarint32();
         ensureBytes(len);

core/src/main/java/io/github/dfa1/vortex/proto/ProtoWriter.java

@@ -93,7 +93,7 @@ void writeEmbedded(byte[] encoded) {
 
     /// Reserves space for a length-delimited region whose payload size is unknown until written.
     /// Returns a mark to pass back to [#endLenDelim(int)]; the caller writes the payload
-    /// in between. Avoids the alloc/copy round-trip of writing into a temporary {@code ProtoWriter}.
+    /// in between. Avoids the alloc/copy round-trip of writing into a temporary `ProtoWriter`.
     ///
     /// Reserves the worst-case 5 bytes for a varint32 length; {@link #endLenDelim} backpatches
     /// the actual length and shifts the payload left if a shorter varint suffices.

inspector/src/main/java/io/github/dfa1/vortex/inspect/InspectorTree.java

@@ -87,7 +87,7 @@ public static InspectorTree build(VortexHandle handle) {
     /// resulting tree contains only structure derived from the file's footer
     /// and layout, so the call is essentially free on remote handles.
     ///
-    /// Use with {@link #peek(Node, VortexHandle)} for lazy on-demand resolution.
+    /// Use with [#peek(Node, VortexHandle)] for lazy on-demand resolution.
     ///
     /// @param handle open file handle
     /// @return immutable shallow inspector tree

inspector/src/main/java/io/github/dfa1/vortex/inspect/ZonedStatsSchema.java

@@ -16,7 +16,7 @@
 ///       vortex-layout/src/layouts/zoned/mod.rs</a> — `ZonedMetadata`):
 ///       bytes [0..4) are the zone length as a little-endian `u32`;
 ///       remaining bytes form a `Stat` bitset (LSB-first per byte). Each
-///       set bit at index `i` indicates that the {@link Stat} with that
+///       set bit at index `i` indicates that the [Stat] with that
 ///       ordinal is present in the auxiliary stats table.
 /// - Schema construction
 ///       (<a href="https://github.com/spiraldb/vortex/blob/develop/vortex-layout/src/layouts/zoned/schema.rs">