refactor: move package.json existence check to top of handlerOptionBuild

RaulCatalinas · May 3, 2024 · 360518b · 360518b
1 parent cbd40c5
commit 360518b
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 13 deletions.
diff --git a/src/controllers/handlers-options.ts b/src/controllers/handlers-options.ts
@@ -9,13 +9,24 @@ import opener from 'opener'
 import { addCommitlint } from '@/utils/add-commitlint'
 import { generateCommitlintConfig } from '@/utils/commitlint'
 import { generateHuskyConfig } from '@/utils/husky-library'
+import { existPackageJson } from '@/utils/package-json'
 import { getPackageManger } from '@/utils/package-managers'
 
 // NodeJS
 import process from 'node:process'
 
 export const handlerOptionBuild = async () => {
   try {
+    const packageJsonPath = `${process.cwd()}/package.json`
+    const existPackageJsonInTheCurrentDirectory =
+      await existPackageJson(packageJsonPath)
+
+    if (!existPackageJsonInTheCurrentDirectory) {
+      console.error(ErrorMessages.NotFound)
+
+      process.exit(1)
+    }
+
     const packageManagerToUse = await getPackageManger()
     const useCommitlint = await addCommitlint()
 

diff --git a/src/utils/package-json.ts b/src/utils/package-json.ts
@@ -12,21 +12,11 @@ import type { PackageJson } from '@/types/package-json'
 interface Props {
   key: string
   value: string
+  packageJsonPath: string
 }
 
-export async function addScript({ key, value }: Props) {
+export async function addScript({ key, value, packageJsonPath }: Props) {
   try {
-    const packageJsonPath = `${process.cwd()}/package.json`
-
-    const existPackageJsonInTheCurrentDirectory =
-      await existPackageJson(packageJsonPath)
-
-    if (!existPackageJsonInTheCurrentDirectory) {
-      console.error(ErrorMessages.NotFound)
-
-      process.exit(1)
-    }
-
     const packageJsonData = await fs.readFile(packageJsonPath, {
       encoding: UTF8_ENCODING
     })
@@ -48,7 +38,7 @@ export async function addScript({ key, value }: Props) {
   }
 }
 
-async function existPackageJson(path: string) {
+export async function existPackageJson(path: string) {
   try {
     return await fs.exists(path)
   } catch {