Skip to content

Add dependency on package:vm_service_interface #2262

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 3 commits into from
Oct 24, 2023
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion dwds/lib/src/dwds_vm_client.dart
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ import 'package:dwds/src/services/debug_service.dart';
import 'package:dwds/src/utilities/synchronized.dart';
import 'package:logging/logging.dart';
import 'package:uuid/uuid.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServerConnection;
import 'package:vm_service_interface/vm_service_interface.dart';
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';

final _logger = Logger('DwdsVmClient');
Expand Down
4 changes: 3 additions & 1 deletion dwds/lib/src/services/chrome_proxy_service.dart
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,9 @@ import 'package:dwds/src/utilities/dart_uri.dart';
import 'package:dwds/src/utilities/shared.dart';
import 'package:logging/logging.dart' hide LogRecord;
import 'package:pub_semver/pub_semver.dart' as semver;
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart'
hide VmServiceInterface, vmServiceVersion;
import 'package:vm_service_interface/vm_service_interface.dart';
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';

/// A proxy from the chrome debug protocol to the dart vm service protocol.
Expand Down
2 changes: 1 addition & 1 deletion dwds/lib/src/services/debug_service.dart
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@ import 'package:shelf/shelf.dart' as shelf;
import 'package:shelf/shelf.dart' hide Response;
import 'package:shelf_web_socket/shelf_web_socket.dart';
import 'package:sse/server/sse_handler.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service_interface/vm_service_interface.dart';
import 'package:web_socket_channel/web_socket_channel.dart';

bool _acceptNewConnections = true;
Expand Down
3 changes: 2 additions & 1 deletion dwds/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ dependencies:
built_value: ^8.3.0
collection: ^1.15.0
crypto: ^3.0.2
dds: ^2.7.1
dds: ^3.0.0
file: ">=6.0.0 <8.0.0"
http: ^0.13.4
http_multi_server: ^3.2.0
Expand All @@ -34,6 +34,7 @@ dependencies:
sse: ^4.1.2
uuid: ^3.0.6
vm_service: ^12.0.0
vm_service_interface: 1.0.0
web_socket_channel: ^2.2.0
webkit_inspection_protocol: ^1.0.1

Expand Down
4 changes: 4 additions & 0 deletions dwds/pubspec_overrides.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -3,3 +3,7 @@ dependency_overrides:
git:
url: https://github.com/dart-lang/sdk.git
path: pkg/vm_service
vm_service_interface:
git:
url: https://github.com/dart-lang/sdk.git
path: pkg/vm_service_interface
3 changes: 2 additions & 1 deletion dwds/test/build_daemon_breakpoint_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,8 @@

import 'package:test/test.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/build_daemon_callstack_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/chrome_proxy_service_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,8 @@ import 'package:path/path.dart' as path;
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';
import 'package:webkit_inspection_protocol/webkit_inspection_protocol.dart';

import 'fixtures/context.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/evaluate_circular_common.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/events_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,8 @@ import 'package:dwds/src/utilities/server.dart';
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';
import 'package:webdriver/async_core.dart';

import 'fixtures/context.dart';
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/fixtures/context.dart
Original file line number Diff line number Diff line change
Expand Up @@ -118,7 +118,7 @@ class TestContext {
ChromeProxyService get service => fetchChromeProxyService(debugConnection);

/// External VM service.
VmServiceInterface get vmService => debugConnection.vmService;
VmService get vmService => debugConnection.vmService;

TestContext(this.project, this.sdkConfigurationProvider)
: nullSafety = project.nullSafety {
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/frontend_server_breakpoint_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/frontend_server_callstack_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/instances/common/instance_inspection_common.dart
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,7 @@ void runTests({
: TestProject.testPackageWithWeakNullSafety();
final context = TestContext(project, provider);

late VmServiceInterface service;
late VmService service;
late Stream<Event> stream;
late String isolateId;
late ScriptRef mainScript;
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/instances/common/patterns_inspection_common.dart
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ void runTests({
TestContext(TestProject.testExperimentWithSoundNullSafety, provider);
final testInspector = TestInspector(context);

late VmServiceInterface service;
late VmService service;
late Stream<Event> stream;
late String isolateId;
late ScriptRef mainScript;
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/instances/common/record_inspection_common.dart
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ void runTests({
TestContext(TestProject.testExperimentWithSoundNullSafety, provider);
final testInspector = TestInspector(context);

late VmServiceInterface service;
late VmService service;
late Stream<Event> stream;
late String isolateId;
late ScriptRef mainScript;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ void runTests({
TestContext(TestProject.testExperimentWithSoundNullSafety, provider);
final testInspector = TestInspector(context);

late VmServiceInterface service;
late VmService service;
late Stream<Event> stream;
late String isolateId;
late ScriptRef mainScript;
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/instances/common/test_inspector.dart
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ class TestInspector {
TestInspector(this.context);
TestContext context;

VmServiceInterface get service => context.debugConnection.vmService;
VmService get service => context.debugConnection.vmService;

Future<void> onBreakPoint(
Stream<Event> stream,
Expand Down
2 changes: 1 addition & 1 deletion dwds/test/instances/common/type_inspection_common.dart
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,7 @@ void runTests({
final context = TestContext(project, provider);
final testInspector = TestInspector(context);

late VmServiceInterface service;
late VmService service;
late Stream<Event> stream;
late String isolateId;
late ScriptRef mainScript;
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/refresh_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,8 @@ import 'dart:async';

import 'package:test/test.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/restore_breakpoints_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,8 @@ import 'dart:async';
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
3 changes: 2 additions & 1 deletion dwds/test/run_request_test.dart
Original file line number Diff line number Diff line change
Expand Up @@ -8,7 +8,8 @@ import 'dart:async';
import 'package:test/test.dart';
import 'package:test_common/logging.dart';
import 'package:test_common/test_sdk_configuration.dart';
import 'package:vm_service/vm_service.dart';
import 'package:vm_service/vm_service.dart' hide VmServiceInterface;
import 'package:vm_service_interface/vm_service_interface.dart';

import 'fixtures/context.dart';
import 'fixtures/project.dart';
Expand Down
4 changes: 3 additions & 1 deletion webdev/CHANGELOG.md
Original file line number Diff line number Diff line change
@@ -1,8 +1,10 @@
## 3.2.0-wip

- Add dependency on `package:vm_service_interface`. - [#2262](https://github.com/dart-lang/webdev/pull/2262)

## 3.1.0

- Update the parameters passed to `Dwds.start`. - [#2231](https://github.com/dart-lang/webdev/pull/2231).
- Update the parameters passed to `Dwds.start`. - [#2231](https://github.com/dart-lang/webdev/pull/2231)
- Update `package:vm_service` constraint to `>=10.1.0 <13.0.0`. - [#2248](https://github.com/dart-lang/webdev/pull/2248)
- Update `dwds` constraint to `21.0.0`.

Expand Down
2 changes: 1 addition & 1 deletion webdev/pubspec.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,7 @@ dependencies:
browser_launcher: ^1.1.0
collection: ^1.15.0
crypto: ^3.0.2
dds: ^2.2.0
dds: ^3.0.0
# Pin DWDS to avoid dependency conflicts with vm_service:
dwds: 22.0.0
http: ^0.13.4
Expand Down