I've been extremely impressed by and interested in but (great work @matthewmueller!!) and I wanted to begin contributing to an interesting bug that I saw in the issues. So I cloned bud and was following the instructions in Contributing.md, but I ran into an error when running make install.

OS: Windows 10 Go Version: 1.18.2 Node.js Version: 16.14.0

When I did 'bud create ff-bud', I got this error: | rename /tmp/bud-create-3425471412 ff-bud: invalid cross-device link Possibly because /tmp is different partition than the current directory. Rename will work only in same partition?

Windows support is currently blocked by:

• https://github.com/rogchap/v8go/pull/234

Bud's compiler will also transition to using Go plugins, which also do not have Windows support yet:

• https://github.com/golang/go/issues/19282 Though we could use something like go-plugin for Windows.

I haven't tested it yet, but you should be able to use Bud with the Windows Subsystem for Linux (WSL).

If you get this working, please let us know how!

| listen tcp 127.0.0.1:3000: bind: address already in use

For example, running on 127.0.0.1:3001 | listen tcp 127.0.0.1:3001: bind: address already in use 127.0.0.1:3002

This addresses #149.

Appends the suffix "action" to all actions and the suffix "controller" to all nested controllers.

I'm sure there's downsides to this approach, so I'm interested in your thoughts. I was able to verify that this solves the problem listed in the issue, both at the root and at other nested levels.

Tests fail, as assertions expect certain names to exist. I can look at that if you agree with this suffix approach.

run create bud_app

| create: Unable to infer a module name. Try again using the module name.

For example, bud create --module=github.com/my/app bud_app

I'm attempting to implement persistence. So I create a new folder models. Write a struct. Update the controllers to use that new struct instead of the autogenerated one. Then run bud run. But bud errors out with

bud/.app/controller/controller.go:10:2: imported and not used: "github/donovanrost/bud-test/models"

I don't think that bud needs to do any code generation with this package. Is there a way to tell bud to ignore it?

Hi, Firstly this project looks great. Really looking forward to working with it.

Getting the error...

bud/.app/controller/controller.go:133:15: too many return values have (nil, error)

Looks the loadController() is missing a return error definination

func loadController() *controller.Controller { controllerController, err := controller.Load() if err != nil { return nil, err } return controllerController }

Hello, After I bud build the project, I remove the *.go and go.mod source files. run ./bud/app in the terminal, but it output the error: unable to find go.mod: file does not exist in "/Users/Works/news"

---- for exmaple input the command in the terminal news % ./bud/app unable to find go.mod: file does not exist in "/Users/Works/news"

Why the app is running and it needs to depend go.mod?

➜  bud-hello bud version
     bud: 0.1.4
  svelte: 3.47.0
   react: 18.0.0
➜  bud-hello bud run
| Listening on http://127.0.0.1:3000
# github.com/livebud/bud/package/scaffold
../../../go/pkg/mod/github.com/livebud/[email protected]/package/scaffold/template.go:28:8: undefined: any
note: module requires Go 1.18
➜  bud-hello go version
go version go1.17.10 darwin/amd64

Reproduce

1. Create a new project bud create project
2. Create a root route (important) bud new controller anything:/ index show
3. bud new controller index or bud new controller show (with or without views)
4. bud run will give an error:

| conjure: generate "bud/.app/main.go". conjure: generate "bud/.app/program/program.go". program: unable to wire. di: unable to wire "bud-app/bud/program".loadApp function. di: unable to find definition for param "bud-app/bud/.app/controller".*Controller in "bud-app/bud/.app/web".*Server . parser: unable to find declaration for "bud-app/bud/.app/controller".Controller in "bud/.app/web/web.go". bud/.app/controller/controller.go:15:2: Index redeclared in this block
	previous declaration at bud/.app/controller/controller.go:13:2

This PR addresses the first proposed solution to this issue. The second proposed solution is not implemented in this PR, but can be added with further information provided.

When launching the dev server, I have to manually open a browser tab and load the URL. It would be cool if it called out to the OS and simply told it to open the dev server URL. The quick and dirty (on macOS) is to shell out to open $URL but I imagine there's a go module somewhere that does this.

Even a flag to do it would be fine, as I invoke bud run from a Makefile (make dev).

c.f.: https://talk.jekyllrb.com/t/automatically-start-browser-on-localhost-4000-after-jekyll-serve/4916/5

In my apps I usually do something like:

var Version   string

func main() {
	os.Exit(httpserver.Run(Version))
}

Then at build time (in my Makefile) I do this:

VERSION := $(shell git describe --always --dirty=-dirty)
GOLDFLAGS += -X main.Version=$(VERSION)
GOFLAGS := -ldflags "$(GOLDFLAGS)"

This embeds the version string into the app so that I can access it, which is useful for me because then in production I can access this value and put it in my footers and error messages so that I know precisely which commit is running.

It would be cool if bud build could detect if it's building out of a git repo, and embed the git shorthash into the built binary in this way. It seems that you could skip the ldflags part and simply "hardcode" the git shorthash into the main.go via the main.gotext template at build filesystem generation time.

Anyway, just an idea, and I'm happy to take a crack at making a patch for this when I have some time, I just wanted to write it down while it's in my head.

golangci-lint run --fix
bud/internal/app/main.go:74:19: Error return value of `budClient.Publish` is not checked (errcheck)
	budClient.Publish("app:ready", nil)

As part of CI I run a linting of my repository. I suggest adding the line //nolint at the top of the template for bud/internal/app/main.go, as well as any other bud-managed template boilerplate files. This way any linters people use will ignore these files (without having to do special configuration to exclude the bud directory like is already automagically done in the .gitignore.)

When using example/placeholder strings that are interpreted as domain names, the domain example.com should be used.

https://github.com/livebud/bud/blob/d9c7ff63a929c9c308921ae35a929f19c5b101c9/internal/cli/create/create.go#L116