feat(export): expose include_code_metric_metadata on export_records (HYBIM-788)#73
Open
etserend wants to merge 1 commit into
Open
feat(export): expose include_code_metric_metadata on export_records (HYBIM-788)#73etserend wants to merge 1 commit into
etserend wants to merge 1 commit into
Conversation
…HYBIM-788) Port upstream commit e42a569 from rungalileo/galileo-python. Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
4 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Port of upstream commit
e42a569fromrungalileo/galileo-python(#587) — first commit in the HYBIM-788 migration, applied in chronological order.include_code_metric_metadata: bool = Falseparam toExportClient.records()and module-levelexport_records()True, includes per-row scorer metadata (the dict returned by code-based scorers via the(score, metadata)tuple-return contract) on eachMetricSuccessin the export — off by default to keep payloads smallLogRecordsExportRequestmodel (log_records_export_request.py) to carry the new field throughto_dict()/from_dict()Falseand opt-in-TrueNamespace translation applied
All
galileo.*references rewritten tosplunk_ao.*per HYBIM-788 translation rules.galileo-coreandgalileo_core.*intentionally unchanged (external dep).Test plan
poetry run pytest tests/test_export.py -v— 17/17 passedpoetry run ruff check src/splunk_ao/export.py tests/test_export.py— cleanPart of
HYBIM-788 — commits applied in order: 1 of 3 (e42a569 → fa93ab2 → 6ccd06a)