[PR #7699] [MERGED] Chore: Use UUID instead of using String #8050

New issue

Closed

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

mirror commented

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

Owner

📋 Pull Request Information

Original PR: https://github.com/AppFlowy-IO/AppFlowy/pull/7699
Author: @appflowy
Created: 4/7/2025
Status: ✅ Merged
Merged: 4/7/2025
Merged by: @appflowy

Base: main ← Head: use_uuid

📝 Commits (4)

995b773 chore: replace str with uuid
91397c9 chore: clippy
24d5733 chore: bump collab id
20bcdd1 chore: fmt

📊 Changes

103 files changed (+1604 additions, -1425 deletions)

View changed files

📝 frontend/rust-lib/Cargo.lock (+33 -55)
📝 frontend/rust-lib/Cargo.toml (+10 -10)
📝 frontend/rust-lib/collab-integrate/Cargo.toml (+1 -2)
📝 frontend/rust-lib/collab-integrate/src/collab_builder.rs (+15 -12)
📝 frontend/rust-lib/collab-integrate/src/persistence/collab_metadata_sql.rs (+11 -4)
📝 frontend/rust-lib/event-integration-test/src/document/document_event.rs (+3 -3)
📝 frontend/rust-lib/event-integration-test/src/folder_event.rs (+6 -3)
📝 frontend/rust-lib/event-integration-test/src/lib.rs (+10 -3)
📝 frontend/rust-lib/event-integration-test/tests/chat/chat_message_test.rs (+6 -4)
📝 frontend/rust-lib/event-integration-test/tests/document/local_test/edit_test.rs (+4 -2)
📝 frontend/rust-lib/event-integration-test/tests/folder/local_test/folder_test.rs (+4 -4)
📝 frontend/rust-lib/event-integration-test/tests/folder/local_test/publish_document_test.rs (+13 -12)
📝 frontend/rust-lib/event-integration-test/tests/user/migration_test/version_test.rs (+0 -35)
📝 frontend/rust-lib/flowy-ai-pub/Cargo.toml (+2 -2)
📝 frontend/rust-lib/flowy-ai-pub/src/cloud.rs (+29 -28)
📝 frontend/rust-lib/flowy-ai/src/ai_manager.rs (+50 -39)
📝 frontend/rust-lib/flowy-ai/src/chat.rs (+18 -17)
📝 frontend/rust-lib/flowy-ai/src/completion.rs (+60 -54)
📝 frontend/rust-lib/flowy-ai/src/entities.rs (+4 -4)
📝 frontend/rust-lib/flowy-ai/src/event_handler.rs (+22 -13)

...and 80 more files

📄 Description

Feature Preview

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

Migrate workspace and view identifiers from string to UUID across the AppFlowy codebase

Enhancements:

Replaced string-based workspace and view identifiers with UUID throughout the project
Updated type signatures to use Uuid instead of String for improved type safety
Refactored code to handle UUID conversions and parsing

Chores:

Updated multiple Cargo.toml files to include uuid dependency
Removed unnecessary dependencies
Standardized UUID handling across different modules

_{🔄 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/7699 **Author:** [@appflowy](https://github.com/appflowy) **Created:** 4/7/2025 **Status:** ✅ Merged **Merged:** 4/7/2025 **Merged by:** [@appflowy](https://github.com/appflowy) **Base:** `main` ← **Head:** `use_uuid` --- ### 📝 Commits (4) - [`995b773`](https://github.com/AppFlowy-IO/AppFlowy/commit/995b773c7404abe9687c619033123d367e56cf93) chore: replace str with uuid - [`91397c9`](https://github.com/AppFlowy-IO/AppFlowy/commit/91397c963a4d63ead18828dc9f9f67bbc24b3b1e) chore: clippy - [`24d5733`](https://github.com/AppFlowy-IO/AppFlowy/commit/24d57336a979829ba6850f0ddf7a9a125bd98c3d) chore: bump collab id - [`20bcdd1`](https://github.com/AppFlowy-IO/AppFlowy/commit/20bcdd1f904e67cf322521a10ae14ad2edb9325e) chore: fmt ### 📊 Changes **103 files changed** (+1604 additions, -1425 deletions) <details> <summary>View changed files</summary> 📝 `frontend/rust-lib/Cargo.lock` (+33 -55) 📝 `frontend/rust-lib/Cargo.toml` (+10 -10) 📝 `frontend/rust-lib/collab-integrate/Cargo.toml` (+1 -2) 📝 `frontend/rust-lib/collab-integrate/src/collab_builder.rs` (+15 -12) 📝 `frontend/rust-lib/collab-integrate/src/persistence/collab_metadata_sql.rs` (+11 -4) 📝 `frontend/rust-lib/event-integration-test/src/document/document_event.rs` (+3 -3) 📝 `frontend/rust-lib/event-integration-test/src/folder_event.rs` (+6 -3) 📝 `frontend/rust-lib/event-integration-test/src/lib.rs` (+10 -3) 📝 `frontend/rust-lib/event-integration-test/tests/chat/chat_message_test.rs` (+6 -4) 📝 `frontend/rust-lib/event-integration-test/tests/document/local_test/edit_test.rs` (+4 -2) 📝 `frontend/rust-lib/event-integration-test/tests/folder/local_test/folder_test.rs` (+4 -4) 📝 `frontend/rust-lib/event-integration-test/tests/folder/local_test/publish_document_test.rs` (+13 -12) 📝 `frontend/rust-lib/event-integration-test/tests/user/migration_test/version_test.rs` (+0 -35) 📝 `frontend/rust-lib/flowy-ai-pub/Cargo.toml` (+2 -2) 📝 `frontend/rust-lib/flowy-ai-pub/src/cloud.rs` (+29 -28) 📝 `frontend/rust-lib/flowy-ai/src/ai_manager.rs` (+50 -39) 📝 `frontend/rust-lib/flowy-ai/src/chat.rs` (+18 -17) 📝 `frontend/rust-lib/flowy-ai/src/completion.rs` (+60 -54) 📝 `frontend/rust-lib/flowy-ai/src/entities.rs` (+4 -4) 📝 `frontend/rust-lib/flowy-ai/src/event_handler.rs` (+22 -13) _...and 80 more files_ </details> ### 📄 Description   ### Feature Preview  ---  #### PR Checklist - [ ] My code adheres to [AppFlowy's Conventions](https://docs.appflowy.io/docs/documentation/software-contributions/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 Migrate workspace and view identifiers from string to UUID across the AppFlowy codebase Enhancements: - Replaced string-based workspace and view identifiers with UUID throughout the project - Updated type signatures to use Uuid instead of String for improved type safety - Refactored code to handle UUID conversions and parsing Chores: - Updated multiple Cargo.toml files to include uuid dependency - Removed unnecessary dependencies - Standardized UUID handling across different modules --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>