Fixed bug for relative URIs in app dev

[georgge293]

closes: https://github.com/shop/issues-event-foundations/issues/793

WHY are these changes introduced?

During development, webhook events need to be delivered to the local development server through a tunnel URL. Previously, relative URIs in event subscriptions were not being resolved, causing webhook delivery failures.

WHAT is this pull request doing?

Implements URI resolution for event subscriptions by modifying transformFromEventsConfig to prepend the application_url to relative URIs (those starting with /). Absolute URIs remain unchanged. The transformation handles edge cases including missing configuration, trailing slashes in application URLs, and empty subscription arrays.

How to test your changes?

1. Create an app with event subscriptions using relative URIs (e.g., /webhooks/orders)
2. Start the development server with a tunnel
3. Verify that the event subscriptions are registered with the full tunnel URL
4. Test with absolute URIs to ensure they remain unchanged
5. Run the test suite to verify all edge cases are handled

[georgge293]

• Fixed bug for relative URIs in app dev #6891 👈 (View in Graphite)
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

[github-actions]

We detected some changes at packages/*/src and there are no updates in the .changeset.
If the changes are user-facing, run pnpm changeset add to track your changes and include them in the next release CHANGELOG.

Caution

DO NOT create changesets for features which you do not wish to be included in the public changelog of the next CLI release.

[github-actions]

Coverage report

St.❔	Category	Percentage	Covered / Total
🟡	Statements	78.81%	14518/18421
🟡	Branches	73.16%	7221/9870
🟡	Functions	79.02%	3691/4671
🟡	Lines	79.15%	13725/17340

Test suite run success

3785 tests passing in 1448 suites.

Report generated by 🧪jest coverage report action from e3e7335

[cdarne]

tophatted like a charm:

[karenxie]

LGTM!

[jantnovi]

Approving the change now because this is the current blessed way to do this sort of transformation, but I think longer term the team would like to be more explicit with transformation interfaces we provide. This is especially true given our impending move to JSON contracts for module validation and given that we want to push as much logic to the backend as possible. Will discuss that internally, but for now this won't block our immediate plans so lgtm.

[georgge293]

Merge activity

• Feb 26, 8:55 PM UTC: A user started a stack merge that includes this pull request via Graphite.
• Feb 26, 8:55 PM UTC: @georgge293 added this pull request to the GitHub merge queue with Graphite.

[binks-code-reviewer]

🤖 Code Review · #projects-dev-ai for questions
React with 👍/👎 or reply — all feedback helps improve the agent.

🚫 PR closed

📋 History

🚫 PR closed