Separate name elements with underscores to improve readability

Author: dacharyc

audit-cli/README.md

@@ -186,17 +186,17 @@ This command helps writers:
 
 **Output Format:**
 
-Extracted files are named: `{heading}-{first-step-title}-{hash}.rst`
+Extracted files are named: `{heading}_{first-step-title}_{hash}.rst`
 
 The filename includes:
 - **Heading**: The section heading above the procedure
 - **First step title**: The title of the first step (for readability)
 - **Hash**: A short 6-character hash of the content (for uniqueness)
 
 Examples:
-- `before-you-begin-pull-the-mongodb-docker-image-e8eeec.rst`
-- `install-mongodb-community-edition-download-the-tarball-44c437.rst`
-- `configuration-create-the-data-and-log-directories-f1d35b.rst`
+- `before-you-begin_pull-the-mongodb-docker-image_e8eeec.rst`
+- `install-mongodb-community-edition_download-the-tarball_44c437.rst`
+- `configuration_create-the-data-and-log-directories_f1d35b.rst`
 
 **Verbose Output:**
 

audit-cli/commands/extract/procedures/parser.go

@@ -107,8 +107,8 @@ func generateVariations(procedure rst.Procedure, sourceFile string, selectionFil
 
 // generateOutputFilename generates the output filename for a procedure.
 //
-// Format: {heading}-{first-step-title}-{hash}.rst
-// Example: "before-you-begin-pull-the-mongodb-docker-image-a1b2c3.rst"
+// Format: {heading}_{first-step-title}_{hash}.rst
+// Example: "before-you-begin_pull-the-mongodb-docker-image_a1b2c3.rst"
 //
 // The hash is a short (6 character) hash of the procedure content to ensure uniqueness.
 func generateOutputFilename(sourceFile string, procedure rst.Procedure, variationName string) string {
@@ -129,10 +129,10 @@ func generateOutputFilename(sourceFile string, procedure rst.Procedure, variatio
 	// If the procedure has steps, use the first step title to make the filename descriptive
 	if len(procedure.Steps) > 0 && procedure.Steps[0].Title != "" {
 		firstStepTitle := sanitizeFilename(procedure.Steps[0].Title)
-		return fmt.Sprintf("%s-%s-%s.rst", title, firstStepTitle, shortHash)
+		return fmt.Sprintf("%s_%s_%s.rst", title, firstStepTitle, shortHash)
 	}
 
-	return fmt.Sprintf("%s-%s.rst", title, shortHash)
+	return fmt.Sprintf("%s_%s.rst", title, shortHash)
 }
 
 // computeContentHash generates a hash of the procedure's content for uniqueness.

audit-cli/commands/extract/procedures/procedures_test.go

@@ -253,7 +253,7 @@ func TestGenerateOutputFilename(t *testing.T) {
 		}
 
 		// Verify filename contains a hash (6 characters before .rst)
-		parts := strings.Split(v.OutputFile, "-")
+		parts := strings.Split(v.OutputFile, "_")
 		if len(parts) < 2 {
 			t.Errorf("Filename should contain at least heading and hash: %s", v.OutputFile)
 		}
@@ -311,8 +311,8 @@ func TestContentHash(t *testing.T) {
 			filename := variations[0].OutputFile
 			// Remove .rst extension
 			nameWithoutExt := strings.TrimSuffix(filename, ".rst")
-			// Get last part after last hyphen (the hash)
-			parts := strings.Split(nameWithoutExt, "-")
+			// Get last part after last underscore (the hash)
+			parts := strings.Split(nameWithoutExt, "_")
 			hash := parts[len(parts)-1]
 			hashes = append(hashes, hash)
 		}