Skip to main content
← Back to list
01Issue
FeatureShippedExtensionsPublic
Assigneesskunk-ape

Relationships

#2637 gatorwalk-factory: integration harness should strip every SWAMP_ variable except an allowlist

Opened by skunk-ape · 9/28/2026· Shipped 9/29/2026

Follow-up to #2619 (GW-7, PR 339), from the verify-reviews code review.

The harness removes a hand-kept list of SWAMP_ environment variables (SWAMP_REPO_DIR, SWAMP_WORKFLOWS_DIR, the serve server URL and token ones) before each swamp call, so swamp cannot be pointed at another repo or server. If swamp adds a new variable with that effect, the list goes stale and nothing notices.

Proposal: remove every variable starting with SWAMP_ except a short explicit allowlist of ones the suite needs (if any), and say so in DESIGN.md. Low severity.

02Bog Flow
✓OPEN✓TRIAGED✓IN PROGRESS✓SHIPPED+ 1 MOREASSIGNED+ 2 MOREREVIEW+ 15 MOREPR_LINKED+ 2 MORESESSION_SUMMARIZED

Shipped

9/29/2026, 2:49:07 PM

Click a lifecycle step above to view its details.

03Sludge Pulse
skunk-ape assigned skunk-ape9/29/2026, 2:28:08 AM

Sign in to post a ripple.