refactor(organize_imports): support bun:, #, absolute imports

crates/biome_js_analyze/src/assists/correctness/organize_imports.rs

@@ -581,6 +581,8 @@ impl PartialEq for ImportKey {
 /// are listed before imports closer to the source file.
 #[derive(Eq, Ord, PartialEq, PartialOrd)]
 enum ImportCategory {
+    /// Anything with an explicit `bun:` prefix.
+    Bun,
     /// Anything with an explicit `node:` prefix, or one of the recognized
     /// Node built-ins, such `"fs"`, `"child_process"`, etc..
     NodeBuiltin,
@@ -594,6 +596,12 @@ enum ImportCategory {
     /// may (incorrectly) include source imports through custom import mappings
     /// as well.
     Library,
+    /// Absolute file imports `/<path>`.
+    Absolute,
+    /// Node allows specifying an import map with name prefixed with `#`.
+    /// See https://nodejs.org/api/packages.html#subpath-imports
+    SharpImport,
+    /// Relative file imports `./<path>`.
     Relative,
     /// Any unrecognized protocols are grouped here. These may include custom
     /// protocols such as supported by bundlers.
@@ -638,10 +646,13 @@ const NODE_BUILTINS: &[&str] = &[
 impl From<&str> for ImportCategory {
     fn from(value: &str) -> Self {
         match value.chars().next().unwrap_or_default() {
+            '#' => Self::SharpImport,
+            '/' => Self::Absolute,
             '.' => Self::Relative,
             _ => {
                 if let Some((protocol, _)) = value.split_once(':') {
                     match protocol {
+                        "bun" => Self::Bun,
                         "http" | "https" => Self::Url,
                         "node" => Self::NodeBuiltin,
                         "npm" => Self::Npm,

crates/biome_js_analyze/tests/specs/correctness/organizeImports/groups.js

@@ -6,3 +6,6 @@ import assert from "node:assert";
 import aunt from "../aunt";
 import { VERSION } from "https://deno.land/std/version.ts";
 import { mock, test } from "node:test";
+import { expect } from "bun:test";
+import { internal } from "#internal";
+import { secret } from "/absolute/path";

crates/biome_js_analyze/tests/specs/correctness/organizeImports/groups.js.snap

@@ -12,25 +12,35 @@ import assert from "node:assert";
 import aunt from "../aunt";
 import { VERSION } from "https://deno.land/std/version.ts";
 import { mock, test } from "node:test";
-
+import { expect } from "bun:test";
+import { internal } from "#internal";
+import { secret } from "/absolute/path";
 ```
 
 # Actions
 ```diff
-@@ -1,8 +1,8 @@
+@@ -1,11 +1,11 @@
+-import uncle from "../uncle";
+-import sibling from "./sibling";
++import { expect } from "bun:test";
 +import assert from "node:assert";
 +import { mock, test } from "node:test";
-+import express from "npm:express";
-+import { VERSION } from "https://deno.land/std/version.ts";
-+import aunt from "../aunt";
- import uncle from "../uncle";
- import sibling from "./sibling";
--import express from "npm:express";
- import imageUrl from "url:./image.png";
+ import express from "npm:express";
+-import imageUrl from "url:./image.png";
 -import assert from "node:assert";
 -import aunt from "../aunt";
--import { VERSION } from "https://deno.land/std/version.ts";
+ import { VERSION } from "https://deno.land/std/version.ts";
 -import { mock, test } from "node:test";
+-import { expect } from "bun:test";
++import { secret } from "/absolute/path";
+ import { internal } from "#internal";
+-import { secret } from "/absolute/path";
+\ No newline at end of file
++import aunt from "../aunt";
++import uncle from "../uncle";
++import sibling from "./sibling";
++import imageUrl from "url:./image.png";
+\ No newline at end of file
 
 ```
 

website/src/content/docs/analyzer/index.mdx

@@ -23,12 +23,15 @@ This feature is enabled by default but can be opted-in/out via configuration:
 
 Import statements are sorted by "distance". Modules that are "farther" from the user are put on the top, modules "closer" to the user are put on the bottom:
 
-1. built-in Node.js modules that are explicitly imported using the `node:` protocol;
-2. modules imported via `npm:` protocol. This is a valid syntax when writing code run by Deno, for example;
-3. modules imported via URL;
-4. modules imported from libraries;
-5. modules imported via relative imports;
-6. modules that couldn't be identified by the previous criteria;
+1. modules imported via `bun:` protocol. This is a valid syntax when writing code run by Bun;
+2. built-in Node.js modules that are explicitly imported using the `node:` protocol;
+3. modules imported via `npm:` protocol. This is a valid syntax when writing code run by Deno, for example;
+4. modules imported via URL;
+5. modules imported from libraries;
+6. modules imported via absolute imports;
+7. modules imported from a name prefixed by `#`. This is a valid syntax when [writing code run by Node](https://nodejs.org/api/packages.html#subpath-imports);
+8. modules imported via relative imports;
+9. modules that couldn't be identified by the previous criteria;
 
 For example, given the following code:
 
@@ -41,15 +44,23 @@ import assert from "node:assert";
 import aunt from "../aunt";
 import { VERSION } from "https://deno.land/std/version.ts";
 import { mock, test } from "node:test";
+import { expect } from "bun:test";
+import { internal } from "#internal";
+import { secret } from "/absolute/path";
+import React from "react";
 ```
 
 They will be sorted like this:
 
 ```ts
+ import { expect } from "bun:test";
  import assert from "node:assert";
  import { mock, test } from "node:test";
  import express from "npm:express";
  import { VERSION } from "https://deno.land/std/version.ts";
+ import React from "react";
+ import { secret } from "/absolute/path";
+ import { internal } from "#internal";
  import aunt from "../aunt";
  import uncle from "../uncle";
  import sibling from "./sibling";