[PR #563] refactor: unified path handle for windows and unix like system #584

Open
opened 2026-02-16 01:16:24 -05:00 by yindo · 0 comments
Owner

📋 Pull Request Information

Original PR: https://github.com/langgenius/dify-plugin-daemon/pull/563
Author: @fatelei
Created: 1/10/2026
Status: 🔄 Open

Base: mainHead: issue-556


📝 Commits (2)

  • 762918e refactor: unified path handle for windows and unix like system
  • 11671f5 chore: add more comment

📊 Changes

10 files changed (+782 additions, -114 deletions)

View changed files

📝 pkg/plugin_packager/decoder/fs.go (+67 -3)
📝 pkg/plugin_packager/decoder/helper.go (+173 -104)
pkg/plugin_packager/decoder/helper_fs_fromslash_test.go (+55 -0)
pkg/plugin_packager/decoder/helper_models_test.go (+134 -0)
pkg/plugin_packager/decoder/helper_providers_test.go (+248 -0)
pkg/plugin_packager/decoder/pathutil.go (+59 -0)
pkg/plugin_packager/decoder/pathutil_test.go (+41 -0)
📝 pkg/utils/cache/helper/keys.go (+1 -1)
📝 pkg/utils/mapping/sync.go (+1 -1)
📝 pkg/utils/mapping/sync_test.go (+3 -5)

📄 Description

Description

fix #556

refactor: unified path handle for windows and unix like system

Type of Change

  • Bug fix
  • New feature
  • Refactor
  • Performance improvement
  • Other

Essential Checklist

Testing

  • I have tested the changes locally and confirmed they work as expected
  • I have added unit tests where necessary and they pass successfully

Bug Fix (if applicable)

  • I have used GitHub syntax to close the related issue (e.g., Fixes #123 or Closes #123)

Additional Information

Please provide any additional context that would help reviewers understand the changes.


🔄 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-plugin-daemon/pull/563 **Author:** [@fatelei](https://github.com/fatelei) **Created:** 1/10/2026 **Status:** 🔄 Open **Base:** `main` ← **Head:** `issue-556` --- ### 📝 Commits (2) - [`762918e`](https://github.com/langgenius/dify-plugin-daemon/commit/762918e9e38179322146cd7d2fe987cc6df7cb70) refactor: unified path handle for windows and unix like system - [`11671f5`](https://github.com/langgenius/dify-plugin-daemon/commit/11671f507b1a9578c7ad9485959ed741934dd11f) chore: add more comment ### 📊 Changes **10 files changed** (+782 additions, -114 deletions) <details> <summary>View changed files</summary> 📝 `pkg/plugin_packager/decoder/fs.go` (+67 -3) 📝 `pkg/plugin_packager/decoder/helper.go` (+173 -104) ➕ `pkg/plugin_packager/decoder/helper_fs_fromslash_test.go` (+55 -0) ➕ `pkg/plugin_packager/decoder/helper_models_test.go` (+134 -0) ➕ `pkg/plugin_packager/decoder/helper_providers_test.go` (+248 -0) ➕ `pkg/plugin_packager/decoder/pathutil.go` (+59 -0) ➕ `pkg/plugin_packager/decoder/pathutil_test.go` (+41 -0) 📝 `pkg/utils/cache/helper/keys.go` (+1 -1) 📝 `pkg/utils/mapping/sync.go` (+1 -1) 📝 `pkg/utils/mapping/sync_test.go` (+3 -5) </details> ### 📄 Description ## Description fix #556 refactor: unified path handle for windows and unix like system ## Type of Change - [x] Bug fix - [ ] New feature - [ ] Refactor - [ ] Performance improvement - [ ] Other ## Essential Checklist ### Testing - [x] I have tested the changes locally and confirmed they work as expected - [ ] I have added unit tests where necessary and they pass successfully ### Bug Fix (if applicable) - [x] I have used GitHub syntax to close the related issue (e.g., `Fixes #123` or `Closes #123`) ## Additional Information Please provide any additional context that would help reviewers understand the changes. --- <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 01:16:24 -05:00
Sign in to join this conversation.
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: langgenius/dify-plugin-daemon#584