Search results

The hardcoded default 'squash' at line 266 duplicates the default value set in config.go line 249. This creates a maintenance issue if the default needs to change. Consider using a constant or accessing the default from a single source of truth.

Status: Open.

#37 In thoreinstein/rig;

· thoreinstein opened on Jan 16, 2026

The error checking relies on string matching which is fragile and may not catch all cases. Consider checking for specific error types or wrapping errors with typed error handling for more robust detection of 'not found' errors.

Status: Open.

#35 In thoreinstein/rig;

· thoreinstein opened on Jan 15, 2026

The config initialization is incomplete - it should also initialize the AI and Beads config fields to ensure the ticket router behaves correctly. Consider adding AI: config.AIConfig{Enabled: false} and Beads: config.BeadsConfig{Enabled: false} to make test dependencies explicit.

Status: Open.

#33 In thoreinstein/rig;

· thoreinstein opened on Jan 15, 2026

The duration calculation logic assumes that the last commit time represents workflow completion, but this may not be accurate. The workflow could have been started long after the last commit was made. Consider documenting this assumption more clearly in a comment, or using actual workflow completion time if available.

Status: Open.

#32 In thoreinstein/rig;

· thoreinstein opened on Jan 15, 2026