refactor: Move Terraform comment to be adjacent to its code

Move the "# If the instance is managed by Terraform, warn user" comment
from line 963 to line 981, placing it directly above the terraform_managed
assignment it describes. The comment was previously orphaned from its
relevant code by 20 lines of unrelated confirmation prompts.

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>

Author: claude

progress.md

@@ -761,3 +761,22 @@ This created a misleading function signature - if a function's return value is n
 
 ---
 
+## 2026-01-18: Move misplaced Terraform comment in `terminate()` function
+
+**File:** `remote/instance.py`
+
+**Issue:** The comment "# If the instance is managed by Terraform, warn user" on line 963 was separated from the code it described by 20 lines. The actual Terraform check (`terraform_managed = any(...)`) was on line 983, with the confirmation prompts and user input validation in between.
+
+This is a code smell because:
+1. Orphaned comments reduce readability
+2. The comment implied the next line would be the Terraform check, but it wasn't
+3. Readers had to mentally reconnect the comment to its relevant code
+
+**Changes:**
+- Removed the comment from line 963 (after the tag fetching try-except block)
+- Added the comment directly above line 981 where `terraform_managed` is assigned
+
+This places the comment immediately before the code it documents, following the principle that comments should be adjacent to the code they describe.
+
+---
+

remote/instance.py

@@ -960,8 +960,6 @@ def terminate(instance_name: str | None = typer.Argument(None, help="Instance na
         typer.secho(f"Error accessing instance information: {e}", fg=typer.colors.RED)
         # Continue with empty tags
 
-    # If the instance is managed by Terraform, warn user
-
     # Confirmation step
     typer.secho(
         f"WARNING: You are about to terminate instance {instance_name}. "
@@ -980,6 +978,7 @@ def terminate(instance_name: str | None = typer.Argument(None, help="Instance na
 
         return
 
+    # If the instance is managed by Terraform, warn user
     terraform_managed = any("terraform" in tag["Value"].lower() for tag in tags)
 
     if terraform_managed: