>>36436 (OP)

>Расскажите там как у вас дела, коллеги

мы отказались от код ревью в принципе. теперь ревью агентам отдали.

если разраб проебал прод из-за своего нейрослопа, то он виноват. одного такого уже уволили.

Не харнессите, небось, дауны?

>>36436 (OP)
Да никак. Так же, как и до ии с кодом мясных коллег - всем похуй. Я не знаю, кто зафорсил эти сраные ревью, чужой код никто никогда не читает. Если лично мне нейрослоп мешает в моей задаче, я поправлю, иначе поебать. Если код работает - он хороший, а техдолг придумали шизы с окр.

>>36436 (OP)
Я конечно не лид, но в чем проблема ревьюить через тесты? Типо есть задача, ее закрыли, ты смотришь на тесты и расширенную границу покрытия включающую твою задачу, если все ок, то закрываешь ревью, если качество покрытия тебя не устраивает, то отклоняешь мердж.
Мне вот интересно, а как вы собираетесь проводить ревью через ии агентов? Вы локально ллм разворачиваете у себя с тонкой настройкой контекста, или на платных площадках? Просто если бизнес в РФ, то сейчас сложно адекватно пользоваться зарубежными сервисами, а самому все это строить как будто бы не стоит того. Мб лучше саму архитектуру поменять разбив на те-же микросервисы, если проект на столько большой, что ревью тратит огромное количество времени

>>36795
проблема в ответственности, квалификации и вовлеченности в проект. почти во всех проектах где работал, ревью было номинальным. лид со своими шавками за пару секунд проводили "ревью" и жали аппрув, задача уходила не пред-пром где все фейлилось, а шишки летели на разраба. т.к. никакой культуры разработки не было, то и тесты были говно, а продовый код был неподдерживаемым месивом куда тупо докидывали нового говна. типичная история это придти, за полгода начать пилить фитчи, а через год срулить в другую компанию, бросив помойный проект перед очередным релизом

>>36805
Согласен, культура многое решает на проекте. Я вот шиз в ddd. Для средненьких по объёму проектах самое то. Главная фича, это общение с писателями тз на 1 языке. Они понимают за домены проекта исходя из своих бизнес задач, и пишут в разы внятнее что конкретно им надо. И не надо тратить кучу времени на лишние созвоны.
По поводу лидов на говнопроектах. Скажем так, даже на урезанном бюджете можно состряпать хотя бы простенькую согласованность. Если ее нет, это говорит о низких скилах самого лида. Как минимум всегда можно нарисовать график с простой структурой и общие правила стиля для кода. Новым ребятам можно дать пару дней на вход и изучение структуры/стиля проекта. Потом недельку за ними последить и поправить косяки, а дальше, если человек не совсем тупой, уже сможет самостоятельно писать относительно чисто и внятно. Но это конечно работает если проект пилится активно, а не доработки на пару часов.
Соответственно время на ревью я бы разделил на 2 этапа, 1 - первую неделю чекать соблюдение стиля и структуры, 2 - все бизнес задачи смотреть только по тестам, я выше в ветке уже писал подробнее. Так не надо будет тратить кучу времени на копание в чужом коде. Ты просто смотришь изменения в тестах, и если они соответствуют покрытию задачи, то пропускаешь, если где-то есть пробелы, то возвращаешь с комментарием, какого покрытия не хватает. Вот и все. А ковырять весь код, как будто обезьяний труд имхо.

>>36913

>Я вот шиз в ddd. Для средненьких по объёму проектах самое то

единицы умеют грамотно в баундед контекст сепарейшн. еще меньше в качественный код на ддд. в основном лепят трехслойку из контроллер, сервис, рипазитари. сервис в сервис, тесты на моках и т.д.

>Главная фича, это общение с писателями тз на 1 языке

обычно они болт клали на разрабов и тупо скидывают хотелки в духе "ну сделайте норм))"

единственный рабочий вариант для лида не читать нейрослоповую кашу на ревью это уволить всех кто пишет код, потому как вероятность того что они пишут код хуже опуса 99% а ревью функционала заказчиком + запрос на покрытие тестами + согласованная архитектура закрывает все что нужно. таким макаром в процессе разработки новым узким местом по времени выступает только сам заказчик.
при невозможности уволить всех, кто пишет код, проще всего уволиться самому, потому как классическая разработка уже мертва и бежит лишь по инерции. даркфактори разработка это ближайшее будущее (6 месяцев). "у всех есть все фичи и все продукты одинаковые" это среднесрочное будущее.
подписывайтесь на мой тед толк.

>>36795

>Типо есть задача, ее закрыли, ты смотришь на тесты и расширенную границу покрытия включающую твою задачу, если все ок, то закрываешь ревью, если качество покрытия тебя не устраивает, то отклоняешь мердж

Как узнать что тесты покрывают все? Я напомню что хорошие тесты должны фиксировать реализацию фичи и если в дальшейшем агент или человек начнет там править код сильно, то тесты должны упасть и показать это. Если ты не прочитал техническую реализации фичи, то сложно оценить насколько хорошо написаны тесты, имхо.

>>36775

>техдолг придумали шизы с окр

Ну это ты просто никогда не работал на проектах с 10-15 летней историей и не видел что такое реальныйт техдолг и к чему он приводит.

>>37367
Говоря о тестах которые нужно проверять, я в первую очередь имею ввиду e2e тесты. Каждая реализация новой задачи для бизнеса является сценарием. Это может быть старый сценарий с фиксами/правками. Тогда ты добавляешь в пакет тестов новые тесты для проверки работы задачи.
Если это новый сценарий, то тогда надо сделать новый пакет тестов под этот сценарий. Я нагрешил только в плане тестов, т.к. делал их через генераторы, чисто для чистоты и читабельности самих тестов. Т.е. 90+% сценариев читается как обычный вызов функций с очевидным действием, единственное что я оставил в самих тестах нетронутым, это проверки данных, на которые и следует обращать внимание для фиксации изменений.
Как понять что уровень покрытия тестов затрагивает все? Тебе и не нужно этого знать на уровне реализации, юнит тесты чекать не надо, это тесты для самих разрабов, а не для лида.То что лид должен знать, это те самые сценарии, ну извините меня, как ты можешь ставить задачу, если нет понимания, что сейчас делает твоя система, и на что она способна.
Пример, есть задача, условно добавить в лк аватарки, ты понимаешь, что это затрагивает агрегат лк, а также сущности файлов и юзера. По итогу должны быть добавлены сценарии и тесты к ним на создание, удаление, выборку и т.д. по ним + проверки, что это не обязательное добавление (твои старые тесты сценариев должны остаться неизменными и т.д.)
И самое прекрасное, что какой бы сложной не была реализация задачи, все сценарии их работы интуитивно понятны даже не разрабам. Все что должен знать лид, это понимание наличия проверок для узких мест, которые опять же крутятся внутри конкретных сущностей и агрегатов, где взаимодействие с конечным пользователем находится внутри цельного сценария. Поэтому и тесты e2e по сути своей и есть описание сценария сродне обычной документации.

>>36436 (OP)
давно уже не слоп, начиная с Opus 4.6 код очень качественный пишется.

>>40001
Хз, сидим на fable / opus 5, нейронка на любой пук в проекте выдает изменений на тысячи строк кода. Предпочитает переизобретать все подряд вместо выстраивания арзитектуры и переиспользования. Бардак растет, читать ПР невозможно, что происхоит в проекте уже кажется никто не знает. Это и есть слоп. То что могло бы решатся твиком 5 строчек и внятными тестами строк на 100 всегда делается минимум 300 строками программного кода + 700 строками тестов.

У нас рано или поздно просле разработки прототипа начинается процесс "деслопизации" и там просто тонны строчек кода идут на помойку

>>40235
У вас:
- код говно
- не сделали с помощью самого Клода доку по асем слоям

Когда Клод вам "изменений на тысячи строк кода" делает, то код точно дерьмо. Проверил лично на говнопроекте.

>>40235
"Дай дураку opus 5, он и с ним слоп сделает"

Harness is everything.

>>44235

>Harness

Дебилы выучили новый баззворд. Сначала "просто модель не та", теперь "просто харнесс не тот", а код как был говном так и остается.

Но святая АИшечка непогрешима, "вы просто не умеете ей пользоваться".

ХХРРРРР-ТЬФУУ

>>44534
Плюсую, меня прикалывает что челы когда отвечают свято уверены что на том конце идиот который не обмазался всем чем надо. Знали бы эти челы в какой компании я работаю, лол, буквально один из передовых ии стартапов с кучей очень головастых челов мирового уровня, эти ваши харнесы тут появились еще до того как это стало мейнстримом.

Если вам ваш "код с харнесами" достаточно хорош, то возможно ваша личная планка качества кода достаточно низка.
Я если буду сравнивать ИИ код с тем гавном которое я писал на галере на 2-3 год своей карьеры разработчика будучи средним формошлепом, тогда ИИ конечно отличный код пишет. Джунов то нейронка уделывает, да и мидлов тоже. Сенек с серьезными проектами за плечами - нет. Это уже инженерный уровень, а не код макакинг.

>>44601

>один из передовых ии стартапов

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

>>44605
AI podzalupkinebius, капитализация свыше 8 млрд долларов. Делаем лопаты для лопат для лопат, чтоб рахмаджон в итоге мог сайты по продже удобрений автоматизировать, платя в токенах 10x

>>44653
ебать нашелся отец-основатель. напомни, в каком году карпатый горбатый писал что градиентный спуск пишет код лучше тебя?

>>44659
А он может и пишет, да только я получаю в опционах 4 миллиона долларов, а ты нет

>>44653
>>44661
не очень бьется твоя история, грант в 4 млн (даже неважно, опционы или rsu) это что-то на синьор менеджерском, хочешь сказать ты до сих пор коды пишешь? учитывая репутацию яндекса, не верю, что ты обычный сенька с таким грантом. когда присоедилися, какой страйк?

>>44675
Не понял, а вам что, не платят столько, гайз?
Не хотел воду шебуршить, считайте, что у меня нет никаких бонусов лол.

>>44680
Хехе, да ладно, я пизжу, гайз. Не хотел никого расстраивать

>>37366

> скора

Жопу не ставь. Она у тебя уже дырявая

>>44534
Лол, освой инструмент которым пользуешься и смотри как умные люди им пользуется чтобы повторять за ними, если сам пока не разбираешься.
Ты сейчас выглядишь как дебик перешедший с перфокарт на обычное программирование, который в блокноте код пишет и ide/git и прочее называет баззвордом, при этом жалуется хули всё так неудобно и медленно разработка идёт.

В веб версии чата гпт с промптом на два предложения просят за один прогон написать всё приложение разом, а потом удивляются чего это код говно.

> ХХРРРРР-ТЬФУУ

Харчу вытереть с себя не забудь

>>44601

> Сенек с серьезными проектами за плечами - нет. Это уже инженерный уровень, а не код макакинг

Анон, если бы ты был дохуя сеньёр инженером 20 лет опыта, то ты бы смог ии объяснить по шагам чего хочешь так, что у нее бы просто места не осталось где ошибиться

> Знали бы эти челы в какой компании я работаю, лол, буквально один из передовых ии стартапов с кучей очень головастых челов мирового уровня, эти ваши харнесы тут появились еще до того как это стало мейнстримом.

У вас с кем начальство составило контракт вы тем и будете пользоваться

>>44762
Чмонь, вот я хуею с твоих проекций. "Освой инструмент" - ну освоил, я например ML занимаюсь с 2014, в сраном Яндексе работаю и там генерация llm-дрисни буквально метрика в перфревью теперь, и кодекс за сотку я сливаю почти в ноль. Я в рамках рабочих задач буквально ПИШУ то что ты называешь харнессом, и агентский цикл знаю получше чем ты своего батю ушедшего за хлебом когда тебе было 6, но нет - чмоня включает проекцию и думает что если кто-то ссыт на его святой "ЭЙ АЙ" - то это дедушка который не осилил и который с волшебным интеллектом в чате общается, он просто ХАРНЕСС не освоил..

Более чем уверен что ты на меня еще и сложность своих рабочих задач спроецировал и сидишь охуеваешь - тебе же LLM-ка спокойно генерит круды и фиксит кнопки на фронте, значит LLM умеет всё!

Знал бы ты как жалко выглядишь, говношлеп.

>>44948
Да ты хоть одну задачу-то назови?
Интегрировать самописный велосипед сеньора Аркаши, на который у ЛЛМ нет весов, потому что это внутреннее говноподелие?

>>44994
Любая задача на стыке DE и DA. Конкретнее - алертинг на отклонение бизнес метрик (число платежек, траты на рекламу и т.д.) Опус мне такой красивый бред с формулами расписал, но реализовать не смог) Вводные по таблицам я дал + у него есть дока, в тз все бизнес требования и ограничения тоже указал
Но правды ради, когда я уже сам все реализовал, внести пару фич поверх он смог


Мимо другой анон

>>45010
Ну и для разных метрик он помог унифицировать саму функцию алерта. Т.е. все что не требует каких-то знаний из вне и сводиться к крудошлепству - решать может, но все остальное уже нет. А это все остальное сейчас такая же часть IT-продуктов.
Гордых бекенд разработчиков опустили до крудошлепов. Эх, колесо сансары дало оборот

>>45012
Хз, тут мне кажется ключевое, что ему формулы расписал, а ептыть, формулы-то откуда взял? Этож самое сложное

>>45030
Я его использовал скорее как второе мнение. Я и без него понимал, что отклонение проще всего искать от среднего/медианы. Но сути не меняет, что адекватно натянуть теорию на имеющуюся базу он не может. Вообще, если ты имеешь ввиду, что он формулы не знает - их он как раз знает

Поэтому пока как DE\DA я страдаю и завидую го-крудошлепам, которые могут 90% тасок закрывать нейронками

>>45083

>Поэтому пока как DE\DA я страдаю и завидую го-крудошлепам, которые могут 90% тасок закрывать нейронками

Так это хорошо что таска нейронкой не закрывается, значит работа будет

>>44605
>>44653
>>44764
Копиум продолжается. Название говоирть не буду, меньше инфы на двачах сливаешь - крепче спишь.

>>36436 (OP)

>мои коллеги постоянно используют fucking ai slop

А кто нет? У нас код ревью это буквально копипаст друг другу ответов клода. Я не шучу. С этими смайликами дурацкими и буллет пойнтсами