Skip to content

feat(guard): treat any non-protected branch as agent-managed - #626

Merged
NagyVikt merged 1 commit into
mainfrom
agent/claude/accept-any-non-protected-branch-name-as-2026-06-05-00-40
Jun 4, 2026
Merged

feat(guard): treat any non-protected branch as agent-managed#626
NagyVikt merged 1 commit into
mainfrom
agent/claude/accept-any-non-protected-branch-name-as-2026-06-05-00-40

Conversation

@NagyVikt

@NagyVikt NagyVikt commented Jun 4, 2026

Copy link
Copy Markdown
Collaborator

Automated by gx branch finish (PR flow).

Agents were forced onto agent/* (plus a fixed claude/codex/cursor allowlist)
to edit or commit; ad-hoc names like vendor/* were blocked at both the
skill_guard.py PreToolUse layer and the git pre-commit hook. Being OFF a
protected base is the only load-bearing rule, so accept any non-protected
branch by default.

- skill_guard.py: is_agent_branch() allows any non-protected branch; prefix
  allowlist now applies only under GUARDEX_AGENT_BRANCH_PREFIXES_ONLY lockdown.
  Callers pass the repo-resolved protected set.
- templates/githooks/pre-commit: non-protected branches allowed for all agent
  sessions; new GUARDEX_REQUIRE_AGENT_BRANCH / multiagent.requireAgentBranch
  lockdown re-imposes agent/*. Protected blocking + Codex managed-only
  exception preserved.
- AGENTS.md/CLAUDE.md quickstart + OpenSpec change documenting the policy.
- Tests updated/added for default-allow, lockdown, and configured-protected.
@NagyVikt
NagyVikt merged commit 30dbf1b into main Jun 4, 2026
@NagyVikt
NagyVikt deleted the agent/claude/accept-any-non-protected-branch-name-as-2026-06-05-00-40 branch June 4, 2026 23:06
NagyVikt added a commit that referenced this pull request Jun 5, 2026
Cross-repo agent coordination release. Minor bump (new gx mcp feature +
backward-compatible default changes) covering PRs #626-#633. Release notes in
the OpenSpec change notes; GitHub release v7.1.0 published separately. Manual
bump + manual npm publish this cycle.

Co-authored-by: NagyVikt <nagy.viktordp@gmail.com>
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