fix: improve global self-update scope detection#337
Conversation
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (4)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe PR hardens global installation detection in ChangesGlobal Installation Detection with XDG Support
Submodule Reference Update
🎯 2 (Simple) | ⏱️ ~12 minutes
✨ Finishing Touches📝 Generate docstrings
🧪 Generate unit tests (beta)
Comment |
|
🚀 Preview is available for this pull request. |
Related Issue
Closes #335
Motivation / Context
dev-tools self-updatewas still behaving as local in certain environments (symlinked/alternate Composer home paths), so global updates could route through local project contexts.The branch introduces a more robust Composer scope detection flow for self-update, including:
realpath/canonicalization fallback),XDG_CONFIG_HOME/composeras a valid Composer home candidate, andChanges
self-updatehome-path canonicalization inComposerSelfUpdateScopeResolver.XDG_CONFIG_HOMEas an additional resolution candidate for global composer home detection.ComposerSelfUpdateScopeResolverTestfor the new candidate path.Verification
composer dev-toolsvia commit-time hookDocumentation / Generated Output
docs/updatedChangelog
CHANGELOG.mdentryReviewer Notes
This is the intermediate self-update fix requested before full git-hooks externalization; the issue-specific change is intentionally scoped.