intake(lote-C): import upstream PR #1599#12
Open
nsalvacao wants to merge 3 commits intobaseline/develop-sync-2026-02-17from
Open
intake(lote-C): import upstream PR #1599#12nsalvacao wants to merge 3 commits intobaseline/develop-sync-2026-02-17from
nsalvacao wants to merge 3 commits intobaseline/develop-sync-2026-02-17from
Conversation
…on failures Clear git environment variables (GIT_INDEX_FILE, GIT_DIR, GIT_WORK_TREE, etc.) from both frontend and backend subprocess environments to prevent contamination from parent processes (pre-commit hooks, IDE integrations, Electron app). Changes: - Frontend: Wrap getAugmentedEnv() with getIsolatedGitEnv() when spawning Python agent subprocess - Backend: Delete git env vars from os.environ before SDK instantiation (SDK merges with os.environ, so deletion is required - setting to "" fails) - Backend: Add HUSKY=0 to disable pre-commit hooks during agent operations This provides defense-in-depth against git environment contamination that can cause "Could not reset index file to revision 'HEAD'" errors in worktrees. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Move GIT_ENV_VARS_TO_CLEAR import from inside create_client() function to module-level imports section, following PEP 8 best practices. Add brief explanatory comment at import site while keeping detailed operational comments at the deletion site for context. Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
|
Warning You have reached your daily quota limit. Please wait up to 24 hours and I will start processing your requests again! |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Upstream intake PR for isolated review.
intake/lote-C-pr-1599baseline/develop-sync-2026-02-17Notes:
apps/backend/core/client.pyfor later resolution.docs/intake-conflicts/pr-1599-conflict-notes.mdto guide resolution.Checklist: