mirror of
https://github.com/GoodOlClint/PSProxmoxVE.git
synced 2026-07-26 07:58:14 +00:00
Clint Branham
f21c7f84b7
- Replace tests/infrastructure/Dockerfile with tests/Dockerfile.test (single multi-stage Dockerfile for both CI and local dev) - CI container-image job now builds from Dockerfile.test target dev-infra - Add ARM support: PowerShell installed via dotnet tool on arm64, APT package on amd64 - Replace tests/dev.sh (bash) with tests/dev.ps1 (PowerShell) for cross-platform support (Windows, macOS, Linux) - Add -DockerHost parameter for running x86 containers on a remote Docker host from ARM Macs (rsyncs repo, uses SSH Docker transport) - Add -NoCleanup switch to keep nested PVE VMs after integration tests - integration command now provisions nested PVE VMs instead of testing against a pre-existing PVE directly - Share /opt/pve-isos host path between CI and local dev (was separate Docker named volume) - Delete tools/Invoke-Tests.ps1 (unused, overlapped with run-integration.sh) - Add .gitignore entries for Terraform state/artifacts - Update all documentation references Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
96 lines
3.5 KiB
Markdown
96 lines
3.5 KiB
Markdown
# PSProxmoxVE — Claude Code Instructions
|
|
|
|
## Project Overview
|
|
|
|
C# binary PowerShell module for managing Proxmox VE (PVE) infrastructure. Two projects:
|
|
- `src/PSProxmoxVE/` — Cmdlets and module surface (targets netstandard2.0)
|
|
- `src/PSProxmoxVE.Core/` — Services, models, HTTP client (targets netstandard2.0)
|
|
|
|
Tests: xUnit (`tests/PSProxmoxVE.Core.Tests/`) and Pester 5 (`tests/PSProxmoxVE.Tests/`).
|
|
|
|
## Development Workflow
|
|
|
|
**All changes go through pull requests.** The `main` branch has branch protection enabled
|
|
(required build checks, required review, admin enforced). Never push directly to main.
|
|
|
|
```bash
|
|
# Create a feature branch
|
|
git checkout -b feat/my-feature
|
|
|
|
# ... make changes ...
|
|
|
|
# Commit using conventional commits
|
|
git commit -m "feat: add new cmdlet"
|
|
|
|
# Push and create PR
|
|
git push -u origin feat/my-feature
|
|
gh pr create
|
|
```
|
|
|
|
### Dev container (recommended)
|
|
|
|
A Docker-based dev environment replicates the full CI setup locally. Works on ARM Macs
|
|
(build + test) and x86 (full provisioning flow).
|
|
|
|
```powershell
|
|
./tests/dev.ps1 # Open pwsh shell in dev container
|
|
./tests/dev.ps1 build # Build the module
|
|
./tests/dev.ps1 test # Run unit tests (ARM + x86)
|
|
./tests/dev.ps1 integration # Provision nested PVE, run tests, cleanup (x86 only)
|
|
./tests/dev.ps1 provision # Provision nested PVE only, no tests (x86 only)
|
|
```
|
|
|
|
Configure parent PVE credentials by copying `tests/.env.test.example` to `tests/.env.test`.
|
|
|
|
### Build & test without container
|
|
|
|
```bash
|
|
# Build
|
|
dotnet build PSProxmoxVE.sln
|
|
|
|
# xUnit tests
|
|
dotnet test tests/PSProxmoxVE.Core.Tests/
|
|
|
|
# Pester tests (requires pwsh)
|
|
pwsh -Command "Invoke-Pester tests/PSProxmoxVE.Tests/ -Output Detailed"
|
|
|
|
# Run all tests via dev container
|
|
./tests/dev.ps1 test
|
|
```
|
|
|
|
## Key Conventions
|
|
|
|
- All cmdlets use `Pve` noun prefix
|
|
- All cmdlet classes must be `sealed`
|
|
- All cmdlets must have `[OutputType]` attribute
|
|
- Destructive cmdlets must set `ConfirmImpact = ConfirmImpact.High`
|
|
- VmId parameters: `[ValidateRange(100, 999999999)]`, nullable when optional
|
|
- JSON: Newtonsoft.Json only (`[JsonProperty]`), no System.Text.Json attributes
|
|
- Task polling: always use `TaskService.WaitForTask`, never inline loops
|
|
- Passwords: `SecureString` type, never plain `string`
|
|
- URL paths: `Uri.EscapeDataString()` on all dynamic path segments
|
|
- No bare `catch {}` blocks — use specific or filtered exceptions
|
|
- Verb class constants required (`VerbsCommon.Get`, not `"Get"`)
|
|
|
|
## Review System
|
|
|
|
This repo uses a structured review system to track findings and prevent regressions.
|
|
|
|
### Key files
|
|
- `docs/review/findings.json` — stable findings database. IDs are permanent (F001, F002...).
|
|
Never renumber. Read this before any coding session to understand open issues.
|
|
- `docs/review/REVIEW_REPORT.md` — latest full review report (scan-7, 2026-03-23)
|
|
- `DECISIONS.md` — architectural decisions and anti-patterns. **Read this before writing
|
|
any new code.** It documents patterns that were deliberately chosen or changed and must
|
|
not be reintroduced.
|
|
|
|
### Before starting a coding session
|
|
1. Read `DECISIONS.md` to understand established patterns
|
|
2. Check `docs/review/findings.json` for open findings relevant to the area you're working in
|
|
3. Do not introduce patterns listed as anti-patterns in DECISIONS.md
|
|
|
|
### Finding ID stability
|
|
Finding IDs (F001, F002...) are permanent. A resolved finding is never deleted from
|
|
findings.json — it is marked `resolved` with evidence of the fix. If a finding reappears,
|
|
it is marked `regressed` and retains its original ID.
|