refactor: standardize attempt counting

Remove Logger's +1 transformation and have all code pass current attempt
(1, 2, 3...) consistently throughout the codebase.

Previously, Logger added +1 internally, creating confusion about whether
code should pass 'current attempt' or 'checkpointed attempt'. This
standardization removes the hidden transformation.

Changes:
- Remove +1 from Logger.from_log_info() (logger.py line 75)
- Update StepContext to pass current attempt by adding +1 (step.py lines 205-208)
- Update logger tests to expect current attempt values

This ensures all components use the same convention: current attempt =
checkpointed attempts + 1, with no hidden transformations.

Author: yaythomas

src/aws_durable_execution_sdk_python/logger.py

@@ -72,7 +72,7 @@ def from_log_info(cls, logger: LoggerInterface, info: LogInfo) -> Logger:
             # Use 'operation_name' instead of 'name' as key because the stdlib LogRecord internally reserved 'name' parameter
             extra["operationName"] = info.name
         if info.attempt is not None:
-            extra["attempt"] = info.attempt + 1
+            extra["attempt"] = info.attempt
         if info.operation_id:
             extra["operationId"] = info.operation_id
         return cls(

src/aws_durable_execution_sdk_python/operation/step.py

@@ -202,9 +202,10 @@ def execute(self, checkpointed_result: CheckpointedResult) -> T:
             ExecutionError: For fatal errors that should not be retried
             May raise other exceptions that will be handled by retry_handler
         """
-        attempt: int = 0
+        # Get current attempt - checkpointed attempts + 1
+        attempt: int = 1
         if checkpointed_result.operation and checkpointed_result.operation.step_details:
-            attempt = checkpointed_result.operation.step_details.attempt
+            attempt = checkpointed_result.operation.step_details.attempt + 1
 
         step_context: StepContext = StepContext(
             logger=self.context_logger.with_log_info(

tests/logger_test.py

@@ -177,7 +177,7 @@ def test_logger_from_log_info_full():
         "parentId": "parent123",
         "operationId": "op123",
         "operationName": "test_name",
-        "attempt": 6,
+        "attempt": 5,
     }
     assert logger._default_extra == expected_extra  # noqa: SLF001
     assert logger._logger is mock_logger  # noqa: SLF001
@@ -202,7 +202,7 @@ def test_logger_from_log_info_partial_fields():
     # Test with attempt but no parent_id or name
     log_info = LogInfo(EXECUTION_STATE, None, None, None, 5)
     logger = Logger.from_log_info(mock_logger, log_info)
-    expected_extra = {"executionArn": "arn:aws:test", "attempt": 6}
+    expected_extra = {"executionArn": "arn:aws:test", "attempt": 5}
     assert logger._default_extra == expected_extra  # noqa: SLF001