Skip to content

feat: substitute LLM bot authors with human assignees in release notes#253

Open
Copilot wants to merge 3 commits intomainfrom
copilot/update-auto-releasers-logic
Open

feat: substitute LLM bot authors with human assignees in release notes#253
Copilot wants to merge 3 commits intomainfrom
copilot/update-auto-releasers-logic

Conversation

Copy link

Copilot AI commented Mar 17, 2026

  • Explore Atmos pattern and existing inline scripts
  • Create .github/actions/substitute-llm-authors/action.yml composite action
  • Move src/release-author-substitutor.js.github/actions/substitute-llm-authors/release-author-substitutor.js
  • Update test/release-author-substitutor.test.js import path
  • Create .github/actions/release-commenter/action.yml composite action
  • Update shared-auto-release.yml to use ./.github/actions/substitute-llm-authors
  • Update shared-release-branches.yml to use ./.github/actions/release-commenter
  • Run tests — 34/34 pass
  • CodeQL security scan — 0 alerts

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@mergify mergify bot added triage Needs triage wip Work in Progress: Not ready for final review or merge labels Mar 17, 2026
- Add src/release-author-substitutor.js with utility functions for
  detecting LLM/AI bot users and replacing their author mentions with
  the PR's human assignee in release note bodies
- Add test/release-author-substitutor.test.js with 34 unit tests
  (Node.js built-in test runner, all passing)
- Add package.json with npm test script
- Update shared-auto-release.yml with new llm-users input and a
  post-processing step that patches the release body when LLM-authored
  PRs are found

Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
Copilot AI changed the title [WIP] Update auto releasers to show human assignee in release notes feat: substitute LLM bot authors with human assignees in release notes Mar 17, 2026
Copilot AI requested a review from nitrocode March 17, 2026 16:47
@nitrocode nitrocode marked this pull request as ready for review March 17, 2026 16:54
@nitrocode nitrocode requested review from a team as code owners March 17, 2026 16:54
@nitrocode nitrocode requested a review from joe-niland March 17, 2026 16:54
@osterman
Copy link
Member

Please look at the localized GitHub Actions pattern that we use in the Atmos repository. Instead of inlining a lot of GitHub script in workflows, move them to composite actions, co-located in the repo, and then reference them with relative paths. Again, see Atmos for examples.

@mergify mergify bot removed the wip Work in Progress: Not ready for final review or merge label Mar 17, 2026
@mergify
Copy link

mergify bot commented Mar 17, 2026

Important

Cloud Posse Engineering Team Review Required

This pull request modifies files that require Cloud Posse's review. Please be patient, and a core maintainer will review your changes.

To expedite this process, reach out to us on Slack in the #pr-reviews channel.

@mergify mergify bot added the needs-cloudposse Needs Cloud Posse assistance label Mar 17, 2026
Extract the two large inline github-script blocks from reusable workflows
into composite actions co-located in .github/actions/, following the Atmos
repository pattern.

- .github/actions/substitute-llm-authors/action.yml: composite action
  extracted from shared-auto-release.yml; uses process.env instead of
  template interpolation for the release-id (security improvement)
- .github/actions/substitute-llm-authors/release-author-substitutor.js:
  moved from src/ to co-locate with the composite action
- .github/actions/release-commenter/action.yml: composite action extracted
  from shared-release-branches.yml; cleaned up var/for-in patterns
- shared-auto-release.yml: 116-line inline script → 4-line composite action call
- shared-release-branches.yml: 131-line inline script → 2-line composite action call
- test/release-author-substitutor.test.js: updated import path

Co-authored-by: nitrocode <7775707+nitrocode@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs-cloudposse Needs Cloud Posse assistance triage Needs triage

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants