Commit 7b6f783
[beta] Dart MCP server updates
Issue description: Without this change, the dart MCP command launch_app will not work for the `chrome` device target because it won't find any DTD URI.
What is the fix: This starts using the `--machine` along with `--print-dtd` arguments to the `flutter run` command, now that it prints the DTD URI in machine mode.
Why cherry-pick: This will enable Flutter developers to develop web apps using the MCP server without hanging.
Risk: Low, this only affects the Dart MCP server's `launch_app` command so the radius is small, and the likely worst thing that could happen is that it would stop working for other devices as well.
See dart-lang/ai#301 and flutter/flutter#176655 for more information.
Cherry-pick: https://dart-review.googlesource.com/c/sdk/+/456480
Bug: dart-lang/ai#302
Change-Id: Ide1ea8d447de43a0a48fae05d40f9051457eaa2c
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/456482
Commit-Queue: Alexander Thomas <[email protected]>
Reviewed-by: Ivan Inozemtsev <[email protected]>
Reviewed-by: Alexander Thomas <[email protected]>1 parent b9a48cb commit 7b6f783
2 files changed
+19
-1
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
123 | 123 | | |
124 | 124 | | |
125 | 125 | | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
126 | 144 | | |
127 | 145 | | |
128 | 146 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
131 | 131 | | |
132 | 132 | | |
133 | 133 | | |
134 | | - | |
| 134 | + | |
135 | 135 | | |
136 | 136 | | |
137 | 137 | | |
| |||
0 commit comments