Fix repo-root guard to work inside a git worktree

[jnasbyupgrade]

Fixes #38.

setup.sh, pgxntool-sync.sh, and update-setup-files.sh guarded the project root with [ -d .git ]. In a linked git worktree (git worktree add), .git is a file (a gitdir: pointer), not a directory, so the guard failed on a perfectly valid working tree:

• setup.sh wrongly re-ran git init
• the sync scripts aborted with ERROR: Not in a git repository.

All three now detect the repo with git rev-parse --git-dir, which works in both a normal clone and a worktree.

Verification

• Reproduced the failure with the old guard and confirmed the fix passes, running the script from inside a real worktree.
• Full suite green (212/212) on PG 17.

Related pgxntool-test PR: Postgres-Extensions/pgxntool-test#24

🤖 Generated with Claude Code

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Pro Plus

Run ID: 355f3df2-34d4-4a74-a394-5f5c73f0f374

📥 Commits

Reviewing files that changed from the base of the PR and between b062fca and b9af993.

📒 Files selected for processing (4)

• HISTORY.asc
• pgxntool-sync.sh
• setup.sh
• update-setup-files.sh

---

📝 Walkthrough

Walkthrough

The setup, synchronization, and setup-file update scripts now detect Git repositories with git rev-parse --git-dir instead of requiring .git to be a directory. This allows the scripts to operate correctly in Git worktrees, where .git is a file. The changelog adds a new stable entry documenting these fixes.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related issues

• Postgres-Extensions/pgxntool issue 17 — Covers replacing .git directory checks with worktree-compatible Git repository detection in the same setup scripts.

Poem

A rabbit hops where worktrees grow,
.git is a file, not a folder below.
The scripts now ask Git, “Are we home?”
No false alarms, no need to roam.
Setup syncs with a cheerful glow.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: making repo-root checks work in Git worktrees.
Description check	✅ Passed	The description accurately describes the worktree guard fix and related script behavior changes.
Linked Issues check	✅ Passed	The scripts now use a worktree-safe Git repo check, satisfying the linked issue's requirements.
Out of Scope Changes check	✅ Passed	The only extra change is the history note, which is directly related to the fix and not unrelated scope.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}