Merge pull request #7595 from saurabhraghuvanshii/build

CodexRaunak · web-flow · commit 6b7a832b29d4 · 2026-04-15T01:11:35.000+05:30
Removing Pr-Preview from netlify, moving to GitHub page
diff --git a/.github/workflows/build-and-deploy-site.yml b/.github/workflows/build-and-deploy-site.yml
@@ -1,49 +1,43 @@
-name: Build and Deploy Site
+name: Deploy Gatsby site to Pages
+
 on:
   push:
     branches: [master]
-    paths-ignore:
-      - .github/*
-      
   workflow_dispatch:
-  workflow_call:
-    
+
+permissions:
+  contents: write
+
+concurrency:
+  group: "pages-deployment"
+  cancel-in-progress: true
+
 jobs:
-  build-and-deploy:
-    runs-on: ubuntu-latest
+  deploy:
+    runs-on: ubuntu-24.04
+
     steps:
-      - name: Checkout 🛎️
-        uses: actions/checkout@master
+      - name: Checkout
+        uses: actions/checkout@v4
+
+      - name: Setup Node
+        uses: actions/setup-node@v4
         with:
-          fetch-depth: 1
-
-      # - name: Cache Gatsby .cache and public folders
-      #   uses: actions/cache@v5
-      #   id: gatsby-cache
-      #   with:
-      #     path: |
-      #       public
-      #       .cache
-      #     key: ${{ runner.os }}-gatsby-${{ hashFiles('**/package-lock.json') }}
-      #     restore-keys: |
-      #       ${{ runner.os }}-gatsby-
-
-      - name: Install 🔧
-        run: make setup
-
-      - name: Build 🏗️
-        # Always run the full production build explicitly.
-        # Do NOT use `make clean` here — that target is a developer convenience
-        # command whose recipe has changed multiple times, causing blog posts and
-        # other collections to be silently excluded from the deployed site when
-        # it ran a lite/dev build instead of the full production build.
-        # `npm run build` is the canonical, stable production build command:
-        #   cross-env BUILD_FULL_SITE=true gatsby build
-        run: npm run build
-
-      - name: Deploy 🚀
-        uses: JamesIves/github-pages-deploy-action@v4
+          node-version: "20"
+
+      - name: Install dependencies
+        run: npm ci
+
+      - name: Build Gatsby site
+        run: |
+          npm run build
+          echo "layer5.io" > public/CNAME
+          touch public/.nojekyll
+
+      - name: Deploy to GitHub Pages
+        uses: peaceiris/actions-gh-pages@v4
         with:
-          branch: site # The branch the action should deploy to.
-          folder: public # The folder the action should deploy.
-          clean: true
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          publish_dir: ./public
+          publish_branch: gh-pages
+          keep_files: true
diff --git a/.github/workflows/build-and-preview-site.yml b/.github/workflows/build-and-preview-site.yml
@@ -1,45 +1,86 @@
-name: Build and Preview Site
+name: PR Preview Deployment (Gatsby + GitHub Pages)
+
 on:
-  pull_request:
+  pull_request_target:
     branches: [master]
-    types: [opened, synchronize, reopened]
+    types: [opened, synchronize, reopened, closed]
+
+permissions:
+  contents: write
+  pull-requests: write
+
+concurrency:
+  group: preview-${{ github.event.pull_request.number || github.run_id }}
+  cancel-in-progress: true
 
 jobs:
-  site-preview:
+  preview:
     runs-on: ubuntu-latest
+
     steps:
-      - name: Checkout 🛎️
-        uses: actions/checkout@master
+
+      - name: Checkout PR
+        if: github.event.action != 'closed'
+        uses: actions/checkout@v4
+        with:
+          ref: ${{ github.event.pull_request.head.sha }}
+
+      - name: Checkout gh-pages for cleanup
+        if: github.event.action == 'closed'
+        uses: actions/checkout@v4
+        with:
+          ref: gh-pages
+
+      - name: Setup Node
+        if: github.event.action != 'closed'
+        uses: actions/setup-node@v4
+        with:
+          node-version: "20"
+
+      - name: Install dependencies
+        if: github.event.action != 'closed'
+        run: npm ci
+
+      - name: Set PATH_PREFIX for preview
+        if: github.event.action != 'closed'
+        run: |
+          echo "PATH_PREFIX=pr-preview/pr-${{ github.event.pull_request.number }}" >> $GITHUB_ENV
+
+      - name: Build PR preview
+        if: github.event.action != 'closed'
+        env:
+          GATSBY_PREVIEW: "true"
+          GATSBY_SITE_URL: https://${{ github.repository_owner }}.github.io
+        run: |
+          PATH_PREFIX=$PATH_PREFIX npm run build:preview
+
+          # Prevent indexing
+          echo -e "User-agent: *\nDisallow: /" > public/robots.txt
+
+      - name: Deploy PR preview
+        if: github.event.action != 'closed'
+        uses: rossjrw/pr-preview-action@v1.6.3
         with:
-          persist-credentials: false
-          fetch-depth: 1
-
-      - name: Install 🔧
-        run: make setup
-
-      - name: Build 🏗️
-        # Always run the full production build explicitly.
-        # Do NOT use `make clean` here — that target is a developer convenience
-        # command whose recipe has changed multiple times, causing blog posts and
-        # other collections to be silently excluded from the deployed site when
-        # it ran a lite/dev build instead of the full production build.
-        # `npm run build` is the canonical, stable production build command:
-        #   cross-env BUILD_FULL_SITE=true gatsby build
-        run: npm run build
-
-      - name: Broken Link Check 🔗
-        uses: technote-space/broken-link-checker-action@v2
+          source-dir: ./public
+          preview-branch: gh-pages
+          umbrella-dir: pr-preview
+          action: auto
+          comment: false
+
+      - name: Comment PR with Preview URL
+        if: github.event.action != 'closed'
+        uses: marocchino/sticky-pull-request-comment@v2
         with:
-          target: ./public/**/*.html  
+          header: pr-preview
+          message: |
+            🚀 **Preview for PR #${{ github.event.pull_request.number }}**
 
-      - name: Zip Site
-        run: bash ./script.sh
+            🌐 https://${{ github.repository_owner }}.github.io/${{ github.event.repository.name }}/pr-preview/pr-${{ github.event.pull_request.number }}/
 
-      - name: Upload files
-        uses: actions/upload-artifact@v6
+      - name: Cleanup PR preview
+        if: github.event.action == 'closed'
+        uses: rossjrw/pr-preview-action@v1.6.3
         with:
-          name: public-dir
-          path: ./public-dir.zip
-          retention-days: 1
-      - name: Trigger Inner workflow
-        run: echo "triggering inner workflow"
+          preview-branch: gh-pages
+          umbrella-dir: pr-preview
+          action: remove
diff --git a/.github/workflows/preview-site.yml b/.github/workflows/preview-site.yml
diff --git a/gatsby-config.js b/gatsby-config.js
@@ -8,6 +8,16 @@ const {
 
 const isDevelopment = process.env.NODE_ENV === "development";
 const isProduction = process.env.NODE_ENV === "production";
+const isPreviewBuild = process.env.GATSBY_PREVIEW === "true";
+const siteOrigin = (process.env.GATSBY_SITE_URL || "https://layer5.io").replace(
+  /\/$/,
+  "",
+);
+const rawPathPrefix = process.env.PATH_PREFIX || "";
+const pathPrefix = rawPathPrefix
+  ? `/${rawPathPrefix.replace(/^\/+|\/+$/g, "")}`
+  : "";
+const siteRootUrl = `${siteOrigin}${pathPrefix}`.replace(/\/$/, "");
 const shouldBuildFullSite = isFullSiteBuild();
 const isLiteDevBuild = isDevelopment && !shouldBuildFullSite;
 const excludedCollections = getExcludedCollections({
@@ -34,11 +44,12 @@ module.exports = {
     description:
       "Expect more from your infrastructure. Cloud native, open source software for your internal development platforms, your DevOps, platform engineering and site reliability engineering teams. Less finger-pointing and more collaborating. Allowing developers to focus on business logic, not infrastructure concerns. Empowering operators to confidently run modern infrastructure.",
     author: "Layer5 Authors",
-    permalink: "https://layer5.io",
-    siteUrl: "https://layer5.io",
+    permalink: siteRootUrl,
+    siteUrl: siteRootUrl,
     image: "/images/layer5-gradient.webp",
     twitterUsername: "@layer5",
   },
+  ...(pathPrefix ? { pathPrefix } : {}),
   flags: {
     FAST_DEV: false,
     DEV_SSR: false,
@@ -560,14 +571,18 @@ module.exports = {
         query: "allMdx",
       },
     },
-    {
-      resolve: "gatsby-plugin-robots-txt",
-      options: {
-        host: "https://layer5.io",
-        sitemap: "https://layer5.io/sitemap-index.xml",
-        policy: [{ userAgent: "*", allow: "/" }],
-      },
-    },
+    ...(!isPreviewBuild
+      ? [
+          {
+            resolve: "gatsby-plugin-robots-txt",
+            options: {
+              host: siteRootUrl,
+              sitemap: `${siteRootUrl}/sitemap-index.xml`,
+              policy: [{ userAgent: "*", allow: "/" }],
+            },
+          },
+        ]
+      : []),
     "gatsby-plugin-meta-redirect",
     // make sure this is always the last one
   ],
diff --git a/package.json b/package.json
@@ -11,6 +11,8 @@
   },
   "scripts": {
     "build": "cross-env BUILD_FULL_SITE=true NODE_OPTIONS=--max-old-space-size=8192 gatsby build",
+    "build:gh-pages": "cross-env BUILD_FULL_SITE=true NODE_OPTIONS=--max-old-space-size=8192 gatsby build --prefix-paths",
+    "build:preview": "cross-env BUILD_FULL_SITE=true GATSBY_PREVIEW=true NODE_OPTIONS=--max-old-space-size=8192 gatsby build --prefix-paths",
     "clean": "gatsby clean",
     "clean:all": "gatsby clean && rimraf node_modules",
     "develop": "cross-env BUILD_FULL_SITE=true GATSBY_CPU_COUNT=4 SHARP_CONCURRENCY=4 UV_THREADPOOL_SIZE=4 NODE_OPTIONS=--max-old-space-size=8192 env-cmd -f .env.development gatsby develop",
diff --git a/src/components/seo.js b/src/components/seo.js
@@ -32,21 +32,32 @@ const SEO = ({
   children,
 }) => {
   const { pathname } = useLocation();
+  const isPreviewBuild = process.env.GATSBY_PREVIEW === "true";
   const {
     title: defaultTitle,
     description: defaultDescription,
     image: siteMetadataImage,
     siteUrl,
     twitterUsername,
   } = useSiteMetadata();
+  const toAbsoluteUrl = (value) => {
+    if (!value) return value;
+    if (/^(?:[a-z]+:)?\/\//i.test(value)) return value;
+    return new URL(value.replace(/^\//, ""), `${siteUrl}/`).toString();
+  };
   const seo = {
     title: title || defaultTitle,
     description: description || defaultDescription,
-    image: `${siteUrl}${image || siteMetadataImage}`,
-    url: `${siteUrl}${pathname.replace(".html", "") || ""}`.replace(/\/$/, ""),
+    image: toAbsoluteUrl(image || siteMetadataImage),
+    url: new URL(
+      (pathname.replace(".html", "") || "/").replace(/^\//, ""),
+      `${siteUrl}/`,
+    )
+      .toString()
+      .replace(/\/$/, ""),
     twitterUsername,
   };
-  if (!canonical) {
+  if (!canonical || isPreviewBuild) {
     canonical = seo.url;
   }
 
@@ -60,6 +71,12 @@ const SEO = ({
       <meta name="description" content={seo.description} />
       <meta name="image" content={seo.image} />
       <meta name="author" content="Layer5, Inc." />
+      {isPreviewBuild && (
+        <>
+          <meta name="robots" content="noindex, nofollow" />
+          <meta name="googlebot" content="noindex, nofollow" />
+        </>
+      )}
 
       {/* OpenGraph Meta Tags (Facebook, LinkedIn, etc.) */}
       <meta property="og:type" content="website" />
