Skip to content

Commit 129896e

Browse files
sheetalkamatsheetalkamat
committed
Add allowPlugins on commandline which is needed to resolve watchFactory
1 parent e77a5b8 commit 129896e

File tree

56 files changed

+1455
-170
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

56 files changed

+1455
-170
lines changed

‎src/compiler/commandLineParser.ts

Lines changed: 52 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -62,9 +62,11 @@ import {
6262
getSupportedExtensions,
6363
getSupportedExtensionsWithJsonIfResolveJsonModule,
6464
getTextOfPropertyName,
65+
getTsConfigObjectLiteralExpression,
6566
getTsConfigPropArrayElementValue,
6667
hasExtension,
6768
hasProperty,
69+
identity,
6870
ImportsNotUsedAsValues,
6971
isArray,
7072
isArrayLiteralExpression,
@@ -527,6 +529,14 @@ export const commonOptionsWithBuild: CommandLineOption[] = [
527529
description: Diagnostics.Set_the_language_of_the_messaging_from_TypeScript_This_does_not_affect_emit,
528530
defaultValueDescription: Diagnostics.Platform_specific
529531
},
532+
{
533+
name: "allowPlugins",
534+
type: "boolean",
535+
category: Diagnostics.Command_line_Options,
536+
isCommandLineOnly: true,
537+
description: Diagnostics.Allow_running_plugins,
538+
defaultValueDescription: false,
539+
},
530540
];
531541

532542
/** @internal */
@@ -1819,6 +1829,10 @@ export function parseCommandLineWorker(
18191829
const errors: Diagnostic[] = [];
18201830

18211831
parseStrings(commandLine);
1832+
if (options.watch && watchOptions?.watchFactory && !options.allowPlugins) {
1833+
errors.push(createCompilerDiagnostic(Diagnostics.Option_watchFactory_cannot_be_specified_without_passing_allowPlugins_on_command_line));
1834+
watchOptions.watchFactory = undefined;
1835+
}
18221836
return {
18231837
options,
18241838
watchOptions,
@@ -2096,6 +2110,7 @@ export function getParsedCommandLineOfConfigFile(
20962110
extendedConfigCache?: Map<string, ExtendedConfigCacheEntry>,
20972111
watchOptionsToExtend?: WatchOptions,
20982112
extraFileExtensions?: readonly FileExtensionInfo[],
2113+
checkAllowPlugins?: boolean,
20992114
): ParsedCommandLine | undefined {
21002115
const configFileText = tryReadFile(configFileName, fileName => host.readFile(fileName));
21012116
if (!isString(configFileText)) {
@@ -2117,7 +2132,8 @@ export function getParsedCommandLineOfConfigFile(
21172132
/*resolutionStack*/ undefined,
21182133
extraFileExtensions,
21192134
extendedConfigCache,
2120-
watchOptionsToExtend
2135+
watchOptionsToExtend,
2136+
checkAllowPlugins,
21212137
);
21222138
}
21232139

@@ -2852,9 +2868,20 @@ export function parseJsonConfigFileContent(json: any, host: ParseConfigHost, bas
28522868
* @param basePath A root directory to resolve relative path entries in the config
28532869
* file to. e.g. outDir
28542870
*/
2855-
export function parseJsonSourceFileConfigFileContent(sourceFile: TsConfigSourceFile, host: ParseConfigHost, basePath: string, existingOptions?: CompilerOptions, configFileName?: string, resolutionStack?: Path[], extraFileExtensions?: readonly FileExtensionInfo[], extendedConfigCache?: Map<string, ExtendedConfigCacheEntry>, existingWatchOptions?: WatchOptions): ParsedCommandLine {
2871+
export function parseJsonSourceFileConfigFileContent(
2872+
sourceFile: TsConfigSourceFile,
2873+
host: ParseConfigHost,
2874+
basePath: string,
2875+
existingOptions?: CompilerOptions,
2876+
configFileName?: string,
2877+
resolutionStack?: Path[],
2878+
extraFileExtensions?: readonly FileExtensionInfo[],
2879+
extendedConfigCache?: Map<string, ExtendedConfigCacheEntry>,
2880+
existingWatchOptions?: WatchOptions,
2881+
checkAllowPlugins?: boolean,
2882+
): ParsedCommandLine {
28562883
tracing?.push(tracing.Phase.Parse, "parseJsonSourceFileConfigFileContent", { path: sourceFile.fileName });
2857-
const result = parseJsonConfigFileContentWorker(/*json*/ undefined, sourceFile, host, basePath, existingOptions, existingWatchOptions, configFileName, resolutionStack, extraFileExtensions, extendedConfigCache);
2884+
const result = parseJsonConfigFileContentWorker(/*json*/ undefined, sourceFile, host, basePath, existingOptions, existingWatchOptions, configFileName, resolutionStack, extraFileExtensions, extendedConfigCache, checkAllowPlugins);
28582885
tracing?.pop();
28592886
return result;
28602887
}
@@ -2898,7 +2925,8 @@ function parseJsonConfigFileContentWorker(
28982925
configFileName?: string,
28992926
resolutionStack: Path[] = [],
29002927
extraFileExtensions: readonly FileExtensionInfo[] = [],
2901-
extendedConfigCache?: Map<string, ExtendedConfigCacheEntry>
2928+
extendedConfigCache?: Map<string, ExtendedConfigCacheEntry>,
2929+
checkAllowPlugins?: boolean,
29022930
): ParsedCommandLine {
29032931
Debug.assert((json === undefined && sourceFile !== undefined) || (json !== undefined && sourceFile === undefined));
29042932
const errors: Diagnostic[] = [];
@@ -2910,6 +2938,21 @@ function parseJsonConfigFileContentWorker(
29102938
extend(existingWatchOptions || {}, parsedConfig.watchOptions || {}) :
29112939
undefined;
29122940

2941+
if (options.watch && watchOptions?.watchFactory && checkAllowPlugins && !options.allowPlugins) {
2942+
const watchFactoryNameProp = forEachPropertyAssignment(
2943+
getTsConfigObjectLiteralExpression(sourceFile),
2944+
"watchOptions",
2945+
prop => isObjectLiteralExpression(prop.initializer) ?
2946+
forEachPropertyAssignment(prop.initializer, "watchFactory", watchFactoryProp => isObjectLiteralExpression(watchFactoryProp.initializer) ?
2947+
forEachPropertyAssignment(watchFactoryProp.initializer, "name", identity) || watchFactoryProp :
2948+
watchFactoryProp
2949+
) :
2950+
undefined
2951+
);
2952+
errors.push(createDiagnosticForNodeInSourceFileOrCompilerDiagnostic(sourceFile, watchFactoryNameProp?.name, Diagnostics.Option_watchFactory_cannot_be_specified_without_passing_allowPlugins_on_command_line));
2953+
watchOptions.watchFactory = undefined;
2954+
}
2955+
29132956
options.configFilePath = configFileName && normalizeSlashes(configFileName);
29142957
const configFileSpecs = getConfigFileSpecs();
29152958
if (sourceFile) sourceFile.configFileSpecs = configFileSpecs;
@@ -3533,7 +3576,11 @@ export function convertJsonOption(
35333576
convertJsonOptionOfListType(opt, value, basePath, errors, propertyAssignment, valueExpression as ArrayLiteralExpression | undefined, sourceFile) :
35343577
convertJsonOption(opt.element, value, basePath, errors, propertyAssignment, valueExpression, sourceFile);
35353578
}
3536-
else if (!isString(opt.type)) {
3579+
if (opt.isCommandLineOnly) {
3580+
errors.push(createDiagnosticForNodeInSourceFileOrCompilerDiagnostic(sourceFile, valueExpression, Diagnostics.Option_0_can_only_be_specified_on_command_line, opt.name));
3581+
return;
3582+
}
3583+
if (!isString(opt.type)) {
35373584
return convertJsonOptionOfCustomType(opt as CommandLineOptionOfCustomType, value as string, errors, valueExpression, sourceFile);
35383585
}
35393586
const validatedValue = validateJsonOptionValue(opt, value, errors, valueExpression, sourceFile);

‎src/compiler/diagnosticMessages.json

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -4333,6 +4333,10 @@
43334333
"category": "Error",
43344334
"code": 5109
43354335
},
4336+
"Option 'watchFactory' cannot be specified without passing '--allowPlugins' on command line.": {
4337+
"category": "Error",
4338+
"code": 5110
4339+
},
43364340

43374341
"Generates a sourcemap for each corresponding '.d.ts' file.": {
43384342
"category": "Message",
@@ -6103,6 +6107,10 @@
61036107
"category": "Message",
61046108
"code": 6804
61056109
},
6110+
"Allow running plugins.": {
6111+
"category": "Message",
6112+
"code": 6805
6113+
},
61066114

61076115
"one of:": {
61086116
"category": "Message",

‎src/compiler/tsbuildPublic.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,7 @@ export interface BuildOptions {
161161
/** @internal */ locale?: string;
162162
/** @internal */ generateCpuProfile?: string;
163163
/** @internal */ generateTrace?: string;
164+
/** @internal */ allowPlugins?: boolean;
164165

165166
[option: string]: CompilerOptionsValue | undefined;
166167
}
@@ -567,7 +568,7 @@ function parseConfigFile<T extends BuilderProgram>(state: SolutionBuilderState<T
567568
}
568569
else {
569570
parseConfigFileHost.onUnRecoverableConfigFileDiagnostic = d => diagnostic = d;
570-
parsed = getParsedCommandLineOfConfigFile(configFileName, baseCompilerOptions, parseConfigFileHost, extendedConfigCache, baseWatchOptions);
571+
parsed = getParsedCommandLineOfConfigFile(configFileName, baseCompilerOptions, parseConfigFileHost, extendedConfigCache, baseWatchOptions, /*extraFileExtensions*/ undefined, state.hostWithWatch.checkAllowPlugins);
571572
parseConfigFileHost.onUnRecoverableConfigFileDiagnostic = noop;
572573
}
573574
configFileCache.set(configFilePath, parsed || diagnostic!);

‎src/compiler/types.ts

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -7141,6 +7141,7 @@ export interface CompilerOptions {
71417141
*/
71427142
pathsBasePath?: string;
71437143
/** @internal */ plugins?: PluginImport[];
7144+
/** @internal */ allowPlugins?: boolean;
71447145
preserveConstEnums?: boolean;
71457146
noImplicitOverride?: boolean;
71467147
preserveSymlinks?: boolean;

‎src/compiler/watch.ts

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -213,10 +213,18 @@ export function createWatchStatusReporter(system: System, pretty?: boolean): Wat
213213
*
214214
* @internal
215215
*/
216-
export function parseConfigFileWithSystem(configFileName: string, optionsToExtend: CompilerOptions, extendedConfigCache: Map<string, ExtendedConfigCacheEntry> | undefined, watchOptionsToExtend: WatchOptions | undefined, system: System, reportDiagnostic: DiagnosticReporter): ParsedCommandLine | undefined {
216+
export function parseConfigFileWithSystem(
217+
configFileName: string,
218+
optionsToExtend: CompilerOptions,
219+
extendedConfigCache: Map<string, ExtendedConfigCacheEntry> | undefined,
220+
watchOptionsToExtend: WatchOptions | undefined,
221+
system: System,
222+
reportDiagnostic: DiagnosticReporter,
223+
checkAllowPlugins: boolean,
224+
): ParsedCommandLine | undefined {
217225
const host: ParseConfigFileHost = system as any;
218226
host.onUnRecoverableConfigFileDiagnostic = diagnostic => reportUnrecoverableDiagnostic(system, reportDiagnostic, diagnostic);
219-
const result = getParsedCommandLineOfConfigFile(configFileName, optionsToExtend, host, extendedConfigCache, watchOptionsToExtend);
227+
const result = getParsedCommandLineOfConfigFile(configFileName, optionsToExtend, host, extendedConfigCache, watchOptionsToExtend, /*extraFileExtensions*/ undefined, checkAllowPlugins);
220228
host.onUnRecoverableConfigFileDiagnostic = undefined!; // TODO: GH#18217
221229
return result;
222230
}

‎src/compiler/watchPublic.ts

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -161,6 +161,7 @@ export interface WatchHost {
161161
setTimeout?(callback: (...args: any[]) => void, ms: number, ...args: any[]): any;
162162
/** If provided, will be used to reset existing delayed compilation */
163163
clearTimeout?(timeoutId: any): void;
164+
/** @internal */ checkAllowPlugins?: boolean;
164165
}
165166
export interface ProgramHost<T extends BuilderProgram> {
166167
/**
@@ -902,7 +903,8 @@ export function createWatchProgram<T extends BuilderProgram>(host: WatchCompiler
902903
parseConfigFileHost,
903904
extendedConfigCache ||= new Map(),
904905
watchOptionsToExtend,
905-
extraFileExtensions
906+
extraFileExtensions,
907+
host.checkAllowPlugins,
906908
)!); // TODO: GH#18217
907909
}
908910

@@ -962,7 +964,9 @@ export function createWatchProgram<T extends BuilderProgram>(host: WatchCompiler
962964
/*optionsToExtend*/ undefined,
963965
parseConfigFileHost,
964966
extendedConfigCache ||= new Map(),
965-
watchOptionsToExtend
967+
watchOptionsToExtend,
968+
host.extraFileExtensions,
969+
host.checkAllowPlugins,
966970
);
967971
parseConfigFileHost.onUnRecoverableConfigFileDiagnostic = onUnRecoverableConfigFileDiagnostic;
968972
return parsedCommandLine;

‎src/executeCommandLine/executeCommandLine.ts

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -645,7 +645,7 @@ function executeCommandLineWorker(
645645
);
646646
if (configFileName) {
647647
const extendedConfigCache = new Map<string, ExtendedConfigCacheEntry>();
648-
const configParseResult = parseConfigFileWithSystem(configFileName, commandLineOptions, extendedConfigCache, commandLine.watchOptions, sys, reportDiagnostic)!; // TODO: GH#18217
648+
const configParseResult = parseConfigFileWithSystem(configFileName, commandLineOptions, extendedConfigCache, commandLine.watchOptions, sys, reportDiagnostic, /*checkAllowPlugins*/ true)!; // TODO: GH#18217
649649
if (commandLineOptions.showConfig) {
650650
if (configParseResult.errors.length !== 0) {
651651
reportDiagnostic = updateReportDiagnostic(
@@ -847,6 +847,7 @@ function performBuild(
847847
);
848848
const solutionPerformance = enableSolutionPerformance(sys, buildOptions);
849849
updateSolutionBuilderHost(sys, cb, buildHost, solutionPerformance);
850+
buildHost.checkAllowPlugins = true;
850851
const onWatchStatusChange = buildHost.onWatchStatusChange;
851852
let reportBuildStatistics = false;
852853
buildHost.onWatchStatusChange = (d, newLine, options, errorCount) => {
@@ -979,6 +980,7 @@ function updateWatchCompilationHost(
979980
cb: ExecuteCommandLineCallbacks,
980981
watchCompilerHost: WatchCompilerHost<EmitAndSemanticDiagnosticsBuilderProgram>,
981982
) {
983+
watchCompilerHost.checkAllowPlugins = true;
982984
updateCreateProgram(sys, watchCompilerHost, /*isBuildMode*/ false);
983985
const emitFilesUsingBuilder = watchCompilerHost.afterProgramCreate!; // TODO: GH#18217
984986
watchCompilerHost.afterProgramCreate = builderProgram => {

‎src/testRunner/unittests/reuseProgramStructure.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -586,7 +586,7 @@ describe("unittests:: Reuse program structure:: isProgramUptoDate", () => {
586586
configFileName,
587587
system
588588
})).getCurrentProgram().getProgram();
589-
const { fileNames, options } = ts.parseConfigFileWithSystem(configFileName, {}, /*extendedConfigCache*/ undefined, /*watchOptionsToExtend*/ undefined, system, ts.notImplemented)!; // TODO: GH#18217
589+
const { fileNames, options } = ts.parseConfigFileWithSystem(configFileName, {}, /*extendedConfigCache*/ undefined, /*watchOptionsToExtend*/ undefined, system, ts.notImplemented, /*checkAllowPlugins*/ true)!; // TODO: GH#18217
590590
verifyProgramIsUptoDate(program, fileNames, options);
591591
}
592592

‎src/testRunner/unittests/tsbuild/outputPaths.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -32,7 +32,7 @@ describe("unittests:: tsbuild - output file paths", () => {
3232

3333
assert.deepEqual(
3434
ts.getOutputFileNames(
35-
ts.parseConfigFileWithSystem("/src/tsconfig.json", {}, /*extendedConfigCache*/ undefined, {}, sys, ts.noop)!,
35+
ts.parseConfigFileWithSystem("/src/tsconfig.json", {}, /*extendedConfigCache*/ undefined, {}, sys, ts.noop, /*checkAllowPlugins*/ true)!,
3636
"/src/src/index.ts",
3737
/*ignoreCase*/ false
3838
),

0 commit comments

Comments
 (0)