From 1635bda15f2bc92cf51bec2b1b691a4b4b7865db Mon Sep 17 00:00:00 2001 From: Zaran Lalvani Date: Mon, 29 Apr 2024 14:44:43 -0400 Subject: [PATCH] fix: do not send empty api key to LLM (#49) --- apps/cli/src/index.tsx | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/apps/cli/src/index.tsx b/apps/cli/src/index.tsx index 110af28..1ee48c4 100644 --- a/apps/cli/src/index.tsx +++ b/apps/cli/src/index.tsx @@ -3,7 +3,6 @@ import React from "react"; import { Option, program } from "@commander-js/extra-typings"; import { select } from "@inquirer/prompts"; import { render } from "ink"; -import { toInt } from "radash"; import { serializeError } from "serialize-error"; import { @@ -54,6 +53,15 @@ if (isNaN(port)) { process.exit(1); } +const resolvedToken = token ?? process.env.LLM_API_KEY; + +if (!resolvedToken) { + console.log( + "LLM token must be provided (either via --token or the LLM_API_KEY environment variable)", + ); + process.exit(1); +} + const bumpFinder = makeBumpFinder({ language, projectRoot: process.cwd(), @@ -95,7 +103,7 @@ if (!pkg) { } const bumpgen = makeBumpgen({ - llmApiKey: token ?? process.env.LLM_API_KEY ?? "", + llmApiKey: resolvedToken, model, packageToUpgrade: { packageName: pkg,