PrimeIntellect/SWE-rebench-V2 huggingface dataset:
Repository: meltano/sdk (python)

Issue:
bug: increment state is breaking when new_value is null
### Singer SDK Version

0.27.0

### Is this a regression?

- [ ] Yes

### Python Version

3.8

### Bug scope

Taps (catalog, state, etc.)

### Operating System

Linux-debian??

### Description


context - state is not present from backend and tap is trying full-refresh with configuration which has incremental key.

incremental state is breaking when new value is null.
so we need to check it's value if it is None or not.

https://github.com/meltano/sdk/blob/973f245d97e34efe6c2cf279998d88daaf4d43cf/singer_sdk/helpers/_state.py#LL217C7-L217C7

### Code

_No response_

Pull request:
fix: Null replication values are now handled when incrementing bookmarks
Related:

* Closes #1714

<!-- readthedocs-preview meltano-sdk start -->
----
📚 Documentation preview 📚: https://meltano-sdk--2438.org.readthedocs.build/en/2438/

<!-- readthedocs-preview meltano-sdk end -->

Patch:
diff --git a/singer_sdk/helpers/_state.py b/singer_sdk/helpers/_state.py
index f3a83110..ed3d345e 100644
--- a/singer_sdk/helpers/_state.py
+++ b/singer_sdk/helpers/_state.py
@@ -218,6 +218,11 @@ def increment_state(
         progress_dict = stream_or_partition_state[PROGRESS_MARKERS]
     old_rk_value = to_json_compatible(progress_dict.get("replication_key_value"))
     new_rk_value = to_json_compatible(latest_record[replication_key])
+
+    if new_rk_value is None:
+        logger.warning("New replication value is null")
+        return
+
     if old_rk_value is None or not check_sorted or new_rk_value >= old_rk_value:
         progress_dict["replication_key"] = replication_key
         progress_dict["replication_key_value"] = new_rk_value


Test patch:
diff --git a/tests/core/test_state_handling.py b/tests/core/test_state_handling.py
index 3f4fff14..f58a0128 100644
--- a/tests/core/test_state_handling.py
+++ b/tests/core/test_state_handling.py
@@ -2,6 +2,8 @@
 
 from __future__ import annotations
 
+import logging
+
 import pytest
 
 from singer_sdk.helpers import _state
@@ -127,3 +129,29 @@ def test_irresumable_state():
             "replication_key_value": "2021-05-17T20:41:16Z",
         },
     }
+
+
+def test_null_replication_value(caplog):
+    stream_state = {
+        "replication_key": "updated_at",
+        "replication_key_value": "2021-05-17T20:41:16Z",
+    }
+    latest_record = {"updated_at": None}
+    replication_key = "updated_at"
+    is_sorted = True
+    check_sorted = False
+
+    with caplog.at_level(logging.WARNING):
+        _state.increment_state(
+            stream_state,
+            latest_record=latest_record,
+            replication_key=replication_key,
+            is_sorted=is_sorted,
+            check_sorted=check_sorted,
+        )
+
+    assert (
+        stream_state["replication_key_value"] == "2021-05-17T20:41:16Z"
+    ), "State should not be updated."
+    assert caplog.records[0].levelname == "WARNING"
+    assert "is null" in caplog.records[0].message
Repository: swiftlang/swift-syntax (swift)

Issue:
New parser fails to parse `@convention` attribute with `cType` argument inside an array type
### Issue Kind

Parse of Valid Source Failed

### Source Code

```swift
_ = [@convention(c, cType: "int (*)(int)") (Int32) -> Int32]()
```


### Description

The issue is that the implementation of `skipTypeAttribute` doesn’t skip over the `cType` argument here. We should probably be able to just `skipSingle` to skip over all the arguments.

https://github.com/apple/swift-syntax/blob/89dd2bde9404950dede113b7898c97f6e2bc2fc4/Sources/SwiftParser/Lookahead.swift#L165-L173

Pull request:
Fixes #2474
rdar://122358055

Patch:
diff --git a/Sources/SwiftParser/Lookahead.swift b/Sources/SwiftParser/Lookahead.swift
index 682e5e46f68..c8d26ce6a14 100644
--- a/Sources/SwiftParser/Lookahead.swift
+++ b/Sources/SwiftParser/Lookahead.swift
@@ -162,14 +162,11 @@ extension Parser.Lookahead {
     if let (attr, handle) = self.at(anyIn: TypeAttribute.self) {
       // Ok, it is a valid attribute, eat it, and then process it.
       self.eat(handle)
-      if case .convention = attr {
-        guard
-          self.consume(if: .leftParen) != nil,
-          self.consume(if: .identifier) != nil,
-          self.consume(if: .rightParen) != nil
-        else {
-          return
-        }
+      switch attr {
+      case .convention:
+        self.skipSingle()
+      default:
+        break
       }
       return
     }


Test patch:
diff --git a/Tests/SwiftParserTest/AttributeTests.swift b/Tests/SwiftParserTest/AttributeTests.swift
index 906eae1abab..3ce6b32508e 100644
--- a/Tests/SwiftParserTest/AttributeTests.swift
+++ b/Tests/SwiftParserTest/AttributeTests.swift
@@ -886,4 +886,25 @@ final class AttributeTests: ParserTestCase {
       """
     )
   }
+
+  func testConventionAttributeInArrayType() {
+    assertParse(
+      """
+      _ = [@convention(c, cType: "int (*)(int)") (Int32) -> Int32]()
+      """,
+      substructure: ConventionAttributeArgumentsSyntax(
+        conventionLabel: .identifier("c"),
+        comma: .commaToken(),
+        cTypeLabel: .keyword(.cType),
+        colon: .colonToken(),
+        cTypeString: StringLiteralExprSyntax(
+          openingQuote: .stringQuoteToken(),
+          segments: StringLiteralSegmentListSyntax([
+            StringLiteralSegmentListSyntax.Element(StringSegmentSyntax(content: .stringSegment("int (*)(int)")))
+          ]),
+          closingQuote: .stringQuoteToken()
+        )
+      )
+    )
+  }
 }