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

feat: add kugetsu session manager skill #12

Closed
shoko wants to merge 0 commits from feat/kugetsu-wrapper into main
pull from: feat/kugetsu-wrapper
merge into: shoko:main
Conversation 3 Commits - Files Changed -
shoko commented 2026-03-29 12:50:31 +02:00
Owner
Copy Link

Summary

  • Add kugetsu skill for opencode session management
  • Provides shell wrapper with commands: start, list [--all], resume, stop, help
  • State tracking: used → idle (graceful) or left (interrupted)
  • Auto-fill last message on resume
  • Confirmation prompt when resuming used session

Files

  • skills/kugetsu/SKILL.md - Skill documentation (agentskills.io format)
  • skills/kugetsu/scripts/kugetsu - Main wrapper script
  • skills/kugetsu/scripts/kugetsu-install.sh - Installation script

Related

  • Issue #11: Support remote agent control - Phase 1
  • Test report: Gitea issue #11 comment #338

Test Results

Test Status
Help ✅
List (empty) ✅
List --all ✅
Start session ✅
Resume with auto-fill ✅
State transitions ✅
## Summary - Add `kugetsu` skill for opencode session management - Provides shell wrapper with commands: `start`, `list [--all]`, `resume`, `stop`, `help` - State tracking: `used` → `idle` (graceful) or `left` (interrupted) - Auto-fill last message on resume - Confirmation prompt when resuming used session ## Files - `skills/kugetsu/SKILL.md` - Skill documentation (agentskills.io format) - `skills/kugetsu/scripts/kugetsu` - Main wrapper script - `skills/kugetsu/scripts/kugetsu-install.sh` - Installation script ## Related - Issue #11: Support remote agent control - Phase 1 - Test report: Gitea issue #11 comment #338 ## Test Results | Test | Status | |------|--------| | Help | ✅ | | List (empty) | ✅ | | List --all | ✅ | | Start session | ✅ | | Resume with auto-fill | ✅ | | State transitions | ✅ |
shoko added 1 commit 2026-03-29 12:50:31 +02:00
feat: add kugetsu session manager skill 7edb54cd3f 
- skills/kugetsu/SKILL.md: Agent skill documentation following agentskills.io spec
- skills/kugetsu/scripts/kugetsu: Shell wrapper for opencode session management
  - Commands: start, list [--all], resume, stop, help
  - State tracking: used → idle (graceful) or left (interrupted)
  - Auto-fill message on resume
  - Confirmation prompt when resuming used session
- skills/kugetsu/scripts/kugetsu-install.sh: Installation script for users

Implements Phase 1 of issue #11 - basic session management layer
for remote agent control without Hermes dependency.
shoko commented 2026-03-29 15:51:47 +02:00
Author
Owner
Copy Link

Test Suite Improvements - Summary

Test Results: 9/12 passing (previously 6/12)

Fixes Applied:

  1. Fixed test 3/6 - Removed premature cleanup call after test 4 that was deleting sessions before tests 5/6 could verify them

  2. Fixed test 8 - Added "Using provided message:" output to kugetsu resume when a message is provided on the command line

  3. Fixed test 9 - Updated grep pattern from "not resumable" to "cannot be resumed" to match actual error message

  4. Added cleanup functions - cleanup_sessions() and cleanup_opencode() to clean up between tests that spawn opencode

Files Modified:

  • skills/kugetsu/tests/test-kugetsu.sh - Fixed test logic and cleanup
  • skills/kugetsu/scripts/kugetsu - Added "Using provided message:" output

Remaining Issue:

Tests 9, 10, 11 fail intermittently due to session state from earlier tests in the same run. This is a test isolation issue - the tests share session state that causes intermittent failures. The core kugetsu functionality works correctly.

## Test Suite Improvements - Summary ### Test Results: 9/12 passing (previously 6/12) ### Fixes Applied: 1. **Fixed test 3/6** - Removed premature `cleanup` call after test 4 that was deleting sessions before tests 5/6 could verify them 2. **Fixed test 8** - Added "Using provided message:" output to `kugetsu resume` when a message is provided on the command line 3. **Fixed test 9** - Updated grep pattern from "not resumable" to "cannot be resumed" to match actual error message 4. **Added cleanup functions** - `cleanup_sessions()` and `cleanup_opencode()` to clean up between tests that spawn opencode ### Files Modified: - `skills/kugetsu/tests/test-kugetsu.sh` - Fixed test logic and cleanup - `skills/kugetsu/scripts/kugetsu` - Added "Using provided message:" output ### Remaining Issue: Tests 9, 10, 11 fail intermittently due to session state from earlier tests in the same run. This is a test isolation issue - the tests share session state that causes intermittent failures. The core kugetsu functionality works correctly.
shoko commented 2026-03-29 16:05:56 +02:00
Author
Owner
Copy Link

Test Suite Update: All 12 Tests Passing

The issue was bash exit status handling with pipes and set -euo pipefail.

Changed from:
if ! timeout 5 bash -c "$KUGETSU resume ..." 2>&1 | grep -q "cannot be resumed";

To:
OUTPUT=$(timeout 5 bash -c "$KUGETSU resume ..." 2>&1 || true)
if echo "$OUTPUT" | grep -q "cannot be resumed";

Also added test isolation cleanup (rm specific session, not all).

All 12 tests now pass consistently.

## Test Suite Update: All 12 Tests Passing The issue was bash exit status handling with pipes and set -euo pipefail. Changed from: if ! timeout 5 bash -c "$KUGETSU resume ..." 2>&1 | grep -q "cannot be resumed"; To: OUTPUT=$(timeout 5 bash -c "$KUGETSU resume ..." 2>&1 || true) if echo "$OUTPUT" | grep -q "cannot be resumed"; Also added test isolation cleanup (rm specific session, not all). **All 12 tests now pass consistently.**
shoko added 2 commits 2026-03-29 16:10:53 +02:00
test: add kugetsu test suite 5a9c3a87a9
fix: resolve test failures - all 12 tests pass b992949314 
- Fix bash pipe/exit status issue with set -euo pipefail
- Change from: if ! cmd | grep -q pattern
- Change to: OUTPUT=$(cmd || true); if echo "$OUTPUT" | grep -q pattern
- Add test isolation cleanup (rm specific session, not all)
- Add 'Using provided message:' output to kugetsu resume
- Fix grep pattern: 'cannot be resumed' not 'not resumable'
shoko added 1 commit 2026-03-29 16:35:03 +02:00
feat: add destroy command and session_id validation dd9a444920 
- Add destroy subcommand for deleting sessions
- Add destroy --all for fresh start with confirmation
- Add -y flag to skip confirmation prompts
- Add validate_session_id() to reject empty session_ids
- Remove misleading force resume error message
- Update SKILL.md to v1.1 with destroy documentation
han approved these changes 2026-03-29 16:36:13 +02:00
han left a comment
First-time contributor
Copy Link

lgtm

lgtm
shoko closed this pull request 2026-03-29 16:38:43 +02:00
shoko deleted branch feat/kugetsu-wrapper 2026-03-29 16:38:43 +02:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No Reviewers
han
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
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
shoko
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: shoko/kugetsu#12
Delete Branch "feat/kugetsu-wrapper"

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?