1.5.2: popup identity fix + bug sweep

Slash popup was caching rows under conflicting identity signals
(ForEach element.id and .id(idx)); unified on positional identity
so rows rebuild when the matches list changes. Removed the 8-item
prefix cap so typing just "/" shows the full command list. Toast
auto-dismiss now verifies it's still the active toast by id to
avoid skipping queued entries. /rewind clamps to [1, 50]. DiffSheet
uses LazyVStack so huge diffs don't hitch on open.

Author: raya-ac

CHANGELOG.md

@@ -4,6 +4,27 @@ All notable changes to Kiln land here. Format loosely follows
 [Keep a Changelog](https://keepachangelog.com/en/1.1.0/). Dates are
 YYYY-MM-DD, versions follow [SemVer](https://semver.org/).
 
+## [1.5.2] — 2026-04-21
+
+### Fixed
+- **Slash popup stale rendering** — conflicting identity signals
+  (ForEach by `element.id` + row `.id(idx)`) kept old rows cached
+  when the matches list shrank. Unified on positional identity so
+  each row's body rebuilds with the current command.
+- **Slash popup now shows all commands** when the input is just `/` —
+  was capped at 8 entries; LazyVStack handles the full list fine.
+- **Toast auto-dismiss race** — between sleep and wake, a manual
+  tap-to-dismiss could cause the next queued toast to be skipped.
+  The auto-dismiss now verifies it's still the active toast by id.
+- **`/rewind` cap** — parse result clamped to [1, 50] so a typo
+  like `/rewind 999` can't quietly shred a session.
+- **DiffSheet performance** — switched to `LazyVStack` so a 10k-line
+  diff doesn't blow out the view hierarchy on open.
+
+### Added
+- Toast feedback for `/rewind` — confirms how many exchanges were
+  dropped.
+
 ## [1.5.1] — 2026-04-21
 
 ### Fixed

Sources/Services/ToastCenter.swift

@@ -43,7 +43,13 @@ final class ToastCenter: ObservableObject {
         dismissTask = Task { [weak self] in
             try? await Task.sleep(nanoseconds: UInt64(toast.duration * 1_000_000_000))
             guard !Task.isCancelled else { return }
-            await MainActor.run { self?.dismissCurrent() }
+            await MainActor.run {
+                // Only auto-dismiss if we're still the active toast —
+                // otherwise a tap-to-dismiss between sleep and wake
+                // would pop the *next* queued toast prematurely.
+                guard self?.current?.id == toast.id else { return }
+                self?.dismissCurrent()
+            }
         }
     }
 }

Sources/Views/Chat/ChatView.swift

@@ -2351,7 +2351,10 @@ struct DiffSheet: View {
             .padding(12)
             Divider()
             ScrollView {
-                VStack(alignment: .leading, spacing: 0) {
+                // LazyVStack so a 10k-line diff doesn't blow out the
+                // view hierarchy up front — only the visible rows get
+                // materialized.
+                LazyVStack(alignment: .leading, spacing: 0) {
                     ForEach(Array(content.split(separator: "\n", omittingEmptySubsequences: false).enumerated()), id: \.offset) { _, line in
                         Text(String(line).isEmpty ? " " : String(line))
                             .font(.system(size: 11, design: .monospaced))

Sources/Views/Chat/ComposerView.swift

@@ -394,26 +394,23 @@ struct ComposerView: View {
     /// shouldn't pull up every command that happens to have "t" in its
     /// description.
     private var slashMatches: [SlashCommand]? {
-        let trimmed = input.trimmingCharacters(in: .whitespaces)
+        let trimmed = input.trimmingCharacters(in: .whitespacesAndNewlines)
         guard trimmed.hasPrefix("/") else { return nil }
         // Only fire while the input is a single word (no spaces yet).
         if trimmed.contains(" ") { return nil }
         let query = String(trimmed.dropFirst()).lowercased()
         let all = SlashCommands.all()
         if query.isEmpty { return all }
-        // Score each candidate: 0 = exact, 1 = prefix, 2 = contains,
-        // nil = not a match. Stable sort within each bucket preserves
-        // the authored order in `SlashCommands.builtins`.
-        let scored: [(SlashCommand, Int)] = all.compactMap { c in
-            let body = c.label.lowercased().dropFirst() // strip the leading "/"
-            let labelLower = String(body)
-            if labelLower == query { return (c, 0) }
-            if labelLower.hasPrefix(query) { return (c, 1) }
-            if labelLower.contains(query) { return (c, 2) }
-            return nil
+        var exact: [SlashCommand] = []
+        var prefix: [SlashCommand] = []
+        var contains: [SlashCommand] = []
+        for c in all {
+            let body = String(c.label.dropFirst()).lowercased()
+            if body == query { exact.append(c) }
+            else if body.hasPrefix(query) { prefix.append(c) }
+            else if body.contains(query) { contains.append(c) }
         }
-        let sorted = scored.sorted { $0.1 < $1.1 }
-        return sorted.map { $0.0 }
+        return exact + prefix + contains
     }
 
     private func insertSlashCommand(_ cmd: SlashCommand) {
@@ -647,10 +644,13 @@ struct ComposerView: View {
             }
         case "/rewind":
             // /rewind N — drop the last N message pairs from the session.
-            // Default N = 1. Non-destructive on disk until saveSession runs.
-            let n = Int(arg.trimmingCharacters(in: .whitespaces)) ?? 1
-            if let id = store.activeSessionId, n > 0 {
+            // Default N = 1. Capped at 50 so a typo can't nuke a long
+            // session. Non-destructive on disk until saveSession runs.
+            let parsed = Int(arg.trimmingCharacters(in: .whitespaces)) ?? 1
+            let n = max(1, min(50, parsed))
+            if let id = store.activeSessionId {
                 store.rewindSession(id, count: n)
+                ToastCenter.shared.show("Rewound \(n) exchange\(n == 1 ? "" : "s")", kind: .info)
             }
         default:
             // Handle dynamic template aliases like `/t:review`.
@@ -1278,7 +1278,12 @@ struct SlashCommandPopup: View {
             ScrollViewReader { proxy in
                 ScrollView {
                     LazyVStack(spacing: 1) {
-                        ForEach(Array(matches.prefix(8).enumerated()), id: \.element.id) { idx, cmd in
+                        // Identity by positional index so SwiftUI rebuilds
+                        // each row's body when `matches` changes (shrinking
+                        // the list used to leave stale rows cached under
+                        // `.id(idx)` while ForEach tracked element.id —
+                        // two conflicting identities). One signal only.
+                        ForEach(Array(matches.enumerated()), id: \.offset) { idx, cmd in
                             SlashCommandRow(cmd: cmd, selected: idx == selected)
                                 .id(idx)
                                 .onTapGesture { onPick(cmd) }

VERSION

@@ -1 +1 @@
-1.5.1
+1.5.2