refactor: исправлены замечания code review

Изменения:
- Исправлено форматирование Markdown в plugin-assistant-system-prompt.md
  (добавлены пустые строки вокруг блоков кода, исправлена нумерация)
- Заменен оператор || на ?? для значений по умолчанию в select полях
  (предотвращает неожиданное поведение при falsy значениях)
- Исправлен bug_risk: используется plugin.settings?.actionsPreset
  (опциональная цепочка для предотвращения ошибок при неинициализированных settings)
- Извлечена функция shouldShowField в общую утилиту lib/pluginSettingsUtils.js
  (устранено дублирование кода между компонентами)

Адресует замечания CodeRabbit и Sourcery.

Author: artemploxoyy

backend/src/ai/plugin-assistant-system-prompt.md

@@ -482,11 +482,13 @@ bot.events.on('core:raw_message', (rawText, jsonMsg) => {
 **Формат options:**
 
 1. **Простой массив строк** - когда значение и label совпадают:
+
 ```javascript
 "options": ["easy", "normal", "hard"]
 ```
 
-2. **Массив объектов** - когда нужны разные value и label:
+1. **Массив объектов** - когда нужны разные value и label:
+
 ```javascript
 "options": [
     { "value": "ru", "label": "Русский" },
@@ -499,6 +501,7 @@ bot.events.on('core:raw_message', (rawText, jsonMsg) => {
 Сохраненное значение всегда будет строкой (например: `"normal"`, `"ru"`)
 
 **Пример использования в плагине:**
+
 ```javascript
 module.exports = (bot, { settings }) => {
     const mode = settings.mode; // "easy" | "normal" | "hard"

frontend/src/components/PluginSettingsDialog.jsx

@@ -15,6 +15,7 @@ import { apiHelper } from '@/lib/api';
 import PluginDetailInfo from './PluginDetailInfo';
 import { Tooltip, TooltipContent, TooltipProvider, TooltipTrigger } from "@/components/ui/tooltip";
 import { useAppStore } from '@/stores/appStore';
+import { shouldShowField } from '@/lib/pluginSettingsUtils';
 
 function JsonEditorDialog({ initialValue, onSave, onCancel }) {
     const [jsonString, setJsonString] = useState('');
@@ -205,7 +206,7 @@ function SettingField({ settingKey, config, value, onChange, readOnly }) {
             return (
                 <div className="space-y-2">
                     <Label htmlFor={id}>{config.label}</Label>
-                    <Select value={value || config.default || ''} onValueChange={readOnly ? undefined : ((newValue) => onChange(settingKey, newValue))} disabled={readOnly}>
+                    <Select value={value ?? config.default ?? ''} onValueChange={readOnly ? undefined : ((newValue) => onChange(settingKey, newValue))} disabled={readOnly}>
                         <SelectTrigger id={id}>
                             <SelectValue placeholder="Выберите значение" />
                         </SelectTrigger>
@@ -384,16 +385,6 @@ export default function PluginSettingsDialog({ bot, plugin, onOpenChange, onSave
         }
     };
 
-    // Определяем, какие поля показывать на основе условий
-    const shouldShowField = (key, config) => {
-        // Если есть поле actionsPreset, то поля enable* показываем только при custom
-        const actionsPresetValue = settings?.actionsPreset;
-        if (actionsPresetValue !== undefined && key.startsWith('enable')) {
-            return actionsPresetValue === 'custom';
-        }
-        return true;
-    };
-
     const renderSettings = () => {
         if (settings === null) return <div className="text-center p-4"><Loader2 className="h-6 w-6 animate-spin mx-auto"/></div>;
         if (Object.keys(manifestSettings).length === 0) return <p className="text-muted-foreground p-4 text-center">У этого плагина нет настроек.</p>;
@@ -407,7 +398,7 @@ export default function PluginSettingsDialog({ bot, plugin, onOpenChange, onSave
                             <AccordionContent className="pt-4 border-t space-y-4">
                                 {Object.entries(categoryConfig)
                                     .filter(([key]) => key !== 'label')
-                                    .filter(([key, config]) => shouldShowField(key, config))
+                                    .filter(([key, config]) => shouldShowField(key, settings?.actionsPreset))
                                     .map(([key, config]) => (
                                         <SettingField
                                             key={key}
@@ -428,7 +419,7 @@ export default function PluginSettingsDialog({ bot, plugin, onOpenChange, onSave
         return (
             <div className="space-y-4">
                 {Object.entries(manifestSettings)
-                    .filter(([key, config]) => shouldShowField(key, config))
+                    .filter(([key, config]) => shouldShowField(key, settings?.actionsPreset))
                     .map(([key, config]) => (
                         <SettingField
                             key={key}

frontend/src/components/PluginSettingsForm.jsx

@@ -9,6 +9,7 @@ import { Edit } from 'lucide-react';
 import { Dialog, DialogContent, DialogDescription, DialogHeader, DialogTitle, DialogFooter, DialogTrigger } from "@/components/ui/dialog";
 import Editor from '@monaco-editor/react';
 import { useAppStore } from '@/stores/appStore';
+import { shouldShowField } from '@/lib/pluginSettingsUtils';
 
 function JsonEditorDialog({ initialValue, onSave, onCancel }) {
     const [jsonString, setJsonString] = useState('');
@@ -201,7 +202,7 @@ function SettingField({ settingKey, config, value, onChange }) {
             return (
                 <div className="space-y-2">
                     <Label htmlFor={id}>{config.label}</Label>
-                    <Select value={value || config.default || ''} onValueChange={(newValue) => onChange(settingKey, newValue)}>
+                    <Select value={value ?? config.default ?? ''} onValueChange={(newValue) => onChange(settingKey, newValue)}>
                         <SelectTrigger id={id}>
                             <SelectValue placeholder="Выберите значение" />
                         </SelectTrigger>
@@ -257,20 +258,10 @@ export default function PluginSettingsForm({ plugin, onSettingsChange }) {
         onSettingsChange(plugin.id, newSettings);
     };
 
-    // Определяем, какие поля показывать на основе условий
-    const shouldShowField = (key, config) => {
-        // Если есть поле actionsPreset, то поля enable* показываем только при custom
-        const actionsPresetValue = plugin.settings.actionsPreset;
-        if (actionsPresetValue !== undefined && key.startsWith('enable')) {
-            return actionsPresetValue === 'custom';
-        }
-        return true;
-    };
-
     return (
         <div className="space-y-6">
             {Object.entries(plugin.manifest.settings)
-                .filter(([key, config]) => shouldShowField(key, config))
+                .filter(([key, config]) => shouldShowField(key, plugin.settings?.actionsPreset))
                 .map(([key, config]) => (
                     <SettingField
                         key={key}

frontend/src/lib/pluginSettingsUtils.js

@@ -0,0 +1,15 @@
+/**
+ * Определяет, должно ли поле настройки плагина отображаться
+ * на основе условий (например, actionsPreset)
+ *
+ * @param {string} key - Ключ поля настройки
+ * @param {any} actionsPresetValue - Значение поля actionsPreset (если есть)
+ * @returns {boolean} - Должно ли поле отображаться
+ */
+export const shouldShowField = (key, actionsPresetValue) => {
+    // Если есть поле actionsPreset, то поля enable* показываем только при custom
+    if (actionsPresetValue !== undefined && key.startsWith('enable')) {
+        return actionsPresetValue === 'custom';
+    }
+    return true;
+};