shoko/kugetsu
1
0
Fork 0
Code Issues 17 Pull Requests Actions Packages Projects Releases 3 Wiki Activity

Code Quality - Shellcheck & Error Handling #121

New Issue
Closed
opened 2026-04-03 11:14:11 +02:00 by shoko · 0 comments
shoko commented 2026-04-03 11:14:11 +02:00
Owner
Copy Link

Summary

Fix shellcheck warnings and standardize error handling.

Depends On

  • #116 (Foundation - Modularize Shell Script)

Details

  • Run shellcheck on all shell scripts
  • Fix all warnings/errors reported
  • Standardize error handling:
    • Decide per-function: either set -e exit on error OR log+continue
    • No mixing within same function
  • Add retry logic for network operations (git clone, opencode calls)

Backport Compatible

Internal cleanup with stricter validation.

Acceptance Criteria

  • Zero shellcheck errors/warnings
  • Consistent error handling across modules
  • Network operations have retry logic (configurable attempts)
## Summary Fix shellcheck warnings and standardize error handling. ## Depends On - #116 (Foundation - Modularize Shell Script) ## Details - Run `shellcheck` on all shell scripts - Fix all warnings/errors reported - Standardize error handling: - Decide per-function: either `set -e` exit on error OR log+continue - No mixing within same function - Add retry logic for network operations (git clone, opencode calls) ## Backport Compatible Internal cleanup with stricter validation. ## Acceptance Criteria - [ ] Zero shellcheck errors/warnings - [ ] Consistent error handling across modules - [ ] Network operations have retry logic (configurable attempts)
shoko added the backportfoundation labels 2026-04-03 11:18:19 +02:00
shoko added this to the v0.2.x milestone 2026-04-03 11:18:20 +02:00
shoko added the ready label 2026-04-07 08:55:27 +02:00
shoko closed this issue 2026-04-08 07:04:27 +02:00
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
Clear labels
0.2.x-only
Available only in 0.2.x, not backportable
 0.3.x-only
Available only in 0.3.x, not backportable
 backport
Backport compatible - can merge to 0.1.x
 blocked
Issue needs clarification/discussion before it can be worked on
 bug
Bug report
 critical
Critical - blocking parallel work
 feature
New feature
 foundation
Foundation work for 0.2.x
 high
High priority - workflow improvements
 meta
Meta issue tracking multiple issues
 need-review
PR needs review before merge
 parallelization
Issues related to parallelization
 ready
Issue is clear and ready to be implemented/delegated
 ready-to-merge
PR approved and ready to merge
 task-delegation
Issues related to task-delegation
No Label backport foundation ready
Milestone
No items
No Milestone v0.2.x
Projects
Clear projects
No project
Assignees
Clear assignees
shoko
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shoko/kugetsu#121
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?