release: 0.0.4 (artisan ^0.0.8) + magic_devtools install repoint#16
Conversation
…evtools dep telescope:install injected the now-removed package:magic/telescope_integration.dart import (the adapter moved to the magic_devtools package). Repoint the injected import to package:magic_devtools/telescope.dart and gate the magic-side wiring on a magic_devtools dependency (_hasMagicDevtoolsDep).
Welcome to Codecov 🎉Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests. Thanks for integrating Codecov - We've got you covered ☂️ |
There was a problem hiding this comment.
Pull request overview
This PR prepares the 0.0.4 release of fluttersdk_telescope, updating the fluttersdk_artisan dependency to ^0.0.8 and repointing the Magic integration injected by telescope:install to the new magic_devtools telescope barrel so downstream dependency resolution remains compatible.
Changes:
- Bump package version to
0.0.4and updatefluttersdk_artisanconstraint to^0.0.8. - Update
telescope:installto injectimport 'package:magic_devtools/telescope.dart';and gate Magic wiring on amagic_devtoolsdependency. - Update tests, example metadata, and changelog entries to match the new release/integration behavior.
Reviewed changes
Copilot reviewed 5 out of 6 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
lib/src/commands/telescope_install_command.dart |
Switch Magic integration injection to magic_devtools and update dependency detection helper. |
test/src/commands/telescope_install_command_test.dart |
Adjust Magic install command test expectations to the new injected import path. |
pubspec.yaml |
Bump version to 0.0.4 and update fluttersdk_artisan constraint to ^0.0.8. |
example/pubspec.yaml |
Update example’s referenced telescope version to 0.0.4. |
example/pubspec.lock |
Refresh lockfile to reflect artisan 0.0.8 and telescope 0.0.4. |
CHANGELOG.md |
Add unreleased/release notes for Magic import repoint and artisan constraint bump. |
| if (_hasMagicDevtoolsDep()) { | ||
| MainDartEditor.addImport( | ||
| mainDartPath, | ||
| "import 'package:magic/telescope_integration.dart';", | ||
| "import 'package:magic_devtools/telescope.dart';", | ||
| ); |
| /// Returns true when the consumer's pubspec.yaml lists `magic_devtools:` | ||
| /// (the package that ships MagicTelescopeIntegration) as a dependency or | ||
| /// dev_dependency (2-space indent). |
…r present (PR #16 review)
|
Both Copilot review notes were addressed in c738a1b (gate the magic_devtools import on a real |
What
version0.0.3->0.0.4andfluttersdk_artisan^0.0.6->^0.0.8. Required for co-installability withfluttersdk_dusk0.0.7 (which needs artisan^0.0.8) inside a downstream package. No public API change.telescope:installnow injectsimport 'package:magic_devtools/telescope.dart';(the Magic telescope adapter moved out of magic core intomagic_devtools) and gates the magic-side wiring on amagic_devtoolsdependency (_hasMagicDevtoolsDep), so a magic-only consumer never gets an unresolvable import.Verification
dart analyze0 issues,dart formatclean, 253 tests green,dart pub publish --dry-runclean (resolves against artisan 0.0.8).Generated via /ac:execute (plan: magic-stabilize-dusk-telescope).