Follow-ups from #135 #138

New Issue

aep · 2026-10-27T18:03:36+03:00

aep commented

2026-10-27 18:03:36 +03:00

Отложенные находки пакета D модуля wiki (PR #136). Источник — финальное ревью ветки (M) и наблюдение контроллера (O).

M-2 — вынесенные из ссылки вложения склеены друг с другом и с остатком текста (convert.py _split_marks/convert_a), предложение — соединять через пробел (финальное ревью).
M-3 — смена состояния одного вложения пересоздаёт components в PageView: перемонтируются все ссылки и картинки страницы (мигание, потеря фокуса); комментарий в download.ts о стабильности объекта неточен (финальное ревью).
M-4 — тип blob картинки берётся из имени вложения; у картинок без расширения остаётся octet-stream, «открыть изображение в новой вкладке» скачивает файл; хаб мог бы отдавать MIME/расширение из адреса (финальное ревью).
M-5 — проверка T8.3 в check_batch не атомарна со вставкой пакета: два одновременных повтора могут оба пройти (финальное ревью).
M-6 — _line в MCP вики схлопывает только \r\n; U+2028, U+0085, \v, \f остаются (финальное ревью).
M-7 — нет компонентного теста клика по нескачанной картинке (start(id, "show")); подмена URL.createObjectURL в WikiPage.test.tsx снимается не в afterEach (финальное ревью).
O-1 — SyntaxWarning: invalid escape sequence '\w' в tests/test_api.py:1172 (докстринг), было до ветки (наблюдение контроллера).

Отложенные находки пакета D модуля wiki (PR #136). Источник — финальное ревью ветки (M) и наблюдение контроллера (O). - M-2 — вынесенные из ссылки вложения склеены друг с другом и с остатком текста (`convert.py` `_split_marks`/`convert_a`), предложение — соединять через пробел (финальное ревью). - M-3 — смена состояния одного вложения пересоздаёт `components` в `PageView`: перемонтируются все ссылки и картинки страницы (мигание, потеря фокуса); комментарий в `download.ts` о стабильности объекта неточен (финальное ревью). - M-4 — тип blob картинки берётся из имени вложения; у картинок без расширения остаётся `octet-stream`, «открыть изображение в новой вкладке» скачивает файл; хаб мог бы отдавать MIME/расширение из адреса (финальное ревью). - M-5 — проверка T8.3 в `check_batch` не атомарна со вставкой пакета: два одновременных повтора могут оба пройти (финальное ревью). - M-6 — `_line` в MCP вики схлопывает только `\r\n`; U+2028, U+0085, `\v`, `\f` остаются (финальное ревью). - M-7 — нет компонентного теста клика по нескачанной картинке (`start(id, "show")`); подмена `URL.createObjectURL` в `WikiPage.test.tsx` снимается не в `afterEach` (финальное ревью). - O-1 — `SyntaxWarning: invalid escape sequence '\w'` в `tests/test_api.py:1172` (докстринг), было до ветки (наблюдение контроллера).

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: aep/sdlc-hub#138