refactor: Extracted replaceContents()

Author: ijlee2

src/parse/preprocess.ts

@@ -2,6 +2,7 @@ import {
   getBuffer,
   parse,
   type Range,
+  replaceContents,
   sliceByteRange,
 } from '../utils/content-tag.js';
 
@@ -17,15 +18,6 @@ export interface Template {
 
 const PLACEHOLDER = '~';
 
-function replaceRange(
-  s: string,
-  start: number,
-  end: number,
-  substitute: string,
-): string {
-  return sliceByteRange(s, 0, start) + substitute + sliceByteRange(s, end);
-}
-
 /**
  * Replace the template with a parsable placeholder that takes up the same
  * range.
@@ -57,14 +49,16 @@ export function preprocessTemplateRange(
 
   // We need to replace forward slash with _something else_, because
   // forward slash breaks the parsed templates.
-  const content = template.contents.replaceAll('/', PLACEHOLDER);
+  const contents = template.contents.replaceAll('/', PLACEHOLDER);
 
   const tplLength = template.range.end - template.range.start;
   const spaces =
-    tplLength - getBuffer(content).length - prefix.length - suffix.length;
-  const total = prefix + content + ' '.repeat(spaces) + suffix;
+    tplLength - getBuffer(contents).length - prefix.length - suffix.length;
 
-  return replaceRange(code, template.range.start, template.range.end, total);
+  return replaceContents(code, {
+    contents: [prefix, contents, ' '.repeat(spaces), suffix].join(''),
+    range: template.range,
+  });
 }
 
 /** Pre-processes the template info, parsing the template content to Glimmer AST. */

src/utils/content-tag.ts

@@ -32,6 +32,22 @@ export function parse(
   return preprocessor.parse(file, options);
 }
 
+export function replaceContents(
+  file: string,
+  options: {
+    contents: string;
+    range: Range;
+  },
+): string {
+  const { contents, range } = options;
+
+  return [
+    sliceByteRange(file, 0, range.start),
+    contents,
+    sliceByteRange(file, range.end),
+  ].join('');
+}
+
 export function sliceByteRange(
   string_: string,
   indexStart: number,