Merge remote-tracking branch 'upstream/main'

Author: costinsin

.changeset/aggregate.mjs

@@ -2,52 +2,50 @@ import fs from "node:fs/promises";
 import path from "node:path";
 import { execSync } from "node:child_process";
 
-const THANKLESS_COMMITTERS = ["thdxr", "fwang", "jayair"];
+const THANKLESS_COMMITTERS = ["thdxr", "fwang", "jayair", "conico974"];
 
 const { version } = JSON.parse(
   await fs.readFile("./packages/open-next/package.json"),
 );
 
-const changesets = JSON.parse(await fs.readFile(".changeset/config.json"));
-const packages = changesets.fixed[0];
-
 const changes = new Set();
-for (const pkg of packages) {
-  const changelog = path.join(
-    "packages",
-    pkg.split("/").at(-1),
-    "CHANGELOG.md",
-  );
-  const lines = (await fs.readFile(changelog)).toString().split("\n");
-  let start = false;
-  for (let line of lines) {
-    if (!start) {
-      if (line === `## ${version}`) {
-        start = true;
-        continue;
-      }
+
+// We only need to look for changes in packages/open-next
+const changelog = path.join(
+  "packages",
+  "open-next",
+  "CHANGELOG.md",
+);
+const lines = (await fs.readFile(changelog)).toString().split("\n");
+let start = false;
+for (let line of lines) {
+  if (!start) {
+    if (line === `## ${version}`) {
+      start = true;
+      continue;
     }
+  }
 
-    if (start) {
-      if (line.startsWith("-") || line.startsWith("*")) {
-        if (line.includes("Updated dependencies")) continue;
-        if (line.includes("@serverless-stack/")) continue;
-
-        for (const user of THANKLESS_COMMITTERS) {
-          line = line.replace(
-            `Thanks [@${user}](https://github.com/${user})! `,
-            "",
-          );
-        }
-        changes.add(line);
-        continue;
-      }
+  if (start) {
+    if (line.startsWith("-") || line.startsWith("*")) {
+      if (line.includes("Updated dependencies")) continue;
+      if (line.includes("@serverless-stack/")) continue;
 
-      if (line.startsWith("## ")) break;
+      for (const user of THANKLESS_COMMITTERS) {
+        line = line.replace(
+          `Thanks [@${user}](https://github.com/${user})! `,
+          "",
+        );
+      }
+      changes.add(line);
+      continue;
     }
+
+    if (line.startsWith("## ")) break;
   }
 }
 
+
 const notes = ["#### Changes", ...changes];
 console.log(notes.join("\n"));
 console.log(`::set-output name=notes::${notes.join("%0A")}`);

.changeset/config.json

@@ -1,6 +1,9 @@
 {
   "$schema": "https://unpkg.com/@changesets/[email protected]/schema.json",
-  "changelog": "@changesets/cli/changelog",
+  "changelog": [
+    "@changesets/changelog-github",
+    { "repo": "Genez-io/open-next" }
+  ],
   "commit": false,
   "fixed": [["@genezio/open-next"]],
   "linked": [],

.changeset/giant-vans-shop.md

@@ -0,0 +1,5 @@
+---
+"@opennextjs/aws": patch
+---
+
+Hides the x-opennext header from server requests when poweredByHeader is false in next config

.changeset/snapshot

@@ -15,7 +15,7 @@ module.exports = {
     ],
     "sonarjs/elseif-without-else": "warn",
     "sonarjs/no-duplicate-string": "warn",
-    "sonarjs/cognitive-complexity": "warn",
+    "sonarjs/cognitive-complexity": ["warn", 35],
 
     // We add some typescript rules - The recommended rules breaks too much stuff
     // TODO: We should add more rules, especially around typescript types
@@ -29,9 +29,15 @@ module.exports = {
     ],
 
     "@typescript-eslint/unbound-method": "error",
-
-    "@typescript-eslint/no-non-null-assertion": "warn",
   },
+  overrides: [
+    {
+      files: ["example/**/*", "examples/**/*"],
+      rules: {
+        "unused-imports/no-unused-vars": "off",
+      },
+    },
+  ],
   parserOptions: {
     project: ["./tsconfig.eslint.json", "./**/tsconfig.json"],
   },

.eslintrc.cjs

@@ -6,12 +6,12 @@ runs:
   steps:
     # Install nodejs. https://github.com/actions/setup-node
     - name: Setup Node.js
-      uses: actions/setup-node@v3
+      uses: actions/setup-node@v4
       with:
         node-version: 18.x
 
     # Install pnpm. https://github.com/pnpm/action-setup
-    - uses: pnpm/action-setup@v2
+    - uses: pnpm/action-setup@v4
       with:
         version: 8
         # run_install: false
@@ -22,7 +22,7 @@ runs:
       run: |
         echo "STORE_PATH=$(pnpm store path --silent)" >> $GITHUB_ENV
 
-    - uses: actions/cache@v3
+    - uses: actions/cache@v4
       name: Setup pnpm cache
       with:
         path: ${{ env.STORE_PATH }}

.github/actions/build-docs/action.yml

@@ -0,0 +1,9 @@
+name: Unit tests
+description: Run unit tests and ensure code quality
+
+runs:
+    using: 'composite'
+    steps:
+        - name: Run tests
+          run: pnpm test
+          shell: bash

.github/actions/pnpm-setup/action.yml

@@ -10,8 +10,8 @@ jobs:
         runs-on: ubuntu-latest
         steps:
             - name: Checkout
-              uses: actions/checkout@v3
+              uses: actions/checkout@v4
             - uses: ./.github/actions/pnpm-setup
             - uses: ./.github/actions/lint
+            - uses: ./.github/actions/test
             - uses: ./.github/actions/build-example
-            - uses: ./.github/actions/build-docs

.github/actions/test/action.yml

@@ -0,0 +1,36 @@
+name: Deploy docs to GitHub Pages
+
+on:
+  workflow_dispatch:
+
+defaults:
+  run:
+    working-directory: docs
+
+permissions:
+  contents: read
+  pages: write
+  id-token: write
+
+concurrency:
+  group: 'pages'
+  cancel-in-progress: false
+
+jobs:
+  deploy:
+    environment:
+      name: github-pages
+      url: ${{ steps.deployment.outputs.page_url }}
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Setup Pages
+        uses: actions/configure-pages@v5
+      - name: Upload artifact
+        uses: actions/upload-pages-artifact@v3
+        with:
+          path: 'docs/'
+      - name: Deploy to GitHub Pages
+        id: deployment
+        uses: actions/deploy-pages@v4