Method vault_intent_update

atomic-repository/src/repository/vault_intent.rs:646–959

Update an intent's fields.

(
        &self,
        intent_id: &str,
        options: IntentUpdateOptions,
    )

    /// Update an intent's fields.
    pub fn vault_intent_update(
        &self,
        intent_id: &str,
        options: IntentUpdateOptions,
    ) -> Result<IntentInfo, RepositoryError> {
        let full_id = self.normalize_intent_id(intent_id)?;
        let intent_file =
            self.find_intent_path(&full_id)?
                .ok_or_else(|| RepositoryError::InvalidOperation {
                    message: format!("Intent '{}' not found", full_id),
                })?;

        // Read from disk first so we pick up any user/agent edits to the
        // markdown body. If the file doesn't exist on disk, fall back to
        // the redb entry.
        let disk_path = self.vault_dir().join(&intent_file);
        let (content_bytes, frontmatter_json) = if disk_path.exists() {
            let file_content = std::fs::read_to_string(&disk_path)?;
            let (fm_json, body) = crate::repository::vault::parse_vault_frontmatter(&file_content);
            (body.into_bytes(), fm_json)
        } else {
            let entry = self.vault_retrieve(&intent_file)?.ok_or_else(|| {
                RepositoryError::InvalidOperation {
                    message: format!("Intent '{}' not found", full_id),
                }
            })?;
            (entry.content_bytes.clone(), entry.frontmatter_json.clone())
        };

        // Update frontmatter
        let mut fm: serde_json::Map<String, serde_json::Value> =
            serde_json::from_str(&frontmatter_json).unwrap_or_default();

        // The status as stored on disk BEFORE this update mutates anything. The
        // T5b lapse check keys off this: a substance edit on an already-`done`
        // intent (that is not itself re-asserting a status) can lapse the grant.
        let prior_status = fm
            .get("status")
            .and_then(|v| v.as_str())
            .map(str::to_string);
        // The authoring view is frozen in frontmatter at create time.
        // TODO(T5b-follow-up): scope-gating uses this frozen authoring `view`
        // (recon gaps #3/#4). A Draft-authored intent whose changes were already
        // inserted into a Shared view still reads its original Draft view here;
        // precise "Shared doneness is immutable" needs the visibility walk.
        let fm_view_name = fm.get("view").and_then(|v| v.as_str()).map(str::to_string);

        if options.content.is_some() && !options.force {
            let manifest = self.vault_manifest()?;
            let summary = manifest.intents.get(&full_id);
            let current_status = summary
                .map(|intent| intent.status.as_str())
                .or_else(|| fm.get("status").and_then(|value| value.as_str()))
                .unwrap_or("unknown");
            let has_linked_goal = summary.is_some_and(|intent| intent.goals > 0)
                || manifest.goals.values().any(|goal| {
                    goal.intent
                        .as_deref()