Skip to content

Commit

Permalink
Omit sources from source maps in bundle.
Browse files Browse the repository at this point in the history
  • Loading branch information
kitsonk authored and ry committed Nov 5, 2018
1 parent e6b3e49 commit 7c76a56
Show file tree
Hide file tree
Showing 6 changed files with 23 additions and 16 deletions.
2 changes: 1 addition & 1 deletion package.json
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@
"flatbuffers": "^1.9.0",
"magic-string": "^0.22.5",
"prettier": "^1.14.0",
"rollup": "^0.63.2",
"rollup": "0.67.0",
"rollup-plugin-alias": "^1.4.0",
"rollup-plugin-analyzer": "^2.1.0",
"rollup-plugin-commonjs": "^9.1.3",
Expand Down
23 changes: 15 additions & 8 deletions rollup.config.js
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
// Copyright 2018 the Deno authors. All rights reserved. MIT license.
import assert from "assert";
// @ts-check
import * as fs from "fs";
import path from "path";
import alias from "rollup-plugin-alias";
Expand Down Expand Up @@ -30,10 +30,14 @@ const tsconfigOverride = {
}
};

// this is a rollup plugin which will look for imports ending with `!string` and resolve
// them with a module that will inline the contents of the file as a string. Needed to
// support `js/assets.ts`.
function strings({ include, exclude } = {}) {
/** this is a rollup plugin which will look for imports ending with `!string` and resolve
* them with a module that will inline the contents of the file as a string. Needed to
* support `js/assets.ts`.
* @param {any} param0
*/
function strings(
{ include, exclude } = { include: undefined, exclude: undefined }
) {
if (!include) {
throw new Error("include option must be passed");
}
Expand Down Expand Up @@ -85,7 +89,9 @@ const osNodeToDeno = {
linux: "linux"
};

// Inject deno.platform.arch and deno.platform.os
/** Inject deno.platform.arch and deno.platform.os
* @param {any} param0
*/
function platform({ include, exclude } = {}) {
if (!include) {
throw new Error("include option must be passed");
Expand Down Expand Up @@ -132,7 +138,7 @@ function resolveGenerated() {
}

function generateDepFile({ outputFile, sourceFiles = [], configFiles = [] }) {
let timestamp = Date.now();
let timestamp = new Date();

// Save the depfile just before the node process exits.
process.once("beforeExit", () =>
Expand Down Expand Up @@ -209,7 +215,8 @@ export default function makeConfig(commandOptions) {
output: {
format: "iife",
name: "denoMain",
sourcemap: true
sourcemap: true,
sourcemapExcludeSources: true
},

plugins: [
Expand Down
4 changes: 2 additions & 2 deletions tests/error_004_missing_module.ts.out
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ NotFound: Cannot resolve module "bad-module.ts" from "[WILDCARD]/tests/error_004
at moduleNames.map.name ([WILDCARD]/js/compiler.ts:[WILDCARD])
at Array.map (<anonymous>)
at DenoCompiler.resolveModuleNames ([WILDCARD]/js/compiler.ts:[WILDCARD])
at Object.compilerHost.resolveModuleNames ([WILDCARD]/third_party/node_modules/typescript/lib/typescript.js:[WILDCARD])
at resolveModuleNamesWorker ([WILDCARD]/third_party/node_modules/typescript/lib/typescript.js:[WILDCARD])
at Object.compilerHost.resolveModuleNames (<anonymous>)
at resolveModuleNamesWorker (<anonymous>)
4 changes: 2 additions & 2 deletions tests/error_005_missing_dynamic_import.ts.out
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ NotFound: Cannot resolve module "bad-module.ts" from "[WILDCARD]/tests/error_005
at moduleNames.map.name ([WILDCARD]/js/compiler.ts:[WILDCARD])
at Array.map (<anonymous>)
at DenoCompiler.resolveModuleNames ([WILDCARD]/js/compiler.ts:[WILDCARD])
at Object.compilerHost.resolveModuleNames ([WILDCARD]/third_party/node_modules/typescript/lib/typescript.js:[WILDCARD])
at resolveModuleNamesWorker ([WILDCARD]/third_party/node_modules/typescript/lib/typescript.js:[WILDCARD])
at Object.compilerHost.resolveModuleNames (<anonymous>)
at resolveModuleNamesWorker (<anonymous>)
4 changes: 2 additions & 2 deletions tests/error_006_import_ext_failure.ts.out
Original file line number Diff line number Diff line change
Expand Up @@ -7,5 +7,5 @@ NotFound: Cannot resolve module "./non-existent" from "[WILDCARD]/tests/error_00
at moduleNames.map.name ([WILDCARD]/js/compiler.ts:[WILDCARD])
at Array.map (<anonymous>)
at DenoCompiler.resolveModuleNames ([WILDCARD]/js/compiler.ts:[WILDCARD])
at Object.compilerHost.resolveModuleNames ([WILDCARD]/third_party/node_modules/typescript/lib/typescript.js:[WILDCARD])
at resolveModuleNamesWorker ([WILDCARD]/third_party/node_modules/typescript/lib/typescript.js:[WILDCARD])
at Object.compilerHost.resolveModuleNames (<anonymous>)
at resolveModuleNamesWorker (<anonymous>)

0 comments on commit 7c76a56

Please sign in to comment.