Skip to content

Brand contributor check comments as AgenTrust#16

Merged
imran-siddique merged 3 commits into
agentrust-io:mainfrom
carloshvp:agenttrust-contributor-check-branding
Jun 29, 2026
Merged

Brand contributor check comments as AgenTrust#16
imran-siddique merged 3 commits into
agentrust-io:mainfrom
carloshvp:agenttrust-contributor-check-branding

Conversation

@carloshvp

@carloshvp carloshvp commented Jun 27, 2026

Copy link
Copy Markdown
Member

What changed

Rebrands the public contributor-check PR/issue comment as AgenTrust-owned while keeping the existing pinned Microsoft Agent Governance Toolkit implementation for the risk checks.

The composite action now runs a small post-processing helper after the AGT check. The helper finds the existing contributor-check comment by its hidden idempotency marker, preserves that marker so future runs update the same comment, and replaces the visible footer with:

Automated check by AgenTrust Contributor Check

Why

The check is part of the AgenTrust repository workflow, so the visible automation should read as AgenTrust infrastructure. The upstream AGT dependency remains acknowledged in the action description and implementation comments, preserving attribution without exposing Microsoft AGT as the user-facing brand on every PR.

Validation

  • python3 -m py_compile .github/actions/contributor-check/brand_comment.py
  • Local _branded_body smoke test confirmed:
    • hidden marker is preserved
    • visible footer changes to AgenTrust
    • visible AGT Contributor Check footer is removed
  • ruby -e "require 'yaml'; YAML.load_file('.github/actions/contributor-check/action.yml')"
  • git diff --check

@carloshvp carloshvp marked this pull request as ready for review June 27, 2026 11:10
@imran-siddique

Copy link
Copy Markdown
Contributor

Thanks for this, the intent is right. One brand fix before we merge: the name is AgenTrust (single T, Agen + Trust), not AgentTrust. Please update the footer to AgenTrust Contributor Check and fix the other occurrences in this PR (brand_comment.py, action.yml). We are standardizing on AgenTrust across the org, so single T everywhere. Otherwise this looks good.

@carloshvp carloshvp changed the title Brand contributor check comments as AgentTrust Brand contributor check comments as AgenTrust Jun 29, 2026
@carloshvp

carloshvp commented Jun 29, 2026

Copy link
Copy Markdown
Member Author

Fixed, thanks for catching that.

Updated the PR-owned occurrences from AgentTrust to AgenTrust in:

  • .github/actions/contributor-check/brand_comment.py
  • .github/actions/contributor-check/action.yml
  • PR title/body

I also renamed the internal footer constant from AGENTTRUST_FOOTER to AGENTRUST_FOOTER, so the action files no longer contain AgentTrust, AGENTTRUST, or lowercase agenttrust.

Validation rerun locally:

  • python3 -m py_compile .github/actions/contributor-check/brand_comment.py
  • local _branded_body smoke test confirms the footer is now AgenTrust Contributor Check, the hidden marker is preserved, and the old AGT-visible footer is removed
  • ruby -e "require 'yaml'; YAML.load_file('.github/actions/contributor-check/action.yml')"
  • git diff --check

@imran-siddique imran-siddique merged commit b117024 into agentrust-io:main Jun 29, 2026
1 check passed
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