Move site-metadata into its own helper, and export node stuff

packages/gatsby-cli/src/init-starter.ts

@@ -9,7 +9,7 @@ import isValid from "is-valid-path"
 import sysPath from "path"
 import prompts from "prompts"
 import url from "url"
-import { createServiceLock } from "gatsby-core-utils/dist/service-lock"
+import { updateSiteMetadata } from "gatsby-core-utils/node"
 import report from "./reporter"
 import { getPackageManager, promptPackageManager } from "./util/package-manager"
 import { isTTY } from "./util/is-tty"
@@ -358,11 +358,14 @@ export async function initStarter(
       )
     })
 
-  await createServiceLock(sitePath, `metadata`, {
-    name: sitePackageJson?.name || rootPath,
-    sitePath,
-    lastRun: Date.now(),
-  }).then(unlock => unlock?.())
+  await updateSiteMetadata(
+    {
+      name: sitePackageJson?.name || rootPath,
+      sitePath,
+      lastRun: Date.now(),
+    },
+    false
+  )
 
   successMessage(rootPath)
   trackCli(`NEW_PROJECT_END`)

packages/gatsby-core-utils/node.d.ts

@@ -0,0 +1 @@
+export * from "./dist/node"

packages/gatsby-core-utils/node.js

@@ -0,0 +1 @@
+module.exports = require("./dist/node")

packages/gatsby-core-utils/package.json

@@ -10,6 +10,7 @@
   "homepage": "https://github.com/gatsbyjs/gatsby/tree/master/packages/gatsby-core-utils#readme",
   "license": "MIT",
   "main": "dist/index.js",
+  "types": "dist/index.d.ts",
   "repository": {
     "type": "git",
     "url": "https://github.com/gatsbyjs/gatsby.git",
@@ -26,7 +27,9 @@
   },
   "files": [
     "dist/",
-    "dist/index.d.ts"
+    "dist/index.d.ts",
+    "node.js",
+    "node.d.ts"
   ],
   "dependencies": {
     "ci-info": "2.0.0",

packages/gatsby-core-utils/src/node.ts

@@ -0,0 +1,2 @@
+export * from "./service-lock"
+export * from "./site-metadata"

packages/gatsby-core-utils/src/site-metadata.ts

@@ -0,0 +1,30 @@
+import { createServiceLock, getService } from "./service-lock"
+
+export interface ISiteMetadata {
+  sitePath: string
+  name?: string
+  pid?: number
+  lastRun?: number
+}
+
+export async function getSiteMetadata(
+  sitePath: string
+): Promise<ISiteMetadata | null> {
+  return getService(sitePath, `metadata`, true)
+}
+
+export async function updateSiteMetadata(
+  metadata: ISiteMetadata,
+  merge = true
+): Promise<void> {
+  if (merge) {
+    const oldMetadata = (await getSiteMetadata(metadata.sitePath)) || {}
+    metadata = { ...oldMetadata, ...metadata }
+  }
+
+  return createServiceLock(
+    metadata.sitePath,
+    `metadata`,
+    metadata
+  ).then(unlock => unlock?.())
+}

packages/gatsby-core-utils/tsconfig.json

@@ -1,4 +1,10 @@
 {
   "extends": "../../tsconfig.json",
-  "exclude": ["node_modules", "src/__tests__", "src/__mocks__", "dist"]
+  "exclude": [
+    "node_modules",
+    "src/__tests__",
+    "src/__mocks__",
+    "dist",
+    "./node.d.ts"
+  ]
 }

packages/gatsby-recipes/src/graphql-server/index.js

@@ -2,10 +2,7 @@ const execa = require(`execa`)
 const path = require(`path`)
 const fs = require(`fs`)
 const detectPort = require(`detect-port`)
-const {
-  getService,
-  createServiceLock,
-} = require(`gatsby-core-utils/dist/service-lock`)
+const { getService, createServiceLock } = require(`gatsby-core-utils/node`)
 
 // NOTE(@mxstbr): The forceStart boolean enforces us to start the recipes graphql server
 // even if another instance might already be running. This is necessary to ensure the gatsby

packages/gatsby-recipes/src/graphql-server/start-dev-cli-server.js

@@ -1,4 +1,4 @@
-const { createServiceLock } = require(`gatsby-core-utils/dist/service-lock`)
+const { createServiceLock } = require(`gatsby-core-utils/node`)
 const execa = require(`execa`)
 
 const startServer = async () => {

packages/gatsby-recipes/src/providers/gatsby/page.js

@@ -1,5 +1,5 @@
 const Joi = require(`@hapi/joi`)
-const { getService } = require(`gatsby-core-utils/dist/service-lock`)
+const { getService } = require(`gatsby-core-utils/node`)
 const fetch = require(`node-fetch`)
 
 const { REQUIRES_KEYS } = require(`./utils/constants`)

packages/gatsby/src/commands/build.ts

@@ -36,7 +36,7 @@ import {
   markWebpackStatusAsPending,
   markWebpackStatusAsDone,
 } from "../utils/webpack-status"
-import { createServiceLock } from "gatsby-core-utils/dist/service-lock"
+import { updateSiteMetadata } from "gatsby-core-utils/node"
 
 let cachedPageData
 let cachedWebpackCompilationHash
@@ -64,11 +64,12 @@ module.exports = async function build(program: IBuildArgs): Promise<void> {
     )
   }
 
-  await createServiceLock(program.directory, `metadata`, {
+  await updateSiteMetadata({
     name: program.sitePackageJson.name,
     sitePath: program.directory,
     lastRun: Date.now(),
-  }).then(unlock => unlock?.())
+    pid: process.pid,
+  })
 
   markWebpackStatusAsPending()
 

packages/gatsby/src/commands/develop.ts

@@ -12,7 +12,8 @@ import { isCI, slash } from "gatsby-core-utils"
 import {
   createServiceLock,
   getService,
-} from "gatsby-core-utils/dist/service-lock"
+  updateSiteMetadata,
+} from "gatsby-core-utils/node"
 import { UnlockFn } from "gatsby-core-utils/src/service-lock"
 import reporter from "gatsby-cli/lib/reporter"
 import { getSslCert } from "../utils/get-ssl-cert"
@@ -268,13 +269,12 @@ module.exports = async (program: IProgram): Promise<void> => {
         port: proxyPort,
       }
     )
-    // We don't need to keep a lock on this, as it's just site metadata
-    await createServiceLock(program.directory, `metadata`, {
+    await updateSiteMetadata({
       name: program.sitePackageJson.name,
       sitePath: program.directory,
       pid: process.pid,
       lastRun: Date.now(),
-    }).then(unlock => unlock?.())
+    })
 
     if (!statusUnlock || !developUnlock) {
       const data = await getService(program.directory, `developproxy`)

packages/gatsby/src/utils/develop-proxy.ts

@@ -3,7 +3,7 @@ import https from "https"
 import httpProxy from "http-proxy"
 import path from "path"
 import fs from "fs-extra"
-import { getServices } from "gatsby-core-utils/dist/service-lock"
+import { getServices } from "gatsby-core-utils/node"
 import st from "st"
 import restartingScreen from "./restarting-screen"
 import { IProgram } from "../commands/types"