fix(cli): commit built webapp/dist so go:embed build passes in CI

A stray `pkg/cli/webapp/dist/` line in .gitignore re-ignored the directory
after the negations meant to keep it tracked (last-match-wins), so nothing
under dist was committed. CI's `go build ./cmd/captain` then failed with
`pattern all:webapp/dist: no matching files found`, which also broke the
test and lint jobs' compilation.

The webapp can't be built in CI or the goreleaser release job (its vite
build uses a local clicky-ui link: dependency), so the built dist must be
committed and embedded. Remove the contradictory ignore line, commit the
production dist, and correct serve.go's stale comment.

Author: moshloop

.gitignore

@@ -20,5 +20,4 @@ pkg/cli/webapp/*.tsbuildinfo
 !pkg/cli/webapp/dist/
 !pkg/cli/webapp/dist/**
 .grite/
-pkg/cli/webapp/dist/
 hack/

pkg/cli/serve.go

@@ -26,9 +26,11 @@ import (
 	"github.com/spf13/cobra"
 )
 
-// all: keeps the committed dist/.gitkeep placeholder in the embed so the binary
-// compiles without a built webapp (dist is gitignored and built on demand or in
-// the release workflow). When index.html is absent, serve.go reports it at runtime.
+// The built webapp is committed under webapp/dist (see .gitignore) because the
+// vite build depends on a local clicky-ui link: dependency that is unavailable
+// in CI and in the goreleaser release job, so the binary embeds the checked-in
+// dist rather than building it. all: ensures dotfiles are embedded too. When
+// index.html is absent, serve.go reports it at runtime.
 //
 //go:embed all:webapp/dist
 var captainWebappFS embed.FS