Skip to content

create tool params and upgrade to v0.5#2

Merged
steven-passynkov merged 2 commits into
mainfrom
feat/createToolParams
Apr 15, 2026
Merged

create tool params and upgrade to v0.5#2
steven-passynkov merged 2 commits into
mainfrom
feat/createToolParams

Conversation

@ipassynk
Copy link
Copy Markdown
Collaborator

@ipassynk ipassynk commented Apr 15, 2026

create tool params and upgrade to v0.5

Summary by CodeRabbit

  • New Features

    • Sandbox creation now supports configurable memory (512–8192 MiB) and CPU (1–8 vCPU) resources.
    • File listing path parameter is now optional, defaulting to workspace directory.
  • Chores

    • Updated Leap0 dependency to ^0.5.0 for enhanced runtime capabilities.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 15, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: c468dba4-8cd2-4e6e-9f21-9355a17daa71

📥 Commits

Reviewing files that changed from the base of the PR and between 9d14c58 and d46fb3b.

📒 Files selected for processing (3)
  • package.json
  • src/mastra/tools/leap0/tools.ts
  • src/mastra/tools/leap0/utils.ts

📝 Walkthrough

Walkthrough

This pull request upgrades the Leap0 dependency from ^0.3.1 to ^0.5.0 and refactors the Leap0 sandbox integration to support workdir-aware path normalization, configurable resource limits (memory and vCPU), and improved sandbox lifecycle management through idle-timeout handling.

Changes

Cohort / File(s) Summary
Dependency Update
package.json
Bumped leap0 dependency from ^0.3.1 to ^0.5.0 to leverage new API capabilities.
Leap0 Sandbox Tools & Utilities
src/mastra/tools/leap0/tools.ts, src/mastra/tools/leap0/utils.ts
Refactored sandbox integration to support workdir-aware path resolution via new getSandboxWithWorkdir() and getSandboxAndNormalizedListPath() utilities; updated normalizeSandboxPath() signature to require explicit workspaceRoot parameter; added optional memoryMiB and vcpu resource limit inputs to createSandbox tool; converted sandbox idle-timeout handling from minutes to seconds; made path parameter optional in listFiles tool; updated delete operations to use object-form API and refactored filesystem calls to resolve sandboxes with their configured workdirs.

Estimated code review effort

🎯 4 (Complex) | ⏱️ ~45 minutes

Possibly related PRs

Based on the provided information, no additional context was available to identify other related pull requests beyond this one.


🐰 A sandbox sprouts, workdirs take root,
Resource limits planted, paths resolving true,
Timeouts tick in seconds, not minutes gone astray,
Leap0 hops forward—refactored today!

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. Write docstrings for the functions missing them to satisfy the coverage threshold.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title 'create tool params and upgrade to v0.5' accurately captures the main changes: adding tool parameters (memoryMiB, vcpu) to createSandbox and upgrading the leap0 dependency from ^0.3.1 to ^0.5.0.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/createToolParams

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@steven-passynkov steven-passynkov merged commit a83c83b into main Apr 15, 2026
1 check 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.

2 participants