Merge pull request #232 from ipnet-mesh/fix/profile-adopted-node-last-seen

fix(web): show node last_seen instead of adopted_at on profile

src/meshcore_hub/api/routes/user_profiles.py

@@ -46,6 +46,7 @@ def _build_adopted_nodes(profile: UserProfile) -> list[AdoptedNodeRead]:
                 name=assoc.node.name,
                 adv_type=assoc.node.adv_type,
                 adopted_at=assoc.adopted_at,
+                last_seen=assoc.node.last_seen,
             )
         )
     return adopted_nodes

src/meshcore_hub/common/schemas/user_profiles.py

@@ -75,6 +75,9 @@ class AdoptedNodeRead(BaseModel):
     name: Optional[str] = Field(default=None, description="Node display name")
     adv_type: Optional[str] = Field(default=None, description="Advertisement type")
     adopted_at: datetime = Field(..., description="When the node was adopted")
+    last_seen: Optional[datetime] = Field(
+        default=None, description="Timestamp of the node's most recent activity"
+    )
 
     class Config:
         from_attributes = True

src/meshcore_hub/web/static/js/spa/pages/profile.js

@@ -7,15 +7,15 @@ import {
 
 function renderAdoptedNode(node) {
     const displayName = node.name || node.public_key.slice(0, 12) + '...';
-    const relTime = formatRelativeTime(node.adopted_at);
-    const fullTime = formatDateTime(node.adopted_at);
+    const relTime = node.last_seen ? formatRelativeTime(node.last_seen) : '-';
+    const fullTime = node.last_seen ? formatDateTime(node.last_seen) : '-';
 
     return html`<a href="/nodes/${node.public_key}" class="flex items-center justify-between gap-3 p-3 bg-base-200 rounded-lg hover:bg-base-300 transition-colors">
         <div class="flex-1 min-w-0">
             <div class="font-medium text-sm truncate">${displayName}</div>
             <div class="font-mono text-xs opacity-60 truncate">${node.public_key}</div>
         </div>
-        <time class="text-xs opacity-60 whitespace-nowrap shrink-0" datetime=${node.adopted_at} title=${fullTime} data-relative-time>${relTime}</time>
+        <time class="text-xs opacity-60 whitespace-nowrap shrink-0" datetime=${node.last_seen || nothing} title=${fullTime} data-relative-time>${relTime}</time>
     </a>`;
 }
 

tests/test_api/test_user_profiles.py

@@ -206,6 +206,7 @@ class TestGetProfile:
         assert len(data["nodes"]) == 1
         assert data["nodes"][0]["public_key"] == "abc123def456abc123def456abc123de"
         assert "adopted_at" in data["nodes"][0]
+        assert data["nodes"][0]["last_seen"] is not None
 
     def test_get_profile_returns_roles(self, client_no_auth, sample_user_profile):
         """Test that profile includes roles."""