@wix/origami-site-runtime 1.0.11 → 1.0.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@wix/origami-site-runtime",
-  "version": "1.0.11",
+  "version": "1.0.12",
   "description": "Vite development and build helpers for wix-import static sites",
   "main": "./src/index.js",
   "files": [
@@ -65,5 +65,5 @@
     "./default-error-404": "./src/default-error-404.jsx"
   },
   "type": "module",
-  "falconPackageHash": "e22a7402b3249122540518d623a21731e722614ef9110bb8cce0eef7"
+  "falconPackageHash": "2e7b7eab8ba52e6afa27dfa363e20a634506137bd61591f8bf6015a3"
 }

package/src/cssPlugins/presets.js

@@ -4,6 +4,9 @@
 // `--preset` off the presets-wrapper classes it renders (see the origami-component-usage skill), so each declaration
 // moves there, as `--preset` in a rule of its own: `.{id}-presets-wrapper`, or `{selector} .{key}-presets-wrapper` so
 // an inner element's preset reaches that instance only. A rule that does not start at a component is left as authored.
+//
+// A preset on a component selector that can't be mapped is reported as a warning and left as authored rather than failing the
+// build: exported sites still carry presets in the older shape, and they must keep building until they are migrated.
 
 import postcss from 'postcss'
 
@@ -24,7 +27,7 @@ const target = (selector, key) => {
 export const presetsPlugin = {
     name: 'presets',
     order: 13,
-    transform(root) {
+    transform(root, context) {
         /** @type {import('postcss').Declaration[]} */
         const declarations = []
         root.walkDecls(PRESET, declaration => {
@@ -46,10 +49,13 @@ export const presetsPlugin = {
             const targets = selectors.map(selector => target(selector, PRESET.exec(declaration.prop)?.[1]))
             if (targets.every(selector => selector === undefined)) continue
             if (!targets.every(Boolean)) {
-                throw new Error(
-                    `${declaration.prop} on "${selectors.join(', ')}": set --preset on #id or .id alone, ` +
-                        `and an inner element's as --preset-{key} on the rule of the element it sits in`
+                context.warn?.(
+                    declaration,
+                    `presets: ${context.relativePath ?? context.filePath}: ` +
+                        `${declaration.prop} on "${selectors.join(', ')}": set --preset on #id or .id alone, ` +
+                        `and an inner element's as --preset-{key} on the rule of the element it sits in. Left as authored.`
                 )
+                continue
             }
             declaration.prop = '--preset'
             const presetRule = postcss.rule({ selector: targets.join(', ') })

package/src/transform/css/postcssPlugin.js

@@ -46,8 +46,11 @@ export const createCssPostcssPlugin = ({ selectPlugins, contextFor, failures })
             declared.add(file)
             result.messages.push({ type: 'dependency', plugin: 'wix-import-css-transform', file, parent: from })
         }
+        // A plugin reports a problem that need not stop the build as a PostCSS warning, which Vite prints with the
+        // node's code frame.
+        const warn = (node, text) => node.warn(result, text)
         try {
-            runCssPlugins({ plugins, root, context: { ...fileContext, addDependency } })
+            runCssPlugins({ plugins, root, context: { ...fileContext, addDependency, warn } })
             failures?.delete(from)
         } catch (error) {
             failures?.set(from, declared)

package/src/transform/types.js

@@ -36,10 +36,12 @@
  * }} CssTransformPlugin
  *
  * @typedef {TransformContext & {
- *     addDependency?: (file: string) => void
+ *     addDependency?: (file: string) => void,
+ *     warn?: (node: import('postcss').Node, text: string) => void
  * }} CssTransformContext
  * `addDependency` declares a file the plugin read besides the stylesheet, so the bundler re-runs the
- * stylesheet when it changes. Absent when no bundler drives the run.
+ * stylesheet when it changes. `warn` reports a problem at `node` without failing the run, as a PostCSS
+ * warning on the stylesheet's result. Both are absent when no bundler drives the run.
  */
 
 export {}