Merge pull request #213 from richardthe3rd/copilot/sub-pr-212

Apply review feedback: use npm ci and remove redundant festivals.json copy

Author: richardthe3rd

.github/workflows/deploy-worker.yml

@@ -91,9 +91,6 @@ jobs:
         working-directory: cloudflare-worker
         run: npm ci
 
-      - name: Copy festivals.json to worker directory
-        run: cp data/festivals.json cloudflare-worker/festivals.json
-
       - name: Run worker tests
         working-directory: cloudflare-worker
         run: npm test

mise.toml

@@ -48,4 +48,4 @@ run = 'flutter analyze --no-fatal-infos'
 description = "Run Cloudflare Worker tests (Vitest + workerd)"
 dir = "cloudflare-worker"
 sources = ['cloudflare-worker/package.json', 'cloudflare-worker/package-lock.json', 'cloudflare-worker/worker.js', 'cloudflare-worker/test/**/*.js', 'cloudflare-worker/vitest.config.js', 'data/festivals.json']
-run = 'npm install && npm test'
+run = 'npm ci && npm test'