claude-plugin-viban 1.3.11 → 1.3.13

package/commands/assign.md

@@ -4,15 +4,13 @@ description: "Assign first backlog issue — clarify if unclear, then finish"
 
 > Tip: Run `/clear` before `/viban:assign` for a clean context.
 
-Run `/viban:assign` to pick up the next backlog issue.
+Run `/viban:assign` to pick up and resolve the next backlog issue.
 
 Usage: `/viban:assign`
 
 What it does:
 1. Assigns the first backlog issue (by priority) to the current session
-2. If the issue description is unclear or lacks context, interviews you to gather missing information and updates the issue
-3. That's it — no implementation, no branch creation
-
-This command is for **assignment and clarification only**. Use other tools to start working on the assigned issue.
+2. If the issue description is unclear, interviews you to gather missing information
+3. Analyzes, implements, verifies, and ships the fix (branch + PR + review)
 
 **IMPORTANT:** Never read or write `viban.json` directly — always use `viban` CLI commands.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "claude-plugin-viban",
-  "version": "1.3.11",
+  "version": "1.3.13",
   "description": "Terminal Kanban TUI for AI-human collaborative issue tracking",
   "main": "bin/viban",
   "bin": {

package/scripts/providers/github.sh

@@ -248,6 +248,14 @@ provider_close_issue() {
     }
 }
 
+provider_push_comment() {
+    local repo="$1" remote_id="$2" body="$3"
+    gh issue comment "$remote_id" --repo "$repo" --body "$body" 2>/dev/null || {
+        echo "Warning: Failed to push comment to issue #$remote_id"
+        return 1
+    }
+}
+
 provider_ensure_labels() {
     local repo="$1"
 

package/scripts/sync.sh

@@ -22,6 +22,7 @@ load_provider() {
         echo "Error: Provider script not found: $PROVIDER_SCRIPT"
         return 1
     fi
+    # shellcheck source=/dev/null
     source "$PROVIDER_SCRIPT"
 
     # Validate provider interface
@@ -29,6 +30,7 @@ load_provider() {
         provider_name provider_check_deps provider_check_auth
         provider_detect_config provider_fetch_issues provider_create_issue
         provider_update_issue provider_close_issue provider_ensure_labels
+        provider_push_comment
     )
     for func in "${required_funcs[@]}"; do
         if ! declare -f "$func" &>/dev/null; then
@@ -70,6 +72,90 @@ set_issue_meta() {
     write_sync_meta "$meta"
 }
 
+# ============================================================
+# Body Enrichment (blocked_by, sub-tasks)
+# ============================================================
+
+# Build enriched body with blocked_by and sub-task metadata for push
+build_enriched_body() {
+    local card_id="$1"
+    jq -r --argjson cid "$card_id" '
+        . as $root |
+        ($root.issues[] | select((.id|tonumber)==$cid)) as $card |
+        ($card.description // "") as $desc |
+        ([$card.blocked_by // [] | .[] | . as $bid |
+            ($root.issues[] | select((.id|tonumber)==($bid|tonumber)) | .external_id // null) as $ext |
+            if ($ext != null and ($ext | startswith("github:"))) then
+                "#" + ($ext | ltrimstr("github:"))
+            else
+                "viban:#" + ($bid|tostring)
+            end
+        ] | join(", ")) as $blocked_refs |
+        ([$root.issues[] | select((.parent_id // -1) == $cid) |
+            (if .external_id != null and (.external_id | startswith("github:")) then
+                "#" + (.external_id | ltrimstr("github:"))
+            else
+                "viban:#" + (.id|tostring)
+            end) as $ref |
+            if .status == "done" then "- [x] " + $ref + ": " + .title
+            else "- [ ] " + $ref + ": " + .title
+            end
+        ]) as $subtask_lines |
+        if ($blocked_refs == "" and ($subtask_lines | length) == 0) then
+            $desc
+        else
+            $desc + "\n\n<!-- viban:meta:start -->\n---\n" +
+            (if $blocked_refs != "" then "**Blocked by:** " + $blocked_refs + "\n" else "" end) +
+            (if ($subtask_lines | length) > 0 then "**Sub-tasks:**\n" + ($subtask_lines | join("\n")) + "\n" else "" end) +
+            "<!-- viban:meta:end -->"
+        end
+    ' "$VIBAN_JSON"
+}
+
+# Strip viban metadata sections from body (used during pull)
+strip_viban_meta() {
+    local body="$1"
+    if [[ "$body" != *"<!-- viban:meta:start -->"* ]]; then
+        printf '%s' "$body"
+        return
+    fi
+    printf '%s\n' "$body" | awk '
+        /<!-- viban:meta:start -->/ { exit }
+        { lines[++n] = $0 }
+        END {
+            while (n > 0 && lines[n] ~ /^[[:space:]]*$/) n--
+            for (i = 1; i <= n; i++) print lines[i]
+        }
+    '
+}
+
+# Push new comments to remote issue (tracks count to avoid duplicates)
+push_new_comments() {
+    local repo="$1" remote_id="$2" viban_id="$3" card_json="$4"
+    local comment_count
+    comment_count=$(echo "$card_json" | jq '.comments // [] | length')
+    [[ "$comment_count" -eq 0 ]] && return 0
+    local meta synced_count
+    meta=$(get_issue_meta "$viban_id")
+    synced_count=$(echo "$meta" | jq -r '.synced_comment_count // 0')
+
+    if [[ "$comment_count" -gt "$synced_count" ]]; then
+        local comments
+        comments=$(echo "$card_json" | jq '.comments // []')
+        for ci in $(seq "$synced_count" $((comment_count - 1))); do
+            local comment_text comment_at
+            comment_text=$(echo "$comments" | jq -r ".[$ci].text")
+            comment_at=$(echo "$comments" | jq -r ".[$ci].created_at")
+            provider_push_comment "$repo" "$remote_id" "**[viban @ ${comment_at}]** ${comment_text}" || true
+        done
+        local meta_full
+        meta_full=$(read_sync_meta)
+        meta_full=$(echo "$meta_full" | jq --arg vid "$viban_id" --argjson cc "$comment_count" \
+            '.issues[$vid].synced_comment_count = $cc')
+        write_sync_meta "$meta_full"
+    fi
+}
+
 # ============================================================
 # Sync Init
 # ============================================================
@@ -135,7 +221,7 @@ sync_pull() {
         local remote_id title description status priority type remote_updated
         remote_id=$(echo "$issue" | jq -r '.remote_id')
         title=$(echo "$issue" | jq -r '.title')
-        description=$(echo "$issue" | jq -r '.description // ""')
+        description=$(strip_viban_meta "$(echo "$issue" | jq -r '.description // ""')")
         status=$(echo "$issue" | jq -r '.status')
         priority=$(echo "$issue" | jq -r '.priority // "P3"')
         type=$(echo "$issue" | jq -r '.type // ""')
@@ -152,7 +238,7 @@ sync_pull() {
             # New remote issue -> import (skip already-closed issues)
             if [[ "$status" == "done" ]]; then
                 echo "  == ${ext_id} \"${title}\" (closed, skipped)"
-                ((unchanged++))
+                ((unchanged++)) || true
                 continue
             fi
             if [[ "$dry_run" == "true" ]]; then
@@ -187,7 +273,7 @@ sync_pull() {
                 set_issue_meta "$viban_id" "$remote_id" "$remote_updated" "$now"
                 echo "  <- ${ext_id} \"${title}\" (new card created)"
             fi
-            ((pulled++))
+            ((pulled++)) || true
         else
             # Existing card - check for changes
             local viban_id viban_updated
@@ -208,7 +294,7 @@ sync_pull() {
                     write_sync_meta "$meta_tmp"
                     echo "  <- ${ext_id} \"${title}\" (closed remotely, card removed)"
                 fi
-                ((pulled++))
+                ((pulled++)) || true
                 continue
             fi
 
@@ -220,7 +306,7 @@ sync_pull() {
                 # First time seeing this linked card in sync - record and skip
                 set_issue_meta "$viban_id" "$remote_id" "$remote_updated" "$viban_updated"
                 echo "  == ${ext_id} \"${title}\" (tracking started)"
-                ((unchanged++))
+                ((unchanged++)) || true
                 continue
             fi
 
@@ -234,7 +320,7 @@ sync_pull() {
 
             if [[ "$remote_changed" == "false" && "$viban_changed" == "false" ]]; then
                 echo "  == ${ext_id} \"${title}\" (no changes)"
-                ((unchanged++))
+                ((unchanged++)) || true
             elif [[ "$remote_changed" == "true" && "$viban_changed" == "false" ]]; then
                 # Only remote changed -> pull
                 if [[ "$dry_run" == "true" ]]; then
@@ -257,11 +343,11 @@ sync_pull() {
                     set_issue_meta "$viban_id" "$remote_id" "$remote_updated" "$now"
                     echo "  <- ${ext_id} \"${title}\" (pulled remote changes)"
                 fi
-                ((updated++))
+                ((updated++)) || true
             elif [[ "$remote_changed" == "false" && "$viban_changed" == "true" ]]; then
                 # Only viban changed -> will be handled in push phase
                 echo "  == ${ext_id} \"${title}\" (local changes, will push)"
-                ((unchanged++))
+                ((unchanged++)) || true
             else
                 # Both changed -> conflict resolution (remote wins by default)
                 if [[ "$dry_run" == "true" ]]; then
@@ -284,7 +370,7 @@ sync_pull() {
                     set_issue_meta "$viban_id" "$remote_id" "$remote_updated" "$now"
                     echo "  !! ${ext_id} \"${title}\" (conflict: remote wins)"
                 fi
-                ((pulled++))
+                ((pulled++)) || true
             fi
         fi
     done
@@ -319,7 +405,7 @@ sync_push() {
         local viban_id ext_id remote_id title status viban_updated
         viban_id=$(echo "$card" | jq -r '.id')
         ext_id=$(echo "$card" | jq -r '.external_id')
-        remote_id="${ext_id#${provider_prefix}}"
+        remote_id="${ext_id#"${provider_prefix}"}"
         title=$(echo "$card" | jq -r '.title')
         status=$(echo "$card" | jq -r '.status')
         viban_updated=$(echo "$card" | jq -r '.updated_at')
@@ -328,7 +414,7 @@ sync_push() {
         local meta
         meta=$(get_issue_meta "$viban_id")
         if [[ -z "$meta" ]]; then
-            ((unchanged++))
+            ((unchanged++)) || true
             continue
         fi
 
@@ -336,23 +422,28 @@ sync_push() {
         last_viban_updated=$(echo "$meta" | jq -r '.viban_updated_at')
 
         if [[ "$viban_updated" == "$last_viban_updated" ]]; then
-            ((unchanged++))
+            ((unchanged++)) || true
             continue
         fi
 
         # Viban card changed since last sync -> push
         if [[ "$dry_run" == "true" ]]; then
             echo "  -> ${ext_id} \"${title}\" (will push local changes)"
-            ((pushed++))
+            ((pushed++)) || true
         else
-            echo "$card" | jq '{
+            local enriched_body
+            enriched_body=$(build_enriched_body "$viban_id")
+            echo "$card" | jq --arg body "$enriched_body" '{
                 title: .title,
-                description: (.description // ""),
+                description: $body,
                 status: .status,
                 priority: (.priority // "P3"),
                 type: (.type // "")
             }' | provider_update_issue "$repo" "$remote_id"
 
+            # Push new comments
+            push_new_comments "$repo" "$remote_id" "$viban_id" "$card"
+
             local now remote_updated
             now=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
             # Fetch updated remote timestamp
@@ -360,7 +451,7 @@ sync_push() {
 
             set_issue_meta "$viban_id" "$remote_id" "$remote_updated" "$viban_updated"
             echo "  -> ${ext_id} \"${title}\" (pushed)"
-            ((pushed++))
+            ((pushed++)) || true
         fi
     done
 
@@ -384,12 +475,14 @@ sync_push() {
 
             if [[ "$dry_run" == "true" ]]; then
                 echo "  -> (new) #${viban_id} \"${title}\" (will create remote issue)"
-                ((pushed++))
+                ((pushed++)) || true
             else
                 local new_remote_id
-                new_remote_id=$(echo "$card" | jq '{
+                local enriched_body
+                enriched_body=$(build_enriched_body "$viban_id")
+                new_remote_id=$(echo "$card" | jq --arg body "$enriched_body" '{
                     title: .title,
-                    description: (.description // ""),
+                    description: $body,
                     status: .status,
                     priority: (.priority // "P3"),
                     type: (.type // "")
@@ -408,8 +501,10 @@ sync_push() {
                     "$VIBAN_JSON" > "${VIBAN_JSON}.tmp" && mv "${VIBAN_JSON}.tmp" "$VIBAN_JSON"
 
                 set_issue_meta "$viban_id" "$new_remote_id" "$now" "$now"
+                # Push comments for newly created issue
+                push_new_comments "$repo" "$new_remote_id" "$viban_id" "$card"
                 echo "  -> ${ext_id} #${viban_id} \"${title}\" (created remote issue)"
-                ((pushed++))
+                ((pushed++)) || true
             fi
         done
     fi
@@ -564,4 +659,6 @@ main() {
     fi
 }
 
-main "$@"
+if [[ "${BASH_SOURCE[0]}" == "${0}" ]]; then
+    main "$@"
+fi

package/scripts/sync_create.sh

@@ -21,6 +21,7 @@ PROVIDER_SCRIPT="${VIBAN_SCRIPT_DIR}/scripts/providers/${VIBAN_PROVIDER}.sh"
 if [[ ! -f "$PROVIDER_SCRIPT" ]]; then
     exit 1
 fi
+# shellcheck source=/dev/null
 source "$PROVIDER_SCRIPT"
 
 # Read card from viban.json

package/skills/assign/SKILL.md

@@ -11,6 +11,13 @@ Assign the first backlog issue and execute the full resolution workflow. If the
 
 ---
 
+## Output Rules
+
+- **Do NOT output any preamble.** No "Your Task:", "I'll now...", "Let me...", or task summaries before starting work.
+- Start executing Step 0 immediately and silently.
+
+---
+
 ## Step 0: Read Workflow (CRITICAL)
 
 Check in priority order — first match wins:

package/skills/parallel-assign/SKILL.md

@@ -13,6 +13,13 @@ Parallel resolution of independent backlog issues via git worktrees.
 
 ---
 
+## Output Rules
+
+- **Do NOT output any preamble.** No "Your Task:", "I'll now...", "Let me...", or task summaries before starting work.
+- Start executing Phase 0 immediately and silently.
+
+---
+
 ## Phase 0: SETUP
 
 ### 0.1 Read Workflow