fix: access JSON output under package name key in stage tests

The logTar utility wraps JSON output under the package name key,
so tests need to access out[pkg] instead of out directly.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>

Author: reggi

smoke-tests/tap-snapshots/test/index.js.test.cjs

@@ -27,7 +27,7 @@ All commands:
     init, install, install-ci-test, install-test, link, ll,
     login, logout, ls, org, outdated, owner, pack, ping, pkg,
     prefix, profile, prune, publish, query, rebuild, repo,
-    restart, root, run, sbom, search, set, shrinkwrap, start,
+    restart, root, run, sbom, search, set, shrinkwrap, stage, start,
     stop, team, test, token, trust, undeprecate, uninstall,
     unpublish, update, version, view, whoami
 

tap-snapshots/smoke-tests/test/index.js.test.cjs

@@ -0,0 +1,14 @@
+/* IMPORTANT
+ * This snapshot file is auto-generated, but designed for humans.
+ * It should be checked into source control and tracked carefully.
+ * Re-generate by setting TAP_SNAPSHOT=1 and running tests.
+ * Make sure to inspect the output below.  Do not ignore changes!
+ */
+'use strict'
+exports[`smoke-tests/test/index.js TAP basic npm (no args) > should have expected no args output 1`] = `
+
+`
+
+exports[`smoke-tests/test/index.js TAP basic npm outdated > should have expected outdated output 1`] = `
+
+`

tap-snapshots/test/lib/commands/publish.js.test.cjs

@@ -205,6 +205,7 @@ Object {
     "man/man1/npm-search.1",
     "man/man1/npm-set.1",
     "man/man1/npm-shrinkwrap.1",
+    "man/man1/npm-stage.1",
     "man/man1/npm-start.1",
     "man/man1/npm-stop.1",
     "man/man1/npm-team.1",
@@ -256,7 +257,7 @@ exports[`test/lib/commands/publish.js TAP no auth dry-run > must match snapshot
 
 exports[`test/lib/commands/publish.js TAP no auth dry-run > warns about auth being needed 1`] = `
 Array [
-  "This command requires you to be logged in to https://registry.npmjs.org/ (dry-run)",
+  "publish This command requires you to be logged in to https://registry.npmjs.org/ (dry-run)",
 ]
 `
 

tap-snapshots/test/lib/docs.js.test.cjs

@@ -147,6 +147,7 @@ Array [
   "search",
   "set",
   "shrinkwrap",
+  "stage",
   "start",
   "stop",
   "team",
@@ -5487,6 +5488,61 @@ Note: This command is unaware of workspaces.
 NO PARAMS
 `
 
+exports[`test/lib/docs.js TAP usage stage > must match snapshot 1`] = `
+Stage packages for publishing, deferring proof-of-presence (2FA) to a later point in time
+
+Usage:
+npm stage
+npm stage publish <package-spec>
+npm stage list [<package-spec>]
+npm stage view <stage-id>
+npm stage approve <stage-id>
+npm stage reject <stage-id>
+npm stage download <stage-id>
+
+Subcommands:
+  publish
+    Stage a package for publishing, deferring proof-of-presence (2FA) to a later point in time
+
+  list
+    List all staged package versions
+
+  view
+    View details of a specific staged package
+
+  approve
+    Approve a staged package, publishing it to the npm registry
+
+  reject
+    Reject a staged package, removing it from the registry
+
+  download
+    Download the tarball of a staged package for inspection
+
+Run "npm stage <subcommand> --help" for more info on a subcommand.
+
+Run "npm help stage" for more info
+
+\`\`\`bash
+npm stage
+\`\`\`
+
+Note: This command is unaware of workspaces.
+
+#### Synopsis
+#### Flags
+#### Synopsis
+#### Flags
+#### Synopsis
+#### Flags
+#### Synopsis
+#### Flags
+#### Synopsis
+#### Flags
+#### Synopsis
+#### Flags
+`
+
 exports[`test/lib/docs.js TAP usage start > must match snapshot 1`] = `
 Start a package
 

tap-snapshots/test/lib/npm.js.test.cjs

@@ -37,9 +37,9 @@ All commands:
     init, install, install-ci-test, install-test, link, ll,
     login, logout, ls, org, outdated, owner, pack, ping, pkg,
     prefix, profile, prune, publish, query, rebuild, repo,
-    restart, root, run, sbom, search, set, shrinkwrap, start,
-    stop, team, test, token, trust, undeprecate, uninstall,
-    unpublish, update, version, view, whoami
+    restart, root, run, sbom, search, set, shrinkwrap, stage,
+    start, stop, team, test, token, trust, undeprecate,
+    uninstall, unpublish, update, version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -86,12 +86,12 @@ All commands:
     query, rebuild, repo,
     restart, root, run,
     sbom, search, set,
-    shrinkwrap, start, stop,
-    team, test, token,
-    trust, undeprecate,
-    uninstall, unpublish,
-    update, version, view,
-    whoami
+    shrinkwrap, stage,
+    start, stop, team, test,
+    token, trust,
+    undeprecate, uninstall,
+    unpublish, update,
+    version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -138,12 +138,12 @@ All commands:
     query, rebuild, repo,
     restart, root, run,
     sbom, search, set,
-    shrinkwrap, start, stop,
-    team, test, token,
-    trust, undeprecate,
-    uninstall, unpublish,
-    update, version, view,
-    whoami
+    shrinkwrap, stage,
+    start, stop, team, test,
+    token, trust,
+    undeprecate, uninstall,
+    unpublish, update,
+    version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -177,9 +177,9 @@ All commands:
     init, install, install-ci-test, install-test, link, ll,
     login, logout, ls, org, outdated, owner, pack, ping, pkg,
     prefix, profile, prune, publish, query, rebuild, repo,
-    restart, root, run, sbom, search, set, shrinkwrap, start,
-    stop, team, test, token, trust, undeprecate, uninstall,
-    unpublish, update, version, view, whoami
+    restart, root, run, sbom, search, set, shrinkwrap, stage,
+    start, stop, team, test, token, trust, undeprecate,
+    uninstall, unpublish, update, version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -226,12 +226,12 @@ All commands:
     query, rebuild, repo,
     restart, root, run,
     sbom, search, set,
-    shrinkwrap, start, stop,
-    team, test, token,
-    trust, undeprecate,
-    uninstall, unpublish,
-    update, version, view,
-    whoami
+    shrinkwrap, stage,
+    start, stop, team, test,
+    token, trust,
+    undeprecate, uninstall,
+    unpublish, update,
+    version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -278,12 +278,12 @@ All commands:
     query, rebuild, repo,
     restart, root, run,
     sbom, search, set,
-    shrinkwrap, start, stop,
-    team, test, token,
-    trust, undeprecate,
-    uninstall, unpublish,
-    update, version, view,
-    whoami
+    shrinkwrap, stage,
+    start, stop, team, test,
+    token, trust,
+    undeprecate, uninstall,
+    unpublish, update,
+    version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -329,8 +329,8 @@ All commands:
     query, rebuild, repo,
     restart, root, run, sbom,
     search, set, shrinkwrap,
-    start, stop, team, test,
-    token, trust,
+    stage, start, stop, team,
+    test, token, trust,
     undeprecate, uninstall,
     unpublish, update,
     version, view, whoami
@@ -368,7 +368,7 @@ All commands:
     link, ll, login, logout, ls, org, outdated, owner, pack,
     ping, pkg, prefix, profile, prune, publish, query, rebuild,
     repo, restart, root, run, sbom, search, set, shrinkwrap,
-    start, stop, team, test, token, trust, undeprecate,
+    stage, start, stop, team, test, token, trust, undeprecate,
     uninstall, unpublish, update, version, view, whoami
 
 Specify configs in the ini-formatted file:
@@ -403,9 +403,9 @@ All commands:
     init, install, install-ci-test, install-test, link, ll,
     login, logout, ls, org, outdated, owner, pack, ping, pkg,
     prefix, profile, prune, publish, query, rebuild, repo,
-    restart, root, run, sbom, search, set, shrinkwrap, start,
-    stop, team, test, token, trust, undeprecate, uninstall,
-    unpublish, update, version, view, whoami
+    restart, root, run, sbom, search, set, shrinkwrap, stage,
+    start, stop, team, test, token, trust, undeprecate,
+    uninstall, unpublish, update, version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}
@@ -439,9 +439,9 @@ All commands:
     init, install, install-ci-test, install-test, link, ll,
     login, logout, ls, org, outdated, owner, pack, ping, pkg,
     prefix, profile, prune, publish, query, rebuild, repo,
-    restart, root, run, sbom, search, set, shrinkwrap, start,
-    stop, team, test, token, trust, undeprecate, uninstall,
-    unpublish, update, version, view, whoami
+    restart, root, run, sbom, search, set, shrinkwrap, stage,
+    start, stop, team, test, token, trust, undeprecate,
+    uninstall, unpublish, update, version, view, whoami
 
 Specify configs in the ini-formatted file:
     {USERCONFIG}

test/lib/commands/stage/index.js

@@ -58,8 +58,8 @@ t.test('stages with --json', async t => {
   registry.nock.post('/-/stage/package/@npmcli%2ftest-package').reply(201, {})
   await npm.exec('stage', ['publish'])
   const out = JSON.parse(joinedOutput())
-  t.equal(out.name, pkg)
-  t.equal(out.version, '1.0.0')
+  t.equal(out[pkg].name, pkg)
+  t.equal(out[pkg].version, '1.0.0')
 })
 
 t.test('stages with --json includes stageId', async t => {
@@ -78,8 +78,8 @@ t.test('stages with --json includes stageId', async t => {
   registry.nock.post('/-/stage/package/@npmcli%2ftest-package').reply(201, { stageId })
   await npm.exec('stage', ['publish'])
   const out = JSON.parse(joinedOutput())
-  t.equal(out.name, pkg)
-  t.equal(out.stageId, stageId)
+  t.equal(out[pkg].name, pkg)
+  t.equal(out[pkg].stageId, stageId)
 })
 
 t.test('throws on invalid semver tag', async t => {