Skip to content

[interop] Implement Diagnostics and Handling Errors #426

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 23, 2025

Conversation

nikeokoronkwo
Copy link
Collaborator

@nikeokoronkwo nikeokoronkwo commented Jul 21, 2025

Fixes #421

This PR adds support for handling diagnostics and errors when parsing TS declaration files for the generator to transform, preventing the generator from handling invalid TS files.

Diagnostics are printed in the following format

<filename> (<line no>,<char no>): <message>

And the process exits with status code 1.

@nikeokoronkwo nikeokoronkwo changed the title [interop] Implement Diagnostics and Parsing Errors [interop] Implement Diagnostics and Handling Errors Jul 21, 2025
Copy link
Contributor

@srujzs srujzs left a comment

Choose a reason for hiding this comment

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

Nice, thanks!

@nikeokoronkwo nikeokoronkwo requested a review from srujzs July 23, 2025 15:28
@srujzs srujzs merged commit 8ffaf5e into dart-lang:main Jul 23, 2025
16 checks passed
@nikeokoronkwo nikeokoronkwo deleted the interop/diagnostics branch July 23, 2025 16:21
nikeokoronkwo added a commit to nikeokoronkwo/web that referenced this pull request Jul 30, 2025
commit da1dd5d
Author: Devon Carew <[email protected]>
Date:   Thu Jul 24 10:24:40 2025 -0700

    switch CI to using workflow files (dart-lang#428)

    * switch CI to using workflow files

    * update the sdk matrix

    * review feedback

    * remove format sdk version check

commit 8ffaf5e
Author: Nikechukwu <[email protected]>
Date:   Wed Jul 23 12:17:55 2025 -0400

    [interop] Implement Diagnostics and Handling Errors (dart-lang#426)

    * implemented diagnostics

    * added extra invalid semantic
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jul 30, 2025
Revisions updated by `dart tools/rev_sdk_deps.dart`.

ai (https://github.com/dart-lang/ai/compare/40607dc..7fbe88a):
  7fbe88a  2025-07-25  Jacob MacDonald  refine the prompt and fix the extension listener (dart-lang/ai#244)
  8d2384b  2025-07-25  Jacob MacDonald  add some intellij files to gitignore (dart-lang/ai#245)
  e09e469  2025-07-24  Jacob MacDonald  add prompt for using flutter driver to write a test for a user journey (dart-lang/ai#243)
  88d7a4d  2025-07-23  Jacob MacDonald  Flutter driver updates, add screenshot support back (dart-lang/ai#241)

http (https://github.com/dart-lang/http/compare/4a90d16..afda310):
  afda310  2025-07-23  Brian Quinlan  [cronet] Support aborting requests (dart-lang/http#1797)

test (https://github.com/dart-lang/test/compare/c201cc9..6aeb1e4):
  6aeb1e4f  2025-07-24  Nate Bosch  Consider NaN a match for itself (dart-lang/test#2524)
  65a37698  2025-07-23  Nate Bosch  Use latest analyzer and language version (dart-lang/test#2523)

web (https://github.com/dart-lang/web/compare/767151e..da1dd5d):
  da1dd5d  2025-07-24  Devon Carew  switch CI to using workflow files (dart-lang/web#428)
  8ffaf5e  2025-07-23  Nikechukwu  [interop] Implement Diagnostics and Handling Errors (dart-lang/web#426)

Change-Id: I499491c4c1466cb1c81cf5e4bbff4a484e880e57
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/442560
Auto-Submit: Devon Carew <[email protected]>
Commit-Queue: Devon Carew <[email protected]>
Reviewed-by: Konstantin Shcheglov <[email protected]>
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.

Interop Gen: Implement Error Diagnostics when parsing input files
2 participants