feat: Enhance output formatting with dynamic section boxes

- Introduced a new function `printSectionBox` to create dynamically-sized boxes around section names in the output, ensuring a minimum width for better readability.
- Updated the `printSectionChanges` function to utilize the new box formatting, improving the visual structure of section headers.
- Corrected error variable naming for consistency in argument validation within the `runVersionsDiff` function.

Author: a-spiker

cmd/diff.go

@@ -351,11 +351,11 @@ func runServerDiff(cmd *cobra.Command, args []string) error {
 // runVersionsDiff compares the configuration between two Aerospike server versions.
 func runVersionsDiff(cmd *cobra.Command, args []string) error {
 	if len(args) < diffVersionsArgMin {
-		return errSchemaDiffTWrongArgs
+		return errSchemaDiffWrongArgs
 	}
 
 	if len(args) > diffVersionsArgMax {
-		return errSchemaDiffTWrongArgs
+		return errSchemaDiffWrongArgs
 	}
 
 	version1 := args[0]

cmd/diff_utils.go

@@ -201,6 +201,11 @@ func extractSection(path string, validSections map[string]bool) string {
 // Output formatting
 // ---------------
 
+const (
+	// minBoxWidth defines the minimum width for the section box.
+	minBoxWidth = 50
+)
+
 // printChangeSummary prints a formatted summary of schema changes.
 func printChangeSummary(summary ChangeSummary, options DiffOptions) {
 	// Print header with versions
@@ -247,9 +252,7 @@ func printHeader(summary ChangeSummary, options DiffOptions) {
 func printSectionChanges(section string, changes SectionChanges, options DiffOptions) {
 	// Section header
 	if options.Verbose {
-		fmt.Fprintf(os.Stdout, "\n╭─────────────────────────────────────────────────────────────╮\n")
-		fmt.Fprintf(os.Stdout, "│ SECTION: %-50s │\n", strings.ToUpper(section))
-		fmt.Fprintf(os.Stdout, "╰─────────────────────────────────────────────────────────────╯\n")
+		printSectionBox(section)
 	} else {
 		fmt.Fprintf(os.Stdout, "\n[SECTION: %s]\n", strings.ToUpper(section))
 	}
@@ -485,6 +488,40 @@ func formatValue(value interface{}) string {
 // Utility functions
 // ---------------
 
+// printSectionBox prints a dynamically-sized box around the section name.
+// The box width adjusts to accommodate the section name while maintaining a minimum width.
+func printSectionBox(section string) {
+	upperSection := strings.ToUpper(section)
+	sectionText := "SECTION: " + upperSection
+
+	// Calculate the required width (text + padding)
+	const padding = 2 // 1 space on each side
+	requiredWidth := len(sectionText) + padding
+
+	// Use the larger of minimum width or required width
+	boxWidth := minBoxWidth
+	if requiredWidth > minBoxWidth {
+		boxWidth = requiredWidth
+	}
+
+	// Calculate padding for centering the text
+	totalPadding := boxWidth - len(sectionText)
+	leftPadding := totalPadding / padding
+	rightPadding := totalPadding - leftPadding
+
+	// Create the box borders
+	topBorder := "╭" + strings.Repeat("─", boxWidth) + "╮"
+	bottomBorder := "╰" + strings.Repeat("─", boxWidth) + "╯"
+
+	// Print the box
+	fmt.Fprintf(os.Stdout, "\n%s\n", topBorder)
+	fmt.Fprintf(os.Stdout, "│%s%s%s│\n",
+		strings.Repeat(" ", leftPadding),
+		sectionText,
+		strings.Repeat(" ", rightPadding))
+	fmt.Fprintf(os.Stdout, "%s\n", bottomBorder)
+}
+
 // formatPath formats a JSON path into a more human-readable form.
 func formatPath(path string) string {
 	parts := strings.Split(path, "/")

cmd/utils.go

@@ -52,7 +52,7 @@ var (
 	)
 
 	// Schema diff errors.
-	errSchemaDiffTWrongArgs = errors.New("diff versions requires exactly 2 version arguments")
+	errSchemaDiffWrongArgs  = errors.New("diff versions requires exactly 2 version arguments")
 	errInvalidSchemaVersion = errors.New("invalid schema version")
 
 	errUnableToGenerateConfigFile       = errors.New("unable to generate config file")