GitHub/storybook
1
0
Fork 0
mirror of https://github.com/storybookjs/storybook.git synced 2025-04-08 11:11:53 +08:00
Code Issues Packages Projects Releases Wiki Activity

Source-loader: Revert sourcemaps

Browse Source
This commit is contained in:
Michael Shilman 2021-03-11 12:33:34 +08:00
parent d051fac86f
commit 9f95a20ec6
2 changed files with 21 additions and 60 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
lib/source-loader/package.json
View File

@ -46,8 +46,7 @@
"loader-utils": "^2.0.0",
"lodash": "^4.17.20",
"prettier": "~2.2.1",
"regenerator-runtime": "^0.13.7",
"source-map": "^0.7.3"
"regenerator-runtime": "^0.13.7"
},
"peerDependencies": {
"react": "^16.8.0 || ^17.0.0",

78
lib/source-loader/src/build.js
View File

@ -1,63 +1,25 @@
import { SourceMapConsumer, SourceNode } from 'source-map';
import { readStory } from './dependencies-lookup/readAsObject';
export async function transform(inputSource, inputSourceMap) {
const callback = this.async();
export async function transform(inputSource) {
const sourceObject = await readStory(this, inputSource);
try {
const sourceObject = await readStory(this, inputSource);
// if source-loader had trouble parsing the story exports, return the original story
// example is
// const myStory = () => xxx
// export { myStory }
if (!sourceObject.source || sourceObject.source.length === 0) {
callback(null, inputSource, inputSourceMap);
return;
}
const { source, sourceJson, addsMap } = sourceObject;
// Use the SourceNode to produce the code. Given that the source mapping here is trivial it's easier to just
// always build a sourcemap rather than to have two different code paths.
let sourceNode;
if (inputSourceMap) {
// The inputSourceMap here should be a SourceMapGenerator, or just a plain source map JSON object.
sourceNode = await SourceMapConsumer.with(
JSON.stringify(inputSourceMap),
null,
(consumer) => {
return SourceNode.fromStringWithSourceMap(source, consumer);
}
);
} else {
// Build an identity sourcemap. Note that "source" is already potentially differing from "inputSource"
// due to other loaders, so while we need to use "source" for the source node contents to generate the correct
// code, we still want to use "inputSource" as the source content.
sourceNode = new SourceNode();
sourceNode.add(
source
.split(/\n/)
.map((line, index) => new SourceNode(index + 1, 0, this.resourcePath, `${line}\n`))
);
sourceNode.setSourceContent(this.resourcePath, inputSource);
}
// Prepend the preamble
const preamble = `
/* eslint-disable */
// @ts-nocheck
// @ts-ignore
var __STORY__ = ${sourceJson};
// @ts-ignore
var __LOCATIONS_MAP__ = ${JSON.stringify(addsMap)};`;
sourceNode.prepend(`${preamble}\n`);
// Generate the code and the source map for the next loader
const { code, map } = sourceNode.toStringWithSourceMap();
callback(null, code, map);
} catch (e) {
callback(e);
// if source-loader had trouble parsing the story exports, return the original story
// example is
// const myStory = () => xxx
// export { myStory }
if (!sourceObject.source || sourceObject.source.length === 0) {
return inputSource;
}
const { source, sourceJson, addsMap } = sourceObject;
const preamble = `
/* eslint-disable */
// @ts-nocheck
// @ts-ignore
var __STORY__ = ${sourceJson};
// @ts-ignore
var __LOCATIONS_MAP__ = ${JSON.stringify(addsMap)};
`;
return `${preamble}\n${source}`;
}