feat(rust/account_groups chat_list): add pin_order attribute and set_chat_pin_order method

[jgmontoya]

Description

Adds the pin_order attribute to chat summary and accounts_groups as well as adds the set_chat_pin_order method so the app can set this value as needed

Type of Change

• ✨ New feature (non-breaking change which adds functionality)
• 🛠️ Bug fix (non-breaking change which fixes an issue)
• ❌ Breaking change (fix or feature that would cause existing functionality to change)
• 🧹 Code refactor
• ✅ Build configuration change
• 📝 Documentation
• 🗑️ Chore
• 🧪 Tests

Checklist

• Run just precommit to ensure that formatting and linting are correct
• Updated the CHANGELOG.md file with your changes (if they affect the user experience)

Summary by CodeRabbit

• New Features

  • Chat pinning: pin/unpin conversations and set a custom pin order.
  • New API to update a chat's pin order so ordering changes take effect immediately.

• Behavior Changes

  • Chat list ordering: pinned chats first (by pin order), then recent unpinned chats, then groups with no messages (by creation date).

• Data Model

  • Account and chat summaries now surface pin order for pinned state.

• Documentation

  • Sorting semantics documented for clarity.

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

[coderabbitai]

Walkthrough

Adds chat pinning support: introduces pinOrder/pin_order fields to AccountGroup and ChatSummary across Dart and Rust, adds a new RPC/method to set a chat's pin order (Dart FFI -> Rust -> whitenoise), and updates generated encoders/decoders and dispatcher funcIds.

Changes

Cohort / File(s)	Summary
Dart API Models & FFI lib/src/rust/api/account_groups.dart, lib/src/rust/api/chat_list.dart, lib/src/rust/frb_generated.dart	Added pinOrder to AccountGroup and ChatSummary; updated constructors, hashCode, and operator==; added setChatPinOrder(...) API and corresponding FFI/task meta; adjusted generated encode/decode usage and funcId wiring.
Rust API Types & Logic rust/src/api/account_groups.rs, rust/src/api/chat_list.rs	Added pin_order: Option<i64> to AccountGroup and ChatSummary; propagate from whitenoise conversions; added pub async fn set_chat_pin_order(...) endpoint and updated chat-list sorting documentation.
Generated Rust Bridge rust/src/frb_generated.rs, rust/src/frb_generated.rs (wire impl)	Added wire handler for set_chat_pin_order; updated SseEncode/SseDecode and IntoDart flows to include pin_order; shifted dispatcher funcIds and updated codegen content hash.
Misc Rust Changes rust/src/api/groups.rs, rust/Cargo.toml	Added PartialEq, Eq derives to GroupType; bumped whitenoise git rev in Cargo.toml.

Sequence Diagram

sequenceDiagram
    actor Client as Dart Client
    participant FFI as FFI Bridge
    participant RustAPI as Rust API
    participant WN as Whitenoise

    Client->>FFI: setChatPinOrder(accountPubkey, mlsGroupId, pinOrder)
    FFI->>FFI: Marshal args (funcId 57)
    FFI->>RustAPI: Invoke set_chat_pin_order(account_pubkey, mls_group_id, pin_order)
    RustAPI->>RustAPI: Validate inputs & await
    RustAPI->>WN: whitenoise.set_chat_pin_order(account_pubkey, mls_group_id, pin_order)
    WN-->>RustAPI: Result/OK
    RustAPI-->>FFI: Return void
    FFI-->>Client: Complete Future<void>

Loading

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~25 minutes

Possibly related PRs

• Show chats in chat list and messages in chat invite screen #15: Modifies AccountGroup and ChatSummary and FRB bindings—likely overlaps in model and wire changes.
• feat(rust/chat-list): extract chat_list module and add chat list streaming capabilities #21: Changes chat-list API and generated bindings; may conflict in chat-list and FRB encoder/decoder updates.

Suggested reviewers

• untreu2
• josefinalliende

🚥 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 accurately describes the main changes: adding pin_order attribute to account_groups and chat_list, plus introducing set_chat_pin_order method.
Docstring Coverage	✅ Passed	Docstring coverage is 93.75% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing touches

• 📝 Generate docstrings

---

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

• X
• Mastodon
• Reddit
• LinkedIn

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

[github-actions]

➡️ Coverage: 99.58% → 99.58% (0.00%)

[coderabbitai]

Actionable comments posted: 1

Caution

Some comments are outside the diff and can’t be posted inline due to platform limitations.

⚠️ Outside diff range comments (1)

rust/src/api/account_groups.rs (1)

18-33: Use #[frb(non_opaque)] for AccountGroup (required for Flutter compatibility).

This struct is part of the Rust API surface and should follow the project guideline for non-opaque FRB structs.

♻️ Proposed fix

-#[frb]
+#[frb(non_opaque)]
 #[derive(Debug, Clone)]
 pub struct AccountGroup {

As per coding guidelines, apply #[frb(non_opaque)] on structs in rust/src/api/**/*.rs.

🤖 Fix all issues with AI agents

In `@rust/src/api/chat_list.rs`:
- Around line 122-153: Reorder input parsing in set_chat_pin_order so all inputs
are parsed before performing lookups: call Whitenoise::get_instance() first,
then parse account_pubkey with PublicKey::parse and decode mls_group_id with
hex::decode / GroupId::from_slice, and only after those parses call
whitenoise.find_account_by_pubkey(&pubkey).await and then
whitenoise.set_chat_pin_order(&account, &group_id, pin_order).await; keep
existing error handling and return types intact.

[github-actions]

➡️ Coverage: 99.58% → 99.58% (0.00%)

[josefinalliende]

It's not related to this PR but it is a blocker for merge: Since this commit cc99f378bb80707005fc81191697f423910d28c0 in the rust crate, the signup turns to be extremely slow... from 2s to above 20s....

marmot-protocol/whitenoise-rs@cc99f37

[github-actions]

➡️ Coverage: 99.6% → 99.6% (0.00%)