Обновление зависимостей `cqrs-adapter` и `buckets-config-adapter`

SlumberyDude commented 9 months ago

С чем связан запрос на фичу?

Обновить версии пакетов в зависимостях

Расскажите как вы это себе видите

Обновить версии зависимости до максимально возможных и зафиксировать таким образом, чтобы ничего не ломалось. Должны без ошибок проходить команды:

yarn install
yarn check
yarn test unit

Если в пакете есть скрипты build и prepack, то они тоже должны проходить без ошибок.

Приложите пример реализаций

https://classic.yarnpkg.com/lang/en/docs/cli/workspace/ yarn workspace <workspace> add <package@1.2.3> <keys>

SlumberyDude commented 9 months ago

@shifuwww Только называй ветки лаконичнее чем в прошлом проекте, там были слишком длинные. Названия должны быть такими, чтобы мы не пустали таски, они должны отражать суть таски и при этом желательно иметь более компактное название.

shifuwww commented 9 months ago

@SlumberyDude привет, после обновления пакета@nestjs/cqrs" до 10.2.5 в директории => services/files/core/infrastructure-module/src/repositories разошлись типы file.repository.ts: Type 'AggregateRoot<IEvent>' is missing the following properties from type 'File': id, ownerId, type, url, and 6 more. Argument of type 'File' is not assignable to parameter of type 'AggregateRoot<IEvent>'. Type 'File' is missing the following properties from type 'AggregateRoot<IEvent>': [IS_AUTO_COMMIT_ENABLED], [INTERNAL_EVENTS] upload.repository.ts Type 'AggregateRoot<IEvent>' is missing the following properties from type 'Upload': id, ownerId, url, name, and 11 more. Argument of type 'Upload' is not assignable to parameter of type 'AggregateRoot<IEvent>'. Type 'Upload' is missing the following properties from type 'AggregateRoot<IEvent>': [IS_AUTO_COMMIT_ENABLED], [INTERNAL_EVENTS] Нужно ли пробовать здесь переписывать типы?

SlumberyDude commented 9 months ago

@shifuwww Ты обновил зависимость только в модуле files/adapters/cqrs-adapter?

shifuwww commented 9 months ago

@SlumberyDude Привет

Ты обновил зависимость только в модуле files/adapters/cqrs-adapter?

нет в buckets-config-adapter

SlumberyDude commented 9 months ago

@shifuwww Хм, понял. Понять бы, почему так происходит и как можно пофиксить. Потому что судя по коду в file.repository.ts все норм, непонятно, почему он перестает воспринимать объект File как AggregateRoot, хотя вроде File экстендитAggregateRoot в domain-module.

shifuwww commented 9 months ago

@shifuwww Хм, понял. Понять бы, почему так происходит и как можно пофиксить. Потому что судя по коду в file.repository.ts все норм, непонятно, почему он перестает воспринимать объект File как AggregateRoot, хотя вроде File экстендитAggregateRoot в domain-module.

Может из за расхождений версий? В domain-module еще старая версия

SlumberyDude commented 9 months ago

@shifuwww Да, понятно, что обновление версии в модуле buckets-config-adapter-module все ломает. Но непонятно каким образом. Если cqrs модуль не обновлять, то тайпчек не расходится? И если обновить версию в domain или infrastructure модуле, то починится? Если так, то может быть обновить версию в buckets-config-adapter-module и сделать на нее временную заплатку в корневом package.json и снять ее потом после обновления уже других модулей

shifuwww commented 9 months ago

Да, понятно, что обновление версии в модуле buckets-config-adapter-module все ломает. Но непонятно каким образом. Если cqrs модуль не обновлять, то тайпчек не расходится? И если обновить версию в domain или infrastructure модуле, то починится? Если так, то может быть обновить версию в buckets-config-adapter-module и сделать на нее временную заплатку в корневом package.json и снять ее потом после обновления уже других модулей

Пробовал сделать обнолвение везде, ошибка с типами не уходит. Пробовал искать версию, но если @nestjs/cqrs обновить выше 9, то вылетает ошибка @SlumberyDude привет, после обновления пакета@nestjs/cqrs" до 10.2.5 в директории => services/files/core/infrastructure-module/src/repositories разошлись типы file.repository.ts: Type 'AggregateRoot' is missing the following properties from type 'File': id, ownerId, type, url, and 6 more. Argument of type 'File' is not assignable to parameter of type 'AggregateRoot'. Type 'File' is missing the following properties from type 'AggregateRoot': [IS_AUTO_COMMIT_ENABLED], [INTERNAL_EVENTS] upload.repository.ts Type 'AggregateRoot' is missing the following properties from type 'Upload': id, ownerId, url, name, and 11 more. Argument of type 'Upload' is not assignable to parameter of type 'AggregateRoot'. Type 'Upload' is missing the following properties from type 'AggregateRoot': [IS_AUTO_COMMIT_ENABLED], [INTERNAL_EVENTS] Нужно ли пробовать здесь переписывать типы?

SlumberyDude commented 9 months ago

@shifuwww Обновить только в infrastructure модуле недостаточно, чтобы ошибка ушла нужно обновление в entrypoint и в infrastructure. Но так как мы до них еще не дошли, то сделай эту заплатку через корневой package.json

SlumberyDude commented 9 months ago

@shifuwww Как дела с таской? Спрашивай если застрял

shifuwww commented 9 months ago

Как дела с таской? Спрашивай если застрял

Обновил @nestjs/cqrs в корневой package.json. И ошибка с типами не ушла. Пока смотрю типы воде бы все там нормально. Гуглю ошибки пока не знаю как исправить

SlumberyDude commented 9 months ago

@shifuwww Смотри в сторону resolutions. Можно селективно там версию приподнять пока мы не обновили другие пакеты, а потом убрать заплатку, когда она будет не нужна

shifuwww commented 9 months ago

@SlumberyDude Привет) Тут у меня один тест не прошел: Login to Google Container Registry, ошибка: Error: Cannot perform an interactive login from a non TTY device. Это мне докер нужно настроить с авс или это что то другое?

SlumberyDude commented 9 months ago

@shifuwww Привет, какой тест? Тут похоже на ошибку связанную с докером, поэтому если это интеграционные тесты не проходят, то думаю из-за него. Ты использовал докер раньше? Эти тесты проходят на ветке мастер без внесенных изменений или тоже с такой ошибкой ложатся?

shifuwww commented 9 months ago

Привет, какой тест? Тут похоже на ошибку связанную с докером, поэтому если это интеграционные тесты не проходят, то думаю из-за него. Ты использовал докер раньше? Эти тесты проходят на ветке мастер без внесенных изменений или тоже с такой ошибкой ложатся?

@SlumberyDude Привет) Тест первый: Preview / Preview (pull_request). Остальные прошли успешно. Если ты про тесты yarn test unit? То все проходит нормально. Докер раншье использовал, но только для деплоя. Тест упал во время пр, когда гит запускает тесты

SlumberyDude commented 9 months ago

@shifuwww Да, вижу. Думаю тут проблема с ci гитхаба и настройкой workflow.

atls / services