Update plugin: table_converter to 0.0.12 (bump dify_plugin requiremen…#2676
Conversation
…t to >=0.9.0 per marketplace review; refocus descriptions on functionality)
|
Please select exactly one risk level in the PR template: Low risk, Medium risk, or High risk. This helps Marketplace reviewers route the submission correctly. |
crazywoola
left a comment
There was a problem hiding this comment.
Thanks for the update. I unpacked the package and checked the package shape, manifest, README, privacy policy, dependency file, and the passing pre-check result.
| Check | Status | Required action |
|---|---|---|
Single .difypkg package |
✅ Pass | None. |
| PR language | ✅ Pass | None. |
| Project structure | ✅ Pass | None. |
| Manifest / icon / privacy file | ✅ Pass | None. |
| README language | ✅ Pass | None; README.md has no Chinese characters. |
| Pre-check CI | ✅ Pass | None. |
| README source repository link | ❌ Fail | Add the plugin source repository link to the packaged README.md, then rebuild table_converter_0.0.12.difypkg. The submission requirements ask the plugin README to include setup/usage details and the source repository link; the PR body has the link, but the package README does not. |
Next steps
Please add the source repository URL to README.md inside the package and regenerate the .difypkg so the Marketplace listing includes that required reference.
|
README updated with the source repository link and setup details, package rebuilt |
crazywoola
left a comment
There was a problem hiding this comment.
❌ Request Changes
Decision: Request changes
Failed Checks
| Check | Status | Detail | Required action |
|---|---|---|---|
| Single .difypkg | ❌ Fail | The PR adds two package files: table_converter_0.0.12.difypkg and table_converter_0_0_12.difypkg. | Keep exactly one correctly named .difypkg artifact in the PR. |
Next steps
- Remove the duplicate package so the PR changes exactly one .difypkg file.
- Push the update so the remaining package checks can run.
crazywoola
left a comment
There was a problem hiding this comment.
❌ Request Changes
Decision: Request changes
Failed Checks
| Check | Status | Detail | Required action |
|---|---|---|---|
Install test |
❌ Fail | Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-s2lmj0y5/.venv/bin/python /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-s2lmj0y5/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=1143, 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.0.12 is available. | None. |
README language |
✅ Pass | README.md CJK ratio=0.0% (zh=0, en=4881, allowed_zh<=0) | None. |
PRIVACY.md |
✅ Pass | PRIVACY.md exists and is non-empty. | None. |
Dependency install |
✅ Pass | requirements installed successfully. | None. |
dify_plugin version |
✅ Pass | dify_plugin version 0.9.1 satisfies >= 0.9.0. | None. |
Install test |
❌ Fail | Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-s2lmj0y5/.venv/bin/python /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-s2lmj0y5/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
- Install test: Fix the plugin so test-plugin-install.py passes.
Please address these issues and push an update.
Plugin Submission
Plugin information
Submission type
Description
Convert tables across Excel/CSV, JSON, and Markdown (four directions), plus
Excel/CSV to row-object arrays for Iteration nodes.
Changes in 0.0.12 (addresses review feedback on 0.0.10):
requested by marketplace review. All SDK APIs used (Tool, ToolProvider,
create_text_message / create_json_message / create_blob_message) are
unchanged in 0.9+, so no code changes were needed.
install-method notes from description fields).
The plugin processes data locally within the Dify runtime. It does not
perform browser automation, arbitrary URL fetching, proxying, or handle any
sensitive/health data, and does not collect or transmit any user data.
Checklist