Update fix_format.sh to use root yarn prettier scripts (#112)

* Update fix_format.sh to use root yarn prettier scripts

Update scripts/fix_format.sh to call root package.json yarn scripts.
Default CHECK_ONLY to true so format checks run by default.
Add prettier:check script to root and workspace package.json files.

TAG=agy
CONV=98e54060-798a-4573-8f25-3c54fe5bb3cf

* Restore correct default value.

Author: jgindin

bridge/package.json

@@ -26,6 +26,7 @@
     "clean": "rm -rf dist",
     "test": "NODE_ENV=test vitest run --coverage",
     "prettier": "prettier --ignore-path ../.prettierignore --write . | grep -v '(unchanged)'",
+    "prettier:check": "prettier --ignore-path ../.prettierignore --check .",
     "lint": "eslint ."
   },
   "devDependencies": {

package.json

@@ -12,6 +12,7 @@
     "test": "yarn workspaces foreach -A --exclude a2ui-composer-root run test",
     "build": "yarn workspaces foreach -A --topological-dev --exclude a2ui-composer-root run build",
     "prettier": "yarn workspaces foreach -A --exclude a2ui-composer-root run prettier",
+    "prettier:check": "yarn workspaces foreach -A --exclude a2ui-composer-root run prettier:check",
     "tsc": "yarn --cwd shell tsc --noEmit && yarn --cwd bridge tsc --project tsconfig.bridge.json --noEmit",
     "clean": "yarn workspaces foreach -A --exclude a2ui-composer-root run clean",
     "lint": "yarn workspaces foreach -A --exclude a2ui-composer-root run lint"

samples/lit-basic-catalog/package.json

@@ -10,6 +10,7 @@
     "preview": "vite preview",
     "test": "vitest run",
     "prettier": "prettier --ignore-path ../../.prettierignore --write . | grep -v '(unchanged)'",
+    "prettier:check": "prettier --ignore-path ../../.prettierignore --check .",
     "clean": "rm -rf dist node_modules/.vite",
     "lint": "eslint ."
   },

samples/ng-basic-catalog/package.json

@@ -7,6 +7,7 @@
     "build": "yarn lint && ng build",
     "test": "vitest run",
     "prettier": "prettier --ignore-path ../../.prettierignore --write . | grep -v '(unchanged)'",
+    "prettier:check": "prettier --ignore-path ../../.prettierignore --check .",
     "clean": "rm -rf dist .angular node_modules/.vite",
     "lint": "eslint ."
   },

samples/react-basic-catalog/package.json

@@ -10,6 +10,7 @@
     "preview": "vite preview",
     "test": "NODE_ENV=test vitest run",
     "prettier": "prettier --ignore-path ../../.prettierignore --write . | grep -v '(unchanged)'",
+    "prettier:check": "prettier --ignore-path ../../.prettierignore --check .",
     "clean": "rm -rf dist node_modules/.vite",
     "lint": "eslint ."
   },

scripts/fix_format.sh

@@ -26,9 +26,9 @@ cd "$REPO_ROOT"
 
 echo "Running Prettier..."
 if [ "$CHECK_ONLY" = true ]; then
-  npx -y prettier --config .prettierrc --check .
+  corepack yarn prettier:check
 else
-  npx -y prettier --config .prettierrc --write .
+  corepack yarn prettier
 fi
 
 echo "Done."

shell/e2e/gallery.e2e.ts

@@ -135,8 +135,7 @@ test.describe('Components Gallery User Journey', () => {
           const createSurface = cmd1['createSurface'] as Record<string, unknown> | undefined;
           const updateComponents = cmd2['updateComponents'] as Record<string, unknown> | undefined;
           const components = updateComponents?.['components'] as
-            | Array<Record<string, unknown>>
-            | undefined;
+            Array<Record<string, unknown>> | undefined;
           const hasComponent =
             Array.isArray(components) &&
             components.some(c => c['component'] === firstComponentName);

shell/package.json

@@ -13,6 +13,7 @@
     "e2e": "playwright test",
     "e2e-headless": "E2E_HEADLESS=true playwright test",
     "prettier": "prettier --ignore-path ../.prettierignore --write . | grep -v '(unchanged)'",
+    "prettier:check": "prettier --ignore-path ../.prettierignore --check .",
     "clean": "rm -rf dist .angular node_modules/.vite",
     "lint": "eslint ."
   },