Skip to content

Update sdk to 3.17.2#16

Merged
marchuk-vlad merged 3 commits intomainfrom
hotfix/sdk-3-17-2
Feb 9, 2026
Merged

Update sdk to 3.17.2#16
marchuk-vlad merged 3 commits intomainfrom
hotfix/sdk-3-17-2

Conversation

@marchuk-vlad
Copy link
Contributor

@marchuk-vlad marchuk-vlad commented Feb 9, 2026

Summary by CodeRabbit

  • Chores

    • Updated application version to 0.14.6
  • Improvements

    • Enhanced file download functionality with improved path handling and streamlined directory operations
    • Added diagnostic output for download operations to provide greater visibility during the process

@marchuk-vlad marchuk-vlad self-assigned this Feb 9, 2026
@coderabbitai
Copy link

coderabbitai bot commented Feb 9, 2026

Walkthrough

This pull request bumps the package version from 0.14.4 to 0.14.6 and refactors the file download command to use path-based assembly instead of filesystem operations, removing directory pre-creation logic while adding a diagnostic print statement.

Changes

Cohort / File(s) Summary
Version Update
package.json
Version field updated from 0.14.4 to 0.14.6.
File Download Refactor
src/commands/filesDownload.addon.ts
Removes filesystem operations (fs.stat/fs.mkdir); replaces with path.join for directory assembly when resource.filepath exists. Adds path module import and diagnostic logging. Directory existence check and pre-creation are removed; resource type validation remains unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 A version hops from point-four to point-six with cheer,
Files download now via paths so clear,
No filesystem checks to make delay,
Just join and print, then download the way!
A cleaner flow, so swift and neat,
This refactored dance is quite the treat! 🎉

🚥 Pre-merge checks | ✅ 2 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Title check ⚠️ Warning The title references an SDK version update (3.17.2), but the actual changes show only a package.json version bump (0.14.4 to 0.14.6) and modifications to file download logic. Update the title to accurately reflect the main changes, such as 'Bump package version to 0.14.6 and refactor file download path handling' or similar that describes the actual modifications made.
✅ Passed checks (2 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

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

✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch hotfix/sdk-3-17-2

No actionable comments were generated in the recent review. 🎉


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.

@marchuk-vlad marchuk-vlad merged commit 45b3e46 into main Feb 9, 2026
5 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