Skip to content

feat/SP-4273/recursive-decompress-skipped-reporting - #146

Merged
isasmendiagus merged 2 commits into
mainfrom
feat/SP-4273/recursive-decompress-skipped-reporting
Apr 23, 2026
Merged

feat/SP-4273/recursive-decompress-skipped-reporting#146
isasmendiagus merged 2 commits into
mainfrom
feat/SP-4273/recursive-decompress-skipped-reporting

Conversation

@isasmendiagus

@isasmendiagus isasmendiagus commented Apr 23, 2026

Copy link
Copy Markdown
Collaborator

Summary by CodeRabbit

  • New Features

    • Version 0.39.0: Decompression now tracks and reports archives skipped when the configured depth limit is reached.
  • Breaking Changes

    • Decompression recursive method return format updated; callers must adapt code to handle the new result structure containing failed files and skipped archives information.

@coderabbitai

coderabbitai Bot commented Apr 23, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: f571863a-80f2-4bf5-aadb-e62202251337

📥 Commits

Reviewing files that changed from the base of the PR and between 029621d and a45dbd3.

📒 Files selected for processing (3)
  • CHANGELOG.md
  • package.json
  • src/sdk/Decompress/DecompressionManager.ts

📝 Walkthrough

Walkthrough

This release (0.39.0) updates the decompression manager API to track archives skipped due to depth limits. Both decompress() and decompressRecursive() return types are modified to include skippedByDepth, with decompressRecursive() changing from returning an array to returning an object with failedFiles and skippedByDepth properties.

Changes

Cohort / File(s) Summary
Version & Documentation
CHANGELOG.md, package.json
Version bumped to 0.39.0 and release notes added documenting API contract updates for decompression manager.
Decompression Manager Implementation
src/sdk/Decompress/DecompressionManager.ts
Updated decompress() and decompressRecursive() to track and return skippedByDepth array alongside failedFiles. Breaking change: decompressRecursive() return shape restructured from array to object with failedFiles and skippedByDepth properties.

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

enhancement

Suggested reviewers

  • agustingroh

Poem

🐰 A rabbit hops through depth-limited archives,
Now tracking paths where decompress thrives!
Return shapes change, but logic holds tight,
Skipped files reported—transparency in sight! ✨

✨ 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/SP-4273/recursive-decompress-skipped-reporting

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.

@isasmendiagus
isasmendiagus merged commit 63628a4 into main Apr 23, 2026
1 check was pending
@isasmendiagus
isasmendiagus deleted the feat/SP-4273/recursive-decompress-skipped-reporting branch April 23, 2026 14:43
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