Skip to content

Commit 29dd157

Browse files
hackall360claude
andcommitted
Refactor: Use copy-assets.js in deployment workflow
- Replace hardcoded file list with centralized copy-assets.js script - Single source of truth for files to copy during build - Adding new assets only requires updating copy-assets.js 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 1656cd3 commit 29dd157

File tree

1 file changed

+10
-18
lines changed

1 file changed

+10
-18
lines changed

.github/workflows/deploy.yml

Lines changed: 10 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -73,24 +73,16 @@ jobs:
7373
7474
- name: Copy additional files to dist
7575
run: |
76-
echo "📋 Copying additional files..."
77-
# Copy files that Vite doesn't process but are needed
78-
cp -r vendor dist/ 2>/dev/null || echo "No vendor directory to copy"
79-
cp -r fonts dist/ 2>/dev/null || echo "No fonts directory to copy"
80-
cp -r PolliLibJS dist/ 2>/dev/null || echo "No PolliLibJS directory to copy"
81-
cp robots.txt dist/ 2>/dev/null || echo "No robots.txt to copy"
82-
cp sitemap.xml dist/ 2>/dev/null || echo "No sitemap.xml to copy"
83-
cp BingSiteAuth.xml dist/ 2>/dev/null || echo "No BingSiteAuth.xml to copy"
84-
cp visitor-tracking.js dist/ 2>/dev/null || echo "No visitor-tracking.js to copy"
85-
86-
# Copy page-specific JS files
87-
cp about/about.js dist/about/ 2>/dev/null || echo "No about.js to copy"
88-
cp ai/demo/age-verification.js dist/ai/demo/ 2>/dev/null || echo "No age-verification.js to copy"
89-
cp ai/demo/js/main.js dist/ai/demo/js/ 2>/dev/null || mkdir -p dist/ai/demo/js && cp ai/demo/js/main.js dist/ai/demo/js/ || echo "No main.js to copy"
90-
mkdir -p dist/apps && cp apps/age-verification.js dist/apps/ 2>/dev/null || echo "No apps/age-verification.js to copy"
91-
cp apps/apps.css dist/apps/ 2>/dev/null || echo "No apps/apps.css to copy"
92-
93-
echo "✅ Additional files copied"
76+
echo "📋 Copying additional files using copy-assets.js..."
77+
# Use the centralized copy-assets.js script for consistency
78+
# This script maintains the list of all files/directories to copy
79+
node copy-assets.js
80+
81+
echo ""
82+
echo "📦 Final dist contents:"
83+
find dist -type f | head -50
84+
echo "..."
85+
echo "Total files: $(find dist -type f | wc -l)"
9486
9587
- name: Upload artifact for deployment
9688
uses: actions/upload-pages-artifact@v3

0 commit comments

Comments
 (0)