fix: skip memory save when review is locked, document lock release

- Guard "Save reviewer memory" with lock-check to prevent skipped runs
  from overwriting memory with stale data
- Add comment explaining lock release failures are safe (TTL fallback)

Author: derekmisler

review-pr/action.yml

@@ -650,7 +650,7 @@ runs:
 
     - name: Release review lock
       if: always() && steps.lock-check.outputs.skip != 'true'
-      continue-on-error: true
+      continue-on-error: true  # Release failures are safe — stale locks expire via TTL (2500s)
       shell: bash
       env:
         GH_TOKEN: ${{ steps.resolve-token.outputs.token }}
@@ -665,7 +665,7 @@ runs:
         echo "🔓 Released review lock for PR #$PR_NUMBER"
 
     - name: Save reviewer memory
-      if: always()
+      if: always() && steps.lock-check.outputs.skip != 'true'
       continue-on-error: true  # Don't fail if memory file doesn't exist (first run)
       uses: actions/cache/save@1bd1e32a3bdc45362d1e726936510720a7c30a57 # v4.2.0
       with: