[PR #2527] [MERGED] feat: add Plivo Verify OTP plugin #2544

Closed
opened 2026-02-16 11:17:17 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-official-plugins/pull/2527
Author: @navi-
Created: 1/31/2026
Status: Merged
Merged: 2/2/2026
Merged by: @crazywoola

Base: mainHead: feat/add-plivo-verify-plugin


📝 Commits (6)

  • b8bb93b feat: add Plivo Verify OTP plugin
  • 315b4df chore: add uv.lock for plivo_verify plugin
  • 5de6cac Merge branch 'main' into feat/add-plivo-verify-plugin
  • a74b9c9 fix: add missing meta.version to plivo_verify manifest
  • 3c2bfb3 Merge branch 'main' into feat/add-plivo-verify-plugin
  • dc6808b refactor: address code review feedback in verify_otp.py

📊 Changes

19 files changed (+2741 additions, -0 deletions)

View changed files

tests/tools/plivo_verify/__init__.py (+0 -0)
tests/tools/plivo_verify/test_integration.py (+223 -0)
tests/tools/plivo_verify/test_manifest_and_provider.py (+129 -0)
tests/tools/plivo_verify/test_manual_full_flow.py (+145 -0)
tests/tools/plivo_verify/test_smoke_invoke.py (+463 -0)
tools/plivo_verify/.env.example (+4 -0)
tools/plivo_verify/README.md (+230 -0)
tools/plivo_verify/_assets/icon.svg (+5 -0)
tools/plivo_verify/main.py (+6 -0)
tools/plivo_verify/manifest.yaml (+31 -0)
tools/plivo_verify/provider/plivo_verify.py (+30 -0)
tools/plivo_verify/provider/plivo_verify.yaml (+43 -0)
tools/plivo_verify/pyproject.toml (+15 -0)
tools/plivo_verify/requirements.txt (+2 -0)
tools/plivo_verify/tools/send_otp.py (+50 -0)
tools/plivo_verify/tools/send_otp.yaml (+63 -0)
tools/plivo_verify/tools/verify_otp.py (+63 -0)
tools/plivo_verify/tools/verify_otp.yaml (+38 -0)
tools/plivo_verify/uv.lock (+1201 -0)

📄 Description

Summary

  • Adds Plivo Verify OTP plugin with send_otp and verify_otp tools
  • Includes comprehensive tests (unit, smoke, integration) for the plugin
  • Adds documentation with usage examples

Test plan

  • Verify plugin manifests are valid (8 tests passed)
  • Run smoke tests for all tools (18 tests passed)
  • Run integration tests against real Plivo API (9 tests passed)
  • Test full OTP flow (send → verify) with real Plivo credentials

🤖 Generated with Claude Code


🔄 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-official-plugins/pull/2527 **Author:** [@navi-](https://github.com/navi-) **Created:** 1/31/2026 **Status:** ✅ Merged **Merged:** 2/2/2026 **Merged by:** [@crazywoola](https://github.com/crazywoola) **Base:** `main` ← **Head:** `feat/add-plivo-verify-plugin` --- ### 📝 Commits (6) - [`b8bb93b`](https://github.com/langgenius/dify-official-plugins/commit/b8bb93bbe48dd077423911b83772965b85bd17dc) feat: add Plivo Verify OTP plugin - [`315b4df`](https://github.com/langgenius/dify-official-plugins/commit/315b4df54b954ad1c406679faa91a06c8726aaa3) chore: add uv.lock for plivo_verify plugin - [`5de6cac`](https://github.com/langgenius/dify-official-plugins/commit/5de6cac9b784577abc7aed997b3a215c6a531022) Merge branch 'main' into feat/add-plivo-verify-plugin - [`a74b9c9`](https://github.com/langgenius/dify-official-plugins/commit/a74b9c980c212d9e1de76948801f9d8011118c1c) fix: add missing meta.version to plivo_verify manifest - [`3c2bfb3`](https://github.com/langgenius/dify-official-plugins/commit/3c2bfb39b080f87b98465b9691a3f5d873dff186) Merge branch 'main' into feat/add-plivo-verify-plugin - [`dc6808b`](https://github.com/langgenius/dify-official-plugins/commit/dc6808b939c5b8a40389a27271ac38ae556acb09) refactor: address code review feedback in verify_otp.py ### 📊 Changes **19 files changed** (+2741 additions, -0 deletions) <details> <summary>View changed files</summary> ➕ `tests/tools/plivo_verify/__init__.py` (+0 -0) ➕ `tests/tools/plivo_verify/test_integration.py` (+223 -0) ➕ `tests/tools/plivo_verify/test_manifest_and_provider.py` (+129 -0) ➕ `tests/tools/plivo_verify/test_manual_full_flow.py` (+145 -0) ➕ `tests/tools/plivo_verify/test_smoke_invoke.py` (+463 -0) ➕ `tools/plivo_verify/.env.example` (+4 -0) ➕ `tools/plivo_verify/README.md` (+230 -0) ➕ `tools/plivo_verify/_assets/icon.svg` (+5 -0) ➕ `tools/plivo_verify/main.py` (+6 -0) ➕ `tools/plivo_verify/manifest.yaml` (+31 -0) ➕ `tools/plivo_verify/provider/plivo_verify.py` (+30 -0) ➕ `tools/plivo_verify/provider/plivo_verify.yaml` (+43 -0) ➕ `tools/plivo_verify/pyproject.toml` (+15 -0) ➕ `tools/plivo_verify/requirements.txt` (+2 -0) ➕ `tools/plivo_verify/tools/send_otp.py` (+50 -0) ➕ `tools/plivo_verify/tools/send_otp.yaml` (+63 -0) ➕ `tools/plivo_verify/tools/verify_otp.py` (+63 -0) ➕ `tools/plivo_verify/tools/verify_otp.yaml` (+38 -0) ➕ `tools/plivo_verify/uv.lock` (+1201 -0) </details> ### 📄 Description ## Summary - Adds Plivo Verify OTP plugin with send_otp and verify_otp tools - Includes comprehensive tests (unit, smoke, integration) for the plugin - Adds documentation with usage examples ## Test plan - [x] Verify plugin manifests are valid (8 tests passed) - [x] Run smoke tests for all tools (18 tests passed) - [x] Run integration tests against real Plivo API (9 tests passed) - [x] Test full OTP flow (send → verify) with real Plivo credentials 🤖 Generated with [Claude Code](https://claude.com/claude-code) --- <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-16 11:17:17 -05:00
yindo closed this issue 2026-02-16 11:17:17 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-official-plugins#2544