Skip to content

Read UDF file systems - #115

Merged
indy2kro merged 3 commits into
mainfrom
feature/udf-filesystem
Oct 6, 2026
Merged

indy2kro merged 3 commits into
mainfrom
feature/udf-filesystem

Conversation

@indy2kro

@indy2kro indy2kro commented Oct 6, 2026

Copy link
Copy Markdown
Owner
  • new FileSystem interface implemented by ISO 9660 volumes and the new UdfFileSystem (shared lookup/read helpers in the BrowsesEntries trait)
  • UDF: anchor + volume descriptor sequence (main/reserve), plain partition maps, file set descriptor, file entries and extended file entries, short/long/inline allocation descriptors, allocation extent chains (bounded), sparse extents, OSTA compressed names, timestamps
  • IsoFile::getUdfFileSystem() / getFileSystem(): UDF only images are browsable like ISO ones; Extractor and isotool (--list, --cat, --find, --json, --extract, info) work on both
  • unsupported structures (sparable/virtual/metadata partitions, other block sizes) are reported with a clear exception; corrupt entries are skipped, loops/chains are bounded
  • tests: real fixtures (UDF bridge images compared against their ISO 9660 tree, pure UDF image) and generated UDF images (9 layouts x a sample tree), hostile cases

Covers FEAT-07.

🤖 Generated with Claude Code

@codecov

codecov Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 97.62712% with 7 lines in your changes missing coverage. Please review.
✅ Project coverage is 97.21%. Comparing base (256960d) to head (1199410).
⚠️ Report is 2 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/Udf/UdfFileSystem.php 97.44% 6 Missing ⚠️
src/BrowsesEntries.php 96.29% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##               main     #115      +/-   ##
============================================
+ Coverage     96.44%   97.21%   +0.77%     
- Complexity      458      582     +124     
============================================
  Files            21       24       +3     
  Lines          1069     1329     +260     
============================================
+ Hits           1031     1292     +261     
+ Misses           38       37       -1     

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

@codacy-production

codacy-production Bot commented Oct 6, 2026 •

Copy link
Copy Markdown

Not up to standards ⛔

🔴 Issues 1 medium

Alerts:
⚠ 1 issue (≤ 0 issues of at least minor severity)

Results:
1 new issue

Category Results
UnusedCode 1 medium

View in Codacy

🟢 Metrics 273 complexity · 2 duplication

Metric Results
Complexity 273
Duplication 2

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 added this pull request to stack #119 October 6, 2026 17:35
@indy2kro
indy2kro merged commit 75401cf into main Oct 6, 2026
7 of 8 checks passed
@indy2kro
indy2kro deleted the feature/udf-filesystem 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