Skip to content

deploy: fix rollback release-root path contamination#131

Merged
benvinegar merged 1 commit into
mainfrom
bentlegen/fix-rollback-release-root-path-normalization
Feb 22, 2026
Merged

deploy: fix rollback release-root path contamination#131
benvinegar merged 1 commit into
mainfrom
bentlegen/fix-rollback-release-root-path-normalization

Conversation

@benvinegar
Copy link
Copy Markdown
Member

Summary

  • fix bin/rollback-release.sh to normalize release paths after env/CLI parsing
  • ensure BAUDBOT_RELEASE_ROOT wins over inherited BAUDBOT_RELEASES_DIR, BAUDBOT_CURRENT_LINK, and BAUDBOT_PREVIOUS_LINK
  • add regression coverage in bin/rollback-release.test.sh for stale rollback path env vars

Validation

  • bin/rollback-release.test.sh
  • bin/update-release.test.sh
  • npm run test:shell

@greptile-apps
Copy link
Copy Markdown

greptile-apps Bot commented Feb 22, 2026

Greptile Summary

Mirrors the fix from PR #130 (update-release.sh) by ensuring BAUDBOT_RELEASE_ROOT always wins over inherited environment variables (BAUDBOT_RELEASES_DIR, BAUDBOT_CURRENT_LINK, BAUDBOT_PREVIOUS_LINK).

  • Changed --release-root argument parsing to set BAUDBOT_RELEASE_ROOT directly instead of immediately calling bb_refresh_release_paths
  • Added normalization call after all argument parsing completes, ensuring paths are derived from the final BAUDBOT_RELEASE_ROOT value
  • Added regression test with stale environment variables to verify the fix prevents path contamination

Confidence Score: 5/5

  • This PR is safe to merge with minimal risk
  • The change follows the exact same pattern as the previous fix in PR deploy: fix update-release release-root metadata path leak #130 for update-release.sh, includes comprehensive test coverage for the regression case, and the logic is straightforward - deferring path normalization until after all argument parsing ensures BAUDBOT_RELEASE_ROOT properly overrides stale environment variables
  • No files require special attention

Important Files Changed

Filename Overview
bin/rollback-release.sh Fixed --release-root CLI argument to set BAUDBOT_RELEASE_ROOT directly instead of calling bb_refresh_release_paths immediately, then normalized all release paths after argument parsing to prevent stale environment variable contamination
bin/rollback-release.test.sh Added test helper run_rollback_with_stale_release_paths and regression test test_rollback_release_root_overrides_stale_release_path_env to verify that BAUDBOT_RELEASE_ROOT correctly overrides stale inherited path environment variables

Last reviewed commit: c2d0dc7

@benvinegar benvinegar merged commit 98e154a into main Feb 22, 2026
10 checks passed
@benvinegar benvinegar deleted the bentlegen/fix-rollback-release-root-path-normalization branch February 22, 2026 22:49
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.

1 participant