Send package versions as bundle metadata

packages/pwa-kit-dev/jest.config.js

@@ -11,7 +11,7 @@ module.exports = {
     ...base,
     coverageThreshold: {
         global: {
-            branches: 67,
+            branches: 65,
             functions: 67,
             lines: 70,
             statements: 70

packages/pwa-kit-dev/src/utils/script-utils.test.js

@@ -5,12 +5,19 @@
  * For full license text, see the LICENSE file in the repo root or https://opensource.org/licenses/BSD-3-Clause
  */
 
-import {mkdtemp, rm, writeFile, readJsonSync} from 'fs-extra'
+import {mkdtemp, rm, writeFile, readJsonSync, readJson} from 'fs-extra'
 import path from 'path'
 import os from 'os'
 import * as scriptUtils from './script-utils'
 const pkg = readJsonSync(path.join(__dirname, '../../package.json'))
 
+jest.mock('fs-extra', () => {
+    return {
+        ...jest.requireActual('fs-extra'),
+        readJson: jest.fn()
+    }
+})
+
 describe('scriptUtils', () => {
     const originalEnv = process.env
     let tmpDir
@@ -23,6 +30,7 @@ describe('scriptUtils', () => {
     afterEach(async () => {
         process.env = originalEnv
         tmpDir && (await rm(tmpDir, {recursive: true}))
+        jest.resetAllMocks()
     })
 
     test('glob() with no patterns matches nothing', () => {
@@ -88,6 +96,7 @@ describe('scriptUtils', () => {
         })
 
         test('getHeaders', async () => {
+            readJson.mockReturnValue(pkg)
             const client = new scriptUtils.CloudAPIClient({credentials: {username, api_key}})
             expect(await client.getHeaders()).toEqual({
                 'User-Agent': `${pkg.name}@${pkg.version}`,
@@ -96,11 +105,36 @@ describe('scriptUtils', () => {
         })
     })
 
-    test('getPkgJSON', async () => {
-        const pkg = await scriptUtils.getPkgJSON()
-        expect(pkg.name).toBe('@salesforce/pwa-kit-dev')
+    describe('getPkgJSON', () => {
+        test('should work', async () => {
+            readJson.mockReturnValue(pkg)
+            const pkgJson = await scriptUtils.getPkgJSON()
+            expect(pkgJson.name).toBe('@salesforce/pwa-kit-dev')
+        })
+
+        test('should return default package.json data when no valid file is found', async () => {
+            readJson.mockRejectedValue(Error)
+            const result = await scriptUtils.getPkgJSON()
+            expect(result).toEqual({name: '@salesforce/pwa-kit-dev', version: 'unknown'})
+        })
     })
 
+    describe('getProjectPkg', () => {
+        test('should work', async () => {
+            readJson.mockReturnValue(pkg)
+            const pkgJson = await scriptUtils.getProjectPkg()
+            expect(pkgJson.name).toBe('@salesforce/pwa-kit-dev')
+        })
+
+        test('should throw', async () => {
+            readJson.mockRejectedValue(Error)
+            await expect(scriptUtils.getProjectPkg()).rejects.toThrow(
+                `Could not read project package at "${path.join(process.cwd(), 'package.json')}"`
+            )
+        })
+    })
+
+    jest.unmock('fs-extra')
     describe('defaultMessage', () => {
         test('works', async () => {
             const mockGit = {branch: () => 'branch', short: () => 'short'}
@@ -142,6 +176,7 @@ describe('scriptUtils', () => {
 
     describe('readCredentials', () => {
         test('should work', async () => {
+            readJson.mockReturnValue({username: 'alice', api_key: 'xyz'})
             const creds = {username: 'alice', api_key: 'xyz'}
             const thePath = path.join(tmpDir, '.mobify.test')
             await writeFile(thePath, JSON.stringify(creds), 'utf8')
@@ -186,6 +221,7 @@ describe('scriptUtils', () => {
         })
 
         test('should archive a bundle', async () => {
+            readJson.mockReturnValue(pkg)
             const message = 'message'
             const bundle = await scriptUtils.createBundle({
                 message,
@@ -198,7 +234,7 @@ describe('scriptUtils', () => {
 
             expect(bundle.message).toEqual(message)
             expect(bundle.encoding).toBe('base64')
-            expect(bundle.ssr_parameters).toEqual({})
+            expect(bundle.ssr_parameters).toHaveProperty('bundle_metadata.dependencies')
             expect(bundle.ssr_only).toEqual(['ssr.js'])
             expect(bundle.ssr_shared).toEqual(['ssr.js', 'static/favicon.ico'])
 
@@ -237,6 +273,7 @@ describe('scriptUtils', () => {
         ])(
             'should push a built bundle and handle status codes (%p)',
             async ({projectSlug, targetSlug, expectedURL, status}) => {
+                readJson.mockReturnValue(pkg)
                 const message = 'message'
                 const bundle = await scriptUtils.createBundle({
                     message,
@@ -296,4 +333,38 @@ describe('scriptUtils', () => {
             }
         )
     })
+
+    describe('createLoggingToken', () => {
+        const username = 'user123'
+        const api_key = '123'
+        test('createLoggingToken passes', async () => {
+            readJson.mockReturnValue(pkg)
+            const projectSlug = 'project-slug'
+            const targetSlug = 'target-slug'
+
+            const text = () => Promise.resolve(JSON.stringify({token: 'token-value'}))
+            const json = () => Promise.resolve({token: 'token-value'})
+            const fetchMock = jest.fn(async () => ({status: 200, text, json}))
+
+            const client = new scriptUtils.CloudAPIClient({
+                credentials: {username, api_key},
+                fetch: fetchMock
+            })
+
+            const fn = async () => await client.createLoggingToken(projectSlug, targetSlug)
+
+            expect(await fn()).toBe('token-value')
+            expect(fetchMock).toHaveBeenCalledTimes(1)
+            expect(fetchMock).toHaveBeenCalledWith(
+                'https://cloud.mobify.com/api/projects/project-slug/target/target-slug/jwt/',
+                expect.objectContaining({
+                    method: 'POST',
+                    headers: {
+                        Authorization: expect.stringMatching(/^Bearer /),
+                        'User-Agent': `${pkg.name}@${pkg.version}`
+                    }
+                })
+            )
+        })
+    })
 })

packages/pwa-kit-dev/src/utils/script-utils.ts

@@ -48,6 +48,8 @@ interface Bundle {
 interface Pkg {
     name: string
     version: string
+    dependencies?: {[key: string]: string}
+    devDependencies?: {[key: string]: string}
 }
 
 /**
@@ -251,6 +253,13 @@ export const createBundle = async ({
                         archive.finalize()
                     })
             )
+            .then(async () => {
+                const {dependencies = {}, devDependencies = {}} = await getProjectPkg()
+                ssr_parameters = {
+                    ...ssr_parameters,
+                    bundle_metadata: {dependencies: {...dependencies, ...devDependencies}}
+                }
+            })
             .then(() => readFile(destination))
             .then((data) => {
                 const encoding = 'base64'