[FR]: Should `output_dir` use `output` as the dir name rather than `name?

### What is the current behavior?

It seems that if `output_dir` is true, a directory artifact is declared using the rule name as the folder name.

https://github.com/aspect-build/rules_esbuild/blob/cede902c94e2313a3ea183b6a5e1dbbbfcf40347/esbuild/private/esbuild.bzl#L255-L257

### Describe the feature

I was surprised that when I defined `output = "dist"` with `output_dir = True`, that the rule would fail with “The following files have no generating action: .../dist". After digging into the code, it seems that `name` is used for the folder name. Is that intentional?

Read esbuild/private/esbuild.bzl around lines 255-257 and reproduce the case with output="dist" and output_dir=True. Done means the declared directory artifact resolves to dist rather than the rule name, without the reported missing-generating-action error.

Written by the indexing model from the issue text.