@proveanything/smartlinks 2.0.50 → 2.0.52

package/openapi.yaml

@@ -19218,16 +19218,113 @@ components:
     AppDataHandle:
       type: object
       properties: {}
+    AppRecordClaimResult:
+      type: object
+      properties:
+        record:
+          type: object
+          additionalProperties: true
+        created:
+          type: boolean
+        conflict:
+          type: string
+          enum:
+            - singleton
+            - customId
+      required:
+        - record
+        - created
+    ServerFunctionCounters:
+      type: object
+      properties: {}
+    ResolvedTag:
+      type: object
+      properties:
+        id:
+          type: string
+        claimSetId:
+          type: string
+        code:
+          type: string
+      required:
+        - id
+        - claimSetId
+        - code
+    ContactUpsertDetails:
+      type: object
+      properties:
+        email:
+          type: string
+        phone:
+          type: string
+        firstName:
+          type: string
+        lastName:
+          type: string
+        displayName:
+          type: string
+        company:
+          type: string
+        locale:
+          type: string
+        timezone:
+          type: string
+        customFields:
+          type: object
+          additionalProperties:
+            type: object
+            additionalProperties: true
+    ServerFunctionInteractionEvent:
+      type: object
+      properties:
+        interactionId:
+          type: string
+        contactId:
+          type: string
+        outcome:
+          type: string
+        metadata:
+          type: object
+          additionalProperties: true
+        productId:
+          type: string
+        proofId:
+          type: string
+        variantId:
+          type: string
+        batchId:
+          type: string
+        eventType:
+          type: string
+      required:
+        - interactionId
     ServerFunctionSl:
       type: object
       properties:
-        appRecords: {}
+        appRecords:
+          type: object
+          additionalProperties: true
+        counters:
+          $ref: "#/components/schemas/ServerFunctionCounters"
+        tags:
+          type: object
+          additionalProperties: true
+        contacts:
+          type: object
+          additionalProperties: true
+        interactions:
+          type: object
+          additionalProperties: true
         products: {}
         attestations: {}
         appData:
           $ref: "#/components/schemas/AppDataHandle"
       required:
         - appRecords
+        - counters
+        - tags
+        - contacts
+        - interactions
         - products
         - attestations
         - appData
@@ -23658,14 +23755,22 @@ components:
       properties:
         email:
           type: string
+        templateId:
+          type: string
         topics:
           type: array
           items:
             type: string
         firstName:
           type: string
+        fields:
+          type: object
+          additionalProperties: true
+        appId:
+          type: string
       required:
         - email
+        - templateId
     CommsSignupResponse:
       type: object
       properties:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@proveanything/smartlinks",
-  "version": "2.0.50",
+  "version": "2.0.52",
   "description": "Official JavaScript/TypeScript SDK for the Smartlinks API",
   "type": "module",
   "main": "dist/index.js",

package/scripts/doctor.mjs

@@ -168,16 +168,26 @@ if (meta.cssBaseline) {
     for (const c of ['sl-mobile-admin']) baseSet.add(c);
     // Class usage lives in compiled markup strings — present in every bundle regardless
     // of module format, so scan both umd + esm across all surfaces.
+    // `sl-…` NOT preceded by a word char or hyphen: a class name, never a CSS custom property
+    // (`var(--sl-color-accent)`, `--sl-widget-accent`) — those are theme tokens, not classes.
+    // Classes the app defines itself (`.sl-foo {…}` in its own CSS) aren't missing either.
     const used = new Set();
+    const defined = new Set();
     for (const [, block] of surfaces) {
-      for (const p of [block?.files?.js?.umd, block?.files?.js?.esm]) {
-        if (!p) continue;
+      for (const p of [block?.files?.js?.umd, block?.files?.js?.esm, block?.files?.css]) {
+        if (!p || typeof p !== 'string') continue;
         const bp = resolveBundle(p);
         if (!bp) continue;
-        for (const m of readFileSync(bp, 'utf8').matchAll(/\bsl-[a-z0-9-]+/g)) used.add(m[0]);
+        const text = readFileSync(bp, 'utf8');
+        for (const m of text.matchAll(/(?<![\w-])sl-[a-z0-9-]+/g)) used.add(m[0]);
+        for (const m of text.matchAll(/\.(sl-[a-z0-9-]+)/g)) defined.add(m[1]);
       }
     }
-    const unknown = [...used].filter((c) => !baseSet.has(c)).sort();
+    const ownSl = [...defined].filter((c) => !baseSet.has(c)).sort();
+    if (ownSl.length) {
+      warnings.push(`the app's own classes use the platform's sl- prefix: ${ownSl.join(', ')} — rename them (e.g. drop the "sl-") so they can't collide with a future baseline class.`);
+    }
+    const unknown = [...used].filter((c) => !baseSet.has(c) && !defined.has(c)).sort();
     if (used.size === 0) {
       console.log(`${DIM}cssBaseline (${baseline.version}) — declared; no sl-* classes found in the bundles.${RESET}`);
     } else if (unknown.length === 0) {
@@ -238,6 +248,51 @@ if (meta.respectsHostTheme) {
   }
 }
 
+// ---- containers: a router of their own must be isolated from the host's ------------------------
+// Since R5, react-router-dom is SHARED with the host, so a container that wraps itself in a
+// MemoryRouter/HashRouter/BrowserRouter inside the host's router throws "You cannot render a <Router>
+// inside another <Router>" — unless it resets the parent's router context first (docs/containers.md →
+// "Pages that read their context from the URL"). Heuristic on the built bundle names; WARN-only.
+{
+  const block = manifest.containers;
+  for (const p of [block?.files?.js?.esm, block?.files?.js?.umd]) {
+    if (!p) continue;
+    const bp = resolveBundle(p);
+    if (!bp) continue;
+    const text = readFileSync(bp, 'utf8');
+    const routers = ['MemoryRouter', 'HashRouter', 'BrowserRouter'].filter((r) => new RegExp(`\\b${r}\\b`).test(text));
+    if (!routers.length) { console.log(`${GREEN}✓${RESET} containers ${DIM}— no router of their own (the host's is used)${RESET}`); break; }
+    if (/UNSAFE_LocationContext/.test(text)) { console.log(`${GREEN}✓${RESET} containers ${DIM}— own ${routers.join('/')} isolated from the host's router${RESET}`); break; }
+    console.log(`${YELLOW}⚠${RESET} containers — ${routers.join('/')} without isolating it from the host's router`);
+    warnings.push(`containers bundle renders its own ${routers.join('/')} but never resets the host's router context — inside an R5 host that throws "You cannot render a <Router> inside another <Router>". Wrap it in UNSAFE_LocationContext.Provider + UNSAFE_NavigationContext.Provider value={null} (docs/containers.md), or drop the router and use the host's.`);
+    break;
+  }
+}
+
+// ---- package.json scripts point at files that exist ------------------------------------------
+// An upgrade that copies a script LINE but not the script FILE breaks every build at that step
+// (e.g. `postbuild: node scripts/smartlinks-deploy.mjs` with no scripts/smartlinks-deploy.mjs).
+// WARN — the doctor runs inside the build, and a skipped hook (Forge builds skip pre/post) is harmless.
+try {
+  const pkg = JSON.parse(readFileSync(join(appDir, 'package.json'), 'utf8'));
+  const missing = [];
+  for (const [name, cmd] of Object.entries(pkg.scripts || {})) {
+    if (typeof cmd !== 'string') continue;
+    for (const m of cmd.matchAll(/(?:^|[;&|]\s*|\s)(?:node|tsx|ts-node|bash|sh)\s+(?:-{1,2}[\w-]+(?:=\S+)?\s+)*((?:\.{1,2}\/)?[\w./-]+\.(?:mjs|cjs|js|ts|sh))\b/g)) {
+      if (!existsSync(join(appDir, m[1]))) missing.push(`"${name}" runs ${m[1]}`);
+    }
+  }
+  if (missing.length) {
+    console.log(`${YELLOW}⚠${RESET} package.json scripts — ${missing.length} point${missing.length === 1 ? 's' : ''} at a file that doesn't exist:`);
+    for (const x of missing) console.log(`    ${YELLOW}${x}${RESET}`);
+    warnings.push(`package.json ${missing.join('; ')} — but the file is missing. Add the file, or remove the script (a pre/post hook like postbuild fails every \`npm run build\` without it).`);
+  } else {
+    console.log(`${GREEN}✓${RESET} package.json scripts ${DIM}— every script file they run exists${RESET}`);
+  }
+} catch {
+  /* no package.json — not an npm app */
+}
+
 console.log('');
 for (const w of warnings) console.log(`${YELLOW}⚠ ${w}${RESET}`);