Unhelpful error messages when using modules

[sipsma]

Collecting unhelpful error messages here, please comment when you hit one 🙂

[marcosnils]

here' one reported by @felipecruz91 in our discord server here: https://discord.com/channels/707636530424053791/1223281904871014532/1223281904871014532

basically the fact that when running dagger call in a directory where dagger.json is not present, you generally get quite confusing error outputs.

[shykes]

• CLI: don't show stderr twice on error #7007

[jedevc]

• Re-exporting go module interfaces fail with unexpected error #7166

[jedevc]

Found another one when working on our own ci.

Non existent secret files don't error:

$ dagger call --host-docker-config=file:path/to/non-existent-file --source=. version string
dev-eca71ba71430c5ae94a3c642578b8391f721ffd7

Non existent directories error very strangely:

$ dagger call --host-docker-config=$HOME/.docker/config.json --source=path/to/non-existent/directory version string
✔ connect 0.8s
✔ initialize 0.8s
✔ ModuleSource.resolveFromCaller: ModuleSource! 0.1s
✘ ModuleSource.resolveDirectoryFromCaller(path: "path/to/non-existent/directory"): Directory! 0.0s
! failed to stat caller path: failed to receive file bytes message: rpc error: code = NotFound desc = rpc error: code = NotFound desc = stat path: lstat path/to/non-existent/directory: no such file or directory
✘ dagger: Dagger! 0.0s
! Query.dagger: missing required argument: "source"

Error: response from query: input: dagger resolve: Query.dagger: missing required argument: "source"

Possibly there's more issues with other flag types as well.

[Excoriate]

Perhaps a tiny addition to this issue is what I've written just some moments ago here. For new module builders, isn't intuitive how the modules are propagated to the user. For example:

Stdout:
invoke: input: container.from.withEnvVariable.withMountedDirectory.withWorkdir.withMountedCache.withEnvVariable.withExec.withExec.withExec
.stdout resolve: process "xyz command" did not complete successfully: exit co
de: 125

It doesn't say much, and it's not human-readable. It'd be great to have a more clear error message such as: Invalid environment variables due to invalid arguments passed XYZ.

[jedevc]

• Unhelpful error messages when using modules #6718
• 🐞 Module names with single-dotted characters cause "main object not found" #7278