git_admin/obsidian-vault
1
0 Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

[2026-09-28] eagle: personal/plans/balda-mm-pr-plan.md personal/plans/balda-mm-pr8-review-fixes.md

Browse Source
This commit is contained in:
Alexey Martemyanov committed 2026-09-28 13:47:01 +06:00
1 parent d6c3ef4ffa
commit 02bff81fa8
2 files changed
+13 -6
Download Patch File Download Diff File Expand all files Collapse all files

No files matched your search

personal/plans/balda-mm-pr-plan.md
+6 -2
View File
@@ -3,6 +3,10 @@
> Обновлён 2026-09-28. Обновлён 2026-09-28 (ревизия: убраны дубли, оставлены факты). > Обновлён 2026-09-28. Обновлён 2026-09-28 (ревизия: убраны дубли, оставлены факты).
> Статус: **PR ОТКРЫТ — baldaworks/balda#8 · OPEN · MERGEABLE · 8/8 CI PASS · ревью апстрима НЕТ.** > Статус: **PR ОТКРЫТ — baldaworks/balda#8 · OPEN · MERGEABLE · 8/8 CI PASS · ревью апстрима НЕТ.**
> https://github.com/baldaworks/balda/pull/8 > https://github.com/baldaworks/balda/pull/8
> 🔧 **2026-09-28: правки по ревью Alex применены — коммит `cd4ce4bc`, CI 8/8 PASS.**
> Закрыты: slash-команды (HTTP-приёмник), `/locator` (`locatorref`), групповой DM (сессия
> по чату) + **регистрация в `ScopeResolver`, которой не было в PR**. Разбор —
> [[personal/plans/balda-mm-pr8-review-fixes]].
> Часть [[personal/projects/balda/balda-mm-delegation-plan]] (Фаза 2 → PR в апстрим). > Часть [[personal/projects/balda/balda-mm-delegation-plan]] (Фаза 2 → PR в апстрим).
--- ---
@@ -14,8 +18,8 @@
| PR | **baldaworks/balda#8** — `feat(mattermost): add Mattermost channel transport` | | PR | **baldaworks/balda#8** — `feat(mattermost): add Mattermost channel transport` |
| Состояние | `OPEN` · `MERGEABLE` · `reviewDecision` **пустой** | | Состояние | `OPEN` · `MERGEABLE` · `reviewDecision` **пустой** |
| База / голова | `baldaworks/balda:main` ← `mallexxx:feat/mattermost-channel` | | База / голова | `baldaworks/balda:main` ← `mallexxx:feat/mattermost-channel` |
| Объём | **41 файл, +5622 / −18**, 7 коммитов | | Объём | **46 файлов, +6760 / −27**, 8 коммитов |
| CI | **8/8 PASS** | | CI | **8/8 PASS** (на `cd4ce4bc`) |
``` ```
architecture-lint pass 2m2s architecture-lint pass 2m2s
personal/plans/balda-mm-pr8-review-fixes.md
+7 -4
View File
@@ -1,8 +1,11 @@
# Balda — PR #8 review fixes: slash commands, /locator, group DM session # Balda — PR #8 review fixes: slash commands, /locator, group DM session
> Создан 2026-09-28. Статус: **✅ РЕАЛИЗОВАНО, ЗАПУШЕНО, CI ИДЁТ.** > Создан 2026-09-28. Статус: **✅ РЕАЛИЗОВАНО, ЗАПУШЕНО, CI 8/8 PASS.**
> Коммит **`cd4ce4bc`** в `feat/mattermost-channel` → PR **baldaworks/balda#8** обновлён. > Коммит **`cd4ce4bc`** в `feat/mattermost-channel` → PR **baldaworks/balda#8** обновлён.
> 46 файлов, +6760/−27, база `0 позади / 8 впереди`. > 46 файлов, +6760/−27, база `0 позади / 8 впереди`. MERGEABLE.
> **CI 8/8 PASS:** architecture-lint 2m12s · ingress-e2e 33s · jetstream-integration 1m1s ·
> lint 2m45s · postgres-integration 1m7s · projection-replay 24s · sqlite-integration 1m21s ·
> test 4m6s.
> Источник: ревью Alex на PR #8 + **четвёртый дефект, найденный мной** (§5). > Источник: ревью Alex на PR #8 + **четвёртый дефект, найденный мной** (§5).
> Часть [[personal/plans/balda-mm-pr-plan]] (PR #8). > Часть [[personal/plans/balda-mm-pr-plan]] (PR #8).
@@ -220,9 +223,9 @@ warn … "no session-memory classifier is registered for \"mattermost\""
| # | Шаг | Статус | | # | Шаг | Статус |
|---|---|---| |---|---|---|
| 1 | Дождаться CI на `cd4ce4bc` (8 джобов) | ⏳ идёт | | 1 | CI на `cd4ce4bc` (8 джобов) | ✅ **8/8 PASS** |
| 2 | Внешнее ревью PR #8 (`reviewDecision` пустой) | ⛔ внешний блокер | | 2 | Внешнее ревью PR #8 (`reviewDecision` пустой) | ⛔ внешний блокер |
| 3 | Живая приёмка slash на Kraken (публичный HTTPS для Request URL!) | ⏳ после 1 | | 3 | Живая приёмка slash на Kraken (публичный HTTPS для Request URL!) | ⏳ следующий шаг |
| 4 | Фаза 4 — исполнитель (делегирование) | 🎯 основной фронт | | 4 | Фаза 4 — исполнитель (делегирование) | 🎯 основной фронт |
⚠️ **Б1 требует публичного HTTPS** для Request URL slash-команды. Стенд Kraken — ⚠️ **Б1 требует публичного HTTPS** для Request URL slash-команды. Стенд Kraken —