Handle pre/post-link/unlink scripts

[Hind-M]

Description

Fix #4310
Fix #3761

Type of Change

• Bugfix
• Feature / enhancement
• CI / Documentation
• Maintenance

Checklist

• My code follows the general style and conventions of the codebase, ensuring consistency
• I have performed a self-review of my code
• I have commented my code, particularly in hard-to-understand areas
• My changes generate no new warnings
• I have run pre-commit run --all locally in the source folder and confirmed that there are no linter errors.
• I have added tests that prove my fix is effective or that my feature works
• New and existing tests pass locally with my changes

[codecov]

Codecov Report

❌ Patch coverage is 92.59259% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.08%. Comparing base (041e86d) to head (f434523).
⚠️ Report is 1 commits behind head on main.

Files with missing lines	Patch %	Lines
libmamba/src/core/transaction.cpp	0.00%	2 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #4313      +/-   ##
==========================================
+ Coverage   54.58%   55.08%   +0.49%     
==========================================
  Files         240      240              
  Lines       30257    30282      +25     
  Branches     3234     3235       +1     
==========================================
+ Hits        16516    16680     +164     
+ Misses      13738    13599     -139     
  Partials        3        3              

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[jjerphan]

Thank you.

I think that we might need to think of ways for users to opt-out of warnings (which will be annoying for them) and from this new behavior.

[Hind-M]

I think that we might need to think of ways for users to opt-out of warnings (which will be annoying for them) and from this new behavior.

Yes I was thinking about that too as they do in Pixi, I think we can add a way to opt-out the scripts execution and in that case not print the warnings. I can do that in a follow-up PR.

[Klaim]

LGTM, my comments are mostly minor improvements, notes and questions which answers probably wont change the validity.