Удаление ненужных сообщений, удаление дублирования кода update viewport final, исправление приведени типов, работа над viewport form, исправление динамических секций

This commit is contained in:
XeRo 2026-01-16 17:44:03 +05:00
parent 300afb5d26
commit ecc89e34c5
11 changed files with 168 additions and 142 deletions

View file

@ -162,10 +162,6 @@ func Recompute(app *types.PackageApplication) {
for j := range section.Fields {
field := &section.Fields[j]
if field.FieldType == "compute" && field.Compute != "" {
if field.Variable != nil {
// todo почему мы пропускаем?
continue
}
value, err := ComputeValue(field.Compute, app)
if err != nil {
value = fmt.Sprintf("Ошибка вычисления: %v", err)

View file

@ -2,7 +2,6 @@ package fields
import (
"bubble/setup/ui"
"bubble/types"
"fmt"
"github.com/charmbracelet/bubbles/textinput"
tea "github.com/charmbracelet/bubbletea"
@ -82,14 +81,8 @@ func (m *DirSelectModel) Update(msg tea.Msg) (FieldModel, tea.Cmd) {
m.dir = newValue
m.input.CursorEnd()
m.open = false
selected := types.SelectedDirMsg{
DirectoryName: fileName,
Index: m.focus,
}
m.focus = 0
return m, func() tea.Msg {
return selected
}
return m, nil
}
}
} else {

View file

@ -2,6 +2,8 @@ package fields
import tea "github.com/charmbracelet/bubbletea"
// todo комментарии к полям
// todo описание поля выводить
type FieldModel interface {
Init() tea.Cmd
Update(msg tea.Msg) (FieldModel, tea.Cmd)

View file

@ -2,7 +2,6 @@ package fields
import (
"bubble/setup/ui"
"bubble/types"
"fmt"
"github.com/charmbracelet/bubbles/textinput"
tea "github.com/charmbracelet/bubbletea"
@ -92,15 +91,9 @@ func (m *FileSelectModel) Update(msg tea.Msg) (FieldModel, tea.Cmd) {
m.file = fullPath
m.input.SetValue(fullPath)
m.open = false
selected := types.SelectedFileMsg{
Filename: fileName,
Index: m.focus,
}
m.input.CursorEnd()
m.focus = 0
return m, func() tea.Msg {
return selected
}
return m, nil
}
}
} else {

View file

@ -1,7 +1,6 @@
package fields
import (
"bubble/types"
"fmt"
"github.com/charmbracelet/bubbles/textinput"
tea "github.com/charmbracelet/bubbletea"
@ -60,15 +59,9 @@ func (m *SelectModel) Update(msg tea.Msg) (FieldModel, tea.Cmd) {
return m, nil
}
m.chosen = m.options[m.focus]
selected := types.SelectedOptionMsg{
FieldIndex: m.focus,
Value: m.chosen,
}
m.open = false
m.focus = len(m.options)
return m, func() tea.Msg {
return selected
}
return m, nil
}
} else {
switch msg.String() {

View file

@ -43,24 +43,6 @@ func (m *FinalModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
case tea.KeyMsg:
//Отслеживаем нажатие клавиш
switch msg.String() {
case "up":
//Обработчик up для прокручивания вверх на 1 позицию
m.viewport.ScrollUp(1)
case "down":
//Обработчик down для прокручивания вниз на 1 позицию
m.viewport.ScrollDown(1)
case "left":
//Обработчик left для прокручивания влево на 1 позицию
m.viewport.ScrollLeft(1)
case "right":
//Обработчик right для прокручивания вправо на 1 позицию
m.viewport.ScrollRight(1)
case "pageup":
//Обработчик pageup для прокручивания вверх на половину допустимой высоты viewport
m.viewport.HalfPageUp()
case "pagedown":
//Обработчик pagedown для прокручивания вниз на половину допустимой высоты viewport
m.viewport.HalfPageDown()
case "home":
//Обработчик home для перехода в самый вверх
m.viewport.GotoTop()
@ -75,7 +57,6 @@ func (m *FinalModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
m.initViewport(msg.Width, msg.Height)
}
//Если сообщение было иным, передаем его напрямую в viewport
//todo проверить есть ли эти функции уже у viewport.Update, возможно мой код дублирует
m.viewport, cmd = m.viewport.Update(msg)
return m, cmd
}
@ -123,7 +104,7 @@ func (m *FinalModel) initViewport(width, height int) {
//Создание видимой области, куда будет помещена таблица
m.viewport = viewport.New(contentWidth, tableHeight)
m.viewport.SetHorizontalStep(1)
//Обновление таблицы
m.rebuildTable(tableWidth)
}

View file

@ -5,6 +5,7 @@ import (
"bubble/setup/ui"
"bubble/types"
"fmt"
"github.com/charmbracelet/bubbles/viewport"
tea "github.com/charmbracelet/bubbletea"
"github.com/charmbracelet/lipgloss"
"github.com/muesli/reflow/wordwrap"
@ -17,7 +18,8 @@ type FormModel struct {
CurrentSection *types.Section //Указатель секции с которой надо работать
FieldModels []fields.FieldModel //Массив моделей полей
focus int //Фокус для отслеживания с каким объектом надо работать
renderedContent string //Строка хранения информации о приложении
renderedContent string //Строка хранения информации о приложении // todo поместить во viewport
viewport viewport.Model //Объект куда помещена информация о приложении для возможности прокручивать её
width int //Ширина окна
height int //Высота окна
}
@ -80,10 +82,13 @@ func NewFormModel(application *types.PackageApplication, sectionIdx, focus, widt
width: width,
height: height,
}
//Формируем описание приложения если секция установлена About
if currentSection.Type == types.SectionAbout {
m.renderedContent = simpleAboutRender(currentSection.Paragraphs, width)
m.focus = 0
////Формируем описание приложения если секция установлена About
//if currentSection.Type == types.SectionAbout {
// m.renderedContent = simpleAboutRender(currentSection.Paragraphs, width)
// m.focus = 0
//}
if width > 0 && height > 0 {
m.initViewport(width, height)
}
return m
}
@ -100,9 +105,10 @@ func (m *FormModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
//Отслеживаем и записываем новые размеры окна, обновляем рендер about
m.width = msg.Width
m.height = msg.Height
if m.CurrentSection.Type == types.SectionAbout {
m.renderedContent = simpleAboutRender(m.CurrentSection.Paragraphs, m.width)
}
m.initViewport(m.width, m.height)
//if m.CurrentSection.Type == types.SectionAbout {
// m.renderedContent = simpleAboutRender(m.CurrentSection.Paragraphs, m.width)
//}
return m, nil
case tea.KeyMsg:
//Отслеживаем нажатие клавиш формы
@ -152,13 +158,20 @@ func (m *FormModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
case "tab", "shift+tab", "down", "up":
//Обработчик клавиш tab, shift+tab, down, up на форме нужен для перехода между объектами формы.
//Получаем точное имя клавиши нажатия, после проверяем где сейчас находится фокус.
//Если фокус не равен количеству полей, то значит фокус сейчас у поля формы.
//Если фокус меньше количества полей, то значит фокус сейчас на поле формы.
// Подробнее. У нас фокус в двух состояниях так сказать,
// фокус равен количество полей, значит он на кнопке следующей секции,
// если меньше количества полей или равен 0, значит фокус на поле формы.
// Если у нас нет полей на форме (секция About), значит количество полей равно 0, а фокус по умолчанию устанавливается 0.
// Чтобы m.FieldModels[m.focus] вызвало ошибку "индекс вне массива",
// нужно чтобы в условие m.focus < len(m.FieldModels) попал фокус меньше 0 (-1,-2 и тд).
// Чтобы этого избежать, есть проверка, которая устанавливает m.focus на последнюю позицию (количество полей на форме).
// Чтобы эта проверка дала сбой, нужно после неё прописать изменение фокуса формы на отрицательный принудительно, так как такое не может произойти в текущем коде.
//Получаем имя поля (так как это старый фокус, то и поле на котором мы находимся это предыдущее поле, отсюда и название переменной),
//Если поле не открыто, то мы можем перейти на следующее поле, увеличив или уменьшив фокус.
//Если фокус равен количеству полей, значит он находится на кнопке переключения секций. Увеличиваем или уменьшаем фокус.
key := msg.String()
if m.focus != len(m.FieldModels) {
//todo исправить фокус
if m.focus < len(m.FieldModels) {
prevField := m.FieldModels[m.focus]
if !prevField.CaptureKeys() {
if key == "tab" || key == "down" {
@ -185,59 +198,81 @@ func (m *FormModel) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
m.focus = len(m.FieldModels)
}
}
case types.SelectedOptionMsg:
//Отслеживаем выбор опции в поле типа select
//todo тут вообще логика нужна? Мы ведь в select, через m.chosen записываем значение, а после GetValue метод возвращаем m.chosen
logrus.Infof("chose %+v", msg)
}
//Если не было возвращено сообщений, значит действие было произведено с полем, а не с самой формой.
//Возвращаем результат функции обновления полей.
return m, m.updateFields(msg)
var cmd tea.Cmd
if m.CurrentSection.Type == types.SectionAbout {
m.viewport, cmd = m.viewport.Update(msg)
return m, cmd
} else {
return m, m.updateFields(msg)
}
}
// View формирует строку для вывода формы и её контента.
func (m *FormModel) View() string {
//Данные о размерах контейнера и ширины контента внутри
containerWidth, containerHeight, contentWidth, _ := ui.ContentSizes(m.width, m.height)
formWidth := contentWidth - 2*1
var strokes []string //отображаемые строки
for i, field := range m.FieldModels {
//Строка с учетом переноса поля.
//Имеется ввиду, что если вводимое значение слишком длинное, то оно будет делиться на несколько строк.
fieldStroke := wordwrap.String(field.View(), formWidth)
//Если фокус равен текущему полю, то выделяем его.
if i == m.focus {
strokes = append(strokes, ui.FormFieldFocused.Render(fieldStroke))
} else {
strokes = append(strokes, ui.FormFieldUnfocused.Render(fieldStroke))
}
}
//Если текущая секция это секция About, то выводим её информацию.
//Не предполагается, что информация о приложении и поля для настройки будут в одной секции, поэтому тут идет обнуление прошлых строк.
if m.CurrentSection.Type == types.SectionAbout {
strokes = append([]string{}, m.renderedContent)
}
//Строка кнопки переключения секций.
//Если позиция фокуса равняется количеству полей, значит он сейчас находится на кнопке.
nextBtn := "[ next section ]"
if m.focus == len(m.FieldModels) {
strokes = append(strokes, ui.FormNextButton.Render(nextBtn))
} else {
strokes = append(strokes, ui.FormFieldUnfocused.Render(nextBtn))
}
//Основной контейнер приложения.
//Устанавливаем ему ширину и высоту, объединяем все строки в одну с соединением, через двойной перевод строки.
containerWidth, containerHeight, contentWidth, contentHeight := ui.ContentSizes(m.width, m.height)
formWidth := contentWidth - 2*1
//formHeight:=contentHeight-2*1
_ = contentHeight - 2*1
m.rebuildContent(formWidth)
container := ui.AppContainer.
Width(containerWidth).
Height(containerHeight).
Render(strings.Join(strokes, "\n\n"))
Render(m.renderedContent)
return lipgloss.Place(
m.width, m.height,
lipgloss.Center, lipgloss.Center,
container,
lipgloss.WithWhitespaceChars(" "),
lipgloss.WithWhitespaceChars("0"),
lipgloss.WithWhitespaceForeground(ui.ColorError),
)
////Данные о размерах контейнера и ширины контента внутри
//containerWidth, containerHeight, contentWidth, _ := ui.ContentSizes(m.width, m.height)
//formWidth := contentWidth - 2*1
//var strokes []string //отображаемые строки
//for i, field := range m.FieldModels {
// //Строка с учетом переноса поля.
// //Имеется ввиду, что если вводимое значение слишком длинное, то оно будет делиться на несколько строк.
// fieldStroke := wordwrap.String(field.View(), formWidth)
// //Если фокус равен текущему полю, то выделяем его.
// if i == m.focus {
// strokes = append(strokes, ui.FormFieldFocused.Render(fieldStroke))
// } else {
// strokes = append(strokes, ui.FormFieldUnfocused.Render(fieldStroke))
// }
//}
////Если текущая секция это секция About, то выводим её информацию.
////Не предполагается, что информация о приложении и поля для настройки будут в одной секции, поэтому тут идет обнуление прошлых строк.
//if m.CurrentSection.Type == types.SectionAbout {
// strokes = append([]string{}, m.renderedContent)
//}
////Строка кнопки переключения секций.
////Если позиция фокуса равняется количеству полей, значит он сейчас находится на кнопке.
//nextBtn := "[ next section ]"
//if m.focus == len(m.FieldModels) {
// strokes = append(strokes, ui.FormNextButton.Render(nextBtn))
//} else {
// strokes = append(strokes, ui.FormFieldUnfocused.Render(nextBtn))
//}
//
////Основной контейнер приложения.
////Устанавливаем ему ширину и высоту, объединяем все строки в одну с соединением, через двойной перевод строки.
//container := ui.AppContainer.
// Width(containerWidth).
// Height(containerHeight).
// Render(strings.Join(strokes, "\n\n"))
//return lipgloss.Place(
// m.width, m.height,
// lipgloss.Center, lipgloss.Center,
// container,
// lipgloss.WithWhitespaceChars(" "),
//)
}
// updateFields обновляет поля и их внутренние данные, формирует единую команду действий для всех полей, в том числе возврат SugarMsg
@ -260,15 +295,19 @@ func (m *FormModel) updateFields(msg tea.Msg) tea.Cmd {
}
//Если поле успешно вернуло обновленное поле, то необходимо:
//обновить модель поля в текущей форме;
//проверить изменилось ли значение переменной в настройках приложения,
//проверить у поля не типа Compute изменилось ли значение переменной в настройках приложения,
//если изменилось, в массив команд помещаем команду по возврату сообщения Sugar
if updated != nil {
m.FieldModels[i] = updated
for _, sectionField := range m.CurrentSection.Fields {
if sectionField.Name == fieldModel.GetLabel() {
if sectionField.Name == fieldModel.GetLabel() && sectionField.FieldType != "compute" {
oldValue := sectionField.Variable.Value
newValue := fieldModel.GetValue()
logrus.Debugf("label %s, oldValue(%v,%T), newValue(%v,%T), equal %t, field info %+v",
sectionField.Name, oldValue, oldValue, newValue, newValue, oldValue == newValue, sectionField,
)
if oldValue != newValue {
sectionField.Variable.Value = newValue //Записываем в переменную новое значение
cmds = append(cmds, func() tea.Msg {
return types.SugarMsg{
VariableName: sectionField.VariableName,
@ -277,8 +316,7 @@ func (m *FormModel) updateFields(msg tea.Msg) tea.Cmd {
Focus: m.focus,
}
})
sectionField.Variable.Value = newValue //recompute вызывать.
//todo что? надо на гите посмотреть в истории, может я это уже сделал, ведь sugar Msg отправляется
}
}
}
@ -292,6 +330,44 @@ func (m *FormModel) updateFields(msg tea.Msg) tea.Cmd {
return tea.Batch(cmds...)
}
func (m *FormModel) initViewport(width, height int) {
_, _, contentWidth, contentHeight := ui.ContentSizes(width, height)
formWidth := contentWidth - 2*1
formHeight := contentHeight - 2*4
m.viewport = viewport.New(contentWidth, formHeight)
m.viewport.SetHorizontalStep(1)
m.rebuildContent(formWidth)
}
func (m *FormModel) rebuildContent(width int) {
var strokes []string
if m.CurrentSection.Type == types.SectionAbout {
strokes = append(strokes, simpleAboutRender(m.CurrentSection.Paragraphs, width))
} else {
for i, field := range m.FieldModels {
//Строка с учетом переноса поля.
//Имеется ввиду, что если вводимое значение слишком длинное, то оно будет делиться на несколько строк.
fieldStroke := wordwrap.String(field.View(), width)
//Если фокус равен текущему полю, то выделяем его.
if i == m.focus {
strokes = append(strokes, ui.FormFieldFocused.Render(fieldStroke))
} else {
strokes = append(strokes, ui.FormFieldUnfocused.Render(fieldStroke))
}
}
}
//Строка кнопки переключения секций.
//Если позиция фокуса равняется количеству полей, значит он сейчас находится на кнопке.
nextBtn := "[ next section ]"
if m.focus == len(m.FieldModels) {
strokes = append(strokes, ui.FormNextButton.Render(nextBtn))
} else {
strokes = append(strokes, ui.FormFieldUnfocused.Render(nextBtn))
}
m.renderedContent = strings.Join(strokes, "\n\n")
m.viewport.SetContent(m.renderedContent)
}
// simpleAboutRender формирует единую строку, описывающую настраиваемое приложение.
func simpleAboutRender(paragraphs []string, width int) string {
var sb strings.Builder

View file

@ -13,6 +13,7 @@ func Start() {
}
}
// todo подсказка горячие клавиши
// Main структура описывающая объект настроек
type Main struct {
current tea.Model //Текущая модель для отображения
@ -94,38 +95,40 @@ func (m *Main) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
}
case types.NextSectionMsg:
//Получаем сообщение о переходе на следующую секцию настраиваемого приложения.
//Увеличиваем индекс текущей секции.
m.CurrentSectionIdx++
//Если новый индекс равен
if m.CurrentSectionIdx == len(m.CurrentApplication.Sections) {
//todo что делать когда секций больше нужного?
return m, tea.Quit
}
for sectionIdx := m.CurrentSectionIdx; sectionIdx < len(m.CurrentApplication.Sections); sectionIdx++ {
//todo переделать. Учесть динамические индексы.
//Ищем индекс следующей секции, если он остался 0, то следующие секции недоступны.
//Возвращаем модальное окно с выходом к списку приложений
nextSectionIndex := 0
for sectionIdx := m.CurrentSectionIdx + 1; sectionIdx < len(m.CurrentApplication.Sections); sectionIdx++ {
section := m.CurrentApplication.Sections[sectionIdx]
if CheckCondition(section.Condition, m.CurrentApplication) {
m.CurrentSectionIdx = sectionIdx
m.CurrentSection = &section
nextSectionIndex = sectionIdx
break
}
}
if m.CurrentSection == nil {
if nextSectionIndex == 0 {
return m, func() tea.Msg {
return types.ShowModalMsg{
Type: types.ModalError,
Title: "Ошибка конфигурации",
Message: "Это приложение не имеет доступных секций для настройки\n" +
"Возможно оно повреждено или не поддерживается",
ModalButtons: []types.ModalButton{{Text: "Выйти", Value: true}},
Type: types.ModalWarning,
Title: "Ошибка секций приложения",
Message: "Указанное приложение не имеет больше доступных секций для обработки",
ModalButtons: []types.ModalButton{{
Text: "Вернутся",
Value: true,
}},
OnResult: func(result any) tea.Cmd {
return tea.Quit
return func() tea.Msg {
return types.BackToListMsg{}
}
},
}
}
}
//todo переделать. Учесть динамические индексы.
if m.CurrentSectionIdx == len(m.CurrentApplication.Sections)-1 && m.CurrentSection.Type == types.SectionFinal {
//Если следующая секция доступна записываем секцию и её индекс.
//Выводим либо финальную форму, если секция последняя и её тип SectionFinal, либо следующую простую секцию.
m.CurrentSectionIdx = nextSectionIndex
m.CurrentSection = &m.CurrentApplication.Sections[nextSectionIndex]
if nextSectionIndex == len(m.CurrentApplication.Sections)-1 && m.CurrentSection.Type == types.SectionFinal {
m.current = NewFinalModel(m.CurrentApplication.Variables, m.Width, m.Height)
} else {
m.current = NewFormModel(m.CurrentApplication, m.CurrentSectionIdx, 0, m.Width, m.Height)
@ -152,7 +155,7 @@ func (m *Main) Update(msg tea.Msg) (tea.Model, tea.Cmd) {
Title: "Ошибка",
Message: "Ошибка выбора приложения. Указанное приложение не имеет доступных секций." +
"Проверьте конфиг настраиваемого приложения на предмет выполнения условий секций",
ModalButtons: []types.ModalButton{{Text: "Вернуться к списку приложений", Value: true}},
ModalButtons: []types.ModalButton{{Text: "Вернуться", Value: true}},
OnResult: func(result any) tea.Cmd {
return func() tea.Msg {
return types.BackToListMsg{}
@ -204,11 +207,8 @@ func (m *Main) Sugar(sugar types.SugarMsg) {
m.DeactivateField(sugar.VariableName, sugar.FieldName)
}
//Если от переменной поля зависит секции или поля,
//то вызываем перерасчет значений compute полей и формируем новую форму с учетом прошлого фокуса на форме
//то формируем новую форму с учетом прошлого фокуса
if IsParent(sugar.VariableName, m.CurrentApplication.Sections) {
if sugar.Compute != "" {
Recompute(m.CurrentApplication)
}
m.current = NewFormModel(m.CurrentApplication, m.CurrentSectionIdx, sugar.Focus, m.Width, m.Height)
}
}