feat: remove unused types

src/PHPFmt.ts

@@ -6,12 +6,31 @@ import detectIndent from 'detect-indent';
 import findUp from 'find-up';
 import * as semver from 'semver';
 import phpfmt, { type PHPFmt as IPHPFmt } from 'phpfmt';
-import type { PHPFmtConfig } from './types';
 import { type Widget, PHPFmtStatus } from './Widget';
 import { Transformation } from './Transformation';
 import { PHPFmtError, PHPFmtSkipError } from './PHPFmtError';
 import { exec } from './utils';
 
+export interface PHPFmtConfig {
+  php_bin: string;
+  use_old_phpfmt: boolean;
+  detect_indent: boolean;
+  psr1: boolean;
+  psr1_naming: boolean;
+  psr2: boolean;
+  wp: boolean;
+  indent_with_space: number | boolean;
+  enable_auto_align: boolean;
+  visibility_order: boolean;
+  ignore: string[];
+  passes: string[];
+  exclude: string[];
+  smart_linebreak_after_curly: boolean;
+  yoda: boolean;
+  cakephp: boolean;
+  custom_arguments: string;
+}
+
 export class PHPFmt {
   private config: PHPFmtConfig;
   private transformation: Transformation;
@@ -64,7 +83,7 @@ export class PHPFmt {
       const spaces = this.config.indent_with_space;
       if (spaces === true) {
         this.args.push('--indent_with_space');
-      } else if (spaces > 0) {
+      } else if (typeof spaces === 'number' && spaces > 0) {
         this.args.push(`--indent_with_space=${spaces}`);
       }
     }
@@ -287,9 +306,6 @@ export class PHPFmt {
       this.widget.logError('Remove temp file failed', err);
     }
 
-    if (formatted.length > 0) {
-      return formatted;
-    }
-    return '';
+    return formatted;
   }
 }

src/PHPFmtProvider.ts

@@ -229,7 +229,7 @@ export class PHPFmtProvider {
 
             const text = await this.phpfmt.format(originalText);
             this.widget.updateStatusBarItem(PHPFmtStatus.Success);
-            if (text !== originalText) {
+            if (text && text !== originalText) {
               return [new TextEdit(range, text)];
             }
           } catch (err) {
@@ -267,7 +267,7 @@ export class PHPFmtProvider {
               text = text.replace(/^<\?php\r?\n/, '');
             }
             this.widget.updateStatusBarItem(PHPFmtStatus.Success);
-            if (text !== originalText) {
+            if (text && text !== originalText) {
               return [new TextEdit(range, text)];
             }
           } catch (err) {

src/Transformation.ts

@@ -1,9 +1,13 @@
 import os from 'os';
 import mem from 'mem';
 import phpfmt, { type PHPFmt as IPHPFmt } from 'phpfmt';
-import type { TransformationItem } from './types';
 import { exec } from './utils';
 
+export interface TransformationItem {
+  key: string;
+  description: string;
+}
+
 export class Transformation {
   public constructor(
     private readonly phpBin: string,

src/Widget.ts

@@ -7,7 +7,6 @@ import {
   ThemeColor
 } from 'vscode';
 import dayjs from 'dayjs';
-import { type LogLevel } from './types';
 
 export enum PHPFmtStatus {
   Ready = 'check-all',
@@ -18,6 +17,8 @@ export enum PHPFmtStatus {
   Disabled = 'circle-slash'
 }
 
+export type LogLevel = 'INFO' | 'WARN' | 'ERROR' | 'NONE';
+
 export class Widget {
   private logLevel: LogLevel = 'INFO';
   private readonly outputChannel: OutputChannel;