Skip to content

Commit 8b8f2f9

Browse files
authored
Merge branch 'main' into dependabot/uv/requests-2.33.0
2 parents 8335d13 + 8c3f8ae commit 8b8f2f9

44 files changed

Lines changed: 4331 additions & 462 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

CLAUDE.md

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -219,6 +219,18 @@ Place it before imports with one blank line after.
219219
- **Never** use the keyword "Kairo" in code - it's a legacy reference that must be removed/replaced
220220
- If found during code review, flag for removal
221221

222+
### Observability Export Configuration — Coordinated Review Required
223+
224+
The following three constants must stay in sync. If a PR changes **any one** of them, the reviewer (human or Copilot) **must** ask the author to confirm the other two are still correct:
225+
226+
| Constant | Location |
227+
|---|---|
228+
| `PROD_OBSERVABILITY_SCOPE` | `libraries/microsoft-agents-a365-runtime/microsoft_agents_a365/runtime/environment_utils.py` |
229+
| `DEFAULT_ENDPOINT_URL` | `libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/agent365_exporter.py` |
230+
| Export URL path pattern | `build_export_url()` in `libraries/microsoft-agents-a365-observability-core/microsoft_agents_a365/observability/core/exporters/utils.py` |
231+
232+
Snapshot tests in `tests/observability/core/test_export_config_consistency.py` will fail if any value drifts, but the developer must also verify the values are correct for the target environment — the tests only catch accidental drift, not intentional-but-incomplete updates.
233+
222234
### Python Conventions
223235

224236
- Type hints required on all function parameters and return types

0 commit comments

Comments
 (0)