Merge pull request #1389 from amzn/prettier-dep

fix: prettier direct dependency & upgrade path-unified

.changeset/rich-cars-unite.md

@@ -0,0 +1,5 @@
+---
+'style-dictionary': patch
+---
+
+Move prettier to dependencies since style-dictionary isn't really a prettier plugin and a direct dependency seems more accurate here.

__tests__/common/transforms.test.js

@@ -11,7 +11,6 @@
  * and limitations under the License.
  */
 import { expect } from 'chai';
-import { join } from 'path-unified';
 import Color from 'tinycolor2';
 import transforms, { isColor } from '../../lib/common/transforms.js';
 
@@ -1391,7 +1390,7 @@ describe('common', () => {
           {},
           {},
         );
-        expect(value).to.equal(join('foo.json'));
+        expect(value).to.equal('foo.json');
       });
     });
   });

docs/src/components/sd-playground.ts

@@ -1,7 +1,7 @@
 import StyleDictionary from 'style-dictionary';
 import { Volume } from '@bundled-es-modules/memfs';
 import { LitElement, html, css } from 'lit';
-import { posix as path } from 'path-unified';
+import { extname, join } from 'path-unified/posix';
 import '@shoelace-style/shoelace/dist/components/radio-button/radio-button.js';
 import '@shoelace-style/shoelace/dist/components/radio-group/radio-group.js';
 import '@shoelace-style/shoelace/dist/components/select/select.js';
@@ -377,7 +377,7 @@ class SdPlayground extends LitElement {
   changeOutputs(filePath: string, changeCurrentFile = false) {
     this.output = JSON.stringify({
       value: this.volume.readFileSync(filePath, 'utf-8'),
-      lang: path.extname(filePath).replace(/^\./g, ''),
+      lang: extname(filePath).replace(/^\./g, ''),
     });
     // call the setter so the editor value updates to the new output selection
     if (this.currentFile === 'output' || changeCurrentFile) {
@@ -389,7 +389,7 @@ class SdPlayground extends LitElement {
   traverseDir(dir = '/', files: string[] = []) {
     let _files = files;
     (this.volume.readdirSync(dir) as string[]).forEach((file: string) => {
-      const fullPath = path.join(dir, file);
+      const fullPath = join(dir, file);
       if (this.volume.lstatSync(fullPath).isDirectory()) {
         _files = [..._files, ...this.traverseDir(fullPath, _files)];
       } else {

lib/StyleDictionary.js

@@ -12,10 +12,9 @@
  */
 
 import JSON5 from 'json5';
-import { extname } from 'path-unified';
+import { extname, dirname } from 'path-unified';
 import { fs } from 'style-dictionary/fs';
 import chalk from 'chalk';
-import { dirname } from 'path-unified';
 
 import combineJSON from './utils/combineJSON.js';
 import deepExtend from './utils/deepExtend.js';

lib/resolve.js

@@ -1,4 +1,5 @@
-import { posix, win32 } from 'path-unified';
+import { resolve as resolvePosix } from 'path-unified/posix';
+import { resolve as resolveWin32 } from 'path-unified/win32';
 import { isNode } from './utils/isNode.js';
 
 /**
@@ -13,7 +14,7 @@ export const resolve = (path, customVolumeUsed = false) => {
     return path;
   }
   if (isNode && process?.platform === 'win32') {
-    return win32.resolve(path);
+    return resolveWin32(path);
   }
-  return posix.resolve(path);
+  return resolvePosix(path);
 };

package.json

@@ -112,7 +112,8 @@
     "is-plain-obj": "^4.1.0",
     "json5": "^2.2.2",
     "patch-package": "^8.0.0",
-    "path-unified": "^0.1.0",
+    "path-unified": "^0.2.0",
+    "prettier": "^3.3.3",
     "tinycolor2": "^1.6.0"
   },
   "devDependencies": {
@@ -162,8 +163,5 @@
     "unist-util-visit": "^5.0.0",
     "uuid": "^9.0.1",
     "yaml": "^2.3.4"
-  },
-  "peerDependencies": {
-    "prettier": "3.x"
   }
 }