@lightnet/cli 4.8.2 → 4.9.1

package/CHANGELOG.md

@@ -1,5 +1,21 @@
 # @lightnet/cli
 
+## 4.9.1
+
+### Patch Changes
+
+- [#471](https://github.com/LightNetDev/LightNet/pull/471) [`667909c`](https://github.com/LightNetDev/LightNet/commit/667909c629835b0bf7f697c6d0a92c1ad92a733e) - Check media links using a range GET and verify the first body byte can be read. Retry unsatisfiable ranges with a plain GET to support empty resources.
+
+## 4.9.0
+
+### Minor Changes
+
+- [#469](https://github.com/LightNetDev/LightNet/pull/469) [`a0961c4`](https://github.com/LightNetDev/LightNet/commit/a0961c40fdc30a93d3d5c814343f9c12ec26b2ef) - Add repeatable --exclude to check-translations for space-separated exact keys to ignore during completeness checks.
+
+### Patch Changes
+
+- [#469](https://github.com/LightNetDev/LightNet/pull/469) [`a0961c4`](https://github.com/LightNetDev/LightNet/commit/a0961c40fdc30a93d3d5c814343f9c12ec26b2ef) - Warn about user-defined ln.* translation overrides that no longer match a built-in English key. Include source filenames and check unused keys without failing check-translations for stale overrides alone.
+
 ## 4.8.2
 
 ### Patch Changes

package/README.md

@@ -8,6 +8,19 @@ Command-line tools for working with LightNet sites. The CLI currently includes t
 pnpm add --D @lightnet/cli
 ```
 
+## Exclude translation keys
+
+Pass a space-separated list of exact keys to skip their completeness checks:
+
+```bash
+pnpm exec lightnet check-translations --exclude ln.details.edit custom.title
+```
+
+You can repeat `--exclude` to add more keys. Matching is case-sensitive and does
+not support wildcards. Exclusions apply to built-in keys, user keys, and inline
+translation-map paths. Stale override warnings are still reported for excluded
+keys. Use `--build` to refresh the cache or `--no-build` to check the last build.
+
 ## LightNet CLI reference
 
 See command-line tools reference documentation at:

package/package.json

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

package/src/check-links.js

@@ -18,9 +18,9 @@ import { cancelPrompt } from "./support/prompt-cancel.js"
 const mediaDir = "src/content/media"
 const defaultConcurrency = 24
 const defaultTimeoutMs = 10_000
-const headFallbackStatuses = new Set([405, 501])
 const requestHeaders = {
   accept: "*/*",
+  "accept-encoding": "identity",
   "user-agent": "@lightnet/cli link checker",
 }
 
@@ -359,22 +359,17 @@ function resolveUrl(url, siteUrl) {
  * @returns {Promise<LinkCheckResult>}
  */
 async function checkLink(url, options) {
-  const headResult = await fetchStatus(url, {
+  const rangeResult = await fetchStatus(url, {
     ...options,
-    headers: requestHeaders,
-    method: "HEAD",
+    headers: { ...requestHeaders, range: "bytes=0-0" },
   })
-  if (headResult.ok || !headFallbackStatuses.has(headResult.status ?? 0)) {
-    return headResult
+  if (rangeResult.status !== 416) {
+    return rangeResult
   }
+  // An empty resource has no first byte; retry without the range constraint.
   return fetchStatus(url, {
     ...options,
-    headers: {
-      ...requestHeaders,
-      "accept-encoding": "identity",
-      range: "bytes=0-0",
-    },
-    method: "GET",
+    headers: requestHeaders,
   })
 }
 
@@ -432,7 +427,6 @@ function formatReferenceSources(sources) {
  * @param {{
  *   fetch: typeof fetch
  *   headers?: Record<string, string>
- *   method: "GET"|"HEAD"
  *   timeoutMs: number
  * }} options
  * @returns {Promise<LinkCheckResult>}
@@ -443,12 +437,42 @@ async function fetchStatus(url, options) {
   try {
     const response = await options.fetch(url, {
       headers: options.headers,
-      method: options.method,
+      method: "GET",
       redirect: "follow",
       signal: controller.signal,
     })
-    await response.body?.cancel()
     const ok = response.status >= 200 && response.status < 400
+    if (ok) {
+      const expectsBody =
+        response.status === 206 ||
+        Number(response.headers.get("content-length")) > 0
+      const reader = response.body?.getReader()
+      let receivedByte = false
+      try {
+        // A server may ignore Range and stream the full file. Stop as soon as
+        // the first nonempty chunk proves that the body can be read.
+        if (reader) {
+          while (true) {
+            const { done, value } = await reader.read()
+            if (done) break
+            if (value.byteLength > 0) {
+              receivedByte = true
+              break
+            }
+          }
+        }
+      } finally {
+        await reader?.cancel()
+      }
+      if (expectsBody && !receivedByte) {
+        return {
+          ok: false,
+          reason: "Response body ended before the first byte",
+        }
+      }
+    } else {
+      await response.body?.cancel()
+    }
     return { ok, status: response.status }
   } catch (error) {
     return {

package/src/check-translations.js

@@ -27,6 +27,7 @@ import { cancelPrompt } from "./support/prompt-cancel.js"
 /**
  * @typedef {{
  *   build?: boolean
+ *   exclude?: string[]
  * }} CheckTranslationsOptions
  */
 
@@ -88,10 +89,24 @@ export async function checkTranslations(options = {}, runtime = {}) {
 
   const translations = await readTranslations(lightnetCachePath, output)
   const languages = await readLanguages(lightnetCachePath, output)
+  const staleOverrides = await readStaleOverrides(lightnetCachePath)
+  if (staleOverrides.length > 0) {
+    output.warn("Stale LightNet translation overrides")
+    for (const { key, path } of staleOverrides.toSorted(
+      (a, b) => a.key.localeCompare(b.key) || a.path.localeCompare(b.path),
+    )) {
+      output.message(`• ${key} — ${path}`)
+    }
+    output.message(
+      "Action: Remove these overrides or rename custom keys without the reserved ln. prefix.",
+    )
+  }
   if (!translations || !languages || translations.length === 0) {
     return false
   }
+  const excludedKeys = new Set(options.exclude ?? [])
   const incompleteTranslations = translations
+    .filter((translation) => !excludedKeys.has(translation.key))
     .map((translation) => ({
       ...translation,
       missingLocales: getMissingLocales(translation, languages),
@@ -99,7 +114,11 @@ export async function checkTranslations(options = {}, runtime = {}) {
     .filter((translation) => translation.missingLocales.length > 0)
 
   if (incompleteTranslations.length === 0) {
-    output.outro("No issues found. 🎉")
+    output.outro(
+      staleOverrides.length > 0
+        ? "No missing translations. Stale overrides found."
+        : "No issues found. 🎉",
+    )
     return true
   }
 
@@ -236,6 +255,32 @@ function getMissingLocales(translation, languages) {
   return languages.locales.filter((locale) => !translation.values[locale])
 }
 
+/**
+ * @param {string} lightnetCachePath
+ * @returns {Promise<{key: string, path: string}[]>}
+ */
+async function readStaleOverrides(lightnetCachePath) {
+  try {
+    return JSON.parse(
+      await readFile(
+        resolve(lightnetCachePath, "stale-translation-overrides.json"),
+        "utf-8",
+      ),
+    )
+  } catch (error) {
+    // Older builds do not include stale override metadata.
+    if (
+      error &&
+      typeof error === "object" &&
+      "code" in error &&
+      error.code === "ENOENT"
+    ) {
+      return []
+    }
+    throw error
+  }
+}
+
 /**
  * @param {string} lightnetCachePath
  * @param {ReturnType<typeof createOutput>} output

package/src/index.js

@@ -25,6 +25,11 @@ program
   .description("check if last build has been missing any translations")
   .option("--build", "run pnpm build before checking translations")
   .option("--no-build", "skip the build prompt and use the latest dist/ output")
+  .option(
+    "--exclude <keys...>",
+    "exclude exact translation keys from completeness checks (repeatable)",
+    [],
+  )
   .action(async (options) => {
     try {
       const checkSuccessful = await checkTranslations(options)