add files

R/asa_api_helpers.R

@@ -1115,49 +1115,65 @@ asa_api_normalize_result <- function(result, include_raw_output = FALSE, include
   output
 }
 
-asa_api_is_hidden_execution_field <- function(name) {
-  field <- asa_api_scalar_chr(name, default = "")
-  if (!nzchar(field)) {
-    return(FALSE)
+asa_api_optional_scalar_chr <- function(value) {
+  text <- asa_api_scalar_chr(value, default = "")
+  if (nzchar(text)) {
+    return(text)
   }
-
-  field <- tolower(trimws(field))
-  field %in% c("base_url", "openai_api_base") ||
-    grepl("(^|_)(backend|provider|model|model_id|model_name)$", field)
+  NULL
 }
 
-asa_api_remove_hidden_execution_fields <- function(value) {
-  if (!is.list(value)) {
-    return(value)
+asa_api_optional_scalar_int <- function(value) {
+  number <- asa_api_scalar_int(value, default = NA_integer_)
+  if (!is.na(number)) {
+    return(number)
   }
+  NULL
+}
 
-  value_names <- names(value) %||% character(0)
-  if (length(value_names)) {
-    keep <- !vapply(value_names, asa_api_is_hidden_execution_field, logical(1))
-    value <- value[keep]
+asa_api_gui_execution_mode <- function(execution, response_mode = NULL) {
+  response_mode <- asa_api_scalar_chr(response_mode, default = "")
+  if (identical(response_mode, "provider_direct_single")) {
+    return("provider_direct")
   }
 
-  if (!length(value)) {
-    return(value)
+  execution <- asa_api_named_list(execution)
+  execution_mode <- asa_api_scalar_chr(execution$mode, default = "")
+  if (identical(execution_mode, "provider_direct")) {
+    return("provider_direct")
   }
 
-  for (i in seq_along(value)) {
-    value[[i]] <- asa_api_remove_hidden_execution_fields(value[[i]])
-  }
+  "asa_agent"
+}
 
-  value
+asa_api_gui_execution_summary <- function(execution, response_mode = NULL) {
+  execution <- asa_api_named_list(asa_api_to_plain(execution))
+
+  asa_api_drop_nulls(list(
+    mode = asa_api_gui_execution_mode(execution, response_mode = response_mode),
+    thread_id = asa_api_optional_scalar_chr(execution$thread_id),
+    backend_status = asa_api_optional_scalar_chr(execution$backend_status),
+    status_code = asa_api_optional_scalar_int(execution$status_code),
+    tool_calls_used = asa_api_optional_scalar_int(execution$tool_calls_used),
+    search_calls_used = asa_api_optional_scalar_int(execution$search_calls_used),
+    action_step_count = asa_api_optional_scalar_int(execution$action_step_count)
+  ))
 }
 
-asa_api_sanitize_gui_result <- function(result) {
+asa_api_project_gui_result <- function(result, response_mode = NULL) {
   if (!is.list(result)) {
     return(result)
   }
 
-  if (!is.null(result$execution)) {
-    result$execution <- asa_api_remove_hidden_execution_fields(result$execution)
-  }
-
-  result
+  asa_api_drop_nulls(list(
+    status = result$status %||% NULL,
+    message = result$message %||% NULL,
+    parsed = result$parsed %||% NULL,
+    elapsed_time_min = result$elapsed_time_min %||% NULL,
+    search_tier = result$search_tier %||% NULL,
+    parsing_status = result$parsing_status %||% NULL,
+    execution = asa_api_gui_execution_summary(result$execution, response_mode = response_mode)
+  ))
 }
 
 asa_api_is_result_like <- function(value) {
@@ -1378,6 +1394,7 @@ asa_api_run_gui_query <- function(payload, request_context = NULL) {
   log_con <- asa_api_request_context_log_con(request_context)
   stage_ref <- new.env(parent = emptyenv())
   stage_ref$value <- "run_single"
+  response_mode <- asa_api_single_mode(payload, allow_direct_provider = TRUE)
   capture <- asa_api_new_runtime_capture()
   capture$prompt <- asa_api_scalar_chr(payload$prompt, default = "")
   run_payload <- payload
@@ -1399,10 +1416,10 @@ asa_api_run_gui_query <- function(payload, request_context = NULL) {
         request_context = list(route = route, log_con = log_con)
       )
 
-      stage_ref$value <- "sanitize_gui_result"
-      asa_api_sanitize_gui_result(result)
+      stage_ref$value <- "project_gui_result"
+      asa_api_project_gui_result(result, response_mode = response_mode)
     },
-    mode = asa_api_single_mode(run_payload, allow_direct_provider = TRUE),
+    mode = response_mode,
     route = route,
     payload = run_payload,
     prompt = capture$prompt,

Tests/api_contract_smoke.R

@@ -452,7 +452,7 @@ assert_true(
 
 {
   original_run_single <- asa_api_run_single
-  original_sanitize_gui_result <- asa_api_sanitize_gui_result
+  original_project_gui_result <- asa_api_project_gui_result
   gui_log_lines <- character(0)
   gui_log_con <- textConnection("gui_log_lines", "w", local = TRUE)
   gui_error <- NULL
@@ -464,7 +464,7 @@ assert_true(
       execution = list(mode = "asa_agent")
     )
   }
-  asa_api_sanitize_gui_result <- function(result) {
+  asa_api_project_gui_result <- function(result, response_mode = NULL) {
     stop("subscript out of bounds", call. = FALSE)
   }
 
@@ -480,7 +480,7 @@ assert_true(
   )
   close(gui_log_con)
   asa_api_run_single <- original_run_single
-  asa_api_sanitize_gui_result <- original_sanitize_gui_result
+  asa_api_project_gui_result <- original_project_gui_result
 
   gui_failure <- asa_api_error_failure(gui_error)
   gui_res_env <- new.env(parent = emptyenv())
@@ -489,10 +489,10 @@ assert_true(
   assert_true(
     inherits(gui_error, "asa_api_runtime_error") &&
       identical(gui_failure$details$route, "/gui/query") &&
-      identical(gui_failure$details$stage, "sanitize_gui_result") &&
-      identical(gui_payload$details$stage, "sanitize_gui_result") &&
-      grepl("stage=sanitize_gui_result", gui_log_text, fixed = TRUE),
-    "GUI sanitization failures should return the safe diagnostic summary and emit matching logs."
+      identical(gui_failure$details$stage, "project_gui_result") &&
+      identical(gui_payload$details$stage, "project_gui_result") &&
+      grepl("stage=project_gui_result", gui_log_text, fixed = TRUE),
+    "GUI projection failures should return the safe diagnostic summary and emit matching logs."
   )
 }
 
@@ -574,31 +574,71 @@ asa_api_refresh_auth_cache(force = TRUE)
   asa_api_run_single_via_direct <- original_direct
 }
 
-gui_result <- asa_api_sanitize_gui_result(list(
+gui_result <- asa_api_project_gui_result(list(
   status = "success",
+  message = "ok",
+  parsed = list(capital = "Paris"),
+  elapsed_time_min = 0.25,
+  search_tier = "unknown",
+  parsing_status = list(valid = TRUE),
   execution = list(
-    mode = "provider_direct",
-    config_snapshot = list(backend = "gemini", model = "gemini-3-flash-preview"),
-    payload_integrity = list(
-      backend = "gemini",
-      model = "gemini-3-flash-preview",
-      released_from = "message_text"
-    ),
-    trace_metadata = list(
-      structured_output_backend = "gemini",
-      model_name = "gemini-3-flash-preview"
-    )
+    thread_id = list("asa_123"),
+    backend_status = list("success"),
+    status_code = list(200L),
+    tool_calls_used = list(2L),
+    search_calls_used = list(1L),
+    action_step_count = list(6L),
+    tool_quality_events = list(list(
+      message_index_in_round = list(1L),
+      tool_name = list("update_plan"),
+      is_empty = list(FALSE),
+      is_off_target = list(FALSE),
+      is_error = list(FALSE),
+      error_type = NULL,
+      elapsed_ms_estimate = list(0L),
+      quality_version = list("v1")
+    )),
+    config_snapshot = list(backend = "gemini", model = "gemini-3-flash-preview")
   )
 ))
 assert_true(
-  is.null(gui_result$execution$config_snapshot$backend) &&
-    is.null(gui_result$execution$config_snapshot$model),
-  "GUI responses should not expose backend/model selection in execution metadata."
+  identical(
+    names(gui_result$execution),
+    c(
+      "mode",
+      "thread_id",
+      "backend_status",
+      "status_code",
+      "tool_calls_used",
+      "search_calls_used",
+      "action_step_count"
+    )
+  ) &&
+    identical(gui_result$execution$mode, "asa_agent") &&
+    identical(gui_result$execution$thread_id, "asa_123") &&
+    identical(gui_result$execution$tool_calls_used, 2L),
+  "GUI success responses should expose only a compact execution summary."
+)
+assert_true(
+  !("tool_quality_events" %in% names(gui_result$execution)) &&
+    !("config_snapshot" %in% names(gui_result$execution)),
+  "GUI projection should avoid recursing through nested execution diagnostics."
+)
+
+gui_direct_result <- asa_api_project_gui_result(
+  list(
+    status = "success",
+    execution = list(
+      mode = list("provider_direct"),
+      status_code = list(200L)
+    )
+  ),
+  response_mode = "provider_direct_single"
 )
 assert_true(
-  is.null(gui_result$execution$payload_integrity$backend) &&
-    is.null(gui_result$execution$payload_integrity$model),
-  "GUI payload integrity metadata should hide backend/model fields."
+  identical(gui_direct_result$execution$mode, "provider_direct") &&
+    identical(gui_direct_result$execution$status_code, 200L),
+  "GUI projection should preserve direct-provider mode in the compact execution summary."
 )
 
 cat("asa-api contract smoke checks passed\n")

www/index.html

@@ -404,6 +404,9 @@
       if (data && data.execution && data.execution.mode === "provider_direct") {
         return "Mode: direct provider";
       }
+      if (data && data.execution && data.execution.mode === "asa_agent") {
+        return "Mode: ASA agent";
+      }
 
       return "Mode: ASA agent";
     }