[PR #7633] [MERGED] feat: add bloc observer #8011

New issue

Closed

opened 2026-03-23 23:22:12 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:22:12 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7633
Author: @LucasXu0
Created: 3/27/2025
Status: ✅ Merged
Merged: 3/27/2025
Merged by: @LucasXu0

Base: main ← Head: bloc_observer

📝 Commits (4)

89230e2 chore: bump version 0.8.8
e1e1c2f feat: add bloc observer
d6b95cc chore: update comment
cb0e65c chore: update comment

📊 Changes

18 files changed (+172 additions, -223 deletions)

View changed files

📝 frontend/Makefile.toml (+1 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/callout/callout_block_component.dart (+1 -7)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_node_extension.dart (+22 -2)
📝 frontend/appflowy_flutter/lib/startup/startup.dart (+1 -1)
📝 frontend/appflowy_flutter/lib/startup/tasks/app_widget.dart (+0 -10)
📝 frontend/appflowy_flutter/lib/startup/tasks/debug_task.dart (+31 -4)
📝 frontend/appflowy_flutter/lib/workspace/application/sidebar/space/space_bloc.dart (+0 -12)
📝 frontend/appflowy_flutter/macos/Podfile.lock (+23 -23)
📝 frontend/appflowy_flutter/packages/appflowy_backend/example/macos/Runner/AppDelegate.swift (+5 -1)
📝 frontend/appflowy_flutter/packages/appflowy_backend/lib/appflowy_backend.dart (+2 -15)
📝 frontend/appflowy_flutter/packages/appflowy_backend/lib/log.dart (+33 -46)
📝 frontend/appflowy_flutter/packages/appflowy_backend/pubspec.yaml (+1 -1)
📝 frontend/appflowy_flutter/packages/appflowy_result/pubspec.yaml (+1 -38)
📝 frontend/appflowy_flutter/packages/flowy_infra/pubspec.yaml (+2 -39)
📝 frontend/appflowy_flutter/packages/flowy_infra_ui/flowy_infra_ui_platform_interface/pubspec.yaml (+1 -3)
📝 frontend/appflowy_flutter/packages/flowy_infra_ui/flowy_infra_ui_web/pubspec.yaml (+2 -2)
📝 frontend/appflowy_flutter/pubspec.lock (+38 -14)
📝 frontend/appflowy_flutter/pubspec.yaml (+8 -4)

📄 Description

Feature Preview

Some blocs have potential assertion or state error issues. Add bloc observer to trace their state.
Replace logger with talker to get a better message reading experience.

PR Checklist

My code adheres to AppFlowy's Conventions
I've listed at least one issue that this PR fixes in the description above.
I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes.
All existing tests are passing.

Summary by Sourcery

Replace logger with Talker for improved logging and add Bloc observer for better state tracing in debug mode

New Features:

Integrate Talker for enhanced logging capabilities
Add Bloc observer to trace bloc state changes in debug mode

Enhancements:

Upgrade Bloc dependencies to latest versions
Improve log filtering and readability
Add more robust error and state logging mechanisms

Chores:

Update project version to 0.8.8
Add homepage URLs to some package configurations

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/AppFlowy-IO/AppFlowy/pull/7633 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 3/27/2025 **Status:** ✅ Merged **Merged:** 3/27/2025 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `bloc_observer` --- ### 📝 Commits (4) - [`89230e2`](https://github.com/AppFlowy-IO/AppFlowy/commit/89230e2dc1fc4bdba26ba961242d9527282046ae) chore: bump version 0.8.8 - [`e1e1c2f`](https://github.com/AppFlowy-IO/AppFlowy/commit/e1e1c2f1e1b09d079700a57d35262d5d34d4a99a) feat: add bloc observer - [`d6b95cc`](https://github.com/AppFlowy-IO/AppFlowy/commit/d6b95ccdd9a97a53a2fb645b36c6adc0265aef2a) chore: update comment - [`cb0e65c`](https://github.com/AppFlowy-IO/AppFlowy/commit/cb0e65cd7e542e5271a4b8a770d481eff010a4b6) chore: update comment ### 📊 Changes **18 files changed** (+172 additions, -223 deletions) <details> <summary>View changed files</summary> 📝 `frontend/Makefile.toml` (+1 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/callout/callout_block_component.dart` (+1 -7) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/simple_table/simple_table_operations/simple_table_node_extension.dart` (+22 -2) 📝 `frontend/appflowy_flutter/lib/startup/startup.dart` (+1 -1) 📝 `frontend/appflowy_flutter/lib/startup/tasks/app_widget.dart` (+0 -10) 📝 `frontend/appflowy_flutter/lib/startup/tasks/debug_task.dart` (+31 -4) 📝 `frontend/appflowy_flutter/lib/workspace/application/sidebar/space/space_bloc.dart` (+0 -12) 📝 `frontend/appflowy_flutter/macos/Podfile.lock` (+23 -23) 📝 `frontend/appflowy_flutter/packages/appflowy_backend/example/macos/Runner/AppDelegate.swift` (+5 -1) 📝 `frontend/appflowy_flutter/packages/appflowy_backend/lib/appflowy_backend.dart` (+2 -15) 📝 `frontend/appflowy_flutter/packages/appflowy_backend/lib/log.dart` (+33 -46) 📝 `frontend/appflowy_flutter/packages/appflowy_backend/pubspec.yaml` (+1 -1) 📝 `frontend/appflowy_flutter/packages/appflowy_result/pubspec.yaml` (+1 -38) 📝 `frontend/appflowy_flutter/packages/flowy_infra/pubspec.yaml` (+2 -39) 📝 `frontend/appflowy_flutter/packages/flowy_infra_ui/flowy_infra_ui_platform_interface/pubspec.yaml` (+1 -3) 📝 `frontend/appflowy_flutter/packages/flowy_infra_ui/flowy_infra_ui_web/pubspec.yaml` (+2 -2) 📝 `frontend/appflowy_flutter/pubspec.lock` (+38 -14) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+8 -4) </details> ### 📄 Description   ### Feature Preview - Some blocs have potential assertion or state error issues. Add bloc observer to trace their state. - Replace logger with talker to get a better message reading experience.  ---  #### PR Checklist - [x] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/conventions) - [x] I've listed at least one issue that this PR fixes in the description above. - [x] I've added a test(s) to validate changes in this PR, or this PR only contains semantic changes. - [x] All existing tests are passing. ## Summary by Sourcery Replace logger with Talker for improved logging and add Bloc observer for better state tracing in debug mode New Features: - Integrate Talker for enhanced logging capabilities - Add Bloc observer to trace bloc state changes in debug mode Enhancements: - Upgrade Bloc dependencies to latest versions - Improve log filtering and readability - Add more robust error and state logging mechanisms Chores: - Update project version to 0.8.8 - Add homepage URLs to some package configurations --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>