Fix: namespace import referencing via value aliases

.changeset/tidy-papayas-clap.md

@@ -0,0 +1,7 @@
+---
+"@modular-css/processor": patch
+---
+
+Fix namespace import referencing via value aliases #964
+
+Updating internal `values` map after importing new values, which was missing previously.

packages/processor/plugins/values-import.js

@@ -116,6 +116,17 @@ module.exports = () => ({
                     throw rule.error(`Unknown @value type, unable to process`);
                 },
             },
+
+            RootExit() {
+                // Update any references that might've been affected by imports
+                for(const name of Object.keys(values)) {
+                    const { value } = values[name];
+
+                    if(value in values) {
+                        values[name] = values[value];
+                    }
+                }
+            },
         };
     },
 });

packages/processor/test/__snapshots__/values.test.js.snap

@@ -36,6 +36,15 @@ exports[`/processor.js values should support importing variables from a file 1`]
 "
 `;
 
+exports[`/processor.js values should support layers of namespaced value references 1`] = `
+"/* packages/processor/test/specimens/deep-namespace/c.css */
+/* packages/processor/test/specimens/deep-namespace/b.css */
+/* packages/processor/test/specimens/deep-namespace/a.css */
+.mc_a {
+    color: cyan;
+}"
+`;
+
 exports[`/processor.js values should support local values in value composition 1`] = `
 "/* packages/processor/test/specimens/folder/folder.css */
 .mc_folder { margin: 2px; }

packages/processor/test/specimens/deep-namespace/a.css

@@ -0,0 +1,5 @@
+@value b from "./b.css";
+
+.a {
+    color: b;
+}

packages/processor/test/specimens/deep-namespace/b.css

@@ -0,0 +1,3 @@
+@value * as values from "./c.css";
+
+@value b: values.c;

packages/processor/test/specimens/deep-namespace/c.css

@@ -0,0 +1 @@
+@value c: cyan;

packages/processor/test/values.test.js

@@ -195,5 +195,13 @@ describe("/processor.js", () => {
 
             expect(css).toMatchSnapshot();
         });
+
+        it("should support layers of namespaced value references", async () => {
+            await processor.file(require.resolve("./specimens/deep-namespace/a.css"));
+
+            const { css } = await processor.output();
+
+            expect(css).toMatchSnapshot();
+        });
     });
 });