4 Commits

Include renames

Author	SHA1	Message	Date
Bot	333bb82c16	fix(git-guard): use 'exec bash "$0"' instead of 'exec "$0"' for self-re-invocation ... Found via live testing on docker-2 after merging #19: git-guard.sh is tracked in git at mode 100644 (not executable). stack-deploy.sh correctly invokes it as `bash deploy/git-guard.sh`, sidestepping the exec bit for the first call -- but the script's own internal `exec "$0" "$@"` calls (used to cleanly re-run after a successful push or stash-pop) rely on the kernel executing the file directly, which requires +x. Since every fresh checkout or fast-forward preserves the tracked 644 mode, this failed with "Permission denied" (exit 126) the moment either self-re-invocation path was actually exercised. Confirmed via a live dry run: simulated the exact #18 incident shape (dirty git-guard.sh on a HEAD 3 commits behind origin/main) on docker-2. The new stash-first logic from #19 worked perfectly end-to-end -- detected dirty+stale, stashed safely, fast-forwarded via resync_with_origin(), and popped the stash cleanly -- but then hit this pre-existing bug on the final `exec "$0" "$@"` re-invocation. This bug pre-dates #19 (the old dirty-commit-then-push-success path had the identical pattern); #19 just added a second trigger point that happened to surface it during testing. Fix: `exec bash "$0" "$@"` explicitly invokes through the interpreter instead of relying on the file's own execute bit -- correct regardless of what git tracks the file's mode as.	2026-09-09 14:01:26 -07:00
Bot	e357907ee6	fix(git-guard): add unmerged-path guard + concrete remediation syntax for every failure branch ... Adds the unmerged-conflict guard flagged in PR review: before the dirty-tree flow can run, check `git ls-files -u` and refuse to auto-commit if unresolved merge conflict markers are present (e.g. left behind by a prior run's failed `git stash pop`). Without this, a re-run's `git add -A` would silently stage literal <<<<<<< / ======= / >>>>>>> markers into a real commit and push them to origin/main. Also expands every failure-path message (stash-push failure, stash-pop conflict vs. untracked-file-collision, resync failure, true divergence, manual-abort cases) to include concrete, copy-pasteable remediation command sequences with multiple options (resolve-in-place vs. abandon-and-reapply vs. drop-if-unneeded), rather than a single generic hint. The true-divergence message now also tells the operator to check `git stash list` first, so a stash created by the new dirty+stale path isn't confused with a second manually-created one. No change to the common dirty-but-current or clean-but-stale behavior.	2026-09-09 13:47:14 -07:00
Bot	bd69cc85d5	fix(git-guard): stash-before-commit when dirty tree is also stale vs origin ... Previously the dirty-tree branch committed unconditionally, then only discovered staleness/divergence when the push was rejected -- leaving a doomed auto-commit sitting on a stale base. Root-caused from a real incident: a stray on-disk edit to ai/ai.yaml sat on a checkout that was already 4 commits behind, git-guard auto-committed on top of it, then the push was rejected (ahead 1, behind 4), aborting the deploy. Fix: when DIRTY=1 and LOCAL != REMOTE, stash the dirty changes first, resync main with origin using the exact same behind/ahead/diverged rules as the clean-tree path (now shared via resync_with_origin()), then reapply the stash and re-run. Never commits on top of a stale base again. On any failure the stash is preserved and printed for manual recovery -- changes are never silently lost.	2026-09-09 10:42:21 -07:00
admin	0bc117c868	Add: git-guard.sh - pre-deploy sync check to prevent local/Gitea divergence	2026-08-26 22:30:47 -07:00