Improve geothermal qualitative extraction + fix LiteLLM runtime bug

[bpulluta]

Summary

This PR finalizes the geothermal qualitative extraction improvements and hardens LiteLLM/OpenAI runtime behavior for provider-specific unsupported kwargs.

Geothermal extraction updates

• Tuned geothermal schema in v2.2.2
• Keep scoped qualitative feature set aligned to required scope (removed extra district features)
• Keep broadened qualitative definitions and applicability/instruction tuning for better recall
• Added tuned collection prompt behavior in plugin_config.yaml

LiteLLM runtime hardening

• Add adaptive retry in compass/services/openai.py for recoverable provider rejections of top-level kwargs
• Automatically remove unsupported kwargs (e.g., temperature) and retry the same call
• Preserve strict behavior for non-recoverable errors (no silent masking of real failures)

Tests

• Add targeted service tests covering:
  • retry behavior when unsupported kwargs are rejected
  • non-recoverable bad request passthrough
  • fallback parsing of param from unstructured error text

Validation

• pixi run -e pdev pytest -q tests/python/unit/services/test_services_openai.py -rapP
  • Passed (8 tests)

[codecov-commenter]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 63.85%. Comparing base (b14fe34) to head (e6775f7).

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #521      +/-   ##
==========================================
+ Coverage   63.78%   63.85%   +0.06%     
==========================================
  Files          78       78              
  Lines        7445     7445              
  Branches      755      755              
==========================================
+ Hits         4749     4754       +5     
+ Misses       2536     2531       -5     
  Partials      160      160              

Flag	Coverage Δ
unittests	63.85% <ø> (+0.06%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[Copilot]

Pull request overview

This PR improves COMPASS’ geothermal electricity one-shot extraction configuration (schema + text-collection prompting) and hardens the OpenAI/LiteLLM call path to dynamically retry after provider-specific rejections of unsupported top-level kwargs.

Changes:

• Added adaptive handling in OpenAIService to detect provider “unsupported parameter” style BadRequestErrors, drop the offending kwarg, and retry.
• Updated geothermal extraction configuration: replaced boolean collection_prompts with an explicit prompt chain and tuned the geothermal qualitative schema (including removing out-of-scope district features).
• Added unit tests covering the unsupported-kwarg retry behavior and kwarg detection logic.

Reviewed changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File	Description
compass/services/openai.py	Adds unsupported-kwarg detection and retry logic for resilient OpenAI/LiteLLM runtime behavior.
tests/python/unit/services/test_services_openai.py	Adds targeted tests for retrying without unsupported kwargs and parsing unsupported params from error text.
compass/extraction/geothermal_electricity/plugin_config.yaml	Switches to explicit collection prompt(s) for geothermal text retention decisions.
compass/extraction/geothermal_electricity/geothermal_schema.json	Updates geothermal qualitative schema definitions and removes extra district features to match intended scope.

[ppinchuk]

Hey Bryon, thank you for this PR.

I am reverting the LiteLLM changes for now because I am not convinced this is the best approach to solve the problem you were experiencing. Let's touch base when you are back to discuss the specific errors you were seeing and what would be the best approach to resolve them.