Skip to content

3136: Run llm code review - #3137

Open
f1sh1918 wants to merge 5 commits into
mainfrom
3136-run-llm-code-review
Open

3136: Run llm code review#3137
f1sh1918 wants to merge 5 commits into
mainfrom
3136-run-llm-code-review

Conversation

@f1sh1918

@f1sh1918 f1sh1918 commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Short Description

Proposed Changes

  • copy get_access_token from lunes-app
  • add llm-pr-review.py
  • add llm-pr-review job and workflow in circle ci
    -add github workflow that triggers circle ci workflow on pr opened
  • update docs

Side Effects

  • N/A

Testing

  • Please check the AI prompt for improvements
  • Check pipeline and pr comment

Resolved Issues

Fixes: #3136

@deliverino

deliverino Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

LLM Review (verdigado-think)

CircleCI Configuration

The changes to .circleci/config.yml are accompanied by corresponding changes in .circleci/src/ (@common.yml, jobs/llm-pr-review.yml, and workflows/llm_review.yml), following the project's auto-generation pattern.

Commit Messages

The commit messages follow the project convention (<issue number>: <message>) and use the present tense.

Labels

The PR currently has no labels. Since these changes are restricted to CI/CD pipelines, scripts, and documentation and have no user-facing effect on the platform, the maintenance label should be added.

Security

  • Secret Handling: The implementation correctly relies on environment variables and GitHub/CircleCI secrets (DELIVERINO_PRIVATE_KEY, CIRCLECI_API_TOKEN, NB_LLM_API_TOKEN) rather than hardcoding sensitive data.
  • Permissions: The GitHub Action .github/workflows/llm-pr-review.yml explicitly sets permissions: {} to follow the principle of least privilege, as it only needs to trigger an external API call.

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.

Run LLM review on commit and pull request open

1 participant