Skip to content

Add progress indicator after detecting staged changes#15

Merged
Iron-Ham merged 1 commit intofeature/timeout-configurationfrom
fix/issue-14-stalling-progress-indicator
Jan 7, 2026
Merged

Add progress indicator after detecting staged changes#15
Iron-Ham merged 1 commit intofeature/timeout-configurationfrom
fix/issue-14-stalling-progress-indicator

Conversation

@Iron-Ham
Copy link
Owner

@Iron-Ham Iron-Ham commented Jan 7, 2026

Summary

  • Adds an "Analyzing staged changes…" progress message after detecting staged changes to provide user feedback during potentially long-running operations
  • Fixes a pre-existing failing test that was checking for outdated prompt text

Fixes #19

Stack

This PR is part of a stack addressing #14 (stalling after detecting staged changes):

  1. PR Enforce LLM timeout configuration that was previously dead code #20 - Fix LLM timeout enforcement (Fixes LLM timeout configuration is defined but never enforced #16)
  2. PR Add timeout and fix resource leaks in Git client #21 - Fix Git timeout + resource leaks (Fixes Git process operations have no timeout and leak file descriptors #17)
  3. PR Add CLI and config file support for LLM timeout settings #22 - Add timeout configuration CLI/config (Fixes Add CLI flag and config file support for LLM timeout settings #18)
  4. PR Add progress indicator after detecting staged changes #15 (this PR) - Progress logging (Fixes Add progress logging during diff analysis #19)

Details

The tool appeared to "stall" after printing [NOTICE] Detected N staged change(s). because there was no progress feedback while potentially long-running operations executed:

  1. git diff --cached (can be slow for large repos or with --function-context)
  2. Diff parsing and summarization
  3. LLM inference

Before:

[INFO] Repository root: /path/to/repo
[NOTICE] Detected 11 staged change(s).
<long silence>
[NOTICE] Summary: 11 file(s), +234 / -89

After:

[INFO] Repository root: /path/to/repo
[NOTICE] Detected 11 staged change(s).
[INFO] Analyzing staged changes…
[NOTICE] Summary: 11 file(s), +234 / -89

Test plan

  • Build passes (swift build)
  • All 49 tests pass (swift test)
  • Verified CLI works with scg generate --help

Fix issue #14 where the tool appeared to stall after printing
"Detected N staged change(s)." by adding an "Analyzing staged changes…"
message before the potentially long-running summarize operation.

Also fix a pre-existing failing test that was checking for prompt text
("You are an AI assistant") that doesn't exist in the current
BatchCombinationPromptBuilder output.
@Iron-Ham Iron-Ham force-pushed the fix/issue-14-stalling-progress-indicator branch from 761eec5 to 149748d Compare January 7, 2026 19:07
@Iron-Ham Iron-Ham changed the base branch from main to feature/timeout-configuration January 7, 2026 19:14
@Iron-Ham Iron-Ham merged commit 9113873 into feature/timeout-configuration Jan 7, 2026
1 check passed
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