fix: rename status() to event_status() to avoid proto field shadowing

SeedCompleted has a proto field named 'status' which shadows the
method. Renamed to event_status() across all UserLevel events and
the base class to prevent runtime errors.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: abhizipstack

backend/visitran/events/base_types.py

@@ -108,7 +108,7 @@ def subtitle(self) -> str:
         """Contextual metadata shown below the title."""
         return ""
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         """One of: running, success, error, warning, info."""
         return "success"
 

backend/visitran/events/eventmgr.py

@@ -131,7 +131,7 @@ def write_line(self, msg: EventMsg) -> None:
                 ts = datetime.utcnow().strftime("%H:%M:%S")
                 title = getattr(msg.data, "title", lambda: msg.info.msg)()
                 subtitle = getattr(msg.data, "subtitle", lambda: "")()
-                event_status = getattr(msg.data, "status", lambda: "info")()
+                event_status = getattr(msg.data, "event_status", lambda: "info")()
                 event_code = getattr(msg.data, "code", lambda: "")()
                 payload = {
                     "level": msg.info.level,

backend/visitran/events/types.py

@@ -846,7 +846,7 @@ def subtitle(self) -> str:
         parts.append(f"{self.source_table} → {self.destination_table}")
         return " · ".join(parts)
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "running"
 
 
@@ -881,7 +881,7 @@ def title(self) -> str:
     def subtitle(self) -> str:
         return (self.error[:150] + "…") if len(self.error) > 150 else self.error
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "error"
 
     def level_tag(self) -> EventLevel:
@@ -945,7 +945,7 @@ def title(self) -> str:
     def subtitle(self) -> str:
         return f"Schema: {self.schema_name}"
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "success" if self.status == "Success" else "error"
 
 
@@ -987,7 +987,7 @@ def message(self) -> str:
     def title(self) -> str:
         return f'Job "{self.job_name}" deleted'
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "warning"
 
 
@@ -1007,7 +1007,7 @@ def subtitle(self) -> str:
         scope_desc = "All models" if self.scope == "job" else f"Model: {self.scope}"
         return f"{scope_desc} · Manual trigger"
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "running"
 
 
@@ -1036,7 +1036,7 @@ def message(self) -> str:
     def title(self) -> str:
         return f'Deleted: {self.file_names}'
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "warning"
 
 
@@ -1099,7 +1099,7 @@ def message(self) -> str:
     def title(self) -> str:
         return f'Connection "{self.connection_name}" deleted'
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "warning"
 
 
@@ -1121,7 +1121,7 @@ def title(self) -> str:
     def subtitle(self) -> str:
         return (self.reason[:150] + "…") if len(self.reason) > 150 else self.reason
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "error"
 
 
@@ -1148,5 +1148,5 @@ def message(self) -> str:
     def title(self) -> str:
         return f'Environment "{self.environment_name}" deleted'
 
-    def status(self) -> str:
+    def event_status(self) -> str:
         return "warning"