feat(cloud-agent): add Gastown badge#3423
Conversation
…rm (#3372) Bug 1: @kilocode/cli@7.2.14 doesn't read KILO_AUTH_CONTENT, causing all kilo serve session-ingest to silently no-op. Bumped to 7.3.1 which has the feature. Verified KILO_AUTH_CONTENT present in binary strings. Bug 2: buildPrewarmEnv didn't set KILO_AUTH_CONTENT, KILO_PLATFORM, or KILO_ORG_ID, so mayor sessions (which go through prewarm) were invisible. Extracted buildKiloAuthEnv helper from buildAgentEnv and used it in both buildAgentEnv and buildPrewarmEnv. Refs #3307 Co-authored-by: John Fawcett <john@kilcoode.ai>
* perf(gastown): shorten mayor cold start path * fix(gastown): address mayor latency review feedback * fix(gastown): remove stale mayor setup parameter * fix(gastown): remove stale mayor setup comment
| </span> | ||
| ); | ||
| } | ||
| } else if (platform === 'slack') { |
There was a problem hiding this comment.
WARNING: Indentation is broken — } else if (platform === 'slack') { is at column 0 instead of matching the surrounding else if branches (8 spaces). This same pattern repeats in apps/web/src/components/cloud-agent/SessionsList.tsx line 74. The code is syntactically correct (React renders fine) but is visually inconsistent and will be a diff noise issue in future PRs.
| </span> | ||
| ); | ||
| } | ||
| } else if (platform === 'slack') { |
There was a problem hiding this comment.
WARNING: Same indentation issue as in cloud-agent-next/SessionsList.tsx — } else if (platform === 'slack') { starts at column 0 instead of being indented to match the surrounding else if blocks.
Code Review SummaryStatus: 2 Issues Found | Recommendation: Address before merge Executive SummaryBroken indentation on both Overview
Issue Details (click to expand)WARNING
Other Observations (not in diff)Bootstrapping gap during cold start (container-dispatch.ts): In Stale
Files Reviewed (18 files)
Fix these issues in Kilo Cloud Reviewed by claude-4.6-sonnet-20260217 · 2,739,640 tokens Review guidance: REVIEW.md from base branch |
|
Superseded by #3424, which targets the correct convoy head branch instead of |
Adds a Gastown badge to SessionsList components for sessions with createdOnPlatform 'gastown'.