fix: check the context of the issue, which injected to the prompt (#38) - #39
Merged
Conversation
FetchIssueContent rendered every comment on the issue into the prompt, including the daemon's own chatter. The pickup comment is posted before the fetch, so it landed in the prompt even on the first run; on a re-run the park comments (with their full error dumps), PR links and stop notices piled up too, so the model read a transcript of previous runs as if it were part of the report. Every status comment now ends with a hidden `<!-- loope:bot -->` marker and FetchIssueContent drops marked comments. Two judgment calls, made headlessly: - needs-info is deliberately NOT marked: it carries the numbered questions a human answers in the following comment, so stripping it would orphan the answer. - comments posted before the marker existed are matched by the exact opening text of each template (legacyBotStatusPrefixes), so issues already carrying chatter — like #38 itself — are cleaned up too, without risking a human comment. The UAT checklist the tool appends to the issue *body* is left in place: it states what the change must satisfy, so it is context, not noise. Closes #38
16 tasks
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Closes #38
Automated by loope (bug flow). Spec and plan, if any, are committed in this branch under docs/.