fix: use single quotes in new script files (#96)

* fix: provide defaults while serializing new script files

* remove unused function

* update tests

* improve

* use tabs

* update snapshots

* Create five-shoes-invent.md

* fix changeset

---------

Co-authored-by: Ben McCann <[email protected]>
Co-authored-by: AdrianGonz97 <[email protected]>

Author: 3 people

.changeset/five-shoes-invent.md

@@ -0,0 +1,5 @@
+---
+'sv': patch
+---
+
+fix: improve formatting on new script files

packages/ast-tooling/index.ts

@@ -1,8 +1,8 @@
 import { parse as tsParse } from 'recast/parsers/typescript.js';
-import { parse as recastParse, print as recastPrint } from 'recast';
-import { Document, Element, Text, type ChildNode } from 'domhandler';
+import { parse as recastParse, print as recastPrint, type Options as RecastOptions } from 'recast';
+import { Document, Element, type ChildNode } from 'domhandler';
 import { ElementType, parseDocument } from 'htmlparser2';
-import { appendChild, prependChild, removeElement, textContent } from 'domutils';
+import { removeElement, textContent } from 'domutils';
 import serializeDom from 'dom-serializer';
 import {
 	Root as CssAst,
@@ -65,8 +65,17 @@ export function parseScript(content: string): AstTypes.Program {
 	return recastOutput.program;
 }
 
-export function serializeScript(ast: AstTypes.ASTNode): string {
-	return recastPrint(ast).code;
+export function serializeScript(ast: AstTypes.ASTNode, previousContent?: string): string {
+	let options: RecastOptions | undefined;
+	if (!previousContent) {
+		// provide sensible defaults if we generate a new file
+		options = {
+			quote: 'single',
+			useTabs: true
+		};
+	}
+
+	return recastPrint(ast, options).code;
 }
 
 export function parseCss(content: string): CssAst {
@@ -143,36 +152,6 @@ export function parseSvelte(content: string): SvelteAst {
 	return { jsAst, htmlAst, cssAst };
 }
 
-export function serializeSvelte(asts: SvelteAst): string {
-	const { jsAst, htmlAst, cssAst } = asts;
-
-	const css = serializeCss(cssAst);
-	const newScriptValue = serializeScript(jsAst);
-
-	if (newScriptValue.length > 0) {
-		const scriptTag = new Element('script', {}, undefined, ElementType.ElementType.Script);
-		for (const child of scriptTag.children) {
-			removeElement(child);
-		}
-
-		appendChild(scriptTag, new Text(newScriptValue));
-		prependChild(htmlAst, scriptTag);
-	}
-
-	if (css.length > 0) {
-		const styleTag = new Element('style', {}, undefined, ElementType.ElementType.Style);
-		for (const child of styleTag.children) {
-			removeElement(child);
-		}
-
-		appendChild(styleTag, new Text(css));
-		appendChild(htmlAst, styleTag);
-	}
-
-	const content = serializeHtml(htmlAst);
-	return content;
-}
-
 export function parseJson(content: string): any {
 	// some of the files we need to process contain comments. The default
 	// node JSON.parse fails parsing those comments.

packages/core/tests/js/arrays/object-array/output.ts

@@ -1,5 +1,5 @@
 const array = [{
   test: true
 }, {
-  test2: "string"
-}];
+  test2: 'string'
+}];

packages/core/tests/js/arrays/string-array/output.ts

@@ -1 +1 @@
-const array = ["test", "test2"];
+const array = ['test', 'test2'];

packages/core/tests/js/exports/default-export-with-variable/output.ts

@@ -1,5 +1,5 @@
 const object = {
-  test: "string"
+  test: 'string'
 };
 
 export default object;

packages/core/tests/js/exports/default-export/output.ts

@@ -1,3 +1,3 @@
 export default {
-  test: "string"
+  test: 'string'
 };

packages/core/tests/js/exports/named-export/output.ts

@@ -1,7 +1,7 @@
 export const variable = {
-  test: "string"
+  test: 'string'
 };
 
 export const variable2 = {
-  test2: "string2"
+  test2: 'string2'
 };

packages/core/tests/js/imports/default-import/output.ts

@@ -1 +1 @@
-import MyPackage from "package";
+import MyPackage from 'package';

packages/core/tests/js/imports/empty-import/output.ts

@@ -1,2 +1,2 @@
-import "package/file.css";
-import "./relativ/file.css";
+import 'package/file.css';
+import './relativ/file.css';

packages/core/tests/js/imports/named-import/output.ts

@@ -1,2 +1,2 @@
-import { Handle } from "@sveltejs/kit";
-import { namedOne } from "package";
+import { Handle } from '@sveltejs/kit';
+import { namedOne } from 'package';