Remove non-implemented postProcessBundleSourcemap from docs and con…

…fig (#890)

Summary:
Remove docs, type and config default for the non-implemented `postProcessBundleSourcemap`

Changelog:
**[Breaking]** Remove `postProcessBundleSourcemap` from config

*(Breaks consumers relying on the config default being populated with a function.)*

-- Original PR description follows --

As [the issue](#400) mentions the function is not implemented therefore I thought it should be removed from the public docs.

- #400

Pull Request resolved: #890

Test Plan: This is only a docs update no need for testing.

Reviewed By: jacdebug

Differential Revision: D41161760

Pulled By: robhogan

fbshipit-source-id: bd57f9db2192f16afec66dc85a1a6180dda84b89

docs/Configuration.md

@@ -472,12 +472,6 @@ Type: `({platform: ?string}) => $ReadOnlyArray<string>`
 
 An optional list of polyfills to include in the bundle. The list defaults to a set of common polyfills for Number, String, Array, Object...
 
-#### `postProcessBundleSourcemap`
-
-Type: `PostProcessBundleSourcemap`
-
-An optional function that can modify the code and source map of the bundle before it is written. Applied once for the entire bundle.
-
 #### `getModulesRunBeforeMainModule`
 
 Type: `(entryFilePath: string) => Array<string>`

packages/metro-config/src/__tests__/__snapshots__/loadConfig-test.js.snap

@@ -83,7 +83,6 @@ Object {
     "getPolyfills": [Function],
     "getRunModuleStatement": [Function],
     "polyfillModuleNames": Array [],
-    "postProcessBundleSourcemap": [Function],
     "processModuleFilter": [Function],
   },
   "server": Object {
@@ -250,7 +249,6 @@ Object {
     "getPolyfills": [Function],
     "getRunModuleStatement": [Function],
     "polyfillModuleNames": Array [],
-    "postProcessBundleSourcemap": [Function],
     "processModuleFilter": [Function],
   },
   "server": Object {
@@ -417,7 +415,6 @@ Object {
     "getPolyfills": [Function],
     "getRunModuleStatement": [Function],
     "polyfillModuleNames": Array [],
-    "postProcessBundleSourcemap": [Function],
     "processModuleFilter": [Function],
   },
   "server": Object {
@@ -584,7 +581,6 @@ Object {
     "getPolyfills": [Function],
     "getRunModuleStatement": [Function],
     "polyfillModuleNames": Array [],
-    "postProcessBundleSourcemap": [Function],
     "processModuleFilter": [Function],
   },
   "server": Object {

packages/metro-config/src/configTypes.flow.js

@@ -15,7 +15,6 @@ import type {CacheStore} from 'metro-cache';
 import typeof MetroCache from 'metro-cache';
 import type {CacheManagerFactory} from 'metro-file-map';
 import type {CustomResolver} from 'metro-resolver';
-import type {MixedSourceMap} from 'metro-source-map';
 import type {JsTransformerConfig} from 'metro-transform-worker';
 import type {TransformResult} from 'metro/src/DeltaBundler';
 import type {
@@ -27,17 +26,6 @@ import type {
 import type {Reporter} from 'metro/src/lib/reporting';
 import type Server from 'metro/src/Server';
 
-export type PostProcessBundleSourcemap = ({
-  code: Buffer | string,
-  map: MixedSourceMap,
-  outFileName: string,
-  ...
-}) => {
-  code: Buffer | string,
-  map: MixedSourceMap | string,
-  ...
-};
-
 export type ExtraTransformOptions = {
   +preloadedModules?: {[path: string]: true, ...} | false,
   +ramGroups?: Array<string>,
@@ -141,7 +129,6 @@ type SerializerConfigT = {
   getPolyfills: ({platform: ?string, ...}) => $ReadOnlyArray<string>,
   getRunModuleStatement: (number | string) => string,
   polyfillModuleNames: $ReadOnlyArray<string>,
-  postProcessBundleSourcemap: PostProcessBundleSourcemap,
   processModuleFilter: (modules: Module<>) => boolean,
 };
 

packages/metro-config/src/defaults/index.js

@@ -56,7 +56,6 @@ const getDefaultValues = (projectRoot: ?string): ConfigT => ({
     getRunModuleStatement: (moduleId: number | string) =>
       `__r(${JSON.stringify(moduleId)});`,
     getPolyfills: () => [],
-    postProcessBundleSourcemap: ({code, map, outFileName}) => ({code, map}),
     getModulesRunBeforeMainModule: () => [],
     processModuleFilter: module => true,
     createModuleIdFactory: defaultCreateModuleIdFactory,