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

cmd_start clones from wrong URL — get_repo_url strips user/org from path #181

New Issue
Closed
opened 2026-04-06 05:52:46 +02:00 by shoko · 0 comments
shoko commented 2026-04-06 05:52:46 +02:00
Owner
Copy Link

Bug

get_repo_url() in kugetsu-worktree.sh constructs repo URL incorrectly for issue refs of the form instance/user/repo#number.

Root cause: Line 44 uses sed 's/.*\///' which removes everything up to the LAST slash. For git.fbrns.co/shoko/kugetsu#158:

  • Expected rest = shoko/kugetsu
  • Actual rest = kugetsu (last path component only)

This makes the function output https://git.fbrns.co/kugetsu.git instead of https://git.fbrns.co/shoko/kugetsu.git.

Affected versions: v0.2.8

Reproduction

git clone https://git.fbrns.co/kugetsu.git /tmp/test-clone
# remote: Not found.
# fatal: repository not found

Fix

Replace line 44:

# WRONG — strips to last path component only
rest=$(echo "$issue_ref" | sed 's/.*\///' | sed 's/#.*//')

# CORRECT — strips instance prefix, preserves user/repo
rest=$(echo "$issue_ref" | sed 's/^[^/]*\///' | sed 's/#.*//')

For git.fbrns.co/shoko/kugetsu#158:

  • sed 's/^[^/]*\///' → shoko/kugetsu#158
  • sed 's/#.*//' → shoko/kugetsu ✓
## Bug get_repo_url() in kugetsu-worktree.sh constructs repo URL incorrectly for issue refs of the form `instance/user/repo#number`. **Root cause:** Line 44 uses `sed 's/.*\///'` which removes everything up to the LAST slash. For `git.fbrns.co/shoko/kugetsu#158`: - Expected `rest = shoko/kugetsu` - Actual `rest = kugetsu` (last path component only) This makes the function output `https://git.fbrns.co/kugetsu.git` instead of `https://git.fbrns.co/shoko/kugetsu.git`. **Affected versions:** v0.2.8 ## Reproduction ```bash git clone https://git.fbrns.co/kugetsu.git /tmp/test-clone # remote: Not found. # fatal: repository not found ``` ## Fix Replace line 44: ```bash # WRONG — strips to last path component only rest=$(echo "$issue_ref" | sed 's/.*\///' | sed 's/#.*//') # CORRECT — strips instance prefix, preserves user/repo rest=$(echo "$issue_ref" | sed 's/^[^/]*\///' | sed 's/#.*//') ``` For `git.fbrns.co/shoko/kugetsu#158`: - `sed 's/^[^/]*\///'` → `shoko/kugetsu#158` - `sed 's/#.*//'` → `shoko/kugetsu` ✓
shoko added the bug label 2026-04-06 05:52:46 +02:00
shoko closed this issue 2026-04-06 07:08:43 +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 bug
Milestone
No items
No Milestone
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#181
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?