Changed interact docs workflow to properly serve all content (#45)

Author: ydaniv

.github/workflows/interactdocs.yml

@@ -42,16 +42,32 @@ jobs:
       - name: Build docs app
         run: yarn workspace @wix/interact-docs build
         env:
-          # Set base path for GitHub Pages deployment under /docs
+          # Set base path for GitHub Pages deployment under /docs/
           VITE_BASE: /docs/
 
       - name: Setup Pages
         uses: actions/configure-pages@1f0c5cde4bc74cd7e1254d0cb4de8d49e9068c7d # v4
 
       - name: Prepare deployment directory
         run: |
+          # Create the site structure:
+          # /           -> root index.html and assets
+          # /docs/      -> docs app
+          # /rules/     -> rules markdown files
+          
           mkdir -p _site/docs
+          mkdir -p _site/rules
+          mkdir -p _site/assets
+          
+          # Copy root landing page and assets
+          cp index.html _site/
+          cp -r assets/* _site/assets/
+          
+          # Copy docs app
           cp -r apps/docs/dist/* _site/docs/
+          
+          # Copy rules from source (served at /rules/, not /docs/rules/)
+          cp -r packages/interact/rules/* _site/rules/
 
       - name: Upload artifact
         uses: actions/upload-pages-artifact@7b1f4a764d45c48632c6b24a0339c27f5614fb0b # v4
@@ -61,7 +77,7 @@ jobs:
   deploy:
     environment:
       name: github-pages
-      url: ${{ steps.deployment.outputs.page_url }}docs/
+      url: ${{ steps.deployment.outputs.page_url }}
     runs-on: ubuntu-latest
     needs: build
     steps:

apps/docs/scripts/copy-docs.js

@@ -8,9 +8,6 @@ const __dirname = dirname(__filename);
 const srcDocs = resolve(__dirname, '../../../packages/interact/docs');
 const destDocs = resolve(__dirname, '../dist/docs');
 
-const srcRules = resolve(__dirname, '../../../packages/interact/rules');
-const destRules = resolve(__dirname, '../dist/rules');
-
 // Create dest directory if it doesn't exist
 if (!existsSync(dirname(destDocs))) {
   mkdirSync(dirname(destDocs), { recursive: true });
@@ -21,8 +18,6 @@ cpSync(srcDocs, destDocs, { recursive: true, force: true });
 
 console.log('✓ Docs copied to dist/docs');
 
-// Copy rules to dist
-cpSync(srcRules, destRules, { recursive: true, force: true });
-
-console.log('✓ Rules copied to dist/rules');
+// Note: Rules are NOT copied here - they are deployed separately to /rules/
+// in the GitHub Actions workflow (not under /docs/rules/)
 

apps/docs/src/App.tsx

@@ -3,8 +3,11 @@ import { Layout } from './components/Layout';
 import { MarkdownPage } from './components/MarkdownPage';
 
 function App() {
+  // Remove trailing slash from BASE_URL for router basename
+  const basename = import.meta.env.BASE_URL.replace(/\/$/, '') || '/';
+  
   return (
-    <BrowserRouter>
+    <BrowserRouter basename={basename}>
       <Layout>
         <Routes>
           <Route path="*" element={<MarkdownPage />} />

apps/docs/src/components/MarkdownPage.tsx

@@ -19,8 +19,10 @@ export function MarkdownPage() {
       const mdPath = getMarkdownPath(location.pathname);
 
       try {
-        // Use the /docs endpoint which serves from packages/interact/docs
-        const response = await fetch(`/docs/${mdPath}`);
+        // Use the base URL + /docs endpoint which serves from packages/interact/docs
+        // In dev (base=/): fetches /docs/README.md
+        // In prod (base=/docs/): fetches /docs/docs/README.md
+        const response = await fetch(`${import.meta.env.BASE_URL}docs/${mdPath}`);
 
         if (!response.ok) {
           throw new Error(`Failed to load: ${mdPath}`);

apps/docs/vite.config.ts

@@ -85,7 +85,7 @@ export default defineConfig({
     serveRulesPlugin()
   ],
   // Use VITE_BASE env var for GitHub Pages deployment, default to '/' for local dev
-  base: process.env.VITE_BASE || './',
+  base: process.env.VITE_BASE || '/',
   resolve: {
     alias: {
       '@': path.resolve(__dirname, 'src'),