Skip to content

fix bugs, silent failures, and simplify create-mod#29

Merged
axelgar merged 3 commits into
mainfrom
feat/code-review
Mar 12, 2026
Merged

fix bugs, silent failures, and simplify create-mod#29
axelgar merged 3 commits into
mainfrom
feat/code-review

Conversation

@axelgar

@axelgar axelgar commented Mar 12, 2026

Copy link
Copy Markdown
Owner

Changes - fix bugs, silent failures, and simpl

Test User and others added 3 commits March 12, 2026 08:11
- config: remove dead AgentCommand source attribution code that was
  immediately overwritten, and unused `_ = resolved` / `defaults` var
- workspace: clean up orphaned worktree if tmux window creation fails
- tui: reset creating/filtering/prCreating flags on errMsg to prevent
  stuck dialogs
- worktree: replace fmt.Sscanf (silent errors) with strconv.Atoi in
  parseNumstat
- github: distinguish "no pull requests found" from real gh CLI errors
  in FetchPRReviews
- tui/commands: surface DiffStats error as "diff unavailable" instead
  of silently showing stale "No changes"; remove redundant WindowID:""
- tui: extract resetCreateMode() helper used across all 5 create-mode
  exit paths
- tui/view: extract renderCIBadge() helper, replace magic layout numbers
  with named constants

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Keep strconv.Atoi over fmt.Sscanf — properly skips malformed input
rather than silencing the error with _, _.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
gh CLI exits with status 4 (resource not found) when a branch has no
PR, not just status 1 with a text message. Check the exit code first,
then fall back to string matching for older gh versions.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@axelgar axelgar merged commit cc5afe1 into main Mar 12, 2026
2 checks 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