Skip to content

add 7z handler#552

Open
neko782 wants to merge 2 commits intop2r3:masterfrom
neko782:7z
Open

add 7z handler#552
neko782 wants to merge 2 commits intop2r3:masterfrom
neko782:7z

Conversation

@neko782
Copy link
Contributor

@neko782 neko782 commented Mar 16, 2026

added 7z handler using 7z-wasm, adds support for a ton of archive formats

@p2r3
Copy link
Owner

p2r3 commented Mar 17, 2026

Does this not obsolete any of the other archive handlers?

@neko782
Copy link
Contributor Author

neko782 commented Mar 17, 2026

Does this not obsolete any of the other archive handlers?

yeah kind of

  • fully replaces jszip
  • can potentially replace tar (if the folder weirdness can be fixed)
  • can potentially replace tarcompressed (needs specific handling/a seperate handler)
  • cant replace lzh (7z cant create lzh archives)

need me to work removing any of those?

@p2r3
Copy link
Owner

p2r3 commented Mar 19, 2026

need me to work removing any of those?

Would be nice. The less bloat the better, the project is already hard to navigate as-is.

@omega-fallon
Copy link
Contributor

We could also close issue #511 with this by adding .cbr support, and the .cb7 format which nobody was asking for but would complete the set.

@neko782
Copy link
Contributor Author

neko782 commented Mar 19, 2026

Would be nice. The less bloat the better, the project is already hard to navigate as-is.

ill try to get as much as i can tomorrow

@neko782
Copy link
Contributor Author

neko782 commented Mar 19, 2026

@p2r3 ready for review

@omega-fallon sorry you might have to implement the comic stuff again

this.supportedFormats = [
...prioritized,
...this.supportedFormats.filter(f => !priority.includes(f.internal))
];
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

just for some better routing

@omega-fallon
Copy link
Contributor

omega-fallon commented Mar 20, 2026

@p2r3 ready for review

@omega-fallon sorry you might have to implement the comic stuff again

Fine by me. Once this PR goes through I can make it work.

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