Skip to content

fix(release): align Python SDK release trigger with python-sdk-v* tag convention - #339

Draft
PetrefiedThunder wants to merge 1 commit into
mainfrom
cursor/fix-python-release-tag-trigger-045a
Draft

PetrefiedThunder wants to merge 1 commit into
mainfrom
cursor/fix-python-release-tag-trigger-045a

Conversation

@PetrefiedThunder

Copy link
Copy Markdown
Collaborator

Summary

The Python SDK release workflow has never run because its tag trigger pattern (python-v*) does not match the actual tagging convention (python-sdk-v*). The existing release tag python-sdk-v0.5.0 followed the convention from the archived agent-middleware-api repository, but the workflow was never updated to match.

This PR aligns the workflow trigger and verification script with the python-sdk-v* convention, allowing tagged releases to actually trigger the PyPI publication workflow.

Related issues

Related to #280 (receipt signing issues). Evidence documented in uploads/amw-sbom-signing-receipts-evidence-2026-11-09.md.

Area and license

  • I identified the repository area changed by this PR.
  • I understand that spec/, verifier/, sdk/python/, sdk/typescript/, integrations/, and examples/ are Apache-2.0.
  • I understand that gateway/ contributions are under FSL-1.1-ALv2 (Fair Source).
  • This PR does not include proprietary hosted-service code or secrets.

Spec changes

  • This PR does not change normative spec behavior.

Validation

  • Tests: Manual verification with GITHUB_EVENT_NAME=push RELEASE_TAG=python-sdk-v0.5.0 python3 .github/scripts/check_python_release.py (passes)
  • Other checks: Verified old pattern python-v0.5.0 now correctly fails validation

Risk and compatibility

  • Low risk: This change only affects the release workflow trigger, not the package itself
  • The PyPI token fallback remains unchanged per requirements
  • No real tags were pushed during testing
  • CI checks (CI result, License result, gitleaks) need to pass before merge

Changes

  • Updated .github/workflows/release-python.yml trigger from python-v* to python-sdk-v*
  • Updated .github/scripts/check_python_release.py to expect python-sdk-v prefix
  • Updated documentation in sdk/python/README.md and sdk/python/CHANGELOG.md

Tag convention evidence

  • Existing tag: python-sdk-v0.5.0 (Nov 5, 2026)
  • Historical convention: python-sdk-v0.4.0 from archived repo
  • Zero workflow runs in gh run list --workflow=release-python.yml
  • Pattern python-sdk-v* matches both existing and historical tags
  • Pattern is specific enough to avoid matching unrelated tags

Contributor checklist

  • I added or updated tests for behavior changes, or explained why tests are not applicable.
  • I updated relevant documentation and examples.
  • Every commit in this PR is signed off under the Developer Certificate of Origin.
  • I agree to submit this contribution under the license applicable to the files I changed.
Open in Web Open in Cursor 

… convention

The release workflow trigger pattern was python-v*, but the actual release
tag followed the python-sdk-v* convention from the archived repository.
This mismatch prevented the workflow from running on tagged releases.

Changes:
- Update workflow trigger from 'python-v*' to 'python-sdk-v*'
- Update check_python_release.py to expect 'python-sdk-v' prefix
- Update documentation in README.md and CHANGELOG.md to reflect the
  actual tag convention

The trigger now matches the existing python-sdk-v0.5.0 tag and the
tag convention documented in earlier releases (python-sdk-v0.4.0).

Refs: #280 (receipt signing issues), evidence in
uploads/amw-sbom-signing-receipts-evidence-2026-11-09.md

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Christopher  <PetrefiedThunder@users.noreply.github.com>
@vercel

vercel Bot commented Nov 10, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
agent-middleware-web Ready Ready Preview Nov 10, 2026 5:48am UTC

Request Review

@coderabbitai

coderabbitai Bot commented Nov 10, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

cursor Bot pushed a commit that referenced this pull request Nov 10, 2026
…ronments

- Change tag trigger from python-v* to python-sdk-v* to match release-python.yml convention (see PR #339)
- Install Python dependencies with uv in virtual environments before scanning to capture resolved transitive dependencies
- Scan installed environments (gateway and SDK Python packages) and node_modules (TypeScript SDK and site) separately, then merge into comprehensive SBOM
- Update SECURITY.md to document SBOM scope: resolved Python dependencies and npm lockfiles
- Note workflow is unvalidated on release tag; will verify on next python-sdk-v* release

Before: 7 packages (only a few with exact pins in requirements.txt)
After: Expected 50+ packages covering full dependency trees

Addresses user feedback on PR #340 about thin coverage and tag pattern mismatch.

Signed-off-by: Cursor Agent <cursoragent@cursor.com>
Co-authored-by: Christopher  <PetrefiedThunder@users.noreply.github.com>

This branch was successfully deployed

1 active deployment
Preview — b8ec655a Deployed Nov 10, 2026 by vercel[bot]
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.

2 participants