Skip to content

Log post-fetch steps in git:pull to find hang#59

Merged
jonathonbyrdziak merged 1 commit intomasterfrom
fix/gitpull-post-fetch-logging
Mar 21, 2026
Merged

Log post-fetch steps in git:pull to find hang#59
jonathonbyrdziak merged 1 commit intomasterfrom
fix/gitpull-post-fetch-logging

Conversation

@jonathonbyrdziak
Copy link
Copy Markdown
Contributor

Summary

  • Adds logging for every step after fetch: reset, composer:install, submodule update
  • Adds GIT_TERMINAL_PROMPT=0 and timeout 60 to git submodule update --init --recursive which fetches over the network and could hang on auth
  • Uses escapeshellarg() for repo paths

Test plan

  • Run protocol start, check /var/log/protocol/protocol-start.log — the last [git:pull] line before the hang reveals the culprit

🤖 Generated with Claude Code

Logs reset, composer:install, and submodule update steps. Also adds
GIT_TERMINAL_PROMPT=0 and timeout 60 to submodule update which may
be the culprit (fetches submodules over network).

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jonathonbyrdziak jonathonbyrdziak merged commit 1104f7a into master Mar 21, 2026
6 of 8 checks passed
@jonathonbyrdziak jonathonbyrdziak deleted the fix/gitpull-post-fetch-logging branch March 21, 2026 01:35
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