Add tim80411/line-connect (Community Fix) v0.10.0 - #2839
Conversation
crazywoola
left a comment
There was a problem hiding this comment.
❌ Request Changes
Decision: Request changes
Failed Checks
| Check | Status | Detail | Required action |
|---|---|---|---|
README language |
❌ Fail | README.md CJK ratio=0.1% (zh=5, en=7914, allowed_zh<=0); Chinese characters are not allowed. Use multilingual README guidance: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme | Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme |
Dependency install |
❌ Fail | Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/.venv/bin/pip install -r /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/unpacked_plugin/requirements.txt error: subprocess-exited-with-e… | Ensure requirements.txt installs successfully in a clean environment. |
dify_plugin version |
❌ Fail | dify_plugin must be installed and >= 0.9.0, but it was not found. | Use dify_plugin >= 0.9.0 in the packaged plugin environment. |
Install test |
❌ Fail | Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/.venv/bin/python /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/toolkit/validator/test-plugin-install.py -d /var/folders/c5/ssq8b54n0rj6… | Fix the plugin so test-plugin-install.py passes. |
Full Check Results
| Check | Status | Detail | Required action |
|---|---|---|---|
PR content language |
✅ Pass | PR title/body CJK ratio=0.0% (zh=0, en=3550, ignored_zh=0, allowed_zh<=0) | None. |
Project structure |
✅ Pass | All expected files present: manifest.yaml, README.md, PRIVACY.md. _assets/: yes. | None. |
Manifest author |
✅ Pass | author is valid. | None. |
Icon validation |
✅ Pass | icon exists: _assets/icon.svg | None. |
Version check |
✅ Pass | version 0.10.0 is available. | None. |
README language |
❌ Fail | README.md CJK ratio=0.1% (zh=5, en=7914, allowed_zh<=0); Chinese characters are not allowed. Use multilingual README guidance: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme | Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme |
PRIVACY.md |
✅ Pass | PRIVACY.md exists and is non-empty. | None. |
Dependency install |
❌ Fail | Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/.venv/bin/pip install -r /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/unpacked_plugin/requirements.txt error: subprocess-exited-with-e… | Ensure requirements.txt installs successfully in a clean environment. |
dify_plugin version |
❌ Fail | dify_plugin must be installed and >= 0.9.0, but it was not found. | Use dify_plugin >= 0.9.0 in the packaged plugin environment. |
Install test |
❌ Fail | Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/.venv/bin/python /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-ftj3dmo4/toolkit/validator/test-plugin-install.py -d /var/folders/c5/ssq8b54n0rj6… | Fix the plugin so test-plugin-install.py passes. |
Packaging test |
✅ Pass | packaging check passed. | None. |
Next steps
- README language: Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme
- Dependency install: Ensure requirements.txt installs successfully in a clean environment.
- dify_plugin version: Use dify_plugin >= 0.9.0 in the packaged plugin environment.
- Install test: Fix the plugin so test-plugin-install.py passes.
Please address these issues and push an update.
crazywoola
left a comment
There was a problem hiding this comment.
❌ Request Changes
Decision: Request changes
Failed Checks
| Check | Status | Detail | Required action |
|---|---|---|---|
README language |
❌ Fail | README.md CJK ratio=0.1% (zh=5, en=7914, allowed_zh<=0); Chinese characters are not allowed. Use multilingual README guidance: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme | Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme |
dify_plugin version |
❌ Fail | dify_plugin version must be >= 0.9.0; found 0.2.4. | Use dify_plugin >= 0.9.0 in the packaged plugin environment. |
Full Check Results
| Check | Status | Detail | Required action |
|---|---|---|---|
PR content language |
✅ Pass | PR title/body CJK ratio=0.0% (zh=0, en=3550, ignored_zh=0, allowed_zh<=0) | None. |
Project structure |
✅ Pass | All expected files present: manifest.yaml, README.md, PRIVACY.md. _assets/: yes. | None. |
Manifest author |
✅ Pass | author is valid. | None. |
Icon validation |
✅ Pass | icon exists: _assets/icon.svg | None. |
Version check |
✅ Pass | version 0.10.0 is available. | None. |
README language |
❌ Fail | README.md CJK ratio=0.1% (zh=5, en=7914, allowed_zh<=0); Chinese characters are not allowed. Use multilingual README guidance: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme | Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme |
PRIVACY.md |
✅ Pass | PRIVACY.md exists and is non-empty. | None. |
Dependency install |
✅ Pass | requirements installed successfully. | None. |
dify_plugin version |
❌ Fail | dify_plugin version must be >= 0.9.0; found 0.2.4. | Use dify_plugin >= 0.9.0 in the packaged plugin environment. |
Install test |
✅ Pass | plugin install test passed. | None. |
Packaging test |
✅ Pass | packaging check passed. | None. |
Next steps
- README language: Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme
- dify_plugin version: Use dify_plugin >= 0.9.0 in the packaged plugin environment.
Please address these issues and push an update.
Superseded by a fresh full review of all open PRs on 2026-08-10.
crazywoola
left a comment
There was a problem hiding this comment.
❌ Request Changes
Decision: Request changes
Failed Checks
| Check | Status | Detail | Required action |
|---|---|---|---|
README language |
❌ Fail | README.md CJK ratio=0.1% (zh=5, en=7914, allowed_zh<=0); Chinese characters are not allowed. Use multilingual README guidance: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme | Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme |
dify_plugin version |
❌ Fail | dify_plugin version must be >= 0.9.0; found 0.2.4. | Use dify_plugin >= 0.9.0 in the packaged plugin environment. |
Full Check Results
| Check | Status | Detail | Required action |
|---|---|---|---|
PR content language |
✅ Pass | PR title/body CJK ratio=0.0% (zh=0, en=3550, ignored_zh=0, allowed_zh<=0) | None. |
Project structure |
✅ Pass | All expected files present: manifest.yaml, README.md, PRIVACY.md. _assets/: yes. | None. |
Manifest author |
✅ Pass | author is valid. | None. |
Icon validation |
✅ Pass | icon exists: _assets/icon.svg | None. |
Version check |
✅ Pass | version 0.10.0 is available. | None. |
README language |
❌ Fail | README.md CJK ratio=0.1% (zh=5, en=7914, allowed_zh<=0); Chinese characters are not allowed. Use multilingual README guidance: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme | Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme |
PRIVACY.md |
✅ Pass | PRIVACY.md exists and is non-empty. | None. |
Dependency install |
✅ Pass | requirements installed successfully. | None. |
dify_plugin version |
❌ Fail | dify_plugin version must be >= 0.9.0; found 0.2.4. | Use dify_plugin >= 0.9.0 in the packaged plugin environment. |
Install test |
✅ Pass | plugin install test passed. | None. |
Packaging test |
✅ Pass | packaging check passed. | None. |
Next steps
- README language: Remove Chinese characters from README.md and use the multilingual README pattern: https://docs.dify.ai/en/develop-plugin/features-and-specs/plugin-types/multilingual-readme#multilingual-readme
- dify_plugin version: Use dify_plugin >= 0.9.0 in the packaged plugin environment.
Please address these issues and push an update.
|
Hi @tim80411 — thanks for preparing this community fix! Two small blockers remain: the root |
|
@crazywoola thanks for the review. I'll update it ASAP |
- Root README.md is now English-only; translations moved to readme/README_zh_Hans.md, readme/README_ja_JP.md, readme/README_th_TH.md per the Dify multilingual README layout. - requirements.txt: dify_plugin>=0.9.0,<0.11.0 (was ~=0.2.1), with Werkzeug>=3.1.8 and requests>=2.33.1 to satisfy the SDK's floors. Verified against dify_plugin 0.10.2 on Python 3.12. - Package no longer carries .pytest_cache/, .gitignore or .difyignore. Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01WC5ZZrs3jEosnP1Ecxm3Jy
|
Hi @tim80411 — here is the current CI summary for commit ❌ Blocking errors
|
Plugin Submission
Plugin information
Submission type
What changed
Community fork of DomT00T/line-connect (MIT, attribution preserved in LICENSE/README) — a LINE Official Account ↔ Dify bridge endpoint plugin. This fork fixes the recurring "conversation resets to the greeting message" bug (upstream issue #1, reproduced again on 2026-08-05 on the latest upstream v0.9.7; upstream has had no commits for ~4 months):
Conversation Not Exists(4xx). Upstream dropped the cid after any two consecutive errors, silently resetting the conversation.exist→setstorage key per webhook event replaces the shared read-modify-write JSON list, removing the race under LINE webhook redelivery.All other functionality (admin dashboard, media, auto-reply, i18n) is unchanged from upstream v0.9.7.
Risk level
Required checks
.envfiles,.gitdirectories, virtual environments, caches, logs, or IDE files.PRIVACY.mdor a hosted privacy policy, andmanifest.yamlreferences it.Security and privacy notes
Same surface as upstream v0.9.7: handles LINE webhook events (user messages, media) and forwards them to the configured Dify app; verifies
X-Line-Signature(HMAC-SHA256); credentials are provided only via endpoint settings. No command/code execution, no SQL, no arbitrary URL fetching.PRIVACY.mdupdated with this fork's contact. No new data collection compared to upstream.Local validation
dify-plugin-darwin-arm64 v0.6.10:plugin packaged successfully(4.0 MB, 41 files; verified no.git/tests/state files included).Conversation Not Exists→ new conversation; on_cid fires on first SSE chunk; on_cid silent when cid unchanged; on_cid survives a cut stream with partial answer.py_compile+ endpoint import check againstdify_plugin==0.2.4(Python 3.12) pass.Reviewer notes
DomT00T/line-connectlisting. Happy to rename if reviewers prefer a different convention for forks.🤖 Generated with Claude Code
https://claude.ai/code/session_01BPwktbNWybDa1easvkqoEu