fix: normalize source map file path on windows (#83)

* chore: upgrade packages, cleanup type errors

Took the opportunity to update TS to latest version, cleanup all type errors.

* fix: normalize source map file path on windows

Updated test snapshot file.
1. Sources paths was cleaned up by upgrading component-compiler-utils.
2. Styles mapping changes are due to newer version of postcss (from component-compiler-utils).
3. postcss rawResult is removed.
 * The rawResult contains ast tree which postcss always prepends current working directory to referencing file name.
 * This is a main pain point of maintaining this snapshot test.
 * The correctness of rawResult is responsibility of postcss, not vue compiler. So it's irrelevant to test it here.

This is a companion PR for vuejs/component-compiler-utils#51

package.json

@@ -47,7 +47,7 @@
  "rollup-plugin-image": "^1.0.2",
  "rollup-plugin-node-resolve": "^3.3.0",
  "ts-jest": "^22.4.2",
- "typescript": "^2.7.2",
+ "typescript": "^3.2.4",
  "typescript-eslint-parser": "^15.0.0",
  "vue": "^2.5.16",
  "vue-template-compiler": "^2.5.16"
@@ -57,7 +57,7 @@
  "vue-template-compiler": "*"
  },
  "dependencies": {
- "@vue/component-compiler-utils": "^2.1.0",
+ "@vue/component-compiler-utils": "^2.5.2",
  "clean-css": "^4.1.11",
  "hash-sum": "^1.0.2",
  "postcss-modules-sync": "^1.0.0",

src/assembler.ts

@@ -74,7 +74,7 @@ export function assembleFromSource(
  script = script || { source: 'export default {}' }
  template = template || { source: '' }
  let map = undefined
- const mapGenerator = new SourceMapGenerator({ file: filename })
+ const mapGenerator = new SourceMapGenerator({ file: filename.replace(/\\/g, '/') })
 
  const hasScopedStyle = styles.some(style => style.scoped === true)
  const hasStyle = styles.some(style => style.source || style.module)

src/compiler.ts

@@ -99,20 +99,23 @@ export class SFCCompiler {
  ? hash(path.basename(filename) + source)
  : hash(filename + source))
 
- const template =
- descriptor.template && this.compileTemplate(filename, descriptor.template)
+ const template = descriptor.template
+ ? this.compileTemplate(filename, descriptor.template)
+ : undefined
 
  const styles = descriptor.styles.map(style =>
  this.compileStyle(filename, scopeId, style)
  )
 
  const { script: rawScript, customBlocks } = descriptor
- const script = rawScript && {
- code: rawScript.src
- ? this.read(rawScript.src, filename)
- : rawScript.content,
- map: rawScript.map
- }
+ const script = rawScript
+ ? {
+ code: rawScript.src
+ ? this.read(rawScript.src, filename)
+ : rawScript.content,
+ map: rawScript.map
+ }
+ : undefined
 
  return {
  scopeId,
@@ -140,8 +143,9 @@ export class SFCCompiler {
  ? hash(path.basename(filename) + source)
  : hash(filename + source))
 
- const template =
- descriptor.template && this.compileTemplate(filename, descriptor.template)
+ const template = descriptor.template
+ ? this.compileTemplate(filename, descriptor.template)
+ : undefined
 
  const styles = await Promise.all(
  descriptor.styles.map(style =>
@@ -150,12 +154,14 @@ export class SFCCompiler {
  )
 
  const { script: rawScript, customBlocks } = descriptor
- const script = rawScript && {
- code: rawScript.src
- ? this.read(rawScript.src, filename)
- : rawScript.content,
- map: rawScript.map
- }
+ const script = rawScript
+ ? {
+ code: rawScript.src
+ ? this.read(rawScript.src, filename)
+ : rawScript.content,
+ map: rawScript.map
+ }
+ : undefined
 
  return {
  scopeId,
@@ -260,7 +266,7 @@ export class SFCCompiler {
  },
 
  prepare: result => ({
- media: style.attrs.media,
+ media: typeof style.attrs.media === 'string' ? style.attrs.media : undefined,
  scoped: style.scoped,
  moduleName: style.module === true ? '$style' : <any>style.module,
  module: tokens,