Text in strings gets mistaken as an import statement and is removed

[blutorange]

The plugin reeds to remove import statements before adding them again in the sorted order. Currently, this removal is done via a regexp https://github.com/IanVS/prettier-plugin-sort-imports/blob/main/src/utils/remove-nodes-from-original-code.ts#L37-L39

While that may be working for most cases, it is fundamentally broken and leads to cases such as

const exampleCode = `

import "./some-import";

`;

import "./some-import";

getting formatted as

import "./some-import";

const exampleCode = `


`;

import "./some-import";

We already know the start and end position of each node in the code. So we could collect the ranges [start, end][] of nodes to remove, sort by start position in reverse order (merging overlapping ranges), then delete each range from the code.

[blutorange]

And since the regex only checks for imports at the beginning of a line (which is not required by the spec), this code

/* foo */ import { foo } from "c";

import { bar } from "a";

function baz() {
}

also results in getting "formatted" as

/* foo */
import { bar } from "a";
import { foo } from "c";

import { foo } from "c";

function baz() {}