From 2e753bb0396b0701373b107aeb3586b552aac361 Mon Sep 17 00:00:00 2001 From: kirillk Date: Fri, 22 May 2026 14:48:12 -0400 Subject: [PATCH] fix(jetbrains): address session review feedback --- packages/kilo-jetbrains/AGENTS.md | 4 ++-- .../kilocode/client/session/controller/SessionController.kt | 4 ++-- 2 files changed, 4 insertions(+), 4 deletions(-) diff --git a/packages/kilo-jetbrains/AGENTS.md b/packages/kilo-jetbrains/AGENTS.md index 30548b1221e..8afdd0ecac3 100644 --- a/packages/kilo-jetbrains/AGENTS.md +++ b/packages/kilo-jetbrains/AGENTS.md @@ -366,11 +366,11 @@ For common spacing lookups, prefer `JBUI.CurrentTheme` area-specific insets (e.g | Simple `BorderLayout` panels | `JBUI.Panels.simplePanel(...)`, `BorderLayoutPanel` | | Simple vertical custom Swing groups | `VerticalLayout` | | Fluent platform panels | `JBPanel.withBorder(...)`, `.andTransparent()`, `.andOpaque()`, `.withBackground(...)` | -| Single-component alignment wrapper | `ai.kilocode.client.ui.Align` — see section below | +| Single-component alignment wrapper | `ai.kilocode.client.ui.layout.Align` — see section below | ### Align — Single-Component Alignment Wrapper -Use `Align` (`ai.kilocode.client.ui.Align`) when a single Swing component must be positioned inside available space without adding visual chrome. It is a transparent, no-border, no-color `JPanel(null)` that lays out its one child according to independent horizontal (`HAlign`) and vertical (`VAlign`) modes. `CenterShrinkPanel` has been removed; use `child.alignCenter()` as a direct replacement. +Use `Align` (`ai.kilocode.client.ui.layout.Align`) when a single Swing component must be positioned inside available space without adding visual chrome. It is a transparent, no-border, no-color `JPanel(null)` that lays out its one child according to independent horizontal (`HAlign`) and vertical (`VAlign`) modes. `CenterShrinkPanel` has been removed; use `child.align(HAlign.CENTER, VAlign.CENTER)` as a direct replacement. **Alignment modes:** diff --git a/packages/kilo-jetbrains/frontend/src/main/kotlin/ai/kilocode/client/session/controller/SessionController.kt b/packages/kilo-jetbrains/frontend/src/main/kotlin/ai/kilocode/client/session/controller/SessionController.kt index 32067597492..a72f6fe55a3 100644 --- a/packages/kilo-jetbrains/frontend/src/main/kotlin/ai/kilocode/client/session/controller/SessionController.kt +++ b/packages/kilo-jetbrains/frontend/src/main/kotlin/ai/kilocode/client/session/controller/SessionController.kt @@ -493,10 +493,10 @@ class SessionController( } } recoverPending(id) - for (child in discovered) trackChild(child) runEdt { if (disposed) return@runEdt if (sid != id) return@runEdt + for (child in discovered) trackChild(child) showSession() loaded(!model.isEmpty()) } @@ -540,9 +540,9 @@ class SessionController( } } recoverPending(session.id) - for (child in discovered) trackChild(child) runEdt { if (disposed) return@runEdt + for (child in discovered) trackChild(child) subscribeEvents() showSession() loaded(!model.isEmpty())