fix withDescription for mapped commands (#397)

.changeset/breezy-berries-think.md

@@ -0,0 +1,5 @@
+---
+"@effect/cli": patch
+---
+
+fix withDescription for mapped commands

src/internal/command.ts

@@ -271,7 +271,7 @@ const mapDescriptor = dual<
     self: Command.Command<Name, R, E, A>,
     f: (_: Descriptor.Command<A>) => Descriptor.Command<A>
   ) => Command.Command<Name, R, E, A>
->(2, (self, f) => makeProto(f(getDescriptor(self)), self.handler, self.tag))
+>(2, (self, f) => makeProto(f(self.descriptor), self.handler, self.tag))
 
 /** @internal */
 export const prompt = <Name extends string, A, R, E>(

src/internal/commandDescriptor.ts

@@ -858,7 +858,7 @@ const withDescriptionInternal = (
       return op
     }
     case "Map": {
-      return map(withDescriptionInternal(self.command as Instruction, description), self.f)
+      return mapOrFail(withDescriptionInternal(self.command as Instruction, description), self.f)
     }
     case "OrElse": {
       // TODO: if both the left and right commands also have help defined, that

test/Command.test.ts

@@ -5,7 +5,7 @@ import { assert, describe, test } from "vitest"
 
 const git = Command.make("git", {
   verbose: Options.boolean("verbose").pipe(Options.withAlias("v"))
-})
+}).pipe(Command.withDescription("the stupid content tracker"))
 
 const clone = Command.make("clone", {
   repository: Args.text({ name: "repository" })
@@ -18,7 +18,7 @@ const clone = Command.make("clone", {
     } else {
       yield* _(log(`Cloning`))
     }
-  }))
+  })).pipe(Command.withDescription("Clone a repository into a new directory"))
 
 const add = Command.make("add", {
   pathspec: Args.text({ name: "pathspec" })
@@ -31,7 +31,7 @@ const add = Command.make("add", {
     } else {
       yield* _(log(`Adding`))
     }
-  }))
+  })).pipe(Command.withDescription("Add file contents to the index"))
 
 const run = git.pipe(
   Command.withSubcommands([clone, add]),