mirror of
https://github.com/rcourtman/Pulse.git
synced 2026-09-21 02:33:32 +00:00
rcourtman
c9c4a07508
Added TestPVESetupScriptArgumentAlignment to prevent future fmt.Sprintf argument mismatch bugs in the PVE quick setup script template. The test uses sentinel values (SENTINEL_URL, SENTINEL_HOST, deadbeef...) to verify that critical placeholders receive the correct argument types: ✓ Repair block INSTALLER_URL uses pulseURL (not authToken) ✓ Repair --pulse-server flags use pulseURL (not authToken) ✓ Authorization headers use runtime $AUTH_TOKEN variable (not hardcoded) ✓ Token ID uses tokenName (pulse-*) (not pulseURL or authToken) This test would have caught the bugs fixed in commits 2bb73d3c7 and 2053bc5e2, where: - authToken appeared in --pulse-server URLs (argument shift) - Authorization headers were hardcoded instead of using runtime variable Recommended by Codex as a safeguard against this class of regression.
Internal API Package
This directory contains the API server implementation for Pulse.
Important Note About frontend-modern/
The frontend-modern/ subdirectory that appears here is:
- AUTO-GENERATED during builds
- NOT the source code - just a build artifact
- IN .gitignore - never committed
- REQUIRED BY GO - The embed directive needs it here
Frontend Development Location
👉 Edit frontend files at: /opt/pulse/frontend-modern/src/
Why This Structure?
Go's //go:embed directive has limitations:
- Cannot use
../paths to access parent directories - Cannot follow symbolic links
- Must embed files within the Go module
This is a known Go limitation and our structure works around it.