chore: run prettier after config change

Author: ignatiusmb

.eslintrc.js

@@ -1,30 +1,30 @@
 module.exports = {
-    root: true,
-    extends: '@sveltejs',
-    plugins: ['import'],
-    env: {
-        node: true
-    },
-    rules: {
-        // enabling these rules makes the linting extremely slow.
-        // (it's conceivable some subset of them could be enabled without impacting speed)
-        // see https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
-        'import/named': 'off',
-        'import/namespace': 'off',
-        'import/default': 'off',
-        'import/no-named-as-default-member': 'off',
-        'import/no-named-as-default': 'off',
-        'import/no-cycle': 'off',
-        'import/no-unused-modules': 'off',
-        'import/no-deprecated': 'off',
-        // project-specific settings
-        'max-len': ['error', { code: 100, ignoreComments: true, ignoreStrings: true }],
-        'no-trailing-spaces': 'error',
-        'one-var': ['error', 'never'],
-        '@typescript-eslint/no-unused-vars': ['error', { args: 'none' }],
-        '@typescript-eslint/no-namespace': 'warn',
-        '@typescript-eslint/no-non-null-assertion': 'warn',
-        // exclude workspace dependencies
-        'import/no-unresolved': [2, { ignore: ['svelte-language-server'] }]
-    }
+	root: true,
+	extends: '@sveltejs',
+	plugins: ['import'],
+	env: {
+		node: true
+	},
+	rules: {
+		// enabling these rules makes the linting extremely slow.
+		// (it's conceivable some subset of them could be enabled without impacting speed)
+		// see https://github.com/typescript-eslint/typescript-eslint/blob/master/docs/getting-started/linting/FAQ.md#eslint-plugin-import
+		'import/named': 'off',
+		'import/namespace': 'off',
+		'import/default': 'off',
+		'import/no-named-as-default-member': 'off',
+		'import/no-named-as-default': 'off',
+		'import/no-cycle': 'off',
+		'import/no-unused-modules': 'off',
+		'import/no-deprecated': 'off',
+		// project-specific settings
+		'max-len': ['error', { code: 100, ignoreComments: true, ignoreStrings: true }],
+		'no-trailing-spaces': 'error',
+		'one-var': ['error', 'never'],
+		'@typescript-eslint/no-unused-vars': ['error', { args: 'none' }],
+		'@typescript-eslint/no-namespace': 'warn',
+		'@typescript-eslint/no-non-null-assertion': 'warn',
+		// exclude workspace dependencies
+		'import/no-unresolved': [2, { ignore: ['svelte-language-server'] }]
+	}
 };

.prettierrc

@@ -1,9 +1,9 @@
 {
-    "arrowParens": "always",
-    "useTabs": true,
-    "printWidth": 100,
-    "tabWidth": 4,
-    "semi": true,
-    "trailingComma": "none",
-    "singleQuote": true
+	"arrowParens": "always",
+	"useTabs": true,
+	"printWidth": 100,
+	"tabWidth": 4,
+	"semi": true,
+	"trailingComma": "none",
+	"singleQuote": true
 }

packages/language-server/src/importPackage.ts

@@ -10,46 +10,46 @@ import { Logger } from './logger';
  * so it's not transformed.
  */
 function dynamicRequire(dynamicFileToRequire: string): any {
-    // prettier-ignore
-    return require(dynamicFileToRequire);
+	// prettier-ignore
+	return require(dynamicFileToRequire);
 }
 
 export function getPackageInfo(packageName: string, fromPath: string) {
-    const packageJSONPath = require.resolve(`${packageName}/package.json`, {
-        paths: [fromPath, __dirname]
-    });
-    // eslint-disable-next-line @typescript-eslint/no-var-requires
-    const { version } = dynamicRequire(packageJSONPath);
-    const [major, minor, patch] = version.split('.');
+	const packageJSONPath = require.resolve(`${packageName}/package.json`, {
+		paths: [fromPath, __dirname]
+	});
+	// eslint-disable-next-line @typescript-eslint/no-var-requires
+	const { version } = dynamicRequire(packageJSONPath);
+	const [major, minor, patch] = version.split('.');
 
-    return {
-        path: dirname(packageJSONPath),
-        version: {
-            full: version,
-            major,
-            minor,
-            patch
-        }
-    };
+	return {
+		path: dirname(packageJSONPath),
+		version: {
+			full: version,
+			major,
+			minor,
+			patch
+		}
+	};
 }
 
 export function importPrettier(fromPath: string): typeof prettier {
-    const pkg = getPackageInfo('prettier', fromPath);
-    const main = resolve(pkg.path);
-    Logger.log('Using Prettier v' + pkg.version.full, 'from', main);
-    return dynamicRequire(main);
+	const pkg = getPackageInfo('prettier', fromPath);
+	const main = resolve(pkg.path);
+	Logger.log('Using Prettier v' + pkg.version.full, 'from', main);
+	return dynamicRequire(main);
 }
 
 export function importSvelte(fromPath: string): typeof svelte {
-    const pkg = getPackageInfo('svelte', fromPath);
-    const main = resolve(pkg.path, 'compiler');
-    Logger.log('Using Svelte v' + pkg.version.full, 'from', main);
-    return dynamicRequire(main);
+	const pkg = getPackageInfo('svelte', fromPath);
+	const main = resolve(pkg.path, 'compiler');
+	Logger.log('Using Svelte v' + pkg.version.full, 'from', main);
+	return dynamicRequire(main);
 }
 
 export function importSveltePreprocess(fromPath: string): typeof sveltePreprocess {
-    const pkg = getPackageInfo('svelte-preprocess', fromPath);
-    const main = resolve(pkg.path);
-    Logger.log('Using svelte-preprocess v' + pkg.version.full, 'from', main);
-    return dynamicRequire(main);
+	const pkg = getPackageInfo('svelte-preprocess', fromPath);
+	const main = resolve(pkg.path);
+	Logger.log('Using svelte-preprocess v' + pkg.version.full, 'from', main);
+	return dynamicRequire(main);
 }

packages/language-server/src/lib/DiagnosticsManager.ts

@@ -5,30 +5,30 @@ export type SendDiagnostics = _Connection['sendDiagnostics'];
 export type GetDiagnostics = (doc: TextDocumentIdentifier) => Thenable<Diagnostic[]>;
 
 export class DiagnosticsManager {
-    constructor(
-        private sendDiagnostics: SendDiagnostics,
-        private docManager: DocumentManager,
-        private getDiagnostics: GetDiagnostics
-    ) {}
+	constructor(
+		private sendDiagnostics: SendDiagnostics,
+		private docManager: DocumentManager,
+		private getDiagnostics: GetDiagnostics
+	) {}
 
-    updateAll() {
-        this.docManager.getAllOpenedByClient().forEach((doc) => {
-            this.update(doc[1]);
-        });
-    }
+	updateAll() {
+		this.docManager.getAllOpenedByClient().forEach((doc) => {
+			this.update(doc[1]);
+		});
+	}
 
-    async update(document: Document) {
-        const diagnostics = await this.getDiagnostics({ uri: document.getURL() });
-        this.sendDiagnostics({
-            uri: document.getURL(),
-            diagnostics
-        });
-    }
+	async update(document: Document) {
+		const diagnostics = await this.getDiagnostics({ uri: document.getURL() });
+		this.sendDiagnostics({
+			uri: document.getURL(),
+			diagnostics
+		});
+	}
 
-    removeDiagnostics(document: Document) {
-        this.sendDiagnostics({
-            uri: document.getURL(),
-            diagnostics: []
-        });
-    }
+	removeDiagnostics(document: Document) {
+		this.sendDiagnostics({
+			uri: document.getURL(),
+			diagnostics: []
+		});
+	}
 }

packages/language-server/src/lib/documents/Document.ts

@@ -9,76 +9,76 @@ import { HTMLDocument } from 'vscode-html-languageservice';
  * Represents a text document contains a svelte component.
  */
 export class Document extends WritableDocument {
-    languageId = 'svelte';
-    scriptInfo: TagInformation | null = null;
-    moduleScriptInfo: TagInformation | null = null;
-    styleInfo: TagInformation | null = null;
-    config!: SvelteConfig;
-    html!: HTMLDocument;
+	languageId = 'svelte';
+	scriptInfo: TagInformation | null = null;
+	moduleScriptInfo: TagInformation | null = null;
+	styleInfo: TagInformation | null = null;
+	config!: SvelteConfig;
+	html!: HTMLDocument;
 
-    constructor(public url: string, public content: string) {
-        super();
-        this.updateDocInfo();
-    }
+	constructor(public url: string, public content: string) {
+		super();
+		this.updateDocInfo();
+	}
 
-    private updateDocInfo() {
-        if (!this.config || this.config.loadConfigError) {
-            this.config = loadConfig(this.getFilePath() || '');
-        }
-        this.html = parseHtml(this.content);
-        const scriptTags = extractScriptTags(this.content, this.html);
-        this.scriptInfo = this.addDefaultLanguage(scriptTags?.script || null, 'script');
-        this.moduleScriptInfo = this.addDefaultLanguage(scriptTags?.moduleScript || null, 'script');
-        this.styleInfo = this.addDefaultLanguage(extractStyleTag(this.content, this.html), 'style');
-    }
+	private updateDocInfo() {
+		if (!this.config || this.config.loadConfigError) {
+			this.config = loadConfig(this.getFilePath() || '');
+		}
+		this.html = parseHtml(this.content);
+		const scriptTags = extractScriptTags(this.content, this.html);
+		this.scriptInfo = this.addDefaultLanguage(scriptTags?.script || null, 'script');
+		this.moduleScriptInfo = this.addDefaultLanguage(scriptTags?.moduleScript || null, 'script');
+		this.styleInfo = this.addDefaultLanguage(extractStyleTag(this.content, this.html), 'style');
+	}
 
-    /**
-     * Get text content
-     */
-    getText(): string {
-        return this.content;
-    }
+	/**
+	 * Get text content
+	 */
+	getText(): string {
+		return this.content;
+	}
 
-    /**
-     * Set text content and increase the document version
-     */
-    setText(text: string) {
-        this.content = text;
-        this.version++;
-        this.updateDocInfo();
-    }
+	/**
+	 * Set text content and increase the document version
+	 */
+	setText(text: string) {
+		this.content = text;
+		this.version++;
+		this.updateDocInfo();
+	}
 
-    /**
-     * Returns the file path if the url scheme is file
-     */
-    getFilePath(): string | null {
-        return urlToPath(this.url);
-    }
+	/**
+	 * Returns the file path if the url scheme is file
+	 */
+	getFilePath(): string | null {
+		return urlToPath(this.url);
+	}
 
-    /**
-     * Get URL file path.
-     */
-    getURL() {
-        return this.url;
-    }
+	/**
+	 * Get URL file path.
+	 */
+	getURL() {
+		return this.url;
+	}
 
-    private addDefaultLanguage(
-        tagInfo: TagInformation | null,
-        tag: 'style' | 'script'
-    ): TagInformation | null {
-        if (!tagInfo) {
-            return null;
-        }
+	private addDefaultLanguage(
+		tagInfo: TagInformation | null,
+		tag: 'style' | 'script'
+	): TagInformation | null {
+		if (!tagInfo) {
+			return null;
+		}
 
-        const defaultLang = Array.isArray(this.config.preprocess)
-            ? this.config.preprocess.find((group) => group.defaultLanguages?.[tag])
-                  ?.defaultLanguages?.[tag]
-            : this.config.preprocess?.defaultLanguages?.[tag];
+		const defaultLang = Array.isArray(this.config.preprocess)
+			? this.config.preprocess.find((group) => group.defaultLanguages?.[tag])
+					?.defaultLanguages?.[tag]
+			: this.config.preprocess?.defaultLanguages?.[tag];
 
-        if (!tagInfo.attributes.lang && !tagInfo.attributes.type && defaultLang) {
-            tagInfo.attributes.lang = defaultLang;
-        }
+		if (!tagInfo.attributes.lang && !tagInfo.attributes.type && defaultLang) {
+			tagInfo.attributes.lang = defaultLang;
+		}
 
-        return tagInfo;
-    }
+		return tagInfo;
+	}
 }