Skip to content

Bug Fix#93

Open
warlordwinter wants to merge 2 commits intobeanlab:mainfrom
warlordwinter:main
Open

Bug Fix#93
warlordwinter wants to merge 2 commits intobeanlab:mainfrom
warlordwinter:main

Conversation

@warlordwinter
Copy link
Copy Markdown

Error Report

During investigation I got this error
KeyError: 'duck-1364046911727865909-1367875125134757908' → While trying to delete an entry from self._workflows:

Recommend action

quest.manager.py tries to clean up by deleting the workflow, and that’s where the KeyError hits because the ID is missing from the dict. My hunch is that when duck channel's id is added to the workflow manager it is not being properly added or stopped.

        if message['channel_id'] in self._duck_channels:
            workflow_id = f'duck-{message["channel_id"]}-{message["message_id"]}'
            self._workflow_manager.start_workflow(
                'duck',
                workflow_id,
                message["channel_id"],
                message
            )

@warlordwinter warlordwinter added the bug Something isn't working label May 2, 2025
@warlordwinter warlordwinter requested a review from byubean May 2, 2025 16:19
@warlordwinter warlordwinter self-assigned this May 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant