Skip to content

feat(claude): web/public media exemption in git-staging-guard - #14

Merged
peter-svensson merged 1 commit into
mainfrom
claude-hook-web-media-exemption
Jun 12, 2026
Merged

feat(claude): web/public media exemption in git-staging-guard#14
peter-svensson merged 1 commit into
mainfrom
claude-hook-web-media-exemption

Conversation

@peter-svensson

Copy link
Copy Markdown
Owner

The ambix web repo serves images and video straight from the repo (no CDN — see ambix-ai/web#158 and the upcoming video PR). The git-staging-guard hook blocked that workflow twice:

  1. Size: screenshots >1MB hit the global cap → 3MB cap for media under public/ in the web repo (and its web.* worktrees).
  2. Extension: .mp4 is in the binary blocklist → mp4/webm (plus the image formats) pass the extension check under the same scoped path.

Shared is_web_public_media() helper gates both. Verified: 1MB+ png and mp4 pass under web*/public/; mp4 still blocked in any other repo; shellcheck clean.

Both changes were applied live via chezmoi during the landing-page screenshot/video work (Peter authorized each interactively); this PR lands them in source control.

The ambix web repo serves images and video straight from the repo (no
CDN). Allow png/jpg/webp/avif/svg/gif/mp4/webm under public/ in the web
repo (and its web.* worktrees) through the binary-extension check, with
a 3MB cap instead of the global 1MB. Everything else unchanged; mp4
stays blocked outside web/public.
@peter-svensson
peter-svensson merged commit 4d3f29c into main Jun 12, 2026
1 check passed
@peter-svensson
peter-svensson deleted the claude-hook-web-media-exemption branch June 12, 2026 14:54
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