fix: address code review findings — cache resolver, fix misleading descriptions

Critical:
- C1: Cache AgentInterfaceResolver.ScanInterfaces() to avoid full assembly scan on every SendToAgent call
- C2: Fix misleading CleanLogsAsync description (it reads logs, doesn't clean)

Important:
- I1: Fix DeployAsync description to not claim rebuild capability (TODO stub)
- I3: Remove deploy-verify job from OnActivateAsync (only schedule from DeployAsync)
- I4: Add Aspire to error message's available agents list

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

Author: LeftTwixWand

src/Agents/Infrastructure/AspireAgent.cs

@@ -28,11 +28,7 @@ await ScheduleRecurringJob("log-monitor", TimeSpan.FromMinutes(30),
                 "Check system health and report any resource errors or warnings.", ct);
         }
 
-        if (!ScheduledJobs.ContainsKey("deploy-verify"))
-        {
-            await ScheduleJob("deploy-verify", TimeSpan.FromSeconds(60),
-                "Verify deployment health: check all resources are running.", ct);
-        }
+        // deploy-verify job is scheduled from DeployAsync, not on every activation
     }
 
     protected override async Task OnScheduledJobDueAsync(ScheduledJobItem job, CancellationToken ct)

src/Agents/Infrastructure/IAspire.cs

@@ -42,9 +42,9 @@ distributed system through the Aspire dashboard.
     [Description("Get recent structured logs for a resource. Shows errors and warnings first.")]
     Task<string> GetLogsAsync(string resourceName, CancellationToken ct = default);
 
-    [Description("Check system health and clean up monitoring state. Reports any unhealthy resources.")]
+    [Description("Read recent logs for a resource and report monitoring state. Same as GetLogs — use for health checks.")]
     Task<string> CleanLogsAsync(string resourceName, CancellationToken ct = default);
 
-    [Description("Deploy code changes. Stops assistant, rebuilds from source via MCP /deploy endpoint, then starts with fresh binary. Use after writing code changes — slower than RestartResource but picks up new code.")]
+    [Description("Restart the assistant resource. NOTE: currently does not rebuild from source — code changes require manual build first. Full deploy via Aspire SDK is planned.")]
     Task<string> DeployAsync(CancellationToken ct = default);
 }

src/Agents/Orchestration/ThreadAgent.cs

@@ -72,7 +72,7 @@ private async Task<string> SendToAgentAsync(string agentName, string request, Ca
         var interfaceType = AgentInterfaceResolver.ResolveByDisplayName(agentName)
                          ?? AgentInterfaceResolver.Resolve(agentName);
         if (interfaceType is null)
-            return $"Unknown agent: {agentName}. Available: Shell, DotNet, FileSystem, Git, Roslyn, GitHub.";
+            return $"Unknown agent: {agentName}. Available: Shell, DotNet, FileSystem, Git, Roslyn, GitHub, Aspire.";
 
         var threadId = this.GetPrimaryKeyString();
         var agent = (IAgent)GrainFactory.GetGrain(interfaceType, $"{threadId}/{interfaceType.Name}");

src/Core/Extensions/AgentInterfaceResolver.cs

@@ -47,8 +47,10 @@ public static class AgentInterfaceResolver
         });
     }
 
+    private static IReadOnlyList<Type>? _cachedInterfaces;
+
     private static IReadOnlyList<Type> ScanInterfaces() =>
-        AppDomain.CurrentDomain.GetAssemblies()
+        _cachedInterfaces ??= AppDomain.CurrentDomain.GetAssemblies()
             .SelectMany(a => { try { return a.GetTypes(); } catch { return []; } })
             .Where(t => t.IsInterface
                         && t != typeof(IAgent)