semconvgen: Add a flag for a path to a custom capitalizations file

.chloggen/528-semconvgen-flag-for-custom-capitalizations.yaml

@@ -0,0 +1,16 @@
+# One of 'breaking', 'deprecation', 'new_component', 'enhancement', 'bug_fix'
+change_type: enhancement
+
+# The name of the component, or a single word describing the area of concern, (e.g. crosslink)
+component: semconvgen
+
+# A brief description of the change.  Surround your text with quotes ("") if it needs to start with a backtick (`).
+note: "Add `--capitalizations-path` to allow users to add additional strings to the static capitalizations slice in generator.go"
+
+# One or more tracking issues related to the change
+issues: [528]
+
+# (Optional) One or more lines of additional information to render under the primary note.
+# These lines will be padded with 2 spaces and then inserted directly into the document.
+# Use pipe (|) for multiline entries.
+subtext:

semconvgen/README.md

@@ -22,13 +22,14 @@ semconvgen -i <path to spec YAML> -t <path to template> -o <path to output>
 
 A full list of available options:
 
-```txt
-  -c, --container string    Container image ID (default "otel/semconvgen")
-  -f, --filename string     Filename for templated output. If not specified 'basename(inputPath).go' will be used.
-  -i, --input string        Path to semantic convention definition YAML. Should be a directory in the specification git repository.
-      --only string         Process only semantic conventions of the specified type. {span, resource, event, metric_group, metric, units, scope, attribute_group}
-  -o, --output string       Path to output target. Must be either an absolute path or relative to the repository root. If unspecified will output to a sub-directory with the name matching the version number specified via --specver flag.
-  -p, --parameters string   List of key=value pairs separated by comma. These values are fed into the template as-is.
-  -s, --specver string      Version of semantic convention to generate. Must be an existing version tag in the specification git repository.
-  -t, --template string     Template filename (default "template.j2")
-```
+````txt
+  -z, --capitalizations-path string   Path to a file containing additional newline-separated capitalization strings.
+  -c, --container string              Container image ID (default "otel/semconvgen")
+  -f, --filename string               Filename for templated output. If not specified 'basename(inputPath).go' will be used.
+  -i, --input string                  Path to semantic convention definition YAML. Should be a directory in the specification git repository.
+      --only string                   Process only semantic conventions of the specified type. {span, resource, event, metric_group, metric, units, scope, attribute_group}
+  -o, --output string                 Path to output target. Must be either an absolute path or relative to the repository root. If unspecified will output to a sub-directory with the name matching the version number specified via --specver flag.
+  -p, --parameters string             List of key=value pairs separated by comma. These values are fed into the template as-is.
+  -s, --specver string                Version of semantic convention to generate. Must be an existing version tag in the specification git repository.
+  -t, --template string               Template filename (default "template.j2")```
+````

semconvgen/generator.go

@@ -15,6 +15,7 @@
 package main
 
 import (
+	"bufio"
 	"bytes"
 	"errors"
 	"fmt"
@@ -48,6 +49,7 @@ func main() {
 	flag.StringVarP(&cfg.outputFilename, "filename", "f", "", "Filename for templated output. If not specified 'basename(inputPath).go' will be used.")
 	flag.StringVarP(&cfg.templateFilename, "template", "t", "template.j2", "Template filename")
 	flag.StringVarP(&cfg.templateParameters, "parameters", "p", "", "List of key=value pairs separated by comma. These values are fed into the template as-is.")
+	flag.StringVarP(&cfg.capitalizationsPath, "capitalizations-path", "z", "", "Path to a file containing additional newline-separated capitalization strings.")
 	flag.Parse()
 
 	cfg, err := validateConfig(cfg)
@@ -74,14 +76,15 @@ func main() {
 }
 
 type config struct {
-	inputPath          string
-	outputPath         string
-	outputFilename     string
-	templateFilename   string
-	templateParameters string
-	onlyType           string
-	containerImage     string
-	specVersion        string
+	inputPath           string
+	outputPath          string
+	outputFilename      string
+	templateFilename    string
+	templateParameters  string
+	onlyType            string
+	containerImage      string
+	specVersion         string
+	capitalizationsPath string
 }
 
 func validateConfig(cfg config) (config, error) {
@@ -126,6 +129,12 @@ func validateConfig(cfg config) (config, error) {
 		cfg.templateFilename = path.Join(pwd, cfg.templateFilename)
 	}
 
+	if cfg.capitalizationsPath != "" {
+		if _, err := os.Stat(cfg.capitalizationsPath); os.IsNotExist(err) {
+			return config{}, fmt.Errorf("capitalizations file does not exist: %w", err)
+		}
+	}
+
 	return cfg, nil
 }
 
@@ -282,7 +291,7 @@ func checkoutSpecToDir(cfg config, toDir string) (doneFunc func(), err error) {
 	return doneFunc, nil
 }
 
-var capitalizations = []string{
+var staticCapitalizations = []string{
 	"ACL",
 	"AIX",
 	"AKS",
@@ -379,6 +388,31 @@ var capitalizations = []string{
 	"OTel",
 }
 
+func capitalizations(capitalizationsPath string) ([]string, error) {
+	c := append([]string(nil), staticCapitalizations...)
+
+	if capitalizationsPath != "" {
+		// #nosec G304
+		file, err := os.Open(capitalizationsPath)
+		if err != nil {
+			return nil, fmt.Errorf("unable to open capitalizations file: %w", err)
+		}
+		defer file.Close()
+
+		scanner := bufio.NewScanner(file)
+		for scanner.Scan() {
+			capitalization := strings.TrimSpace(scanner.Text())
+			c = append(c, capitalization)
+		}
+
+		if err := scanner.Err(); err != nil {
+			return nil, fmt.Errorf("unable to read capitalizations file: %w", err)
+		}
+	}
+
+	return c, nil
+}
+
 // These are not simple capitalization fixes, but require string replacement.
 // All occurrences of the key will be replaced with the corresponding value.
 var replacements = map[string]string{
@@ -394,6 +428,10 @@ func fixIdentifiers(cfg config) error {
 		return fmt.Errorf("unable to read file: %w", err)
 	}
 
+	capitalizations, err := capitalizations(cfg.capitalizationsPath)
+	if err != nil {
+		return fmt.Errorf("unable to combine custom and static capitalizations: %w", err)
+	}
 	caser := cases.Title(language.Und)
 	for _, init := range capitalizations {
 		// Match the title-cased capitalization target, asserting that its followed by