feat: support extracting 7z archives for static backends

[yjoer]

This update uses file::un7z, which is currently gated behind Windows. Another known limitation of this approach is that strip_components is not supported.

[Copilot]

Pull Request Overview

This PR adds initial support for recognizing and extracting 7z archives in static backends.

• Introduces a new SevenZip variant in TarFormat and maps the "7z" extension to it.
• Updates open_tar to explicitly bail on 7z formats.
• Adds a 7z extraction branch in static_helpers.rs that invokes un7z on Windows.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/file.rs	Added SevenZip variant and parsing; updated open_tar to bail on 7z
src/backend/static_helpers.rs	Added SevenZip extraction branch invoking un7z on Windows

Comments suppressed due to low confidence (2)

src/file.rs:756

• [nitpick] Currently open_tar explicitly bails on the SevenZip variant, so 7z archives cannot be processed via this function. If extraction support is intended, consider delegating to un7z here or providing a clear alternative API.

        TarFormat::SevenZip => bail!("7z format not supported"),

src/file.rs:649

• No tests were added to verify the new SevenZip variant or the parsing logic for .7z extensions. Consider adding unit tests to ensure correct detection and handling of 7z archives.

    SevenZip,

[jdx]

I don't think it will work very well without strip_components

[yjoer]

The same implementation for strip_components from file::untar can be reused. I believe this also applies to #5631. What remains is to determine how to support should_strip_components.

[yjoer]

Perhaps I could add a branch to support 7z below zip.

[jdx]

I think @risu729 is looking into this for zip

[risu729]

I refactored it, so when #5631 is merged, you can use strip_archive_path_components.

[yjoer]

Uh-oh, I refactored in the opposite direction. Nevermind, I will refactor again when #5631 lands.

[yjoer]

This is now ready with the strip component and automatic detection.

[jdx]

Can you add an e2e test?

If needed I can publish a tarball in the mise s3 bucket

[yjoer]

I might need another 7z archive with a nested directory structure to test strip components.

[jdx]

If you create one I can put it up so you can fetch it with the http backend

[yjoer]

7z2500-extra.7z.zip

Here it is. You'll need to remove the .zip extension. I added it to trick GitHub into letting me upload the file here. This is created from 7z2500-extra.7z, where I relocate the executable to 7za/bin/7za.exe

[jdx]

https://mise.jdx.dev/test-fixtures/7z2500-extra.7z

[jdx]

huh it's in there, but I get a 404...

EDIT: was caching, works now

[jdx]

use unimplemented!, no reason not to panic in this case. it'd also be cleaner to have separate functions, e.g.:

#[cfg(not(windows))]
pub fn inspect_7z_contents(archive: &Path) -> Result<Vec<(String, bool)>> {
  unimplemented!("")
}

[yjoer]

Applied in b672276

[jdx]

bugbot run

[cursor]

Bug: 7z Extraction Error on Non-Windows

On non-Windows platforms, 7z archive extraction fails. The #[cfg(windows)] directive guards the un7z call, causing the code to fall through to generic tar extraction logic. This results in incorrect processing and confusing errors. An explicit #[cfg(not(windows))] branch is needed to return an error or unimplemented!(), consistent with inspect_7z_contents.

src/file.rs#L689-L699

mise/src/file.rs

Lines 689 to 699 in b672276

	);
	} else if format == TarFormat::SevenZip {
	#[cfg(windows)]
	return un7z(
	archive,
	dest,
	&SevenZipOptions {
	strip_components: opts.strip_components,
	},
	);
	}

Fix in Cursor • Fix in Web

---

BugBot free trial expires on July 22, 2025
Learn more in the Cursor dashboard.

Was this report helpful? Give feedback by reacting with 👍 or 👎