checkout-preview should also use sdk preview branch

[rgarcia]

No description provided.

[mesa-dot-dev]

Mesa Description

This pull request updates the checkout-preview functionality to utilize the sdk preview branch. This change ensures that the preview process aligns with the SDK's preview branch, likely for early integration or testing purposes.

^{Description generated by Mesa. Update settings}

[mesa-dot-dev]

Performed full review of 7a633ef...d642eb6

Analysis

1. Environment Variable Persistence Issue: The use-sdk-preview script exports GOPRIVATE, but when invoked as a subprocess from checkout-preview, this export doesn't persist to the user's shell, potentially causing subsequent Go commands to fail when accessing the private SDK repo.

2. Branch vs Commit Disambiguation Bug: The regex pattern for detecting commit SHAs will incorrectly classify branch names that happen to be valid hex strings as commits, leading to unexpected failures when using such branch names.

3. Lack of Error Recovery: The script doesn't include cleanup traps, so if failures occur after cloning the temporary repo but before completion, temporary directories persist unnecessarily.

4. Git Configuration Validation Weakness: The current approach to checking git configuration may produce false negatives if multiple URL rewrites exist.

Tip

Help

Slash Commands:

• /review - Request a full code review
• /review latest - Review only changes since the last review
• /describe - Generate PR description. This will update the PR body or issue comment depending on your configuration
• /help - Get help with Mesa commands and configuration options

^{3 files reviewed | 0 comments | Edit Agent Settings • Read Docs}