mirror of
https://github.com/GoodOlClint/PSProxmoxVE.git
synced 2026-08-10 14:26:52 +00:00
dependabot[bot]
aff2ffc752
Bumps [actions/checkout](https://github.com/actions/checkout) from 6 to 7. - [Release notes](https://github.com/actions/checkout/releases) - [Changelog](https://github.com/actions/checkout/blob/main/CHANGELOG.md) - [Commits](https://github.com/actions/checkout/compare/v6...v7) --- updated-dependencies: - dependency-name: actions/checkout dependency-version: '7' dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com>
57 lines
2.2 KiB
YAML
57 lines
2.2 KiB
YAML
name: Claude Code Review
|
|
|
|
on:
|
|
pull_request:
|
|
types: [opened, synchronize, ready_for_review, reopened]
|
|
|
|
jobs:
|
|
claude-review:
|
|
if: ${{ !github.event.pull_request.draft }}
|
|
runs-on: ubuntu-latest
|
|
permissions:
|
|
contents: read
|
|
pull-requests: write
|
|
issues: write
|
|
id-token: write
|
|
|
|
steps:
|
|
- name: Checkout repository
|
|
uses: actions/checkout@v7
|
|
with:
|
|
fetch-depth: 0
|
|
|
|
- name: Run Claude Code Review
|
|
id: claude-review
|
|
uses: anthropics/claude-code-action@v1
|
|
with:
|
|
claude_code_oauth_token: ${{ secrets.CLAUDE_CODE_OAUTH_TOKEN }}
|
|
allowed_bots: 'dependabot[bot]'
|
|
track_progress: true
|
|
prompt: |
|
|
REPO: ${{ github.repository }}
|
|
PR NUMBER: ${{ github.event.pull_request.number }}
|
|
|
|
Review this pull request for the PSProxmoxVE PowerShell module.
|
|
|
|
Focus areas:
|
|
1. **DECISIONS.md compliance** — Check against the 13 architectural
|
|
decisions (D001-D013). Any violation is a regression.
|
|
2. **Code quality** — Cmdlet conventions (sealed, OutputType,
|
|
ConfirmImpact.High for destructive, VmId ValidateRange),
|
|
SecureString for passwords, Uri.EscapeDataString on path params,
|
|
no bare catch blocks, Newtonsoft-only JSON.
|
|
3. **API correctness** — Parameter names and enum values must match
|
|
the PVE OpenAPI spec (see tests/PSProxmoxVE.Core.Tests/Fixtures/
|
|
pve-api-enums.pve*.json for valid values per PVE version).
|
|
4. **Tests** — New cmdlets should have xUnit service tests and
|
|
Pester parameter-validation tests.
|
|
5. **Security** — No hardcoded credentials, no secrets in logs,
|
|
TLS verification on by default.
|
|
|
|
Provide inline comments for specific issues and a summary comment
|
|
for general observations. Skip nitpicks unless they indicate a
|
|
real problem.
|
|
|
|
claude_args: |
|
|
--allowedTools "mcp__github_inline_comment__create_inline_comment,Bash(gh pr comment:*),Bash(gh pr diff:*),Bash(gh pr view:*)"
|