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

[SlumberyDude]

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

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

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

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

• yarn install
• yarn check
• yarn test unit

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

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

https://classic.yarnpkg.com/lang/en/docs/cli/workspace/
yarn workspace <workspace> add <[email protected]> <keys>

[SlumberyDude]

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

[shifuwww]

@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]

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

[shifuwww]

@SlumberyDude Привет

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

нет в buckets-config-adapter

[SlumberyDude]

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

[shifuwww]

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

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

[SlumberyDude]

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

[shifuwww]

Да, понятно, что обновление версии в модуле 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]

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

[SlumberyDude]

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

[shifuwww]

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

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

[SlumberyDude]

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

[shifuwww]

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

[SlumberyDude]

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

[shifuwww]

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

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

[SlumberyDude]

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