claude-autopm 2.8.2 → 2.8.3

package/packages/plugin-pm/scripts/pm/issue-sync/post-comment.sh

@@ -0,0 +1,330 @@
+#!/bin/bash
+# Post Comment Script for Issue Sync
+# Posts formatted comments to GitHub issues
+
+set -euo pipefail
+
+# Load libraries
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+source "${SCRIPT_DIR}/../../lib/logging-utils.sh"
+source "${SCRIPT_DIR}/../../lib/github-utils.sh"
+source "${SCRIPT_DIR}/../../lib/validation-utils.sh"
+source "${SCRIPT_DIR}/../../lib/datetime-utils.sh"
+
+# Script configuration
+readonly ISSUE_NUMBER="${1:-}"
+readonly COMMENT_FILE="${2:-}"
+readonly IS_COMPLETION="${3:-false}"
+
+# Global variables
+declare -g comment_id=""
+declare -g comment_url=""
+
+# Main function
+main() {
+    print_banner "GitHub Comment Poster" "1.0.0"
+
+    # Validate inputs
+    log_info "Posting comment to issue #$ISSUE_NUMBER"
+    validate_inputs || exit 1
+
+    # Check GitHub authentication
+    with_error_handling "GitHub authentication check" \
+        check_github_authentication
+
+    # Check if issue exists
+    with_error_handling "Issue validation" \
+        validate_issue_exists
+
+    # Post comment to GitHub
+    with_error_handling "Post comment to GitHub" \
+        post_github_comment
+
+    # Verify comment was posted
+    with_error_handling "Verify comment posted" \
+        verify_comment_posted
+
+    # Display results
+    display_results
+
+    # Return comment URL
+    echo "$comment_url"
+}
+
+# Validate script inputs
+validate_inputs() {
+    log_function_entry "validate_inputs"
+
+    if [[ -z "$ISSUE_NUMBER" ]]; then
+        log_error "Issue number is required"
+        return 1
+    fi
+
+    if [[ ! -f "$COMMENT_FILE" ]]; then
+        log_error "Comment file not found: $COMMENT_FILE"
+        return 1
+    fi
+
+    # Validate issue number format
+    if ! [[ "$ISSUE_NUMBER" =~ ^[0-9]+$ ]]; then
+        log_error "Invalid issue number format: $ISSUE_NUMBER"
+        return 1
+    fi
+
+    log_function_exit "validate_inputs"
+    return 0
+}
+
+# Check GitHub authentication
+check_github_authentication() {
+    log_function_entry "check_github_authentication"
+
+    log_info "Verifying GitHub CLI authentication"
+
+    if ! validate_github_auth; then
+        log_error "GitHub CLI authentication failed"
+        echo "❌ GitHub CLI is not authenticated"
+        echo "Please run: gh auth login"
+        return 1
+    fi
+
+    log_success "GitHub CLI authenticated"
+    log_function_exit "check_github_authentication"
+    return 0
+}
+
+# Validate that the issue exists
+validate_issue_exists() {
+    log_function_entry "validate_issue_exists"
+
+    log_info "Validating issue #$ISSUE_NUMBER exists"
+
+    local issue_state
+    issue_state=$(check_issue_exists "$ISSUE_NUMBER")
+
+    if [[ $? -ne 0 ]]; then
+        log_error "Issue #$ISSUE_NUMBER not found"
+        echo "❌ Issue #$ISSUE_NUMBER not found"
+        return 1
+    fi
+
+    log_info "Issue #$ISSUE_NUMBER exists with state: $issue_state"
+
+    # Warn if issue is closed
+    if [[ "$issue_state" == "closed" ]]; then
+        log_warning "Issue #$ISSUE_NUMBER is closed"
+        echo "⚠️ Issue #$ISSUE_NUMBER is closed"
+
+        # Interactive confirmation for closed issues
+        if [[ -t 0 ]] && [[ "${AUTOPM_FORCE_SYNC:-}" != "true" ]]; then
+            if ! confirm "Post comment to closed issue?"; then
+                echo "Comment posting cancelled"
+                exit 0
+            fi
+        fi
+    fi
+
+    log_function_exit "validate_issue_exists"
+    return 0
+}
+
+# Post comment to GitHub issue
+post_github_comment() {
+    log_function_entry "post_github_comment"
+
+    log_info "Posting comment to issue #$ISSUE_NUMBER"
+
+    # Check for dry run mode
+    if [[ "${AUTOPM_DRY_RUN:-false}" == "true" ]]; then
+        log_warning "DRY RUN mode - not posting comment"
+        echo "🔸 DRY RUN: Would post comment to issue #$ISSUE_NUMBER"
+        echo "Comment preview:"
+        head -20 "$COMMENT_FILE" | sed 's/^/  /'
+        echo "  ..."
+        comment_id="DRY-RUN"
+        comment_url="https://github.com/DRYRUN/issues/$ISSUE_NUMBER#issuecomment-DRYRUN"
+        log_function_exit "post_github_comment"
+        return 0
+    fi
+
+    # Post the comment using GitHub CLI
+    local response
+    response=$(gh issue comment "$ISSUE_NUMBER" \
+        --body-file "$COMMENT_FILE" \
+        --json id,url 2>&1) || {
+        log_error "Failed to post comment: $response"
+        echo "❌ Failed to post comment to issue #$ISSUE_NUMBER"
+        echo "Error: $response"
+        return 1
+    }
+
+    # Extract comment ID and URL from response
+    comment_id=$(echo "$response" | jq -r '.id' 2>/dev/null || echo "")
+    comment_url=$(echo "$response" | jq -r '.url' 2>/dev/null || echo "")
+
+    if [[ -z "$comment_id" ]] || [[ "$comment_id" == "null" ]]; then
+        # Try alternative parsing for non-JSON response
+        log_debug "Attempting alternative response parsing"
+
+        # GitHub CLI sometimes returns just the URL
+        if [[ "$response" =~ https://github.com/.*/issues/.*#issuecomment-.* ]]; then
+            comment_url="$response"
+            comment_id=$(echo "$response" | sed 's/.*#issuecomment-//')
+        else
+            log_error "Could not parse comment response: $response"
+            echo "⚠️ Comment may have been posted but could not parse response"
+            # Continue anyway as the comment might have been posted
+        fi
+    fi
+
+    log_success "Comment posted successfully"
+    log_debug "Comment ID: $comment_id"
+    log_debug "Comment URL: $comment_url"
+
+    log_function_exit "post_github_comment"
+    return 0
+}
+
+# Verify comment was posted
+verify_comment_posted() {
+    log_function_entry "verify_comment_posted"
+
+    # Skip verification in dry run mode
+    if [[ "${AUTOPM_DRY_RUN:-false}" == "true" ]]; then
+        log_debug "Skipping verification in dry run mode"
+        log_function_exit "verify_comment_posted"
+        return 0
+    fi
+
+    log_info "Verifying comment was posted"
+
+    # If we don't have a comment URL, try to get the latest comment
+    if [[ -z "$comment_url" ]] || [[ "$comment_url" == "null" ]]; then
+        log_debug "No comment URL, checking latest comments"
+
+        # Get the last comment on the issue
+        local latest_comment
+        latest_comment=$(gh issue view "$ISSUE_NUMBER" \
+            --json comments \
+            --jq '.comments[-1].url' 2>/dev/null || echo "")
+
+        if [[ -n "$latest_comment" ]]; then
+            comment_url="$latest_comment"
+            log_info "Found latest comment: $comment_url"
+        else
+            log_warning "Could not verify comment was posted"
+            # Not a fatal error - comment might still have been posted
+        fi
+    else
+        # Verify the comment exists by trying to view it
+        local verify_response
+        verify_response=$(gh api "${comment_url#https://github.com}" \
+            --jq '.id' 2>/dev/null || echo "")
+
+        if [[ -n "$verify_response" ]]; then
+            log_success "Comment verified: exists at $comment_url"
+        else
+            log_warning "Could not verify comment at URL: $comment_url"
+        fi
+    fi
+
+    log_function_exit "verify_comment_posted"
+    return 0
+}
+
+# Display posting results
+display_results() {
+    print_section "✅ Comment Posting Results"
+
+    echo "Issue: #$ISSUE_NUMBER"
+
+    if [[ "$IS_COMPLETION" == "true" ]]; then
+        echo "Comment Type: Completion Comment"
+    else
+        echo "Comment Type: Progress Update"
+    fi
+
+    # Show comment size
+    local comment_size
+    comment_size=$(wc -c < "$COMMENT_FILE" 2>/dev/null || echo "0")
+    echo "Comment Size: $comment_size characters"
+
+    # Show posting status
+    if [[ "${AUTOPM_DRY_RUN:-false}" == "true" ]]; then
+        echo ""
+        echo "🔸 DRY RUN - No actual comment posted"
+        echo "Review the formatted comment at: $COMMENT_FILE"
+    else
+        if [[ -n "$comment_url" ]] && [[ "$comment_url" != "null" ]]; then
+            echo ""
+            echo "✅ Comment posted successfully"
+            echo "🔗 View comment: $comment_url"
+        else
+            echo ""
+            echo "⚠️ Comment posting status uncertain"
+            echo "Please check issue #$ISSUE_NUMBER manually"
+        fi
+    fi
+
+    # Show timestamp
+    local current_time
+    current_time=$(get_current_datetime)
+    echo ""
+    echo "Posted at: $current_time"
+}
+
+# Error handling
+handle_error() {
+    local exit_code=$?
+    log_error "Comment posting failed with exit code: $exit_code"
+    log_error "Failed to post comment to issue: #$ISSUE_NUMBER"
+
+    # Provide recovery suggestions
+    echo ""
+    echo "Recovery options:"
+    echo "1. Check GitHub authentication: gh auth status"
+    echo "2. Verify issue exists: gh issue view $ISSUE_NUMBER"
+    echo "3. Try manual posting: gh issue comment $ISSUE_NUMBER --body-file $COMMENT_FILE"
+    echo "4. Check network connectivity"
+
+    exit "$exit_code"
+}
+
+# Set up error handling
+trap handle_error ERR
+
+# Validate arguments
+if [[ $# -lt 2 ]]; then
+    echo "Usage: $0 <issue_number> <comment_file> [is_completion]"
+    echo ""
+    echo "Posts a formatted comment to a GitHub issue."
+    echo ""
+    echo "Arguments:"
+    echo "  issue_number    GitHub issue number to post comment to"
+    echo "  comment_file    Path to formatted comment file"
+    echo "  is_completion   Optional: 'true' if completion comment (default: false)"
+    echo ""
+    echo "Environment Variables:"
+    echo "  AUTOPM_DRY_RUN=true     Perform dry run without posting"
+    echo "  AUTOPM_FORCE_SYNC=true  Skip confirmations for closed issues"
+    echo "  AUTOPM_LOG_LEVEL=0      Enable debug logging"
+    echo ""
+    echo "Examples:"
+    echo "  $0 123 /tmp/formatted-comment.md"
+    echo "  $0 456 /tmp/completion-comment.md true"
+    echo "  AUTOPM_DRY_RUN=true $0 789 /tmp/test-comment.md"
+    echo ""
+    echo "Output:"
+    echo "  Prints comment URL on success"
+    echo "  Returns 0 on success, non-zero on failure"
+    echo ""
+    echo "Error Recovery:"
+    echo "  - If posting fails, the formatted comment is preserved"
+    echo "  - You can manually post using: gh issue comment <number> --body-file <file>"
+    echo "  - Check GitHub status: gh auth status"
+    echo ""
+    exit 1
+fi
+
+# Run main function
+main "$@"

package/packages/plugin-pm/scripts/pm/issue-sync/preflight-validation.sh

@@ -0,0 +1,348 @@
+#!/bin/bash
+# Preflight Validation Script for Issue Sync
+# Performs all necessary checks before syncing issue updates
+
+set -euo pipefail
+
+# Load libraries
+SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
+source "${SCRIPT_DIR}/../../lib/logging-utils.sh"
+source "${SCRIPT_DIR}/../../lib/github-utils.sh"
+source "${SCRIPT_DIR}/../../lib/frontmatter-utils.sh"
+source "${SCRIPT_DIR}/../../lib/validation-utils.sh"
+source "${SCRIPT_DIR}/../../lib/datetime-utils.sh"
+
+# Script configuration
+readonly ISSUE_NUMBER="${1:-}"
+
+# Global variables
+declare -g epic_name=""
+declare -g updates_dir=""
+declare -g progress_file=""
+
+# Main function
+main() {
+    print_banner "Issue Sync Preflight Validation" "1.0.0"
+
+    # Validate inputs
+    log_info "Validating issue: #$ISSUE_NUMBER"
+    validate_inputs || exit 1
+
+    # Run all preflight checks
+    with_error_handling "Repository protection check" \
+        check_repository_protection
+
+    with_error_handling "GitHub authentication check" \
+        check_github_authentication
+
+    with_error_handling "Issue validation" \
+        validate_issue_exists
+
+    with_error_handling "Local updates check" \
+        check_local_updates
+
+    with_error_handling "Sync timing check" \
+        check_sync_timing
+
+    with_error_handling "Changes verification" \
+        verify_changes_exist
+
+    # Output validation results
+    display_validation_results
+
+    log_success "All preflight checks passed successfully"
+}
+
+# Validate script inputs
+validate_inputs() {
+    log_function_entry "validate_inputs"
+
+    validate_issue_number "$ISSUE_NUMBER" || return 1
+
+    log_function_exit "validate_inputs"
+    return 0
+}
+
+# Check repository protection
+check_repository_protection() {
+    log_function_entry "check_repository_protection"
+
+    log_info "Checking repository protection against template repos"
+    check_repo_protection
+
+    log_function_exit "check_repository_protection"
+}
+
+# Check GitHub authentication
+check_github_authentication() {
+    log_function_entry "check_github_authentication"
+
+    log_info "Verifying GitHub CLI authentication"
+    validate_github_auth
+
+    log_function_exit "check_github_authentication"
+}
+
+# Validate that the issue exists and get its state
+validate_issue_exists() {
+    log_function_entry "validate_issue_exists"
+
+    log_info "Validating issue #$ISSUE_NUMBER exists"
+
+    local issue_state
+    issue_state=$(check_issue_exists "$ISSUE_NUMBER")
+
+    if [[ $? -ne 0 ]]; then
+        log_error "Issue #$ISSUE_NUMBER not found"
+        echo "❌ Issue #$ISSUE_NUMBER not found"
+        echo "Please verify the issue number and try again."
+        return 1
+    fi
+
+    log_info "Issue #$ISSUE_NUMBER exists with state: $issue_state"
+
+    # Warn if issue is closed but we're still syncing
+    if [[ "$issue_state" == "closed" ]]; then
+        log_warning "Issue #$ISSUE_NUMBER is closed"
+        echo "⚠️ Issue #$ISSUE_NUMBER is closed but you're syncing updates."
+        echo "This is unusual but allowed - the sync will proceed."
+    fi
+
+    log_function_exit "validate_issue_exists"
+    return 0
+}
+
+# Check for local updates directory and files
+check_local_updates() {
+    log_function_entry "check_local_updates"
+
+    log_info "Checking for local updates directory"
+
+    # Find the epic that contains this issue
+    local found_epic=""
+    for epic_dir in .claude/epics/*/; do
+        [[ -d "$epic_dir" ]] || continue
+
+        local epic_updates_dir="$epic_dir/updates/$ISSUE_NUMBER"
+        if [[ -d "$epic_updates_dir" ]]; then
+            found_epic=$(basename "$epic_dir")
+            updates_dir="$epic_updates_dir"
+            progress_file="$epic_updates_dir/progress.md"
+            break
+        fi
+    done
+
+    if [[ -z "$found_epic" ]]; then
+        log_error "No local updates found for issue #$ISSUE_NUMBER"
+        echo "❌ No local updates found for issue #$ISSUE_NUMBER"
+        echo "Initialize issue tracking with: /pm:issue-start $ISSUE_NUMBER"
+        return 1
+    fi
+
+    epic_name="$found_epic"
+    log_info "Found updates in epic: $epic_name"
+    log_debug "Updates directory: $updates_dir"
+
+    # Check if progress.md exists
+    if [[ ! -f "$progress_file" ]]; then
+        log_error "No progress tracking found"
+        echo "❌ No progress tracking found for issue #$ISSUE_NUMBER"
+        echo "Initialize with: /pm:issue-start $ISSUE_NUMBER"
+        return 1
+    fi
+
+    log_success "Local updates directory found: $updates_dir"
+    log_function_exit "check_local_updates"
+    return 0
+}
+
+# Check timing of last sync to avoid too frequent syncs
+check_sync_timing() {
+    log_function_entry "check_sync_timing"
+
+    log_info "Checking last sync timing"
+
+    local last_sync
+    last_sync=$(get_frontmatter_field "$progress_file" "last_sync" 2>/dev/null || echo "")
+
+    if [[ -n "$last_sync" ]]; then
+        local current_time
+        current_time=$(get_current_datetime)
+
+        local time_diff
+        time_diff=$(datetime_diff_minutes "$last_sync" "$current_time" 2>/dev/null || echo "999")
+
+        log_debug "Last sync: $last_sync, Current: $current_time, Diff: ${time_diff}m"
+
+        if [[ "$time_diff" -lt 5 ]]; then
+            log_warning "Recent sync detected (${time_diff} minutes ago)"
+            echo "⚠️ Issue was synced recently (${time_diff} minutes ago at $last_sync)"
+
+            # Interactive confirmation in non-automated environments
+            if [[ -t 0 ]] && [[ "${AUTOPM_FORCE_SYNC:-}" != "true" ]]; then
+                echo "Force sync anyway? Set AUTOPM_FORCE_SYNC=true to skip this check."
+                if ! confirm "Continue with sync?"; then
+                    echo "Sync cancelled by user"
+                    exit 0
+                fi
+            fi
+        fi
+    else
+        log_info "No previous sync timestamp found - this appears to be the first sync"
+    fi
+
+    log_function_exit "check_sync_timing"
+    return 0
+}
+
+# Verify that there are actually changes to sync
+verify_changes_exist() {
+    log_function_entry "verify_changes_exist"
+
+    log_info "Verifying that changes exist to sync"
+
+    # Get last sync timestamp
+    local last_sync
+    last_sync=$(get_frontmatter_field "$progress_file" "last_sync" 2>/dev/null || echo "")
+
+    local has_changes=false
+
+    # Check if progress.md has been modified since last sync
+    if [[ -f "$progress_file" ]]; then
+        if [[ -z "$last_sync" ]]; then
+            # No previous sync, so there are changes
+            has_changes=true
+            log_debug "No previous sync - treating as changes exist"
+        else
+            # Check file modification time vs last sync
+            local file_mtime_iso
+            if command -v gstat >/dev/null 2>&1; then
+                # GNU stat (macOS with coreutils)
+                file_mtime_iso=$(gstat -c %Y "$progress_file" | xargs -I {} date -u -d "@{}" +"%Y-%m-%dT%H:%M:%SZ")
+            else
+                # BSD stat (macOS default) or Linux stat
+                file_mtime_iso=$(stat -f %m "$progress_file" 2>/dev/null | xargs -I {} date -u -r {} +"%Y-%m-%dT%H:%M:%SZ" ||
+                                 stat -c %Y "$progress_file" | xargs -I {} date -u -d "@{}" +"%Y-%m-%dT%H:%M:%SZ")
+            fi
+
+            if datetime_is_after "$file_mtime_iso" "$last_sync" 2>/dev/null; then
+                has_changes=true
+                log_debug "Progress file modified after last sync"
+            fi
+        fi
+    fi
+
+    # Check for other update files
+    if [[ -d "$updates_dir" ]]; then
+        local update_files
+        update_files=$(find "$updates_dir" -name "*.md" -type f | wc -l)
+        if [[ "$update_files" -gt 1 ]]; then  # More than just progress.md
+            has_changes=true
+            log_debug "Found $update_files update files"
+        fi
+    fi
+
+    if [[ "$has_changes" != "true" ]]; then
+        local last_sync_formatted
+        last_sync_formatted=$(echo "$last_sync" | sed 's/T/ /' | sed 's/Z//')
+
+        log_info "No new updates found since last sync"
+        echo "ℹ️ No new updates to sync since $last_sync_formatted"
+        echo "All changes have already been synchronized to GitHub."
+        exit 0
+    fi
+
+    log_success "Changes detected - sync can proceed"
+    log_function_exit "verify_changes_exist"
+    return 0
+}
+
+# Display validation results summary
+display_validation_results() {
+    print_section "✅ Preflight Validation Results"
+
+    echo "Issue: #$ISSUE_NUMBER"
+    echo "Epic: $epic_name"
+    echo "Updates Directory: $updates_dir"
+    echo "Progress File: $progress_file"
+
+    # Show basic issue info if possible
+    if command -v gh >/dev/null 2>&1; then
+        local issue_title
+        issue_title=$(gh issue view "$ISSUE_NUMBER" --json title -q .title 2>/dev/null || echo "Unknown")
+        echo "Issue Title: $issue_title"
+    fi
+
+    # Show completion status
+    local completion
+    completion=$(get_frontmatter_field "$progress_file" "completion" 2>/dev/null || echo "Unknown")
+    echo "Current Completion: $completion"
+
+    # Show last sync info
+    local last_sync
+    last_sync=$(get_frontmatter_field "$progress_file" "last_sync" 2>/dev/null || echo "Never")
+    echo "Last Sync: $last_sync"
+
+    echo ""
+    echo "✅ All preflight checks passed"
+    echo "✅ Ready to sync updates to GitHub"
+}
+
+# Get epic name for the issue
+get_epic_name() {
+    echo "$epic_name"
+}
+
+# Get updates directory path
+get_updates_directory() {
+    echo "$updates_dir"
+}
+
+# Get progress file path
+get_progress_file() {
+    echo "$progress_file"
+}
+
+# Error handling
+handle_error() {
+    local exit_code=$?
+    log_error "Preflight validation failed with exit code: $exit_code"
+    log_error "Issue sync preflight failed for: #$ISSUE_NUMBER"
+    exit "$exit_code"
+}
+
+# Set up error handling
+trap handle_error ERR
+
+# Validate arguments
+if [[ $# -ne 1 ]]; then
+    echo "Usage: $0 <issue_number>"
+    echo ""
+    echo "Performs preflight validation for issue sync operation."
+    echo ""
+    echo "Checks performed:"
+    echo "  1. Repository protection (prevents syncing to template repos)"
+    echo "  2. GitHub CLI authentication"
+    echo "  3. Issue existence and state validation"
+    echo "  4. Local updates directory and files"
+    echo "  5. Sync timing (prevents too frequent syncs)"
+    echo "  6. Changes verification (ensures there's something to sync)"
+    echo ""
+    echo "Environment Variables:"
+    echo "  AUTOPM_FORCE_SYNC=true    Skip timing checks and force sync"
+    echo "  AUTOPM_LOG_LEVEL=0        Enable debug logging"
+    echo ""
+    echo "Examples:"
+    echo "  $0 123"
+    echo "  AUTOPM_FORCE_SYNC=true $0 456"
+    echo ""
+    echo "Output Functions (for use by other scripts):"
+    echo "  get_epic_name             Returns the epic name containing the issue"
+    echo "  get_updates_directory     Returns the path to updates directory"
+    echo "  get_progress_file         Returns the path to progress.md file"
+    echo ""
+    exit 1
+fi
+
+# Run main function
+main "$@"