Scary note

sync3/caches/user.go

@@ -52,6 +52,7 @@ type UserRoomData struct {
 	// Avatars set in m.room.avatar take precedence; if this is missing and the room is
 	// a DM with one other user joined or invited, we fall back to that user's
 	// avatar (if any) as specified in their membership event in that room.
+	// XXX: this field appears to be write-only?!!?
 	ResolvedAvatarURL string
 
 	// Spaces is the set of room IDs of spaces that this room is part of.

tests-e2e/lists_test.go

@@ -1652,6 +1652,7 @@ func TestAvatarFieldInRoomResponse(t *testing.T) {
 	})
 
 	t.Run("Changing DM flag", func(t *testing.T) {
+		// XXX: the test's expectations might very well be out of date now
 		t.Skip("TODO: unimplemented")
 		t.Log("Alice clears the DM flag on Bob's room.")
 		alice.MustSetGlobalAccountData(t, "m.direct", map[string]interface{}{