You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The generated description/changelog doesn't account for ignored and "bad extension" files. Additionally, if I make a PR that only changes some images the describe/update changelog command doesn't complete due to every file being ignored.
I think in this case it's valid to know these files changed (even if the LLM won't know what changed in them) so that it can be mentioned in the PR description or changelog.
The PR diff logic would need to be updated to mention every file but only show diff contents as desired.
Note there are two meanings of "ignore" now - "ignore" as in "don't compute a diff and report on changes" and "ignore" as in "never consider these files". The configuration would need to reflect that somehow.
Example LLM input:
PR Info:
Previous title: 'some title'
Previous description:
=====
nothing
=====
Branch: 'my-test'
The PR Diff:
=====
## file 'test1.png' was added
## file 'test-foo.png' was added
## file 'test-bar.png' was added
## file 'test-bla.png' was added
## file 'test-cat.png' was added
## file 'test-dog.png' was added
## file 'test9.png' was updated
## file 'test10.png' was deleted
=====
Note that lines in the diff body are prefixed with a symbol that represents the type of change: '-' for deletions, '+' for additions, and ' ' (a space) for unchanged lines.
Response (should be a valid YAML, and nothing else):
```yaml
Example output
type:
- Other
description: |-
- Added new image files: `test1.png`, `test-foo.png`, `test-bar.png`, `test-bla.png`, `test-cat.png`, and `test-dog.png`.
- Updated image file: `test9.png`.
- Deleted image file: `test10.png`.
title: |-
Add and update image files
The text was updated successfully, but these errors were encountered:
some stream-of-thought about it, since it's not an easy issue to solve:
When thinking about how to address it, I think of three questions:
how hard is it to solve it ?
can we solve it completely?
what is the current level of bad impact ?
let's start with Q2:
some files like .png,'.zip and others, are worth mentioning. other files, like auto-generated ones, are not as important
It can be quite challenging to distinguish between the two types. it's not a clear cut.
Q1:
It will require a significant change to pass also the raw files list, from all git providers, to the relevant tools prompt. Not impossible, but a significant change.
Q3:
Yeah, the update-changlog may skip some ignored files. is it really that bad ? Since the AI cannot know what was changed inside them anyway, wouldn't the user anyway need to adjust the output a bit in those cases ?
The generated description/changelog doesn't account for ignored and "bad extension" files. Additionally, if I make a PR that only changes some images the describe/update changelog command doesn't complete due to every file being ignored.
I think in this case it's valid to know these files changed (even if the LLM won't know what changed in them) so that it can be mentioned in the PR description or changelog.
The PR diff logic would need to be updated to mention every file but only show diff contents as desired.
Note there are two meanings of "ignore" now - "ignore" as in "don't compute a diff and report on changes" and "ignore" as in "never consider these files". The configuration would need to reflect that somehow.
Example LLM input:
Example output
The text was updated successfully, but these errors were encountered: