Skip to content

Enhance GitHub workflows for code generation and auto-merge (#43) - #44

Merged
ptr727 merged 1 commit into
mainfrom
develop
Apr 13, 2026
Merged

Enhance GitHub workflows for code generation and auto-merge (#43)#44
ptr727 merged 1 commit into
mainfrom
develop

Conversation

@ptr727

Copy link
Copy Markdown
Owner

Improve security and prevent abuse in merge-bot workflows. Introduce a new workflow for running the codegen app and creating pull requests, along with a periodic workflow for weekly execution. Update documentation to include Git and commit rules for AI coding agents. Adjust date in the CodeGen class for accurate quote generation.


Signed-off-by: Pieter Viljoen ptr727@users.noreply.github.com

Improve security and prevent abuse in merge-bot workflows. Introduce a
new workflow for running the codegen app and creating pull requests,
along with a periodic workflow for weekly execution. Update
documentation to include Git and commit rules for AI coding agents.
Adjust date in the CodeGen class for accurate quote generation.
---------
Signed-off-by: Pieter Viljoen <ptr727@users.noreply.github.com>
CopilotAI review requested due to automatic review settings April 13, 2026 22:09

CopilotAI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR strengthens and expands the automation around CodeGen PR creation and auto-merge, adding an alternative GitHub App authentication path alongside the existing PAT approach, and documenting the operational/security rationale.

Changes:

  • Added a scheduled GitHub App–based CodeGen workflow and corresponding reusable workflow task.
  • Hardened merge-bot conditions to reduce abuse risk (same-repo checks, actor constraints) and updated docs accordingly.
  • Updated CodeGen internals and documentation for agent git/commit constraints and corrected generated quote timestamp.

Reviewed changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated 4 comments.

Show a summary per file
FileDescription
README.mdDocuments PAT vs GitHub App flows, auto-merge conditions, and the weekly schedule.
CodeGen/HttpClientFactory.csRefactors resilience handler creation and exposes CreateHttpClient() internally.
CodeGen/CodeGen.csUpdates generated quote timestamp value.
AGENTS.mdAdds “Git and Commit Rules” for AI agents.
.github/workflows/run-periodic-codegen-pull-request.ymlAdjusts schedule comment and concurrency group for PAT-based weekly codegen.
.github/workflows/run-periodic-codegen-app-pull-request.ymlAdds weekly scheduled workflow using GitHub App credentials.
.github/workflows/run-codegen-pull-request-task.ymlAdds clarifying comments for PAT/GITHUB_TOKEN usage.
.github/workflows/run-codegen-app-pull-request-task.ymlAdds reusable workflow to run codegen and open PR using a GitHub App token.
.github/workflows/merge-bot-pull-request.ymlTightens auto-merge if: guards and adds merge path for app-created codegen PRs.
.github/copilot-instructions.mdAdds “Git and Commit Rules” guidance for AI agents.

Comment thread.github/workflows/merge-bot-pull-request.yml
Comment threadCodeGen/HttpClientFactory.cs
Comment thread.github/workflows/run-periodic-codegen-pull-request.yml
Comment thread.github/workflows/run-periodic-codegen-app-pull-request.yml
@ptr727
ptr727 merged commit bd21912 into mainApr 13, 2026
40 checks passed
Sign up for freeto 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

@ptr727