Skip to content

Запретить legacy-опцию -all на входе CLI#268

Merged
nixel2007 merged 4 commits into
masterfrom
copilot/all
Jun 7, 2026
Merged

Запретить legacy-опцию -all на входе CLI#268
nixel2007 merged 4 commits into
masterfrom
copilot/all

Conversation

Copilot AI commented Jun 7, 2026

Copy link
Copy Markdown
Contributor

CLI все еще принимал legacy-синтаксис -all, хотя для длинных опций в проекте уже используется POSIX-форма --all. Этот PR добавляет явный отказ на точке входа и оставляет поддерживаемым только корректный вариант записи.

  • Что меняется

    • На входе приложения добавлена проверка сырых АргументыКоманднойСтроки до разбора команд.
    • При обнаружении -all запуск прерывается с явным сообщением: использовать нужно --all.
  • Где внесены изменения

    • src/cmd/opm.os
      • вызов ранней валидации аргументов до инициализации/запуска консольного приложения
    • src/cmd/Модули/ПараметрыПриложенияOpm.os
      • выделена проверка аргументов командной строки
      • добавлен запрет на token -all
    • tests/cli-args-test.os
      • добавлены сценарии на отклонение -all
      • добавлены сценарии на сохранение поддержки --all
  • Поведение

    # было: legacy-форма проходила дальше по пайплайну
    opm install -all
    
    # стало: ранний отказ с подсказкой
    Опция -all не поддерживается. Используйте --all.
    
    # корректный вариант остается валидным
    opm install --all

Copilot AI and others added 3 commits June 7, 2026 07:58
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Co-authored-by: nixel2007 <1132840+nixel2007@users.noreply.github.com>
Comment thread src/cmd/Модули/ПараметрыПриложенияOpm.os Outdated
@nixel2007 nixel2007 marked this pull request as ready for review June 7, 2026 08:12
@nixel2007 nixel2007 merged commit 85eae21 into master Jun 7, 2026
16 of 18 checks passed
@nixel2007 nixel2007 deleted the copilot/all branch June 7, 2026 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants