add --partial option

Author: ds300

README.md

@@ -107,15 +107,17 @@ Then, update your hash key to include a checksum of that file:
 ```yaml
 - restore_cache:
  key:
- app-node_modules-v1-{{ checksum "yarn.lock" }}-{{ checksum "patches.hash" }}
+ app-node_modules-v1-{{ checksum "yarn.lock" }}-{{ checksum "patches.hash"
+ }}
 ```
 
 As well as the save_cache
 
 ```yaml
 - save_cache:
  key:
- app-node_modules-v1-{{ checksum "yarn.lock" }}-{{ checksum "patches.hash" }}
+ app-node_modules-v1-{{ checksum "yarn.lock" }}-{{ checksum "patches.hash"
+ }}
  paths:
  - ./node_modules
 ```
@@ -342,6 +344,21 @@ If you deleted one of the patch files other than the last one, you don't need to
 update the sequence numbers in the successive patch file names, but you might
 want to do so to keep things tidy.
 
+#### Partially applying a broken patch file
+
+Normally patch application is atomic per patch file. i.e. if a patch file
+contains an error anywhere then none of the changes in the patch file will be
+applied and saved to disk.
+
+This can be problematic if you have a patch with many changes and you want to
+keep some of them and update others.
+
+In this case you can use the `--partial` option. Patch-package will apply as
+many of the changes as it can and then leave it to you to fix the rest.
+
+Any errors encountered will be written to a file `./patch-package-errors.log` to
+help you keep track of what needs fixing.
+
 ## Benefits of patching over forking
 
 - Sometimes forks need extra build steps, e.g. with react-native for Android.

integration-tests/partial-apply/__snapshots__/partial-apply.test.ts.snap

@@ -0,0 +1,50 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`Test partial-apply: 00: patch-package fails when one of the patches in the sequence fails 1`] = `
+"SNAPSHOT: patch-package fails when one of the patches in the sequence fails
+patch-package 0.0.0
+Applying patches...
+left-pad@1.3.0 (1 hello) ✔
+
+⛔ ERROR
+
+Failed to apply patch file left-pad+1.3.0+002+world.patch.
+
+If this patch file is no longer useful, delete it and run
+
+ patch-package
+
+To partially apply the patch (if possible) and output a log of errors to fix, run
+
+ patch-package --partial
+
+After which you should make any required changes inside node_modules/left-pad, and finally run
+
+ patch-package left-pad
+
+to update the patch file.
+
+END SNAPSHOT"
+`;
+
+exports[`Test partial-apply: 01: patch-package --partial saves a log 1`] = `
+"SNAPSHOT: patch-package --partial saves a log
+patch-package 0.0.0
+Applying patches...
+left-pad@1.3.0 (1 hello) ✔
+Saving errors to ./patch-package-errors.log
+patch-package-errors.log
+Cannot apply hunk 0 for file node_modules/left-pad/index.js
+\`\`\`diff
+@@ -3,7 +3,7 @@
+ * and/or modify it under the terms of the Do What The Fuck You Want
+ * To Public License, Version 2, as published by Sam Hocevar. See
+ * http://www.wtfpl.net/ for more details. */
+-'use oops';
++'use world';
+ module.exports = leftPad;
+ 
+ var cache = [
+\`\`\`
+END SNAPSHOT"
+`;

integration-tests/partial-apply/package-lock.json

@@ -0,0 +1,11 @@
+{
+ "name": "partial-apply",
+ "version": "1.0.0",
+ "description": "integration test for patch-package",
+ "main": "index.js",
+ "author": "",
+ "license": "ISC",
+ "dependencies": {
+ "left-pad": "^1.3.0"
+ }
+}

integration-tests/partial-apply/package.json

@@ -0,0 +1,26 @@
+#!/bin/bash
+# make sure errors stop the script
+set -e
+
+npm install
+
+echo "add patch-package"
+npm add $1
+
+function patch-package {
+ ./node_modules/.bin/patch-package "$@"
+}
+
+echo "SNAPSHOT: patch-package fails when one of the patches in the sequence fails"
+if patch-package
+then
+ exit 1
+fi
+echo "END SNAPSHOT"
+
+
+echo "SNAPSHOT: patch-package --partial saves a log"
+patch-package --partial
+echo 'patch-package-errors.log'
+cat patch-package-errors.log
+echo "END SNAPSHOT"

integration-tests/partial-apply/partial-apply.sh

@@ -0,0 +1,5 @@
+import { runIntegrationTest } from "../runIntegrationTest"
+runIntegrationTest({
+ projectName: "partial-apply",
+ shouldProduceSnapshots: true,
+})

integration-tests/partial-apply/partial-apply.test.ts

@@ -0,0 +1,13 @@
+diff --git a/node_modules/left-pad/index.js b/node_modules/left-pad/index.js
+index e90aec3..1a2ec5f 100644
+--- a/node_modules/left-pad/index.js
++++ b/node_modules/left-pad/index.js
+@@ -3,7 +3,7 @@
+ * and/or modify it under the terms of the Do What The Fuck You Want
+ * To Public License, Version 2, as published by Sam Hocevar. See
+ * http://www.wtfpl.net/ for more details. */
+-'use strict';
++'use hello';
+ module.exports = leftPad;
+ 
+ var cache = [

integration-tests/partial-apply/patches/left-pad+1.3.0+001+hello.patch

@@ -0,0 +1,13 @@
+diff --git a/node_modules/left-pad/index.js b/node_modules/left-pad/index.js
+index 1a2ec5f..5aa41be 100644
+--- a/node_modules/left-pad/index.js
++++ b/node_modules/left-pad/index.js
+@@ -3,7 +3,7 @@
+ * and/or modify it under the terms of the Do What The Fuck You Want
+ * To Public License, Version 2, as published by Sam Hocevar. See
+ * http://www.wtfpl.net/ for more details. */
+-'use oops';
++'use world';
+ module.exports = leftPad;
+ 
+ var cache = [

integration-tests/partial-apply/patches/left-pad+1.3.0+002+world.patch

@@ -0,0 +1,13 @@
+diff --git a/node_modules/left-pad/index.js b/node_modules/left-pad/index.js
+index 5aa41be..5ee751b 100644
+--- a/node_modules/left-pad/index.js
++++ b/node_modules/left-pad/index.js
+@@ -3,7 +3,7 @@
+ * and/or modify it under the terms of the Do What The Fuck You Want
+ * To Public License, Version 2, as published by Sam Hocevar. See
+ * http://www.wtfpl.net/ for more details. */
+-'use world';
++'goodbye world';
+ module.exports = leftPad;
+ 
+ var cache = [

integration-tests/partial-apply/patches/left-pad+1.3.0+003+goodbye.patch

@@ -132,7 +132,7 @@ export function executeTestCase(testCase: TestCase) {
  fs.setWorkingFiles({ ...testCase.cleanFiles })
  reportingFailures(() => {
  const effects = parsePatchFile(patchFileContents)
- executeEffects(effects, { dryRun: false })
+ executeEffects(effects, { dryRun: false, bestEffort: false })
  expect(fs.getWorkingFiles()).toEqual(testCase.modifiedFiles)
  })
  })
@@ -141,7 +141,7 @@ export function executeTestCase(testCase: TestCase) {
  fs.setWorkingFiles({ ...testCase.modifiedFiles })
  reportingFailures(() => {
  const effects = reversePatch(parsePatchFile(patchFileContents))
- executeEffects(effects, { dryRun: false })
+ executeEffects(effects, { dryRun: false, bestEffort: false })
  expect(fs.getWorkingFiles()).toEqual(testCase.cleanFiles)
  })
  })