Bugfix in the seriatim adapter

internal/config/config.go

@@ -216,7 +216,7 @@ type RenderConfig struct {
 	Format            string `yaml:"format"`
 	Title             string `yaml:"title"`
 	IncludeTimestamps *bool  `yaml:"include_timestamps"`
-	IncludeSegmentIDs bool   `yaml:"include_segment_ids"`
+	IncludeSegmentIDs *bool  `yaml:"include_segment_ids"`
 	IncludeMetadata   bool   `yaml:"include_metadata"`
 }
 

internal/config/defaults.go

@@ -44,7 +44,7 @@ const (
 	DefaultRenderFormat       = "markdown"
 	DefaultRenderTitle        = ""
 	DefaultRenderTimestamps   = true
-	DefaultRenderSegmentIDs   = false
+	DefaultRenderSegmentIDs   = true
 	DefaultRenderMetadata     = false
 
 	DefaultNormalizeOutputPath   = artifactmodel.TranscriptPathFinal

internal/config/load.go

@@ -527,6 +527,9 @@ func applyRenderDefaults(cfg **RenderConfig) {
 	if (*cfg).IncludeTimestamps == nil {
 		(*cfg).IncludeTimestamps = boolPtr(DefaultRenderTimestamps)
 	}
+	if (*cfg).IncludeSegmentIDs == nil {
+		(*cfg).IncludeSegmentIDs = boolPtr(DefaultRenderSegmentIDs)
+	}
 }
 
 func applyNormalizeDefaults(cfg *NormalizeConfig) {

internal/config/render_test.go

@@ -30,8 +30,8 @@ func TestRenderLoadAndValidate(t *testing.T) {
 				if cfg.Pipeline.Render.IncludeTimestamps == nil || !*cfg.Pipeline.Render.IncludeTimestamps {
 					t.Fatalf("render.include_timestamps = %#v, want true", cfg.Pipeline.Render.IncludeTimestamps)
 				}
-				if cfg.Pipeline.Render.IncludeSegmentIDs {
-					t.Fatalf("render.include_segment_ids = true, want false")
+				if cfg.Pipeline.Render.IncludeSegmentIDs == nil || !*cfg.Pipeline.Render.IncludeSegmentIDs {
+					t.Fatalf("render.include_segment_ids = %#v, want true", cfg.Pipeline.Render.IncludeSegmentIDs)
 				}
 				if cfg.Pipeline.Render.IncludeMetadata {
 					t.Fatalf("render.include_metadata = true, want false")
@@ -59,14 +59,26 @@ func TestRenderLoadAndValidate(t *testing.T) {
 				if cfg.Pipeline.Render.IncludeTimestamps == nil || *cfg.Pipeline.Render.IncludeTimestamps {
 					t.Fatalf("render.include_timestamps = %#v, want false", cfg.Pipeline.Render.IncludeTimestamps)
 				}
-				if !cfg.Pipeline.Render.IncludeSegmentIDs {
-					t.Fatalf("render.include_segment_ids = false, want true")
+				if cfg.Pipeline.Render.IncludeSegmentIDs == nil || !*cfg.Pipeline.Render.IncludeSegmentIDs {
+					t.Fatalf("render.include_segment_ids = %#v, want true", cfg.Pipeline.Render.IncludeSegmentIDs)
 				}
 				if !cfg.Pipeline.Render.IncludeMetadata {
 					t.Fatalf("render.include_metadata = false, want true")
 				}
 			},
 		},
+		{
+			name: "explicit segment ids false overrides default",
+			renderYAML: `render:
+  include_segment_ids: false
+`,
+			assert: func(t *testing.T, cfg *Config) {
+				t.Helper()
+				if cfg.Pipeline.Render.IncludeSegmentIDs == nil || *cfg.Pipeline.Render.IncludeSegmentIDs {
+					t.Fatalf("render.include_segment_ids = %#v, want false", cfg.Pipeline.Render.IncludeSegmentIDs)
+				}
+			},
+		},
 		{
 			name: "invalid render format fails",
 			renderYAML: `render:

internal/config/validate.go

@@ -312,6 +312,9 @@ func validateRender(cfg *RenderConfig) error {
 	if cfg.IncludeTimestamps == nil {
 		return fmt.Errorf("pipeline.render.include_timestamps must be set (defaults should populate this)")
 	}
+	if cfg.IncludeSegmentIDs == nil {
+		return fmt.Errorf("pipeline.render.include_segment_ids must be set (defaults should populate this)")
+	}
 	format := strings.TrimSpace(cfg.Format)
 	if format != "markdown" {
 		return fmt.Errorf("pipeline.render.format must be markdown")