fix(all): Fix linter warnings

packages/moxxmpp/lib/src/message.dart

@@ -66,7 +66,7 @@ class MessageManager extends XmppManagerBase {
           stanzaTag: 'message',
           callback: _onMessage,
           priority: messageHandlerPriority,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/presence.dart

@@ -66,7 +66,7 @@ class PresenceManager extends XmppManagerBase {
           stanzaTag: 'presence',
           callback: _onPresence,
           priority: presenceHandlerPriority,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/roster/roster.dart

@@ -122,7 +122,7 @@ class RosterManager extends XmppManagerBase {
           tagName: 'query',
           tagXmlns: rosterXmlns,
           callback: _onRosterPush,
-        )
+        ),
       ];
 
   @override
@@ -277,7 +277,7 @@ class RosterManager extends XmppManagerBase {
               attributes: {
                 'ver': await _stateManager.getRosterVersion() ?? '',
               },
-            )
+            ),
           ],
         ),
       ),
@@ -319,14 +319,12 @@ class RosterManager extends XmppManagerBase {
                   tag: 'item',
                   attributes: <String, String>{
                     'jid': jid,
-                    ...title == jid.split('@')[0]
+                    if (title == jid.split('@')[0]) 'name': title,
-                        ? <String, String>{}
-                        : <String, String>{'name': title}
                   },
                   children: (groups ?? [])
                       .map((group) => XMLNode(tag: 'group', text: group))
                       .toList(),
-                )
+                ),
               ],
             ),
           ],
@@ -357,13 +355,13 @@ class RosterManager extends XmppManagerBase {
               children: [
                 XMLNode(
                   tag: 'item',
-                  attributes: <String, String>{
+                  attributes: {
                     'jid': jid,
-                    'subscription': 'remove'
+                    'subscription': 'remove',
                   },
-                )
+                ),
               ],
-            )
+            ),
           ],
         ),
       ),

packages/moxxmpp/lib/src/stanza.dart

@@ -248,15 +248,14 @@ XMLNode buildErrorElement(String type, String condition, {String? text}) {
       XMLNode.xmlns(
         tag: condition,
         xmlns: fullStanzaXmlns,
-        children: text != null
+        children: [
-            ? [
+          if (text != null)
             XMLNode.xmlns(
               tag: 'text',
               xmlns: fullStanzaXmlns,
               text: text,
-                )
+            ),
-              ]
+        ],
-            : [],
       ),
     ],
   );

packages/moxxmpp/lib/src/xeps/xep_0004.dart

@@ -10,14 +10,14 @@ class DataFormOption {
   XMLNode toXml() {
     return XMLNode(
       tag: 'option',
-      attributes: label != null
+      attributes: {
-          ? <String, dynamic>{'label': label}
+        if (label != null) 'label': label,
-          : <String, dynamic>{},
+      },
       children: [
         XMLNode(
           tag: 'value',
           text: value,
-        )
+        ),
       ],
     );
   }
@@ -45,19 +45,22 @@ class DataFormField {
     return XMLNode(
       tag: 'field',
       attributes: <String, dynamic>{
-        ...varAttr != null
+        if (varAttr != null) 'var': varAttr,
-            ? <String, dynamic>{'var': varAttr}
+        if (type != null) 'type': type,
-            : <String, dynamic>{},
+        if (label != null) 'label': label,
-        ...type != null ? <String, dynamic>{'type': type} : <String, dynamic>{},
-        ...label != null
-            ? <String, dynamic>{'label': label}
-            : <String, dynamic>{}
       },
       children: [
-        ...description != null ? [XMLNode(tag: 'desc', text: description)] : [],
+        if (description != null)
-        ...isRequired ? [XMLNode(tag: 'required')] : [],
+          XMLNode(
+            tag: 'desc',
+            text: description,
+          ),
+        if (isRequired)
+          XMLNode(
+            tag: 'required',
+          ),
         ...values.map((value) => XMLNode(tag: 'value', text: value)),
-        ...options.map((option) => option.toXml())
+        ...options.map((option) => option.toXml()),
       ],
     );
   }

packages/moxxmpp/lib/src/xeps/xep_0030/helpers.dart

@@ -13,8 +13,10 @@ Stanza buildDiscoInfoQueryStanza(JID entity, String? node) {
       XMLNode.xmlns(
         tag: 'query',
         xmlns: discoInfoXmlns,
-        attributes: node != null ? {'node': node} : {},
+        attributes: {
-      )
+          if (node != null) 'node': node,
+        },
+      ),
     ],
   );
 }
@@ -27,8 +29,10 @@ Stanza buildDiscoItemsQueryStanza(JID entity, {String? node}) {
       XMLNode.xmlns(
         tag: 'query',
         xmlns: discoItemsXmlns,
-        attributes: node != null ? {'node': node} : {},
+        attributes: {
-      )
+          if (node != null) 'node': node,
+        },
+      ),
     ],
   );
 }

packages/moxxmpp/lib/src/xeps/xep_0030/types.dart

@@ -19,13 +19,11 @@ class Identity {
   XMLNode toXMLNode() {
     return XMLNode(
       tag: 'identity',
-      attributes: <String, dynamic>{
+      attributes: {
         'category': category,
         'type': type,
         'name': name,
-        ...lang == null
+        if (lang != null) 'xml:lang': lang,
-            ? <String, dynamic>{}
-            : <String, dynamic>{'xml:lang': lang}
       },
     );
   }

packages/moxxmpp/lib/src/xeps/xep_0054.dart

@@ -38,7 +38,7 @@ class VCardManager extends XmppManagerBase {
           tagName: 'x',
           tagXmlns: vCardTempUpdate,
           callback: _onPresence,
-        )
+        ),
       ];
 
   @override
@@ -108,7 +108,7 @@ class VCardManager extends XmppManagerBase {
             XMLNode.xmlns(
               tag: 'vCard',
               xmlns: vCardTempXmlns,
-            )
+            ),
           ],
         ),
         encrypted: true,

packages/moxxmpp/lib/src/xeps/xep_0060/xep_0060.dart

@@ -38,26 +38,20 @@ class PubSubPublishOptions {
           varAttr: 'FORM_TYPE',
           type: 'hidden',
         ),
-        ...accessModel != null
+        if (accessModel != null)
-            ? [
           DataFormField(
             options: [],
             isRequired: false,
             values: [accessModel!],
             varAttr: 'pubsub#access_model',
-                )
+          ),
-              ]
+        if (maxItems != null)
-            : [],
-        ...maxItems != null
-            ? [
           DataFormField(
             options: [],
             isRequired: false,
             values: [maxItems!],
             varAttr: 'pubsub#max_items',
           ),
-              ]
-            : [],
       ],
     ).toXml();
   }
@@ -87,7 +81,7 @@ class PubSubManager extends XmppManagerBase {
           tagName: 'event',
           tagXmlns: pubsubEventXmlns,
           callback: _onPubsubMessage,
-        )
+        ),
       ];
 
   @override
@@ -314,11 +308,11 @@ class PubSubManager extends XmppManagerBase {
                   children: [
                     XMLNode(
                       tag: 'item',
-                      attributes: id != null
+                      attributes: {
-                          ? <String, String>{'id': id}
+                        if (id != null) 'id': id,
-                          : <String, String>{},
+                      },
                       children: [payload],
-                    )
+                    ),
                   ],
                 ),
                 if (pubOptions != null)
@@ -327,7 +321,7 @@ class PubSubManager extends XmppManagerBase {
                     children: [pubOptions.toXml()],
                   ),
               ],
-            )
+            ),
           ],
         ),
         shouldEncrypt: false,
@@ -422,7 +416,7 @@ class PubSubManager extends XmppManagerBase {
                   },
                 ),
               ],
-            )
+            ),
           ],
         ),
         shouldEncrypt: false,

packages/moxxmpp/lib/src/xeps/xep_0066.dart

@@ -45,7 +45,7 @@ class OOBManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the message manager
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0085.dart

@@ -69,7 +69,7 @@ class ChatStateManager extends XmppManagerBase {
           callback: _onChatStateReceived,
           // Before the message handler
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0184.dart

@@ -66,7 +66,7 @@ class MessageDeliveryReceiptManager extends XmppManagerBase {
           callback: _onDeliveryRequestReceived,
           // Before the message handler
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0191.dart

@@ -27,7 +27,7 @@ class BlockingManager extends XmppManagerBase {
           tagName: 'block',
           tagXmlns: blockingXmlns,
           callback: _blockPush,
-        )
+        ),
       ];
 
   @override
@@ -107,10 +107,12 @@ class BlockingManager extends XmppManagerBase {
               children: items.map((item) {
                 return XMLNode(
                   tag: 'item',
-                  attributes: <String, String>{'jid': item},
+                  attributes: {
+                    'jid': item,
+                  },
                 );
               }).toList(),
-            )
+            ),
           ],
         ),
       ),
@@ -128,7 +130,7 @@ class BlockingManager extends XmppManagerBase {
             XMLNode.xmlns(
               tag: 'unblock',
               xmlns: blockingXmlns,
-            )
+            ),
           ],
         ),
       ),
@@ -152,11 +154,13 @@ class BlockingManager extends XmppManagerBase {
                   .map(
                     (item) => XMLNode(
                       tag: 'item',
-                      attributes: <String, String>{'jid': item},
+                      attributes: {
+                        'jid': item,
+                      },
                     ),
                   )
                   .toList(),
-            )
+            ),
           ],
         ),
       ),
@@ -174,7 +178,7 @@ class BlockingManager extends XmppManagerBase {
             XMLNode.xmlns(
               tag: 'blocklist',
               xmlns: blockingXmlns,
-            )
+            ),
           ],
         ),
       ),

packages/moxxmpp/lib/src/xeps/xep_0198/nonzas.dart

@@ -5,7 +5,10 @@ class StreamManagementEnableNonza extends XMLNode {
   StreamManagementEnableNonza()
       : super(
           tag: 'enable',
-          attributes: <String, String>{'xmlns': smXmlns, 'resume': 'true'},
+          attributes: {
+            'xmlns': smXmlns,
+            'resume': 'true',
+          },
         );
 }
 
@@ -13,10 +16,10 @@ class StreamManagementResumeNonza extends XMLNode {
   StreamManagementResumeNonza(String id, int h)
       : super(
           tag: 'resume',
-          attributes: <String, String>{
+          attributes: {
             'xmlns': smXmlns,
             'previd': id,
-            'h': h.toString()
+            'h': h.toString(),
           },
         );
 }
@@ -25,7 +28,10 @@ class StreamManagementAckNonza extends XMLNode {
   StreamManagementAckNonza(int h)
       : super(
           tag: 'a',
-          attributes: <String, String>{'xmlns': smXmlns, 'h': h.toString()},
+          attributes: {
+            'xmlns': smXmlns,
+            'h': h.toString(),
+          },
         );
 }
 
@@ -33,7 +39,7 @@ class StreamManagementRequestNonza extends XMLNode {
   StreamManagementRequestNonza()
       : super(
           tag: 'r',
-          attributes: <String, String>{
+          attributes: {
             'xmlns': smXmlns,
           },
         );

packages/moxxmpp/lib/src/xeps/xep_0198/xep_0198.dart

@@ -140,7 +140,7 @@ class StreamManagementManager extends XmppManagerBase {
           nonzaTag: 'a',
           nonzaXmlns: smXmlns,
           callback: _handleAckResponse,
-        )
+        ),
       ];
 
   @override
@@ -148,14 +148,14 @@ class StreamManagementManager extends XmppManagerBase {
         StanzaHandler(
           callback: _onServerStanzaReceived,
           priority: 9999,
-        )
+        ),
       ];
 
   @override
   List<StanzaHandler> getOutgoingPostStanzaHandlers() => [
         StanzaHandler(
           callback: _onClientStanzaSent,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0280.dart

@@ -49,7 +49,7 @@ class CarbonsManager extends XmppManagerBase {
           tagXmlns: carbonsXmlns,
           callback: _onMessageSent,
           priority: -98,
-        )
+        ),
       ];
 
   @override
@@ -124,7 +124,7 @@ class CarbonsManager extends XmppManagerBase {
             XMLNode.xmlns(
               tag: 'enable',
               xmlns: carbonsXmlns,
-            )
+            ),
           ],
         ),
       ),
@@ -154,7 +154,7 @@ class CarbonsManager extends XmppManagerBase {
             XMLNode.xmlns(
               tag: 'disable',
               xmlns: carbonsXmlns,
-            )
+            ),
           ],
         ),
       ),

packages/moxxmpp/lib/src/xeps/xep_0308.dart

@@ -40,7 +40,7 @@ class LastMessageCorrectionManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the message handler
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0333.dart

@@ -100,7 +100,7 @@ class ChatMarkerManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the message handler
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0359.dart

@@ -81,7 +81,7 @@ class StableIdManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the MessageManager
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0363/xep_0363.dart

@@ -161,9 +161,9 @@ class HttpFileUploadManager extends XmppManagerBase {
               attributes: {
                 'filename': filename,
                 'size': filesize.toString(),
-                ...contentType != null ? {'content-type': contentType} : {}
+                if (contentType != null) 'content-type': contentType,
               },
-            )
+            ),
           ],
         ),
       ),

packages/moxxmpp/lib/src/xeps/xep_0385.dart

@@ -82,7 +82,7 @@ class SIMSManager extends XmppManagerBase {
           tagXmlns: referenceXmlns,
           // Before the message handler
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0424.dart

@@ -31,7 +31,7 @@ class MessageRetractionManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the MessageManager
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0447.dart

@@ -133,7 +133,7 @@ class SFSManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the message handler
           priority: -98,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/lib/src/xeps/xep_0461.dart

@@ -96,7 +96,7 @@ class MessageRepliesManager extends XmppManagerBase {
           callback: _onMessage,
           // Before the message handler
           priority: -99,
-        )
+        ),
       ];
 
   @override

packages/moxxmpp/test/sasl/scram_test.dart

@@ -14,9 +14,9 @@ final scramSha1StreamFeatures = XMLNode(
         XMLNode(
           tag: 'mechanism',
           text: 'SCRAM-SHA-1',
-        )
+        ),
       ],
-    )
+    ),
   ],
 );
 final scramSha256StreamFeatures = XMLNode(
@@ -29,9 +29,9 @@ final scramSha256StreamFeatures = XMLNode(
         XMLNode(
           tag: 'mechanism',
           text: 'SCRAM-SHA-256',
-        )
+        ),
       ],
-    )
+    ),
   ],
 );
 

packages/moxxmpp/test/stanzahandler_test.dart

@@ -171,7 +171,7 @@ void main() {
         ),
         tagName: '3',
         priority: 50,
-      )
+      ),
     ]..sort(stanzaHandlerSortComparator);
 
     expect(handlerList[0].tagName, '1');

packages/moxxmpp/test/xeps/xep_0060_test.dart

@@ -158,7 +158,7 @@ void main() {
 </iq>''',
           ignoreId: true,
           adjustId: true,
-        )
+        ),
       ],
     );
 

packages/moxxmpp/test/xeps/xep_0115_test.dart

@@ -69,7 +69,7 @@ class StubbedDiscoManager extends DiscoManager {
                   isRequired: false,
                   varAttr: 'FORM_TYPE',
                   type: 'hidden',
-                )
+                ),
               ],
               reported: [],
               items: [],
@@ -153,14 +153,14 @@ void main() {
         'http://jabber.org/protocol/caps',
         'http://jabber.org/protocol/disco#info',
         'http://jabber.org/protocol/disco#items',
-        'http://jabber.org/protocol/muc'
+        'http://jabber.org/protocol/muc',
       ],
       const [
         Identity(
           category: 'client',
           type: 'pc',
           name: 'Exodus 0.9.1',
-        )
+        ),
       ],
       const [],
       null,
@@ -179,7 +179,7 @@ void main() {
         'http://jabber.org/protocol/caps',
         'http://jabber.org/protocol/disco#info',
         'http://jabber.org/protocol/disco#items',
-        'http://jabber.org/protocol/muc'
+        'http://jabber.org/protocol/muc',
       ],
       const [
         Identity(
@@ -295,14 +295,14 @@ void main() {
         'urn:xmpp:message-correct:0',
         'urn:xmpp:ping',
         'urn:xmpp:receipts',
-        'urn:xmpp:time'
+        'urn:xmpp:time',
       ],
       const [
         Identity(
           category: 'client',
           type: 'phone',
           name: 'Conversations',
-        )
+        ),
       ],
       const [],
       null,

packages/moxxmpp/test/xeps/xep_0198_test.dart

@@ -433,7 +433,8 @@ void main() {
     });
 
     test('Test a failed stream resumption', () async {
-      final fakeSocket = StubTCPSocket([
+      final fakeSocket = StubTCPSocket(
+        [
           StringExpectation(
             "<stream:stream xmlns='jabber:client' version='1.0' xmlns:stream='http://etherx.jabber.org/streams' to='test.server' from='polynomdivision@test.server' xml:lang='en'>",
             '''
@@ -486,8 +487,9 @@ void main() {
           StringExpectation(
             "<enable xmlns='urn:xmpp:sm:3' resume='true' />",
             '<enabled xmlns="urn:xmpp:sm:3" id="id-2" resume="true" />',
-        )
+          ),
-      ]);
+        ],
+      );
 
       final conn = XmppConnection(
         TestingReconnectionPolicy(),

packages/moxxmpp/test/xeps/xep_0334_test.dart

@@ -115,7 +115,8 @@ void main() {
   test('Test sending a message processing hint', () async {
     final manager = MessageManager();
     final holder = TestingManagerHolder(
-      stubSocket: StubTCPSocket([
+      stubSocket: StubTCPSocket(
+        [
           StanzaExpectation(
             '''
 <message to="user@example.org" type="chat">
@@ -124,8 +125,9 @@ void main() {
 </message>
 ''',
             '',
-        )
+          ),
-      ]),
+        ],
+      ),
     );
 
     await holder.register([

packages/moxxmpp/test/xeps/xep_0363_test.dart

@@ -6,7 +6,7 @@ void main() {
     test('invariance', () {
       final headers = {
         'authorization': 'Basic Base64String==',
-        'cookie': 'foo=bar; user=romeo'
+        'cookie': 'foo=bar; user=romeo',
       };
       expect(
         prepareHeaders(headers),
@@ -16,7 +16,7 @@ void main() {
     test('invariance through uppercase', () {
       final headers = {
         'Authorization': 'Basic Base64String==',
-        'Cookie': 'foo=bar; user=romeo'
+        'Cookie': 'foo=bar; user=romeo',
       };
       expect(
         prepareHeaders(headers),
@@ -27,7 +27,7 @@ void main() {
       final headers = {
         'Authorization': 'Basic Base64String==',
         'Cookie': 'foo=bar; user=romeo',
-        'X-Tracking': 'Base64String=='
+        'X-Tracking': 'Base64String==',
       };
       expect(prepareHeaders(headers), {
         'Authorization': 'Basic Base64String==',