Skip to content

cherry-studio@1.5.6: Switch Cherry Studio manifest to portable version#16005

Closed
GeorgeDong32 wants to merge 231 commits intoScoopInstaller:masterfrom
GeorgeDong32:update-Cherry-Studio
Closed

cherry-studio@1.5.6: Switch Cherry Studio manifest to portable version#16005
GeorgeDong32 wants to merge 231 commits intoScoopInstaller:masterfrom
GeorgeDong32:update-Cherry-Studio

Conversation

@GeorgeDong32
Copy link
Copy Markdown
Contributor

@GeorgeDong32 GeorgeDong32 commented Aug 17, 2025

  • Switch Cherry Studio manifest to portable version
  • Add notes and depends
  • Optimize persist
  • Use conventional PR title: <manifest-name[@version]|chore>: <general summary of the pull request>
  • I have read the Contributing Guide

- Switch Cherry Studio manifest to portable version
- Add notes and depends
- Optimize persist
@github-actions
Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

cherry-studio

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@GeorgeDong32
Copy link
Copy Markdown
Contributor Author

/verify

@github-actions
Copy link
Copy Markdown
Contributor

All changes look good.

Wait for review from human collaborators.

cherry-studio

  • Description
  • License
  • Hashes
  • Checkver
  • Autoupdate

@GeorgeDong32
Copy link
Copy Markdown
Contributor Author

Hi here, I am a core developer of Cherry Studio. This PR aims to resolve previously encountered data and dependency issues during Scoop installation. Please let us know of any inaccuracies, and we will promptly address them.

Comment on lines +22 to +39
"# Handle data directory migration",
"if (Test-Path \"$persist_dir\\data\") {",
" $dataItems = Get-ChildItem \"$persist_dir\\data\" -ErrorAction SilentlyContinue",
" if (-not $dataItems) {",
" # persist_dir\\data exists but is empty, check if we need to copy from APPDATA",
" if ((Test-Path \"$env:APPDATA\\CherryStudio\") -and (Get-ChildItem \"$env:APPDATA\\CherryStudio\" -ErrorAction SilentlyContinue)) {",
" Copy-Item \"$env:APPDATA\\CherryStudio\\*\" \"$persist_dir\\data\" -Recurse -Force",
" info \"Data copied from '$env:APPDATA\\CherryStudio' to persist directory.\"",
" }",
" }",
"} else {",
" # persist_dir\\data does not exist",
" if ((Test-Path \"$env:APPDATA\\CherryStudio\") -and (Get-ChildItem \"$env:APPDATA\\CherryStudio\" -ErrorAction SilentlyContinue)) {",
" ensure \"$persist_dir\\data\" | Out-Null",
" Copy-Item \"$env:APPDATA\\CherryStudio\\*\" \"$persist_dir\\data\" -Recurse -Force",
" info \"Data copied from '$env:APPDATA\\CherryStudio' to persist directory.\"",
" }",
"}"
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.

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 your help. Based on the recommendation in ScoopInstaller/Main#6462, as long as this change does not result in data loss or issues for existing users, we should be fine. If user data is preserved and persisted correctly, a clear note in the manifest should be sufficient. So I would recommend you to remove the post_install script.

@GeorgeDong32
Copy link
Copy Markdown
Contributor Author

/verify

@GeorgeDong32 GeorgeDong32 deleted the update-Cherry-Studio branch August 23, 2025 14:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants