Apply some more buildifier fixes

android/android_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of android_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

android/android_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of android_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

android/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `android` rules to one loadable file
+"""android protobuf and grpc rules."""
+
 load(":android_proto_compile.bzl", _android_proto_compile = "android_proto_compile")
 load(":android_grpc_compile.bzl", _android_grpc_compile = "android_grpc_compile")
 load(":android_proto_library.bzl", _android_proto_library = "android_proto_library")
 load(":android_grpc_library.bzl", _android_grpc_library = "android_grpc_library")
 
+# Export android rules
 android_proto_compile = _android_proto_compile
 android_grpc_compile = _android_grpc_compile
 android_proto_library = _android_proto_library

closure/closure_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of closure_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

closure/defs.bzl

@@ -1,6 +1,8 @@
-# Aggregate all `closure` rules to one loadable file
+"""closure protobuf and grpc rules."""
+
 load(":closure_proto_compile.bzl", _closure_proto_compile = "closure_proto_compile")
 load(":closure_proto_library.bzl", _closure_proto_library = "closure_proto_library")
 
+# Export closure rules
 closure_proto_compile = _closure_proto_compile
 closure_proto_library = _closure_proto_library

cpp/cpp_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of cpp_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

cpp/cpp_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of cpp_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

cpp/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `cpp` rules to one loadable file
+"""cpp protobuf and grpc rules."""
+
 load(":cpp_proto_compile.bzl", _cpp_proto_compile = "cpp_proto_compile")
 load(":cpp_grpc_compile.bzl", _cpp_grpc_compile = "cpp_grpc_compile")
 load(":cpp_proto_library.bzl", _cpp_proto_library = "cpp_proto_library")
 load(":cpp_grpc_library.bzl", _cpp_grpc_library = "cpp_grpc_library")
 
+# Export cpp rules
 cpp_proto_compile = _cpp_proto_compile
 cpp_grpc_compile = _cpp_grpc_compile
 cpp_proto_library = _cpp_proto_library

csharp/csharp_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of csharp_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

csharp/csharp_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of csharp_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

csharp/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `csharp` rules to one loadable file
+"""csharp protobuf and grpc rules."""
+
 load(":csharp_proto_compile.bzl", _csharp_proto_compile = "csharp_proto_compile")
 load(":csharp_grpc_compile.bzl", _csharp_grpc_compile = "csharp_grpc_compile")
 load(":csharp_proto_library.bzl", _csharp_proto_library = "csharp_proto_library")
 load(":csharp_grpc_library.bzl", _csharp_grpc_library = "csharp_grpc_library")
 
+# Export csharp rules
 csharp_proto_compile = _csharp_proto_compile
 csharp_grpc_compile = _csharp_grpc_compile
 csharp_proto_library = _csharp_proto_library

d/d_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of d_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

d/defs.bzl

@@ -1,6 +1,8 @@
-# Aggregate all `d` rules to one loadable file
+"""d protobuf and grpc rules."""
+
 load(":d_proto_compile.bzl", _d_proto_compile = "d_proto_compile")
 load(":d_proto_library.bzl", _d_proto_library = "d_proto_library")
 
+# Export d rules
 d_proto_compile = _d_proto_compile
 d_proto_library = _d_proto_library

go/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `go` rules to one loadable file
+"""go protobuf and grpc rules."""
+
 load(":go_proto_compile.bzl", _go_proto_compile = "go_proto_compile")
 load(":go_grpc_compile.bzl", _go_grpc_compile = "go_grpc_compile")
 load(":go_proto_library.bzl", _go_proto_library = "go_proto_library")
 load(":go_grpc_library.bzl", _go_grpc_library = "go_grpc_library")
 
+# Export go rules
 go_proto_compile = _go_proto_compile
 go_grpc_compile = _go_grpc_compile
 go_proto_library = _go_proto_library

go/go_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of go_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

go/go_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of go_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

grpc-gateway/defs.bzl

@@ -1,8 +1,10 @@
-# Aggregate all `grpc-gateway` rules to one loadable file
+"""grpc-gateway protobuf and grpc rules."""
+
 load(":gateway_grpc_compile.bzl", _gateway_grpc_compile = "gateway_grpc_compile")
 load(":gateway_openapiv2_compile.bzl", _gateway_openapiv2_compile = "gateway_openapiv2_compile")
 load(":gateway_grpc_library.bzl", _gateway_grpc_library = "gateway_grpc_library")
 
+# Export grpc-gateway rules
 gateway_grpc_compile = _gateway_grpc_compile
 gateway_openapiv2_compile = _gateway_openapiv2_compile
 gateway_grpc_library = _gateway_grpc_library

grpc-gateway/gateway_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of gateway_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

grpc-gateway/gateway_openapiv2_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of gateway_openapiv2_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

grpc-web/closure_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of closure_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

grpc-web/commonjs_dts_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of commonjs_dts_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

grpc-web/commonjs_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of commonjs_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

grpc-web/defs.bzl

@@ -1,10 +1,12 @@
-# Aggregate all `grpc-web` rules to one loadable file
+"""grpc-web protobuf and grpc rules."""
+
 load(":closure_grpc_compile.bzl", _closure_grpc_compile = "closure_grpc_compile")
 load(":commonjs_grpc_compile.bzl", _commonjs_grpc_compile = "commonjs_grpc_compile")
 load(":commonjs_dts_grpc_compile.bzl", _commonjs_dts_grpc_compile = "commonjs_dts_grpc_compile")
 load(":ts_grpc_compile.bzl", _ts_grpc_compile = "ts_grpc_compile")
 load(":closure_grpc_library.bzl", _closure_grpc_library = "closure_grpc_library")
 
+# Export grpc-web rules
 closure_grpc_compile = _closure_grpc_compile
 commonjs_grpc_compile = _commonjs_grpc_compile
 commonjs_dts_grpc_compile = _commonjs_dts_grpc_compile

grpc-web/ts_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of ts_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

java/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `java` rules to one loadable file
+"""java protobuf and grpc rules."""
+
 load(":java_proto_compile.bzl", _java_proto_compile = "java_proto_compile")
 load(":java_grpc_compile.bzl", _java_grpc_compile = "java_grpc_compile")
 load(":java_proto_library.bzl", _java_proto_library = "java_proto_library")
 load(":java_grpc_library.bzl", _java_grpc_library = "java_grpc_library")
 
+# Export java rules
 java_proto_compile = _java_proto_compile
 java_grpc_compile = _java_grpc_compile
 java_proto_library = _java_proto_library

java/java_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of java_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

java/java_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of java_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

nodejs/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `nodejs` rules to one loadable file
+"""nodejs protobuf and grpc rules."""
+
 load(":nodejs_proto_compile.bzl", _nodejs_proto_compile = "nodejs_proto_compile")
 load(":nodejs_grpc_compile.bzl", _nodejs_grpc_compile = "nodejs_grpc_compile")
 load(":nodejs_proto_library.bzl", _nodejs_proto_library = "nodejs_proto_library")
 load(":nodejs_grpc_library.bzl", _nodejs_grpc_library = "nodejs_grpc_library")
 
+# Export nodejs rules
 nodejs_proto_compile = _nodejs_proto_compile
 nodejs_grpc_compile = _nodejs_grpc_compile
 nodejs_proto_library = _nodejs_proto_library

nodejs/nodejs_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of nodejs_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

nodejs/nodejs_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of nodejs_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

objc/defs.bzl

@@ -1,8 +1,10 @@
-# Aggregate all `objc` rules to one loadable file
+"""objc protobuf and grpc rules."""
+
 load(":objc_proto_compile.bzl", _objc_proto_compile = "objc_proto_compile")
 load(":objc_grpc_compile.bzl", _objc_grpc_compile = "objc_grpc_compile")
 load(":objc_proto_library.bzl", _objc_proto_library = "objc_proto_library")
 
+# Export objc rules
 objc_proto_compile = _objc_proto_compile
 objc_grpc_compile = _objc_grpc_compile
 objc_proto_library = _objc_proto_library

objc/objc_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of objc_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

objc/objc_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of objc_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

php/defs.bzl

@@ -1,6 +1,8 @@
-# Aggregate all `php` rules to one loadable file
+"""php protobuf and grpc rules."""
+
 load(":php_proto_compile.bzl", _php_proto_compile = "php_proto_compile")
 load(":php_grpc_compile.bzl", _php_grpc_compile = "php_grpc_compile")
 
+# Export php rules
 php_proto_compile = _php_proto_compile
 php_grpc_compile = _php_grpc_compile

php/php_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of php_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

php/php_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of php_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

python/defs.bzl

@@ -1,4 +1,5 @@
-# Aggregate all `python` rules to one loadable file
+"""python protobuf and grpc rules."""
+
 load(":python_proto_compile.bzl", _python_proto_compile = "python_proto_compile")
 load(":python_grpc_compile.bzl", _python_grpc_compile = "python_grpc_compile")
 load(":python_grpclib_compile.bzl", _python_grpclib_compile = "python_grpclib_compile")
@@ -6,6 +7,7 @@ load(":python_proto_library.bzl", _python_proto_library = "python_proto_library"
 load(":python_grpc_library.bzl", _python_grpc_library = "python_grpc_library")
 load(":python_grpclib_library.bzl", _python_grpclib_library = "python_grpclib_library")
 
+# Export python rules
 python_proto_compile = _python_proto_compile
 python_grpc_compile = _python_grpc_compile
 python_grpclib_compile = _python_grpclib_compile

python/python_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of python_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

python/python_grpclib_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of python_grpclib_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

python/python_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of python_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

ruby/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `ruby` rules to one loadable file
+"""ruby protobuf and grpc rules."""
+
 load(":ruby_proto_compile.bzl", _ruby_proto_compile = "ruby_proto_compile")
 load(":ruby_grpc_compile.bzl", _ruby_grpc_compile = "ruby_grpc_compile")
 load(":ruby_proto_library.bzl", _ruby_proto_library = "ruby_proto_library")
 load(":ruby_grpc_library.bzl", _ruby_grpc_library = "ruby_grpc_library")
 
+# Export ruby rules
 ruby_proto_compile = _ruby_proto_compile
 ruby_grpc_compile = _ruby_grpc_compile
 ruby_proto_library = _ruby_proto_library

ruby/ruby_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of ruby_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

ruby/ruby_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of ruby_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

rust/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `rust` rules to one loadable file
+"""rust protobuf and grpc rules."""
+
 load(":rust_proto_compile.bzl", _rust_proto_compile = "rust_proto_compile")
 load(":rust_grpc_compile.bzl", _rust_grpc_compile = "rust_grpc_compile")
 load(":rust_proto_library.bzl", _rust_proto_library = "rust_proto_library")
 load(":rust_grpc_library.bzl", _rust_grpc_library = "rust_grpc_library")
 
+# Export rust rules
 rust_proto_compile = _rust_proto_compile
 rust_grpc_compile = _rust_grpc_compile
 rust_proto_library = _rust_proto_library

rust/rust_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of rust_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

rust/rust_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of rust_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

scala/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `scala` rules to one loadable file
+"""scala protobuf and grpc rules."""
+
 load(":scala_proto_compile.bzl", _scala_proto_compile = "scala_proto_compile")
 load(":scala_grpc_compile.bzl", _scala_grpc_compile = "scala_grpc_compile")
 load(":scala_proto_library.bzl", _scala_proto_library = "scala_proto_library")
 load(":scala_grpc_library.bzl", _scala_grpc_library = "scala_grpc_library")
 
+# Export scala rules
 scala_proto_compile = _scala_proto_compile
 scala_grpc_compile = _scala_grpc_compile
 scala_proto_library = _scala_proto_library

scala/scala_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of scala_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

scala/scala_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of scala_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

swift/defs.bzl

@@ -1,9 +1,11 @@
-# Aggregate all `swift` rules to one loadable file
+"""swift protobuf and grpc rules."""
+
 load(":swift_proto_compile.bzl", _swift_proto_compile = "swift_proto_compile")
 load(":swift_grpc_compile.bzl", _swift_grpc_compile = "swift_grpc_compile")
 load(":swift_proto_library.bzl", _swift_proto_library = "swift_proto_library")
 load(":swift_grpc_library.bzl", _swift_grpc_library = "swift_grpc_library")
 
+# Export swift rules
 swift_proto_compile = _swift_proto_compile
 swift_grpc_compile = _swift_grpc_compile
 swift_proto_library = _swift_proto_library

swift/swift_grpc_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of swift_grpc_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

swift/swift_proto_compile.bzl

@@ -1,3 +1,5 @@
+"""Generated definition of swift_proto_compile."""
+
 load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",

tools/rulegen/common.go

@@ -33,11 +33,13 @@ var libraryRuleAttrs = append(append([]*Attr(nil), compileRuleAttrs...), []*Attr
 }...)
 
 
-var aspectRuleTemplate = mustTemplate(`load("@rules_proto//proto:defs.bzl", "ProtoInfo")
+var aspectRuleTemplate = mustTemplate(`"""Generated definition of {{ .Rule.Name }}."""
+
+load("@rules_proto//proto:defs.bzl", "ProtoInfo")
 load(
     "//:defs.bzl",
-    "ProtoPluginInfo",
     "ProtoLibraryAspectNodeInfo",
+    "ProtoPluginInfo",
     "proto_compile_aspect_attrs",
     "proto_compile_aspect_impl",
     "proto_compile_attrs",

tools/rulegen/main.go

@@ -220,15 +220,20 @@ func mustWriteLanguageExampleBazelrcFile(dir string, lang *Language, rule *Rule)
 
 func mustWriteLanguageDefs(dir string, lang *Language) {
 	out := &LineWriter{}
-	out.w("# Aggregate all `%s` rules to one loadable file", lang.Name)
+	out.w("\"\"\"%s protobuf and grpc rules.\"\"\"", lang.Name)
+	out.ln()
+
 	for _, rule := range lang.Rules {
 		out.w(`load(":%s.bzl", _%s = "%s")`, rule.Name, rule.Name, rule.Name)
 	}
 	out.ln()
+
+	out.w("# Export %s rules", lang.Name)
 	for _, rule := range lang.Rules {
 		out.w(`%s = _%s`, rule.Name, rule.Name)
 	}
 	out.ln()
+
 	if len(lang.Aliases) > 0 {
 		out.w(`# Aliases`)