fix: СинхронизацияОбъектовМетаданныхИФайлов: Не анализировать каталог объекта метаданных если он пустой (без файлов)

[Kyrales]

fix: В СинхронизацияОбъектовМетаданныхИФайлов: Не анализировать каталог объекта метаданных если он пустой (без файлов).

Плюс скорректированы тесты BDD

Summary by CodeRabbit

• Улучшения

  • Пустые каталоги при синхронизации теперь автоматически пропускаются вместо ошибки.

• Новые функции

  • Добавлена проверка наличия файлов в каталоге.
  • Добавлена процедура проверки идентичности содержимого файлов.

• Тесты

  • Обновлены сценарии: сравнение содержимого файлов ожидает идентичность; пути в тестах и шаги установки приведены к корню репозитория и учтён новый параметр установки.

_{✏️ Tip: You can customize this high-level summary in your review settings.}

[coderabbitai]

Walkthrough

Добавлены проверка равенства содержимого файлов и новый хелпер для обнаружения файлов в каталоге; тесты переключены на работу из корня репозитория (включая флаг -source-dir .); синхронизация метаданных теперь игнорирует пустые каталоги без соответствующих объектов.

Changes

Cohort / File(s)	Описание
Вспомогательные процедуры тестирования features/step_definitions/precommitonec_feature.os	Добавлена экспортируемая процедура СодержимоеФайлаИФайлаОдинаковое(ИсходныйФайл, КонечныйФайл) — сравнивает содержимое двух файлов после нормализации путей и вызывает утверждение об их равенстве.
Обновление сценариев тестирования features/ВыполнениеПрекоммита.feature, features/ФайловыеОперации.feature	Тесты изменены для проверки файлов относительно корня репозитория (.) вместо src; в команде установки добавлен параметр -source-dir .; в одном сценарии ассерция изменена с проверки различия на проверку идентичности содержимого.
Модуль работы с файлами src/Модули/ФайловыеОперации.os	Добавлена экспортируемая функция ЕстьФайлыВКаталоге(ФайлКаталога) — рекурсивно находит файлы в каталоге, фильтрует директории и возвращает логическое значение наличия файлов.
Логика синхронизации метаданных и файлов src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os	В НайтиКаталогиОбъектов добавлена проверка пустоты каталога: если каталог не соответствует объекту и в нём нет файлов, он пропускается; если есть файлы — сохраняется существующая обработка как ошибка.
Тесты сценариев обработки tests/ТестПроверкаСценариевОбработки.os	Добавлены и переименованы кейсы пустых каталогов (EDT и Configuration/пустой объект метаданных), обновлены ожидаемые сообщений об игнорировании пустых каталогов.

Sequence Diagram(s)

(пропущено)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

• Анализ наличия физических файлов, которые указываются в ПВХ #48 — содержит изменения в src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os и проверках наличия физических файлов в каталогах, пересекается по логике пропуска/валидации пустых каталогов.

Poem

🐇 Я — кролик, по коду скак,

Сравнил файлы, пустоты проскочил.
Тесты в корне весело шумят,
Хелпер файлы тихо считает.
Пусть ревью мчится — я за вами гляжу!

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Title check	✅ Passed	The PR title clearly and specifically describes the main change: skipping empty metadata object directories in the synchronization component. The title matches the core functionality added across multiple files.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@src/Модули/ФайловыеОперации.os`:
- Around line 244-264: В функции ЕстьФайлыВКаталоге исправьте опечатку имени
переменной: переименуйте ИскатьВПодкаталагах в ИскатьВПодкаталогах, а также
уберите ненужное накопление в МассивФайловКаталога — вместо создания массива и
подсчёта верните Истина сразу при обнаружении первого элемента из МассивФайлов
(элемент Файл, для которого Не Файл.ЭтоКаталог()), и верните Ложь если цикл
завершился без нахождения файлов; оставьте вызов
НайтиФайлы(ФайлКаталога.ПолноеИмя, "*", ИскатьВПодкаталогах) как есть.

🧹 Nitpick comments (1)

src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os (1)

225-233: Проверьте логику: условие НЕ ПустойКаталог может быть избыточным.

После проверки ЕстьФайлыВКаталоге(Каталог) возвращающей Истина (есть файлы рекурсивно), условие НЕ ПустойКаталог(Каталог.ПолноеИмя) всегда будет истинным, так как каталог содержит файлы.

Возможные варианты:

1. Если поведение верное — упростить код, убрав проверку ПустойКаталог:

♻️ Упрощённый вариант

 		Если МассивОбъектов.Найти(Каталог.ИмяБезРасширения) = Неопределено Тогда
 			
 			Если Не ФайловыеОперации.ЕстьФайлыВКаталоге(Каталог) Тогда
 				Продолжить;
 			КонецЕсли;
 			
-			Если НЕ ПустойКаталог(Каталог.ПолноеИмя) Тогда
-				МассивОшибок.Добавить(СтрШаблон("Каталог не является каталогом объекта %1", Каталог.ПолноеИмя));
-			КонецЕсли;
+			МассивОшибок.Добавить(СтрШаблон("Каталог не является каталогом объекта %1", Каталог.ПолноеИмя));
 			
 		Иначе

2. Если ПустойКаталог учитывает специфические случаи (например, каталог с только вложенными пустыми папками) — оставить как есть.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d3f1efd and 5335959.

📒 Files selected for processing (5)

• features/step_definitions/precommitonec_feature.os
• features/ВыполнениеПрекоммита.feature
• features/ФайловыеОперации.feature
• src/Модули/ФайловыеОперации.os
• src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os

🧰 Additional context used

🧠 Learnings (1)

📚 Learning: 2025-11-03T09:51:07.133Z

Learnt from: Kyrales
Repo: bia-technologies/precommit4onec PR: 48
File: src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os:0-0
Timestamp: 2025-11-03T09:51:07.133Z
Learning: В функции СинхронизироватьМетаданныеИФайлыПВХ в файле src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os массив МассивОшибокПриПоискеФайловОбъектов намеренно игнорируется, так как эти ошибки актуальны только для функции СинхронизироватьМетаданныеИФайлы(), но не для обработки планов видов характеристик (ПВХ).

Applied to files:

• src/СценарииОбработки/СинхронизацияОбъектовМетаданныхИФайлов.os

🔇 Additional comments (6)

features/ФайловыеОперации.feature (1)

23-23: Изменение логики теста выглядит корректно.

Проверка на идентичность содержимого файлов после коммита соответствует цели теста — убедиться, что кодировка и содержимое файла не изменяются в процессе прекоммита.

features/step_definitions/precommitonec_feature.os (2)

151-162: Реализация корректна и соответствует существующему паттерну.

Новая процедура СодержимоеФайлаИФайлаОдинаковое является симметричным аналогом существующей СодержимоеФайлаИФайлаРазное и корректно использует Ожидаем.Что(...).Равно(...) для проверки идентичности содержимого файлов.

---

23-23: LGTM!

Шаг корректно добавлен в список доступных шагов.

features/ВыполнениеПрекоммита.feature (3)

34-34: Добавление параметра -source-dir . корректно настраивает корневой каталог.

Изменение согласуется с обновлёнными путями проверки файлов в сценариях ниже.

---

43-64: Пути проверки файлов обновлены консистентно.

Все проверки корректно изменены с src на . (корень репозитория), что соответствует новому параметру -source-dir . в команде установки.

---

125-128: LGTM!

Пути в этом сценарии также согласованы с изменениями в корневом каталоге.

_{✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.}