fix(config): default profiles is empty array, not nil

config.Default() set Media.Profiles to nil, which encoding/json
emits as "profiles": null. ONVIF clients and downstream JSON
consumers expect that field to always be an array — null vs []
is a needless schema gotcha. Initialize with an empty slice so
the on-disk shape is "profiles": [].

Add TestDefaultProfilesIsEmptyArrayNotNull to lock both the in-
memory non-nil-ness and the marshaled JSON form.

Replace the hardcoded Version: 1 fixture literal with
config.CurrentVersion in cmd/cli/control_test.go,
internal/simulator/{simulator,rtsp}_test.go, and
internal/tui/{adapter,mock}_test.go so a future schema bump
fails the tests at compile time rather than silently passing
with a stale literal.

Author: GyeongHoKim

cmd/cli/control_test.go

@@ -24,7 +24,7 @@ func writeTempConfig(t *testing.T) (cfgPath string, cleanup func()) {
 	t.Helper()
 	dir := t.TempDir()
 	cfg := config.Config{
-		Version: 1,
+		Version: config.CurrentVersion,
 		Device: config.DeviceConfig{
 			UUID:         "urn:uuid:00000000-0000-4000-8000-000000000001",
 			Manufacturer: "Test", Model: "SimCam", Serial: "SN-1",

internal/config/config.go

@@ -923,7 +923,10 @@ func Default() Config {
 			},
 		},
 		Network: NetworkConfig{HTTPPort: 8080, RTSPPort: DefaultRTSPPort},
-		Media:   MediaConfig{Profiles: nil},
+		// Use an empty slice (not nil) so json.Marshal emits
+		// "profiles": [] rather than "profiles": null. ONVIF clients
+		// and JSON consumers expect the field to always be an array.
+		Media: MediaConfig{Profiles: []ProfileConfig{}},
 		Events: EventsConfig{
 			MaxPullPoints:       defaultMaxPullPoints,
 			SubscriptionTimeout: "1h",

internal/config/path_test.go

@@ -1,10 +1,12 @@
 package config_test
 
 import (
+	"encoding/json"
 	"errors"
 	"os"
 	"path/filepath"
 	"runtime"
+	"strings"
 	"sync"
 	"testing"
 
@@ -147,6 +149,28 @@ func TestDefaultPassesValidate(t *testing.T) {
 	}
 }
 
+// TestDefaultProfilesIsEmptyArrayNotNull pins the JSON shape of
+// Default(): media.profiles must marshal to "[]" rather than "null"
+// so ONVIF clients (and json-schema-style consumers) always see an
+// array. A nil slice would emit "null" and break that contract.
+func TestDefaultProfilesIsEmptyArrayNotNull(t *testing.T) {
+	t.Parallel()
+	cfg := config.Default()
+	if cfg.Media.Profiles == nil {
+		t.Fatal("Default().Media.Profiles is nil; expected an empty slice")
+	}
+	if len(cfg.Media.Profiles) != 0 {
+		t.Fatalf("Default().Media.Profiles is not empty: %+v", cfg.Media.Profiles)
+	}
+	data, err := json.Marshal(cfg.Media)
+	if err != nil {
+		t.Fatalf("marshal: %v", err)
+	}
+	if !strings.Contains(string(data), `"profiles":[]`) {
+		t.Errorf("expected profiles to marshal as [], got: %s", string(data))
+	}
+}
+
 // TestDefaultUUIDIsUnique guards against the original hardcoded
 // urn:uuid:00000000-0000-4000-8000-000000000001 sneaking back in. Two
 // successive Default() calls must not collide; ONVIF clients (and

internal/simulator/rtsp_test.go

@@ -39,7 +39,7 @@ func newSimulatorWithMediaFile(t *testing.T, mediaPath string) (sim *Simulator,
 	cfgPath := filepath.Join(dir, config.FileName)
 
 	cfg := config.Config{
-		Version: 1,
+		Version: config.CurrentVersion,
 		Device: config.DeviceConfig{
 			UUID:         "urn:uuid:00000000-0000-4000-8000-000000000001",
 			Manufacturer: "Test",

internal/simulator/simulator_test.go

@@ -44,7 +44,7 @@ func newTestSimulator(t *testing.T) (sim *Simulator, cleanup func()) {
 	cfgPath := filepath.Join(dir, config.FileName)
 
 	cfg := config.Config{
-		Version: 1,
+		Version: config.CurrentVersion,
 		Device: config.DeviceConfig{
 			UUID:         "urn:uuid:00000000-0000-4000-8000-000000000001",
 			Manufacturer: "Test",

internal/tui/adapter_test.go

@@ -31,7 +31,7 @@ func newTestAdapter(t *testing.T) (sa *simulatorAdapter, cleanup func()) {
 	}
 
 	cfg := config.Config{
-		Version: 1,
+		Version: config.CurrentVersion,
 		Device: config.DeviceConfig{
 			UUID:         "urn:uuid:00000000-0000-4000-8000-aaaaaaaaaaaa",
 			Manufacturer: "Test",

internal/tui/mock_test.go

@@ -23,7 +23,7 @@ type mockSim struct {
 func newMockSim() *mockSim {
 	return &mockSim{
 		snapshot: config.Config{
-			Version: 1,
+			Version: config.CurrentVersion,
 			Device: config.DeviceConfig{
 				UUID:         "urn:uuid:11111111-1111-1111-1111-111111111111",
 				Manufacturer: "Acme",