Skip to content

AX-1792: Remove JFROG_URL validation from SessionStart injector#30

Merged
MatanEden1 merged 1 commit into
mainfrom
AX-1792-jfrog-url-check-before-agent-guard-force-enable
Jul 7, 2026
Merged

AX-1792: Remove JFROG_URL validation from SessionStart injector#30
MatanEden1 merged 1 commit into
mainfrom
AX-1792-jfrog-url-check-before-agent-guard-force-enable

Conversation

@MatanEden1

Copy link
Copy Markdown
Collaborator

Summary

  • The JFROG_URL missing/trailing-slash check ran before the Agent Guard force-enable/settings check, so it could exit the SessionStart hook even when Agent Guard was disabled or force-enabled.
  • Removes the JFROG_URL validation block entirely from plugin/scripts/inject-instructions.mjs.

Test plan

  • node scripts/validate-hook-injector.mjs passes
  • Manually confirm SessionStart hook still injects instructions when Agent Guard is enabled/force-enabled, with and without JFROG_URL set

🤖 Generated with Claude Code

The check exited the hook before the Agent Guard force-enable check
ran, so a missing/malformed JFROG_URL blocked sessions even when
Agent Guard wasn't enabled or was force-enabled to bypass checks.

Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@MatanEden1 MatanEden1 merged commit 7f8e069 into main Jul 7, 2026
1 check passed
@MatanEden1 MatanEden1 deleted the AX-1792-jfrog-url-check-before-agent-guard-force-enable branch July 7, 2026 08:51
MatanEden1 pushed a commit that referenced this pull request Jul 7, 2026
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.

2 participants