MOBILE-323: Add the SwiftUI embedded block - #756
Merged
Vailence merged 3 commits intoAug 12, 2026
Merged
Conversation
added 2 commits
August 10, 2026 17:48
Тот же блок в SwiftUI: создаётся с id и высотой, ставится куда угодно, а плейсхолдер и экран ошибки задаются модификаторами на самом блоке. Слои хоста рисует обёртка, а не контейнер. Вью, отданная контейнеру через отдельный UIHostingController, не входит в дерево SwiftUI и не видит его окружения: плейсхолдер с @EnvironmentObject просто падает, а заданные хостом шрифт, цвет и локаль до него не доезжают. Поэтому контейнер получает под каждый заявленный слой прозрачную заглушку — держать место, — а красит это место SwiftUI поверх. Отсюда и снимок показа: обёртке нужен не только размер, но и текущий слой. И id, и высоту контейнер получает при создании и потом не меняет, поэтому другое значение любого из них — это другой блок, который надо собрать заново. Без явной идентичности хост, подставивший другой id, продолжал бы видеть содержимое прежнего: SwiftUI переиспользовал бы уже созданный контейнер. Координатор переставляется на свежие замыкания на каждом проходе body, а заглушки ставятся и снимаются на каждом обновлении: модификатор мог быть применён по условию, поэтому слой может появиться после первого прохода — и точно так же исчезнуть.
Модификаторы, идентичность блока и мост между контейнером и обёрткой: заглушки под заявленные слои ставятся и снимаются по состоянию модификаторов, а показ и исход доезжают до обёртки. Идентичность проверяется отдельно, потому что на ней держится главное: другой id или другая высота — это другой блок, а не обновление текущего.
Contributor
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds a new SwiftUI API (MindboxEmbeddedBlock) that wraps the existing UIKit container (MindboxEmbeddedBlockView) to render embedded blocks inside SwiftUI layouts while keeping placeholder/error layers inside the SwiftUI view tree.
Changes:
- Introduces
MindboxEmbeddedBlockSwiftUI view with.placeholder {}and.errorView {}modifiers and identity management via.id(...). - Implements a
UIViewRepresentablebridge that syncs transparent stand-ins into the UIKit container and relays container presentation changes into SwiftUI state. - Adds a dedicated test suite covering modifier semantics, identity behavior, and stand-in syncing.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| Mindbox/EmbeddedBlocks/Public/MindboxEmbeddedBlock/MindboxEmbeddedBlock.swift | New SwiftUI wrapper + representable bridge, managing identity, presentation state, and stand-in layer wiring to the UIKit container. |
| MindboxTests/EmbeddedBlocks/MindboxEmbeddedBlockTests.swift | New unit tests validating wrapper modifier contract, identity stability, and correct container stand-in behavior. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Comment on lines
+52
to
+60
| public init(id: String, | ||
| height: CGFloat, | ||
| onLoad: (() -> Void)? = nil, | ||
| onFail: (() -> Void)? = nil) { | ||
| self.id = id | ||
| self.height = height | ||
| self.onLoad = onLoad | ||
| self.onFail = onFail | ||
| } |
justSmK
reviewed
Aug 11, 2026
justSmK
left a comment
justSmK
left a comment
Collaborator
There was a problem hiding this comment.
Обёртка аккуратная, отдельно понравилось, что слои хоста рисуются в дереве SwiftUI, а не отдаются контейнеру через UIHostingController — окружение бы потерялось.
Собрал все четыре ветки в один worktree (свелись чисто, конфликт только в project.pbxproj), собрал и прогнал тесты. С одной правкой (см. блокер в #755) сборка проходит: 1407 тестов, 0 падений.
Про тесты. 16 тестов покрывают контракт модификаторов и идентичность — для юнитов это правильный объём, рендер SwiftUI толком не проверишь. Но тогда кейсы «блок в List / LazyVStack / после поворота / после ухода за экран и обратно» должны быть в ручной приёмке. У Android они прописаны явно (кейсы 5–6, 9–10), в описании этого PR их нет.
Про порядок мержа стека. 754 и 755 не содержат двух последних коммитов фундамента (42b04e22, 250447cf), а 756 не содержит de08d916. Мержить надо строго по порядку с ребейзами.
Отдельно: 250447cf переформатировал project.pbxproj примерно на 150 строк. При сведении стека это гарантированный конфликт — я его получил, когда собирал ветки вместе локально. Лучше решить заранее, а не в момент мержа.
| /// этого хост, подставивший в блок другой id, продолжал бы видеть содержимое прежнего: SwiftUI | ||
| /// переиспользовал бы уже созданный контейнер. | ||
| var identity: Identity { | ||
| Identity(id: id, height: height) |
Collaborator
There was a problem hiding this comment.
🟡 Высота входит в идентичность и идёт в .id(…), поэтому её смена пересобирает контейнер и перезагружает вебвью с нуля. Хост, который считает высоту из GeometryReader или анимирует её, получит перезагрузку на каждое изменение.
Причина понятна: preferredHeight в контейнере — let. Правильное решение — сделать его var и обновлять в updateUIView, тогда высота перестанет быть частью идентичности, а id останется.
Если оставляем как есть — надо написать в доке жирно, что высота обязана быть константой.
| /// Пишется на следующем витке главной очереди: контейнер может доложить о смене слоя прямо | ||
| /// посреди прохода body, а менять состояние в этот момент нельзя. | ||
| func update(_ newPresentation: EmbeddedBlockPresentation) { | ||
| DispatchQueue.main.async { [weak self] in |
Collaborator
There was a problem hiding this comment.
🟡 dismantleUIView (218) обнуляет onPresentationChange, но уже поставленный в очередь блок всё равно выполнится и запишет в состояние снятой вью. [weak self] тут не спасает: координатор жив, его держит SwiftUI.
Нужен флаг «снят», который ставится в dismantleUIView и проверяется внутри async.
| private let onLoad: (() -> Void)? | ||
| private let onFail: (() -> Void)? | ||
|
|
||
| private(set) var placeholderBuilder: (() -> AnyView)? |
Collaborator
There was a problem hiding this comment.
🟢 AnyView стирает тип и мешает диффингу SwiftUI. Здесь это приемлемый размен — дженерики по двум слотам убили бы эргономику API. Менять не предлагаю, отмечаю осознанность выбора.
6 tasks
Base automatically changed from
feature/MOBILE-323-block-container
to
feature/MOBILE-323-block-foundation
August 12, 2026 11:01
…E-323-block-swiftui
Vailence
force-pushed
the
feature/MOBILE-323-block-swiftui
branch
from
August 12, 2026 11:10
0ac17f9 to
2d66e8f
Compare
Vailence
merged commit Aug 12, 2026
0a4b51a
into
feature/MOBILE-323-block-foundation
3 checks passed
Vailence
added a commit
that referenced
this pull request
Aug 12, 2026
…, actions (#751) * MOBILE-323: Add the embeddedBlocks log category Своя категория логов у встроенных блоков: их путь — резолв id, страница, бюджет ожидания — читается в логах отдельно от инаппов. Счётчик категорий в LogPrimitivesTests обновлён здесь же: это не новый тест, а утверждение, которое обязано ехать вместе с самим enum. * MOBILE-323: Add the embedded block web contract Словарь, на котором страница блока разговаривает с нативной стороной, и типы, которыми говорят между собой все остальные слои. Ядро разбирает только core-слой — ready, heightChanged и empty, они нужны любому блоку. Всё остальное с валидным конвертом уходит механике как action: ядро не знает и не должно знать словарь конкретной механики. Состояния контейнера намеренно internal: хост узнаёт только исход — показан блок или нет, — поэтому менять сам путь к исходу можно без изменения публичного API. Папка Mindbox/EmbeddedBlocks заведена синхронизируемой группой, поэтому файлы в ней дальше подхватываются без правок проекта. * MOBILE-323: Add the embedded block content resolver Отвечает на единственный вопрос: что показывает блок с данным id. Резолвер — общая точка всех контейнеров: несколько блоков с одним id разрешаются одними данными, за которыми ходим один раз. Кэш на id и очередь ожидающих — это и есть «одна загрузка на id»: второй блок встаёт в очередь, а не идёт за данными сам. forceRefresh нужен перезагрузке блока: переехавший или выключенный блок иначе вечно доставал бы из кэша прежний адрес. Пока конфига из админки нет, любой id разрешается в статическую страницу ленты сторизов. Это единственное место, которое заменит настоящий конфиг: кэш и очередь при этом не изменятся. Рядом — подмена содержимого по id: она встаёт ровно на место будущего конфига и сильнее кэша, потому что приёмка переключает сценарий на ходу. * MOBILE-323: Add the embedded block page in WKWebView Страница блока и шов, за которым живёт весь WebKit: перевод её сообщений в состояния блока дальше проверяется без реального вебвью и без сети. Вебвью берётся из InAppWebViewFactory — того же места, где настраиваются вебвью инаппов, — поэтому блок получает тот же user agent, тот же WKWebsiteDataStore и общий с инаппами HTTP-кеш. Навигация судит исключительно о своём: загрузка не состоялась или документ доехал. Готовность блока из этого не следует — о ней говорит сама страница своим ready. Отменённая навигация при этом провалом не считается: WebKit отдаёт NSURLErrorCancelled и когда навигацию вытеснил клиентский редирект, и когда её остановил наш собственный cancel() на уехавшем с экрана блоке. Выдать это за провал значит свернуть исправный блок насовсем. Мост живёт столько же, сколько страница, и держится слабым прокси: WKUserContent- Controller держит обработчик сильно, иначе страница и вебвью не освободятся. Отладочная подмена готовности — для страниц, которые веб-контракт ещё не умеют. * MOBILE-323: Add the embedded block page action router Универсальный словарь действий страницы — один на все механики. Блок не знает, какая механика внутри, поэтому любая страница, говорящая этим словарём, получает нативное поведение без нового кода в SDK. Незнакомое действие — не ошибка: словарь у веб-стороны может быть новее, чем у SDK. openUrl открывает не что попало. Страница блока приезжает из сети, поэтому решать за пользователя, что откроет система, ей не положено: tel:, sms:, itms-apps: и схемы чужих приложений — это уже не переход по контенту, а действие от его имени, и canOpenURL для них проходит. Разрешено то, что никуда пользователя не увозит: веб-адреса и диплинки в само это приложение из CFBundleURLTypes. Открытие вынесено за шов EmbeddedBlockURLOpening — и чтобы политику можно было проверить тестами, и на будущее: в SDK открытие ссылок уже живёт в MindboxURLHandlerDelegate, и когда блоки поедут на общий мост инаппов, здесь окажется он, а не UIApplication напрямую. * MOBILE-323: Add the debug content override SPI Отладочное управление содержимым блоков — для тестового приложения и приёмки. Подменяет ответ на вопрос «что стоит за этим id», то есть встаёт ровно на место конфига из админки. Всё ниже — резолвер, страница, а дальше провайдер и бюджет ожидания у контейнера — работает без изменений, поэтому приёмка проверяет боевой путь, а не отдельный тестовый режим. Разметкой задаются сценарии, которых в сети нет: страница, сообщающая «пусто», молчащая страница, страница с ответом после таймаута. Не часть публичного API: доступно только через @_spi(Internal) import Mindbox. Из релизных сборок не вырезано намеренно — QA проверяет ровно то, что уходит клиентам, — поэтому каждая установка подмены пишется в лог. * MOBILE-323: Add test infrastructure for embedded blocks Тег для сьютов блоков, папка MindboxTests/EmbeddedBlocks синхронизируемой группой и заготовка моков. В моках пока только то, что нужно тестам этой части: адрес страницы-заготовка и открыватель ссылок, который ничего не открывает, — политику схем иначе не проверить, canOpenURL в тестовом окружении пропускает системные схемы. * MOBILE-323: Add tests for the embedded block content resolver Резолвер проверяется главным своим обещанием: сколько блоков ни спросило бы про один id, за данными идём один раз. Пока конфиг синхронный это незаметно, с сетью это разница между одним запросом и N. Рядом — кэш, обход кэша перезагрузкой и то, что отладочная подмена сильнее и данных, и кэша: приёмка переключает сценарий на ходу. * MOBILE-323: Add tests for the embedded block page Страница судит только о своём, и проверяется именно это разделение: настоящая ошибка сети — провал, а отменённая навигация — нет. Отмена при этом страницу не глушит: следующая настоящая ошибка приходит как обычно. * MOBILE-323: Add tests for the page action router Обе ветки handle и вся политика схем: веб-адреса и диплинки хоста открываются, tel:, sms:, itms-apps:, mailto: и схемы чужих приложений — нет, причём отказ не отменяется тем, что система такую ссылку умеет открыть. Плюс кривые payload'ы, незнакомое действие и разбор CFBundleURLTypes. * MOBILE-323: Add the embedded block content provider Переводит сообщения страницы в состояния блока. Не рисует контент и не знает механик: спрашивает у резолвера, что стоит за id, разбирает core-слой, а действия сверх него отдаёт универсальному обработчику. Готовность определяет только сама страница: ready — показываем, empty — показывать нечего. Навигация судит исключительно о своём, молчащая страница готовой не становится — её добьёт бюджет ожидания у контейнера. Нулевая высота в ready значит сломанную вёрстку: «показывать нечего» страница сообщает явным empty. Исход попытки хранится явно, а не пачкой флагов. Он переживает stop(), потому что это свойство страницы, а не факта нахождения в окне: уход блока с экрана не выбрасывает уже отрендеренную страницу, и возврат показывает её снова, без сети и без шиммера. При этом провал и empty страницу не убивают — она жива и может продолжать говорить, — поэтому известный исход служит и признаком того, что блока на экране больше нет: действия от невидимого блока не выполняются. За ним не стоит ни одного касания пользователя, а openUrl увёл бы человека из приложения на пустом месте. Экземпляр принадлежит одному контейнеру: ничего общего между контейнерами здесь нет, это и делает возможными несколько независимых блоков с одним id. Номер попытки отсекает резолв, доехавший уже после остановки или перезагрузки. Счётчик живых блоков на id — диагностика, а не механика: два блока с одним id законны, но чаще это скопированный id или переиспользованная ячейка, а у обоих случаев нет симптомов кроме «блок оказался не там, где ждали». * MOBILE-323: Add the content provider factory Собирает провайдер под конкретный блок: резолвер и обработчик действий общие на все блоки, а провайдер — свой на каждый. Это и делает блоки с одинаковым id независимыми друг от друга. Потребителей у фабрики появится два, и оба в следующей части: DI-регистрация и публичный init контейнера. Здесь она едет вместе с провайдером, потому что описывает его модель владения, а не способ его достать. * MOBILE-323: Add test doubles for the block content provider Дописывает моки до среза этой части: страница без WebKit, фабрика страниц со счётчиком созданных, резолвер с отложенным ответом, обработчик действий и общая заготовка провайдера со всеми подменёнными зависимостями. Резолвер умеет держать ответ до отдельной команды: так проверяется резолв, доехавший уже после остановки или перезагрузки блока. Фабрика считает страницы, потому что перезагрузка обязана создать новую, а возврат блока в окно — нет. * MOBILE-323: Add tests for the embedded block content provider Весь путь блока без WebKit и без сети: резолв, показ по ready, пустой блок, сломанная нулевая высота, провал загрузки, действия страницы, отладочная подмена готовности, остановка и перезапуск, перезагрузка. Отдельно закреплено то, на что опираются соседние слои: после stop() провайдер молчит целиком — иначе контейнер не смог бы свернуть просроченный блок; уже отрендеренная страница на возврате в окно показывается как есть, без сети и шиммера, а блок, который показать не удалось, получает новую попытку; выброшенная перезагрузкой страница не может доложить в новую попытку ни сообщением, ни провалом, ни через подмену готовности. Действия проверяются с обеих сторон: из показанного блока доходят до обработчика, из схлопнутого — нет, ни после empty, ни после провала, ни после нулевой высоты, — а новая попытка снова их принимает. Счётчику живых блоков в каждом тесте свой id: он общий на процесс, иначе тесты, идущие параллельно, считали бы блоки друг друга. * MOBILE-323: Add tests for the content provider factory Фабрика приехала без тестов, а её обещание — то, на чём держится независимость блоков с одинаковым id: провайдер свой на каждый блок, резолвер общий на все. Проверяется и то и другое, плюс что провайдер собран под запрошенный id. Резолвер в тестах отвечает «пусто»: страницу для такого блока не создают, поэтому настоящий вебвью фабрике здесь не нужен. * MOBILE-323: Trim duplicated comments in the content provider Док класса пересказывал то, что уже сказано ниже по файлу: правила готовности — у apply(height:) и handleLoadFinish, владение высотой — у heightChanged, а мысль «уход из окна не выбрасывает страницу» шла трижды — в доке класса, в доке свойства page и во встроенном комментарии в start(). Осталось два абзаца: чем провайдер является и что после stop() он обязан молчать — это межтиповой инвариант, из одного файла его не видно. Убраны и три дока, ушедшие из своей ответственности или пересказывавшие подпись: contentView рассказывал, как контейнер растягивает вью; второй абзац reload() — про плейсхолдер и события хосту; isShown повторял собственное имя. Встроенные комментарии не тронуты: каждый объясняет отсутствующую строку, развилку или внешнюю причину — то, чего в коде не прочитать. * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * MOBILE-323: Add the block layer host Держит в контейнере ровно одну вью, растянутую по его краям. Слои блока — плейсхолдер, контент, экран ошибки — взаимоисключающие: показать новый значит снять прежний. Показанная вью запоминается отдельно от свойств контейнера, потому что подменить её хост может в любой момент, а снимать надо ту, что действительно висит, а не ту, что лежит в свойстве сейчас. * MOBILE-323: Add the block ready timeout Бюджет на то, чтобы показаться, принадлежит контейнеру, а не контенту: чем бы блок ни оказался внутри, вёрстка хоста не ждёт его вечно. Считается время ожидания пользователя, а не календарное: пока блока никто не ждёт — приложение в фоне, контейнер вне окна — отсчёт стоит. Но именно стоит, а не начинается заново: потраченное запоминается, и попытка продолжает бюджет с того места, где её прервали. Пауза, отдающая полный бюджет заново, не заканчивается никогда — пользователь, переключающийся между приложениями каждые пять секунд, продлевал бы ожидание блока бесконечно, и вёрстка ждала бы его вечно. Ровно то, против чего бюджет и заведён. Полный бюджет получает только новая попытка. Часы вынесены отдельным швом: считать потраченное без них нельзя, а тесты не могут ждать бюджет целиком — им нужно уметь сказать, что время прошло. Загрузку пауза не трогает: она идёт своим чередом, в фоне её тормозит система, а не SDK. * MOBILE-323: Add the default loading shimmer Плейсхолдер, который блок показывает, пока грузится, если хост не дал своего. Занимает весь контейнер: место под блок должно быть занято сразу, а занятое место не должно выглядеть пустым. * MOBILE-323: Add the UIKit embedded block container Публичный API: контейнер создаётся с id блока из админки и высотой, которую блок должен занять. Хост ставит его куда угодно и задаёт только положение и ширину — высоту контейнер заявляет сам через intrinsicContentSize: данную при создании, пока контент грузится и показан, и 0, когда показывать нечего. Оба исхода настраиваются: placeholderView заменяет штатный шиммер, errorView — это согласие показать провал вместо схлопывания. Жизненным циклом владеет SDK: контент стартует, когда блок попадает в окно, и останавливается, когда уходит. Публичного способа запустить его руками нет. Внутри — машина из четырёх состояний контента и одного видимого слоя на каждое. Три решения, которые стоят за ней: Место, единожды отданное хосту, назад не забирается. Блок, который показать не удалось, на возврате в окно пробует снова, но контейнер под эту попытку места уже не занимает и шиммером не мигает — иначе он дёргал бы вёрстку на свою высоту и мигал на каждый свой проход по экрану, ничего в итоге не показывая. Разворачивает блок только показанный контент или явная перезагрузка. Тот же делегат — не новый подписчик. Хост штатно переприсваивает его на каждой переиспользованной ячейке, и отдавать ему на это уже услышанный исход нельзя: на исход он перестраивает вёрстку, а перестройка вёрстки снова переприсваивает делегата. Исходы отдаются на следующем витке главной очереди: состояние может измениться посреди прохода layout, и заходить оттуда в код хоста — верный способ сломать его вёрстку. EmbeddedBlockPresentation — снимок показа для SwiftUI-обёртки: она сама назначает себе высоту и сама рисует слои хоста, потому что вью, отданная контейнеру через отдельный UIHostingController, выпадает из дерева SwiftUI и теряет его окружение. * MOBILE-323: Register embedded blocks in DI Резолвер общий: его кэш на id и очередь ожидающих — это и есть «одна загрузка данных на id» для всех контейнеров сразу. Обработчик действий общий потому, что не имеет состояния. Провайдеры, наоборот, фабрика делает на каждый блок свой — так блоки остаются независимыми друг от друга. Файл прописан в проект вручную: Mindbox/DI/Injections — обычная группа, а не синхронизируемая, в отличие от папок самих блоков. * MOBILE-323: Add tests for the block layer host Хост слоёв приехал без прямого покрытия: контейнер задевает его косвенно, но собственное обещание — ровно одна вью, растянутая по краям, — проверить надо отдельно. Кроме подмены и снятия закреплены две тонкости: повторный показ той же вью констрейнты не пересобирает (контейнер зовёт show на каждую смену состояния), а вью, снятую снаружи, показ обязан вернуть на место. * MOBILE-323: Add tests for the block ready timeout Сколько бюджета «уже потрачено», тесты задают подменёнными часами, а ждут только тот огрызок, который остался. Иначе проверка «продолжается остаток, а не выдаётся полный бюджет» сводилась бы к измерению задержек секундомером. Главный тест — пять пауз по четверти бюджета не растягивают его сверх срока: это ровно тот сценарий, из-за которого пауза со сбросом непригодна. Уход в фон и возврат здесь не проверяются: это глобальные нотификации, они долетят до блоков из тестов, идущих рядом. Провод от них к паузе проверяет контейнер, у которого блок один. * MOBILE-323: Add tests for the UIKit embedded block container Высота, слои, события хосту и жизненный цикл по окну — всё через настоящий провайдер: единственный шов внутри блока это страница, и подменять больше нечего. Закреплены три решения, из-за которых контейнер и выглядит так: Блок, который показать не удалось, на возврате в окно пробует снова, но остаётся свёрнутым и шиммером не мигает; разворачивает его только показанный контент или явная перезагрузка. Пустой блок ведёт себя так же. Тот же делегат, присвоенный повторно, уже услышанный исход не получает — иначе хост, перестраивающий вёрстку в onFail, укатился бы в цикл на скролле. А другой делегат исход обязан услышать, даже если тот случился до подписки. Бюджет ожидания в фоне встаёт и на возврате продолжается: пользователь не должен возвращаться в приложение к блоку, который сдался, ни разу не побывав на экране. Моки дописаны последним нужным дублёром — делегатом со списком услышанных событий. * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * MOBILE-323 Less comments * MOBILE-323: Translate embedded block comments to English Also carries the pending working-tree changes: the URL opener is dropped from the action router until the in-app web view route logic is reused, resolver answers are delivered on the main thread, and the page no longer detaches its bridge in deinit. * MOBILE-323: Fix the broken applyColors signature in the shimmer A stray text fragment had been committed into the declaration line of applyColors(), leaving the Mindbox target uncompilable since de08d91. Every commit after it, tests included, was stacked on code that never built. Restores the signature and keeps the deinit that the same change added. * MOBILE-323: Split the nothing-to-show branch out of the layer host Showing nothing was falling through the same guard as showing a view, where the superview comparison against a nil view is always true and the condition therefore read as the opposite of what it meant. Behaviour is unchanged; adds the missing test for showing nothing on a host that already shows nothing. * MOBILE-323: Make the ready timeout testable and report a zero height The timeout kept its notification centre and its scheduler hardwired, so the pause in the background was untestable and every budget test had to wait real time. Both become injectable, as the clock already was, and the container takes the whole budget instead of its duration. Drops all 20 sleeps from the embedded block tests, which now declare that time is up and assert the delay a countdown was armed with. Adds the background and foreground cases the consumed accounting exists for. A block created with a height of zero or less also reserves no space and stays invisible while still reporting its outcome, which is the likeliest integration mistake and now goes to the log as an error. * Update Mindbox/EmbeddedBlocks/Resolver/EmbeddedBlockResolver.swift Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com> * MOBILE-323 PR Fix * MOBILE-323: Embedded block content provider (#754) * MOBILE-323: Add the embedded block content provider Переводит сообщения страницы в состояния блока. Не рисует контент и не знает механик: спрашивает у резолвера, что стоит за id, разбирает core-слой, а действия сверх него отдаёт универсальному обработчику. Готовность определяет только сама страница: ready — показываем, empty — показывать нечего. Навигация судит исключительно о своём, молчащая страница готовой не становится — её добьёт бюджет ожидания у контейнера. Нулевая высота в ready значит сломанную вёрстку: «показывать нечего» страница сообщает явным empty. Исход попытки хранится явно, а не пачкой флагов. Он переживает stop(), потому что это свойство страницы, а не факта нахождения в окне: уход блока с экрана не выбрасывает уже отрендеренную страницу, и возврат показывает её снова, без сети и без шиммера. При этом провал и empty страницу не убивают — она жива и может продолжать говорить, — поэтому известный исход служит и признаком того, что блока на экране больше нет: действия от невидимого блока не выполняются. За ним не стоит ни одного касания пользователя, а openUrl увёл бы человека из приложения на пустом месте. Экземпляр принадлежит одному контейнеру: ничего общего между контейнерами здесь нет, это и делает возможными несколько независимых блоков с одним id. Номер попытки отсекает резолв, доехавший уже после остановки или перезагрузки. Счётчик живых блоков на id — диагностика, а не механика: два блока с одним id законны, но чаще это скопированный id или переиспользованная ячейка, а у обоих случаев нет симптомов кроме «блок оказался не там, где ждали». * MOBILE-323: Add the content provider factory Собирает провайдер под конкретный блок: резолвер и обработчик действий общие на все блоки, а провайдер — свой на каждый. Это и делает блоки с одинаковым id независимыми друг от друга. Потребителей у фабрики появится два, и оба в следующей части: DI-регистрация и публичный init контейнера. Здесь она едет вместе с провайдером, потому что описывает его модель владения, а не способ его достать. * MOBILE-323: Add test doubles for the block content provider Дописывает моки до среза этой части: страница без WebKit, фабрика страниц со счётчиком созданных, резолвер с отложенным ответом, обработчик действий и общая заготовка провайдера со всеми подменёнными зависимостями. Резолвер умеет держать ответ до отдельной команды: так проверяется резолв, доехавший уже после остановки или перезагрузки блока. Фабрика считает страницы, потому что перезагрузка обязана создать новую, а возврат блока в окно — нет. * MOBILE-323: Add tests for the embedded block content provider Весь путь блока без WebKit и без сети: резолв, показ по ready, пустой блок, сломанная нулевая высота, провал загрузки, действия страницы, отладочная подмена готовности, остановка и перезапуск, перезагрузка. Отдельно закреплено то, на что опираются соседние слои: после stop() провайдер молчит целиком — иначе контейнер не смог бы свернуть просроченный блок; уже отрендеренная страница на возврате в окно показывается как есть, без сети и шиммера, а блок, который показать не удалось, получает новую попытку; выброшенная перезагрузкой страница не может доложить в новую попытку ни сообщением, ни провалом, ни через подмену готовности. Действия проверяются с обеих сторон: из показанного блока доходят до обработчика, из схлопнутого — нет, ни после empty, ни после провала, ни после нулевой высоты, — а новая попытка снова их принимает. Счётчику живых блоков в каждом тесте свой id: он общий на процесс, иначе тесты, идущие параллельно, считали бы блоки друг друга. * MOBILE-323: Add tests for the content provider factory Фабрика приехала без тестов, а её обещание — то, на чём держится независимость блоков с одинаковым id: провайдер свой на каждый блок, резолвер общий на все. Проверяется и то и другое, плюс что провайдер собран под запрошенный id. Резолвер в тестах отвечает «пусто»: страницу для такого блока не создают, поэтому настоящий вебвью фабрике здесь не нужен. * MOBILE-323: Trim duplicated comments in the content provider Док класса пересказывал то, что уже сказано ниже по файлу: правила готовности — у apply(height:) и handleLoadFinish, владение высотой — у heightChanged, а мысль «уход из окна не выбрасывает страницу» шла трижды — в доке класса, в доке свойства page и во встроенном комментарии в start(). Осталось два абзаца: чем провайдер является и что после stop() он обязан молчать — это межтиповой инвариант, из одного файла его не видно. Убраны и три дока, ушедшие из своей ответственности или пересказывавшие подпись: contentView рассказывал, как контейнер растягивает вью; второй абзац reload() — про плейсхолдер и события хосту; isShown повторял собственное имя. Встроенные комментарии не тронуты: каждый объясняет отсутствующую строку, развилку или внешнюю причину — то, чего в коде не прочитать. * MOBILE-323 PR Fix * MOBILE-323 Remove unused tests --------- Co-authored-by: Vailence <utekeshev@mindbox.cloud> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * MOBILE-323 Less comments * MOBILE-323: Translate embedded block comments to English Also carries the pending working-tree changes: the URL opener is dropped from the action router until the in-app web view route logic is reused, resolver answers are delivered on the main thread, and the page no longer detaches its bridge in deinit. * Update Mindbox/EmbeddedBlocks/Resolver/EmbeddedBlockResolver.swift Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com> * MOBILE-323 PR Fix * MOBILE-323: Embedded block content provider (#754) * MOBILE-323: Add the embedded block content provider Переводит сообщения страницы в состояния блока. Не рисует контент и не знает механик: спрашивает у резолвера, что стоит за id, разбирает core-слой, а действия сверх него отдаёт универсальному обработчику. Готовность определяет только сама страница: ready — показываем, empty — показывать нечего. Навигация судит исключительно о своём, молчащая страница готовой не становится — её добьёт бюджет ожидания у контейнера. Нулевая высота в ready значит сломанную вёрстку: «показывать нечего» страница сообщает явным empty. Исход попытки хранится явно, а не пачкой флагов. Он переживает stop(), потому что это свойство страницы, а не факта нахождения в окне: уход блока с экрана не выбрасывает уже отрендеренную страницу, и возврат показывает её снова, без сети и без шиммера. При этом провал и empty страницу не убивают — она жива и может продолжать говорить, — поэтому известный исход служит и признаком того, что блока на экране больше нет: действия от невидимого блока не выполняются. За ним не стоит ни одного касания пользователя, а openUrl увёл бы человека из приложения на пустом месте. Экземпляр принадлежит одному контейнеру: ничего общего между контейнерами здесь нет, это и делает возможными несколько независимых блоков с одним id. Номер попытки отсекает резолв, доехавший уже после остановки или перезагрузки. Счётчик живых блоков на id — диагностика, а не механика: два блока с одним id законны, но чаще это скопированный id или переиспользованная ячейка, а у обоих случаев нет симптомов кроме «блок оказался не там, где ждали». * MOBILE-323: Add the content provider factory Собирает провайдер под конкретный блок: резолвер и обработчик действий общие на все блоки, а провайдер — свой на каждый. Это и делает блоки с одинаковым id независимыми друг от друга. Потребителей у фабрики появится два, и оба в следующей части: DI-регистрация и публичный init контейнера. Здесь она едет вместе с провайдером, потому что описывает его модель владения, а не способ его достать. * MOBILE-323: Add test doubles for the block content provider Дописывает моки до среза этой части: страница без WebKit, фабрика страниц со счётчиком созданных, резолвер с отложенным ответом, обработчик действий и общая заготовка провайдера со всеми подменёнными зависимостями. Резолвер умеет держать ответ до отдельной команды: так проверяется резолв, доехавший уже после остановки или перезагрузки блока. Фабрика считает страницы, потому что перезагрузка обязана создать новую, а возврат блока в окно — нет. * MOBILE-323: Add tests for the embedded block content provider Весь путь блока без WebKit и без сети: резолв, показ по ready, пустой блок, сломанная нулевая высота, провал загрузки, действия страницы, отладочная подмена готовности, остановка и перезапуск, перезагрузка. Отдельно закреплено то, на что опираются соседние слои: после stop() провайдер молчит целиком — иначе контейнер не смог бы свернуть просроченный блок; уже отрендеренная страница на возврате в окно показывается как есть, без сети и шиммера, а блок, который показать не удалось, получает новую попытку; выброшенная перезагрузкой страница не может доложить в новую попытку ни сообщением, ни провалом, ни через подмену готовности. Действия проверяются с обеих сторон: из показанного блока доходят до обработчика, из схлопнутого — нет, ни после empty, ни после провала, ни после нулевой высоты, — а новая попытка снова их принимает. Счётчику живых блоков в каждом тесте свой id: он общий на процесс, иначе тесты, идущие параллельно, считали бы блоки друг друга. * MOBILE-323: Add tests for the content provider factory Фабрика приехала без тестов, а её обещание — то, на чём держится независимость блоков с одинаковым id: провайдер свой на каждый блок, резолвер общий на все. Проверяется и то и другое, плюс что провайдер собран под запрошенный id. Резолвер в тестах отвечает «пусто»: страницу для такого блока не создают, поэтому настоящий вебвью фабрике здесь не нужен. * MOBILE-323: Trim duplicated comments in the content provider Док класса пересказывал то, что уже сказано ниже по файлу: правила готовности — у apply(height:) и handleLoadFinish, владение высотой — у heightChanged, а мысль «уход из окна не выбрасывает страницу» шла трижды — в доке класса, в доке свойства page и во встроенном комментарии в start(). Осталось два абзаца: чем провайдер является и что после stop() он обязан молчать — это межтиповой инвариант, из одного файла его не видно. Убраны и три дока, ушедшие из своей ответственности или пересказывавшие подпись: contentView рассказывал, как контейнер растягивает вью; второй абзац reload() — про плейсхолдер и события хосту; isShown повторял собственное имя. Встроенные комментарии не тронуты: каждый объясняет отсутствующую строку, развилку или внешнюю причину — то, чего в коде не прочитать. * MOBILE-323 PR Fix * MOBILE-323 Remove unused tests --------- Co-authored-by: Vailence <utekeshev@mindbox.cloud> * MOBILE-323 Unused tests removed * MOBILE-323: Add the SwiftUI embedded block (#756) * MOBILE-323: Add the SwiftUI embedded block Тот же блок в SwiftUI: создаётся с id и высотой, ставится куда угодно, а плейсхолдер и экран ошибки задаются модификаторами на самом блоке. Слои хоста рисует обёртка, а не контейнер. Вью, отданная контейнеру через отдельный UIHostingController, не входит в дерево SwiftUI и не видит его окружения: плейсхолдер с @EnvironmentObject просто падает, а заданные хостом шрифт, цвет и локаль до него не доезжают. Поэтому контейнер получает под каждый заявленный слой прозрачную заглушку — держать место, — а красит это место SwiftUI поверх. Отсюда и снимок показа: обёртке нужен не только размер, но и текущий слой. И id, и высоту контейнер получает при создании и потом не меняет, поэтому другое значение любого из них — это другой блок, который надо собрать заново. Без явной идентичности хост, подставивший другой id, продолжал бы видеть содержимое прежнего: SwiftUI переиспользовал бы уже созданный контейнер. Координатор переставляется на свежие замыкания на каждом проходе body, а заглушки ставятся и снимаются на каждом обновлении: модификатор мог быть применён по условию, поэтому слой может появиться после первого прохода — и точно так же исчезнуть. * MOBILE-323: Add tests for the SwiftUI embedded block Модификаторы, идентичность блока и мост между контейнером и обёрткой: заглушки под заявленные слои ставятся и снимаются по состоянию модификаторов, а показ и исход доезжают до обёртки. Идентичность проверяется отдельно, потому что на ней держится главное: другой id или другая высота — это другой блок, а не обновление текущего. --------- Co-authored-by: Vailence <utekeshev@mindbox.cloud> * MOBILE-323 Remove Identity * MOBILE-323 isDetached * MOBILE-323: Translate embedded block comments and restore test coverage Translate all remaining Russian comments in EmbeddedBlocks sources and tests to English. Fix the ready timeout's clock to be monotonic so an NTP correction or manual clock change cannot shrink or negate spent budget. Restore the previously deleted container, provider, factory and SwiftUI wrapper test suites, adapted to the current identity and timeout seams. * Update Mindbox/EmbeddedBlocks/Public/MindboxEmbeddedBlock/MindboxEmbeddedBlock.swift Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com> * Update Mindbox/EmbeddedBlocks/Public/MindboxEmbeddedBlock/MindboxEmbeddedBlock.swift Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com> --------- Co-authored-by: Vailence <utekeshev@mindbox.cloud> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
MOBILE-323
Adds
MindboxEmbeddedBlock— the SwiftUI wrapper overMindboxEmbeddedBlockView. Stacked on top of #755 (the UIKit container).Public API
MindboxEmbeddedBlock(id:height:onLoad:onFail:)— aViewcreated with the block id from the admin panel and the height the block occupies while loading and shown; an empty block collapses to zero height..placeholder { }— replaces the stock loading shimmer..errorView { }— opts into showing a failure instead of collapsing (applies only to failures; an empty block always collapses).Both modifiers return
Self, so they come before any SwiftUI modifier.Internals
ZStackover the container, not handed to it viaUIHostingController— a hosting controller is outside the SwiftUI tree, so a placeholder with@EnvironmentObjectwould crash and the host's font, color and locale would not reach it. The container instead gets a transparent stand-in per declared layer to hold the space.bodypass..id(Identity(id:height:))— both values are fixed at creation, so a different value of either is a different block that must be rebuilt rather than an update of the current one; without it a host that swapped the id would keep seeing the previous block's content.presentationon the next main-queue turn — the container can report a layer change mid-body, and state cannot be mutated at that point.Type of Change
Test Procedure
MindboxEmbeddedBlockTests— 16 tests over the wrapper:Pre-flight Checklist
MOBILE-323: DescriptionformatMindboxEmbeddedBlock