refactor: Clean up DpathExtractor extract_records logic

Apply cleaner logic using 'yield from' consistently:
- When extracted is a list without record_expander, use 'yield from extracted'
- Check 'if not self.record_expander' instead of nested if/else
- Remove unnecessary 'yield from []' for empty case

All 24 tests passing. Suggested by @DanyloGL.

Co-Authored-By: unknown <>

Author: devin-ai-integration[bot]

airbyte_cdk/sources/declarative/extractors/dpath_extractor.py

@@ -101,15 +101,13 @@ def extract_records(self, response: requests.Response) -> Iterable[MutableMappin
                 else:
                     extracted = dpath.get(body, path, default=[])  # type: ignore # extracted will be a MutableMapping, given input data structure
             if isinstance(extracted, list):
-                for record in extracted:
-                    if self.record_expander:
+                if not self.record_expander:
+                    yield from extracted
+                else:
+                    for record in extracted:
                         yield from self.record_expander.expand_record(record)
-                    else:
-                        yield record
             elif extracted:
                 if self.record_expander:
                     yield from self.record_expander.expand_record(extracted)
                 else:
                     yield extracted
-            else:
-                yield from []