[DEV-3763] Removed cloudfront function build (#2130)

* removed cloudfront function build. Just go with js. There is no need to use type script

* + changeset

Author: uolter

.changeset/chubby-suits-build.md

@@ -0,0 +1,5 @@
+---
+"infrastructure": minor
+---
+
+Removed cloudfront function build. From now on we'll just go with java script

.github/workflows/code_review_infra.yaml

@@ -31,16 +31,7 @@ jobs:
         uses: actions/checkout@c85c95e3d7251135ab7dc9ce3241c5835cc595a9 # v3.5.3
         with:
           submodules: recursive
-
-      - name: Setup Node.JS
-        uses: ./.github/actions/setup-node
-
-      - name: Install dependencies
-        run: npm ci --audit=false --fund=false
-
-      - name: Compile Cloudfront Functions
-        run: npm run compile -w cloudfront-functions
-
+            
       - name: Pull & update submodules recursively
         run: git submodule update --init --recursive
 

.github/workflows/deploy_infra.yml

@@ -34,7 +34,7 @@ permissions:
 jobs:
   deploy:
     name: Terraform apply (manual on ${{ inputs.environment }})
-    runs-on: ubuntu-latest
+    runs-on: ubuntu-24.04
     continue-on-error: false
     environment: ${{ inputs.environment }}
 
@@ -48,15 +48,6 @@ jobs:
         with:
           submodules: recursive
 
-      - name: Setup Node.JS
-        uses: ./.github/actions/setup-node
-
-      - name: Install dependencies
-        run: npm ci --audit=false --fund=false
-
-      - name: Compile Cloudfront Functions
-        run: npm run compile -w cloudfront-functions
-
       - name: Pull & update submodules recursively
         run: git submodule update --init --recursive
 

apps/cloudfront-functions/.eslintrc.json

@@ -0,0 +1,39 @@
+"use strict";
+// This code is executed in the CloudFront Functions JavaScript runtime. In this
+// context we prefer performance over immutability.
+// https://docs.aws.amazon.com/AmazonCloudFront/latest/DeveloperGuide/writing-function-code.html#function-code-modify-request
+/* eslint-disable functional/immutable-data */
+/* eslint-disable functional/no-expression-statements */
+// eslint-disable-next-line @typescript-eslint/no-unused-vars
+var handler = function (event) {
+    if (event.context.eventType === 'viewer-request') {
+        // do the rewrite
+        var request = event.request;
+        var uri_1 = request.uri;
+        // Check if the uri refers to a gitbook asset
+        var isGitbookAssets = uri_1.startsWith('/gitbook/docs');
+        var isWoff2 = uri_1.endsWith('.woff2'); // woff2 is a font format
+        var uriEndsWithSlash = uri_1.endsWith('/');
+        var isHomepage = uri_1 === '/';
+        // List of special cases (add more as needed)
+        var specialCases = ['.bollo', '.a', '.b', '.c'];
+        // Function to check if URI ends with any of the special cases
+        var isSpecialCase = specialCases.some(function (caseExt) { return uri_1.endsWith(caseExt); });
+        if (!isHomepage) {
+            if (uriEndsWithSlash) {
+                request.uri = uri_1.replace(/\/$/, '');
+            }
+            // Always add .html if there's no file extension, including special cases
+            if (!isGitbookAssets &&
+                !isWoff2 &&
+                (!/\.[a-zA-Z]+$/.test(uri_1) || isSpecialCase)) {
+                request.uri += '.html';
+            }
+        }
+        return request;
+    }
+    else {
+        // do nothing
+        return event.request;
+    }
+};