Replace unzip with bsdtar

There are situations where a Zip archive from Sourcegraph causes the
directory to be improperly recreated within the container. Using
libarchive-tools fixes this.

Author: lowjoel

docker/batch-change-volume-workspace/Dockerfile

@@ -4,4 +4,4 @@
 
 FROM alpine:3.15.0@sha256:21a3deaa0d32a8057914f36584b5288d2e5ecc984380bc0118285c70fa8c9300
 
-RUN apk add --update git unzip
+RUN apk add --update git libarchive-tools

internal/batches/workspace/volume_workspace.go

@@ -153,7 +153,7 @@ func (wc *dockerVolumeWorkspaceCreator) unzipRepoIntoVolume(ctx context.Context,
 		opts,
 		DockerVolumeWorkspaceImage,
 		"sh", "-c",
-		fmt.Sprintf("unzip /tmp/zip; rm /work/%s", dummy),
+		fmt.Sprintf("bsdtar -xf /tmp/zip && rm /work/%s", dummy),
 	)
 
 	if out, err := exec.CommandContext(ctx, "docker", opts...).CombinedOutput(); err != nil {

internal/batches/workspace/volume_workspace_test.go

@@ -85,7 +85,7 @@ func TestVolumeWorkspaceCreator(t *testing.T) {
 					"--user", "0:0",
 					"--mount", "type=volume,source="+volumeID+",target=/work",
 					DockerVolumeWorkspaceImage,
-					"sh", "-c", "unzip /tmp/zip; rm /work/*",
+					"sh", "-c", "bsdtar -xf /tmp/zip && rm /work/*",
 				),
 				expect.NewGlob(
 					expect.Success,
@@ -122,7 +122,7 @@ func TestVolumeWorkspaceCreator(t *testing.T) {
 					"--user", "0:0",
 					"--mount", "type=volume,source="+volumeID+",target=/work",
 					DockerVolumeWorkspaceImage,
-					"sh", "-c", "unzip /tmp/zip; rm /work/*",
+					"sh", "-c", "bsdtar -xf /tmp/zip && rm /work/*",
 				),
 				expect.NewGlob(
 					expect.Success,
@@ -163,7 +163,7 @@ func TestVolumeWorkspaceCreator(t *testing.T) {
 					"--user", "1:2",
 					"--mount", "type=volume,source="+volumeID+",target=/work",
 					DockerVolumeWorkspaceImage,
-					"sh", "-c", "unzip /tmp/zip; rm /work/*",
+					"sh", "-c", "bsdtar -xf /tmp/zip && rm /work/*",
 				),
 				expect.NewGlob(
 					expect.Success,
@@ -242,7 +242,7 @@ func TestVolumeWorkspaceCreator(t *testing.T) {
 					"--user", "0:0",
 					"--mount", "type=volume,source="+volumeID+",target=/work",
 					DockerVolumeWorkspaceImage,
-					"sh", "-c", "unzip /tmp/zip; rm /work/*",
+					"sh", "-c", "bsdtar -xf /tmp/zip && rm /work/*",
 				),
 				expect.NewGlob(
 					expect.Behaviour{ExitCode: 1},
@@ -284,7 +284,7 @@ func TestVolumeWorkspaceCreator(t *testing.T) {
 					"--user", "0:0",
 					"--mount", "type=volume,source="+volumeID+",target=/work",
 					DockerVolumeWorkspaceImage,
-					"sh", "-c", "unzip /tmp/zip; rm /work/*",
+					"sh", "-c", "bsdtar -xf /tmp/zip && rm /work/*",
 				),
 			},
 			steps: []batcheslib.Step{
@@ -318,7 +318,7 @@ func TestVolumeWorkspaceCreator(t *testing.T) {
 					"--user", "0:0",
 					"--mount", "type=volume,source="+volumeID+",target=/work",
 					DockerVolumeWorkspaceImage,
-					"sh", "-c", "unzip /tmp/zip; rm /work/*",
+					"sh", "-c", "bsdtar -xf /tmp/zip && rm /work/*",
 				),
 				expect.NewGlob(
 					expect.Success,