Ошибка чтения параметров команды convert

[Asakra]

Может кто-то столкнется с такой же проблемой и ему это поможет.

Столкнулся с тем, что не могу указать путь к репозиторию в команде convert.
Пишу команду:

ParentConfigConverter convert "." --dump="config/ConfigDumpInfo.xml" --in="config/Ext/ParentConfigurations.bin"

В итоге получаю ошибку OneScript\lib\cli\src\core\Классы\КомандаПриложения.os / Ошибка в строке: 416 / Ошибка чтения параметров команды;

Зашел в исходники. Первое что заметил это Аргумент REPO, может быть не обязательным. Сделал это:

Команда.Аргумент("REPO", ТекущийКаталог(), "Путь к git-репозиторию")
	.ТСтрока()
	.Обязательный(Ложь);

Проверил без аргумента - да, работает! Но в целом проблема не ушла.

ParentConfigConverter convert --dump="config/ConfigDumpInfo.xml" --in="config/Ext/ParentConfigurations.bin"

Не плохо было бы это исправить внутри пакета.

Решил разобраться, как передать аргумент REPO в команду. Нашел в документации к пакету cli, что такая проблема может быть и надо парсеру подсказать, где находится аргумент команды. Это делается с помощью указателя --. Только в справке не указано, что он обязательно должен быть последним. В итоге команда должна выглядеть:

ParentConfigConverter convert --dump="config/ConfigDumpInfo.xml" --in="config/Ext/ParentConfigurations.bin" -- '.'

Или

ParentConfigConverter convert --dump="config/ConfigDumpInfo.xml" --in="config/Ext/ParentConfigurations.bin" -- 'C:\repo\МойПроект'

p.s.
Команды в моих примерах нельзя использовать, так как опция dump-cache является по факту обязательной, потому что в обработчике команды нет проверки, что опция dump-cache не указана. Поэтому правильно будет указывать dump-cache, даже если он вам не нужен:

ParentConfigConverter convert --dump="config/ConfigDumpInfo.xml" --in="config/Ext/ParentConfigurations.bin" --dump-cache="build/ConfigID.xml" -- "."

ИНФОРМАЦИЯ - Выполнена конвертация файла поддержки config/Ext/ParentConfigurations.bin в файл src/Ext/ParentConfigurations_mod.bin

Вот тут не хватает проверки:

[240596448]

Порядок указания определяется в cli.
Порядок не автором придуман, есть определенные правила. И это не правила самого oscript или библиотеки.

Могу сейчас наврать, но общий порядок cli примерно такой:

1. Имя приложенния
2. Опции (именные) приложения.
3. Команда приложения
4. Опции (именные) команды.
5. Позиционные аргументы команды.

У вас в примере 5 идёт раньше 4.

Делать дополнительную защиту от дурака - можно. Но нужно ли? Так можно скатиться перепроверять вообще все подряд, что должно безусловно работать.

По хорошему cli должна обеспечить выборос исключения и не дать приложению запуститься. Неправильная строка запуска - не ответственность приложения.
Если cli не даёт достаточно информативную ошибку, то перепроверка за ней будет костылём. Костыли допускаю, но не люблю)

[Asakra]

Делать дополнительную защиту от дурака - можно. Но нужно ли?

Нет конечно. Я не для этого написал. Я создал ишью, чтобы тот кто с этим столкнется быстрее нашел решение.

Про проверку я писал, только для опции dump-cache. Она или должна быть обязательной для команды convert, либо ее значение не надо передавать в метод ПодготовитьФайлКЗаписи(), если опцию dump-cache пользователь не указал.

[Asakra]

Вот такая будет ошибка, если не передать dump-cache.

КРИТИЧНАЯОШИБКА - {Модуль C:\Users\aleksey_k\AppData\Local\Programs\OneScript\lib\fs\Модули\ФС.os / Ошибка в строке: 78 / Внешнее исключение (System.ArgumentException): The value cannot be an empty string. (Parameter 'path')}

[Asakra]

@240596448 , можете пояснить, когда нужно использовать кэш и нужно ли его хранить в репозитории или его надо хранить только локально?

[240596448]

Исправил 1.4.2