alpha-xone
diff --git a/‎CHANGELOG.md‎
Lines changed: 5 additions & 0 deletions b/‎CHANGELOG.md‎
Lines changed: 5 additions & 0 deletions
diff --git a/‎crates/xbbg-async/src/engine/mod.rs‎
Lines changed: 107 additions & 1 deletion b/‎crates/xbbg-async/src/engine/mod.rs‎
Lines changed: 107 additions & 1 deletion
@@ -7,6 +7,11 @@ and this project adheres to [Semantic Versioning 2.0.0](https://semver.org/spec/
 
 ## [Unreleased]
 
+### Fixed
+
+- **`bsrch` / `absrch` Excel grid requests restored (#333, #334)**: Python now sends BSRCH search parameters as `ExcelGetGridRequest` `Overrides[]` entries while keeping `Domain` as the top-level request element, so Excel-style weather searches no longer fail with Bloomberg element validation errors. The Rust extractor now parses `GridResponse.ColumnTitles[]` and `DataRecords[].DataFields[]` directly instead of the BEQS schema, preserves Bloomberg column names, propagates grid errors, and validates reported row/field counts rather than returning an empty placeholder ticker column.
+- **`bsrch` / `absrch` request plumbing hardened (#333, #334)**: Low-level `arequest()` now preserves `//blp/exrsvc` overrides as overrides instead of rewriting them into root elements, and the Rust engine normalizes `ExcelGetGridRequest` `Domain`/override pairs consistently across generated endpoints, raw requests, and direct request kwargs. `RawRequest` calls whose effective operation is `ExcelGetGridRequest` now pick the BSRCH extractor by default, with regression coverage for malformed Python override pairs and live weather-grid requests.
+
 ## [1.2.2] - 2026-05-06
 
 ### Fixed
 
@@ -879,10 +879,17 @@ impl RequestParams {
         }
     }
 
+    pub(crate) fn is_excel_get_grid_request(&self) -> bool {
+        matches!(
+            parse_operation_lossless(self.effective_operation()),
+            Operation::ExcelGetGrid
+        )
+    }
+
     /// Apply default values derived from operation semantics.
     pub fn with_defaults(mut self) -> Self {
         if !self.extractor_set && self.extractor == ExtractorType::default() {
-            let operation = parse_operation_lossless(&self.operation);
+            let operation = parse_operation_lossless(self.effective_operation());
             self.extractor = operation.default_extractor();
         }
         self
@@ -1110,6 +1117,43 @@ impl RequestParams {
     }
 }
 
+fn normalize_excel_grid_params(params: &mut RequestParams, kwargs: HashMap<String, String>) {
+    let mut domain: Option<String> = None;
+    let mut grid_overrides: Vec<(String, String)> = Vec::new();
+
+    fn route_pair(
+        domain: &mut Option<String>,
+        grid_overrides: &mut Vec<(String, String)>,
+        key: String,
+        value: String,
+    ) {
+        if key.eq_ignore_ascii_case("Domain") {
+            *domain = Some(value);
+        } else if !key.is_empty() && !key.eq_ignore_ascii_case("Overrides") {
+            grid_overrides.push((key, value));
+        }
+    }
+
+    for (key, value) in params.elements.take().unwrap_or_default() {
+        route_pair(&mut domain, &mut grid_overrides, key, value);
+    }
+
+    for (key, value) in params.overrides.take().unwrap_or_default() {
+        route_pair(&mut domain, &mut grid_overrides, key, value);
+    }
+
+    let mut keys: Vec<String> = kwargs.keys().cloned().collect();
+    keys.sort();
+    for key in keys {
+        if let Some(value) = kwargs.get(&key) {
+            route_pair(&mut domain, &mut grid_overrides, key, value.clone());
+        }
+    }
+
+    params.elements = domain.map(|value| vec![("Domain".to_string(), value)]);
+    params.overrides = (!grid_overrides.is_empty()).then_some(grid_overrides);
+}
+
 fn merge_raw_kwargs_into_elements(params: &mut RequestParams, kwargs: HashMap<String, String>) {
     if kwargs.is_empty() {
         return;
@@ -1415,6 +1459,11 @@ impl Engine {
         params.validate()?;
 
         let kwargs = params.kwargs.take().unwrap_or_default();
+        if params.is_excel_get_grid_request() {
+            normalize_excel_grid_params(&mut params, kwargs);
+            return Ok(params);
+        }
+
         if params.is_raw_request() {
             merge_raw_kwargs_into_elements(&mut params, kwargs);
             return Ok(params);
@@ -2365,6 +2414,63 @@ mod tests {
         );
     }
 
+    #[test]
+    fn normalize_excel_grid_params_routes_domain_and_grid_overrides() {
+        let mut params = RequestParams {
+            operation: Operation::ExcelGetGrid.to_string(),
+            elements: Some(vec![
+                ("Domain".to_string(), "FI:OLD".to_string()),
+                ("provider".to_string(), "wsi".to_string()),
+            ]),
+            overrides: Some(vec![
+                ("location".to_string(), "nwe".to_string()),
+                ("Domain".to_string(), "COMDTY:WEATHER".to_string()),
+            ]),
+            ..Default::default()
+        };
+
+        normalize_excel_grid_params(
+            &mut params,
+            HashMap::from([("model".to_string(), "ecmwf".to_string())]),
+        );
+
+        assert_eq!(
+            params.elements,
+            Some(vec![("Domain".to_string(), "COMDTY:WEATHER".to_string())])
+        );
+        assert_eq!(
+            params.overrides,
+            Some(vec![
+                ("provider".to_string(), "wsi".to_string()),
+                ("location".to_string(), "nwe".to_string()),
+                ("model".to_string(), "ecmwf".to_string()),
+            ])
+        );
+    }
+
+    #[test]
+    fn excel_grid_detection_uses_raw_request_operation() {
+        let params = RequestParams {
+            operation: Operation::RawRequest.to_string(),
+            request_operation: Some(Operation::ExcelGetGrid.to_string()),
+            ..Default::default()
+        };
+
+        assert!(params.is_excel_get_grid_request());
+    }
+
+    #[test]
+    fn raw_excel_grid_defaults_to_bsrch_extractor() {
+        let params = RequestParams {
+            operation: Operation::RawRequest.to_string(),
+            request_operation: Some(Operation::ExcelGetGrid.to_string()),
+            ..Default::default()
+        }
+        .with_defaults();
+
+        assert_eq!(params.extractor, ExtractorType::Bsrch);
+    }
+
     #[test]
     fn subscription_status_records_failure_and_removes_active_topic() {
         let metric = Arc::new(SubscriptionMetrics {