Finish improvements (probably)

.github/workflows/macos-latest.yml

@@ -8,7 +8,7 @@ jobs:
 
     strategy:
       matrix:
-        go-version: [1.22]
+        go-version: [1.23]
     runs-on: macos-latest
     steps:
     - name: Install Go

.github/workflows/ubuntu-latest.yml

@@ -8,7 +8,7 @@ jobs:
 
     strategy:
       matrix:
-        go-version: [1.22]
+        go-version: [1.23]
     runs-on: ubuntu-latest
     steps:
     - name: Install Go

.github/workflows/windows-latest.yml

@@ -8,7 +8,7 @@ jobs:
 
     strategy:
       matrix:
-        go-version: [1.22]
+        go-version: [1.23]
     runs-on: windows-latest
     steps:
     - name: Install Go

7z.go

@@ -31,13 +31,13 @@ type SevenZip struct {
 	Password string
 }
 
-func (z SevenZip) Name() string { return ".7z" }
+func (z SevenZip) Extension() string { return ".7z" }
 
-func (z SevenZip) Match(filename string, stream io.Reader) (MatchResult, error) {
+func (z SevenZip) Match(_ context.Context, filename string, stream io.Reader) (MatchResult, error) {
 	var mr MatchResult
 
 	// match filename
-	if strings.Contains(strings.ToLower(filename), z.Name()) {
+	if strings.Contains(strings.ToLower(filename), z.Extension()) {
 		mr.ByName = true
 	}
 
@@ -104,7 +104,7 @@ func (z SevenZip) Extract(ctx context.Context, sourceArchive io.Reader, pathsInA
 				if err != nil {
 					return nil, err
 				}
-				return archivedFile{openedFile, fi}, nil
+				return fileInArchive{openedFile, fi}, nil
 			},
 		}
 

README.md

@@ -1,6 +1,6 @@
 # archiver [![Go Reference](https://pkg.go.dev/badge/github.com/mholt/archiver/v4.svg)](https://pkg.go.dev/github.com/mholt/archiver/v4) [![Ubuntu-latest](https://github.com/mholt/archiver/actions/workflows/ubuntu-latest.yml/badge.svg)](https://github.com/mholt/archiver/actions/workflows/ubuntu-latest.yml) [![Macos-latest](https://github.com/mholt/archiver/actions/workflows/macos-latest.yml/badge.svg)](https://github.com/mholt/archiver/actions/workflows/macos-latest.yml) [![Windows-latest](https://github.com/mholt/archiver/actions/workflows/windows-latest.yml/badge.svg)](https://github.com/mholt/archiver/actions/workflows/windows-latest.yml)
 
-Introducing **Archiver 4.0** - a cross-platform, multi-format archive utility and Go library. A powerful and flexible library meets an elegant CLI in this generic replacement for several platform-specific or format-specific archive utilities.
+Introducing **Archiver 4.0 (alpha)** - a cross-platform, multi-format archive utility and Go library. A powerful and flexible library meets an elegant CLI in this generic replacement for several platform-specific or format-specific archive utilities.
 
 **:warning: v4 is in ALPHA. The core library APIs work pretty well but the command has not been implemented yet, nor have most automated tests. If you need the `arc` command, stick with v3 for now.**
 
@@ -11,8 +11,8 @@ Introducing **Archiver 4.0** - a cross-platform, multi-format archive utility an
 	- By file name
 	- By header
 - Traverse directories, archive files, and any other file uniformly as [`io/fs`](https://pkg.go.dev/io/fs) file systems:
-	- [`DirFS`](https://pkg.go.dev/github.com/mholt/archiver/v4#DirFS)
 	- [`FileFS`](https://pkg.go.dev/github.com/mholt/archiver/v4#FileFS)
+	- [`DirFS`](https://pkg.go.dev/github.com/mholt/archiver/v4#DirFS)
 	- [`ArchiveFS`](https://pkg.go.dev/github.com/mholt/archiver/v4#ArchiveFS)
 - Compress and decompress files
 - Create and extract archive files
@@ -117,7 +117,7 @@ If you want all the files, pass in a nil list of file paths.
 
 ```go
 // the type that will be used to read the input stream
-format := archiver.Zip{}
+var format archiver.Zip
 
 // the list of files we want out of the archive; any
 // directories will include all their contents unless
@@ -141,7 +141,7 @@ if err != nil {
 Have an input stream with unknown contents? No problem, archiver can identify it for you. It will try matching based on filename and/or the header (which peeks at the stream):
 
 ```go
-format, input, err := archiver.Identify("filename.tar.zst", input)
+format, input, err := archiver.Identify(ctx, "filename.tar.zst", input)
 if err != nil {
 	return err
 }
@@ -165,7 +165,7 @@ if decom, ok := format.(archiver.Decompressor); ok {
 }
 ```
 
-`Identify()` works by reading an arbitrary number of bytes from the beginning of the stream (just enough to check for file headers). It buffers them and returns a new reader that lets you re-read them anew.
+`Identify()` works by reading an arbitrary number of bytes from the beginning of the stream (just enough to check for file headers). It buffers them and returns a new reader that lets you re-read them anew. If your input stream is `io.Seeker` however, no buffer is created (it uses `Seek()` instead).
 
 ### Virtual file systems
 
@@ -212,7 +212,7 @@ if dir, ok := f.(fs.ReadDirFile); ok {
 		return err
 	}
 	for _, e := range entries {
-		fmt.Println(e.Name())
+		fmt.Println(e.Extension())
 	}
 }
 ```
@@ -225,7 +225,7 @@ if err != nil {
 	return err
 }
 for _, e := range entries {
-	fmt.Println(e.Name())
+	fmt.Println(e.Extension())
 }
 ```
 
@@ -247,6 +247,8 @@ if err != nil {
 }
 ```
 
+**Important .tar note:** Tar files do not efficiently implement file system semantics due to their roots in sequential-access design for tapes. File systems inherently assume random access, but tar files need to be read from the beginning to access something at the end. This is especially slow when the archive is compressed. Optimizations have been implemented to amortize `ReadDir()` calls so that `fs.WalkDir()` only has to scan the archive once, but they use more memory. Open calls require another scan to find the file. It may be more efficient to use `Tar.Extract()` directly if file system semantics are not important to you.
+
 #### Use with `http.FileServer`
 
 It can be used with http.FileServer to browse archives and directories in a browser. However, due to how http.FileServer works, don't directly use http.FileServer with compressed files; instead wrap it like following:

archiver.go

@@ -219,7 +219,7 @@ type FromDiskOptions struct {
 // memory, and skipping lots of directories may run up your memory bill.
 //
 // Any other returned error will terminate a walk and be returned to the caller.
-type FileHandler func(ctx context.Context, f FileInfo) error
+type FileHandler func(ctx context.Context, info FileInfo) error
 
 // openAndCopyFile opens file for reading, copies its
 // contents to w, then closes file.

archiver_test.go

@@ -245,7 +245,8 @@ func TestNameOnDiskToNameInArchive(t *testing.T) {
 		},
 	} {
 		if !strings.HasPrefix(tc.nameOnDisk, tc.rootOnDisk) {
-			t.Fatalf("Test %d: Invalid test case! Filename (on disk) will have rootOnDisk as a prefix according to the fs.WalkDirFunc godoc.", i)
+			t.Errorf("Test %d: Invalid test case! Filename (on disk) will have rootOnDisk as a prefix according to the fs.WalkDirFunc godoc.", i)
+			continue
 		}
 		if tc.windows && runtime.GOOS != "windows" {
 			t.Logf("Test %d: Skipping test that is only compatible with Windows", i)

brotli.go

@@ -1,6 +1,7 @@
 package archiver
 
 import (
+	"context"
 	"io"
 	"strings"
 
@@ -16,19 +17,25 @@ type Brotli struct {
 	Quality int
 }
 
-func (Brotli) Name() string { return ".br" }
+func (Brotli) Extension() string { return ".br" }
 
-func (br Brotli) Match(filename string, stream io.Reader) (MatchResult, error) {
+func (br Brotli) Match(_ context.Context, filename string, stream io.Reader) (MatchResult, error) {
 	var mr MatchResult
 
 	// match filename
-	if strings.Contains(strings.ToLower(filename), br.Name()) {
+	if strings.Contains(strings.ToLower(filename), br.Extension()) {
 		mr.ByName = true
 	}
 
-	// brotli does not have well-defined file headers; the
-	// best way to match the stream would be to try decoding
-	// part of it, and this is not implemented for now
+	// brotli does not have well-defined file headers or a magic number;
+	// the best way to match the stream is probably to try decoding part
+	// of it, but we'll just have to guess a large-enough size that is
+	// still small enough for the smallest streams we'll encounter
+	r := brotli.NewReader(stream)
+	buf := make([]byte, 16)
+	if _, err := io.ReadFull(r, buf); err == nil {
+		mr.ByStream = true
+	}
 
 	return mr, nil
 }

bz2.go

@@ -2,6 +2,7 @@ package archiver
 
 import (
 	"bytes"
+	"context"
 	"io"
 	"strings"
 
@@ -17,13 +18,13 @@ type Bz2 struct {
 	CompressionLevel int
 }
 
-func (Bz2) Name() string { return ".bz2" }
+func (Bz2) Extension() string { return ".bz2" }
 
-func (bz Bz2) Match(filename string, stream io.Reader) (MatchResult, error) {
+func (bz Bz2) Match(_ context.Context, filename string, stream io.Reader) (MatchResult, error) {
 	var mr MatchResult
 
 	// match filename
-	if strings.Contains(strings.ToLower(filename), bz.Name()) {
+	if strings.Contains(strings.ToLower(filename), bz.Extension()) {
 		mr.ByName = true
 	}