@lystech/core 3.0.87 → 3.0.88

package/bin/check-pwa.cjs

@@ -19,9 +19,13 @@ function checkManifest(autoCreate = false) {
     findFileUpwards("public/manifest.json"),
     findFileUpwards("manifest.json"),
   ].filter(Boolean);
+  // Nom déduit du package.json de la BOUTIQUE : un manifeste créé automatiquement
+  // baptisait toute nouvelle boutique « Lystech Core » sur l'écran d'accueil des
+  // clients (le manifeste fait foi pour iOS 16.4+ et Android).
+  const fallbackName = resolveAppName(null);
   const defaultManifest = {
-    name: "Lystech Core",
-    short_name: "Lystech",
+    name: fallbackName,
+    short_name: fallbackName,
     start_url: ".",
     display: "standalone",
     background_color: "#ffffff",
@@ -184,7 +188,34 @@ function checkServiceWorker(autoCreate = false) {
   return false;
 }
 
-function checkIndexHtmlForManifestAndAppleTags(autoCreate = false) {
+/**
+ * Nom de l'app tel que LA BOUTIQUE le déclare. Le script tourne dans le CI de chaque
+ * boutique : il ne doit jamais y injecter le nom de la plateforme. Priorité au
+ * manifeste (qui fait foi pour iOS 16.4+), repli sur package.json.
+ */
+function resolveAppName(manifestPath) {
+  try {
+    if (manifestPath && fs.existsSync(manifestPath)) {
+      const manifest = JSON.parse(fs.readFileSync(manifestPath, "utf8"));
+      const name = manifest?.short_name || manifest?.name;
+      if (name) return String(name);
+    }
+  } catch {
+    /* manifeste illisible -> repli */
+  }
+  try {
+    const pkgPath = findFileUpwards("package.json");
+    if (pkgPath) {
+      const pkg = JSON.parse(fs.readFileSync(pkgPath, "utf8"));
+      if (pkg?.name) return String(pkg.name);
+    }
+  } catch {
+    /* ignore */
+  }
+  return "Lystech Core";
+}
+
+function checkIndexHtmlForManifestAndAppleTags(autoCreate = false, appName = "Lystech Core") {
   const indexPaths = [
     findFileUpwards("public/index.html"),
     findFileUpwards("index.html"),
@@ -198,16 +229,37 @@ function checkIndexHtmlForManifestAndAppleTags(autoCreate = false) {
   const indexPath = indexPaths[0];
   let lines = fs.readFileSync(indexPath, "utf8").split(/\r?\n/);
   let ok = true;
-  const requiredLines = [
-    '<link rel="manifest" href="/manifest.json" />',
-    '<meta name="apple-mobile-web-app-capable" content="yes" />',
-    '<meta name="apple-mobile-web-app-status-bar-style" content="default" />',
-    '<meta name="apple-mobile-web-app-title" content="Lystech Core" />',
-    '<link rel="apple-touch-icon" href="/logo.png" />',
+  // Détection par ATTRIBUT, pas par chaîne exacte. Avant, on cherchait la valeur
+  // littérale ('content="Lystech Core"') : une boutique brandée autrement ne matchait
+  // jamais, et --fix lui ajoutait une SECONDE balise contradictoire à chaque fois.
+  // On ne touche donc à rien si la balise existe déjà, quelle que soit sa valeur.
+  const requiredTags = [
+    {
+      detect: /rel=["']manifest["']/i,
+      tag: '<link rel="manifest" href="/manifest.json" />',
+    },
+    {
+      detect: /name=["']apple-mobile-web-app-capable["']/i,
+      tag: '<meta name="apple-mobile-web-app-capable" content="yes" />',
+    },
+    {
+      detect: /name=["']apple-mobile-web-app-status-bar-style["']/i,
+      tag: '<meta name="apple-mobile-web-app-status-bar-style" content="default" />',
+    },
+    {
+      detect: /name=["']apple-mobile-web-app-title["']/i,
+      // Nom DÉRIVÉ de la boutique : ce titre s'affiche sous l'icône iOS. Coder
+      // « Lystech Core » en dur y faisait fuiter le nom de la plateforme.
+      tag: `<meta name="apple-mobile-web-app-title" content="${appName}" />`,
+    },
+    {
+      detect: /rel=["']apple-touch-icon["']/i,
+      tag: '<link rel="apple-touch-icon" href="/logo.png" />',
+    },
   ];
   let changed = false;
-  for (const tag of requiredLines) {
-    if (lines.some((l) => l.includes(tag.replace(/\s*\/?>$/, "")))) {
+  for (const { detect, tag } of requiredTags) {
+    if (lines.some((l) => detect.test(l))) {
       console.log(`✅ ${tag} trouvé dans`, indexPath);
     } else {
       ok = false;
@@ -237,8 +289,10 @@ function main() {
     process.argv.includes("--fix") || process.argv.includes("--create");
   const manifest = checkManifest(autoCreate);
   const serviceWorker = checkServiceWorker(autoCreate);
-  const indexHasManifestAndApple =
-    checkIndexHtmlForManifestAndAppleTags(autoCreate);
+  const indexHasManifestAndApple = checkIndexHtmlForManifestAndAppleTags(
+    autoCreate,
+    resolveAppName(manifest),
+  );
 
   // Ajout automatique du script check-pwa dans package.json si absent
   const pkgPath = findFileUpwards("package.json");