Skip to content

Conversation

wolovim
Copy link
Member

@wolovim wolovim commented Jul 31, 2025

this is a major overhaul of the bot that manages protocol call resources (e.g., calendar, zoom, ethmag posts)

background:

  • most of this originates with changing the data structure of mapping json file, which is effectively the database. today zoom ids are the key for each call object. often, facilitators bring their own meeting and forego usage of the zoom account managed by this bot. some categories of problems start there. now, the call series (e.g., acde, l2interop, etc.) serves as the key name, with more specifics within each occurrence of the call.
  • another major category of bugs today is data input-related, as the current markdown template is a bit too permissive. a new form template with dropdowns & checkboxes replaces the old template.
  • introducing those two big changes opened the door to rethinking the connective tissue in between. the monolithic handle_issue script has been replaced with a slightly smaller script but one that is more observable and resilient, with a set of unit-tested subcomponents - 97 unit tests to start.

id like to acknowledge:

  • this PR is impractical to meaningfully review, given its size. sorry bout that. parts of this could have been achieved incrementally, but i went down that path a couple times before ultimately deciding we would be better served by ripping the bandaid off at once while addressing the root issues.
  • something will break when this meets reality. im unable to test all the integrations locally. ill test existing issues one at a time though by applying a label to kick off the workflow, then address issues as they arise in follow-up PRs.

Copy link
Member

@taxmeifyoucan taxmeifyoucan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All provided tests are passing and the Zoom integration works when trying with my account. I couldn't try other features or read the whole thing but no major issue found

@wolovim wolovim merged commit f22317f into ethereum:master Aug 1, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants