Skip to content

feat: Command Line SDK update for version 14.0.0#274

Closed
premtsd-code wants to merge 1 commit intomasterfrom
dev
Closed

feat: Command Line SDK update for version 14.0.0#274
premtsd-code wants to merge 1 commit intomasterfrom
dev

Conversation

@premtsd-code
Copy link
Contributor

@premtsd-code premtsd-code commented Mar 5, 2026

This PR contains updates to the Command Line SDK for version 14.0.0.

Summary by CodeRabbit

  • New Features

    • Added encryption support for text-based attributes during creation.
  • Improvements

    • Enhanced installation detection to recognize Node Version Manager (NVM) paths for npm-based installations.
    • Updated core dependencies for improved stability and compatibility.

@coderabbitai
Copy link

coderabbitai bot commented Mar 5, 2026

Caution

Review failed

Pull request was closed or merged during review

Walkthrough

This pull request upgrades the tar library from version 6.1.11 to 7.4.3 and refactors related code. The tar library usage is updated from default imports to named imports (create and extract) in two files. The isInstalledViaNpm check is expanded to recognize NVM installation paths. Encryption support is added to text-based attribute types by propagating the encrypt flag through to database service methods. The corresponding TypeScript types package is also updated.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and overly broad, using generic phrasing 'Command Line SDK update for version 14.0.0' that doesn't convey the specific changes made. Consider using a more descriptive title that highlights the primary changes, such as 'feat: Update tar dependency and add encryption support for string attributes' or similar that better reflects the actual modifications.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

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

✨ Finishing Touches
  • 📝 Generate docstrings (stacked PR)
  • 📝 Generate docstrings (commit on current branch)
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch dev

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.

@premtsd-code premtsd-code deleted the dev branch March 5, 2026 09:45
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.

1 participant