fix: skip dist check and guard ow.package when packages is empty

When a manifest declares `packages: {}` the build step produces no
output so the dist directory is never created. The existing dist check
was throwing "missing files in dist/…, maybe you forgot to build your
actions?" even though there was nothing to build.

* deploy-actions.js: gate the dist check on `hasAnyActions` — only
  validate the build directory when at least one package actually
  defines actions. Two tests cover both branches of `pkg.actions || {}`
  (empty packages and package with no actions key), keeping 100%
  branch coverage on deploy-actions.js.
* utils.js (replacePackagePlaceHolder): guard the `packageNames[0]`
  assignment so an empty packages object no longer clobbers `ow.package`
  with `undefined`.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: iivvaannxx

src/deploy-actions.js

@@ -57,9 +57,12 @@ async function deployActions (config, deployConfig = {}, logFunc) {
   // checks
   // a. missing credentials
   utils.checkOpenWhiskCredentials(config)
-  // b. missing build files
+  // b. missing build files — only required when at least one package defines actions
   const dist = config.actions.dist
+  const hasAnyActions = Object.values(config.manifest.full.packages)
+    .some(pkg => Object.keys(pkg.actions || {}).length > 0)
   if (
+    hasAnyActions &&
     (!deployConfig.filterEntities || deployConfig.filterEntities.actions) &&
     (!fs.pathExistsSync(dist) || !fs.lstatSync(dist).isDirectory() || !fs.readdirSync(dist).length === 0)
   ) {

src/utils.js

@@ -2104,7 +2104,9 @@ function replacePackagePlaceHolder (config) {
     // Using custom package name.
     // Set config.ow.package so that syncProject can use it as project name for annotations.
     const packageNames = Object.keys(packages)
-    modifiedConfig.ow.package = packageNames[0]
+    if (packageNames.length > 0) {
+      modifiedConfig.ow.package = packageNames[0]
+    }
   }
   return modifiedConfig
 }

test/deploy.actions.test.js

@@ -583,6 +583,20 @@ test('Deploy actions should pass if there are no build files and filter does not
   await expect(deployActions(global.sampleAppConfig, { filterEntities: { triggers: ['trigger1'] } })).resolves.toEqual({})
 })
 
+test('Deploy actions should succeed when packages: {} (empty packages, no actions defined)', async () => {
+  const emptyPackagesConfig = deepCopy(global.sampleAppConfig)
+  emptyPackagesConfig.manifest.full.packages = {}
+  runtimeLibUtils.processPackage.mockReturnValue(deepCopy(mockEntities))
+  await expect(deployActions(emptyPackagesConfig)).resolves.toBeDefined()
+})
+
+test('Deploy actions should succeed when a package has no actions key (pkg.actions || {} guard)', async () => {
+  const noActionsPkgConfig = deepCopy(global.sampleAppConfig)
+  noActionsPkgConfig.manifest.full.packages = { emptyPkg: {} }
+  runtimeLibUtils.processPackage.mockReturnValue(deepCopy(mockEntities))
+  await expect(deployActions(noActionsPkgConfig)).resolves.toBeDefined()
+})
+
 // lonely
 test('if actions are deployed and part of the manifest it should return their url', async () => {
   addSampleAppReducedFiles()

test/utils.test.js

@@ -3097,3 +3097,43 @@ describe('loadIMSCredentialsFromEnv', () => {
     expect(result.scopes).toBe('not json')
   })
 })
+
+describe('replacePackagePlaceHolder', () => {
+  test('leaves ow.package unchanged when packages is empty (packages: {})', () => {
+    const config = {
+      ow: { package: 'my-pkg' },
+      manifest: {
+        packagePlaceholder: '__APP_PACKAGE__',
+        full: { packages: {} }
+      }
+    }
+    const result = utils.replacePackagePlaceHolder(config)
+    expect(result.ow.package).toBe('my-pkg')
+  })
+
+  test('renames placeholder package to ow.package', () => {
+    const config = {
+      ow: { package: 'my-pkg' },
+      manifest: {
+        packagePlaceholder: '__APP_PACKAGE__',
+        full: { packages: { __APP_PACKAGE__: { actions: {} } } }
+      }
+    }
+    const result = utils.replacePackagePlaceHolder(config)
+    expect(result.ow.package).toBe('my-pkg')
+    expect(result.manifest.full.packages['my-pkg']).toBeDefined()
+    expect(result.manifest.full.packages.__APP_PACKAGE__).toBeUndefined()
+  })
+
+  test('sets ow.package to first package name when no placeholder matches', () => {
+    const config = {
+      ow: { package: 'ignored' },
+      manifest: {
+        packagePlaceholder: '__APP_PACKAGE__',
+        full: { packages: { 'custom-pkg': { actions: {} } } }
+      }
+    }
+    const result = utils.replacePackagePlaceHolder(config)
+    expect(result.ow.package).toBe('custom-pkg')
+  })
+})