fix: skip when formatting ranges (#112)

Co-authored-by: simonhaenisch <simonhaenisch@users.noreply.github.com>
simonhaenisch · Nov 9, 2023 · 5ebe7d1 · 5ebe7d1
1 parent 9a10490
commit 5ebe7d1
Show file tree

Hide file tree

Showing 3 changed files with 20 additions and 0 deletions.
diff --git a/.gitignore b/.gitignore
@@ -1 +1,2 @@
 node_modules
+.idea
diff --git a/index.js b/index.js
@@ -19,6 +19,15 @@ const organizeImports = (code, options) => {
 		return code;
 	}
 
+	const isRange =
+		Boolean(options.originalText) ||
+		options.rangeStart !== 0 ||
+		(options.rangeEnd !== Infinity && options.rangeEnd !== code.length);
+
+	if (isRange) {
+		return code; // processing a range doesn't make sense
+	}
+
 	try {
 		return organize(code, options);
 	} catch (error) {

diff --git a/test/main.js b/test/main.js
@@ -70,6 +70,16 @@ test(
 	{ transformer: (res) => res.split('\n')[1] },
 );
 
+test('skips when formatting a range', async (t) => {
+	const code = 'import { foo } from "./bar";';
+
+	const formattedCode1 = await prettify(code, { rangeEnd: 10 });
+	const formattedCode2 = await prettify(code, { rangeStart: 10 });
+
+	t.is(formattedCode1, code);
+	t.is(formattedCode2, code);
+});
+
 test('does not remove unused imports with `organizeImportsSkipDestructiveCodeActions` enabled', async (t) => {
 	const code = `import { foo } from "./bar";
 `;