bump: Replace discontinued deps and update fl_lib dep#1024
bump: Replace discontinued deps and update fl_lib dep#1024GT-610 merged 4 commits intolollipopkit:mainfrom
fl_lib dep#1024Conversation
…ependencies Update the fl_lib dependency version in the pubspec.yaml and pubspec.lock files to v1.0.361, and simultaneously update the dependency versions of camera_avfoundation and objective_c
📝 WalkthroughWalkthroughReplaced imports of Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Suggested reviewers
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: Organization UI Review profile: CHILL Plan: Pro ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
🚧 Files skipped from review as they are similar to previous changes (1)
✏️ Tip: You can disable this entire section by setting Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 Fix all issues with AI agents
In `@lib/view/page/server/detail/view.dart`:
- Line 5: The project is missing the flutter_markdown_plus package required by
the import and usage of MarkdownBody and MarkdownStyleSheet; add
flutter_markdown_plus: ^1.0.7 to the pubspec.yaml dependencies, run flutter pub
get, then re-run the analyzer/build to ensure the import line "import
'package:flutter_markdown_plus/flutter_markdown_plus.dart';" and usages of
MarkdownBody/MarkdownStyleSheet resolve without errors.
♻️ Duplicate comments (1)
lib/view/page/snippet/edit.dart (1)
3-3: Same dependency gap as ServerDetailPage import.Please add
flutter_markdown_plustopubspec.yaml(and removeflutter_markdownif present) so this import resolves.
📜 Review details
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
pubspec.lockis excluded by!**/*.lock
📒 Files selected for processing (3)
lib/view/page/server/detail/view.dartlib/view/page/snippet/edit.dartpubspec.yaml
🧰 Additional context used
📓 Path-based instructions (2)
lib/view/**/*.dart
📄 CodeRabbit inference engine (CLAUDE.md)
lib/view/**/*.dart: Prefer widgets/utilities from fl_lib for common UI (e.g., CustomAppBar, context.showRoundDialog, Input, Btnx.cancelOk)
Prefer using libL10n strings before adding new ones to project l10n
Split UI into build, actions, and utils; use extension on to separate concerns
Files:
lib/view/page/snippet/edit.dartlib/view/page/server/detail/view.dart
pubspec.yaml
📄 CodeRabbit inference engine (CLAUDE.md)
Use hive_ce instead of hive for Hive integration
Files:
pubspec.yaml
🧠 Learnings (5)
📓 Common learnings
Learnt from: CR
Repo: lollipopkit/flutter_server_box PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-10-08T09:36:41.682Z
Learning: Applies to lib/view/**/*.dart : Prefer widgets/utilities from fl_lib for common UI (e.g., CustomAppBar, context.showRoundDialog, Input, Btnx.cancelOk)
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR
Repo: lollipopkit/flutter_server_box PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-10-08T09:36:41.682Z
Learning: Applies to lib/view/**/*.dart : Prefer widgets/utilities from fl_lib for common UI (e.g., CustomAppBar, context.showRoundDialog, Input, Btnx.cancelOk)
Applied to files:
lib/view/page/snippet/edit.dartlib/view/page/server/detail/view.dart
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR
Repo: lollipopkit/flutter_server_box PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-10-08T09:36:41.682Z
Learning: Use Riverpod with code generation (riverpod_annotation) for state management
Applied to files:
lib/view/page/snippet/edit.dart
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR
Repo: lollipopkit/flutter_server_box PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-10-08T09:36:41.682Z
Learning: Applies to lib/view/**/*.dart : Split UI into build, actions, and utils; use extension on to separate concerns
Applied to files:
lib/view/page/snippet/edit.dartlib/view/page/server/detail/view.dart
📚 Learning: 2025-10-08T09:36:41.682Z
Learnt from: CR
Repo: lollipopkit/flutter_server_box PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-10-08T09:36:41.682Z
Learning: Applies to lib/view/**/*.dart : Prefer using libL10n strings before adding new ones to project l10n
Applied to files:
lib/view/page/snippet/edit.dartlib/view/page/server/detail/view.dart
🪛 GitHub Actions: flutter analysis
lib/view/page/snippet/edit.dart
[error] 3-3: Target of URI doesn't exist: 'package:flutter_markdown_plus/flutter_markdown_plus.dart'. Try creating the file referenced by the URI, or try using a URI for a file that does exist. - uri_does_not_exist
[error] 206-206: The method 'MarkdownStyleSheet' isn't defined for the type '_SnippetEditPageState'. Try correcting the name to the name of an existing method, or defining a method named 'MarkdownStyleSheet'. - undefined_method
lib/view/page/server/detail/view.dart
[error] 5-5: Target of URI doesn't exist: 'package:flutter_markdown_plus/flutter_markdown_plus.dart'. Try creating the file referenced by the URI, or try using a URI for a file that does exist. - uri_does_not_exist
[error] 767-767: The method 'MarkdownBody' isn't defined for the type '_ServerDetailPageState'. Try correcting the name to the name of an existing method, or defining a method named 'MarkdownBody'. - undefined_method
[error] 770-770: Undefined name 'MarkdownStyleSheet'. Try correcting the name to one that is defined, or defining the name. - undefined_identifier
🔇 Additional comments (1)
pubspec.yaml (1)
69-69: fl_lib v1.0.361 tag verified and exists.The tag is valid and already pinned in the repository. No breaking changes or compatibility issues were identified during verification.
✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.
f0e3337
flutter_markdownwithflutter_markdown_plusfl_libto v1.0.361Summary by CodeRabbit
✏️ Tip: You can customize this high-level summary in your review settings.