Исправлена ошибка из-за которой набор не обнаруживался если там есть закомментированный конструктор

[sfaqer]

closes: #8

Summary by CodeRabbit

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

[coderabbitai]

Walkthrough

Рефакторинг логики обнаружения конструктора объекта: замена проверки вхождения подстроки на построчное сканирование текста с требованием, чтобы целевые процедуры начинались в начале строки. Также добавлены комментарии в тестовый файл.

Changes

Cohort / File(s)	Изменения
Рефакторинг обнаружения конструктора src/core/internal/Классы/ЗаготовкаТестирование.os	Изменена логика поиска конструктора с использования подстроки (indexOf) на итеративное построчное сканирование; теперь требуется, чтобы "Процедура ПриСозданииОбъекта" или "Procedure OnObjectCreation" начинали строку
Обновление тестовых данных tests/fixtures/НаборБезКонструктора.os	Добавлены закомментированные блоки процедур без изменения исполняемого кода

Sequence Diagram

sequenceDiagram
    participant Old as Старый подход
    participant New as Новый подход
    participant Text as Текст

    rect rgba(200, 220, 255, 0.3)
    Note over Old: indexOf/contains
    Old->>Text: Поиск подстроки<br/>в произвольном месте
    Text-->>Old: Найдена? true/false
    end

    rect rgba(220, 255, 200, 0.3)
    Note over New: Построчное сканирование
    loop Каждая строка
        New->>Text: Строка начинает<br/>с целевой фразы?
        alt Совпадение в начале
            Text-->>New: true → возврат
        else Совпадение не в начале
            New->>New: Перейти к следующей
        end
    end
    New-->>New: Возврат false<br/>после цикла
    end

Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 минут

Обратить внимание:

• Изменение семантики поиска: требуется соответствие в начале строки вместо поиска по всему тексту — может влиять на обработку нестандартного форматирования кода
• Убедиться, что все существующие тесты проходят с новой логикой обнаружения

Poem

🐰 Скан строка за строкой, не спеша,
Конструктор в начале найти спешит,
Процедура ПриСозданья поёт,
Рефакторинг логик в коде живёт! 🌟

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	Название четко описывает основное изменение: исправление ошибки обнаружения набора при наличии закомментированного конструктора.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch bugfix/8

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 8250275 and 5c6cb33.

📒 Files selected for processing (2)

• src/core/internal/Классы/ЗаготовкаТестирование.os (1 hunks)
• tests/fixtures/НаборБезКонструктора.os (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (3)

• GitHub Check: test (dev) / build (ubuntu-latest, dev)
• GitHub Check: test (default) / build (ubuntu-latest, default)
• GitHub Check: test (default) / build (windows-latest, default)

🔇 Additional comments (2)

tests/fixtures/НаборБезКонструктора.os (1)

6-7: Правильное дополнение тестового сценария.

Добавление закомментированных конструкторов в фикстуру "НаборБезКонструктора" правильно тестирует исправленный баг. Новая логика обнаружения должна корректно игнорировать эти комментарии и не обнаруживать конструктор в данном наборе.

Also applies to: 10-11

src/core/internal/Классы/ЗаготовкаТестирование.os (1)

88-99: Исправление подтверждено, запрос проверки отклонен.

Построчное сканирование с СтрНачинаетсяС корректно решает проблему с закомментированными конструкторами и соответствует структуре кодовой базы. Проверка показала, что в репозитории отсутствуют конструкторы с отступами на уровне модуля, поэтому дополнительная обработка пробелов не требуется. Реализация правильна для фактической структуры кода.

---

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.}