feat: add PR link checking workflow, fix links and add fallback to english pages (#2192)

Author: bjohansebas

.github/workflows/build.yml

@@ -75,3 +75,36 @@ jobs:
       - name: Build Astro site
         working-directory: astro
         run: npm run build
+
+  linkChecker:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v6
+        with:
+          persist-credentials: false
+
+      # TODO: use astro action, when pnpm action is enabled in the org
+      - name: Set up Node.js
+        uses: actions/setup-node@v6
+        with:
+          # node-version-file: ".nvmrc" use when .nvmrc is on root of the repo
+          node-version: "24.13"
+          # cache: "npm" use when package-lock.json is on root of the repo
+
+      - name: Install Node.js dependencies
+        working-directory: astro
+        run: npm ci
+
+      - name: Build Astro site
+        working-directory: astro
+        run: npm run build
+
+      - name: Check links
+        uses: lycheeverse/lychee-action@a8c4c7cb88f0c7386610c35eb25108e448569cb0 # v2.7.0
+        with:
+          workingDirectory: astro # TODO: change when Astro site is moved to root
+          args: |
+            --root-dir $PWD/dist
+            --remap 'https://expressjs\.com\/(.*)/ file://'$PWD'/dist/$1/index.html'
+            dist/
+          fail: true

astro/.lycheeignore

@@ -0,0 +1,17 @@
+# Prevent rate limit for npmjs links
+
+http://npmjs.com/.*
+https://npmjs.com/.*
+https://www.npmjs.com/.*
+https://npmjs.org/.*
+https://www.npmjs.org/.*
+
+# Prevent github rate limit
+https://github.com/.*
+
+# Exclude 404 pages
+dist/.*/404/index.html$
+dist/404/index.html$
+
+# Exclude Open Collective links
+https://opencollective.com/.*

astro/CONTRIBUTING.md

@@ -3,7 +3,7 @@
 This is the contribution documentation for the [expressjs.com](https://github.com/expressjs/expressjs.com) website.
 
 > [!NOTE]
-> This is not the repo for Express.js framework. To contribute to the _[Express.js framework](https://github.com/expressjs/express)_, check out the [GitHub repo contributing page](https://github.com/expressjs/express?tab=contributing-ov-file) or the website's [Contributing to Express](https://expressjs.com/en/resources/contributing.html) page.
+> This is not the repo for Express.js framework. To contribute to the _[Express.js framework](https://github.com/expressjs/express)_, check out the [GitHub repo contributing page](https://github.com/expressjs/express?tab=contributing-ov-file) or the website's [Contributing to Express](https://expressjs.com/en/resources/contributing) page.
 
 ### Common contributions
 

astro/astro.config.mjs

@@ -4,9 +4,16 @@ import mdx from '@astrojs/mdx';
 import icon from 'astro-icon';
 import expressiveCode from 'astro-expressive-code';
 
+/* https://docs.netlify.com/configure-builds/environment-variables/#read-only-variables */
+const NETLIFY_PREVIEW_SITE = process.env.CONTEXT !== 'production' && process.env.DEPLOY_PRIME_URL;
+
+const site = NETLIFY_PREVIEW_SITE || 'https://expressjs.com';
+
+// TODO: add redirecto for blog posts
+
 // https://astro.build/config
 export default defineConfig({
-  site: 'https://expressjs.com',
+  site,
   integrations: [
     expressiveCode({
       themes: ['github-dark'],

astro/eslint.config.js

@@ -3,11 +3,19 @@ import tseslint from 'typescript-eslint';
 import eslintAstroPlugin from 'eslint-plugin-astro';
 import jsxA11y from 'eslint-plugin-jsx-a11y';
 import eslintConfigPrettier from 'eslint-config-prettier';
+import globals from 'globals';
+import { globalIgnores } from 'eslint/config';
 
 export default [
   // Global ignores
+  globalIgnores(['dist/*', '.astro/*', 'node_modules/*', 'package-lock.json']),
   {
-    ignores: ['dist/', '.astro/', 'node_modules/', 'package-lock.json'],
+    languageOptions: {
+      globals: {
+        ...globals.browser,
+        ...globals.node,
+      },
+    },
   },
 
   // Base ESLint recommended rules