Skip to content

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

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

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

Conversation

Copy link
Copy Markdown

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 RB (nitrocode) March 17, 2026 16:47
@nitrocode RB (nitrocode) marked this pull request as ready for review March 17, 2026 16:54
@nitrocode RB (nitrocode) requested review from a team as code owners March 17, 2026 16:54
@osterman
Copy link
Copy Markdown
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
Copy Markdown

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