Skip to content

Throwing an error when reloaded sources when reloaded_sources.json is empty or null#2810

Merged
Markzipan merged 13 commits intodart-lang:mainfrom
Markzipan:reloaded-sources
Apr 10, 2026
Merged

Throwing an error when reloaded sources when reloaded_sources.json is empty or null#2810
Markzipan merged 13 commits intodart-lang:mainfrom
Markzipan:reloaded-sources

Conversation

@Markzipan
Copy link
Copy Markdown
Contributor

This was causing some webdev tests to error and time out.

Also includes a change to set chrome proxy service's canary flag when DDC modules are specified.

@Markzipan Markzipan requested review from jyameo and nshahan April 2, 2026 21:15
Comment thread dwds/lib/src/handlers/injector.dart Outdated
Comment thread dwds/lib/src/handlers/injector.dart Outdated
Comment thread dwds/pubspec.yaml Outdated
Comment thread dwds/web/reloader/ddc_library_bundle_restarter.dart Outdated
Comment thread dwds/web/reloader/ddc_library_bundle_restarter.dart Outdated
@Markzipan Markzipan marked this pull request as ready for review April 3, 2026 00:25
@Markzipan Markzipan changed the title Defaulting reloaded sources to an empty list when reloaded_sources.json is invalid Throwing an error when reloaded sources when reloaded_sources.json is empty or null Apr 7, 2026
@Markzipan Markzipan requested a review from nshahan April 7, 2026 20:12
Copy link
Copy Markdown
Contributor

@jyameo jyameo left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

Comment thread dwds/CHANGELOG.md Outdated
Comment thread dwds/lib/src/services/chrome/chrome_proxy_service.dart Outdated
Comment thread dwds/web/reloader/ddc_library_bundle_restarter.dart Outdated
@Markzipan Markzipan requested a review from nshahan April 8, 2026 22:15
Comment thread dwds/lib/src/services/chrome/chrome_proxy_service.dart Outdated
Comment thread dwds/web/reloader/ddc_library_bundle_restarter.dart Outdated
@Markzipan Markzipan requested a review from nshahan April 9, 2026 17:05
Copy link
Copy Markdown
Contributor

@nshahan nshahan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@Markzipan Markzipan merged commit 4e1e7da into dart-lang:main Apr 10, 2026
47 checks passed
@Markzipan Markzipan deleted the reloaded-sources branch April 10, 2026 16:57
Markzipan added a commit that referenced this pull request Apr 13, 2026
… empty or null (#2810)

This error was being swallowed, making debugging a pain.

Also includes a change to set chrome proxy service's canary flag when DDC modules are specified.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants