Skip to content

feat(core): bump galileo-core to ^4.4.0, drop fallback ControlSpan tests (HYBIM-788)#75

Draft
etserend wants to merge 1 commit into
feat/HYBIM-788-port-6ccd06afrom
feat/HYBIM-788-port-fa93ab2
Draft

feat(core): bump galileo-core to ^4.4.0, drop fallback ControlSpan tests (HYBIM-788)#75
etserend wants to merge 1 commit into
feat/HYBIM-788-port-6ccd06afrom
feat/HYBIM-788-port-fa93ab2

Conversation

@etserend

Copy link
Copy Markdown
Contributor

Summary

Port of upstream commit fa93ab2 from rungalileo/galileo-python (#615) — third commit in the HYBIM-788 migration.

  • Bumps galileo-core from ^4.3.0^4.4.0 in both main deps and test group
  • Runs poetry lock to update the lock file to galileo-core 4.4.0
  • Replaces test_logged_trace_roundtrip_with_fallback_control_span and test_fallback_control_span_rejects_non_uuidish_id_fields with direct native equivalents — the builtins.__import__ + importlib.reload fallback machinery is dead code now that 4.4.0 ships native ControlSpan unconditionally
  • Removes unused top-level imports (builtins, importlib, three splunk_ao.* module refs) from both test files

Namespace translation applied

No from galileo.* refs — this commit only touches pyproject.toml, poetry.lock, and two test files. galileo-core dep name is intentionally unchanged (external dep).

Test plan

  • poetry run pytest tests/schemas/test_logged.py tests/test_agent_control_bridge.py -v — 49 passed
  • poetry run ruff check src/splunk_ao/ --exclude src/splunk_ao/resources/ — clean
  • grep -r "from galileo\." src/splunk_ao/ — no results

Part of

HYBIM-788 — commits applied in order: 3 of 3 (e42a569 → 6ccd06a → fa93ab2)

Stacked on: #74 (feat/HYBIM-788-port-6ccd06a — jsonl_flat)

🤖 Generated with Claude Code

…sts (HYBIM-788)

Ports upstream commit fa93ab2 from rungalileo/galileo-python.

galileo-core 4.4.0 ships native ControlSpan unconditionally, so the
builtins.__import__ + importlib.reload fallback test machinery is no longer
needed. Replace the two fallback tests with direct native equivalents and
remove the now-unused top-level module imports from both test files.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant