Fix CI failures: add display_name support and fix auto-generation

- Add 'nodisplayname' to spell-check dictionary
- Remove incorrect auto-generation of DisplayName in loader.go
- Update test expectations for optional display_name field
- Add explicit display_name to test metadata files

Co-authored-by: SteveYi <[email protected]>

Author: thc1006

.github/workflows/utils/cspell.json

@@ -320,6 +320,7 @@
       "nonclobbering",
       "nopexporter",
       "nopreceiver",
+      "nodisplayname",
       "nosuchprocessor",
       "notls",
       "obsreceiver",

cmd/mdatagen/internal/loader.go

@@ -59,9 +59,6 @@ func LoadMetadata(filePath string) (Metadata, error) {
 	if md.GeneratedPackageName == "" {
 		md.GeneratedPackageName = "metadata"
 	}
-	if md.DisplayName == "" {
-		md.DisplayName = generateDefaultDisplayName(md.Type)
-	}
 
 	if err := md.Validate(); err != nil {
 		return md, err
@@ -73,13 +70,6 @@ func LoadMetadata(filePath string) (Metadata, error) {
 	return md, nil
 }
 
-func generateDefaultDisplayName(componentType string) string {
-	if componentType == "" {
-		return ""
-	}
-	return strings.ToUpper(componentType[:1]) + componentType[1:]
-}
-
 var componentTypes = []string{
 	"connector",
 	"exporter",

cmd/mdatagen/internal/loader_test.go

@@ -559,7 +559,7 @@ func TestLoadMetadata(t *testing.T) {
 			name: "testdata/no_display_name.yaml",
 			want: Metadata{
 				Type:                 "nodisplayname",
-				DisplayName:          "Nodisplayname",
+				DisplayName:          "",
 				GeneratedPackageName: "metadata",
 				ScopeName:            "go.opentelemetry.io/collector/cmd/mdatagen/internal/testdata",
 				PackageName:          "go.opentelemetry.io/collector/cmd/mdatagen/internal/testdata",

cmd/mdatagen/internal/samplereceiver/README.md

@@ -4,6 +4,7 @@ This receiver is used for testing purposes to check the output of mdatagen.
 # Sample
 
 <!-- status autogenerated section -->
+# Sample
 | Status        |           |
 | ------------- |-----------|
 | Stability     | [deprecated]: profiles   |

cmd/mdatagen/internal/samplereceiver/metadata.yaml

@@ -1,6 +1,7 @@
 # Sample metadata file with all available configurations for a receiver.
 
 type: sample
+display_name: Sample
 scope_name: go.opentelemetry.io/collector/internal/receiver/samplereceiver
 github_project: open-telemetry/opentelemetry-collector
 

cmd/mdatagen/internal/templates/readme.md.tmpl

@@ -1,8 +1,8 @@
+<!-- status autogenerated section -->
 {{- if .DisplayName }}
 # {{ .DisplayName }}
 
-{{ end -}}
-<!-- status autogenerated section -->
+{{- end }}
 {{- if len .Status.Stability }}
 | Status        |           |
 | ------------- |-----------|

cmd/mdatagen/internal/testdata/empty_test_config.yaml

@@ -1,4 +1,5 @@
 type: test
+display_name: Test
 
 status:
   class: receiver

cmd/mdatagen/internal/testdata/generated_package_name.yaml

@@ -1,4 +1,5 @@
 type: custom
+display_name: Custom
 
 generated_package_name: customname
 

cmd/mdatagen/internal/testdata/parent.yaml

@@ -1,3 +1,4 @@
 type: subcomponent
+display_name: Subcomponent
 
 parent: parentComponent