fix: address reviewer feedback on cache optimization PR (#5200)

* fix: remove duplicate cache entries and fix YAML formatting

- Remove duplicate path entries in dev-release.yaml
- Remove redundant node_modules/.cache entries since setup-node handles npm caching
- Fix YAML indentation issues across workflow files

Addresses reviewer feedback on PR #5117 about duplication and maintenance

* fix: add cache-dependency-path for setup-node in test-ui workflow

The test-ui workflow runs at repo root but package-lock.json is in ComfyUI_frontend/
subdirectory, causing setup-node cache to fail. Added cache-dependency-path to fix this.

Author: christian-byrne

.github/workflows/chromatic.yaml

@@ -56,7 +56,6 @@ jobs:
         with:
           path: |
             .cache
-            node_modules/.cache
             storybook-static
             tsconfig.tsbuildinfo
           key: storybook-cache-${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}-${{ hashFiles('src/**/*.{ts,vue,js}', '*.config.*', '.storybook/**/*') }}

.github/workflows/dev-release.yaml

@@ -27,11 +27,7 @@ jobs:
           path: |
             .cache
             dist
-            node_modules/.cache
             tsconfig.tsbuildinfo
-            .cache
-            tsconfig.tsbuildinfo
-            dist
           key: dev-release-tools-cache-${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             dev-release-tools-cache-${{ runner.os }}-

.github/workflows/i18n.yaml

@@ -23,7 +23,6 @@ jobs:
       with:
         path: |
           ComfyUI_frontend/.cache
-          ComfyUI_frontend/node_modules/.cache
           ComfyUI_frontend/.cache
         key: i18n-tools-cache-${{ runner.os }}-${{ hashFiles('ComfyUI_frontend/package-lock.json') }}
         restore-keys: |

.github/workflows/release.yaml

@@ -28,7 +28,6 @@ jobs:
         uses: actions/cache@v4
         with:
           path: |
-            node_modules/.cache
             .cache
             tsconfig.tsbuildinfo
           key: release-tools-cache-${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}
@@ -136,7 +135,6 @@ jobs:
         uses: actions/cache@v4
         with:
           path: |
-            node_modules/.cache
             .cache
             tsconfig.tsbuildinfo
             dist

.github/workflows/test-ui.yaml

@@ -41,6 +41,7 @@ jobs:
         with:
           node-version: lts/*
           cache: 'npm'
+          cache-dependency-path: 'ComfyUI_frontend/package-lock.json'
 
       - name: Get current time
         id: current-time

.github/workflows/update-electron-types.yaml

@@ -25,7 +25,6 @@ jobs:
         with:
           path: |
             .cache
-            node_modules/.cache
           key: electron-types-tools-cache-${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             electron-types-tools-cache-${{ runner.os }}-

.github/workflows/update-manager-types.yaml

@@ -29,7 +29,6 @@ jobs:
         uses: actions/cache@v4
         with:
           path: |
-            node_modules/.cache
             .cache
           key: update-manager-tools-cache-${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |

.github/workflows/update-registry-types.yaml

@@ -29,7 +29,6 @@ jobs:
         with:
           path: |
             .cache
-            node_modules/.cache
           key: update-registry-tools-cache-${{ runner.os }}-${{ hashFiles('**/package-lock.json') }}
           restore-keys: |
             update-registry-tools-cache-${{ runner.os }}-