fix(openflare): improve SWOfflineDomains validation and snapshot diff logic

internal/apps/openflare/config_version/logics.go

@@ -540,7 +540,7 @@ func diffOpenRestyOptionDetails(left openRestyConfigSnapshot, right openRestyCon
 	appendIfChanged("OriginErrorPageGetOnly", fmt.Sprintf("%t", left.OriginErrorPageGetOnly), fmt.Sprintf("%t", right.OriginErrorPageGetOnly))
 	appendIfChanged("SWOfflineEnabled", fmt.Sprintf("%t", left.SWOfflineEnabled), fmt.Sprintf("%t", right.SWOfflineEnabled))
 	appendIfChanged("SWOfflineHTML", left.SWOfflineHTML, right.SWOfflineHTML)
-	appendIfChanged("SWOfflineDomains", strings.Join(left.SWOfflineDomains, ","), strings.Join(right.SWOfflineDomains, ","))
+	appendIfChanged("SWOfflineDomains", encodeSWOfflineDomains(left.SWOfflineDomains), encodeSWOfflineDomains(right.SWOfflineDomains))
 	return changes
 }
 
@@ -555,6 +555,17 @@ func encodeOriginErrorPageStatusCodes(tags []string) string {
 	return string(payload)
 }
 
+func encodeSWOfflineDomains(domains []string) string {
+	if len(domains) == 0 {
+		return ""
+	}
+	payload, err := json.Marshal(domains)
+	if err != nil {
+		return strings.Join(domains, ",")
+	}
+	return string(payload)
+}
+
 func extractOptionDiffKeys(details []ConfigOptionDiffItem) []string {
 	keys := make([]string, 0, len(details))
 	for _, item := range details {

internal/apps/openflare/config_version/origin_error_page_snapshot_test.go

@@ -101,3 +101,21 @@ func TestDiffOpenRestyOptionDetailsOriginErrorPage(t *testing.T) {
 	assert.Equal(t, "", keys["OriginErrorPageHTML"].PreviousValue)
 	assert.Equal(t, "<p>x</p>", keys["OriginErrorPageHTML"].CurrentValue)
 }
+
+func TestDiffOpenRestyOptionDetailsSWOfflineDomains(t *testing.T) {
+	t.Parallel()
+
+	left := openRestyConfigSnapshot{
+		SWOfflineDomains: []string{"a.com,b.com"},
+	}
+	right := openRestyConfigSnapshot{
+		SWOfflineDomains: []string{"a.com", "b.com"},
+	}
+	details := diffOpenRestyOptionDetails(left, right)
+	keys := make(map[string]ConfigOptionDiffItem, len(details))
+	for _, item := range details {
+		keys[item.Key] = item
+	}
+	assert.Equal(t, `["a.com,b.com"]`, keys["SWOfflineDomains"].PreviousValue)
+	assert.Equal(t, `["a.com","b.com"]`, keys["SWOfflineDomains"].CurrentValue)
+}

internal/apps/openflare/option/openresty_validators.go

@@ -260,7 +260,7 @@ func validateSWOfflineHTML(key, value string) error {
 
 func validateSWOfflineDomains(key, value string) error {
 	var domains []string
-	if err := json.Unmarshal([]byte(value), &domains); err != nil {
+	if err := json.Unmarshal([]byte(value), &domains); err != nil || domains == nil {
 		return fmt.Errorf("%s 必须为 JSON 字符串数组", key)
 	}
 	if len(domains) > maxSWOfflineDomains {

internal/apps/openflare/option/openresty_validators_test.go

@@ -106,6 +106,7 @@ func TestValidateSWOfflineDomains(t *testing.T) {
 		{"single", `["example.com"]`, true},
 		{"multiple", `["example.com","api.example.com"]`, true},
 		{"invalid json", `not-json`, false},
+		{"null", "null", false},
 		{"empty element", `[""]`, false},
 		{"duplicate", `["example.com","example.com"]`, false},
 		{"whitespace dedup", `[" Example.com ","example.com"]`, false},