Completion Command Always Returns Error

VERSION

@@ -1 +1 @@
-0.8.0
+0.8.1

internal/cmd/completion.go

@@ -24,20 +24,24 @@ func Completion() *cobra.Command {
 func completionRunE(c *cobra.Command, args []string) error {
  switch args[0] {
  case "bash":
- err := Root().GenBashCompletion(os.Stdout)
- return fmt.Errorf("failed to generate bash completion: %w", err)
+ if err := Root().GenBashCompletion(os.Stdout); err != nil {
+ return fmt.Errorf("failed to generate bash completion: %w", err)
+ }
  case "fish":
- err := Root().GenFishCompletion(os.Stdout, true)
- return fmt.Errorf("failed to generate fish completion: %w", err)
+ if err := Root().GenFishCompletion(os.Stdout, true); err != nil {
+ return fmt.Errorf("failed to generate fish completion: %w", err)
+ }
  case "powershell":
- err := Root().GenPowerShellCompletionWithDesc(os.Stdout)
- return fmt.Errorf("failed to generate powershell completion: %w", err)
+ if err := Root().GenPowerShellCompletionWithDesc(os.Stdout); err != nil {
+ return fmt.Errorf("failed to generate powershell completion: %w", err)
+ }
  case "zsh":
- err := Root().GenZshCompletion(os.Stdout)
- return fmt.Errorf("failed to generate zsh completion: %w", err)
- default:
- return nil
+ if err := Root().GenZshCompletion(os.Stdout); err != nil {
+ return fmt.Errorf("failed to generate zsh completion: %w", err)
+ }
  }
+
+ return nil
 }
 
 func completionLong(rootName string) string {

internal/note/note.go

@@ -50,7 +50,6 @@ func NewNote(noteSchema string) (Note, error) {
  }
 
  if errs := validate.NewFilenameValidator().Validate(note.Filename); errs != nil {
- log.Info().Msgf("creating note: %s", filename)
  return Note{}, fmt.Errorf("note filename validation failed: %v", errs)
  }
 

internal/validate/filename.go

@@ -21,11 +21,11 @@ func NewFilenameValidator() filenameValidator { //nolint:revive
 
 // Validate implements the validate.Validator.Valid() interface. It carries out
 // a number of checks:
-// - If the file extension is correct
-// - Has a valid base schema (e.g. "area" from "area.language.go.error")
-// - If the filename matches a combined kebab-case dot notation
-// (e.g. "area.languagego.error-handling.md")
-// - If the filename has the required number of parts (e.g. not "area.md").
+//  - If the file extension is correct
+//  - Has a valid base schema (e.g. "area" from "area.language.go.error")
+//  - If the filename matches a combined kebab-case dot notation
+//  (e.g. "area.languagego.error-handling.md")
+//  - If the filename has the required number of parts (e.g. not "area.md").
 func (f filenameValidator) Validate(filename string) []error {
  var errs []error