[PR #215] [MERGED] refactor: de-duplicate lark event dispatcher setup #229

Closed
opened 2026-02-15 21:16:20 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-sdks/pull/215
Author: @Yeuoly
Created: 10/13/2025
Status: Merged
Merged: 10/13/2025
Merged by: @Yeuoly

Base: feat/triggerHead: codex/audit-lark-trigger-implementation


📝 Commits (7)

  • 0226b5a refactor: reuse lark event dispatch setup
  • fc32deb fix: expose structured lark trigger payloads
  • 81615b5 Update python/examples/lark_trigger/events/contact/user_updated_v3.py
  • a8beb4d Update python/examples/lark_trigger/events/contact/user_updated_v3.py
  • 9970888 Update python/examples/lark_trigger/events/contact/user_updated_v3.py
  • 5159a14 Update python/examples/lark_trigger/events/contact/user_updated_v3.py
  • 490383e Update python/examples/lark_trigger/events/contact/user_updated_v3.py

📊 Changes

26 files changed (+2391 additions, -65 deletions)

View changed files

📝 python/examples/lark_trigger/TODO.md (+137 -42)
python/examples/lark_trigger/events/__init__.py (+0 -0)
python/examples/lark_trigger/events/_shared.py (+74 -0)
python/examples/lark_trigger/events/calendar/__init__.py (+0 -0)
python/examples/lark_trigger/events/calendar/calendar_acl_created_v4.py (+45 -0)
python/examples/lark_trigger/events/calendar/calendar_acl_created_v4.yaml (+96 -0)
python/examples/lark_trigger/events/calendar/calendar_acl_deleted_v4.py (+45 -0)
python/examples/lark_trigger/events/calendar/calendar_acl_deleted_v4.yaml (+96 -0)
python/examples/lark_trigger/events/contact/__init__.py (+0 -0)
python/examples/lark_trigger/events/contact/department_deleted_v3.py (+97 -0)
python/examples/lark_trigger/events/contact/department_deleted_v3.yaml (+155 -0)
python/examples/lark_trigger/events/contact/department_updated_v3.py (+86 -0)
python/examples/lark_trigger/events/contact/department_updated_v3.yaml (+155 -0)
python/examples/lark_trigger/events/contact/user_updated_v3.py (+204 -0)
python/examples/lark_trigger/events/contact/user_updated_v3.yaml (+388 -0)
python/examples/lark_trigger/events/drive/__init__.py (+0 -0)
python/examples/lark_trigger/events/drive/file_deleted_v1.py (+42 -0)
python/examples/lark_trigger/events/drive/file_deleted_v1.yaml (+96 -0)
python/examples/lark_trigger/events/drive/file_edit_v1.py (+38 -0)
python/examples/lark_trigger/events/drive/file_edit_v1.yaml (+117 -0)

...and 6 more files

📄 Description

Summary

  • extract a dispatch_single_event helper to centralize Lark trigger request validation and payload capture
  • refactor calendar, drive, and contact handlers to reuse the helper and remove repeated dispatcher boilerplate

Testing

  • ruff check python/examples/lark_trigger/events/_shared.py python/examples/lark_trigger/events/calendar/calendar_acl_created_v4.py python/examples/lark_trigger/events/calendar/calendar_acl_deleted_v4.py python/examples/lark_trigger/events/drive/file_deleted_v1.py python/examples/lark_trigger/events/drive/file_edit_v1.py python/examples/lark_trigger/events/drive/file_permission_member_added_v1.py python/examples/lark_trigger/events/drive/file_permission_member_removed_v1.py python/examples/lark_trigger/events/contact/department_deleted_v3.py python/examples/lark_trigger/events/contact/department_updated_v3.py python/examples/lark_trigger/events/contact/user_updated_v3.py

https://chatgpt.com/codex/tasks/task_b_68ecf38753ac8326a9878432c081b1d4


🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.

## 📋 Pull Request Information **Original PR:** https://github.com/langgenius/dify-plugin-sdks/pull/215 **Author:** [@Yeuoly](https://github.com/Yeuoly) **Created:** 10/13/2025 **Status:** ✅ Merged **Merged:** 10/13/2025 **Merged by:** [@Yeuoly](https://github.com/Yeuoly) **Base:** `feat/trigger` ← **Head:** `codex/audit-lark-trigger-implementation` --- ### 📝 Commits (7) - [`0226b5a`](https://github.com/langgenius/dify-plugin-sdks/commit/0226b5a31ea5b311117ba8ff94d3f671e20b96dd) refactor: reuse lark event dispatch setup - [`fc32deb`](https://github.com/langgenius/dify-plugin-sdks/commit/fc32deb40fcf2478d436d7f72f2c346dfa6e1d04) fix: expose structured lark trigger payloads - [`81615b5`](https://github.com/langgenius/dify-plugin-sdks/commit/81615b5f0ac87abbf2b8aa620acbdfa331a4ff74) Update python/examples/lark_trigger/events/contact/user_updated_v3.py - [`a8beb4d`](https://github.com/langgenius/dify-plugin-sdks/commit/a8beb4d6d4e3d6b8674b49e546085459a1fa7ca7) Update python/examples/lark_trigger/events/contact/user_updated_v3.py - [`9970888`](https://github.com/langgenius/dify-plugin-sdks/commit/99708883a47dd19ce4357d49591c3f824936dd2f) Update python/examples/lark_trigger/events/contact/user_updated_v3.py - [`5159a14`](https://github.com/langgenius/dify-plugin-sdks/commit/5159a14d57c132acf15c0bd6986aab5988d7f9e4) Update python/examples/lark_trigger/events/contact/user_updated_v3.py - [`490383e`](https://github.com/langgenius/dify-plugin-sdks/commit/490383e5b56696b6de0eaa7c63b8a21b8e5a9a62) Update python/examples/lark_trigger/events/contact/user_updated_v3.py ### 📊 Changes **26 files changed** (+2391 additions, -65 deletions) <details> <summary>View changed files</summary> 📝 `python/examples/lark_trigger/TODO.md` (+137 -42) ➕ `python/examples/lark_trigger/events/__init__.py` (+0 -0) ➕ `python/examples/lark_trigger/events/_shared.py` (+74 -0) ➕ `python/examples/lark_trigger/events/calendar/__init__.py` (+0 -0) ➕ `python/examples/lark_trigger/events/calendar/calendar_acl_created_v4.py` (+45 -0) ➕ `python/examples/lark_trigger/events/calendar/calendar_acl_created_v4.yaml` (+96 -0) ➕ `python/examples/lark_trigger/events/calendar/calendar_acl_deleted_v4.py` (+45 -0) ➕ `python/examples/lark_trigger/events/calendar/calendar_acl_deleted_v4.yaml` (+96 -0) ➕ `python/examples/lark_trigger/events/contact/__init__.py` (+0 -0) ➕ `python/examples/lark_trigger/events/contact/department_deleted_v3.py` (+97 -0) ➕ `python/examples/lark_trigger/events/contact/department_deleted_v3.yaml` (+155 -0) ➕ `python/examples/lark_trigger/events/contact/department_updated_v3.py` (+86 -0) ➕ `python/examples/lark_trigger/events/contact/department_updated_v3.yaml` (+155 -0) ➕ `python/examples/lark_trigger/events/contact/user_updated_v3.py` (+204 -0) ➕ `python/examples/lark_trigger/events/contact/user_updated_v3.yaml` (+388 -0) ➕ `python/examples/lark_trigger/events/drive/__init__.py` (+0 -0) ➕ `python/examples/lark_trigger/events/drive/file_deleted_v1.py` (+42 -0) ➕ `python/examples/lark_trigger/events/drive/file_deleted_v1.yaml` (+96 -0) ➕ `python/examples/lark_trigger/events/drive/file_edit_v1.py` (+38 -0) ➕ `python/examples/lark_trigger/events/drive/file_edit_v1.yaml` (+117 -0) _...and 6 more files_ </details> ### 📄 Description ## Summary - extract a `dispatch_single_event` helper to centralize Lark trigger request validation and payload capture - refactor calendar, drive, and contact handlers to reuse the helper and remove repeated dispatcher boilerplate ## Testing - ruff check python/examples/lark_trigger/events/_shared.py python/examples/lark_trigger/events/calendar/calendar_acl_created_v4.py python/examples/lark_trigger/events/calendar/calendar_acl_deleted_v4.py python/examples/lark_trigger/events/drive/file_deleted_v1.py python/examples/lark_trigger/events/drive/file_edit_v1.py python/examples/lark_trigger/events/drive/file_permission_member_added_v1.py python/examples/lark_trigger/events/drive/file_permission_member_removed_v1.py python/examples/lark_trigger/events/contact/department_deleted_v3.py python/examples/lark_trigger/events/contact/department_updated_v3.py python/examples/lark_trigger/events/contact/user_updated_v3.py ------ https://chatgpt.com/codex/tasks/task_b_68ecf38753ac8326a9878432c081b1d4 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>
yindo added the pull-request label 2026-02-15 21:16:20 -05:00
yindo closed this issue 2026-02-15 21:16:20 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-plugin-sdks#229