Skip to content

chore: add gt operational directories to .gitignore#196

Open
jrf0110 wants to merge 1 commit intomainfrom
merge/hq-rim
Open

chore: add gt operational directories to .gitignore#196
jrf0110 wants to merge 1 commit intomainfrom
merge/hq-rim

Conversation

@jrf0110
Copy link
Contributor

@jrf0110 jrf0110 commented Feb 14, 2026

Summary

  • Adds .gitignore entries for Gas Town operational directories (.runtime/, .claude/, .beads/, .logs/)
  • Adds git-lfs husky hooks (post-checkout, post-commit, post-merge, pre-push)

Source

Polecat: chrome (branch polecat/chrome/hq-rim@mlmw0c0d)
MR: ki-cwh

Test plan

  • Config-only change — no source code modified
  • Rebased cleanly on main (already up to date)

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
.env*.local

# Gas Town (added by gt)
.runtime/
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

SUGGESTION: Anchor ignore pattern to repo root to avoid accidentally ignoring nested .runtime/ directories

In a monorepo, .runtime/ without a leading slash will ignore that directory name anywhere, including inside subprojects. If this is intended to be root-only, consider anchoring with a leading /.

Suggested change
.runtime/
/.runtime/

@@ -0,0 +1,3 @@
#!/bin/sh
command -v git-lfs >/dev/null 2>&1 || { printf >&2 "\n%s\n\n" "This repository is configured for Git LFS but 'git-lfs' was not found on your path. If you no longer wish to use Git LFS, remove this hook by deleting the 'pre-push' file in the hooks directory (set by 'core.hookspath'; usually '.git/hooks')."; exit 2; }
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

WARNING: Hook hard-fails when git-lfs is not installed

This hook exits non-zero (exit 2) if git-lfs isn’t on PATH, which will block git push for contributors/CI environments that don’t have LFS installed. If LFS is required, consider ensuring installation is enforced via onboarding/CI and making the hook message non-blocking, or gating this behavior to only repos that truly require LFS for the push.

@kiloconnect
Copy link
Contributor

kiloconnect bot commented Feb 14, 2026

Code Review Summary

Status: 2 Issues Found | Recommendation: Address before merge

Overview

Severity Count
CRITICAL 0
WARNING 1
SUGGESTION 1
Issue Details (click to expand)

WARNING

File Line Issue
.husky/_/pre-push 2 Hook hard-fails when git-lfs is not installed (blocks git push)

SUGGESTION

File Line Issue
.gitignore 85 Consider anchoring .runtime/ ignore to repo root (/.runtime/) to avoid ignoring nested directories
Other Observations (not in diff)

Issues found in unchanged code that cannot receive inline comments:

File Line Issue
Files Reviewed (5 files)
  • .gitignore - 1 issue
  • .husky/_/post-checkout
  • .husky/_/post-commit
  • .husky/_/post-merge
  • .husky/_/pre-push - 1 issue

@jrf0110 jrf0110 enabled auto-merge (squash) February 14, 2026 22:41
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