Skip to content

Update plugin: table_converter to 0.0.10 (fix hang on large xlsx: blo…#2648

Closed
fangyong20062006 wants to merge 1 commit into
langgenius:mainfrom
fangyong20062006:update-table_converter-0_0_10
Closed

Update plugin: table_converter to 0.0.10 (fix hang on large xlsx: blo…#2648
fangyong20062006 wants to merge 1 commit into
langgenius:mainfrom
fangyong20062006:update-table_converter-0_0_10

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.10:

  • Fix node hang on large .xlsx inputs: hard timeout on lazy file blob fetch
    (falls back to URL download), and large text results are now emitted in
    newline-aligned ~512KB chunks instead of one oversized message.
  • Fix "Attempt to use ZIP archive that was already closed" when converting
    only the first sheet (read-only workbook was closed prematurely by a
    generator finalizer).
  • Fix escaped pipe cells (|) being split into extra columns in
    Markdown -> Excel/CSV round-trips.
  • Tolerate ragged rows in Excel -> JSON and treat placeholder sheet_name
    values ("/", "None", "null") as unset.

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.

…b hard-timeout + chunked text output; fix read-only workbook closed prematurely; fix escaped-pipe cells in markdown->excel; tolerate ragged rows and placeholder sheet_name)

@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 submission. A few required checks need fixes before this can be approved.

Check Status Required action
Single .difypkg ✅ Pass None.
PR language ✅ Pass None.
Project structure ✅ Pass None.
Manifest author ✅ Pass None.
Icon ✅ Pass None.
Version availability ✅ Pass None.
README language ✅ Pass None.
PRIVACY.md ✅ Pass None.
Dependencies / install / packaging ✅ Pass CI pre-check passed.
dify_plugin version ❌ Fail requirements.txt declares dify_plugin>=0.2.0,<0.3.0; please update it to dify_plugin>=0.9.0 or newer and rebuild the package.

Next steps

  1. requirements.txt declares dify_plugin>=0.2.0,<0.3.0; please update it to dify_plugin>=0.9.0 or newer and rebuild the package.

@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
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 ❌ Fail Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-eadfd9j1/.venv/bin/python /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-eadfd9j1/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=1262, 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.10 is available. None.
README language ✅ Pass README.md CJK ratio=0.0% (zh=0, en=4563, 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 ❌ 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 ❌ Fail Command failed (1): /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-eadfd9j1/.venv/bin/python /var/folders/c5/ssq8b54n0rj6rbmghy7_2xdm0000gn/T/pr-review-helper-eadfd9j1/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

  • 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 crazywoola closed this Jul 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants