Fix @cobalt-ui/plugin-sass’ listModes

terrazzoapp · May 23, 2022 · cd82302 · cd82302
1 parent 5d458e8
commit cd82302
Show file tree

Hide file tree

Showing 5 changed files with 447 additions and 264 deletions.
diff --git a/.changeset/green-icons-kneel.md b/.changeset/green-icons-kneel.md
@@ -0,0 +1,5 @@
+---
+'@cobalt-ui/plugin-sass': patch
+---
+
+Fix listModes bug
diff --git a/package.json b/package.json
@@ -44,17 +44,17 @@
   },
   "devDependencies": {
     "@changesets/cli": "^2.22.0",
-    "@typescript-eslint/eslint-plugin": "^5.12.1",
-    "@typescript-eslint/parser": "^5.12.1",
+    "@typescript-eslint/eslint-plugin": "^5.25.0",
+    "@typescript-eslint/parser": "^5.25.0",
     "del-cli": "^4.0.1",
-    "esbuild": "^0.14.31",
-    "eslint": "^8.12.0",
+    "esbuild": "^0.14.39",
+    "eslint": "^8.16.0",
     "eslint-config-prettier": "^8.5.0",
     "eslint-plugin-prettier": "^4.0.0",
     "npm-run-all": "^4.1.5",
-    "pnpm": "^6.32.4",
+    "pnpm": "^7.1.3",
     "prettier": "^2.6.2",
-    "typescript": "^4.6.3"
+    "typescript": "^4.6.4"
   },
   "volta": {
     "node": "16.14.2"

diff --git a/packages/plugin-sass/src/index.ts b/packages/plugin-sass/src/index.ts
@@ -97,7 +97,7 @@ ${cbClose}`
     @error "No token named \\"#{$tokenName}\\""${semi}
   ${cbClose}
   $_modes: ();
-  @each $k in map.get($${VAR_TOKENS})${cbOpen}
+  @each $k in map.get($${VAR_TOKENS}, $tokenName)${cbOpen}
     @if $k != "default"${cbOpen}
       $_modes: list.append($_modes, $k);
     ${cbClose}

diff --git a/packages/plugin-sass/test/fixtures/basic/want.scss b/packages/plugin-sass/test/fixtures/basic/want.scss
@@ -101,7 +101,7 @@ $__token-typography-mixins: (
     @error "No token named \"#{$tokenName}\"";
   }
   $_modes: ();
-  @each $k in map.get($__token-values) {
+  @each $k in map.get($__token-values, $tokenName) {
     @if $k != "default" {
       $_modes: list.append($_modes, $k);
     }