fix(sync): expose viewSectionIds to the local API and exclude it from MCP writes (#1090)

Author: dongdongbh

apps/desktop/src-tauri/src/local_api.rs

@@ -2655,6 +2655,10 @@ fn sanitize_task_patch_map(patch: &mut Map<String, Value>) -> Result<(), String>
             "attachments" => value.is_null() || valid_attachments(value),
             "relativeStartOffset" => value.is_null() || valid_relative_start_offset(value),
             "recurrence" => value.is_null() || valid_recurrence(value),
+            // Presentational grouping per view (#1090). Keys are view scopes and
+            // values are section ids from the settings catalogue; this engine
+            // never resolves them, so it only checks the shape.
+            "viewSectionIds" => value.is_null() || valid_view_section_ids(value),
             _ => return Err(format!("Unsupported task field: {key}")),
         };
         if !valid {
@@ -2664,6 +2668,13 @@ fn sanitize_task_patch_map(patch: &mut Map<String, Value>) -> Result<(), String>
     Ok(())
 }
 
+fn valid_view_section_ids(value: &Value) -> bool {
+    let Some(map) = value.as_object() else {
+        return false;
+    };
+    map.values().all(|entry| entry.as_str().is_some_and(|id| !id.trim().is_empty()))
+}
+
 fn js_string_length(value: &str) -> usize {
     value.encode_utf16().count()
 }

apps/mcp-server/src/task-write-field-exclusions.ts

@@ -15,4 +15,7 @@ export const TASK_WRITE_FIELD_EXCLUSIONS: Readonly<Partial<Record<keyof Task, st
         + 'file bytes this text interface cannot supply.',
     orderNum: 'Legacy alias of `order` (same SQLite column) — this surface exposes only '
         + '`order` to avoid two keys writing one value.',
+    viewSectionIds: 'Presentational per-view grouping whose section ids live in a settings '
+        + 'catalogue this surface cannot enumerate or create, so a raw id map would be '
+        + 'unusable from a tool call (#1090).',
 };