Skip to content

Conversation

@mateuuszzzzz
Copy link
Contributor

@mateuuszzzzz mateuuszzzzz commented Nov 6, 2025

Summary

  • Added ccache to the list of cleanup tasks
  • Implemented isCCacheInstalled() to check if ccache is available using which ccache
  • Changed createCleanupTasks() to async to support spawn-based availability checks

Test plan

When ccache is installed:

  • Run npx rock clean
  • Verify that [C/C++] CCache compiler cache appears in the cleanup options
  • Select it and verify it runs ccache --clear successfully

When ccache is NOT installed:

  • Run npx rock clean
  • Verify that ccache option does NOT appear in the menu

You can easily test both scenarios using:

brew install ccache   # Test with ccache
brew uninstall ccache # Test without ccache

cc: @adhorodyski


Note

Adds a conditional ccache cleanup task and makes task creation async to detect availability.

  • CLI: clean plugin
    • Add ccache cleanup task (ccache --clear), enabled only when available via isCCacheInstalled (which ccache).
    • Make createCleanupTasks async and update cleanProject to await it.
    • Extend CLEANUP_TASK_NAMES with ccache.
  • Changeset
    • Patch bumps for rock and create-rock.

Written by Cursor Bugbot for commit 235188b. This will update automatically on new commits. Configure here.

@vercel
Copy link

vercel bot commented Nov 6, 2025

@mateuuszzzzz is attempting to deploy a commit to the Callstack Team on Vercel.

A member of the Team first needs to authorize it.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

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

This PR is being reviewed by Cursor Bugbot

Details

Your team is on the Bugbot Free tier. On this plan, Bugbot will review limited PRs each billing cycle for each member of your team.

To receive Bugbot reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Copy link
Collaborator

@adhorodyski adhorodyski left a comment

Choose a reason for hiding this comment

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

Only one change requested on the which command, other that that one NIT. Thanks for shipping this!

@thymikee
Copy link
Contributor

thymikee commented Nov 11, 2025

@cursoragent add changeset with patch version bump, use PR title

@cursor
Copy link

cursor bot commented Nov 11, 2025

Please finish setting up background agents. Go to Cursor

@thymikee
Copy link
Contributor

@cursoragent try again

@adhorodyski
Copy link
Collaborator

@mateuuszzzzz can I ask you to please add a changeset for this? It's clear AI won't replace us just yet 😅

@thymikee thymikee merged commit 3070360 into callstackincubator:main Nov 14, 2025
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants