Skip to content

Document tag-driven releases and Control Tower roadmap - #15

Merged
pramodbn27 merged 2 commits into
mainfrom
agent/release-automation-and-control-tower-docs
Aug 16, 2026
Merged

Document tag-driven releases and Control Tower roadmap#15
pramodbn27 merged 2 commits into
mainfrom
agent/release-automation-and-control-tower-docs

Conversation

@pramodbn27

Copy link
Copy Markdown
Contributor

What changed

This PR packages two related documentation updates:

  • clarifies the release process in AGENTS.md and updates release-pypi.yml so tag pushes drive PyPI publish, GitHub Release creation, and MCP Registry publish from a single flow
  • expands ROADMAP.md with the first planned Control Tower integration scope, including read-only inventory/health/posture queries and explicit guardrails around write-capable actions

Why

The release guidance was too terse for a repo that now has multiple release artifacts, and the roadmap needed a clearer statement of how this package should relate to agenticops-control-tower instead of reimplementing control-plane behavior locally.

Impact

Maintainers get a clearer release checklist and more aligned automation, while roadmap readers get a more concrete picture of the intended Control Tower integration boundary.

Validation

These are documentation and workflow-file changes only; no code or test commands were required.

@pramodbn27
pramodbn27 marked this pull request as ready for review August 16, 2026 05:11
@pramodbn27
pramodbn27 requested a review from a team as a code owner August 16, 2026 05:11
@pramodbn27
pramodbn27 merged commit dec8e8f into main Aug 16, 2026
5 checks passed

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: 035b166843

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

publish-mcp-registry:
create-github-release:
runs-on: ubuntu-latest
needs: publish-pypi

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Validate release notes before publishing to PyPI

When a pushed tag lacks the exact ## <version> - changelog heading, publish-pypi has already completed before this job runs, so the extractor exits after an immutable PyPI version has been published but before the GitHub Release and MCP Registry entry are created. Correcting the changelog in a later commit will not change the tagged checkout, while rerunning the whole workflow can fail on the duplicate PyPI upload; perform the deterministic changelog extraction/validation before the publish job and pass the generated notes forward.

AGENTS.md reference: AGENTS.md:L170-L179

Useful? React with 👍 / 👎.

Comment on lines 8 to +9
permissions:
contents: read
contents: write

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Restrict contents write access to the release job

This workflow-level grant is inherited by the build job, so checkout, dependency setup, build commands, and third-party actions all run with a token capable of modifying repository contents even though only create-github-release needs that privilege. A compromised dependency or action during a tag build could therefore alter tags or repository content; keep workflow/build access at contents: read and grant contents: write only on the release-creation job.

Useful? React with 👍 / 👎.

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