Skip to content

fix: verify powermem-mcp PyPI release publishing - #1134

Open
knqiufan wants to merge 1 commit into
oceanbase:mainfrom
knqiufan:fix/issue-1117-powermem-mcp-pypi
Open

fix: verify powermem-mcp PyPI release publishing#1134
knqiufan wants to merge 1 commit into
oceanbase:mainfrom
knqiufan:fix/issue-1117-powermem-mcp-pypi

Conversation

@knqiufan

@knqiufan knqiufan commented Jul 2, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Add a PyPI release verification script that checks both powermem and powermem-mcp publish the requested version with wheel and sdist artifacts.
  • Run the verification after the powermem-mcp publish job so missing wrapper releases fail the publish workflow instead of passing silently.
  • Update release docs to describe the two PyPI projects, Trusted Publisher setup, and the backfill flow for missing package versions.

Test Plan

  • python -m pytest tests/unit/test_check_pypi_release.py -v
  • python -m pytest tests/unit/test_release_binary_ci.py::test_pypi_publish_workflow_verifies_both_python_packages -v
  • python scripts/check_package_versions.py
  • git diff --cached --check

Notes

  • The powermem-mcp PyPI project still needs a Trusted Publisher entry for oceanbase/powermem and publish.yml before the failed release can be backfilled.

Closes #1117

Add a PyPI visibility check after publishing both Python packages so missing powermem-mcp releases fail the workflow instead of passing unnoticed.

@wayyoungboy wayyoungboy left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Reviewed the publish workflow and the new PyPI visibility checker. The verification job waits for the powermem-mcp publish job, checks both powermem and powermem-mcp, and the script correctly validates the requested release version plus wheel/sdist presence with retries.

Targeted verification passed locally: tests/unit/test_check_pypi_release.py and tests/unit/test_release_binary_ci.py (24 tests). No blocker from my side.

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.

[Bug]: powermem-mcp 1.1.6 is missing from PyPI after release

2 participants