bretton.dev / coves
A community based topic aggregation platform built on atproto
fork atom

fix(communities): preserve moderation fields on update

Fixes data loss bug where moderationType and contentWarnings
would be erased if not explicitly provided in update request.

Previously, omitting these fields in an update request would
remove them from the PDS record. Now follows the same pattern
as other optional fields (displayName, description, etc.) by
preserving existing values when not being updated.

Impact: Prevents accidental erasure of moderation configuration
when updating other community properties.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>

bretton.dev d08c9e30 746d64cd

options
unified split
Changed files
+6
internal
core
communities
service.go
+6
internal/core/communities/service.go 
···

       
260

       
 

       
		}


     

       
261

       
 

       
	}


     

       
262

       
 

       



     

       

       

       
     

       

       

       
     

       
263

       
 

       
	if req.ModerationType != nil {


     

       
264

       
 

       
		profile["moderationType"] = *req.ModerationType


     

       

       

       
     

       

       

       
     

       
265

       
 

       
	}


     

       
266

       
 

       



     

       
267

       
 

       
	if len(req.ContentWarnings) > 0 {


     

       
268

       
 

       
		profile["contentWarnings"] = req.ContentWarnings


     

       

       

       
     

       

       

       
     

       
269

       
 

       
	}


     

       
270

       
 

       



     

       
271

       
 

       
	// Preserve counts


     
···

       
260

       
 

       
		}


     

       
261

       
 

       
	}


     

       
262

       
 

       



     

       
263

       
+

       
	// Preserve moderation settings (even if empty)


     

       
264

       
+

       
	// These fields are optional but should not be erased on update


     

       
265

       
 

       
	if req.ModerationType != nil {


     

       
266

       
 

       
		profile["moderationType"] = *req.ModerationType


     

       
267

       
+

       
	} else if existing.ModerationType != "" {


     

       
268

       
+

       
		profile["moderationType"] = existing.ModerationType


     

       
269

       
 

       
	}


     

       
270

       
 

       



     

       
271

       
 

       
	if len(req.ContentWarnings) > 0 {


     

       
272

       
 

       
		profile["contentWarnings"] = req.ContentWarnings


     

       
273

       
+

       
	} else if len(existing.ContentWarnings) > 0 {


     

       
274

       
+

       
		profile["contentWarnings"] = existing.ContentWarnings


     

       
275

       
 

       
	}


     

       
276

       
 

       



     

       
277

       
 

       
	// Preserve counts