Skip to content

Read images from streams (stdin, pipes) - #117

Merged
indy2kro merged 1 commit into
mainfrom
feature/stream-input
Oct 6, 2026
Merged

indy2kro merged 1 commit into
mainfrom
feature/stream-input

Conversation

@indy2kro

@indy2kro indy2kro commented Oct 6, 2026

Copy link
Copy Markdown
Owner

Stacked on #115 (merge that one first, this PR then retargets main).

  • IsoFile::fromStream(): copies a non-seekable stream to a temporary file (removed on destruct, size limit, cleanup on failure)
  • isotool -f - reads the image from the standard input (cat image.iso | isotool -f - --list)
  • tests for ISO and UDF streams, limit, cleanup, CLI

Covers FEAT-08.

🤖 Generated with Claude Code

@codacy-production

Copy link
Copy Markdown

Up to standards ✅

🟢 Issues 0 issues

Results:
0 new issues

View in Codacy

🟢 Metrics 22 complexity · 0 duplication

Metric Results
Complexity 22
Duplication 0

View in Codacy

NEW Get contextual insights on your PRs based on Codacy's metrics, along with PR and Jira context, without leaving GitHub. Enable AI reviewer
TIP This summary will be updated as you push new changes.

@indy2kro
indy2kro force-pushed the feature/stream-input branch from 6404331 to 705a664 Compare October 6, 2026 17:29
@indy2kro
indy2kro added this pull request to stack #119 October 6, 2026 17:35
Base automatically changed from feature/udf-filesystem to main October 6, 2026 17:35
@indy2kro
indy2kro force-pushed the feature/stream-input branch from 705a664 to 265faf5 Compare October 6, 2026 17:35
@codecov

codecov Bot commented Oct 6, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 90.90909% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.04%. Comparing base (75401cf) to head (265faf5).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/IsoFile.php 85.71% 3 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #117      +/-   ##
============================================
- Coverage     97.21%   97.04%   -0.17%     
- Complexity      582      595      +13     
============================================
  Files            24       24              
  Lines          1329     1355      +26     
============================================
+ Hits           1292     1315      +23     
- Misses           37       40       +3     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

@indy2kro
indy2kro merged commit b64673f into main Oct 6, 2026
8 checks passed
@indy2kro
indy2kro deleted the feature/stream-input branch October 7, 2026 19:31
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