refactor(config): remove unused buffer configuration parameter

examples/advanced.yaml

@@ -25,7 +25,6 @@ prefix:
 
 output:
   format: "text"
-  buffer: "line"
 
 log_level:
   default_stdout: "INFO"

examples/basic.yaml

@@ -23,7 +23,6 @@ prefix:
 
 output:
   format: "text"
-  buffer: "line"
 
 log_level:
   default_stdout: "INFO"

examples/minimal.yaml

@@ -17,7 +17,6 @@ prefix:
 
 output:
   format: "text"
-  buffer: "line"
 
 log_level:
   default_stdout: "INFO"

internal/testutils/helpers.go

@@ -191,7 +191,6 @@ prefix:
 
 output:
   format: "text"
-  buffer: "line"
 
 log_level:
   default_stdout: "INFO"

pkg/apperrors/apperrors.go

@@ -13,7 +13,6 @@ var (
 	ErrInvalidUserFormat           = errors.New("invalid user format")
 	ErrInvalidPIDFormat            = errors.New("invalid PID format")
 	ErrInvalidOutputFormat         = errors.New("invalid output format")
-	ErrInvalidBufferMode           = errors.New("invalid buffer mode")
 	ErrInvalidStdoutLogLevel       = errors.New("invalid default stdout log level")
 	ErrInvalidStderrLogLevel       = errors.New("invalid default stderr log level")
 	ErrInvalidLogLevel             = errors.New("invalid log level")

pkg/apperrors/apperrors_test.go

@@ -46,11 +46,6 @@ func TestConfigurationErrors(t *testing.T) {
 			err:      ErrInvalidOutputFormat,
 			expected: "invalid output format",
 		},
-		{
-			name:     "ErrInvalidBufferMode",
-			err:      ErrInvalidBufferMode,
-			expected: "invalid buffer mode",
-		},
 		{
 			name:     "ErrInvalidStdoutLogLevel",
 			err:      ErrInvalidStdoutLogLevel,
@@ -245,7 +240,6 @@ func TestErrorWrapping(t *testing.T) {
 		ErrInvalidUserFormat,
 		ErrInvalidPIDFormat,
 		ErrInvalidOutputFormat,
-		ErrInvalidBufferMode,
 		ErrInvalidStdoutLogLevel,
 		ErrInvalidStderrLogLevel,
 		ErrInvalidLogLevel,

pkg/config/config.go

@@ -57,7 +57,6 @@ type PIDConfig struct {
 // OutputConfig contains output formatting configuration.
 type OutputConfig struct {
 	Format string `yaml:"format"`
-	Buffer string `yaml:"buffer"`
 }
 
 // LogLevelConfig contains log level detection configuration.
@@ -133,7 +132,6 @@ func getDefaultConfig() *Config {
 		},
 		Output: OutputConfig{
 			Format: "text",
-			Buffer: "line",
 		},
 		LogLevel: LogLevelConfig{
 			DefaultStdout: "INFO",

pkg/config/config_test.go

@@ -31,7 +31,6 @@ func TestLoadConfig_DefaultConfig(t *testing.T) {
 	assert.True(t, cfg.Prefix.PID.Enabled)
 	assert.Equal(t, "decimal", cfg.Prefix.PID.Format)
 	assert.Equal(t, "text", cfg.Output.Format)
-	assert.Equal(t, "line", cfg.Output.Buffer)
 	assert.Equal(t, "INFO", cfg.LogLevel.DefaultStdout)
 	assert.Equal(t, "ERROR", cfg.LogLevel.DefaultStderr)
 	assert.True(t, cfg.LogLevel.Detection.Enabled)
@@ -57,7 +56,6 @@ func TestLoadConfig_WithValidConfigFile(t *testing.T) {
 	assert.True(t, cfg.Prefix.PID.Enabled)
 	assert.Equal(t, "decimal", cfg.Prefix.PID.Format)
 	assert.Equal(t, "text", cfg.Output.Format)
-	assert.Equal(t, "line", cfg.Output.Buffer)
 	assert.Equal(t, "INFO", cfg.LogLevel.DefaultStdout)
 	assert.Equal(t, "ERROR", cfg.LogLevel.DefaultStderr)
 	assert.True(t, cfg.LogLevel.Detection.Enabled)
@@ -416,7 +414,6 @@ func TestGetDefaultConfig(t *testing.T) {
 	assert.True(t, cfg.Prefix.PID.Enabled)
 	assert.Equal(t, "decimal", cfg.Prefix.PID.Format)
 	assert.Equal(t, "text", cfg.Output.Format)
-	assert.Equal(t, "line", cfg.Output.Buffer)
 	assert.Equal(t, "INFO", cfg.LogLevel.DefaultStdout)
 	assert.Equal(t, "ERROR", cfg.LogLevel.DefaultStderr)
 	assert.True(t, cfg.LogLevel.Detection.Enabled)

pkg/config/validation.go

@@ -130,13 +130,6 @@ func (c *Config) validateOutput() error {
 			apperrors.ErrInvalidOutputFormat, c.Output.Format, strings.Join(validFormats, ", "))
 	}
 
-	validBuffers := []string{"line", "none", "full"}
-
-	if !slices.Contains(validBuffers, c.Output.Buffer) {
-		return fmt.Errorf("%w '%s', valid modes: %s",
-			apperrors.ErrInvalidBufferMode, c.Output.Buffer, strings.Join(validBuffers, ", "))
-	}
-
 	return nil
 }
 

pkg/config/validation_test.go

@@ -268,46 +268,27 @@ func TestConfig_ValidateOutput(t *testing.T) {
 	tests := []struct {
 		name        string
 		format      string
-		buffer      string
 		expectError bool
 		expectedErr error
 	}{
 		{
-			name:   "valid text format with line buffer",
+			name:   "valid text format",
 			format: "text",
-			buffer: "line",
 		},
 		{
-			name:   "valid json format with none buffer",
+			name:   "valid json format",
 			format: "json",
-			buffer: "none",
 		},
 		{
-			name:   "valid structured format with full buffer",
+			name:   "valid structured format",
 			format: "structured",
-			buffer: "full",
 		},
 		{
 			name:        "invalid format",
 			format:      "invalid",
-			buffer:      "line",
 			expectError: true,
 			expectedErr: apperrors.ErrInvalidOutputFormat,
 		},
-		{
-			name:        "invalid buffer",
-			format:      "text",
-			buffer:      "invalid",
-			expectError: true,
-			expectedErr: apperrors.ErrInvalidBufferMode,
-		},
-		{
-			name:        "both invalid",
-			format:      "invalid",
-			buffer:      "invalid",
-			expectError: true,
-			expectedErr: apperrors.ErrInvalidOutputFormat, // First error encountered
-		},
 	}
 
 	for _, tt := range tests {
@@ -316,7 +297,6 @@ func TestConfig_ValidateOutput(t *testing.T) {
 
 			cfg := getDefaultConfig()
 			cfg.Output.Format = tt.format
-			cfg.Output.Buffer = tt.buffer
 
 			err := cfg.Validate()
 
@@ -535,7 +515,6 @@ func TestConfig_ValidateIntegration(t *testing.T) {
 		},
 		Output: OutputConfig{
 			Format: "invalid", // Invalid: not text/json/structured
-			Buffer: "invalid", // Invalid: not line/none/full
 		},
 		LogLevel: LogLevelConfig{
 			DefaultStdout: "INVALID", // Invalid: not a valid level
@@ -968,46 +947,6 @@ func TestConfig_ValidateColors_CaseInsensitivity(t *testing.T) {
 	}
 }
 
-// TestConfig_ValidateBufferMode_AllModes tests all valid and invalid buffer modes.
-func TestConfig_ValidateBufferMode_AllModes(t *testing.T) {
-	t.Parallel()
-
-	tests := []struct {
-		name        string
-		bufferMode  string
-		expectError bool
-	}{
-		// Valid modes
-		{name: "valid line", bufferMode: "line", expectError: false},
-		{name: "valid none", bufferMode: "none", expectError: false},
-		{name: "valid full", bufferMode: "full", expectError: false},
-		// Invalid modes
-		{name: "invalid auto", bufferMode: "auto", expectError: true},
-		{name: "invalid block", bufferMode: "block", expectError: true},
-		{name: "invalid empty", bufferMode: "", expectError: true},
-		{name: "invalid mixed case", bufferMode: "Line", expectError: true},
-		{name: "invalid uppercase", bufferMode: "LINE", expectError: true},
-	}
-
-	for _, tt := range tests {
-		t.Run(tt.name, func(t *testing.T) {
-			t.Parallel()
-
-			cfg := getDefaultConfig()
-			cfg.Output.Buffer = tt.bufferMode
-
-			err := cfg.Validate()
-
-			if tt.expectError {
-				assert.Error(t, err)
-				assert.ErrorIs(t, err, apperrors.ErrInvalidBufferMode)
-			} else {
-				assert.NoError(t, err)
-			}
-		})
-	}
-}
-
 // TestConfig_ValidateOutputFormat_AllFormats tests all valid and invalid output formats.
 func TestConfig_ValidateOutputFormat_AllFormats(t *testing.T) {
 	t.Parallel()