fix: throw an error while missing script start or file server.js

.changeset/tricky-coats-join.md

@@ -0,0 +1,5 @@
+---
+"@pnpm/lifecycle": patch
+---
+
+Throw an error while missing script start or file server.js

exec/lifecycle/package.json

@@ -37,6 +37,7 @@
   "dependencies": {
     "@pnpm/core-loggers": "workspace:*",
     "@pnpm/directory-fetcher": "workspace:*",
+    "@pnpm/error": "workspace:*",
     "@pnpm/npm-lifecycle": "^2.0.0",
     "@pnpm/read-package-json": "workspace:*",
     "@pnpm/store-controller-types": "workspace:*",

exec/lifecycle/src/runLifecycleHook.ts

@@ -2,6 +2,8 @@ import { lifecycleLogger } from '@pnpm/core-loggers'
 import { globalWarn } from '@pnpm/logger'
 import lifecycle from '@pnpm/npm-lifecycle'
 import { DependencyManifest, ProjectManifest } from '@pnpm/types'
+import { PnpmError } from '@pnpm/error'
+import { existsSync } from 'fs'
 
 function noop () {} // eslint-disable-line:no-empty
 
@@ -35,6 +37,9 @@ export async function runLifecycleHook (
 
   if (stage === 'start' && !m.scripts.start) {
     m.scripts.start = 'node server.js'
+    if (!existsSync('server.js')) {
+      throw new PnpmError('NO_SCRIPT_OR_SERVER', 'Missing script start or file server.js')
+    }
   }
   if (opts.args?.length && m.scripts?.[stage]) {
     const escapedArgs = opts.args.map((arg) => JSON.stringify(arg))

exec/lifecycle/test/fixtures/without-scriptstart-serverjs/package.json

@@ -0,0 +1,5 @@
+{
+  "name": "without-scriptstart-serverjs",
+  "version": "1.0.0",
+  "scripts": {}
+}

exec/lifecycle/test/index.ts

@@ -3,6 +3,7 @@ import path from 'path'
 import { runLifecycleHook, runPostinstallHooks } from '@pnpm/lifecycle'
 import loadJsonFile from 'load-json-file'
 import rimraf from '@zkochan/rimraf'
+import { PnpmError } from '@pnpm/error'
 
 const fixtures = path.join(__dirname, 'fixtures')
 const rootModulesDir = path.join(__dirname, '..', 'node_modules')
@@ -51,3 +52,18 @@ test('runPostinstallHooks()', async () => {
 
   expect(loadJsonFile.sync(path.join(pkgRoot, 'output.json'))).toStrictEqual(['preinstall', 'install', 'postinstall'])
 })
+
+test('runLifecycleHook() should throw an error while missing script start or file server.js', async () => {
+  const pkgRoot = path.join(fixtures, 'without-scriptstart-serverjs')
+  const pkg = await import(path.join(pkgRoot, 'package.json'))
+  await expect(
+    runLifecycleHook('start', pkg, {
+      depPath: '/without-scriptstart-serverjs/1.0.0',
+      optional: false,
+      pkgRoot,
+      rawConfig: {},
+      rootModulesDir,
+      unsafePerm: true,
+    })
+  ).rejects.toThrow(new PnpmError('NO_SCRIPT_OR_SERVER', 'Missing script start or file server.js'))
+})

exec/lifecycle/tsconfig.json

@@ -15,6 +15,9 @@
     {
       "path": "../../packages/core-loggers"
     },
+    {
+      "path": "../../packages/error"
+    },
     {
       "path": "../../packages/types"
     },