Merge pull request #144 from ChetanXpro/feat/add-workflow-for-code-formatting

feat: add workflow for code formatting

Author: ChetanXpro

.github/workflows/ci.yml

@@ -6,18 +6,35 @@ on:
   pull_request:
     branches: [main]
 
-env:
-  NODE_VERSION: 16.x
-
 jobs:
-  typecheck:
+  build:
     runs-on: ubuntu-latest
+    permissions:
+      contents: write 
 
     steps:
-      - uses: actions/setup-node@v3
-        with:
-          node-version: ${{ env.NODE_VERSION }}
-      - uses: actions/checkout@v3
-      - run: |
-          npm install
-          npm run typecheck
+    - uses: actions/checkout@v4
+      with:
+        ref: ${{ github.head_ref }}
+        token: ${{ secrets.GITHUB_TOKEN }}
+        
+    - name: Setup Node.js
+      uses: actions/setup-node@v4
+      with:
+        node-version: '18'
+        cache: 'npm'
+    
+    - name: Install dependencies
+      run: npm ci
+    
+    - name: Run type check
+      run: npm run typecheck
+    
+    - name: Format code
+      run: npm run format
+      
+    - name: Commit changes
+      uses: stefanzweifel/git-auto-commit-action@v5
+      with:
+        commit_message: "style: format code with prettier"
+        branch: ${{ github.head_ref }}

cpp/whisper.cpp

@@ -8,7 +8,7 @@
 		"access": "public"
 	},
 	"bin": {
-		"download": "dist/downloadModel.js"
+		"nodejs-whisper": "dist/downloadModel.js"
 	},
 	"repository": {
 		"type": "git",
@@ -43,13 +43,7 @@
 		"nodejs-whisper",
 		"audio-to-subtitles"
 	],
-	"author": "chetan baliyan",
-	"contributors": [
-		{
-			"name": "Michael Heuberger",
-			"email": "[email protected]"
-		}
-	],
+	"author": "chetan",
 	"license": "MIT",
 	"homepage": "https://github.com/ChetanXpro/nodejs-whisper#readme",
 	"bugs": {

package.json

@@ -27,7 +27,7 @@ export const constructCommand = (filePath: string, args: IOptions): string => {
 	}
 
 	const modelName = MODEL_OBJECT[args.modelName as keyof typeof MODEL_OBJECT]
-	 let command = `./main  ${constructOptionsFlags(args)} -l ${args.whisperOptions?.language ? args.whisperOptions?.language : 'auto'} -m "./models/${modelName}"  -f "${filePath}"`
+	let command = `./main  ${constructOptionsFlags(args)} -l ${args.whisperOptions?.language ? args.whisperOptions?.language : 'auto'} -m "./models/${modelName}"  -f "${filePath}"`
 
 	return command
 }

src/WhisperHelper.ts

@@ -47,21 +47,22 @@ export default async function autoDownloadModel(
 		shell.cd('../')
 
 		let compileCommand
-        if (process.platform === 'win32') {
-            // Try mingw32-make first
-            if (shell.which('mingw32-make')) {
-                compileCommand = withCuda ? 'WHISPER_CUDA=1 mingw32-make -j' : 'mingw32-make -j'
-            } else if (shell.which('make')) {
-                compileCommand = withCuda ? 'WHISPER_CUDA=1 make -j' : 'make -j'
-            } else {
-                throw new Error('[Nodejs-whisper] Neither mingw32-make nor make found. Please install MinGW-w64 or MSYS2.')
-            }
-        } else {
-            compileCommand = withCuda ? 'WHISPER_CUDA=1 make -j' : 'make -j'
-        }
-
-        const compileResult = shell.exec(compileCommand)
+		if (process.platform === 'win32') {
+			// Try mingw32-make first
+			if (shell.which('mingw32-make')) {
+				compileCommand = withCuda ? 'WHISPER_CUDA=1 mingw32-make -j' : 'mingw32-make -j'
+			} else if (shell.which('make')) {
+				compileCommand = withCuda ? 'WHISPER_CUDA=1 make -j' : 'make -j'
+			} else {
+				throw new Error(
+					'[Nodejs-whisper] Neither mingw32-make nor make found. Please install MinGW-w64 or MSYS2.'
+				)
+			}
+		} else {
+			compileCommand = withCuda ? 'WHISPER_CUDA=1 make -j' : 'make -j'
+		}
 
+		const compileResult = shell.exec(compileCommand)
 
 		if (compileResult.code !== 0) {
 			throw new Error(`[Nodejs-whisper] Failed to compile model: ${compileResult.stderr}`)

src/autoDownloadModel.ts

@@ -46,7 +46,7 @@ const askIfUserWantToUseCuda = async (logger = console) => {
 	}
 }
 
-export default async function downloadModel(logger = console) {
+async function downloadModel(logger = console) {
 	try {
 		shell.cd(path.join(__dirname, '..', './cpp/whisper.cpp/models'))
 
@@ -60,10 +60,9 @@ export default async function downloadModel(logger = console) {
 		})
 
 		if (anyModelExist.length > 0) {
-			return
-			// logger.log('Models already exist. Skipping download.')
-		} else {
-			logger.log('[Nodejs-whisper] Models do not exist. Please Select a model to download.\n')
+			console.log('\n[Nodejs-whisper] Currently installed models:')
+			anyModelExist.forEach(model => console.log(`- ${model}`))
+			console.log('\n[Nodejs-whisper] You can install additional models from the list below.\n')
 		}
 
 		logger.log(`
@@ -115,4 +114,7 @@ export default async function downloadModel(logger = console) {
 	}
 }
 // run on npx nodejs-whisper download
-downloadModel()
+downloadModel().catch(error => {
+	console.error('Failed to download:', error)
+	process.exit(1)
+})