Skip to content

Create worker Sessions for every issue and PR comment#1528

Open
kelos-bot[bot] wants to merge 1 commit into
mainfrom
self-development-all-comment-sessions
Open

Create worker Sessions for every issue and PR comment#1528
kelos-bot[bot] wants to merge 1 commit into
mainfrom
self-development-all-comment-sessions

Conversation

@kelos-bot

@kelos-bot kelos-bot Bot commented Jul 20, 2026

Copy link
Copy Markdown
Contributor

🤖 Kelos Agent @gjkim42

What type of PR is this?

/kind cleanup

What this PR does / why we need it:

  • makes kelos-workers create a durable Session for every new non-bot issue or PR conversation comment
  • keeps /kelos pick-up as the explicit request to handle the complete issue or PR
  • uses the PR head branch for PR comments and kelos-task- for issue comments
  • updates the self-development documentation and adds matching and branch-template coverage

Which issue(s) this PR is related to:

N/A

Special notes for your reviewer:

Validation:

  • make verify
  • make test

This intentionally covers GitHub issue_comment deliveries, which represent issue comments and PR conversation comments. It does not add inline review-comment events.

Does this PR introduce a user-facing change?

NONE

Summary by cubic

Create a durable Session for every new non-bot issue or PR conversation comment. Keep /kelos pick-up as the opt-in to take full ownership; PR comments run on the PR head branch and issues use kelos-task-<number>, with a persistent workspace.

  • New Features

    • Match all issue_comment created events for issues and PR conversations; exclude kelos-bot[bot]; inline review comments are not included.
    • Expanded initial prompt: PVC-backed workspace note, full comment context (body and URL), and clear immediate-request vs. /kelos pick-up ownership.
    • PR vs. issue flow: detect context up front, update incrementally, avoid self-closing issue refs on PR-triggered work, and clarify squash/CI steps; initialBranch uses PR head or falls back to kelos-task-{{.Number}}.
    • Switch to shared AgentConfig kelos-dev-agent and make guidance persistence-neutral; remove the inline worker AgentConfig.
    • Tests cover webhook matching and bot exclusion, persistent-workspace prompt, agent config reference, and branch rendering.
    • README: new trigger/behavior, shared AgentConfig, CommentBody/CommentURL template variables, and githubWebhook fields for SessionSpawner.
  • Migration

    • Replace the old TaskSpawner and install the shared agent config:
      • kubectl delete taskspawner kelos-workers --ignore-not-found
      • kubectl apply -f self-development/agentconfig.yaml
      • kubectl apply -f self-development/kelos-workers.yaml

Written for commit 9d1f846. Summary will update on new commits.

Review in cubic

@kelos-bot
kelos-bot Bot force-pushed the self-development-all-comment-sessions branch from 6d83a12 to 4769921 Compare July 20, 2026 01:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant