is Mux.jl an appropriate home for HTTP/Revise glue? about mux.jl HOT 7 OPEN

I managed to use Revise with this script:

using Pkg
Pkg.activate(@__DIR__)
using Revise: entr, revise
using MyApp


const t = MyApp.main() # call serve(app), nothing special


entr(String[], [MyApp]; all = true) do
	revise(MyApp)
end

from mux.jl.

Ah, yes, that's a nice simple approach! Very nice 😃 Would you be happy to make a PR adding something like this to the readme?

from mux.jl.

I've gotten my baby TodoMVC.jl to work with this glue. It's not great, I still think Mux.jl may be the most appropriate place for targeting this sort of developer oriented workflow. The bulk of the work happens in live_server.jl which probably needs a different name, but if it could be integrated here, that might be useful.

from mux.jl.

I think that having Mux work with Revise is worthwhile, but I'm not totally sold on this approach. I don't really like that we're closing and re-opening the socket, either, though maybe that's fine.

Re-evaluating the @app expression should be sufficient for Mux.serve to use the latest version of the code. Perhaps we can do something with that?

Looking at the revise docs, it seems like Revise.includet doesn't evaluate data definitions again if they change in the file, it just evaluates them on the first go around, then updates only methods. Which would explain why the top-level statements generated by the macro aren't being re-evaluated when they change. There might be something we can do there with Revise.track directly (though my initial attempts at that aren't succeeding).

from mux.jl.

I think that having Mux work with Revise is worthwhile, but I'm not totally sold on this approach.

Awesome. We're on the same page. I don't like the opening/closing of the socket either. Frankly, there's parts of the code I don't grok.

Which would explain why the top-level statements generated by the macro aren't being re-evaluated when they change.

With the existing TodoMVC, it did re-evaluate the @app macro, I think it only worked due to the clever if/else logic there. I've not tried an example using with Revise.includet.

from mux.jl.

Stumbling upon this because also for me it seems that revise is not working with Mux.jl which is very unfortunate...

Are there any updates on makeing Revise work with Mux.jl?

from mux.jl.

I just found a simple way: not to use @app.

I guess the advantage that Revise works out of the box like this is really huge.

module MyModule
using Mux: serve, mux, App
myserve(req) = req |> mux(
    ...
)

function main()
    serve(App(myserve), 1234)
end
end # module

Looks only very slightly alien ;-) but as Revise works and we have one macro less, I am actually quite happy with it.

from mux.jl.