feat: "pnpm publish" supports package.{json5,yaml}

ref #1803

packages/pnpm/package.json

@@ -84,7 +84,8 @@
     "text-table": "0.2.0",
     "tree-kill": "1.2.1",
     "update-notifier": "2.5.0",
-    "version-selector-type": "2.0.1"
+    "version-selector-type": "2.0.1",
+    "write-json-file": "3.2.0"
   },
   "devDependencies": {
     "@pnpm/assert-project": "link:../../privatePackages/assert-project",
@@ -127,7 +128,6 @@
     "ts-node": "8.1.0",
     "tslint": "5.16.0",
     "typescript": "3.4.5",
-    "write-json-file": "3.2.0",
     "write-pkg": "4.0.0",
     "write-yaml-file": "2.0.0"
   },

packages/pnpm/src/bin/pnpm.ts

@@ -41,7 +41,6 @@ if (argv.includes('--help') || argv.includes('-h') || argv.includes('--h')) {
     case 'ping':
     case 'prefix':
     case 'profile':
-    case 'publish':
     case 'repo':
     case 's':
     case 'se':

packages/pnpm/src/cmd/help.ts

@@ -215,6 +215,13 @@ function getHelpText (command: string) {
         Removes extraneous packages
       `
 
+    case 'publish':
+      return stripIndent`
+        pnpm publish [<tarball>|<folder>] [--tag <tag>] [--access <public|restricted>]
+
+        Publishes a package to the npm registry.
+      `
+
     case 'install-test':
       return stripIndent`
         pnpm install-test
@@ -474,6 +481,7 @@ function getHelpText (command: string) {
           - list
           - outdated
           - prune
+          - publish
           - rebuild
           - restart
           - root

packages/pnpm/src/cmd/index.ts

@@ -6,6 +6,7 @@ import link from './link'
 import list from './list'
 import outdated from './outdated'
 import prune from './prune'
+import publish from './publish'
 import rebuild from './rebuild'
 import recursive from './recursive'
 import root from './root'
@@ -25,6 +26,7 @@ export default {
   list,
   outdated,
   prune,
+  publish,
   rebuild,
   recursive,
   restart,

packages/pnpm/src/cmd/publish.ts

@@ -0,0 +1,27 @@
+import readImporterManifest from '@pnpm/read-importer-manifest'
+import path = require('path')
+import rimraf = require('rimraf-then')
+import writeJsonFile = require('write-json-file')
+import { PnpmOptions } from '../types'
+import runNpm from './runNpm'
+
+export default async function (
+  args: string[],
+  opts: PnpmOptions,
+  command: string,
+) {
+  if (args.length && args[0].endsWith('.tgz')) {
+    await runNpm(['publish', ...args])
+    return
+  }
+  const prefix = args.length && args[0] || process.cwd()
+  const { fileName, manifest, writeImporterManifest } = await readImporterManifest(prefix)
+  if (fileName !== 'package.json') {
+    await writeJsonFile(path.join(prefix, 'package.json'), manifest)
+  }
+  await runNpm(['publish', ...args])
+  if (fileName !== 'package.json') {
+    await rimraf(path.join(prefix, 'package.json'))
+    await writeImporterManifest(manifest)
+  }
+}

packages/pnpm/src/main.ts

@@ -38,6 +38,7 @@ type CANONICAL_COMMAND_NAMES = 'help'
   | 'list'
   | 'outdated'
   | 'prune'
+  | 'publish'
   | 'rebuild'
   | 'recursive'
   | 'restart'
@@ -60,6 +61,7 @@ const supportedCmds = new Set<CANONICAL_COMMAND_NAMES>([
   'update',
   'link',
   'prune',
+  'publish',
   'install-test',
   'restart',
   'server',

packages/pnpm/test/index.ts

@@ -7,6 +7,7 @@ import './list'
 import './monorepo'
 import './outdated'
 import './prune'
+import './publish'
 import './rebuild'
 import './recursive'
 import './root'

packages/pnpm/test/publish.ts

@@ -0,0 +1,53 @@
+import {
+  prepareWithYamlManifest,
+} from '@pnpm/prepare'
+import tape = require('tape')
+import promisifyTape from 'tape-promise'
+import { execPnpm } from './utils'
+
+const test = promisifyTape(tape)
+const testOnly = promisifyTape(tape.only)
+
+const CREDENTIALS = [
+  '--//localhost:4873/:username=username',
+  `--//localhost:4873/:_password=${Buffer.from('password').toString('base64')}`,
+  '--//localhost:4873/:email=foo@bar.net',
+]
+
+test('publish: package with package.json', async (t: tape.Test) => {
+  prepareWithYamlManifest(t, {
+    name: 'test-publish-package.json',
+    version: '0.0.0',
+  })
+
+  await execPnpm('publish', ...CREDENTIALS)
+})
+
+test('publish: package with package.yaml', async (t: tape.Test) => {
+  prepareWithYamlManifest(t, {
+    name: 'test-publish-package.yaml',
+    version: '0.0.0',
+  })
+
+  await execPnpm('publish', ...CREDENTIALS)
+})
+
+test('publish: package with package.json5', async (t: tape.Test) => {
+  prepareWithYamlManifest(t, {
+    name: 'test-publish-package.json5',
+    version: '0.0.0',
+  })
+
+  await execPnpm('publish', ...CREDENTIALS)
+})
+
+test('publish: package with package.json5 running publish from different folder', async (t: tape.Test) => {
+  prepareWithYamlManifest(t, {
+    name: 'test-publish-package.json5',
+    version: '0.0.1',
+  })
+
+  process.chdir('..')
+
+  await execPnpm('publish', 'project', ...CREDENTIALS)
+})

packages/read-importer-manifest/src/index.ts

@@ -15,12 +15,14 @@ import {
 const stat = promisify(fs.stat)
 
 export default async function readImporterManifest (importerDir: string): Promise<{
+  fileName: string,
   manifest: ImporterManifest
   writeImporterManifest: (manifest: ImporterManifest) => Promise<void>
 }> {
   const result = await tryReadImporterManifest(importerDir)
   if (result.manifest !== null) {
     return result as {
+      fileName: string,
       manifest: ImporterManifest
       writeImporterManifest: (manifest: ImporterManifest) => Promise<void>
     }
@@ -36,6 +38,7 @@ export async function readImporterManifestOnly (importerDir: string): Promise<Im
 }
 
 export async function tryReadImporterManifest (importerDir: string): Promise<{
+  fileName: string,
   manifest: ImporterManifest | null
   writeImporterManifest: (manifest: ImporterManifest) => Promise<void>
 }> {
@@ -44,6 +47,7 @@ export async function tryReadImporterManifest (importerDir: string): Promise<{
     const { data, text } = await readJsonFile(manifestPath)
     const { indent } = detectIndent(text)
     return {
+      fileName: 'package.json',
       manifest: data,
       writeImporterManifest: createManifestWriter({
         indent,
@@ -59,6 +63,7 @@ export async function tryReadImporterManifest (importerDir: string): Promise<{
     const { data, text } = await readJson5File(manifestPath)
     const { indent } = detectIndent(text)
     return {
+      fileName: 'package.json5',
       manifest: data,
       writeImporterManifest: createManifestWriter({
         indent,
@@ -73,6 +78,7 @@ export async function tryReadImporterManifest (importerDir: string): Promise<{
     const manifestPath = path.join(importerDir, 'package.yaml')
     const manifest = await readPackageYaml(manifestPath)
     return {
+      fileName: 'package.yaml',
       manifest,
       writeImporterManifest: createManifestWriter({ initialManifest: manifest, manifestPath }),
     }
@@ -95,6 +101,7 @@ export async function tryReadImporterManifest (importerDir: string): Promise<{
   }
   const filePath = path.join(importerDir, 'package.json')
   return {
+    fileName: 'package.json',
     manifest: null,
     writeImporterManifest: writeImporterManifest.bind(null, filePath),
   }