feat(typescript-estree): add fragment options

yoyo930021 · yoyo930021 · commit db796149a319 · 2019-11-06T19:17:42.000+08:00
diff --git a/packages/experimental-utils/src/ts-eslint/ParserOptions.ts b/packages/experimental-utils/src/ts-eslint/ParserOptions.ts
@@ -19,4 +19,5 @@ export interface ParserOptions {
   tsconfigRootDir?: string;
   useJSXTextNode?: boolean;
   warnOnUnsupportedTypeScriptVersion?: boolean;
+  fragment?: boolean;
 }
diff --git a/packages/parser/src/parser.ts b/packages/parser/src/parser.ts
@@ -66,6 +66,9 @@ export function parseForESLint(
   if (typeof options.ecmaFeatures !== 'object') {
     options.ecmaFeatures = {};
   }
+  if (!('fragment' in options)) {
+    options.fragment = false;
+  }
 
   const parserOptions: TSESTreeOptions = {};
   Object.assign(parserOptions, options, {
diff --git a/packages/typescript-estree/src/parser-options.ts b/packages/typescript-estree/src/parser-options.ts
@@ -20,6 +20,7 @@ export interface Extra {
   tokens: null | TSESTree.Token[];
   tsconfigRootDir: string;
   useJSXTextNode: boolean;
+  fragment: boolean;
 }
 
 export interface TSESTreeOptions {
@@ -38,6 +39,7 @@ export interface TSESTreeOptions {
   tokens?: boolean;
   tsconfigRootDir?: string;
   useJSXTextNode?: boolean;
+  fragment?: boolean;
 }
 
 // This lets us use generics to type the return value, and removes the need to
diff --git a/packages/typescript-estree/src/parser.ts b/packages/typescript-estree/src/parser.ts
@@ -59,12 +59,15 @@ function getProgramAndAST(
   code: string,
   shouldProvideParserServices: boolean,
   shouldCreateDefaultProgram: boolean,
+  shouldFragment: boolean,
 ): ASTAndProgram | undefined {
   return (
     (shouldProvideParserServices &&
+      !shouldFragment &&
       createProjectProgram(code, shouldCreateDefaultProgram, extra)) ||
     (shouldProvideParserServices &&
       shouldCreateDefaultProgram &&
+      !shouldFragment &&
       createDefaultProgram(code, extra)) ||
     createIsolatedProgram(code, extra)
   );
@@ -105,6 +108,7 @@ function resetExtra(): void {
     tokens: null,
     tsconfigRootDir: process.cwd(),
     useJSXTextNode: false,
+    fragment: false,
   };
 }
 
@@ -228,6 +232,8 @@ function applyParserOptionsToExtra(options: TSESTreeOptions): void {
   extra.createDefaultProgram =
     typeof options.createDefaultProgram === 'boolean' &&
     options.createDefaultProgram;
+
+  extra.fragment = typeof options.fragment === 'boolean' && options.fragment;
 }
 
 function warnAboutTSVersion(): void {
@@ -363,6 +369,7 @@ function parseAndGenerateServices<T extends TSESTreeOptions = TSESTreeOptions>(
     code,
     shouldProvideParserServices,
     extra.createDefaultProgram,
+    extra.fragment,
   )!;
 
   /**

Original file line number	Diff line number	Diff line change
`@@ -19,4 +19,5 @@ export interface ParserOptions {`
`19`	`19`	`tsconfigRootDir?: string;`
`20`	`20`	`useJSXTextNode?: boolean;`
`21`	`21`	`warnOnUnsupportedTypeScriptVersion?: boolean;`
	`22`	`+ fragment?: boolean;`
`22`	`23`	`}`
Original file line number	Diff line number	Diff line change
`@@ -66,6 +66,9 @@ export function parseForESLint(`
`66`	`66`	`if (typeof options.ecmaFeatures !== 'object') {`
`67`	`67`	`options.ecmaFeatures = {};`
`68`	`68`	`}`
	`69`	`+ if (!('fragment' in options)) {`
	`70`	`+ options.fragment = false;`
	`71`	`+ }`
`69`	`72`
`70`	`73`	`const parserOptions: TSESTreeOptions = {};`
`71`	`74`	`Object.assign(parserOptions, options, {`
Original file line number	Diff line number	Diff line change
`@@ -20,6 +20,7 @@ export interface Extra {`
`20`	`20`	`tokens: null \| TSESTree.Token[];`
`21`	`21`	`tsconfigRootDir: string;`
`22`	`22`	`useJSXTextNode: boolean;`
	`23`	`+ fragment: boolean;`
`23`	`24`	`}`
`24`	`25`
`25`	`26`	`export interface TSESTreeOptions {`
`@@ -38,6 +39,7 @@ export interface TSESTreeOptions {`
`38`	`39`	`tokens?: boolean;`
`39`	`40`	`tsconfigRootDir?: string;`
`40`	`41`	`useJSXTextNode?: boolean;`
	`42`	`+ fragment?: boolean;`
`41`	`43`	`}`
`42`	`44`
`43`	`45`	`// This lets us use generics to type the return value, and removes the need to`