-
Notifications
You must be signed in to change notification settings - Fork 3
Description
Describe the bug
Amazon Q keeps saying "Something went wrong and I was unable to complete the pull request revision. This may be because of content guidelines that I cannot comply with." when asking to implement something in a PR. Literally 100% there's nothing wrong in "content guidelines" and it's probably just a few symbols that trigger the filter. All I have to do is reword it a couple times until it works.
Expected Behavior
Just work. It's not like I'm asking it to make a virus or something.
Current Behavior
I told it mapStateToProps is missing the exportingSpriteId field, so it had to add that, and it replied with that. To fix this i just removed like all symbols and direct lines of code in the comment so I had to spoonfeed it the english words.
Reproduction Steps
"/q fix this pls
The real issue: mapStateToProps is missing the exportingSpriteId field. Just add this line to the object in src/containers/target-pane.jsx:
const mapStateToProps = state => ({
editingTarget: state.scratchGui.targets.editingTarget,
hoveredTarget: {
...state.scratchGui.hoveredTarget,
sprite: state.scratchGui.hoveredTarget.sprite === null ?
null :
String(state.scratchGui.hoveredTarget.sprite)
},
exportingSpriteId: state.scratchGui.exportJust.exportingSpriteId, // <-- Add this line
isRtl: state.locales.isRtl,
spriteLibraryVisible: state.scratchGui.modals.spriteLibrary,
sprites: state.scratchGui.targets.sprites,
stage: state.scratchGui.targets.stage,
raiseSprites: state.scratchGui.blockDrag,
workspaceMetrics: state.scratchGui.workspaceMetrics
});
That should fix the build error!"
Possible Solution
Make the filters better and have less false positives
Additional Information/Context
Keep it up, the bot is hallucinating way less with recent updates and is actually useful, it's just little nitpicks like this that are annoying