Skip to content

feat(cloud-agent): add Gastown platform filter to sessions UI#3426

Merged
jrf0110 merged 2 commits into
gastown-stagingfrom
convoy/tag-gastown-sessions-correctly-in-cloud-/2ad5ef09/head
May 22, 2026
Merged

feat(cloud-agent): add Gastown platform filter to sessions UI#3426
jrf0110 merged 2 commits into
gastown-stagingfrom
convoy/tag-gastown-sessions-correctly-in-cloud-/2ad5ef09/head

Conversation

@jrf0110
Copy link
Copy Markdown
Contributor

@jrf0110 jrf0110 commented May 22, 2026

Summary

Add 'gastown' as a filterable platform option in the cloud sessions sidebar (web and mobile), so Gastown sessions are correctly tagged and filterable in the cloud sessions UI. Also removes a stale comment in CloudSidebarLayout's platform switch and adds a clarifying comment on the default branch that now handles gastown pass-through.

Verification

Manually verified that the platform filter list and label mapping are consistent across web ChatSidebar and mobile platform-filter-modal.

Visual Changes

N/A — filter chip addition matches existing pattern.

Reviewer Notes

The default case in CloudSidebarLayout.tsx already passes unknown platform strings through unchanged, which correctly handles 'gastown' without special-casing. The KILO_PLATFORM: 'gastown' value is set by the gastown process manager (confirmed by existing tests). Minor: line 59 of CloudSidebarLayout.tsx has a stray blank line from the removed comment.

@kilo-code-bot
Copy link
Copy Markdown
Contributor

kilo-code-bot Bot commented May 22, 2026

Code Review Summary

Status: No Issues Found | Recommendation: Merge

Executive Summary

This PR cleanly adds 'gastown' as a filterable platform option to both web and mobile session sidebars, with no logic bugs, security concerns, or behavioral regressions.

Files Reviewed (4 files)
  • apps/mobile/src/components/agents/platform-filter-modal.tsx
  • apps/web/src/components/cloud-agent-next/ChatSidebar.tsx
  • apps/web/src/components/cloud-agent-next/CloudSidebarLayout.tsx
  • services/gastown/container/src/process-manager.test.ts

Reviewed by claude-sonnet-4.6 · 304,386 tokens

Review guidance: REVIEW.md from base branch gastown-staging

@jrf0110 jrf0110 merged commit 8d648c5 into gastown-staging May 22, 2026
2 checks passed
@jrf0110 jrf0110 deleted the convoy/tag-gastown-sessions-correctly-in-cloud-/2ad5ef09/head branch May 22, 2026 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant