Skip to content

Harden GitHub helper outputs#135

Merged
codeforester merged 1 commit into
mainfrom
fix/131-20260707-gh-helper-followups
Jul 7, 2026
Merged

Harden GitHub helper outputs#135
codeforester merged 1 commit into
mainfrom
fix/131-20260707-gh-helper-followups

Conversation

@codeforester

@codeforester codeforester commented Jul 7, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • rename GitHub helper pass-by-name intermediates to avoid shadowing caller output variables
  • log an actionable error when gh_infer_repo_from_origin cannot infer a repo in non-optional mode
  • deduplicate worktree parsing loops by building the git command once

Fixes #131.
Fixes #132.
Fixes #133.

Validation

  • bats lib/bash/gh/tests/lib_gh.bats
  • ./tests/validate.sh
  • git diff --check

@codeforester codeforester merged commit 31eabfa into main Jul 7, 2026
2 checks passed
@codeforester codeforester deleted the fix/131-20260707-gh-helper-followups branch July 7, 2026 23:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment