Skip to content

Update plugin: table_converter to 0.0.12 (bump dify_plugin requiremen…#2676

Merged
crazywoola merged 3 commits into
langgenius:mainfrom
fangyong20062006:update-table_converter-0_0_12
Jul 11, 2026
Merged

Update plugin: table_converter to 0.0.12 (bump dify_plugin requiremen…#2676
crazywoola merged 3 commits into
langgenius:mainfrom
fangyong20062006:update-table_converter-0_0_12

Conversation

@fangyong20062006

Copy link
Copy Markdown
Contributor

Plugin Submission

Plugin information

Submission type

  • Update existing plugin

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):

  • Bump dify_plugin requirement from ">=0.2.0,<0.3.0" to ">=0.9.0" as
    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.
  • Polish plugin descriptions to focus on functionality (removed
    install-method notes from description fields).
  • No functional/code changes since 0.0.10.

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

  • I have read the contributing guidelines.
  • The plugin has been tested locally and works as expected.
  • The plugin includes a privacy policy (PRIVACY.md).
  • The manifest contains valid author contact information.
  • The plugin's purpose is reasonable within the Dify ecosystem.

…t to >=0.9.0 per marketplace review; refocus descriptions on functionality)
@github-actions github-actions Bot added the risk: missing Missing or invalid Marketplace risk selection label Jul 8, 2026
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

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 crazywoola left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

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.

@fangyong20062006

Copy link
Copy Markdown
Contributor Author

README updated with the source repository link and setup details, package rebuilt

@crazywoola crazywoola left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

❌ 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 crazywoola left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

❌ 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.

@crazywoola crazywoola merged commit 7ca6baa into langgenius:main Jul 11, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: missing Missing or invalid Marketplace risk selection

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants