lib/l10n/app_nl.arb (1)

288-299: NL copy nits: improve imperatives.

Small, optional tweaks for natural Dutch:

-  "askAi": "AI vragen",
+  "askAi": "Vraag AI",
-  "askAiConfirmExecute": "Bevestigen voor uitvoeren",
+  "askAiConfirmExecute": "Bevestigen vóór uitvoeren",

lib/l10n/app_pt.arb (1)

287-304: gen-l10n + placeholder metadata for askAiConfigMissing.

• Run flutter gen-l10n after ARB edits.
• Add metadata for {fields} to improve generated API clarity.

+  "@askAiConfigMissing": {
+    "description": "Exibido quando faltam configurações obrigatórias de IA.",
+    "placeholders": {
+      "fields": { "type": "String", "example": "URL base / Chave de API / Modelo" }
+    }
+  },

Based on coding guidelines.

lib/l10n/app_uk.arb (2)

287-304: gen-l10n + placeholder metadata for askAiConfigMissing.

• Run flutter gen-l10n.
• Add metadata for {fields}.

+  "@askAiConfigMissing": {
+    "description": "Показується, коли відсутні необхідні налаштування ШІ.",
+    "placeholders": {
+      "fields": { "type": "String", "example": "Базова URL-адреса / Ключ API / Модель" }
+    }
+  },

Based on coding guidelines.

---

290-292: UA copy nit: base URL phrasing.

More natural in UA:

-  "askAiBaseUrl": "Базова URL",
+  "askAiBaseUrl": "Базова URL-адреса",

lib/l10n/app_ja.arb (2)

287-304: gen-l10n + placeholder metadata for askAiConfigMissing.

• Run flutter gen-l10n.
• Add metadata for {fields}.

+  "@askAiConfigMissing": {
+    "description": "必須のAI設定が未構成の場合に表示されます。",
+    "placeholders": {
+      "fields": { "type": "String", "example": "ベースURL / APIキー / モデル" }
+    }
+  },

Based on coding guidelines.

---

291-291: JA typography nit.

Japanese commonly omits the space before “URL”.

-  "askAiBaseUrl": "ベース URL",
+  "askAiBaseUrl": "ベースURL",

lib/l10n/app_zh.arb (2)

287-304: gen-l10n + placeholder metadata for askAiConfigMissing.

• Run flutter gen-l10n.
• Add metadata for {fields}.

+  "@askAiConfigMissing": {
+    "description": "必填的 AI 设置缺失时显示。",
+    "placeholders": {
+      "fields": { "type": "String", "example": "基础 URL / API 密钥 / 模型" }
+    }
+  },

Based on coding guidelines.

---

299-300: ZH copy nit: phrasing for “insert into terminal”.

Reads more naturally with “到”.

-  "askAiInsertTerminal": "插入终端",
+  "askAiInsertTerminal": "插入到终端",

lib/l10n/app_fr.arb (1)

287-304: Run gen-l10n + small copy tweak + placeholder metadata

• After ARB edits, run: flutter gen-l10n. As per coding guidelines.
• Replace the visual "|" with natural language to avoid odd rendering.
• Consider adding placeholders metadata for fields in askAiConfigMissing.

Apply this tweak:

-  "writeScriptTip": "Après la connexion au serveur, un script sera écrit dans `~/.config/server_box` \n | `/tmp/server_box` pour surveiller l'état du système. Vous pouvez examiner le contenu du script.",
+  "writeScriptTip": "Après la connexion au serveur, un script sera écrit dans `~/.config/server_box` ou `/tmp/server_box` pour surveiller l'état du système. Vous pouvez examiner le contenu du script.",

Please confirm:

• gen-l10n was re-generated.
• All locales include the new askAi* keys to prevent missing-string fallbacks. Based on coding guidelines.

lib/data/model/ai/ask_ai_models.dart (1)

26-74: Make value/event types easier to compare and debug

• Mark leaf classes final to respect the sealed hierarchy.
• Add ==/hashCode (or use package:equatable) and toString for cleaner state tests/logs.

Example (selected types):

-@immutable
-class AskAiCommand {
+@immutable
+final class AskAiCommand {
   const AskAiCommand({
     required this.command,
     this.description = '',
     this.toolName,
   });
 
   final String command;
   final String description;
   final String? toolName;
+
+  @override
+  String toString() => 'AskAiCommand(command: $command, tool: $toolName)';
+  @override
+  bool operator ==(Object other) =>
+      identical(this, other) ||
+      other is AskAiCommand &&
+          command == other.command &&
+          description == other.description &&
+          toolName == other.toolName;
+  @override
+  int get hashCode => Object.hash(command, description, toolName);
 }
 
-class AskAiContentDelta extends AskAiEvent {
+final class AskAiContentDelta extends AskAiEvent {
   const AskAiContentDelta(this.delta);
   final String delta;
 }
 
-class AskAiToolSuggestion extends AskAiEvent {
+final class AskAiToolSuggestion extends AskAiEvent {
   const AskAiToolSuggestion(this.command);
   final AskAiCommand command;
 }
 
-class AskAiCompleted extends AskAiEvent {
+final class AskAiCompleted extends AskAiEvent {
   const AskAiCompleted({
     required this.fullText,
     required this.commands,
   });

lib/l10n/app_tr.arb (1)

287-304: Run gen-l10n + small copy tweak + placeholder metadata

• After ARB edits, run: flutter gen-l10n. As per coding guidelines.
• Replace the "|" with natural language for clarity.
• Consider adding placeholders metadata for askAiConfigMissing.

Apply this tweak:

-  "writeScriptTip": "Sunucuya bağlandıktan sonra, sistem durumunu izlemek için `~/.config/server_box` \n | `/tmp/server_box` dizinine bir betik yazılacak. Betik içeriğini inceleyebilirsiniz.",
+  "writeScriptTip": "Sunucuya bağlandıktan sonra, sistem durumunu izlemek için `~/.config/server_box` veya `/tmp/server_box` dizinine bir betik yazılacak. Betik içeriğini inceleyebilirsiniz.",

Please confirm gen-l10n was regenerated and all locales received the askAi* keys. Based on coding guidelines.

lib/view/page/setting/entry.dart (1)

124-124: Localize the “AI” section title

Use existing l10n (askAi) instead of a hardcoded string. As per coding guidelines.

-        [const CenterGreyTitle('App'), _buildApp(), const CenterGreyTitle('AI'), _buildAskAiConfig()],
+        [const CenterGreyTitle('App'), _buildApp(), CenterGreyTitle(libL10n.askAi), _buildAskAiConfig()],

lib/view/page/ssh/page/page.dart (2)

249-253: Verify TerminalView.toolbarBuilder support in your xterm version

Ensure the xterm/ui TerminalView you depend on exposes toolbarBuilder with CustomTextEditState; otherwise this will fail at compile/runtime. Gate with isMobile (since showToolbar is mobile-only) or null out builder when unsupported.

Example guard:

-          toolbarBuilder: _buildTerminalToolbar,
+          toolbarBuilder: isMobile ? _buildTerminalToolbar : null,

---

9-26: Trim + bound selection length before invoking AI

Very long selections can bloat prompts and UI. Consider limiting to N KB (e.g., 8–16 KB) and indicating truncation in the sheet to keep latency/cost predictable.

-  void _showAskAiSheet(String selection) async {
+  void _showAskAiSheet(String selection) async {
+    const max = 16 * 1024; // 16KB
+    final trimmed = selection.trim();
+    final safe = trimmed.length > max ? '${trimmed.substring(0, max)}…' : trimmed;
     ...
-        return _AskAiSheet(selection: selection, localeHint: localeHint, onCommandApply: _applyAiCommand);
+        return _AskAiSheet(selection: safe, localeHint: localeHint, onCommandApply: _applyAiCommand);

lib/view/page/setting/entries/ai.dart (2)

55-94: Add per-field validation (URL format, model non-empty) to dialog

Introduce an optional validator to _showAskAiFieldDialog and validate base URL (https scheme, no trailing spaces) and non-empty model. Prevents misconfig and reduces support churn.

-  Future<void> _showAskAiFieldDialog({
+  Future<void> _showAskAiFieldDialog({
     required HiveProp<String> prop,
     required String title,
     required String hint,
     bool obscure = false,
+    String? Function(String value)? validator,
   }) async {
 ...
-      void onSave() {
-        prop.put(ctrl.text.trim());
+      void onSave() {
+        final v = ctrl.text.trim();
+        final err = validator?.call(v);
+        if (err != null) {
+          context.showSnackBar(err);
+          return;
+        }
+        prop.put(v);
         context.pop();
       }

Usage example for Base URL:

_showAskAiFieldDialog(
  prop: _setting.askAiBaseUrl,
  title: l10n.askAiBaseUrl,
  hint: 'https://api.openai.com',
  validator: (v) {
    final u = Uri.tryParse(v);
    if (u == null || !(u.isScheme('https') || u.isScheme('http')) || u.host.isEmpty) {
      return libL10n.invalid;
    }
    return null;
  },
);

---

36-49: Offer “Paste” and “Reveal” toggles for API key input

Small UX wins: add a paste shortcut and an eye icon to temporarily reveal the key.

lib/l10n/app_ru.arb (1)

287-304: Add placeholders metadata for askAiConfigMissing and regenerate l10n

Same as DE locale; declare “fields”.

  "askAiConfigMissing": "Настройте {fields} в настройках.",
+  "@askAiConfigMissing": {
+    "placeholders": {
+      "fields": { "type": "String" }
+    }
+  },

As per coding guidelines.

lib/view/page/ssh/page/ask_ai.dart (4)

363-373: Add a “Stop” action to cancel streaming

Allow users to abort long generations, canceling the subscription and marking idle.

-                  if (_isStreaming)
-                    const SizedBox(height: 16, width: 16, child: CircularProgressIndicator(strokeWidth: 2)),
+                  if (_isStreaming) ...[
+                    const SizedBox(height: 16, width: 16, child: CircularProgressIndicator(strokeWidth: 2)),
+                    const SizedBox(width: 8),
+                    IconButton(
+                      tooltip: 'Stop',
+                      icon: const Icon(Icons.stop_circle_outlined),
+                      onPressed: () {
+                        _subscription?.cancel();
+                        setState(() {
+                          _isStreaming = false;
+                          _streamingContent = null;
+                        });
+                      },
+                    ),
+                  ],

---

183-193: Throttle auto-scroll to reduce jank during rapid deltas

Multiple animateTo calls per frame can stutter. Throttle or use jumpTo when close to bottom.

-  void _scheduleAutoScroll() {
+  void _scheduleAutoScroll() {
     if (!_scrollController.hasClients) return;
     WidgetsBinding.instance.addPostFrameCallback((_) {
       if (!_scrollController.hasClients) return;
-      _scrollController.animateTo(
+      final pos = _scrollController.position;
+      final nearBottom = (pos.maxScrollExtent - pos.pixels) < 80;
+      (nearBottom ? _scrollController.animateTo : _scrollController.jumpTo)(
         _scrollController.position.maxScrollExtent,
         duration: const Duration(milliseconds: 180),
         curve: Curves.easeOutCubic,
       );
     });
   }

---

295-304: Ensure markdown rendering is safe

If SimpleMarkdown supports HTML, disable raw HTML to avoid accidental rendering of unsafe content from LLMs.

Example:

SimpleMarkdown(
  data: content,
  enableHtml: false, // or equivalent flag in your widget
)

---

221-235: Keep “insert vs execute” wording consistent

Dialog title says “Confirm execute” but action only inserts text into the terminal. Consider renaming to “Confirm insertion” to avoid confusion.

lib/data/provider/ai/ask_ai.dart (10)

11-13: Use autoDispose for the provider.

This repository is stateless; autoDispose reduces leak risk in short‑lived pages.

-final askAiRepositoryProvider = Provider<AskAiRepository>((ref) {
-  return AskAiRepository();
-});
+final askAiRepositoryProvider =
+    Provider.autoDispose<AskAiRepository>((ref) => AskAiRepository());

---

42-47: Add SSE-friendly headers and optional language hint.

Improves intermediary behavior and localization.

   final headers = <String, String>{
     Headers.acceptHeader: 'text/event-stream',
     Headers.contentTypeHeader: Headers.jsonContentType,
     'Authorization': authHeader,
+    'Cache-Control': 'no-cache',
   };
+  if (localeHint != null && localeHint.isNotEmpty) {
+    headers['Accept-Language'] = localeHint;
+  }

---

63-65: SSE streams need longer/no receiveTimeout.

2 minutes can prematurely abort long generations. Use a large value or disable receiveTimeout.

-          receiveTimeout: const Duration(minutes: 2),
+          // Long-lived SSE
+          receiveTimeout: const Duration(hours: 1),

Based on learnings (Dio 5.x timeouts).

---

85-88: Handle CRLF separators in SSE framing.

Split on blank line with optional CR to avoid stray “\r” in payloads.

-        final segments = carry.toString().split('\n\n');
+        final segments = carry.toString().split(RegExp(r'\r?\n\r?\n'));
         carry
           ..clear()
           ..write(segments.removeLast());

---

90-101: Assemble multi-line SSE data: events before decoding.

Some servers send multiple data: lines per event. Join them with “\n” before jsonDecode; only then handle [DONE].

// Inside "for (final segment in segments) {"
final dataLines = <String>[];
for (final rawLine in segment.split('\n')) {
  final line = rawLine.trim();
  if (line.startsWith('data:')) {
    dataLines.add(line.substring(5).trim());
  }
}
if (dataLines.isEmpty) continue;

final payload = dataLines.join('\n');
if (payload == '[DONE]') { /* completed */ }
// else jsonDecode(payload)...

---

196-215: Move system prompt to i18n and neutral fallback.

Hardcoding CN text and CN fallback may surprise non‑CN users. Externalize prompt and default to English when localeHint is absent.

---

231-258: Consider explicit tool choice.

Some OpenAI‑compatible servers honor tool_choice: 'auto'. If supported, set it explicitly; otherwise skip.

     return {
       'model': model,
       'stream': true,
       'messages': messages,
+      'tool_choice': 'auto',
       'tools': [

---

261-265: Prefer URI resolve for safer joining.

Handles edge cases (query, base with path) more robustly.

-  Uri _composeUri(String base, String path) {
-    final sanitizedBase = base.replaceAll(RegExp(r'/+$'), '');
-    final sanitizedPath = path.replaceFirst(RegExp(r'^/+'), '');
-    return Uri.parse('$sanitizedBase/$sanitizedPath');
-  }
+  Uri _composeUri(String base, String path) {
+    final baseUri = Uri.parse(base.trim());
+    return baseUri.resolve(path);
+  }

---

288-293: Clear accumulated arguments after emission.

Prevents accidental re‑parsing if _emitted is reset in future changes.

       _emitted = true;
+      arguments.clear();
       return AskAiCommand(

---

55-69: Redact Authorization in logs (if LogInterceptor is used).

Ensure debug logging doesn’t leak API keys. Configure a custom LogInterceptor or disable request headers in debug.

Based on learnings (Dio 5.x).

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro