fix: Allow detection of experiments in config early in init, but re-parse the root module after it might have been impacted by use of the -from-module flag (parsing may yield something different the second time!). Diagnostics still need to be passed though.

Author: SarahFrench

internal/command/init_run.go

@@ -130,14 +130,13 @@ func (c *InitCommand) run(initArgs *arguments.Init, view views.Init, rootModEarl
 		return 0
 	}
 
-	// If the passed root module is nil, attempt to parse it.
-	// At this point we load just the root module to begin backend and module initialization
+	// We parse the root module here because it may have been affected by use of init's -from-module flag.
+	// However we need to use diagnostics passed in from calling code due to detection of experiments in the config.
+	// See calling code for more context.
 	//
-	// TODO(SarahFrench/radeksimko): Once PSS's experiment is over, remove use of arguments and
-	// restore parsing of config to always happen here in this code instead of calling code.
-	if rootModEarly == nil {
-		rootModEarly, earlyConfDiags = c.loadSingleModuleWithTests(path, initArgs.TestsDirectory)
-	}
+	// TODO(SarahFrench/radeksimko): Once PSS's experiment is over, remove use of earlyConfDiags argument and
+	// restore parsing of config to only happen here in this code instead of calling code.
+	rootModEarly, _ = c.loadSingleModuleWithTests(path, initArgs.TestsDirectory)
 
 	// There may be parsing errors in config loading but these will be shown later _after_
 	// checking for core version requirement errors. Not meeting the version requirement should

internal/command/init_run_experiment.go

@@ -138,14 +138,13 @@ func (c *InitCommand) runPssInit(initArgs *arguments.Init, view views.Init, root
 		return 0
 	}
 
-	// If the passed root module is nil, attempt to parse it.
-	// At this point we load just the root module to begin backend and module initialization
+	// We parse the root module here because it may have been affected by use of init's -from-module flag.
+	// However we need to use diagnostics passed in from calling code due to detection of experiments in the config.
+	// See calling code for more context.
 	//
-	// TODO(SarahFrench/radeksimko): Once PSS's experiment is over, remove use of arguments and
-	// restore parsing of config to always happen here in this code instead of calling code.
-	if rootModEarly == nil {
-		rootModEarly, earlyConfDiags = c.loadSingleModuleWithTests(path, initArgs.TestsDirectory)
-	}
+	// TODO(SarahFrench/radeksimko): Once PSS's experiment is over, remove use of earlyConfDiags argument and
+	// restore parsing of config to only happen here in this code instead of calling code.
+	rootModEarly, _ = c.loadSingleModuleWithTests(path, initArgs.TestsDirectory)
 
 	// There may be parsing errors in config loading but these will be shown later _after_
 	// checking for core version requirement errors. Not meeting the version requirement should