Skip to content

801: pls work#833

Closed
naanci wants to merge 3 commits intomainfrom
801-ai
Closed

801: pls work#833
naanci wants to merge 3 commits intomainfrom
801-ai

Conversation

@naanci
Copy link
Collaborator

@naanci naanci commented Mar 5, 2026

801

Description of changes

Checklist before review

  • I have done a thorough self-review of the PR
  • Copilot has reviewed my latest changes, and all comments have been fixed and/or closed.
  • If I have made database changes, I have made sure I followed all the db repo rules listed in the wiki here. (check if no db changes)
  • All tests have passed
  • I have successfully deployed this PR to staging
  • I have done manual QA in both dev (and staging if possible) and attached screenshots below.

Screenshots

Dev

Staging

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Available PR Commands

  • /ai - Triggers all AI review commands at once
  • /review - AI review of the PR changes
  • /describe - AI-powered description of the PR
  • /improve - AI-powered suggestions
  • /deploy - Deploy to staging

See: https://github.com/tahminator/codebloom/wiki/CI-Commands

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Workflow Trigger Scope

The push_commands are configured to run specific AI review actions on pushes. Confirm if running /describe, /review, and /improve on every commit/push is the intended behavior, or if these commands should be triggered only for specific events, such as PR creation or updates, to optimize AI usage and reduce potential noise.

github_action_config.push_commands: '["/describe", "/review", "/improve"]'

@naanci
Copy link
Collaborator Author

naanci commented Mar 5, 2026

/deploy

@tahminator
Copy link
Owner

/review

@tahminator
Copy link
Owner

/describe

@tahminator
Copy link
Owner

/improve

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

Preparing PR description...

@github-actions
Copy link
Contributor

github-actions bot commented Mar 5, 2026

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 3 🔵🔵🔵⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ Recommended focus areas for review

Indirect Triggering

The new triggerAIOnSync job posts comments (/review, /describe, /improve) to trigger the AI review process on pull_request synchronize events. This indirect method adds extra steps and comments to the PR history. Consider if the AI review jobs could be triggered more directly from triggerAIOnSync or if the getPRHead job's condition could be adjusted to handle synchronize events without relying on issue_comment events.

triggerAIOnSync:
  name: Trigger AI review on push
  runs-on: ubuntu-latest
  if: github.event_name == 'pull_request' && github.event.action == 'synchronize'
  steps:
    - name: Checkout Repository
      uses: actions/checkout@v4

    - name: Setup CI
      uses: ./.github/composite/setup-ci
      with:
        GPG_PRIVATE_KEY: ${{ secrets.GPG_PRIVATE_KEY }}
        GPG_PASSPHRASE: ${{ secrets.GPG_PASSPHRASE }}

    - name: Load secrets
      uses: ./.github/composite/load-secrets
      with:
        GPG_PRIVATE_KEY: ${{ secrets.GPG_PRIVATE_KEY }}
        GPG_PASSPHRASE: ${{ secrets.GPG_PASSPHRASE }}
        UNLOAD_ENVIRONMENTS: ci

    - name: Post /review command
      uses: ./.github/composite/send-message
      with:
        prId: ${{ github.event.number }}
        message: "/review"
        token: ${{ env.GH_PAT }}

    - name: Post /describe command
      uses: ./.github/composite/send-message
      with:
        prId: ${{ github.event.number }}
        message: "/describe"
        token: ${{ env.GH_PAT }}

    - name: Post /improve command
      uses: ./.github/composite/send-message
      with:
        prId: ${{ github.event.number }}
        message: "/improve"
        token: ${{ env.GH_PAT }}

@naanci naanci closed this Mar 5, 2026
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