feat(cli): add dimos cache clean#3178
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Codecov Report❌ Patch coverage is
@@ Coverage Diff @@
## main #3178 +/- ##
==========================================
- Coverage 74.17% 73.78% -0.39%
==========================================
Files 1108 1113 +5
Lines 104052 104660 +608
Branches 9530 9595 +65
==========================================
+ Hits 77179 77223 +44
- Misses 24129 24741 +612
+ Partials 2744 2696 -48
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 35 files with indirect coverage changes 🚀 New features to boost your workflow:
|
Greptile SummaryAdds a coordinated cache-cleaning command and centralizes regenerable assets beneath the platform cache directory.
Confidence Score: 5/5The PR appears safe to merge, with no blocking failures remaining in the fixes related to the previous review threads. The cleanup gate and per-run usage marker now mutually exclude cache deletion and run startup throughout the relevant lifecycle. Important Files Changed
Sequence DiagramsequenceDiagram
participant Run as dimos run
participant Gate as Cleanup gate lock
participant Usage as Usage marker lock
participant Clean as dimos cache clean
participant Cache as CACHE_DIR
Run->>Gate: acquire
Run->>Usage: create and acquire marker
Run-->>Gate: release
Run->>Run: build and execute blueprint
Clean->>Gate: wait to acquire
Clean->>Usage: test existing markers
Usage-->>Clean: locked / cache in use
Clean-->>Clean: refuse cleanup
Run-->>Usage: release and remove marker
Clean->>Gate: acquire
Clean->>Usage: verify no active markers
Clean->>Cache: remove cache hierarchy
Clean-->>Gate: release
Reviews (12): Last reviewed commit: "fix(cache): synchronize cleanup with run..." | Re-trigger Greptile |
|
Preview deployment for your docs. Learn more about Mintlify Previews.
💡 Tip: Enable Workflows to automatically generate PRs for you. |
Contribution path
Problem
DimOS writes regenerable assets to several unrelated platform, state, and temporary directories, but users have no single command to clear them. This becomes more important with #2505, which adds cached robot source checkouts and derived URDFs.
Solution
Move all DimOS-owned caches beneath the platform-specific
CACHE_DIR, then adddimos cache cleanto remove that hierarchy.The command:
--forceis supplied;y/Nconfirmation by default;--yes, independently of--force;How to Test
AI assistance
OpenAI Codex with GPT-5 implemented and tested the change under the author's direction.
Checklist