dockerfile-optimizer: improves PR message

samalba · samalba · commit 05135caa414d · 2025-03-11T10:06:42.000-07:00
Signed-off-by: Sam Alba &lt;216487+samalba@users.noreply.github.com&gt;
diff --git a/dockerfile-optimizer/go.mod b/dockerfile-optimizer/go.mod
@@ -26,6 +26,7 @@ require (
 
 require (
 	github.com/cenkalti/backoff/v4 v4.3.0 // indirect
+	github.com/dustin/go-humanize v1.0.1 // indirect
 	github.com/go-logr/logr v1.4.2 // indirect
 	github.com/go-logr/stdr v1.2.2 // indirect
 	github.com/google/uuid v1.6.0 // indirect
diff --git a/dockerfile-optimizer/go.sum b/dockerfile-optimizer/go.sum
@@ -8,6 +8,8 @@ github.com/cenkalti/backoff/v4 v4.3.0 h1:MyRJ/UdXutAwSAT+s3wNd7MfTIcy71VQueUuFK3
 github.com/cenkalti/backoff/v4 v4.3.0/go.mod h1:Y3VNntkOUPxTVeUxJ/G5vcM//AlwfmyYozVcomhLiZE=
 github.com/davecgh/go-spew v1.1.1 h1:vj9j/u1bqnvCEfJOwUhtlOARqs3+rkHYY13jYWTU97c=
 github.com/davecgh/go-spew v1.1.1/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
+github.com/dustin/go-humanize v1.0.1 h1:GzkhY7T5VNhEkwH0PVJgjz+fX1rhBrR7pRT3mDkpeCY=
+github.com/dustin/go-humanize v1.0.1/go.mod h1:Mu1zIs6XwVuF/gI1OepvI0qD18qycQx+mFykh5fBlto=
 github.com/go-logr/logr v1.2.2/go.mod h1:jdQByPbusPIv2/zmleS9BjJVeZ6kBagPoEUsqbVz/1A=
 github.com/go-logr/logr v1.4.2 h1:6pFjapn8bFcIbiKo3XT4j/BhANplGihG6tvd+8rYgrY=
 github.com/go-logr/logr v1.4.2/go.mod h1:9T104GzyrTigFIr8wt5mBrctHMim0Nb2HLGrmQ40KvY=
diff --git a/dockerfile-optimizer/main.go b/dockerfile-optimizer/main.go
@@ -23,6 +23,7 @@ import (
 
 	"dagger/dockerfile-optimizer/internal/dagger"
 
+	"github.com/dustin/go-humanize"
 	"github.com/google/uuid"
 )
 
@@ -181,8 +182,8 @@ func (m *DockerfileOptimizer) OptimizeDockerfile(ctx context.Context, githubToke
 	}
 
 	answer += "\n\nImage info:\n"
-	answer += fmt.Sprintf("- The original image has %d layers and is %d bytes in size.\n", originalImgInfo[0], originalImgInfo[1])
-	answer += fmt.Sprintf("- The optimized image has %d layers and is %d bytes in size.\n", lastImgInfo[0], lastImgInfo[1])
+	answer += fmt.Sprintf("- The original image has %d layers and is %s in size.\n", originalImgInfo[0], humanize.Bytes(uint64(originalImgInfo[1])))
+	answer += fmt.Sprintf("- The optimized image has %d layers and is %s in size.\n", lastImgInfo[0], humanize.Bytes(uint64(lastImgInfo[1])))
 
 	return createPR(ctx, githubToken, repoURL, lastState, dockerfile, answer)
 }

Original file line number	Diff line number	Diff line change
`@@ -23,6 +23,7 @@ import (`
`23`	`23`
`24`	`24`	`"dagger/dockerfile-optimizer/internal/dagger"`
`25`	`25`
	`26`	`+ "github.com/dustin/go-humanize"`
`26`	`27`	`"github.com/google/uuid"`
`27`	`28`	`)`
`28`	`29`
`@@ -181,8 +182,8 @@ func (m *DockerfileOptimizer) OptimizeDockerfile(ctx context.Context, githubToke`
`181`	`182`	`}`
`182`	`183`
`183`	`184`	`answer += "\n\nImage info:\n"`
`184`		`- answer += fmt.Sprintf("- The original image has %d layers and is %d bytes in size.\n", originalImgInfo[0], originalImgInfo[1])`
`185`		`- answer += fmt.Sprintf("- The optimized image has %d layers and is %d bytes in size.\n", lastImgInfo[0], lastImgInfo[1])`
	`185`	`+ answer += fmt.Sprintf("- The original image has %d layers and is %s in size.\n", originalImgInfo[0], humanize.Bytes(uint64(originalImgInfo[1])))`
	`186`	`+ answer += fmt.Sprintf("- The optimized image has %d layers and is %s in size.\n", lastImgInfo[0], humanize.Bytes(uint64(lastImgInfo[1])))`
`186`	`187`
`187`	`188`	`return createPR(ctx, githubToken, repoURL, lastState, dockerfile, answer)`
`188`	`189`	`}`