Upgrade Cmd Removes All Files Including Virtual Environment

[axif0]

Terms

• I have searched all open bug reports
• I agree to follow Scribe-Data's Code of Conduct

Behavior

Description

Running upgrade.py as,

scribe-data --upgrade

Removes all root files, including venv, instead of only updating necessary files which have changes in newer scribe version. Might be for this line, shutil.rmtree(Path.cwd() / item.name)

Expected behavior might be

• Only update required files.

• Preserve venv and other user-created files.

Also, add tests for upgrade.py.

Contribution

Happy to support this! I can also provide a review and check the PR once it's opened 😊

[andrewtavis]

Feel free to add labels and the tags for the Scribe Board to new issues, @axif0 :)

[meerhaasham1]

Hello @andrewtavis
can I work on this

[andrewtavis]

Sounds good, @meerhaasham1 :) Please let us know if you need any support!

[meerhaasham1]

Hi @andrewtavis
Just to update you, I have completed the task of identifying the issue with the scribe-data --upgrade command.

I’ve made the necessary changes to ensure that the command now behaves as expected, updating only the required files while preserving the venv and other user-created files.

Currently, I’m working on writing test cases for test_upgrade.py to ensure everything is functioning correctly.

I’ll update you once the tests are ready, and the pull request is submitted.

Thanks

[andrewtavis]

Thanks for the update, @meerhaasham1! 😊

[meerhaasham1]

Hi @andrewtavis,
Just a quick update — I’m currently working on the remaining issues related to “Add missing tests for files with low coverage” (#598).
Also, a gentle reminder that I’ve submitted a PR for issue #596 earlier and just wanted to follow up in case it might have been missed.
Thanks again!