fix(kugetsu-session): extract_issue_ref_from_message fix URL parsing

Fix issue where full URLs like #158 were incorrectly parsed to 'shoko/kugetsu/issues#158' instead of 'git.fbrns.co/shoko/kugetsu#158'. The regex now correctly extracts instance, owner, repo, and number using bash regex capture groups instead of fragile sed/cut pipeline.
Merge pull request 'fix: queue daemon crashes on every task (issue #174 )' (#175 ) from fix/issue-174-queue-daemon-crashes into main
2026-04-06 02:51:51 +00:00 · 2026-04-06 04:16:12 +02:00 · 2026-04-06 02:14:27 +00:00 · 2026-04-06 03:09:30 +02:00
2 changed files with 10 additions and 7 deletions
--- a/skills/kugetsu/scripts/kugetsu
+++ b/skills/kugetsu/scripts/kugetsu
@@ -854,6 +854,11 @@ EOF
 }

 parse_issue_ref_from_message() {
+    # DEPRECATED: This function is not called anywhere.
+    # The active implementation is extract_issue_ref_from_message()
+    # in kugetsu-session.sh which is used by cmd_delegate.
+    # This function is kept for backwards compatibility and will
+    # be removed in a future release.
    local message="$1"
    
    local gitserver=""
--- a/skills/kugetsu/scripts/kugetsu-session.sh
+++ b/skills/kugetsu/scripts/kugetsu-session.sh
@@ -156,13 +156,11 @@ extract_issue_ref_from_message() {
        return
    fi
    
-    if [[ "$message" =~ (https?://[a-zA-Z0-9.-]+/[a-zA-Z0-9._-]+/[a-zA-Z0-9._-]+/(issues|pull)/[0-9]+) ]]; then
-        local url="${BASH_REMATCH[1]}"
-        local path=$(echo "$url" | sed 's|https\?://||' | cut -d'/' -f2-)
-        local instance=$(echo "$path" | cut -d'/' -f1)
-        local owner=$(echo "$path" | cut -d'/' -f2)
-        local repo=$(echo "$path" | cut -d'/' -f3)
-        local num=$(echo "$path" | grep -oE '[0-9]+$')
+    if [[ "$message" =~ (https?://)?([a-zA-Z0-9.-]+)/([a-zA-Z0-9._-]+)/([a-zA-Z0-9._-]+)/(issues|pull)/([0-9]+) ]]; then
+        local instance="${BASH_REMATCH[2]}"
+        local owner="${BASH_REMATCH[3]}"
+        local repo="${BASH_REMATCH[4]}"
+        local num="${BASH_REMATCH[6]}"
        echo "${instance}/${owner}/${repo}#${num}"
        return
    fi
Author	SHA1	Message	Date
shokollm	d68a63af41	fix(kugetsu-session): extract_issue_ref_from_message fix URL parsing Fix issue where full URLs like #158 were incorrectly parsed to 'shoko/kugetsu/issues#158' instead of 'git.fbrns.co/shoko/kugetsu#158'. The regex now correctly extracts instance, owner, repo, and number using bash regex capture groups instead of fragile sed/cut pipeline.	2026-04-06 02:51:51 +00:00
shoko	56310755b8	Merge pull request 'fix: queue daemon crashes on every task (issue #174 )' (#175 ) from fix/issue-174-queue-daemon-crashes into main	2026-04-06 04:16:12 +02:00
shokollm	fb33be3a64	fix: queue daemon crashes on every task - 3 bugs Fix 3 bugs from issue #174 that caused silent failure loop: 1. kugetsu-log.sh: Fix json.loads with newlines - Previously, notifications JSON was embedded in a single-quoted Python string literal, but newlines in the JSON broke the Python parser. - Fix: Pass JSON via stdin to Python instead of embedding in string. 2. kugetsu-queue-daemon.sh: Create logs directory during init - The logs/ directory ($LOGS_DIR) was never created during kugetsu init. - Fix: Add mkdir -p for LOGS_DIR, WORKTREES_DIR, QUEUE_DIR, and QUEUE_ITEMS_DIR to ensure_dirs() and ensure_queue_dirs(). 3. kugetsu: Fix parse_issue_ref_from_message URL parsing - The function used buggy grep/sed to parse URLs like #158 - Fix: Use bash regex (=~) for reliable URL parsing with proper capture groups. Additional improvements: - ensure_dirs() now creates all necessary directories instead of just SESSIONS_DIR - ensure_queue_dirs() now also creates QUEUE_DIR and LOGS_DIR - parse_issue_ref_from_message uses consistent bash regex approach for all URL patterns	2026-04-06 02:14:27 +00:00
shoko	1b19c9a92c	Merge pull request 'fix: init script captures wrong session IDs when old sessions exist' (#173 ) from fix/issue-172-init-script-wrong-session-ids into main	2026-04-06 03:09:30 +02:00