Enhance update output/mechanism

Author: cezaraugusto

programs/cli/check-updates.ts

@@ -16,7 +16,10 @@ function isStableVersion(version: string) {
   return Boolean(v && v.prerelease.length === 0)
 }
 
-export default async function checkUpdates(): Promise<string | null> {
+export default async function checkUpdates(): Promise<{
+  suffix: string
+  message: string
+} | null> {
   const packageJson = getCliPackageJson()
   let update = null
 
@@ -29,7 +32,9 @@ export default async function checkUpdates(): Promise<string | null> {
   }
 
   if (update && isStableVersion(update.latest)) {
-    return messages.checkUpdates(packageJson, update)
+    if (isStableVersion(packageJson.version)) {
+      return messages.checkUpdates(packageJson, update)
+    }
   }
 
   return null

programs/cli/cli-lib/messages.ts

@@ -81,12 +81,14 @@ export function checkUpdates(
   packageJson: Record<string, any>,
   update: {latest: string}
 ) {
-  return (
+  const suffix = colors.gray(`(version ${String(update.latest)} is available!)`)
+  const message =
     `${getLoggingPrefix('info')} 🧩 ${colors.blue('Extension.js')} update available.\n\n` +
     `You are currently using version ${colors.red(String(packageJson.version))}. ` +
     `Latest stable is ${colors.green(String(update.latest))}.\n` +
     `Update to the latest stable to get fixes and new features.`
-  )
+
+  return {suffix, message}
 }
 
 export function unsupportedNodeVersion() {

programs/cli/index.ts

@@ -21,18 +21,28 @@ import {registerPreviewCommand} from './commands/preview'
 import {registerBuildCommand} from './commands/build'
 
 const cliPackageJson = getCliPackageJson()
-process.env.EXTENSION_CLI_VERSION = cliPackageJson.version
+
+function developVersion() {
+  try {
+    const pkg = require('extension-develop/package.json')
+    return pkg?.version || cliPackageJson.version
+  } catch {
+    return cliPackageJson.version
+  }
+}
+
+process.env.EXTENSION_DEVELOP_VERSION = developVersion()
 
 checkUpdates().then((updateMessage) => {
   if (!updateMessage) return
 
   if (process.env.EXTENSION_CLI_BANNER_PRINTED === 'true') {
     // eslint-disable-next-line no-console
-    console.log(updateMessage)
+    console.log(updateMessage.message)
     return
   }
 
-  process.env.EXTENSION_CLI_UPDATE_MESSAGE = updateMessage
+  process.env.EXTENSION_CLI_UPDATE_SUFFIX = updateMessage.suffix
 })
 
 const extensionJs = program

programs/develop/webpack/plugin-browsers/browsers-lib/banner.ts

@@ -18,11 +18,14 @@ type HostPort = {host?: string; port?: number | string}
 
 const printedKeys = new Set<string>()
 
-function readUpdateMessageOnce() {
-  const message = process.env.EXTENSION_CLI_UPDATE_MESSAGE
-  if (!message) return null
-  delete process.env.EXTENSION_CLI_UPDATE_MESSAGE
-  return message
+function readUpdateSuffixOnce() {
+  const suffix = process.env.EXTENSION_CLI_UPDATE_SUFFIX
+
+  if (!suffix) return null
+
+  delete process.env.EXTENSION_CLI_UPDATE_SUFFIX
+
+  return suffix
 }
 
 function keyFor(
@@ -50,6 +53,7 @@ export async function printDevBannerOnce(opts: {
 
   const info = (await opts.getInfo()) || null
   const manifestPath = path.join(opts.outPath, 'manifest.json')
+  const updateSuffix = readUpdateSuffixOnce()
 
   if (!fs.existsSync(manifestPath)) return false
 
@@ -72,13 +76,10 @@ export async function printDevBannerOnce(opts: {
       manifest,
       opts.browser,
       message,
-      opts.browserVersionLine
+      opts.browserVersionLine,
+      updateSuffix || undefined
     )
   )
-  const updateMessage = readUpdateMessageOnce()
-  if (updateMessage) {
-    console.log(updateMessage)
-  }
   console.log(messages.emptyLine())
   markBannerPrinted()
   process.env.EXTENSION_CLI_BANNER_PRINTED = 'true'
@@ -101,6 +102,7 @@ export async function printProdBannerOnce(opts: {
     opts.browserVersionLine && opts.browserVersionLine.trim().length > 0
       ? opts.browserVersionLine.trim()
       : String(opts.browser || 'unknown')
+  const updateSuffix = readUpdateSuffixOnce()
 
   try {
     const manifestPath = path.join(opts.outPath, 'manifest.json')
@@ -123,13 +125,10 @@ export async function printProdBannerOnce(opts: {
           manifest,
           opts.browser,
           message,
-          browserLabel
+          browserLabel,
+          updateSuffix || undefined
         )
       )
-      const updateMessage = readUpdateMessageOnce()
-      if (updateMessage) {
-        console.log(updateMessage)
-      }
       console.log(messages.emptyLine())
     } else {
       // Fallback: manifest-only summary using the unified dev/preview layout.
@@ -149,23 +148,16 @@ export async function printProdBannerOnce(opts: {
           manifest,
           opts.browser,
           message,
-          browserLabel
+          browserLabel,
+          updateSuffix || undefined
         )
       )
-      const updateMessage = readUpdateMessageOnce()
-      if (updateMessage) {
-        console.log(updateMessage)
-      }
       console.log(messages.emptyLine())
     }
   } catch {
     // Fallback: if anything goes wrong, still try to print a minimal card
     console.log(messages.emptyLine())
     console.log(coreMessages.runningInProduction(opts.outPath, browserLabel))
-    const updateMessage = readUpdateMessageOnce()
-    if (updateMessage) {
-      console.log(updateMessage)
-    }
     console.log(messages.emptyLine())
   }
 

programs/develop/webpack/plugin-browsers/browsers-lib/messages.ts

@@ -942,7 +942,8 @@ export function runningInDevelopment(
   manifest: DevManifestInfo,
   browser: DevOptions['browser'],
   message: DevClientMessage,
-  browserVersionLine?: string
+  browserVersionLine?: string,
+  updateSuffix?: string
 ) {
   const capitalize = (str: string) => str.charAt(0).toUpperCase() + str.slice(1)
   const manifestName = manifest.name || 'Extension.js'
@@ -994,6 +995,7 @@ export function runningInDevelopment(
   }
   // Note: keep dynamic import here to avoid ESM JSON import issues at compile time in some bundlers
   const extensionVersion =
+    process.env.EXTENSION_DEVELOP_VERSION ||
     process.env.EXTENSION_CLI_VERSION ||
     (() => {
       try {
@@ -1091,10 +1093,12 @@ export function runningInDevelopment(
 
   const lines: string[] = []
 
+  const updateNotice = updateSuffix ? ` ${updateSuffix}` : ''
+
   lines.push(
     ` 🧩 ${colors.brightBlue('Extension.js')} ${colors.gray(
       `${extensionVersion}`
-    )}`,
+    )}${updateNotice}`,
     `    Browser        ${colors.gray(browserLabel)}`,
     `    Extension      ${colors.gray(
       version ? `${name} ${version}` : name || manifestName

programs/develop/webpack/plugin-extension/feature-special-folders/__spec__/warn-upon-folder-changes.spec.ts

@@ -45,7 +45,11 @@ const createFakeCompiler = () => {
 
 const runCycle = (
   compiler: any,
-  compilation: {warnings: any[]; errors: any[]; contextDependencies: Set<string>}
+  compilation: {
+    warnings: any[]
+    errors: any[]
+    contextDependencies: Set<string>
+  }
 ) => {
   ;(compiler.hooks.watchRun as any).__invokeAll()
   ;(compiler.hooks.thisCompilation as any).__invokeAll(compilation)