Skip to content

bump: Replace discontinued deps and update fl_lib dep#1024

Merged
GT-610 merged 4 commits intolollipopkit:mainfrom
GT-610:bump
Jan 24, 2026
Merged

bump: Replace discontinued deps and update fl_lib dep#1024
GT-610 merged 4 commits intolollipopkit:mainfrom
GT-610:bump

Conversation

@GT-610
Copy link
Copy Markdown
Collaborator

@GT-610 GT-610 commented Jan 24, 2026

  • Replace flutter_markdown with flutter_markdown_plus
  • Update fl_lib to v1.0.361

Summary by CodeRabbit

  • Chores
    • Replaced the project's markdown rendering dependency with an updated markdown package.
    • Added the new markdown package to project dependencies.
    • Updated a supporting library to a newer release for improved compatibility.

✏️ Tip: You can customize this high-level summary in your review settings.

…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
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jan 24, 2026

📝 Walkthrough

Walkthrough

Replaced imports of flutter_markdown with flutter_markdown_plus in two view files and bumped the fl_lib git ref from v1.0.359 to v1.0.361 in pubspec.yaml.

Changes

Cohort / File(s) Summary
Markdown import updates
lib/view/page/server/detail/view.dart, lib/view/page/snippet/edit.dart
Replaced imports from flutter_markdown/flutter_markdown.dart to flutter_markdown_plus/flutter_markdown_plus.dart; no other code changes observed
Dependency update
pubspec.yaml
Updated git ref for fl_lib from v1.0.359v1.0.361

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested reviewers

  • lollipopkit
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'bump: Replace discontinued deps and update fl_lib dep' accurately describes the main changes: replacing flutter_markdown with flutter_markdown_plus (discontinued dependency replacement) and updating fl_lib from v1.0.359 to v1.0.361.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 47e5e9f and f0e3337.

⛔ Files ignored due to path filters (1)
  • pubspec.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • pubspec.yaml
🚧 Files skipped from review as they are similar to previous changes (1)
  • pubspec.yaml

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

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_plus to pubspec.yaml (and remove flutter_markdown if present) so this import resolves.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e49b31e and a030be5.

⛔ Files ignored due to path filters (1)
  • pubspec.lock is excluded by !**/*.lock
📒 Files selected for processing (3)
  • lib/view/page/server/detail/view.dart
  • lib/view/page/snippet/edit.dart
  • pubspec.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.dart
  • lib/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.dart
  • lib/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.dart
  • lib/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.dart
  • lib/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.

@coderabbitai coderabbitai bot requested a review from lollipopkit January 24, 2026 06:34
coderabbitai[bot]
coderabbitai bot previously approved these changes Jan 24, 2026
lollipopkit
lollipopkit previously approved these changes Jan 24, 2026
@GT-610 GT-610 dismissed stale reviews from lollipopkit and coderabbitai[bot] via f0e3337 January 24, 2026 06:49
@coderabbitai coderabbitai bot requested a review from lollipopkit January 24, 2026 06:49
@GT-610 GT-610 merged commit 7d47c9d into lollipopkit:main Jan 24, 2026
2 checks passed
@GT-610 GT-610 deleted the bump branch January 24, 2026 07:04
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.

2 participants