[PR #6856] [MERGED] feat: support skipping in-memory update transaction #7608

New issue

Closed

opened 2026-03-23 23:20:25 +00:00 by mirror · 0 comments

mirror commented

2026-03-23 23:20:25 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/6856
Author: @LucasXu0
Created: 11/25/2024
Status: ✅ Merged
Merged: 11/25/2024
Merged by: @LucasXu0

Base: main ← Head: fix_845

📝 Commits (9)

8a232d9 feat: support skipping in-memory update transaction
6c4790d fix: flutter analyze
a29e1ff feat: add sentence mode
26e08a3 test: support skipping in-memory update transaction
105cfc8 test: add sentence mode
c913a0e test: add sentence mode (2)
70e4e0d chore: set enableDocumentInternalLog to false
58dee45 fix: integration test
5c8c600 fix: integration test

📊 Changes

14 files changed (+725 additions, -53 deletions)

View changed files

➕ frontend/appflowy_flutter/integration_test/desktop/cloud/document/document_ai_writer_test.dart (+47 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/cloud/document/document_test_runner.dart (+2 -0)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_image_block_test.dart (+0 -1)
📝 frontend/appflowy_flutter/integration_test/desktop/document/document_with_multi_image_block_test.dart (+2 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/application/document_bloc.dart (+6 -0)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart (+2 -1)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/text_robot.dart (+60 -25)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/openai/widgets/auto_completion_node_widget.dart (+17 -4)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/outline/outline_block_component.dart (+1 -2)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/slash_menu/slash_menu_items.dart (+26 -8)
📝 frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/transaction_handler/editor_transaction_service.dart (+7 -4)
📝 frontend/appflowy_flutter/pubspec.lock (+4 -4)
📝 frontend/appflowy_flutter/pubspec.yaml (+2 -2)
➕ frontend/appflowy_flutter/test/unit_test/document/text_robot/text_robot_test.dart (+549 -0)

📄 Description

Feature Preview

skip the auto completion block update. (only in memory)
test
add sentence mode for text robot
test

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.

_{🔄 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/6856 **Author:** [@LucasXu0](https://github.com/LucasXu0) **Created:** 11/25/2024 **Status:** ✅ Merged **Merged:** 11/25/2024 **Merged by:** [@LucasXu0](https://github.com/LucasXu0) **Base:** `main` ← **Head:** `fix_845` --- ### 📝 Commits (9) - [`8a232d9`](https://github.com/AppFlowy-IO/AppFlowy/commit/8a232d96cdaa6e27d0de711f4dd6657699bcb4a8) feat: support skipping in-memory update transaction - [`6c4790d`](https://github.com/AppFlowy-IO/AppFlowy/commit/6c4790d66e9ec1c07aed40600636eecc11e372f1) fix: flutter analyze - [`a29e1ff`](https://github.com/AppFlowy-IO/AppFlowy/commit/a29e1ffd6112d0c8cf19220f2d9d457a209818fe) feat: add sentence mode - [`26e08a3`](https://github.com/AppFlowy-IO/AppFlowy/commit/26e08a338873a66cfb8f3a7f28f87337af7e493a) test: support skipping in-memory update transaction - [`105cfc8`](https://github.com/AppFlowy-IO/AppFlowy/commit/105cfc8e05f5f561def9ffe0fd970ed210403b74) test: add sentence mode - [`c913a0e`](https://github.com/AppFlowy-IO/AppFlowy/commit/c913a0e52c4b4029dc388b259bdacaff557f28c9) test: add sentence mode (2) - [`70e4e0d`](https://github.com/AppFlowy-IO/AppFlowy/commit/70e4e0d2644a2600fbf4eb244cbcb9810e387e1a) chore: set enableDocumentInternalLog to false - [`58dee45`](https://github.com/AppFlowy-IO/AppFlowy/commit/58dee45a061e6e27e9dfee9fc45a54ed3c2205e0) fix: integration test - [`5c8c600`](https://github.com/AppFlowy-IO/AppFlowy/commit/5c8c6002de1ca2cfafe68963a891f2e21e90b4ec) fix: integration test ### 📊 Changes **14 files changed** (+725 additions, -53 deletions) <details> <summary>View changed files</summary> ➕ `frontend/appflowy_flutter/integration_test/desktop/cloud/document/document_ai_writer_test.dart` (+47 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/cloud/document/document_test_runner.dart` (+2 -0) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_image_block_test.dart` (+0 -1) 📝 `frontend/appflowy_flutter/integration_test/desktop/document/document_with_multi_image_block_test.dart` (+2 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/application/document_bloc.dart` (+6 -0) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_page.dart` (+2 -1) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/base/text_robot.dart` (+60 -25) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/openai/widgets/auto_completion_node_widget.dart` (+17 -4) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/outline/outline_block_component.dart` (+1 -2) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/slash_menu/slash_menu_items.dart` (+26 -8) 📝 `frontend/appflowy_flutter/lib/plugins/document/presentation/editor_plugins/transaction_handler/editor_transaction_service.dart` (+7 -4) 📝 `frontend/appflowy_flutter/pubspec.lock` (+4 -4) 📝 `frontend/appflowy_flutter/pubspec.yaml` (+2 -2) ➕ `frontend/appflowy_flutter/test/unit_test/document/text_robot/text_robot_test.dart` (+549 -0) </details> ### 📄 Description   ### Feature Preview - [x] skip the auto completion block update. (only in memory) - [x] test - [x] add sentence mode for text robot - [x] test  ---  #### 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. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>