fix: address review feedback - remove unused import and fix namespace

- Remove unused SessionPhaseBadge import from sessions-section.tsx
- Replace SessionPhaseBadge with AgentStatusIndicator using correct pattern
- Fix Makefile kind-port-forward to use $(NAMESPACE) variable instead of hardcoded "ambient-code"
- Simplify redundant conditional logic for agent status derivation

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

Author: 2 people

Makefile

@@ -735,8 +735,8 @@ kind-port-forward: check-kubectl check-local-context ## Port-forward kind servic
 	@echo "$(COLOR_YELLOW)Press Ctrl+C to stop$(COLOR_RESET)"
 	@echo ""
 	@trap 'kill 0; echo ""; echo "$(COLOR_GREEN)✓$(COLOR_RESET) Port forwarding stopped"; exit 0' INT; \
-	kubectl port-forward -n ambient-code svc/frontend-service $(KIND_FWD_FRONTEND_PORT):3000 >/dev/null 2>&1 & \
-	kubectl port-forward -n ambient-code svc/backend-service $(KIND_FWD_BACKEND_PORT):8080 >/dev/null 2>&1 & \
+	kubectl port-forward -n $(NAMESPACE) svc/frontend-service $(KIND_FWD_FRONTEND_PORT):3000 >/dev/null 2>&1 & \
+	kubectl port-forward -n $(NAMESPACE) svc/backend-service $(KIND_FWD_BACKEND_PORT):8080 >/dev/null 2>&1 & \
 	wait
 
 dev-bootstrap: check-kubectl check-local-context ## Bootstrap developer workspace with API key and integrations

components/frontend/src/components/workspace-sections/sessions-section.tsx

@@ -23,7 +23,6 @@ import {
 import { getPageNumbers } from '@/lib/pagination';
 import { EmptyState } from '@/components/empty-state';
 import { SessionStatusDot } from '@/components/session-status-dot';
-import { SessionPhaseBadge } from '@/components/status-badge';
 import { AgentStatusIndicator } from '@/components/agent-status-indicator';
 import { deriveAgentStatusFromPhase } from '@/hooks/use-agent-status';
 import { CreateSessionDialog } from '@/components/create-session-dialog';
@@ -305,7 +304,10 @@ export function SessionsSection({ projectName }: SessionsSectionProps) {
                                   <p className="text-sm font-semibold truncate mr-2">
                                     {session.spec.displayName || session.metadata.name}
                                   </p>
-                                  <SessionPhaseBadge phase={phase} stoppedReason={session.status?.stoppedReason} />
+                                  <AgentStatusIndicator
+                                    status={session.status?.agentStatus ?? deriveAgentStatusFromPhase(phase)}
+                                    compact
+                                  />
                                 </div>
                                 {session.spec.displayName && (
                                   <p className="text-xs text-muted-foreground">{session.metadata.name}</p>
@@ -333,11 +335,9 @@ export function SessionsSection({ projectName }: SessionsSectionProps) {
                           </HoverCard>
                         </TableCell>
                         <TableCell>
-                          <AgentStatusIndicator status={
-                            (phase === 'Completed' || phase === 'Failed' || phase === 'Stopped')
-                              ? deriveAgentStatusFromPhase(phase)
-                              : (session.status?.agentStatus ?? deriveAgentStatusFromPhase(phase))
-                          } />
+                          <AgentStatusIndicator
+                            status={session.status?.agentStatus ?? deriveAgentStatusFromPhase(phase)}
+                          />
                         </TableCell>
                         <TableCell className="hidden md:table-cell">
                           <div className="text-sm truncate max-w-[160px]">