chore: code cleanup

Author: demolaf

packages/googleapis_firestore/lib/src/field_value.dart

@@ -426,11 +426,6 @@ void _validateUserInput(
 
     case Map<Object?, Object?>():
       for (final entry in value.entries) {
-        // FieldPath keys (from UpdateMap) are already validated field paths
-        // and must be used as-is to preserve multi-segment paths.
-        // String keys are literal Firestore map keys and must NOT be routed
-        // through _StringFieldMask, which rejects valid key characters such
-        // as '/'. Construct a single-segment FieldPath directly instead.
         final keyPath = switch (entry.key) {
           final FieldPath fp => fp,
           final String s => FieldPath([s]),

packages/googleapis_firestore/test/firestore_test.dart

@@ -168,20 +168,11 @@ void main() {
         expect(batch, isA<WriteBatch>());
       });
 
-      // Regression test for https://github.com/invertase/dart_firebase_admin/issues/83
-      //
-      // Firestore allows '/' characters inside map *keys* (e.g. document
-      // reference paths stored as map keys). The SDK was incorrectly routing
-      // map keys through field-path validation, which rejects '/', causing an
-      // ArgumentError before any network call was made.
-      group('map keys with "/" characters (issue #83)', () {
-        late Firestore firestore;
-
-        setUp(() {
-          firestore = Firestore(settings: const Settings(projectId: 'test'));
-        });
-
-        test('set() should not throw for a map key containing "/"', () {
+      group('set() with map keys containing "/"', () {
+        test('accepts a top-level map value with "/" in key', () {
+          final firestore = Firestore(
+            settings: const Settings(projectId: 'test'),
+          );
           final batch = firestore.batch();
           final docRef = firestore.doc('activities/new-activity');
 
@@ -194,7 +185,10 @@ void main() {
           );
         });
 
-        test('set() should not throw for nested maps with "/" in keys', () {
+        test('accepts nested maps with "/" in keys', () {
+          final firestore = Firestore(
+            settings: const Settings(projectId: 'test'),
+          );
           final batch = firestore.batch();
           final docRef = firestore.doc('col/doc');