neoagent 3.2.1-beta.9 → 3.3.0

package/flutter_app/lib/main_controller.dart

@@ -132,6 +132,7 @@ class NeoAgentController extends ChangeNotifier {
       const <LinkedAuthProviderItem>[];
   QrLoginChallenge? qrLoginChallenge;
   Map<String, dynamic> settings = const <String, dynamic>{};
+  Map<String, dynamic> behaviorConfig = const <String, dynamic>{};
   Map<String, dynamic>? versionInfo;
   Map<String, dynamic>? backendHealthStatus;
   HealthBridgeStatus? deviceHealthStatus;
@@ -1420,6 +1421,7 @@ class NeoAgentController extends ChangeNotifier {
     usageAndLimits = null;
     linkedAuthProviders = const <LinkedAuthProviderItem>[];
     settings = const <String, dynamic>{};
+    behaviorConfig = const <String, dynamic>{};
     chatMessages = const <ChatEntry>[];
     _resetChatHistoryPagination();
     agentProfiles = const <AgentProfile>[];
@@ -1764,6 +1766,9 @@ class NeoAgentController extends ChangeNotifier {
         );
       case 'sharedActorRules':
         return policy.copyWith(
+          directPolicy: policy.directPolicy == 'disabled'
+              ? 'allowlist'
+              : policy.directPolicy,
           sharedPolicy: policy.sharedPolicy == 'disabled'
               ? 'allowlist'
               : policy.sharedPolicy,
@@ -1772,6 +1777,16 @@ class NeoAgentController extends ChangeNotifier {
             suggestion.rule,
           ]),
         );
+      case 'sharedMemberRules':
+        return policy.copyWith(
+          sharedPolicy: policy.sharedPolicy == 'disabled'
+              ? 'allowlist'
+              : policy.sharedPolicy,
+          sharedMemberRules: _dedupeAccessRules(<MessagingAccessRule>[
+            ...policy.sharedMemberRules,
+            suggestion.rule,
+          ]),
+        );
       default:
         return policy.copyWith(
           sharedPolicy: policy.sharedPolicy == 'disabled'
@@ -1958,6 +1973,11 @@ class NeoAgentController extends ChangeNotifier {
         _backendClient.fetchSettings(backendUrl, agentId: agentId),
         const <String, dynamic>{},
       );
+      final behaviorFuture = _softRefreshLoad<Map<String, dynamic>>(
+        'behavior_config',
+        _backendClient.fetchBehaviorConfig(backendUrl, agentId: agentId),
+        const <String, dynamic>{},
+      );
       final runsFuture = _softRefreshLoad<Map<String, dynamic>>(
         'runs',
         _backendClient.fetchRuns(backendUrl, agentId: agentId),
@@ -2094,6 +2114,7 @@ class NeoAgentController extends ChangeNotifier {
       final modelsResponse = await modelsFuture;
       final providersResponse = await providersFuture;
       final settingsResponse = await settingsFuture;
+      final behaviorResponse = await behaviorFuture;
       final runsResponse = await runsFuture;
       final timelineResponse = await timelineFuture;
       final versionResponse = await versionFuture;
@@ -2137,6 +2158,9 @@ class NeoAgentController extends ChangeNotifier {
       );
 
       settings = Map<String, dynamic>.from(settingsResponse);
+      behaviorConfig = behaviorResponse['config'] is Map
+          ? Map<String, dynamic>.from(behaviorResponse['config'] as Map)
+          : const <String, dynamic>{};
       recentRuns = _decodeModelList(
         'runs',
         runsResponse['runs'],
@@ -4221,6 +4245,27 @@ class NeoAgentController extends ChangeNotifier {
     }
   }
 
+  Future<void> saveBehaviorConfig(Map<String, dynamic> config) async {
+    isSavingSettings = true;
+    errorMessage = null;
+    notifyListeners();
+    try {
+      final response = await _backendClient.saveBehaviorConfig(
+        backendUrl,
+        config,
+        agentId: _scopedAgentId,
+      );
+      behaviorConfig = response['config'] is Map
+          ? Map<String, dynamic>.from(response['config'] as Map)
+          : Map<String, dynamic>.from(config);
+    } catch (error) {
+      errorMessage = _friendlyErrorMessage(error);
+    } finally {
+      isSavingSettings = false;
+      notifyListeners();
+    }
+  }
+
   void _applyAccountResponse(Map<String, dynamic> response) {
     if (response['user'] is Map) {
       user = Map<String, dynamic>.from(response['user'] as Map);
@@ -5338,10 +5383,20 @@ class NeoAgentController extends ChangeNotifier {
   }
 
   Future<void> updateAssistantBehaviorNotes(String content) async {
-    await _backendClient.saveSettings(backendUrl, <String, dynamic>{
-      'assistant_behavior_notes': content,
-    }, agentId: _scopedAgentId);
-    await refreshMemory();
+    isSavingSettings = true;
+    errorMessage = null;
+    notifyListeners();
+    try {
+      await _backendClient.saveSettings(backendUrl, <String, dynamic>{
+        'assistant_behavior_notes': content,
+      }, agentId: _scopedAgentId);
+      await refreshMemory();
+    } catch (error) {
+      errorMessage = _friendlyErrorMessage(error);
+    } finally {
+      isSavingSettings = false;
+      notifyListeners();
+    }
   }
 
   Future<void> updateCoreMemory(String key, String value) async {

package/flutter_app/lib/main_models.dart

@@ -719,6 +719,9 @@ class MessagingAccessRule {
     required this.scope,
     required this.value,
     this.label,
+    this.spaceScope,
+    this.spaceValue,
+    this.spaceLabel,
   });
 
   factory MessagingAccessRule.fromJson(Map<String, dynamic> json) {
@@ -726,23 +729,38 @@ class MessagingAccessRule {
       scope: json['scope']?.toString() ?? 'chat',
       value: json['value']?.toString() ?? '',
       label: json['label']?.toString(),
+      spaceScope: json['spaceScope']?.toString(),
+      spaceValue: json['spaceValue']?.toString(),
+      spaceLabel: json['spaceLabel']?.toString(),
     );
   }
 
   final String scope;
   final String value;
   final String? label;
+  final String? spaceScope;
+  final String? spaceValue;
+  final String? spaceLabel;
 
   Map<String, dynamic> toJson() => <String, dynamic>{
     'scope': scope,
     'value': value,
     if (label != null && label!.trim().isNotEmpty) 'label': label,
+    if (spaceScope != null && spaceScope!.trim().isNotEmpty)
+      'spaceScope': spaceScope,
+    if (spaceValue != null && spaceValue!.trim().isNotEmpty)
+      'spaceValue': spaceValue,
+    if (spaceLabel != null && spaceLabel!.trim().isNotEmpty)
+      'spaceLabel': spaceLabel,
   };
 
-  String get id => '$scope:$value';
+  String get id => '$scope:$value:${spaceScope ?? ''}:${spaceValue ?? ''}';
 
   String get displayLabel => label?.ifEmpty(value) ?? value;
 
+  String get spaceDisplayLabel =>
+      spaceLabel?.ifEmpty(spaceValue ?? '') ?? spaceValue ?? '';
+
   String get scopeLabel {
     switch (scope) {
       case 'phone_number':
@@ -767,23 +785,63 @@ class MessagingAccessRule {
   }
 }
 
+class MessagingSharedParticipationRule {
+  const MessagingSharedParticipationRule({
+    required this.scope,
+    required this.value,
+    required this.allowUntagged,
+    this.label,
+  });
+
+  factory MessagingSharedParticipationRule.fromJson(Map<String, dynamic> json) {
+    return MessagingSharedParticipationRule(
+      scope: json['scope']?.toString() ?? 'chat',
+      value: json['value']?.toString() ?? '',
+      allowUntagged: json['allowUntagged'] == true,
+      label: json['label']?.toString(),
+    );
+  }
+
+  final String scope;
+  final String value;
+  final bool allowUntagged;
+  final String? label;
+
+  String get id => '$scope:$value';
+  String get displayLabel => label?.ifEmpty(value) ?? value;
+
+  Map<String, dynamic> toJson() => <String, dynamic>{
+    'scope': scope,
+    'value': value,
+    'allowUntagged': allowUntagged,
+    if (label != null && label!.trim().isNotEmpty) 'label': label,
+  };
+}
+
 class MessagingAccessPolicy {
   const MessagingAccessPolicy({
+    required this.schemaVersion,
     required this.directPolicy,
     required this.sharedPolicy,
-    required this.requireMentionInShared,
+    required this.defaultAllowUntaggedInShared,
     required this.directRules,
     required this.sharedSpaceRules,
     required this.sharedActorRules,
+    required this.sharedMemberRules,
+    required this.sharedParticipationRules,
   });
 
   factory MessagingAccessPolicy.fromJson(Map<String, dynamic> json) {
+    final schemaVersion = (json['schemaVersion'] as num?)?.toInt() ?? 2;
     return MessagingAccessPolicy(
+      schemaVersion: schemaVersion,
       directPolicy:
           json['directPolicy']?.toString().ifEmpty('allowlist') ?? 'allowlist',
       sharedPolicy:
           json['sharedPolicy']?.toString().ifEmpty('allowlist') ?? 'allowlist',
-      requireMentionInShared: json['requireMentionInShared'] == true,
+      defaultAllowUntaggedInShared: schemaVersion < 3
+          ? json['requireMentionInShared'] != true
+          : json['defaultAllowUntaggedInShared'] != false,
       directRules:
           (json['directRules'] is List
                   ? json['directRules'] as List
@@ -817,48 +875,85 @@ class MessagingAccessPolicy {
                 ),
               )
               .toList(growable: false),
+      sharedMemberRules:
+          (json['sharedMemberRules'] is List
+                  ? json['sharedMemberRules'] as List
+                  : const <dynamic>[])
+              .whereType<Map>()
+              .map(
+                (item) => MessagingAccessRule.fromJson(
+                  Map<String, dynamic>.from(item),
+                ),
+              )
+              .toList(growable: false),
+      sharedParticipationRules:
+          (json['sharedParticipationRules'] is List
+                  ? json['sharedParticipationRules'] as List
+                  : const <dynamic>[])
+              .whereType<Map>()
+              .map(
+                (item) => MessagingSharedParticipationRule.fromJson(
+                  Map<String, dynamic>.from(item),
+                ),
+              )
+              .where((item) => item.value.isNotEmpty)
+              .toList(growable: false),
     );
   }
 
   const MessagingAccessPolicy.defaults({
+    this.schemaVersion = 3,
     this.directPolicy = 'allowlist',
     this.sharedPolicy = 'allowlist',
-    this.requireMentionInShared = true,
+    this.defaultAllowUntaggedInShared = true,
     this.directRules = const <MessagingAccessRule>[],
     this.sharedSpaceRules = const <MessagingAccessRule>[],
     this.sharedActorRules = const <MessagingAccessRule>[],
+    this.sharedMemberRules = const <MessagingAccessRule>[],
+    this.sharedParticipationRules = const <MessagingSharedParticipationRule>[],
   });
 
+  final int schemaVersion;
   final String directPolicy;
   final String sharedPolicy;
-  final bool requireMentionInShared;
+  final bool defaultAllowUntaggedInShared;
   final List<MessagingAccessRule> directRules;
   final List<MessagingAccessRule> sharedSpaceRules;
   final List<MessagingAccessRule> sharedActorRules;
+  final List<MessagingAccessRule> sharedMemberRules;
+  final List<MessagingSharedParticipationRule> sharedParticipationRules;
 
   MessagingAccessPolicy copyWith({
+    int? schemaVersion,
     String? directPolicy,
     String? sharedPolicy,
-    bool? requireMentionInShared,
+    bool? defaultAllowUntaggedInShared,
     List<MessagingAccessRule>? directRules,
     List<MessagingAccessRule>? sharedSpaceRules,
     List<MessagingAccessRule>? sharedActorRules,
+    List<MessagingAccessRule>? sharedMemberRules,
+    List<MessagingSharedParticipationRule>? sharedParticipationRules,
   }) {
     return MessagingAccessPolicy(
+      schemaVersion: schemaVersion ?? this.schemaVersion,
       directPolicy: directPolicy ?? this.directPolicy,
       sharedPolicy: sharedPolicy ?? this.sharedPolicy,
-      requireMentionInShared:
-          requireMentionInShared ?? this.requireMentionInShared,
+      defaultAllowUntaggedInShared:
+          defaultAllowUntaggedInShared ?? this.defaultAllowUntaggedInShared,
       directRules: directRules ?? this.directRules,
       sharedSpaceRules: sharedSpaceRules ?? this.sharedSpaceRules,
       sharedActorRules: sharedActorRules ?? this.sharedActorRules,
+      sharedMemberRules: sharedMemberRules ?? this.sharedMemberRules,
+      sharedParticipationRules:
+          sharedParticipationRules ?? this.sharedParticipationRules,
     );
   }
 
   Map<String, dynamic> toJson() => <String, dynamic>{
+    'schemaVersion': schemaVersion,
     'directPolicy': directPolicy,
     'sharedPolicy': sharedPolicy,
-    'requireMentionInShared': requireMentionInShared,
+    'defaultAllowUntaggedInShared': defaultAllowUntaggedInShared,
     'directRules': directRules
         .map((rule) => rule.toJson())
         .toList(growable: false),
@@ -868,10 +963,19 @@ class MessagingAccessPolicy {
     'sharedActorRules': sharedActorRules
         .map((rule) => rule.toJson())
         .toList(growable: false),
+    'sharedMemberRules': sharedMemberRules
+        .map((rule) => rule.toJson())
+        .toList(growable: false),
+    'sharedParticipationRules': sharedParticipationRules
+        .map((rule) => rule.toJson())
+        .toList(growable: false),
   };
 
   int get totalRuleCount =>
-      directRules.length + sharedSpaceRules.length + sharedActorRules.length;
+      directRules.length +
+      sharedSpaceRules.length +
+      sharedActorRules.length +
+      sharedMemberRules.length;
 }
 
 class MessagingAccessCapabilities {
@@ -879,6 +983,7 @@ class MessagingAccessCapabilities {
     this.supportsDirectPolicy = true,
     this.supportsSharedPolicy = true,
     this.supportsMentionGate = false,
+    this.supportsUntaggedGroupToggle = true,
     this.supportsDiscovery = false,
     this.directRuleScopes = const <String>[],
     this.sharedSpaceRuleScopes = const <String>[],
@@ -899,6 +1004,7 @@ class MessagingAccessCapabilities {
       supportsDirectPolicy: json['supportsDirectPolicy'] != false,
       supportsSharedPolicy: json['supportsSharedPolicy'] != false,
       supportsMentionGate: json['supportsMentionGate'] == true,
+      supportsUntaggedGroupToggle: json['supportsUntaggedGroupToggle'] != false,
       supportsDiscovery: json['supportsDiscovery'] == true,
       directRuleScopes: stringList(json['directRuleScopes']),
       sharedSpaceRuleScopes: stringList(json['sharedSpaceRuleScopes']),
@@ -910,6 +1016,7 @@ class MessagingAccessCapabilities {
   final bool supportsDirectPolicy;
   final bool supportsSharedPolicy;
   final bool supportsMentionGate;
+  final bool supportsUntaggedGroupToggle;
   final bool supportsDiscovery;
   final List<String> directRuleScopes;
   final List<String> sharedSpaceRuleScopes;
@@ -920,6 +1027,7 @@ class MessagingAccessCapabilities {
     'supportsDirectPolicy': supportsDirectPolicy,
     'supportsSharedPolicy': supportsSharedPolicy,
     'supportsMentionGate': supportsMentionGate,
+    'supportsUntaggedGroupToggle': supportsUntaggedGroupToggle,
     'supportsDiscovery': supportsDiscovery,
     'directRuleScopes': directRuleScopes,
     'sharedSpaceRuleScopes': sharedSpaceRuleScopes,
@@ -936,18 +1044,41 @@ class MessagingAccessTarget {
     required this.value,
     required this.label,
     required this.subtitle,
+    this.spaceScope,
+    this.spaceValue,
+    this.spaceLabel,
   });
 
   factory MessagingAccessTarget.fromJson(Map<String, dynamic> json) {
+    final nestedRule = _jsonMap(json['rule']);
+    final scope =
+        json['scope']?.toString() ?? nestedRule['scope']?.toString() ?? 'chat';
+    final value =
+        json['value']?.toString() ?? nestedRule['value']?.toString() ?? '';
     return MessagingAccessTarget(
       source: json['source']?.toString() ?? 'manual',
       bucket: json['bucket']?.toString() ?? 'sharedSpaceRules',
-      scope: json['scope']?.toString() ?? 'chat',
-      value: json['value']?.toString() ?? '',
+      scope: scope,
+      value: value,
       label:
-          json['label']?.toString().ifEmpty(json['value']?.toString() ?? '') ??
-          (json['value']?.toString() ?? ''),
-      subtitle: json['subtitle']?.toString() ?? '',
+          nestedRule['label']?.toString().ifEmpty(
+            json['label']?.toString() ?? value,
+          ) ??
+          json['label']?.toString().ifEmpty(value) ??
+          value,
+      subtitle:
+          json['subtitle']?.toString() ??
+          nestedRule['spaceLabel']?.toString() ??
+          '',
+      spaceScope:
+          json['spaceScope']?.toString() ??
+          nestedRule['spaceScope']?.toString(),
+      spaceValue:
+          json['spaceValue']?.toString() ??
+          nestedRule['spaceValue']?.toString(),
+      spaceLabel:
+          json['spaceLabel']?.toString() ??
+          nestedRule['spaceLabel']?.toString(),
     );
   }
 
@@ -957,11 +1088,21 @@ class MessagingAccessTarget {
   final String value;
   final String label;
   final String subtitle;
-
-  MessagingAccessRule get asRule =>
-      MessagingAccessRule(scope: scope, value: value, label: label);
-
-  String get id => '$bucket:$scope:$value';
+  final String? spaceScope;
+  final String? spaceValue;
+  final String? spaceLabel;
+
+  MessagingAccessRule get asRule => MessagingAccessRule(
+    scope: scope,
+    value: value,
+    label: label,
+    spaceScope: spaceScope,
+    spaceValue: spaceValue,
+    spaceLabel: spaceLabel,
+  );
+
+  String get id =>
+      '$bucket:$scope:$value:${spaceScope ?? ''}:${spaceValue ?? ''}';
 
   Map<String, dynamic> toJson() => <String, dynamic>{
     'source': source,
@@ -970,6 +1111,12 @@ class MessagingAccessTarget {
     'value': value,
     'label': label,
     'subtitle': subtitle,
+    if (spaceScope != null && spaceScope!.trim().isNotEmpty)
+      'spaceScope': spaceScope,
+    if (spaceValue != null && spaceValue!.trim().isNotEmpty)
+      'spaceValue': spaceValue,
+    if (spaceLabel != null && spaceLabel!.trim().isNotEmpty)
+      'spaceLabel': spaceLabel,
   };
 }
 
@@ -2548,7 +2695,8 @@ class OfficialIntegrationAppItem {
   String get effectiveStatus =>
       !isConnected && hasExpiredAccounts ? 'expired' : connection.status;
 
-  String get statusLabel => _titleCase(effectiveStatus.replaceAll('_', ' '));
+  String get statusLabel =>
+      effectiveStatus == 'expired' ? 'Expired' : connection.statusLabel;
 }
 
 class OfficialIntegrationEnvStatus {
@@ -2780,7 +2928,8 @@ class OfficialIntegrationItem {
   String get effectiveStatus =>
       !isConnected && hasExpiredAccounts ? 'expired' : connection.status;
 
-  String get statusLabel => _titleCase(effectiveStatus.replaceAll('_', ' '));
+  String get statusLabel =>
+      effectiveStatus == 'expired' ? 'Expired' : connection.statusLabel;
 }
 
 class SkillDocument {

package/flutter_app/lib/main_operations.dart

@@ -1663,11 +1663,6 @@ class _MemoryPanelState extends State<MemoryPanel>
                 icon: Icon(Icons.bug_report_outlined),
                 label: Text('Inspect'),
               ),
-              OutlinedButton.icon(
-                onPressed: () => _openBehaviorNotesEditor(context, controller),
-                icon: Icon(Icons.edit_outlined),
-                label: Text('Behavior Notes'),
-              ),
               FilledButton.icon(
                 onPressed: () => _openMemoryCreator(context, controller),
                 icon: Icon(Icons.add),
@@ -2291,50 +2286,6 @@ class _MemoryPanelState extends State<MemoryPanel>
     );
   }
 
-  Future<void> _openBehaviorNotesEditor(
-    BuildContext context,
-    NeoAgentController controller,
-  ) async {
-    final contentController = TextEditingController(
-      text: controller.memoryOverview.assistantBehaviorNotes,
-    );
-    await showDialog<void>(
-      context: context,
-      builder: (context) {
-        return AlertDialog(
-          backgroundColor: _bgCard,
-          title: Text('Edit Assistant Behavior Notes'),
-          content: SizedBox(
-            width: 720,
-            child: TextField(
-              controller: contentController,
-              minLines: 16,
-              maxLines: 24,
-              decoration: const InputDecoration(
-                labelText: 'assistant_behavior_notes',
-              ),
-            ),
-          ),
-          actions: <Widget>[
-            TextButton(
-              onPressed: () => Navigator.of(context).pop(),
-              child: Text('Cancel'),
-            ),
-            FilledButton(
-              onPressed: () async {
-                await controller.updateAssistantBehaviorNotes(
-                  contentController.text,
-                );
-                if (context.mounted) Navigator.of(context).pop();
-              },
-              child: Text('Save'),
-            ),
-          ],
-        );
-      },
-    );
-  }
-
   Future<void> _openCoreMemoryEditor(
     BuildContext context,
     NeoAgentController controller, {