@lightnet/cli 4.7.1 → 4.8.1

package/CHANGELOG.md

@@ -1,5 +1,25 @@
 # @lightnet/cli
 
+## 4.8.1
+
+### Patch Changes
+
+- [#454](https://github.com/LightNetDev/LightNet/pull/454) [`5de5b93`](https://github.com/LightNetDev/LightNet/commit/5de5b931575a9e864c5acfdf991e0e9b2980030b) - Rename check-files scope `thumbnails` to `images`.
+
+## 4.8.0
+
+### Minor Changes
+
+- [#449](https://github.com/LightNetDev/LightNet/pull/449) [`b4b1ba9`](https://github.com/LightNetDev/LightNet/commit/b4b1ba9e591af8f6870f2b3eb04029820a008393) - Add repeatable `--exclude` glob exclusions to `check-links` for explicitly exempt media URLs.
+
+### Patch Changes
+
+- [#449](https://github.com/LightNetDev/LightNet/pull/449) [`b4b1ba9`](https://github.com/LightNetDev/LightNet/commit/b4b1ba9e591af8f6870f2b3eb04029820a008393) - Update dependencies
+
+- [#449](https://github.com/LightNetDev/LightNet/pull/449) [`b4b1ba9`](https://github.com/LightNetDev/LightNet/commit/b4b1ba9e591af8f6870f2b3eb04029820a008393) - Support non interactive usage of check-translations cli script.
+
+- [#449](https://github.com/LightNetDev/LightNet/pull/449) [`b4b1ba9`](https://github.com/LightNetDev/LightNet/commit/b4b1ba9e591af8f6870f2b3eb04029820a008393) - Internal fix typings after dependency update.
+
 ## 4.7.1
 
 ### Patch Changes

package/package.json

@@ -3,7 +3,7 @@
   "description": "CLI for managing LightNet projects",
   "type": "module",
   "license": "MIT",
-  "version": "4.7.1",
+  "version": "4.8.1",
   "publishConfig": {
     "access": "public"
   },
@@ -25,11 +25,11 @@
     "CHANGELOG.md"
   ],
   "dependencies": {
-    "@clack/prompts": "^1.7.0",
+    "@clack/prompts": "^1.8.0",
     "commander": "^15.0.0"
   },
   "devDependencies": {
-    "vitest": "^4.1.9"
+    "vitest": "^5.0.0"
   },
   "engines": {
     "node": ">=22"

package/src/check-files.js

@@ -24,7 +24,7 @@ import {
 import { cancelPrompt } from "./support/prompt-cancel.js"
 import { createR2FileStorage } from "./support/r2.js"
 
-const supportedScopes = new Set(["content-files", "thumbnails"])
+const supportedScopes = new Set(["content-files", "images"])
 
 const mediaDir = "src/content/media"
 const mediaImagesDir = "src/content/media/images"
@@ -83,7 +83,7 @@ export async function checkFiles(options, runtime = {}) {
 
   const scopes = parseScopes(options.scope)
   const includeContentFiles = scopes.has("content-files")
-  const includeThumbnails = scopes.has("thumbnails")
+  const includeImages = scopes.has("images")
 
   intro("check-files")
 
@@ -102,9 +102,9 @@ export async function checkFiles(options, runtime = {}) {
   /** @type {string[]} */
   let orphanedContentFiles = []
   /** @type {string[]} */
-  let orphanedMediaThumbnails = []
+  let orphanedMediaImages = []
   /** @type {string[]} */
-  let orphanedCategoryThumbnails = []
+  let orphanedCategoryImages = []
   /** @type {string[]} */
   let removedItems = []
   /** @type {string[]} */
@@ -112,8 +112,8 @@ export async function checkFiles(options, runtime = {}) {
 
   /** @type {FileStorage|undefined} */
   let contentFileStorage = undefined
-  const mediaThumbnailStorage = files(cwd, { rootDir: mediaImagesDir })
-  const categoryThumbnailStorage = files(cwd, { rootDir: categoryImagesDir })
+  const mediaImageStorage = files(cwd, { rootDir: mediaImagesDir })
+  const categoryImageStorage = files(cwd, { rootDir: categoryImagesDir })
 
   if (includeContentFiles) {
     contentFileStorage = options.r2
@@ -153,7 +153,7 @@ export async function checkFiles(options, runtime = {}) {
       orphanedContentFiles = r2Result.orphanedFiles
       if (r2Result.referenceCount === 0) {
         warnings.push(
-          'No R2-backed content file references found. Use "--scope=thumbnails" or check your "--r2" setup.',
+          'No R2-backed content file references found. Use "--scope=images" or check your "--r2" setup.',
         )
       }
     } else {
@@ -177,21 +177,21 @@ export async function checkFiles(options, runtime = {}) {
     }
   }
 
-  if (includeThumbnails) {
-    const thumbnailsResult = await runSpinner({
-      error: "Thumbnail check failed.",
-      start: "Checking thumbnails",
-      stop: (result) => formatThumbnailCheckSummary(result),
+  if (includeImages) {
+    const imagesResult = await runSpinner({
+      error: "Image check failed.",
+      start: "Checking images",
+      stop: (result) => formatImageCheckSummary(result),
       task: () =>
-        validateThumbnails(
+        validateImages(
           mediaItems,
           categories,
-          mediaThumbnailStorage,
-          categoryThumbnailStorage,
+          mediaImageStorage,
+          categoryImageStorage,
         ),
     })
-    orphanedMediaThumbnails = thumbnailsResult.orphanedMediaThumbnails
-    orphanedCategoryThumbnails = thumbnailsResult.orphanedCategoryThumbnails
+    orphanedMediaImages = imagesResult.orphanedMediaImages
+    orphanedCategoryImages = imagesResult.orphanedCategoryImages
   }
 
   if (options.fix || options.fixWithoutConfirm) {
@@ -212,7 +212,7 @@ export async function checkFiles(options, runtime = {}) {
       contentDeletions.push(deletion)
       deletions.push(deletion)
     }
-    for (const filePath of orphanedMediaThumbnails) {
+    for (const filePath of orphanedMediaImages) {
       const deletion = {
         displayPath: filePath,
         target: filePath,
@@ -220,7 +220,7 @@ export async function checkFiles(options, runtime = {}) {
       localDeletions.push(deletion)
       deletions.push(deletion)
     }
-    for (const filePath of orphanedCategoryThumbnails) {
+    for (const filePath of orphanedCategoryImages) {
       const deletion = {
         displayPath: filePath,
         target: filePath,
@@ -257,22 +257,22 @@ export async function checkFiles(options, runtime = {}) {
             task: async () => {
               const [
                 deletedContentTargets,
-                deletedMediaThumbnailTargets,
-                deletedCategoryThumbnailTargets,
+                deletedMediaImageTargets,
+                deletedCategoryImageTargets,
               ] = await Promise.all([
                 contentFileStorage
                   ? contentFileStorage.delete(
                       contentDeletions.map((deletion) => deletion.target),
                     )
                   : [],
-                mediaThumbnailStorage.delete(
+                mediaImageStorage.delete(
                   localDeletions
                     .filter((deletion) =>
                       deletion.target.startsWith(mediaImagesDir),
                     )
                     .map((deletion) => deletion.target),
                 ),
-                categoryThumbnailStorage.delete(
+                categoryImageStorage.delete(
                   localDeletions
                     .filter((deletion) =>
                       deletion.target.startsWith(categoryImagesDir),
@@ -283,8 +283,8 @@ export async function checkFiles(options, runtime = {}) {
               return {
                 deletedContentTargets,
                 deletedLocalTargets: [
-                  ...deletedMediaThumbnailTargets,
-                  ...deletedCategoryThumbnailTargets,
+                  ...deletedMediaImageTargets,
+                  ...deletedCategoryImageTargets,
                 ],
               }
             },
@@ -304,10 +304,10 @@ export async function checkFiles(options, runtime = {}) {
         orphanedContentFiles = orphanedContentFiles.filter(
           (item) => !deletedTargets.has(item),
         )
-        orphanedMediaThumbnails = orphanedMediaThumbnails.filter(
+        orphanedMediaImages = orphanedMediaImages.filter(
           (item) => !deletedTargets.has(item),
         )
-        orphanedCategoryThumbnails = orphanedCategoryThumbnails.filter(
+        orphanedCategoryImages = orphanedCategoryImages.filter(
           (item) => !deletedTargets.has(item),
         )
       }
@@ -318,8 +318,8 @@ export async function checkFiles(options, runtime = {}) {
     missingContentFiles.length > 0 ||
     wrongTypeR2ContentFiles.length > 0 ||
     orphanedContentFiles.length > 0 ||
-    orphanedMediaThumbnails.length > 0 ||
-    orphanedCategoryThumbnails.length > 0
+    orphanedMediaImages.length > 0 ||
+    orphanedCategoryImages.length > 0
 
   for (const warning of warnings) {
     log.warn(warning)
@@ -344,8 +344,8 @@ export async function checkFiles(options, runtime = {}) {
       formatContentFilePath(filePath, options),
     ),
   )
-  printSection("Orphaned media thumbnails", orphanedMediaThumbnails)
-  printSection("Orphaned category thumbnails", orphanedCategoryThumbnails)
+  printSection("Orphaned media images", orphanedMediaImages)
+  printSection("Orphaned category images", orphanedCategoryImages)
   printSection("Removed items", removedItems)
 
   outro(hasIssues ? "Issues found. 🚧" : "Cleanup complete. 🧹")
@@ -458,17 +458,16 @@ function formatContentCheckSummary(result) {
 /**
  * @param {{
  *   fileCount: number
- *   orphanedCategoryThumbnails: string[]
- *   orphanedMediaThumbnails: string[]
+ *   orphanedCategoryImages: string[]
+ *   orphanedMediaImages: string[]
  *   referenceCount: number
  * }} result
  */
-function formatThumbnailCheckSummary(result) {
-  const orphanedThumbnails =
-    result.orphanedMediaThumbnails.length +
-    result.orphanedCategoryThumbnails.length
+function formatImageCheckSummary(result) {
+  const orphanedImages =
+    result.orphanedMediaImages.length + result.orphanedCategoryImages.length
 
-  return `Thumbnails: ${formatCount(result.referenceCount, "reference")}, ${formatCount(result.fileCount, "file")}; orphaned: ${orphanedThumbnails} (${result.orphanedMediaThumbnails.length} media, ${result.orphanedCategoryThumbnails.length} categories).`
+  return `Images: ${formatCount(result.referenceCount, "reference")}, ${formatCount(result.fileCount, "file")}; orphaned: ${orphanedImages} (${result.orphanedMediaImages.length} media, ${result.orphanedCategoryImages.length} categories).`
 }
 
 /**
@@ -502,7 +501,7 @@ function formatDeletionProgress(action, count) {
  * @param {string|undefined} rawScope
  */
 function parseScopes(rawScope) {
-  const values = (rawScope ?? "content-files,thumbnails")
+  const values = (rawScope ?? "content-files,images")
     .split(",")
     .map((value) => value.trim())
     .filter(Boolean)
@@ -510,7 +509,7 @@ function parseScopes(rawScope) {
   if (values.length === 0) {
     throw new CliError(
       // intentionally kept as a CLI-facing validation error
-      'Expected "--scope" to include at least one of "content-files" or "thumbnails".',
+      'Expected "--scope" to include at least one of "content-files" or "images".',
     )
   }
 
@@ -518,7 +517,7 @@ function parseScopes(rawScope) {
   for (const scope of scopes) {
     if (!supportedScopes.has(scope)) {
       throw new CliError(
-        `Unsupported scope "${scope}". Allowed values: content-files, thumbnails.`,
+        `Unsupported scope "${scope}". Allowed values: content-files, images.`,
       )
     }
   }
@@ -540,31 +539,28 @@ async function assertLightNetSiteRoot(cwd) {
 /**
  * @param {MediaItem[]} mediaItems
  * @param {Category[]} categories
- * @param {FileStorage} mediaThumbnailStorage
- * @param {FileStorage} categoryThumbnailStorage
+ * @param {FileStorage} mediaImageStorage
+ * @param {FileStorage} categoryImageStorage
  */
-async function validateThumbnails(
+async function validateImages(
   mediaItems,
   categories,
-  mediaThumbnailStorage,
-  categoryThumbnailStorage,
+  mediaImageStorage,
+  categoryImageStorage,
 ) {
   const mediaReferences = new Set(
     mediaItems
-      .map((item) => toThumbnailPath("media", item.image))
+      .map((item) => toImagePath("media", item.image))
       .filter((value) => value !== undefined),
   )
   const categoryReferences = new Set(
     categories
-      .map((item) => toThumbnailPath("categories", item.image))
+      .map((item) => toImagePath("categories", item.image))
       .filter((value) => value !== undefined),
   )
 
   const [initializedMediaStorage, initializedCategoryStorage] =
-    await Promise.all([
-      mediaThumbnailStorage.init(),
-      categoryThumbnailStorage.init(),
-    ])
+    await Promise.all([mediaImageStorage.init(), categoryImageStorage.init()])
   const [mediaFiles, categoryFiles] = await Promise.all([
     initializedMediaStorage.list(),
     initializedCategoryStorage.list(),
@@ -572,10 +568,10 @@ async function validateThumbnails(
 
   return {
     fileCount: mediaFiles.length + categoryFiles.length,
-    orphanedMediaThumbnails: mediaFiles.filter(
+    orphanedMediaImages: mediaFiles.filter(
       (file) => !mediaReferences.has(file),
     ),
-    orphanedCategoryThumbnails: categoryFiles.filter(
+    orphanedCategoryImages: categoryFiles.filter(
       (file) => !categoryReferences.has(file),
     ),
     referenceCount: mediaReferences.size + categoryReferences.size,
@@ -700,7 +696,7 @@ function addFileReference(references, filePath, displayPath, sourceFileName) {
  * @param {"media"|"categories"} kind
  * @param {string|undefined} image
  */
-function toThumbnailPath(kind, image) {
+function toImagePath(kind, image) {
   if (typeof image !== "string") {
     return undefined
   }
@@ -772,5 +768,5 @@ async function defaultPromptConfirm(message) {
   if (isCancel(answer)) {
     cancelPrompt()
   }
-  return answer
+  return Boolean(answer)
 }

package/src/check-links.js

@@ -1,6 +1,6 @@
 // @ts-check
 
-import { posix, resolve } from "node:path"
+import { matchesGlob, posix, resolve } from "node:path"
 import { cwd as processCwd, stdin, stdout } from "node:process"
 
 import { intro, isCancel, log, outro, progress, text } from "@clack/prompts"
@@ -28,6 +28,7 @@ const requestHeaders = {
 
 /**
  * @typedef {{
+ *   exclude?: string[]
  *   timeout?: string
  * }} CheckLinksOptions
  */
@@ -49,6 +50,14 @@ const requestHeaders = {
  * }} LinkReference
  */
 
+/**
+ * @typedef {{
+ *   displayUrl: string
+ *   patterns: Set<string>
+ *   sources: Set<string>
+ * }} ExcludedLinkReference
+ */
+
 /**
  * @typedef {{
  *   ok: boolean
@@ -73,6 +82,7 @@ export async function checkLinks(options, runtime = {}) {
     "--timeout",
     defaultTimeoutMs,
   )
+  const excludePatterns = parseExcludePatterns(options.exclude)
 
   intro("check-links")
 
@@ -82,17 +92,28 @@ export async function checkLinks(options, runtime = {}) {
   const mediaItems = await collections.getMediaItems()
   const siteUrl = await initSiteUrl({
     cwd,
+    excludePatterns,
     interactive,
     mediaItems,
     promptText,
   })
-  const references = collectLinkReferences(mediaItems, siteUrl)
+  const { excludedReferences, references } = collectLinkReferences(
+    mediaItems,
+    siteUrl,
+    excludePatterns,
+  )
   log.message(
     `Checking ${references.length} unique links from ${mediaItems.length} media items.`,
   )
 
+  printExcludedLinks(excludedReferences)
+
   if (references.length === 0) {
-    log.warn("No media content URLs found.")
+    log.warn(
+      excludedReferences.length > 0
+        ? "No media content URLs require checking."
+        : "No media content URLs found.",
+    )
     outro("No issues found. 🎉")
     return true
   }
@@ -179,14 +200,25 @@ async function checkReferencesWithProgress(references, options) {
 /**
  * @param {{
  *   cwd: string
+ *   excludePatterns: string[]
  *   interactive: boolean
  *   mediaItems: MediaItem[]
  *   promptText: (message: string) => Promise<string>
  * }} args
  */
-async function initSiteUrl({ cwd, interactive, mediaItems, promptText }) {
+async function initSiteUrl({
+  cwd,
+  excludePatterns,
+  interactive,
+  mediaItems,
+  promptText,
+}) {
   const needsSiteUrl = mediaItems.some((item) =>
-    item.content.some((contentItem) => isRootRelativeUrl(contentItem.url)),
+    item.content.some(
+      (contentItem) =>
+        isRootRelativeUrl(contentItem.url) &&
+        findMatchingPatterns(contentItem.url, excludePatterns).length === 0,
+    ),
   )
   if (!needsSiteUrl) {
     return undefined
@@ -236,13 +268,36 @@ function parseSiteUrl(config) {
 /**
  * @param {MediaItem[]} mediaItems
  * @param {string|undefined} siteUrl
+ * @param {string[]} excludePatterns
  */
-function collectLinkReferences(mediaItems, siteUrl) {
+function collectLinkReferences(mediaItems, siteUrl, excludePatterns) {
   /** @type {Map<string, LinkReference>} */
   const references = new Map()
+  /** @type {Map<string, ExcludedLinkReference>} */
+  const excludedReferences = new Map()
   for (const item of mediaItems) {
     const sourceFileName = posix.basename(toPosixPath(item.path))
     for (const contentItem of item.content) {
+      const matchingPatterns = findMatchingPatterns(
+        contentItem.url,
+        excludePatterns,
+      )
+      if (matchingPatterns.length > 0) {
+        const current = excludedReferences.get(contentItem.url)
+        if (current) {
+          current.sources.add(sourceFileName)
+          for (const pattern of matchingPatterns) {
+            current.patterns.add(pattern)
+          }
+        } else {
+          excludedReferences.set(contentItem.url, {
+            displayUrl: contentItem.url,
+            patterns: new Set(matchingPatterns),
+            sources: new Set([sourceFileName]),
+          })
+        }
+        continue
+      }
       const resolvedUrl = resolveUrl(contentItem.url, siteUrl)
       const current = references.get(resolvedUrl)
       if (current) {
@@ -259,9 +314,14 @@ function collectLinkReferences(mediaItems, siteUrl) {
       }
     }
   }
-  return [...references.values()].sort((a, b) =>
-    a.displayUrl.localeCompare(b.displayUrl),
-  )
+  return {
+    excludedReferences: [...excludedReferences.values()].sort((a, b) =>
+      a.displayUrl.localeCompare(b.displayUrl),
+    ),
+    references: [...references.values()].sort((a, b) =>
+      a.displayUrl.localeCompare(b.displayUrl),
+    ),
+  }
 }
 
 /**
@@ -334,6 +394,25 @@ function printProtectedLinks(links) {
   }
 }
 
+/**
+ * @param {ExcludedLinkReference[]} links
+ */
+function printExcludedLinks(links) {
+  if (links.length === 0) {
+    return
+  }
+
+  log.info(`Excluded links (${links.length})`)
+  for (const link of links) {
+    const patterns = [...link.patterns]
+      .toSorted()
+      .map((pattern) => `"${pattern}"`)
+    log.message(
+      `• ${link.displayUrl}\n  Excluded by: ${patterns.join(", ")}\n${formatReferenceSources(link.sources)}`,
+    )
+  }
+}
+
 /**
  * @param {Set<string>} sources
  */
@@ -426,6 +505,63 @@ function parsePositiveInteger(rawValue, optionName, defaultValue) {
   return value
 }
 
+/**
+ * @param {string[]|undefined} patterns
+ */
+function parseExcludePatterns(patterns) {
+  return (patterns ?? []).map((pattern) => {
+    if (!isValidGlobPattern(pattern)) {
+      throw new CliError(`Invalid "--exclude" glob pattern "${pattern}".`)
+    }
+    return pattern
+  })
+}
+
+/**
+ * @param {string} url
+ * @param {string[]} patterns
+ */
+function findMatchingPatterns(url, patterns) {
+  return patterns.filter((pattern) => matchesGlob(url, pattern))
+}
+
+/**
+ * Node's glob matcher treats unclosed groups as non-matches, so validate them
+ * explicitly to turn common malformed patterns into actionable CLI errors.
+ *
+ * @param {string} pattern
+ */
+function isValidGlobPattern(pattern) {
+  /** @type {Partial<Record<string, string>>} */
+  const pairs = { "[": "]", "{": "}", "(": ")" }
+  /** @type {string[]} */
+  const expectedClosers = []
+  let escaped = false
+
+  for (const character of pattern) {
+    if (escaped) {
+      escaped = false
+      continue
+    }
+    if (character === "\\") {
+      escaped = true
+      continue
+    }
+    const closer = pairs[character]
+    if (closer) {
+      expectedClosers.push(closer)
+      continue
+    }
+    if (character === "]" || character === "}" || character === ")") {
+      if (expectedClosers.pop() !== character) {
+        return false
+      }
+    }
+  }
+
+  return !escaped && expectedClosers.length === 0
+}
+
 /**
  * @param {string} value
  * @param {string} label

package/src/check-translations.js

@@ -3,7 +3,7 @@
 import { spawn } from "node:child_process"
 import { readFile } from "node:fs/promises"
 import { resolve } from "node:path"
-import { cwd } from "node:process"
+import { cwd, stdin, stdout } from "node:process"
 
 import { confirm, intro, isCancel, log, outro, taskLog } from "@clack/prompts"
 
@@ -30,7 +30,14 @@ import { cancelPrompt } from "./support/prompt-cancel.js"
  * }} CheckTranslationsOptions
  */
 
-const lightnetCachePath = resolve(cwd(), "node_modules", ".cache", "lightnet")
+/**
+ * @typedef {{
+ *   cwd?: string
+ *   isInteractive?: boolean
+ *   promptRunBuild?: () => Promise<boolean>
+ *   write?: (message: string) => void
+ * }} CheckTranslationsRuntime
+ */
 
 /** @type {{type:Translation["type"], title:string, action:string}[]} */
 const translationSources = [
@@ -54,18 +61,33 @@ const translationSources = [
 
 /**
  * @param {CheckTranslationsOptions} [options]
+ * @param {CheckTranslationsRuntime} [runtime]
  */
-export async function checkTranslations(options = {}) {
-  intro("check-translations")
+export async function checkTranslations(options = {}, runtime = {}) {
+  const lightnetCachePath = resolve(
+    runtime.cwd ?? cwd(),
+    "node_modules",
+    ".cache",
+    "lightnet",
+  )
+  const interactive =
+    runtime.isInteractive ?? Boolean(stdin.isTTY && stdout.isTTY)
+  const output = createOutput(interactive, runtime.write)
+
+  output.intro("check-translations")
 
-  const buildAvailable = await runBuild(options.build)
+  const buildAvailable = await runBuild(options.build, {
+    interactive,
+    output,
+    promptRunBuild: runtime.promptRunBuild,
+  })
   if (!buildAvailable) {
-    outro("Build failed. 🚧")
+    output.outro("Build failed. 🚧")
     return false
   }
 
-  const translations = await readTranslations()
-  const languages = await readLanguages()
+  const translations = await readTranslations(lightnetCachePath, output)
+  const languages = await readLanguages(lightnetCachePath, output)
   if (!translations || !languages || translations.length === 0) {
     return false
   }
@@ -77,7 +99,7 @@ export async function checkTranslations(options = {}) {
     .filter((translation) => translation.missingLocales.length > 0)
 
   if (incompleteTranslations.length === 0) {
-    outro("No issues found. 🎉")
+    output.outro("No issues found. 🎉")
     return true
   }
 
@@ -86,29 +108,42 @@ export async function checkTranslations(options = {}) {
     (translation) => translation.type,
   )
 
-  log.error("Translation check failed")
+  output.error("Translation check failed")
   for (const source of translationSources) {
-    printMissingTranslations(source, grouped[source.type])
+    printMissingTranslations(source, grouped[source.type], output)
   }
 
-  outro("Issues found. 🚧")
+  output.outro("Issues found. 🚧")
 
   return false
 }
 
 /**
- *
  * @param {boolean|undefined} build
- * @returns
+ * @param {{
+ *   interactive: boolean
+ *   output: ReturnType<typeof createOutput>
+ *   promptRunBuild?: () => Promise<boolean>
+ * }} runtime
  */
-async function runBuild(build) {
-  const shouldRunBuild = build ?? (await promptRunBuild())
+async function runBuild(build, { interactive, output, promptRunBuild }) {
+  const shouldRunBuild =
+    build ??
+    (interactive ? await (promptRunBuild ?? defaultPromptRunBuild)() : false)
   if (!shouldRunBuild) {
     return true
   }
-  const buildLog = taskLog({
-    title: "Running pnpm build",
-  })
+  const buildLog = interactive
+    ? taskLog({ title: "Running pnpm build" })
+    : {
+        error: output.error,
+        message: output.message,
+        success: output.message,
+      }
+
+  if (!interactive) {
+    output.message("Running pnpm build")
+  }
 
   const child = spawn("pnpm", ["build"], {
     shell: process.platform === "win32",
@@ -154,7 +189,7 @@ async function runBuild(build) {
   }
 }
 
-async function promptRunBuild() {
+async function defaultPromptRunBuild() {
   const answer = await confirm({
     message:
       "Run pnpm build now? Command requires an up-to-date dist/ directory.",
@@ -170,13 +205,14 @@ async function promptRunBuild() {
  *
  * @param {{title:string, action:string}} source
  * @param {(Translation & {missingLocales:string[]})[]|undefined} translations
+ * @param {ReturnType<typeof createOutput>} output
  */
-function printMissingTranslations(source, translations) {
+function printMissingTranslations(source, translations, output) {
   if (!translations || translations.length === 0) {
     return
   }
 
-  log.warn(source.title)
+  output.warn(source.title)
   translations
     .toSorted(
       (t1, t2) =>
@@ -184,10 +220,10 @@ function printMissingTranslations(source, translations) {
         t1.key.localeCompare(t2.key),
     )
     .forEach(({ key, missingLocales }) => {
-      log.message(`• ${key} > Missing: ${missingLocales.join(", ")}`)
+      output.message(`• ${key} > Missing: ${missingLocales.join(", ")}`)
     })
 
-  log.message(`Action: ${source.action}`)
+  output.message(`Action: ${source.action}`)
 }
 
 /**
@@ -201,9 +237,11 @@ function getMissingLocales(translation, languages) {
 }
 
 /**
+ * @param {string} lightnetCachePath
+ * @param {ReturnType<typeof createOutput>} output
  * @returns {Promise<Translation[]|undefined>}
  */
-async function readTranslations() {
+async function readTranslations(lightnetCachePath, output) {
   try {
     const translationsText = await readFile(
       resolve(lightnetCachePath, "translations.jsonl"),
@@ -214,16 +252,18 @@ async function readTranslations() {
       .filter((line) => line.trim())
       .map((line) => JSON.parse(line))
   } catch {
-    log.error("No translation build cache found.")
-    log.error("Action: Run build and try lightnet check-translations again.")
+    output.error("No translation build cache found.")
+    output.error("Action: Run build and try lightnet check-translations again.")
     return undefined
   }
 }
 
 /**
+ * @param {string} lightnetCachePath
+ * @param {ReturnType<typeof createOutput>} output
  * @returns {Promise<Languages|undefined>}
  */
-async function readLanguages() {
+async function readLanguages(lightnetCachePath, output) {
   try {
     const languagesText = await readFile(
       resolve(lightnetCachePath, "languages.json"),
@@ -231,8 +271,37 @@ async function readLanguages() {
     )
     return JSON.parse(languagesText)
   } catch {
-    log.error("No language manifest found from the last build.")
-    log.error("Action: Run build and try lightnet check-translations again.")
+    output.error("No language manifest found from the last build.")
+    output.error("Action: Run build and try lightnet check-translations again.")
     return undefined
   }
 }
+
+/**
+ * @param {boolean} interactive
+ * @param {(message: string) => void | undefined} write
+ */
+function createOutput(
+  interactive,
+  write = (message) => {
+    stdout.write(`${message}\n`)
+  },
+) {
+  if (interactive) {
+    return {
+      error: log.error,
+      intro,
+      message: log.message,
+      outro,
+      warn: log.warn,
+    }
+  }
+
+  return {
+    error: write,
+    intro: write,
+    message: write,
+    outro: write,
+    warn: write,
+  }
+}

package/src/index.js

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 // @ts-check
 
-import { Command } from "commander"
+import { Command, Option } from "commander"
 
 import pkg from "../package.json" with { type: "json" }
 import { checkFiles } from "./check-files.js"
@@ -37,7 +37,7 @@ program
 program
   .command("check-files")
   .description(
-    "check for missing and orphaned content files and thumbnails in a LightNet site",
+    "check for missing and orphaned content files and images in a LightNet site",
   )
   .option("--fix", "remove orphaned files")
   .option("--fix-without-confirm", "remove orphaned files without confirmation")
@@ -45,10 +45,7 @@ program
     "--r2",
     "validate remote content files in Cloudflare R2 instead of public/files",
   )
-  .option(
-    "--scope <values>",
-    "comma-separated scopes: content-files,thumbnails",
-  )
+  .option("--scope <values>", "comma-separated scopes: content-files,images")
   .action(async (options) => {
     try {
       const checkSuccessful = await checkFiles(options)
@@ -62,6 +59,14 @@ program
   .command("check-links")
   .description("check media content links in a LightNet site")
   .option("--timeout <ms>", "request timeout per link in milliseconds")
+  .addOption(
+    new Option(
+      "--exclude <pattern>",
+      "exclude media content URLs matching a glob pattern (repeatable)",
+    )
+      .default([])
+      .argParser((pattern, patterns) => [...patterns, pattern]),
+  )
   .action(async (options) => {
     try {
       const checkSuccessful = await checkLinks(options)

package/src/support/r2.js

@@ -617,9 +617,10 @@ async function promptRequiredSecret({ interactive = true } = {}) {
     if (isCancel(value)) {
       cancelPrompt()
     }
-    if (value.trim()) {
-      processEnv[sessionSecretEnvName] = value
-      return value
+    const secret = typeof value === "string" ? value : ""
+    if (secret.trim()) {
+      processEnv[sessionSecretEnvName] = secret
+      return secret
     }
   }
 }