Skip to content

HYBIM 719 rebrand package metadata#19

Merged
keith-decker merged 2 commits into
mainfrom
HYBIM-719-rebrand-package-metadata
Jun 16, 2026
Merged

HYBIM 719 rebrand package metadata#19
keith-decker merged 2 commits into
mainfrom
HYBIM-719-rebrand-package-metadata

Conversation

@keith-decker

@keith-decker keith-decker commented Jun 12, 2026

Copy link
Copy Markdown
Contributor

Summary

HYBIM-719 rebrands the root Agent Observability SDK package metadata from Galileo to the approved Splunk AO package identity.

This PR is built on top of #17 due to the folder/class renames. Changeset will be larger as a result.

Changes include:

  • rename root distribution metadata to splunk-ao
  • reset the root SDK version from 2.3.0 to 0.1.0
  • update author, description, and repository metadata to Splunk AO values
  • keep runtime splunk_ao.__version__ synchronized with pyproject.toml
  • update package-version lookup to query the splunk-ao distribution
  • add focused tests for the new distribution lookup and missing-package fallback

The existing galileo-python/ telemetry header prefix is intentionally preserved per HYBIM-719 scope.

Scope Notes

This change is limited to root SDK package metadata/versioning and directly affected package-version helper tests.

Not changed:

  • adjacent galileo-a2a / galileo-adk package metadata
  • generated resources under src/galileo/resources/
  • dependency versions or lock files
  • broad namespace, environment variable, backend URL, or documentation rebranding

@keith-decker

keith-decker commented Jun 12, 2026

Copy link
Copy Markdown
Contributor Author

Result output from speckit implement:

  • Root package metadata parse: PASS
    • project.name == "splunk-ao"
    • project.version == "0.1.0"
    • project.description contains Splunk Agent Observability SDK
    • project.authors == [{"name": "Splunk Inc."}]
    • project.urls.Repository == "https://github.com/splunk/splunk-ao-python"
    • [tool.poetry].packages includes both splunk_ao and galileo
  • Runtime version and semantic-release sync: PASS
    • src/splunk_ao/__init__.py:__version__ == "0.1.0"
    • tool.semantic_release.version_toml == ["pyproject.toml:project.version"]
    • tool.semantic_release.version_variables == ["src/splunk_ao/__init__.py:__version__"]
  • Python compile checks: PASS
    • src/splunk_ao/utils/headers_data.py
    • tests/test_api_headers.py
    • tests/test_public_imports.py
  • Tracked lock-file consistency: PASS
    • uvx --from poetry==2.1.3 poetry check --lock
    • poetry.lock did not require changes
    • uv.lock is untracked/ignored in this branch and was not changed
  • Focused tests: PASS after switching package source away from Splunk
    Artifactory
    • Direct uv run pytest tests/test_api_headers.py tests/test_public_imports.py
      no longer hit the Artifactory package-index failure, but the default uv
      project environment does not include pytest because test dependencies are
      declared in Poetry's test group.
    • Project-scoped focused test command passed with the Poetry test-group
      dependencies supplied explicitly:
      uv run --with pytest --with pytest-xdist --with pytest-timeout --with pytest-socket --with pytest-asyncio --with pytest-env --with requests-mock --with openai --with 'galileo-core[testing]==4.3.0' pytest tests/test_api_headers.py tests/test_public_imports.py
    • Result: 7 passed
  • Package build: PASS
    • uv build
    • Built dist/splunk_ao-0.1.0.tar.gz
    • Built dist/splunk_ao-0.1.0-py3-none-any.whl
    • Wheel metadata reports Name: splunk-ao, Version: 0.1.0,
      Author: Splunk Inc., and repository URL
      https://github.com/signalfx/splunk-ao-python
    • Wheel includes both splunk_ao/ and galileo/
  • Scope review: PASS
    • No diff in ../splunk-ao-python/galileo-a2a/pyproject.toml
    • No diff in ../splunk-ao-python/galileo-adk/pyproject.toml
    • No diff under ../splunk-ao-python/src/galileo/resources/
    • No modified files in ../galileo-python
    • No lock-file diff

@fercor-cisco

Copy link
Copy Markdown
Collaborator

I'll review #19 after #17 has been merged and its diff is smaller

@keith-decker keith-decker force-pushed the HYBIM-719-rebrand-package-metadata branch from b7f3fa1 to f510437 Compare June 16, 2026 17:32

@fercor-cisco fercor-cisco left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 This review was generated by the Astra agent. It may contain mistakes.

Verdict: approve — Focused, low-risk metadata rebrand; version is consistently synced and the distribution-name change is covered by tests.

Follow-ups

Suggested follow-up work that could be tracked as Shortcut stories:

  • src/splunk_ao/utils/headers_data.py:68-68: The telemetry SDK header still emits the galileo-python/ prefix. This is intentionally out of scope for HYBIM-719 per the PR description, but a future ticket should rebrand the header prefix (and the X-Galileo-SDK header name) to the Splunk AO identity if/when downstream consumers are ready.

Comment thread pyproject.toml Outdated

[project.urls]
Repository = "https://github.com/rungalileo/galileo-python"
Repository = "https://github.com/signalfx/splunk-ao-python"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔵 nit (question): The Repository URL points to the signalfx org, but this repository lives at github.com/splunk/splunk-ao-python. Is signalfx/splunk-ao-python the intended canonical/publish location? If not, this should match the splunk org. (Your speckit verification lists the signalfx URL as expected, so flagging only to confirm it's deliberate.)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the speckit was built before the repo was moved. I'll update this!

@fercor-cisco fercor-cisco left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🤖 This review was generated by the Astra agent. It may contain mistakes.

Verdict: approve — Clean, consistent package-metadata rebrand with correct tests; version lookup and semantic-release config are in sync, and the one open review comment is already resolved.

@keith-decker keith-decker merged commit 410a0e6 into main Jun 16, 2026
16 checks passed
@keith-decker keith-decker deleted the HYBIM-719-rebrand-package-metadata branch June 16, 2026 19:43
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 16, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants