Skip to content

Allow VS Code commits on protected local-only branches#86

Merged
NagyVikt merged 2 commits intomainfrom
agent/codex/pia-edix-hu-dup-2-vscode-local-branch-commit-allowance
Apr 13, 2026
Merged

Allow VS Code commits on protected local-only branches#86
NagyVikt merged 2 commits intomainfrom
agent/codex/pia-edix-hu-dup-2-vscode-local-branch-commit-allowance

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Summary\n- allow VS Code Source Control commits on protected branches that exist only locally (no upstream and no remote counterpart)\n- keep protected-branch blocking for shared/remote branches and for Codex/OMX sessions\n- stop using TERM_PROGRAM=vscode as a VS Code SCM signal to avoid terminal false positives\n\n## Validation\n- node --test test/install.test.js\n- npm test

NagyVikt and others added 2 commits April 13, 2026 23:28
Users working in local-only feature branches inside VS Code Source Control should not be blocked by protected-branch hooks when no remote/upstream branch exists yet. The hook now keeps protection for shared/remote branches while permitting local-only VS Code commits, and codex-session protection remains unchanged.

Constraint: Codex/OMX sessions must stay blocked on protected branches

Rejected: Keep TERM_PROGRAM=vscode as VS Code context signal | over-matches terminal sessions and weakens branch guardrails

Rejected: Allow all protected-branch VS Code commits | would permit unsafe writes on shared protected branches

Confidence: high

Scope-risk: narrow

Reversibility: clean

Directive: Keep local-only exception tied to both missing upstream and missing remote counterpart

Tested: node --test test/install.test.js; npm test
@NagyVikt NagyVikt merged commit 1480802 into main Apr 13, 2026
7 checks passed
@NagyVikt NagyVikt deleted the agent/codex/pia-edix-hu-dup-2-vscode-local-branch-commit-allowance branch April 13, 2026 21:37
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