refactor: inline reason alias in run_loop stop-event emit

The `reason = state.status.reason` local was read exactly once on the
next line as the `reason=` kwarg to `RunStoppedData(...)`.  Collapsing
to `reason=state.status.reason` matches the inline-alias pattern from
ce487d3 (`text`) / 52e0272 (`msg`) / 497c028 (`agent`) / fc5e1cb
(`total_in`).

The property access is safe at this call site: the immediately-preceding
`if state.status == RunStatus.RUNNING: state.status = RunStatus.COMPLETED`
ensures `state.status` is always a terminal status (STOPPED / COMPLETED
/ FAILED), so `RunStatus.reason` cannot raise.

Co-authored-by: Ralphify <noreply@ralphify.co>

Author: 2 people

src/ralphify/engine.py

@@ -369,11 +369,10 @@ def run_loop(
     if state.status == RunStatus.RUNNING:
         state.status = RunStatus.COMPLETED
 
-    reason = state.status.reason
     emit(
         EventType.RUN_STOPPED,
         RunStoppedData(
-            reason=reason,
+            reason=state.status.reason,
             total=state.total,
             completed=state.completed,
             failed=state.failed,