@wix/himalaya-cli 0.1213.0 → 0.1214.0

package/dist/authoring-runtime/node_modules/@wix/himalaya/dist/_types/core/dev-server/src/l10n-strings.d.ts

@@ -47,7 +47,20 @@ export interface L10nStringFinding {
  * Bindings are skipped (they render worker state, which this cannot see — use
  * `collectBoundServicePaths` + the worker's own state for that half).
  */
-export declare function collectDescriptorStrings(descriptor: unknown): L10nStringFinding[];
+export declare function collectDescriptorStrings(descriptor: unknown, options?: {
+    /**
+     * Values the app has DECLARED as copy (its `strings/<locale>.json` table), for which
+     * the "has a letter" rule of `isUserVisibleCopy` is waived — and only that rule: the
+     * binding/id/url/hex/length filters still apply. Exists for letterless copy — a phone
+     * or number format such as "+1 555 0100" has no letter, so the heuristic calls it
+     * noise, and the extractor dropped `reserve.phonePlaceholder` (#3320). Only exact
+     * table values qualify, so an arbitrary "4.8" in sample data stays out.
+     *
+     * Passed by the extractor only. The `l10n` validate pass does not pass it yet, so it
+     * still treats letterless copy as non-copy (as it always has).
+     */
+    copyValues?: ReadonlySet<string>;
+}): L10nStringFinding[];
 /**
  * The `$service:` bindings sitting in a user-visible prop — i.e. the state paths whose
  * value is PAINTED as copy.

package/dist/cli.mjs

@@ -6818,7 +6818,6 @@ function walk(node, registry, path2, out2) {
 var SYSTEM_FONT_FAMILY, TONAL_ELEVATION_ALPHA, COLOR_TOKEN_PROPS, TYPOGRAPHY_TOKEN_PROPS;
 var init_tokens = __esm({
   "../../core/ts/src/tokens.ts"() {
-    "use strict";
     SYSTEM_FONT_FAMILY = "$system";
     TONAL_ELEVATION_ALPHA = [0, 0.05, 0.08, 0.11, 0.12, 0.14];
     COLOR_TOKEN_PROPS = {
@@ -6869,7 +6868,6 @@ function resolveTranslation(value, catalog, sourceCatalog) {
 var BINDING_SIGILS, BINDING_SIGIL_RE, TRANSLATION_PREFIX, TRANSLATION_KEY_RE;
 var init_bindings = __esm({
   "../../core/ts/src/bindings.ts"() {
-    "use strict";
     BINDING_SIGILS = ["service", "flag", "item", "t"];
     BINDING_SIGIL_RE = new RegExp(
       `^\\$(?:${BINDING_SIGILS.filter((s) => s !== "t").join("|")})\\b|^\\$t:`
@@ -7988,14 +7986,19 @@ var init_token_suggest = __esm({
 
 // ../../core/dev-server/src/l10n-strings.ts
 function isUserVisibleCopy(s) {
+  return isCopyShaped(s) && /[A-Za-z֐-׿؀-ۿ]/.test(s);
+}
+function isCopyShaped(s) {
   if (typeof s !== "string" || s.length === 0 || s.length > 200) return false;
   if (s.startsWith("$") || s.startsWith("_")) return false;
   if (/^https?:\/\//.test(s)) return false;
   if (/^#[0-9A-Fa-f]+$/.test(s)) return false;
   if (!s.includes(" ") && /^[a-z0-9_\-./]+$/.test(s)) return false;
-  return /[A-Za-z֐-׿؀-ۿ]/.test(s);
+  return true;
 }
-function collectDescriptorStrings(descriptor) {
+function collectDescriptorStrings(descriptor, options = {}) {
+  const { copyValues } = options;
+  const isCopy2 = (v) => isUserVisibleCopy(v) || copyValues !== void 0 && copyValues.has(v) && isCopyShaped(v);
   const out2 = [];
   const seen = /* @__PURE__ */ new Set();
   const push = (f) => {
@@ -8018,7 +8021,7 @@ function collectDescriptorStrings(descriptor) {
       const mapped = STRING_FIELDS_BY_TYPE[type] ?? [];
       for (const [prop, v] of Object.entries(props)) {
         if (typeof v !== "string" || BINDING_SIGIL.test(v)) continue;
-        if (!isUserVisibleCopy(v)) continue;
+        if (!isCopy2(v)) continue;
         const at2 = `${path2}.body.props.${prop}`;
         if (mapped.includes(prop) || SECONDARY_FIELDS.has(prop)) {
           push({ value: v, bucket: "reachable", type, prop, path: at2 });
@@ -8027,7 +8030,7 @@ function collectDescriptorStrings(descriptor) {
         }
       }
       const templ = props.template;
-      if (templ && typeof templ.template === "string" && isUserVisibleCopy(templ.template)) {
+      if (templ && typeof templ.template === "string" && isCopy2(templ.template)) {
         push({
           value: templ.template,
           bucket: "reachable",
@@ -8039,7 +8042,7 @@ function collectDescriptorStrings(descriptor) {
     }
     for (const [k, v] of Object.entries(obj3)) {
       if (k === "body") continue;
-      if (typeof v === "string" && SECONDARY_FIELDS.has(k) && !BINDING_SIGIL.test(v) && isUserVisibleCopy(v)) {
+      if (typeof v === "string" && SECONDARY_FIELDS.has(k) && !BINDING_SIGIL.test(v) && isCopy2(v)) {
         push({ value: v, bucket: "reachable", type, prop: k, path: `${path2}.${k}` });
       }
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/himalaya-cli",
-  "version": "0.1213.0",
+  "version": "0.1214.0",
   "type": "module",
   "description": "himi — the Himalaya authoring CLI. Build a Tier-5 content package on your own machine, push it to the production serve as a draft, preview it on web + Himi Player, and read worker logs back — the remote dev loop. Wraps @himalaya/serve-cli for transport.",
   "keywords": [
@@ -67,5 +67,5 @@
       "artifactId": "himalaya-cli"
     }
   },
-  "falconPackageHash": "85a85909eb5f201a49f5414ffcc4d36dbf9a3dcf7acca4a0b3eec160"
+  "falconPackageHash": "4665b1c60527def192c75337a09421d697bbdd3949d74c93be76a087"
 }