MOBILE-323: Embedded blocks foundation — web contract, resolver, page, actions - #751
Merged
Merged
Conversation
added 10 commits
August 10, 2026 16:33
Своя категория логов у встроенных блоков: их путь — резолв id, страница, бюджет ожидания — читается в логах отдельно от инаппов. Счётчик категорий в LogPrimitivesTests обновлён здесь же: это не новый тест, а утверждение, которое обязано ехать вместе с самим enum.
Словарь, на котором страница блока разговаривает с нативной стороной, и типы, которыми говорят между собой все остальные слои. Ядро разбирает только core-слой — ready, heightChanged и empty, они нужны любому блоку. Всё остальное с валидным конвертом уходит механике как action: ядро не знает и не должно знать словарь конкретной механики. Состояния контейнера намеренно internal: хост узнаёт только исход — показан блок или нет, — поэтому менять сам путь к исходу можно без изменения публичного API. Папка Mindbox/EmbeddedBlocks заведена синхронизируемой группой, поэтому файлы в ней дальше подхватываются без правок проекта.
Отвечает на единственный вопрос: что показывает блок с данным id. Резолвер — общая точка всех контейнеров: несколько блоков с одним id разрешаются одними данными, за которыми ходим один раз. Кэш на id и очередь ожидающих — это и есть «одна загрузка на id»: второй блок встаёт в очередь, а не идёт за данными сам. forceRefresh нужен перезагрузке блока: переехавший или выключенный блок иначе вечно доставал бы из кэша прежний адрес. Пока конфига из админки нет, любой id разрешается в статическую страницу ленты сторизов. Это единственное место, которое заменит настоящий конфиг: кэш и очередь при этом не изменятся. Рядом — подмена содержимого по id: она встаёт ровно на место будущего конфига и сильнее кэша, потому что приёмка переключает сценарий на ходу.
Страница блока и шов, за которым живёт весь WebKit: перевод её сообщений в состояния блока дальше проверяется без реального вебвью и без сети. Вебвью берётся из InAppWebViewFactory — того же места, где настраиваются вебвью инаппов, — поэтому блок получает тот же user agent, тот же WKWebsiteDataStore и общий с инаппами HTTP-кеш. Навигация судит исключительно о своём: загрузка не состоялась или документ доехал. Готовность блока из этого не следует — о ней говорит сама страница своим ready. Отменённая навигация при этом провалом не считается: WebKit отдаёт NSURLErrorCancelled и когда навигацию вытеснил клиентский редирект, и когда её остановил наш собственный cancel() на уехавшем с экрана блоке. Выдать это за провал значит свернуть исправный блок насовсем. Мост живёт столько же, сколько страница, и держится слабым прокси: WKUserContent- Controller держит обработчик сильно, иначе страница и вебвью не освободятся. Отладочная подмена готовности — для страниц, которые веб-контракт ещё не умеют.
Универсальный словарь действий страницы — один на все механики. Блок не знает, какая механика внутри, поэтому любая страница, говорящая этим словарём, получает нативное поведение без нового кода в SDK. Незнакомое действие — не ошибка: словарь у веб-стороны может быть новее, чем у SDK. openUrl открывает не что попало. Страница блока приезжает из сети, поэтому решать за пользователя, что откроет система, ей не положено: tel:, sms:, itms-apps: и схемы чужих приложений — это уже не переход по контенту, а действие от его имени, и canOpenURL для них проходит. Разрешено то, что никуда пользователя не увозит: веб-адреса и диплинки в само это приложение из CFBundleURLTypes. Открытие вынесено за шов EmbeddedBlockURLOpening — и чтобы политику можно было проверить тестами, и на будущее: в SDK открытие ссылок уже живёт в MindboxURLHandlerDelegate, и когда блоки поедут на общий мост инаппов, здесь окажется он, а не UIApplication напрямую.
Отладочное управление содержимым блоков — для тестового приложения и приёмки. Подменяет ответ на вопрос «что стоит за этим id», то есть встаёт ровно на место конфига из админки. Всё ниже — резолвер, страница, а дальше провайдер и бюджет ожидания у контейнера — работает без изменений, поэтому приёмка проверяет боевой путь, а не отдельный тестовый режим. Разметкой задаются сценарии, которых в сети нет: страница, сообщающая «пусто», молчащая страница, страница с ответом после таймаута. Не часть публичного API: доступно только через @_spi(Internal) import Mindbox. Из релизных сборок не вырезано намеренно — QA проверяет ровно то, что уходит клиентам, — поэтому каждая установка подмены пишется в лог.
Тег для сьютов блоков, папка MindboxTests/EmbeddedBlocks синхронизируемой группой и заготовка моков. В моках пока только то, что нужно тестам этой части: адрес страницы-заготовка и открыватель ссылок, который ничего не открывает, — политику схем иначе не проверить, canOpenURL в тестовом окружении пропускает системные схемы.
Резолвер проверяется главным своим обещанием: сколько блоков ни спросило бы про один id, за данными идём один раз. Пока конфиг синхронный это незаметно, с сетью это разница между одним запросом и N. Рядом — кэш, обход кэша перезагрузкой и то, что отладочная подмена сильнее и данных, и кэша: приёмка переключает сценарий на ходу.
Страница судит только о своём, и проверяется именно это разделение: настоящая ошибка сети — провал, а отменённая навигация — нет. Отмена при этом страницу не глушит: следующая настоящая ошибка приходит как обычно.
Обе ветки handle и вся политика схем: веб-адреса и диплинки хоста открываются, tel:, sms:, itms-apps:, mailto: и схемы чужих приложений — нет, причём отказ не отменяется тем, что система такую ссылку умеет открыть. Плюс кривые payload'ы, незнакомое действие и разбор CFBundleURLTypes.
added 5 commits
August 10, 2026 17:00
Переводит сообщения страницы в состояния блока. Не рисует контент и не знает механик: спрашивает у резолвера, что стоит за id, разбирает core-слой, а действия сверх него отдаёт универсальному обработчику. Готовность определяет только сама страница: ready — показываем, empty — показывать нечего. Навигация судит исключительно о своём, молчащая страница готовой не становится — её добьёт бюджет ожидания у контейнера. Нулевая высота в ready значит сломанную вёрстку: «показывать нечего» страница сообщает явным empty. Исход попытки хранится явно, а не пачкой флагов. Он переживает stop(), потому что это свойство страницы, а не факта нахождения в окне: уход блока с экрана не выбрасывает уже отрендеренную страницу, и возврат показывает её снова, без сети и без шиммера. При этом провал и empty страницу не убивают — она жива и может продолжать говорить, — поэтому известный исход служит и признаком того, что блока на экране больше нет: действия от невидимого блока не выполняются. За ним не стоит ни одного касания пользователя, а openUrl увёл бы человека из приложения на пустом месте. Экземпляр принадлежит одному контейнеру: ничего общего между контейнерами здесь нет, это и делает возможными несколько независимых блоков с одним id. Номер попытки отсекает резолв, доехавший уже после остановки или перезагрузки. Счётчик живых блоков на id — диагностика, а не механика: два блока с одним id законны, но чаще это скопированный id или переиспользованная ячейка, а у обоих случаев нет симптомов кроме «блок оказался не там, где ждали».
Собирает провайдер под конкретный блок: резолвер и обработчик действий общие на все блоки, а провайдер — свой на каждый. Это и делает блоки с одинаковым id независимыми друг от друга. Потребителей у фабрики появится два, и оба в следующей части: DI-регистрация и публичный init контейнера. Здесь она едет вместе с провайдером, потому что описывает его модель владения, а не способ его достать.
Дописывает моки до среза этой части: страница без WebKit, фабрика страниц со счётчиком созданных, резолвер с отложенным ответом, обработчик действий и общая заготовка провайдера со всеми подменёнными зависимостями. Резолвер умеет держать ответ до отдельной команды: так проверяется резолв, доехавший уже после остановки или перезагрузки блока. Фабрика считает страницы, потому что перезагрузка обязана создать новую, а возврат блока в окно — нет.
Весь путь блока без WebKit и без сети: резолв, показ по ready, пустой блок, сломанная нулевая высота, провал загрузки, действия страницы, отладочная подмена готовности, остановка и перезапуск, перезагрузка. Отдельно закреплено то, на что опираются соседние слои: после stop() провайдер молчит целиком — иначе контейнер не смог бы свернуть просроченный блок; уже отрендеренная страница на возврате в окно показывается как есть, без сети и шиммера, а блок, который показать не удалось, получает новую попытку; выброшенная перезагрузкой страница не может доложить в новую попытку ни сообщением, ни провалом, ни через подмену готовности. Действия проверяются с обеих сторон: из показанного блока доходят до обработчика, из схлопнутого — нет, ни после empty, ни после провала, ни после нулевой высоты, — а новая попытка снова их принимает. Счётчику живых блоков в каждом тесте свой id: он общий на процесс, иначе тесты, идущие параллельно, считали бы блоки друг друга.
Фабрика приехала без тестов, а её обещание — то, на чём держится независимость блоков с одинаковым id: провайдер свой на каждый блок, резолвер общий на все. Проверяется и то и другое, плюс что провайдер собран под запрошенный id. Резолвер в тестах отвечает «пусто»: страницу для такого блока не создают, поэтому настоящий вебвью фабрике здесь не нужен.
Contributor
|
Contributor
There was a problem hiding this comment.
Pull request overview
Adds the “foundation layer” for Embedded Blocks: defines the web ↔ native contract, resolves block IDs to web content, hosts that content in a WKWebView page with a dedicated JS handler, and routes page actions with a strict URL-opening policy. This is explicitly internal-facing for now (SPI/debug entry points), with tests for resolver/page/action routing.
Changes:
- Introduces Embedded Blocks core types: page message contract (
EmbeddedBlockPageMessage), state (EmbeddedBlockState), and web content descriptor (EmbeddedBlockWebContent). - Implements infrastructure: resolver with caching/coalescing (
EmbeddedBlockResolver), WKWebView-backed page with bridge (EmbeddedBlockWebViewPage), and action routing (EmbeddedBlockActionRouter) + debug overrides. - Adds logging category + test suites for resolver, page navigation failure semantics, and action routing URL policy.
Reviewed changes
Copilot reviewed 18 out of 18 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| MindboxTests/Extensions/Tag+Extensions.swift | Adds .embeddedBlocks test tag for grouping new suites. |
| MindboxTests/EmbeddedBlocks/EmbeddedBlockWebViewPageTests.swift | Tests load-failure reporting, especially ignoring NSURLErrorCancelled. |
| MindboxTests/EmbeddedBlocks/EmbeddedBlockResolverTests.swift | Tests resolver caching, coalescing, force refresh, and overrides behavior. |
| MindboxTests/EmbeddedBlocks/EmbeddedBlockMocks.swift | Adds shared test stubs/mocks for Embedded Blocks tests. |
| MindboxTests/EmbeddedBlocks/EmbeddedBlockActionRouterTests.swift | Tests openUrl allow/deny policy and Info.plist scheme parsing. |
| MindboxLoggerTests/LogPrimitivesTests.swift | Updates expected LogCategory count for new category. |
| MindboxLogger/Shared/Group/LogCategory.swift | Adds .embeddedBlocks log category and emoji mapping. |
| Mindbox/EmbeddedBlocks/WebView/EmbeddedBlockWebViewPage.swift | Implements WKWebView page hosting + JS message bridge + navigation failure reporting. |
| Mindbox/EmbeddedBlocks/WebView/EmbeddedBlockReadinessOverrides.swift | Adds debug readiness override for treating “document loaded” as ready. |
| Mindbox/EmbeddedBlocks/WebView/EmbeddedBlockPageMessage.swift | Defines/parses the Embedded Blocks page → native message contract. |
| Mindbox/EmbeddedBlocks/WebView/EmbeddedBlockPageHosting.swift | Introduces the hosting protocol seam for page implementations. |
| Mindbox/EmbeddedBlocks/Resolver/EmbeddedBlockWebContent.swift | Defines web content source (URL vs inline HTML). |
| Mindbox/EmbeddedBlocks/Resolver/EmbeddedBlockResolver.swift | Implements ID→content resolver with cache and in-flight request coalescing. |
| Mindbox/EmbeddedBlocks/Resolver/EmbeddedBlockContentOverrides.swift | Implements debug content overrides (by id) used by QA/acceptance flows. |
| Mindbox/EmbeddedBlocks/Public/MindboxEmbeddedBlockDebug.swift | Exposes SPI debug API for content/readiness overrides. |
| Mindbox/EmbeddedBlocks/Container/EmbeddedBlockState.swift | Defines internal container state machine states. |
| Mindbox/EmbeddedBlocks/Actions/EmbeddedBlockActionRouter.swift | Routes page actions; implements strict openUrl policy + host scheme extraction. |
| Mindbox.xcodeproj/project.pbxproj | Adds synchronized groups for EmbeddedBlocks sources/tests. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
Док класса пересказывал то, что уже сказано ниже по файлу: правила готовности — у apply(height:) и handleLoadFinish, владение высотой — у heightChanged, а мысль «уход из окна не выбрасывает страницу» шла трижды — в доке класса, в доке свойства page и во встроенном комментарии в start(). Осталось два абзаца: чем провайдер является и что после stop() он обязан молчать — это межтиповой инвариант, из одного файла его не видно. Убраны и три дока, ушедшие из своей ответственности или пересказывавшие подпись: contentView рассказывал, как контейнер растягивает вью; второй абзац reload() — про плейсхолдер и события хосту; isShown повторял собственное имя. Встроенные комментарии не тронуты: каждый объясняет отсутствующую строку, развилку или внешнюю причину — то, чего в коде не прочитать.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
6 tasks
added 6 commits
August 10, 2026 17:41
Держит в контейнере ровно одну вью, растянутую по его краям. Слои блока — плейсхолдер, контент, экран ошибки — взаимоисключающие: показать новый значит снять прежний. Показанная вью запоминается отдельно от свойств контейнера, потому что подменить её хост может в любой момент, а снимать надо ту, что действительно висит, а не ту, что лежит в свойстве сейчас.
Бюджет на то, чтобы показаться, принадлежит контейнеру, а не контенту: чем бы блок ни оказался внутри, вёрстка хоста не ждёт его вечно. Считается время ожидания пользователя, а не календарное: пока блока никто не ждёт — приложение в фоне, контейнер вне окна — отсчёт стоит. Но именно стоит, а не начинается заново: потраченное запоминается, и попытка продолжает бюджет с того места, где её прервали. Пауза, отдающая полный бюджет заново, не заканчивается никогда — пользователь, переключающийся между приложениями каждые пять секунд, продлевал бы ожидание блока бесконечно, и вёрстка ждала бы его вечно. Ровно то, против чего бюджет и заведён. Полный бюджет получает только новая попытка. Часы вынесены отдельным швом: считать потраченное без них нельзя, а тесты не могут ждать бюджет целиком — им нужно уметь сказать, что время прошло. Загрузку пауза не трогает: она идёт своим чередом, в фоне её тормозит система, а не SDK.
Плейсхолдер, который блок показывает, пока грузится, если хост не дал своего. Занимает весь контейнер: место под блок должно быть занято сразу, а занятое место не должно выглядеть пустым.
Публичный API: контейнер создаётся с id блока из админки и высотой, которую блок должен занять. Хост ставит его куда угодно и задаёт только положение и ширину — высоту контейнер заявляет сам через intrinsicContentSize: данную при создании, пока контент грузится и показан, и 0, когда показывать нечего. Оба исхода настраиваются: placeholderView заменяет штатный шиммер, errorView — это согласие показать провал вместо схлопывания. Жизненным циклом владеет SDK: контент стартует, когда блок попадает в окно, и останавливается, когда уходит. Публичного способа запустить его руками нет. Внутри — машина из четырёх состояний контента и одного видимого слоя на каждое. Три решения, которые стоят за ней: Место, единожды отданное хосту, назад не забирается. Блок, который показать не удалось, на возврате в окно пробует снова, но контейнер под эту попытку места уже не занимает и шиммером не мигает — иначе он дёргал бы вёрстку на свою высоту и мигал на каждый свой проход по экрану, ничего в итоге не показывая. Разворачивает блок только показанный контент или явная перезагрузка. Тот же делегат — не новый подписчик. Хост штатно переприсваивает его на каждой переиспользованной ячейке, и отдавать ему на это уже услышанный исход нельзя: на исход он перестраивает вёрстку, а перестройка вёрстки снова переприсваивает делегата. Исходы отдаются на следующем витке главной очереди: состояние может измениться посреди прохода layout, и заходить оттуда в код хоста — верный способ сломать его вёрстку. EmbeddedBlockPresentation — снимок показа для SwiftUI-обёртки: она сама назначает себе высоту и сама рисует слои хоста, потому что вью, отданная контейнеру через отдельный UIHostingController, выпадает из дерева SwiftUI и теряет его окружение.
Резолвер общий: его кэш на id и очередь ожидающих — это и есть «одна загрузка данных на id» для всех контейнеров сразу. Обработчик действий общий потому, что не имеет состояния. Провайдеры, наоборот, фабрика делает на каждый блок свой — так блоки остаются независимыми друг от друга. Файл прописан в проект вручную: Mindbox/DI/Injections — обычная группа, а не синхронизируемая, в отличие от папок самих блоков.
Хост слоёв приехал без прямого покрытия: контейнер задевает его косвенно, но собственное обещание — ровно одна вью, растянутая по краям, — проверить надо отдельно. Кроме подмены и снятия закреплены две тонкости: повторный показ той же вью констрейнты не пересобирает (контейнер зовёт show на каждую смену состояния), а вью, снятую снаружи, показ обязан вернуть на место.
added 3 commits
August 11, 2026 19:46
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.
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.
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.
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
This comment was marked as spam.
This comment was marked as spam.
Sorry, something went wrong.
Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com>
* 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>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
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.
Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com>
* 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>
…ion' into feature/MOBILE-323-block-foundation
* 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>
added 3 commits
August 12, 2026 18:09
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.
justSmK
reviewed
Aug 12, 2026
justSmK
reviewed
Aug 12, 2026
…ddedBlock.swift Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com>
…ddedBlock.swift Co-authored-by: Sergei Semko <28645140+justSmK@users.noreply.github.com>
justSmK
approved these changes
Aug 12, 2026
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
Нижний слой встроенных блоков: как страница блока разговаривает с нативной стороной, откуда берётся её адрес и что ей позволено делать.
Это первая из четырёх частей, на которые разбита подсистема встроенных блоков. Публичного API для хост-приложения здесь ещё нет — ни UIKit-контейнера, ни SwiftUI-обёртки: наружу смотрит только
@_spi(Internal)для приёмки. Для клиентов код в этой части инертен.Что внутри
EmbeddedBlockPageMessage,EmbeddedBlockState,EmbeddedBlockWebContentready,heightChanged,empty; всё остальное с валидным конвертом уходит механике какactionEmbeddedBlockResolverEmbeddedBlockWebViewPageWKWebViewза швомEmbeddedBlockPageHosting. Вебвью изInAppWebViewFactory— тот же user agent, тот жеWKWebsiteDataStore, общий с инаппами HTTP-кешEmbeddedBlockActionRouterMindboxEmbeddedBlockDebugКоммиты разделены по слоям, продакшн и тесты — отдельно.
На что стоит смотреть
EmbeddedBlockPageMessage. Ядро не знает и не должно знать словарь конкретной механики.EmbeddedBlockResolver— общая точка всех контейнеров, при этом страница и состояние у каждого блока свои.forceRefreshнужен перезагрузке: переехавший или выключенный блок иначе вечно доставал бы из кэша прежний адрес.NSURLErrorCancelledи когда навигацию вытеснил клиентский редирект, и когда её остановил наш собственныйcancel()на уехавшем с экрана блоке. Выдать это за провал значит свернуть исправный блок насовсем.openUrl. Страница блока приезжает из сети, поэтому решать за пользователя, что откроет система, ей не положено:tel:,sms:,itms-apps:и схемы чужих приложений — это уже действие от его имени, иcanOpenURLдля них проходит. Разрешено то, что никуда пользователя не увозит: веб-адреса и диплинки в само это приложение изCFBundleURLTypes. Это сознательно строже, чемnavigateу инаппов (TransparentView.swift), — обсуждаемо.Что здесь заведомо не готово
mindboxEmbeddedBlock) вместо общегоMindboxWebBridgeинаппов — сведение отдельной задачей.decidePolicyFor: тап по ссылке внутри блока пока уводит навигацию в сам блок. Правильное решение — переиспользовать хукMindboxWebBridge, а не копировать switch изTransparentView; это часть того же переезда на общий мост.EmbeddedBlockPageMessageбез тестов — договорились сделать позже.EmbeddedBlockReadinessOverridesиMindboxEmbeddedBlockDebugтоже без покрытия; их логичнее отдать в следующие части, к провайдеру и SwiftUI-обёртке соответственно.Правки проекта
Mindbox/EmbeddedBlocksиMindboxTests/EmbeddedBlocksзаведены синхронизируемыми группами — 7 строк вproject.pbxprojбез единого удаления, файлы в этих папках дальше подхватываются без правок проекта.Type of Change
Test Procedure
Mindboxscheme, iPhone 16 Pro simulator:implicit_optional_initialization,function_name_whitespace), к этим изменениям отношения не имеющиеНовые сьюты:
EmbeddedBlockResolverTests,EmbeddedBlockWebViewPageTests,EmbeddedBlockActionRouterTests.Pre-flight Checklist
EmbeddedBlockPageMessageи отладочных подмен)