feat: replace environment variable release flags with CLI arguments and refactor release fetching logic

Author: Armaldio

src/main.ts

@@ -172,9 +172,52 @@ app.whenReady().then(async () => {
     await handleProtocolUrl(url)
   })
 
-  if (!is.dev || process.env.APP_UPDATE_URL || process.env.PIPELAB_OVERRIDE_RELEASE) {
+  const config = {
+    options: {
+      /** project: path to file .pipelab */
+      project: {
+        type: 'string',
+        short: 'p'
+      },
+      /** action: run | open  */
+      action: {
+        type: 'string',
+        short: 'a'
+      },
+      /** output: path to output result */
+      output: {
+        type: 'string',
+        short: 'o'
+      },
+      inspect: {
+        type: 'boolean'
+      },
+      prerelease: {
+        type: 'boolean'
+      },
+      'override-release': {
+        type: 'string'
+      },
+      'app-update-url': {
+        type: 'boolean'
+      }
+    }
+  } satisfies ParseArgsConfig
+
+  const { values } = parseArgs(config)
+
+  const hasAppUpdateUrlFlag = !!values['app-update-url']
+  const hasOverrideReleaseFlag = !!values['override-release']
+  const hasPrereleaseFlag = !!values['prerelease']
+  const overrideReleaseVal = values['override-release']
+
+  if (!is.dev || hasAppUpdateUrlFlag || hasOverrideReleaseFlag || hasPrereleaseFlag) {
     try {
-      const latestRelease = await fetchLatestRelease('@pipelab/app')
+      const latestRelease = await fetchLatestRelease({
+        packageName: '@pipelab/app',
+        prerelease: hasPrereleaseFlag,
+        overrideRelease: overrideReleaseVal
+      })
       if (latestRelease) {
         logger().info('Found latest release:', latestRelease.tag_name)
         logger().info('Release API URL:', latestRelease.url)
@@ -294,39 +337,13 @@ app.whenReady().then(async () => {
     logger().error('Failed to start WebSocket server:', error)
   }
 
-  const config = {
-    options: {
-      /** project: path to file .pipelab */
-      project: {
-        type: 'string',
-        short: 'p'
-      },
-      /** action: run | open  */
-      action: {
-        type: 'string',
-        short: 'a'
-      },
-      /** output: path to output result */
-      output: {
-        type: 'string',
-        short: 'o'
-      },
-      inspect: {
-        type: 'boolean'
-      }
-    }
-  } satisfies ParseArgsConfig
-
-  const { values } = parseArgs(config)
-
   logger().info('values', values)
 
   createWindow()
 
-  delete values['inspect']
-
-  // exit if values are passed
-  if (Object.keys(values).length > 0) {
+  // exit if values are passed for batch execution
+  const hasPipelineArgs = values.action || values.project || values.output
+  if (hasPipelineArgs) {
     logger().info('Processing graph...')
 
     const { action, project, output } = values
@@ -406,7 +423,7 @@ app.whenReady().then(async () => {
     mainWindow.show()
     mainWindow.maximize()
 
-    if (app.isPackaged || process.env.APP_UPDATE_URL || process.env.PIPELAB_OVERRIDE_RELEASE) {
+    if (app.isPackaged || hasAppUpdateUrlFlag || hasOverrideReleaseFlag || hasPrereleaseFlag) {
       autoUpdater.checkForUpdates()
       setTimeout(() => {
         autoUpdater.checkForUpdates()

src/main/releases.ts

@@ -34,36 +34,43 @@ export const fetchAllReleases = async (): Promise<Release[]> => {
   }
 }
 
+export type FetchLatestReleaseOptions = {
+  packageName?: string
+  prerelease?: boolean
+  overrideRelease?: string
+}
+
 /**
  * Fetches the latest release for a specific package.
- * Supports PIPELAB_OVERRIDE_RELEASE environment variable.
  */
-export const fetchLatestRelease = async (packageName = '@pipelab/app'): Promise<Release | null> => {
+export const fetchLatestRelease = async (
+  options: FetchLatestReleaseOptions = {}
+): Promise<Release | null> => {
+  const { packageName = '@pipelab/app', prerelease = false, overrideRelease } = options
   const { logger } = useLogger()
 
   try {
     const releases = await fetchAllReleases()
 
-    // Handle environment variable override
-    const override = process.env.PIPELAB_OVERRIDE_RELEASE
-    if (override) {
-      logger().info('Using release override:', override)
-      const targetTag = override.includes('@') ? override : `${packageName}@${override}`
+    // Handle command-line override
+    if (overrideRelease) {
+      logger().info('Using release override:', overrideRelease)
+      const targetTag = overrideRelease.includes('@') ? overrideRelease : `${packageName}@${overrideRelease}`
       const release = releases.find((r) => r.tag_name === targetTag)
       if (release) {
         return release
       }
-      logger().warn(`Release override ${override} not found for package ${packageName}`)
+      logger().warn(`Release override ${overrideRelease} not found for package ${packageName}`)
     }
 
     // Helper to get version from tag name
     const getVersion = (tagName: string) => tagName.split('@').pop() || '0.0.0'
 
-    // Filter for releases that follow the {packageName}@X.Y.Z tag pattern and filter out prereleases unconditionally unless PRERELEASE is true
+    // Filter for releases that follow the {packageName}@X.Y.Z tag pattern and filter out prereleases unconditionally unless prerelease is true
     const packageReleases = releases
       .filter((r) => r.tag_name.startsWith(`${packageName}@`))
       .filter((r) => {
-        if (process.env.PRERELEASE === 'true') {
+        if (prerelease) {
           return true
         }
         if (r.prerelease) {