fix(kugetsu): add kugetsu post-comment helper for PM agent #94
Closed
shoko
wants to merge 1 commits from
fix/issue-45-post-comment-v2 into main
pull from: fix/issue-45-post-comment-v2
merge into: shoko:main
shoko:main
shoko:fix/issue-254
shoko:fix/issue-252
shoko:fix/issue-248
shoko:fix/issue-246
shoko:fix/issue-165
shoko:fix/issue-158
shoko:fix/issue-244
shoko:fix/issue-121
shoko:fix/issue-119
shoko:fix/issue-166
shoko:fix/issue-229-pr-conflict-check
shoko:fix/issue-212
shoko:fix/issue-229-user-message-with-base-workflow
shoko:fix/issue-229-ensure-session-worktree-bug
shoko:fix/issue-168
shoko:fix/issue-225-cmd-continue-message-truncation
shoko:fix/issue-223-pr-creation-instructions
shoko:fix/issue-220-pm-context-enhancement
shoko:fix/issue-210-msg-file-race-condition
shoko:fix/issue-207-queue-daemon-set-debug-mode
shoko:fix/issue-message-encoding
shoko:fix/issue-118
shoko:fix/issue-daemon-worktree-session-handling
shoko:fix/issue-write-index-quoting
shoko:fix/issue-cmd-destroy-unbound-var
shoko:fix/issue-daemon-worktree-path-fix
shoko:fix/issue-syntax-error-372
shoko:fix/issue-187-start-forks-agent
shoko:fix/issue-185-worktree-wrong-directory
shoko:fix/issue-183-destroy-base-requires-target
shoko:fix/issue-181-get-repo-url-strips-user-org
shoko:fix/issue-179-worktree-path-doubled
shoko:fix/issue-176-extract-issue-ref
shoko:fix/issue-174-queue-daemon-crashes
shoko:fix/issue-172-init-script-wrong-session-ids
shoko:fix/issue-170-duplicate-update-queue
shoko:fix/issue-167-notification-bash
shoko:fix/issue-156-queue-fixes
shoko:fix/issue-155-queue-list-json
shoko:fix/issue-160-gitea-token-from-pm-agent
shoko:fix/issue-156
shoko:fix/issue-150
shoko:fix/issue-116-modularize-script
shoko:fix/issue-120
shoko:fix/issue-144-parse-issue-ref-format-v2
shoko:fix/issue-queue-daemon-excess-agents
shoko:fix/issue-142-process-queue-missing-parens
No Reviewers
Labels
Clear labels
0.2.x-only
0.3.x-only
backport
blocked
bug
critical
feature
foundation
high
meta
need-review
parallelization
ready
ready-to-merge
task-delegation
Available only in 0.2.x, not backportable
Available only in 0.3.x, not backportable
Backport compatible - can merge to 0.1.x
Issue needs clarification/discussion before it can be worked on
Bug report
Critical - blocking parallel work
New feature
Foundation work for 0.2.x
High priority - workflow improvements
Meta issue tracking multiple issues
PR needs review before merge
Issues related to parallelization
Issue is clear and ready to be implemented/delegated
PR approved and ready to merge
Issues related to task-delegation
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: shoko/kugetsu#94
Reference in New Issue
Block a user
Blocking a user prevents them from interacting with repositories, such as opening or commenting on pull requests or issues. Learn more about blocking a user.
Delete Branch "fix/issue-45-post-comment-v2"
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?
Summary
Fix for #45 - PM agent cannot write to /tmp for comment posting.
Problem
PM agent tried to use
write_fileto create temp files in /tmp, but opencode blocks/tmp/*access in headless mode.Solution
Add
kugetsu post-commentcommand that posts Gitea comments directly without needing temp files.Usage
The command:
Fixes #45
if we merged #92 or #93, does this PR is still necessary?
Yes, #94 is still necessary independently. It serves a different purpose:
The post-comment helper is a tool/command that the PM agent can use to communicate back to the issue. Its not related to temp dirs or permissions.
It would be useful if the PM agent could append its output/context to the issue as a comment for transparency and for the user to track progress.
why do we need a specific tool/command when the PM agent can just write a file in a temporary directory now (thanks to #92)? what if we have 100 git servers, do we have to add 100 commands?
You make a fair point! With #92, the PM agent can write files to temp directory.
However,
kugetsu post-commentprovides:But youre right - for 100 git servers, wed need different commands or make it more generic. We could:
Option 3 seems cleanest - the PM agent has GITEA_TOKEN and can use curl directly. Want me to close this PR in favor of that approach?
Pull request closed