don't override existing hooks

filiptronicek · filiptronicek · commit ec1be4391fc1 · 2025-01-21T12:57:45.000Z
diff --git a/components/supervisor/pkg/supervisor/supervisor.go b/components/supervisor/pkg/supervisor/supervisor.go
@@ -847,6 +847,10 @@ func setupGitMessageHook(path string) error {
 	}
 
 	fn := filepath.Join(path, "prepare-commit-msg")
+	// do not override existing hooks. Relevant for workspaces based off of prebuilds, which might already have a hook.
+	if _, err := os.Stat(fn); err == nil {
+		return nil
+	}
 	if err := os.WriteFile(fn, []byte(hookContent), 0755); err != nil {
 		return err
 	}

Original file line number	Diff line number	Diff line change
`@@ -847,6 +847,10 @@ func setupGitMessageHook(path string) error {`
`847`	`847`	`}`
`848`	`848`
`849`	`849`	`fn := filepath.Join(path, "prepare-commit-msg")`
	`850`	`+ // do not override existing hooks. Relevant for workspaces based off of prebuilds, which might already have a hook.`
	`851`	`+ if _, err := os.Stat(fn); err == nil {`
	`852`	`+ return nil`
	`853`	`+ }`
`850`	`854`	`if err := os.WriteFile(fn, []byte(hookContent), 0755); err != nil {`
`851`	`855`	`return err`
`852`	`856`	`}`