fix: correct behavior when config option is missing

cybozu · Mar 29, 2024 · e2b1ab2 · e2b1ab2
1 parent 1b8c6bb
commit e2b1ab2
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 6 deletions.
diff --git a/src/analyze.ts b/src/analyze.ts
@@ -18,7 +18,7 @@ export const analyze = async ({
   configFilePath,
   ...cliArgs
 }: AnalyzeArgs & {
-  configFilePath?: string;
+  configFilePath: string;
 }) => {
   console.log("🔎 Analyzing dependencies...");
 

diff --git a/src/extract.ts b/src/extract.ts
@@ -22,7 +22,7 @@ export const extract = async ({
   configFilePath,
   ...cliArgs
 }: ExtractArgs & {
-  configFilePath?: string;
+  configFilePath: string;
 }) => {
   console.log("🔪 Extract licenses...");
 

diff --git a/src/functions/loadConfigScript.ts b/src/functions/loadConfigScript.ts
@@ -2,10 +2,8 @@ import fs from "fs";
 import path from "path";
 import { Config } from "../types";
 
-export const loadConfigScript = async (
-  relativeConfigPath: string = "license-manager.config.js"
-): Promise<Partial<Config>> => {
-  const configPath = path.join(process.cwd(), relativeConfigPath);
+export const loadConfigScript = async (relativeConfigPath: string): Promise<Partial<Config>> => {
+  const configPath = path.join(process.cwd(), relativeConfigPath || "license-manager.config.js");
 
   if (!fs.existsSync(configPath)) {
     return {};