Wrap non-CssSyntaxError into stylelint results

src/index.js

@@ -38,22 +38,40 @@ module.exports = options => ({
       )
       return extractedCSS
     } catch (e) {
-      // incorrect interpolations will throw CssSyntaxError and they'll be handled by stylelint
+      // Always save the error
+      errorWasThrown[absolutePath] = e
+      // Incorrect interpolations will throw CssSyntaxError and they'll be handled by stylelint
+      // so we can throw it out but not for others
       if (e.name === 'CssSyntaxError') {
-        errorWasThrown[absolutePath] = true
         throw e
       }
       return ''
     }
   },
   // Fix sourcemaps
   result(stylelintResult, filepath) {
-    if (errorWasThrown[filepath]) {
-      // We threw an error ourselves, in this case we have already put correct
-      // line/column numbers so no source maps are needed
-      // (and would actually break the line numbers)
-      delete errorWasThrown[filepath]
-      return stylelintResult
+    const err = errorWasThrown[filepath]
+    if (err) {
+      if (err.name === 'CssSyntaxError') {
+        // We threw an error ourselves, in this case we have already put correct
+        // line/column numbers so no source maps are needed
+        // (and would actually break the line numbers)
+        return stylelintResult
+      } else {
+        // For other errors, wrap them into the result
+        return Object.assign({}, stylelintResult, {
+          errored: true,
+          parseErrors: [
+            {
+              line: err.loc && err.loc.line,
+              column: err.loc && err.loc.column,
+              rule: 'parseError',
+              severity: 'error',
+              text: `${err.message}`
+            }
+          ]
+        })
+      }
     }
     const interpolationLines = interpolationLinesMap[filepath] || []
     const lineCorrection = sourceMapsCorrections[filepath]

test/fixtures/typescript/ts-syntax-parse-error.ts

@@ -0,0 +1,17 @@
+import * as React from 'react';
+import styled, { keyframes } from 'styled-components';
+
+export interface IAccordionContainerProps {
+    className?: string;
+    borderWidth: number,
+    onChange?(id: string): void;
+}
+export interface IAccordionContainerState {
+    selected: string;
+}
+
+export const AccordionContainerDiv = styled.div<{ width#: string }>`
+  border: solid  ${(props: IAccordionContainerProps) => props.borderWidth}px;
+  border-bottom-width: 0;
+  width: auto;
+`;

test/typescript.test.js

@@ -56,6 +56,16 @@ describe('Typescript files, both TS and TSX should parse and report any errors c
     })
   })
 
+  it('should report errors when parsing typescript files with SyntaxError', done => {
+    const fixture = path.join(__dirname, './fixtures/typescript/ts-syntax-parse-error.ts')
+    doLint(fixture, done).then(data => {
+      expect(data.results.length).toEqual(1)
+      expect(data.results[0].parseErrors.length).toEqual(1)
+      expect(data.results[0].parseErrors[0].rule).toEqual('parseError')
+      done()
+    })
+  })
+
   it('should report errors in TSX files(typescript + JSX)', done => {
     const fixture = path.join(__dirname, './fixtures/typescript/ts-syntax-jsx-invalid.tsx')
     doLint(fixture, done).then(data => {