From 5485bb1e93b0fcfbcc2c17a96f1f5dba70cdd3e9 Mon Sep 17 00:00:00 2001
From: Bryan Helmkamp <bryan@brynary.com>
Date: Fri, 28 Aug 2026 15:15:39 -0400
Subject: [PATCH] Hydrate nested structured blob references

---
 lib/components/fabro-workflow/src/artifact.rs | 84 ++++++++++++++++---
 1 file changed, 71 insertions(+), 13 deletions(-)

diff --git a/lib/components/fabro-workflow/src/artifact.rs b/lib/components/fabro-workflow/src/artifact.rs
index 6d2eb1e79..a61f0ba0e 100644
--- a/lib/components/fabro-workflow/src/artifact.rs
+++ b/lib/components/fabro-workflow/src/artifact.rs
@@ -521,23 +521,47 @@ pub async fn resolve_text_or_blob_ref(value: &Value, run_store: &RunStoreHandle)
     }
 }
 
-/// Resolve a structured JSON value from inline context or a Fabro-managed
-/// blob reference.
+/// Resolve a structured JSON value from inline context or Fabro-managed blob
+/// references at any depth.
 ///
 /// Managed `file://` references are normalized through their content-addressed
 /// blob hash instead of reading an execution-local path. Ordinary strings and
 /// ordinary file references remain unchanged for the caller to validate.
-pub(crate) async fn resolve_json_value(value: Value, run_store: &RunStoreHandle) -> Result<Value> {
-    let blob_hash = value.as_str().and_then(|reference| {
-        parse_blob_ref(reference).or_else(|| parse_managed_blob_file_ref(reference))
-    });
-    let Some(blob_hash) = blob_hash else {
-        return Ok(value);
-    };
-
-    let bytes = read_required_blob(&blob_hash, run_store).await?;
-    serde_json::from_slice(&bytes)
-        .map_err(|err| Error::engine_with_source("artifact blob was not valid JSON", err))
+pub(crate) fn resolve_json_value<'a>(
+    value: Value,
+    run_store: &'a RunStoreHandle,
+) -> BoxFuture<'a, Result<Value>> {
+    Box::pin(async move {
+        match value {
+            Value::String(reference) => {
+                let blob_hash =
+                    parse_blob_ref(&reference).or_else(|| parse_managed_blob_file_ref(&reference));
+                let Some(blob_hash) = blob_hash else {
+                    return Ok(Value::String(reference));
+                };
+                let bytes = read_required_blob(&blob_hash, run_store).await?;
+                let resolved = serde_json::from_slice(&bytes).map_err(|err| {
+                    Error::engine_with_source("artifact blob was not valid JSON", err)
+                })?;
+                resolve_json_value(resolved, run_store).await
+            }
+            Value::Array(items) => {
+                let mut resolved = Vec::with_capacity(items.len());
+                for item in items {
+                    resolved.push(resolve_json_value(item, run_store).await?);
+                }
+                Ok(Value::Array(resolved))
+            }
+            Value::Object(items) => {
+                let mut resolved = serde_json::Map::with_capacity(items.len());
+                for (key, item) in items {
+                    resolved.insert(key, resolve_json_value(item, run_store).await?);
+                }
+                Ok(Value::Object(resolved))
+            }
+            primitive => Ok(primitive),
+        }
+    })
 }
 
 /// Resolve a flat workflow context key (`context.NAME` or `NAME`) to a
@@ -966,6 +990,40 @@ mod tests {
         );
     }
 
+    #[tokio::test]
+    async fn resolve_json_value_hydrates_nested_parallel_branch_values() {
+        let run_store = make_run_store("nested-structured-json-resolution").await;
+        let finder_output = serde_json::json!({
+            "findings": [{"file": "src/lib.rs", "line": 7}]
+        });
+        let finder_blob = run_store
+            .write_blob(&serde_json::to_vec(&finder_output).unwrap())
+            .await
+            .unwrap();
+        let parallel_results = serde_json::json!([{
+            "id": "finder",
+            "index": 0,
+            "status": "succeeded",
+            "context_updates": {
+                "output.finder": format_blob_ref(&finder_blob),
+                "small": "kept inline"
+            }
+        }]);
+
+        let resolved = resolve_json_value(parallel_results, &run_store.into())
+            .await
+            .unwrap();
+
+        assert_eq!(
+            resolved[0]["context_updates"]["output.finder"],
+            finder_output
+        );
+        assert_eq!(
+            resolved[0]["context_updates"]["small"],
+            serde_json::json!("kept inline")
+        );
+    }
+
     #[tokio::test]
     async fn offload_preserves_parallel_results_and_replaces_large_context_updates() {
         let run_store = make_run_store("parallel-result-artifact-offload").await;