Skip to content

Fix two Windows-only bugs in Git Bash #224

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
anarchitect wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Fix two Windows-only bugs in Git Bash #224

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
19949ed
Detect Windows shells that report OSTYPE as cygwin
anarchitect Jul 30, 2026
91c2aad
Strip carriage returns before base64 decode
anarchitect Jul 30, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
39 changes: 23 additions & 16 deletions gh-repo-stats
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -558,13 +558,20 @@ CheckAdminRights() {
################################################################################
#### Function isGHEC ###########################################################
isCloud() {
if [[ "$OSTYPE" == "msys" ]]; then
# If the operating system is Windows escape leading slash
ROOT_DATA=$(gh api //)
else
# If the operating system is Unix-like (e.g., Linux, macOS)
ROOT_DATA=$(gh api /)
fi
# Git Bash reports OSTYPE inconsistently across builds: "msys" on the common
# MINGW64 build, but "cygwin" on others such as the ARM64 CLANG build. Match
# all of them, otherwise the leading slash is rewritten into a filesystem
# path and this check silently misidentifies github.com as GHES.
case "${OSTYPE}" in
msys* | cygwin* | win32)
# If the operating system is Windows escape leading slash
ROOT_DATA=$(gh api //)
;;
*)
# If the operating system is Unix-like (e.g., Linux, macOS)
ROOT_DATA=$(gh api /)
;;
esac
USER_URL=$(echo -n "${ROOT_DATA}" | jq -r '.current_user_url')

if [[ "${USER_URL}" != "https://api.github.com/user" ]]; then
Expand Down Expand Up @@ -870,7 +877,7 @@ ParseRepos() {
REPOS=$(echo "${PARSE_DATA}" | jq -r '.data.organization.repositories.nodes')
for REPO_DATA in $(echo -n "${REPOS}" | jq -r '.[] | @base64'); do
_jq() {
echo -n "${REPO_DATA}" | base64 --decode | jq -r "${1}"
echo -n "${REPO_DATA}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

OWNER=$(_jq '.owner.login' | tr '[:upper:]' '[:lower:]')
Expand Down Expand Up @@ -1000,7 +1007,7 @@ ParseRepoData() {
REPO_DATA=$1
# Convert the format to JSON
_jq() {
echo -n "${REPO_DATA}" | base64 --decode | jq -r "${1}"
echo -n "${REPO_DATA}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}
Comment on lines 1007 to 1011

OWNER=$(_jq '.owner.login' | tr '[:upper:]' '[:lower:]')
Expand Down Expand Up @@ -1127,7 +1134,7 @@ AnalyzeIssues() {
THIS_REPO=$1

_pr_issue_jq() {
echo -n "${THIS_REPO}" | base64 --decode | jq -r "${1}"
echo -n "${THIS_REPO}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

ISSUES=$(_pr_issue_jq '.issues.nodes')
Expand All @@ -1144,7 +1151,7 @@ AnalyzeIssues() {

for ISSUE in $(echo -n "${ISSUES}" | jq -r '.[] | @base64'); do
_issue_jq() {
echo -n "${ISSUE}" | base64 --decode | jq -r "${1}"
echo -n "${ISSUE}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

EVENT_CT=$(_issue_jq '.timeline.totalCount')
Expand Down Expand Up @@ -1240,7 +1247,7 @@ GetNextIssues() {
AnalyzePullRequests() {
PR_REPO=$1
_pr_repo_jq() {
echo -n "${PR_REPO}" | base64 --decode | jq -r "${1}"
echo -n "${PR_REPO}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

Debug "Analyzing Pull Requests for: ${REPO_NAME}"
Expand All @@ -1258,7 +1265,7 @@ AnalyzePullRequests() {
PR_NEXT_PAGE=$(_pr_repo_jq '.pullRequests.pageInfo.endCursor')
for PR in $(echo -n "${PRS}" | jq -r '.[] | @base64'); do
_pr_jq() {
echo -n "${PR}" | base64 --decode | jq -r "${1}"
echo -n "${PR}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

PR_NUMBER=$(_pr_jq '.number')
Expand Down Expand Up @@ -1397,7 +1404,7 @@ AnalyzeReviews() {
REVIEW_PR=$1

_review_jq() {
echo -n "${REVIEW_PR}" | base64 --decode | jq -r "${1}"
echo -n "${REVIEW_PR}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

REVIEWS=$(_review_jq '.reviews.nodes')
Expand All @@ -1417,7 +1424,7 @@ AnalyzeReviews() {

for REVIEW in $(echo -n "${REVIEWS}" | jq -r '.[] | @base64'); do
_pr_jq() {
echo -n "${REVIEW}" | base64 --decode | jq -r "${1}"
echo -n "${REVIEW}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

REVIEW_COMMENT_CT=$(_pr_jq '.comments.totalCount')
Expand Down Expand Up @@ -1581,7 +1588,7 @@ GetTeams() {

for TEAM in $(echo -n "${TEAMS}" | jq -r '.[] | @base64'); do
_team_jq() {
echo -n "${TEAM}" | base64 --decode | jq -r "${1}"
echo -n "${TEAM}" | tr -d '\r' | base64 --decode | jq -r "${1}"
}

TEAM_NAME=$(_team_jq '.slug')
Expand Down
Loading