fix(kugetsu): remove duplicate update_queue_item_state to use fixed version from kugetsu-index.sh #171

shoko · 2026-04-06T02:41:08+02:00

shoko commented

2026-04-06 02:41:08 +02:00

Summary

Remove the duplicate update_queue_item_state() definition from the main kugetsu script so it uses the fixed version from kugetsu-index.sh.

Problem

Issue #170: The main kugetsu script had its own update_queue_item_state() definition that overwrote the fixed version from kugetsu-index.sh. This happened because kugetsu sources kugetsu-index.sh first, then later in the same file defined its own version with broken os.system() calls.

Impact

Queue daemon worked correctly (used fixed version from kugetsu-index.sh)
Main kugetsu command failed for notifications from check_task_timeouts() (used buggy version from kugetsu)

Fix

Remove the duplicate update_queue_item_state() definition from kugetsu. Now kugetsu will use the fixed version from kugetsu-index.sh since that module is sourced before any local function definitions.

Fixes #170

## Summary Remove the duplicate update_queue_item_state() definition from the main kugetsu script so it uses the fixed version from kugetsu-index.sh. ## Problem Issue #170: The main kugetsu script had its own update_queue_item_state() definition that overwrote the fixed version from kugetsu-index.sh. This happened because kugetsu sources kugetsu-index.sh first, then later in the same file defined its own version with broken os.system() calls. ## Impact - Queue daemon worked correctly (used fixed version from kugetsu-index.sh) - Main kugetsu command failed for notifications from check_task_timeouts() (used buggy version from kugetsu) ## Fix Remove the duplicate update_queue_item_state() definition from kugetsu. Now kugetsu will use the fixed version from kugetsu-index.sh since that module is sourced before any local function definitions. Fixes #170

shoko added 1 commit 2026-04-06 02:41:09 +02:00

fix(kugetsu): remove duplicate update_queue_item_state to use fixed version from kugetsu-index.sh 59f6a4883e

The main kugetsu script had its own update_queue_item_state() definition
with broken os.system() calls that overwrote the fixed version in kugetsu-index.sh.

Now kugetsu will use the fixed version from kugetsu-index.sh (which sources
kugetsu-log.sh) since that module is sourced first.

Fixes #170

han approved these changes 2026-04-06 02:42:15 +02:00

han left a comment

First-time contributor

lgtm

shoko merged commit 7234837284 into main

2026-04-06 02:42:25 +02:00

shoko referenced this issue from a commit

2026-04-06 02:42:25 +02:00

Merge pull request 'fix(kugetsu): remove duplicate update_queue_item_state to use fixed version from kugetsu-index.sh' (#171) from fix/issue-170-duplicate-update-queue into main

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: shoko/kugetsu#171