@lightnet/cli 4.3.0 → 4.4.0

package/CHANGELOG.md

@@ -1,5 +1,15 @@
 # @lightnet/cli
 
+## 4.4.0
+
+### Minor Changes
+
+- [#429](https://github.com/LightNetDev/LightNet/pull/429) [`00faa68`](https://github.com/LightNetDev/LightNet/commit/00faa6806bbdb74aef665d71b813449f2a15137f) - Add `check-translations --no-build` and `check-translations --build` support and improve CLI check commands.
+
+### Patch Changes
+
+- [#429](https://github.com/LightNetDev/LightNet/pull/429) [`00faa68`](https://github.com/LightNetDev/LightNet/commit/00faa6806bbdb74aef665d71b813449f2a15137f) - Rename `check-files --yes` to `check-files --no-confirm` for clearer unattended deletion.
+
 ## 4.3.0
 
 ### Minor Changes

package/package.json

@@ -3,7 +3,7 @@
   "description": "CLI for managing LightNet projects",
   "type": "module",
   "license": "MIT",
-  "version": "4.3.0",
+  "version": "4.4.0",
   "publishConfig": {
     "access": "public"
   },

package/src/check-files.js

@@ -25,7 +25,7 @@ const categoryImagesDir = "src/content/categories/images"
  * @typedef {{
  *   scope?: string
  *   fix?: boolean
- *   yes?: boolean
+ *   confirm?: boolean
  *   r2?: boolean
  * }} CheckFilesOptions
  */
@@ -185,13 +185,14 @@ export async function checkFiles(options, runtime = {}) {
     }
 
     if (deletions.length > 0) {
-      if (!options.yes && !interactive) {
+      const shouldSkipConfirmation = options.confirm === false
+      if (!shouldSkipConfirmation && !interactive) {
         throw new CliError(
-          'Deletion requires confirmation. Re-run with "--fix --yes" in non-interactive environments.',
+          'Deletion requires confirmation. Re-run with "--fix --no-confirm" in non-interactive environments.',
         )
       }
       const shouldDelete =
-        options.yes ||
+        shouldSkipConfirmation ||
         (await confirmDeletion({
           deletions,
           promptConfirm,

package/src/check-translations.js

@@ -22,6 +22,12 @@ import { confirm, intro, log, outro, taskLog } from "@clack/prompts"
  * }} Languages
  */
 
+/**
+ * @typedef {{
+ *   build?: boolean
+ * }} CheckTranslationsOptions
+ */
+
 const lightnetCachePath = resolve(cwd(), "node_modules", ".cache", "lightnet")
 
 /** @type {{type:Translation["type"], title:string, action:string}[]} */
@@ -44,14 +50,18 @@ const translationSources = [
   },
 ]
 
-export async function checkTranslations() {
+/**
+ * @param {CheckTranslationsOptions} [options]
+ */
+export async function checkTranslations(options = {}) {
   intro("check-translations")
 
-  const buildAvailable = await runBuild()
+  const buildAvailable = await runBuild(options.build)
   if (!buildAvailable) {
     outro("Build failed. 🚧")
     return false
   }
+
   const translations = await readTranslations()
   const languages = await readLanguages()
   if (!translations || !languages || translations.length === 0) {
@@ -84,12 +94,19 @@ export async function checkTranslations() {
   return false
 }
 
-async function runBuild() {
-  const shouldRunBuild = await confirm({
-    message:
-      "Run pnpm build now? Command requires an up-to-date dist/ directory.",
-    initialValue: false,
-  })
+/**
+ *
+ * @param {boolean|undefined} build
+ * @returns
+ */
+async function runBuild(build) {
+  const shouldRunBuild =
+    build ??
+    (await confirm({
+      message:
+        "Run pnpm build now? Command requires an up-to-date dist/ directory.",
+      initialValue: false,
+    }))
   if (!shouldRunBuild) {
     return true
   }

package/src/index.js

@@ -21,8 +21,10 @@ program
 program
   .command("check-translations")
   .description("check if last build has been missing any translations")
-  .action(async () => {
-    const checkSuccessful = await checkTranslations()
+  .option("--build", "run pnpm build before checking translations")
+  .option("--no-build", "skip the build prompt and use the latest dist/ output")
+  .action(async (options) => {
+    const checkSuccessful = await checkTranslations(options)
     commandExitCode = checkSuccessful ? 0 : 1
   })
 
@@ -32,7 +34,7 @@ program
     "check for missing and orphaned content files and thumbnails in a LightNet site",
   )
   .option("--fix", "remove orphaned files")
-  .option("--yes", "skip deletion confirmation when used with --fix")
+  .option("--no-confirm", "skip deletion confirmation when used with --fix")
   .option(
     "--r2",
     "validate remote content files in Cloudflare R2 instead of public/files",

package/src/support/cli-config.js

@@ -6,9 +6,7 @@ import { resolve } from "node:path"
 import { CliError } from "./cli-error.js"
 import { pathExists } from "./filesystem.js"
 
-export const cliConfigFileName = ".lightnet-cli.config.json"
-
-const gitIgnoreEntry = `${cliConfigFileName}\n`
+export const cliConfigFileName = "lightnet-cli.json"
 
 /**
  * @param {string} cwd
@@ -36,22 +34,4 @@ export async function writeCliConfig(cwd, config) {
     `${JSON.stringify(config, null, 2)}\n`,
     "utf8",
   )
-  await ensureGitignoreContainsConfig(cwd)
-}
-
-/**
- * @param {string} cwd
- */
-async function ensureGitignoreContainsConfig(cwd) {
-  const filePath = resolve(cwd, ".gitignore")
-  const contents = (await pathExists(filePath))
-    ? await readFile(filePath, "utf8")
-    : ""
-  if (contents.includes(cliConfigFileName)) {
-    return
-  }
-  const next = contents
-    ? `${contents}${contents.endsWith("\n") ? "" : "\n"}${gitIgnoreEntry}`
-    : gitIgnoreEntry
-  await writeFile(filePath, next, "utf8")
 }