@financial-times/dotcom-middleware-app-context 7.2.7 → 7.3.1

package/package.json

@@ -1,18 +1,17 @@
 {
   "name": "@financial-times/dotcom-middleware-app-context",
-  "version": "7.2.7",
+  "version": "7.3.1",
   "description": "",
   "main": "dist/node/index.js",
   "types": "src/index.ts",
   "scripts": {
     "test": "echo \"Error: no test specified\" && exit 1",
-    "tsc": "../../node_modules/.bin/tsc --incremental",
     "clean": "npm run clean:dist && npm run clean:node_modules",
     "clean:dist": "rm -rf dist",
     "clean:node_modules": "rm -rf node_modules",
+    "build:node": "tsc",
     "build": "npm run build:node",
     "dev": "npm run build:node -- --watch",
-    "build:node": "npm run tsc -- --module commonjs --outDir ./dist/node",
     "preinstall": "[ \"$INIT_CWD\" != \"$PWD\" ] || npm_config_yes=true npx check-engine"
   },
   "keywords": [],
@@ -25,12 +24,15 @@
     "node-mocks-http": "^1.7.5"
   },
   "dependencies": {
-    "@financial-times/dotcom-server-app-context": "^7.2.7"
+    "@financial-times/dotcom-server-app-context": "file:../dotcom-server-app-context"
   },
   "engines": {
     "node": ">= 14.0.0",
     "npm": "7.x || 8.x"
   },
+  "files": [
+    "dist/"
+  ],
   "repository": {
     "type": "git",
     "repository": "https://github.com/Financial-Times/dotcom-page-kit.git",
@@ -40,4 +42,4 @@
   "volta": {
     "extends": "../../package.json"
   }
-}
+}

package/src/index.ts

@@ -1,38 +0,0 @@
-import { Request, Response, NextFunction } from 'express'
-import pkg from '@financial-times/dotcom-server-app-context/package.json'
-import { AppContext, TAppContext } from '@financial-times/dotcom-server-app-context'
-
-export type TMiddlewareOptions = {
-  appContext?: Partial<TAppContext>
-}
-
-export function init(options: TMiddlewareOptions = {}) {
-  return (request: Request, response: Response, next: NextFunction) => {
-    const appContext = {
-      // TODO: improve how we retrieve the app name.
-      // HACK: this is plucked from the debug headers set by n-express:
-      // https://github.com/Financial-Times/n-express/blob/HEAD/main.js#L76-L80
-      appName: response.get('ft-app-name'),
-      product: 'next',
-      edition: request.get('ft-edition'),
-      appVersion: process.env.CIRCLE_SHA1 || process.env.SOURCE_VERSION || process.env.HEROKU_SLUG_COMMIT,
-      // Many headers are set to a default value of "-" by the CDN so we need to ignore those
-      // https://github.com/Financial-Times/ft.com-cdn/blob/HEAD/src/vcl/next-preflight.vcl
-      abTestState: request.get('ft-ab') === '-' ? undefined : request.get('ft-ab'),
-      isProduction: process.env.NODE_ENV === 'production',
-      // This is set by the membership session service as part of preflight
-      // https://github.com/Financial-Times/next-preflight/blob/HEAD/server/tasks/membership/session.js
-      isUserLoggedIn: request.get('ft-anonymous-user') === 'false',
-      pageKitVersion: pkg.version === '0.0.0' ? 'development' : pkg.version,
-      ...options.appContext
-    }
-
-    try {
-      response.locals.appContext = new AppContext({ appContext })
-    } catch (error) {
-      next(error)
-    }
-
-    next()
-  }
-}