Skip to content

internal: pass base_url through to run()#60

Merged
anilanar merged 1 commit into
masterfrom
feat/run-settings-base-url
May 13, 2026
Merged

internal: pass base_url through to run()#60
anilanar merged 1 commit into
masterfrom
feat/run-settings-base-url

Conversation

@buscape
Copy link
Copy Markdown
Contributor

@buscape buscape commented Apr 29, 2026

makes baseUrl available for the messenger bundle know which origin served its loader.

Backwards compatible: baseUrl is optional. Existing umm bundles ignore the new field; bundles that read it fall back to their env-baked URL when older messenger-internal omits it.

makes baseUrl available for the messenger bundle know which origin
served its loader.

Backwards compatible: baseUrl is optional. Existing umm bundles ignore
the new field; bundles that read it fall back to their env-baked URL
when older messenger-internal omits it.
@buscape buscape self-assigned this Apr 29, 2026
@buscape buscape requested a review from anilanar May 13, 2026 09:50
@buscape buscape assigned anilanar and unassigned buscape May 13, 2026
@anilanar anilanar merged commit 4c1b7dc into master May 13, 2026
@anilanar anilanar deleted the feat/run-settings-base-url branch May 13, 2026 09:52
anilanar added a commit that referenced this pull request May 22, 2026
Companion to the next commit. PR #60's original commit (4c1b7dc) used
the non-conventional 'internal:' type prefix that release-please does
not recognize, so the fix has been sitting on master unreleased.

We can't just push an empty conventional-commit marker because
GitHub's merge strategies drop empty commits.

Workaround: revert here, re-apply with the right message in the next
commit. Net code change on master: zero.
anilanar added a commit that referenced this pull request May 22, 2026
Re-applies the change originally landed in 4c1b7dc (#60) on 2026-05-13
with a conventional-commit subject so release-please picks it up. The
previous commit on this branch reverts the change; this one re-applies
it. Net code change on master: zero. The diff is identical to
4c1b7dc's diff.

Original change: makes baseUrl available so the messenger bundle knows
which origin served its loader. Backwards compatible: baseUrl is
optional. Existing umm bundles ignore the new field; bundles that
read it fall back to their env-baked URL when older messenger-internal
omits it.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants