feat(dna): Group DNA per-group coordination DHT with Sweettest suite (#107)

* feat(dna/group): Group DNA integrity and coordinator zomes

Introduces the Group DNA as the per-group coordination layer of the
Lobby → Group → NDO hierarchy. Each group runs in its own cloned cell
(separate DHT, same DNA template) provisioned via clone_cell.

Entry types (integrity zome):
- GroupProfile — name, description; identity from action header
- GroupMembership — group_hash, optional role; member from action header
- WorkLog — group_hash, description, hours (> 0); author from action header
- SoftLink — group_hash, target_ndo_hash, description; planning-only (ADR-GROUP-04)

Coordinator externs (16):
- create_group, get_group, get_my_group, update_group (NotAuthor guard)
- join_group (AlreadyMember guard), leave_group, get_group_members, is_member
- log_work, get_work_logs, get_my_work_logs, delete_work_log (NotAuthor guard)
- create_soft_link, get_soft_links, delete_soft_link (NotAuthor guard), init

happ.yaml: group role added with deferred: true and clone_limit: 64
Shared crate: GroupError domain error enum added

* feat(dna/lobby): replace NdoAnnouncement with GroupAnnouncement

The Lobby DHT is now the registry for group cells, not NDOs directly.
NDOs are discovered through group cells (Lobby → Groups → NDOs).

- NdoAnnouncement entry type and announce_ndo, get_all_ndo_announcements,
  get_ndo_announcement, update_ndo_announcement replaced by GroupAnnouncement
  and announce_group, get_all_group_announcements, get_group_announcement_by_dna_hash
- get_my_groups now returns real GroupDescriptorStub from DHT instead of
  stub solo workspace
- AnnounceGroupInput replaces AnnounceNdoInput and UpdateNdoAnnouncementInput
  in shared crate

* test(group): Sweettest suite — 13 tests for Group DNA and Lobby update

Group DNA Sweettest (dnas/group/tests/):
- create_group_returns_profile, get_group_by_hash
- join_group_creates_membership, leave_group_removes_membership
- is_member_reflects_membership_state, duplicate_join_returns_already_member_error
- log_work_and_get_work_logs, get_my_work_logs_returns_own_logs
- create_soft_link_and_get_soft_links, get_my_group_returns_created_group
- update_group_changes_name
- empty_group_name_rejected, work_log_zero_hours_rejected

Thread limit: 13 tests × 2 conductors; --test-threads 6 required.

Lobby Sweettest updated for GroupAnnouncement API (announce_group,
get_my_group_announcements, get_group_announcement_by_dna_hash).

* feat(ui): TypeScript service layer for Group DNA

GroupService stub replaced with real callZome implementation. Interface
takes groupCellId: CellId (DnaHash + AgentPubKey) — CellId is the correct
addressing for cloned group cells (ADR-GROUP-03).

- group.types.ts: GroupProfile, GroupMembership, WorkLog, SoftLink types
  and input types; GroupGovernanceRule removed
- group.service.ts: getMembers, getWorkLogs, getSoftLinks implemented via
  callGroupZome helper; GroupServiceLive built as Effect-TS Layer
- cell.manager.ts: getGroupCellHandle added to resolve DnaHash to group CellId
- lobby.service.ts: GroupAnnouncement API wired; get_my_groups updated
- lobby.types.ts: NdoAnnouncement → GroupAnnouncement; shared-types updated

* docs(group): API reference, architecture overview, test commands

- group_zome.md: full API reference for Group DNA (entry types, link types,
  16 coordinator functions including delete_work_log and delete_soft_link,
  validation rules, error types, ADRs, test commands)
- architecture_overview.md: updated to multi-DNA hApp structure with
  Nondominium DNA / Lobby DNA / Group DNA sections
- lobby_zome.md: updated for GroupAnnouncement API
- API_REFERENCE.md, IMPLEMENTATION_STATUS.md, TEST_COMMANDS.md,
  DOCUMENTATION_INDEX.md updated for Group DNA

Author: Soushi888

Cargo.lock

@@ -14,8 +14,11 @@ members = [
     "dnas/nondominium/zomes/integrity/zome_resource",
     "dnas/nondominium/zomes/coordinator/zome_gouvernance",
     "dnas/nondominium/zomes/integrity/zome_gouvernance",
+    "dnas/group/zomes/coordinator/zome_group",
+    "dnas/group/zomes/integrity/zome_group_integrity",
     "crates/shared",
     "dnas/nondominium/tests",
+    "dnas/group/tests",
     "dnas/lobby/tests",
 ]
 # default-members excludes Sweettest crates so that
@@ -32,6 +35,8 @@ default-members = [
     "dnas/nondominium/zomes/coordinator/zome_gouvernance",
     "dnas/nondominium/zomes/integrity/zome_gouvernance",
     "dnas/nondominium/zomes/coordinator/misc",
+    "dnas/group/zomes/coordinator/zome_group",
+    "dnas/group/zomes/integrity/zome_group_integrity",
     "crates/shared",
     "dnas/lobby/zomes/coordinator/zome_lobby_coordinator",
     "dnas/lobby/zomes/integrity/zome_lobby_integrity",

Cargo.toml

@@ -165,3 +165,33 @@ impl From<GovernanceError> for WasmError {
     wasm_error!(WasmErrorInner::Guest(err.to_string()))
   }
 }
+
+// =============================================================================
+// GroupError — domain errors for zome_group
+// =============================================================================
+
+#[derive(Debug, thiserror::Error)]
+pub enum GroupError {
+  #[error("Group not found: {0}")]
+  GroupNotFound(String),
+  #[error("Already a member of this group")]
+  AlreadyMember,
+  #[error("Not a member of this group")]
+  NotMember,
+  #[error("Not the author of this entry")]
+  NotAuthor,
+  #[error("Serialization error: {0}")]
+  SerializationError(String),
+  #[error("Entry operation failed: {0}")]
+  EntryOperationFailed(String),
+  #[error("Link operation failed: {0}")]
+  LinkOperationFailed(String),
+  #[error("Invalid input: {0}")]
+  InvalidInput(String),
+}
+
+impl From<GroupError> for WasmError {
+  fn from(err: GroupError) -> Self {
+    wasm_error!(WasmErrorInner::Guest(err.to_string()))
+  }
+}

crates/shared/src/errors.rs

@@ -1,20 +1,6 @@
-use crate::types::{LifecycleStage, PropertyRegime, ResourceNature};
 use hdi::prelude::*;
 use serde::{Deserialize, Serialize};
 
-/// Input to `announce_ndo` in `zome_lobby_coordinator`.
-#[derive(Debug, Serialize, Deserialize)]
-pub struct AnnounceNdoInput {
-  pub ndo_name: String,
-  pub ndo_dna_hash: DnaHash,
-  pub network_seed: String,
-  pub ndo_identity_hash: ActionHash,
-  pub lifecycle_stage: LifecycleStage,
-  pub property_regime: PropertyRegime,
-  pub resource_nature: ResourceNature,
-  pub description: Option<String>,
-}
-
 /// Input to `upsert_lobby_agent_profile` in `zome_lobby_coordinator`.
 #[derive(Debug, Serialize, Deserialize)]
 pub struct LobbyAgentProfileInput {
@@ -23,15 +9,19 @@ pub struct LobbyAgentProfileInput {
   pub bio: Option<String>,
 }
 
-/// Input to `update_ndo_announcement` in `zome_lobby_coordinator`.
+/// Input to `announce_group` in `zome_lobby_coordinator`.
+///
+/// Follows the Lobby → Groups → NDOs hierarchy: the Lobby is the registry for groups,
+/// groups host NDOs. NDOs are discovered through group cells, not the Lobby DHT.
 #[derive(Debug, Serialize, Deserialize)]
-pub struct UpdateNdoAnnouncementInput {
-  pub original_action_hash: ActionHash,
-  pub new_lifecycle_stage: LifecycleStage,
+pub struct AnnounceGroupInput {
+  pub group_name: String,
+  pub group_dna_hash: DnaHash,
+  pub network_seed: String,
+  pub description: Option<String>,
 }
 
-/// Minimal group descriptor stub until Group DNA ships in issue #101.
-/// Used by `get_my_groups` in `zome_lobby_coordinator`.
+/// Minimal group descriptor stub returned by `get_my_groups`.
 #[derive(Debug, Serialize, Deserialize)]
 pub struct GroupDescriptorStub {
   pub id: String,

crates/shared/src/io/lobby.rs

@@ -14,7 +14,7 @@ pub mod errors;
 #[cfg(feature = "coordinator")]
 pub mod paths;
 #[cfg(feature = "coordinator")]
-pub use errors::{CommonError, GovernanceError, PersonError, ResourceError};
+pub use errors::{CommonError, GovernanceError, GroupError, PersonError, ResourceError};
 
 #[cfg(feature = "coordinator")]
 use hdk::prelude::*;
@@ -139,3 +139,4 @@ pub mod validation {
     Ok(())
   }
 }
+

crates/shared/src/lib.rs

@@ -0,0 +1,22 @@
+[package]
+name = "group_sweettest"
+version = "0.1.0"
+edition = "2021"
+
+# This crate is NOT compiled to WASM. It uses the holochain test_utils feature
+# to run Sweettest in-process against the compiled .dna bundles.
+#
+# IMPORTANT: Run with CARGO_TARGET_DIR=target/native-tests to avoid
+# artifact conflicts with the wasm32-unknown-unknown build directory.
+#
+# Prerequisites: `bun run build:happ` must be run before `cargo test`
+
+[dependencies]
+holochain                  = { workspace = true }
+tokio                      = { workspace = true }
+serde                      = { workspace = true }
+holochain_serialized_bytes = { workspace = true }
+
+[[test]]
+name = "group"
+path = "src/group/mod.rs"

dnas/group/tests/Cargo.toml

@@ -0,0 +1,42 @@
+use holochain::prelude::*;
+use holochain::sweettest::*;
+use std::sync::atomic::{AtomicU64, Ordering};
+
+/// Path to the compiled group DNA bundle, resolved relative to this crate's Cargo.toml.
+pub const GROUP_DNA_PATH: &str = concat!(
+    env!("CARGO_MANIFEST_DIR"),
+    "/../workdir/group.dna"
+);
+
+// Each test invocation gets a unique monotonic ID. Combined with the process PID this
+// guarantees distinct network seeds even when multiple test processes run in parallel.
+static TEST_INSTANCE: AtomicU64 = AtomicU64::new(0);
+
+pub fn unique_seed() -> NetworkSeed {
+    let id = TEST_INSTANCE.fetch_add(1, Ordering::SeqCst);
+    format!("test-{}-{}", std::process::id(), id).into()
+}
+
+/// Spin up two conductors, each with the group DNA installed.
+///
+/// Returns `(conductors, cell_alice, cell_bob)`.
+pub async fn setup_two_agents() -> (SweetConductorBatch, SweetCell, SweetCell) {
+    let mut conductors =
+        SweetConductorBatch::from_config_rendezvous(2, SweetConductorConfig::standard()).await;
+
+    let dna = SweetDnaFile::from_bundle(std::path::Path::new(GROUP_DNA_PATH))
+        .await
+        .expect("Failed to load group DNA bundle. Did you run `bun run build:happ`?")
+        .with_network_seed(unique_seed())
+        .await;
+
+    let apps = conductors
+        .setup_app("group", &[dna])
+        .await
+        .expect("Failed to install group app on conductors");
+
+    conductors.exchange_peer_info().await;
+
+    let ((cell_alice,), (cell_bob,)) = apps.into_tuples();
+    (conductors, cell_alice, cell_bob)
+}

dnas/group/tests/src/common/conductors.rs

@@ -0,0 +1,2 @@
+pub mod conductors;
+pub use conductors::*;