Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update appcleaner extension #17115

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

tborychowski
Copy link
Contributor

@tborychowski tborychowski commented Feb 16, 2025

Description

Screencast

Checklist

- fixes app icon finding crash
- Merge branch \'contributions/merge-1739747587815\'
- Pull contributions
- Merge branch \'contributions/merge-1739747490405\'
- Pull contributions
- Merge branch \'contributions/merge-1739747471183\'
- Pull contributions
@raycastbot raycastbot added extension fix / improvement Label for PRs with extension's fix improvements extension: appcleaner Issues related to the appcleaner extension OP is author The OP of the PR is the author of the extension labels Feb 16, 2025
@raycastbot
Copy link
Collaborator

raycastbot commented Feb 16, 2025

Thank you for the update! 🎉

Due to our current reduced availability, the initial review may take up to 10-15 business days

Copy link

@greptile-apps greptile-apps bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

PR Summary

This PR enhances the AppCleaner extension's icon handling capabilities by implementing robust fallback mechanisms for applications with non-standard folder structures.

  • Added comprehensive icon path fallbacks in src/lib/app-utils.ts to handle apps without standard /Contents/Resources structure
  • Improved icon file detection with support for both .icns and .png formats using regex patterns
  • Simplified app filtering logic in src/lib/app-manager.ts by moving system app checks earlier in the process
  • Enhanced error handling for file system operations when scanning application directories
  • Updated PearCleaner attribution in README with proper GitHub links

💡 (1/5) You can manually trigger the bot by mentioning @greptileai in a comment!

6 file(s) reviewed, 5 comment(s)
Edit PR Review Bot Settings | Greptile

@pernielsentikaer pernielsentikaer self-assigned this Feb 18, 2025
tborychowski and others added 4 commits February 18, 2025 08:53
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Co-authored-by: greptile-apps[bot] <165735046+greptile-apps[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
extension: appcleaner Issues related to the appcleaner extension extension fix / improvement Label for PRs with extension's fix improvements OP is author The OP of the PR is the author of the extension
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[App Cleaner] Extension crashes after installtion [App Cleaner] ... error after installation
3 participants