implement module_ctx.root_module_has_non_dev_dependency

Author: EdSchouten

pkg/model/analysis/module_extension_repos.go

@@ -418,18 +418,19 @@ func (c *baseComputer[TReference, TMetadata]) ComputeModuleExtensionReposValue(c
 
 		moduleCtx := model_starlark.NewStructFromDict[TReference, TMetadata](nil, map[string]any{
 			// Fields shared with repository_ctx.
-			"download":             starlark.NewBuiltin("module_ctx.download", moduleContext.doDownload),
-			"download_and_extract": starlark.NewBuiltin("module_ctx.download_and_extract", moduleContext.doDownloadAndExtract),
-			"execute":              starlark.NewBuiltin("module_ctx.execute", moduleContext.doExecute),
-			"extract":              starlark.NewBuiltin("module_ctx.extract", moduleContext.doExtract),
-			"file":                 starlark.NewBuiltin("module_ctx.file", moduleContext.doFile),
-			"getenv":               starlark.NewBuiltin("module_ctx.getenv", moduleContext.doGetenv),
-			"os":                   newRepositoryOS[TReference, TMetadata](thread, repoPlatform.Message),
-			"path":                 starlark.NewBuiltin("module_ctx.path", moduleContext.doPath),
-			"read":                 starlark.NewBuiltin("module_ctx.read", moduleContext.doRead),
-			"report_progress":      starlark.NewBuiltin("module_ctx.report_progress", moduleContext.doReportProgress),
-			"watch":                starlark.NewBuiltin("module_ctx.watch", moduleContext.doWatch),
-			"which":                starlark.NewBuiltin("module_ctx.which", moduleContext.doWhich),
+			"download":                           starlark.NewBuiltin("module_ctx.download", moduleContext.doDownload),
+			"download_and_extract":               starlark.NewBuiltin("module_ctx.download_and_extract", moduleContext.doDownloadAndExtract),
+			"execute":                            starlark.NewBuiltin("module_ctx.execute", moduleContext.doExecute),
+			"extract":                            starlark.NewBuiltin("module_ctx.extract", moduleContext.doExtract),
+			"file":                               starlark.NewBuiltin("module_ctx.file", moduleContext.doFile),
+			"getenv":                             starlark.NewBuiltin("module_ctx.getenv", moduleContext.doGetenv),
+			"os":                                 newRepositoryOS[TReference, TMetadata](thread, repoPlatform.Message),
+			"path":                               starlark.NewBuiltin("module_ctx.path", moduleContext.doPath),
+			"read":                               starlark.NewBuiltin("module_ctx.read", moduleContext.doRead),
+			"report_progress":                    starlark.NewBuiltin("module_ctx.report_progress", moduleContext.doReportProgress),
+			"root_module_has_non_dev_dependency": starlark.Bool(usedModuleExtension.RootModuleHasNonDevDependency),
+			"watch":                              starlark.NewBuiltin("module_ctx.watch", moduleContext.doWatch),
+			"which":                              starlark.NewBuiltin("module_ctx.which", moduleContext.doWhich),
 
 			// Fields specific to module_ctx.
 			"extension_metadata": starlark.NewBuiltin(

pkg/model/analysis/used_module_extensions.go

@@ -134,6 +134,9 @@ func (h *usedModuleExtensionExtractingModuleDotBazelHandler[TReference, TMetadat
 		}
 		h.usedModuleExtensions[moduleExtensionName] = ume
 	}
+	if h.isRoot && !devDependency {
+		ume.message.RootModuleHasNonDevDependency = true
+	}
 
 	meu, ok := ume.users[h.moduleInstance]
 	if !ok {

pkg/proto/model/analysis/analysis.pb.go

@@ -1963,6 +1963,11 @@ message ModuleExtension {
   // list is the same as a breadth-first search starting from the root
   // module.
   repeated User users = 2;
+
+  // Whether the MODULE.bazel file of the root module contains one or
+  // more calls to use_extension() for this extension with
+  // dev_dependency set to False.
+  bool root_module_has_non_dev_dependency = 3;
 }
 
 message RepositoryRuleObject {