feat: watch command execute on successful build

[SrIzan10]

absolute cinema

[SrIzan10]

@jacoobes i also request a review from you :)

[jacoobes]

is it possible to timeout this?

[EvolutionX-10]

Everything else Looks good to me, confirm testing with @jacoobes

[SrIzan10]

it works well for me

[jacoobes]

Will check tonight! One thing I was thinking about is how often does it refresh? We want it to be reasonable with api publisher limits

…

On Fri, Jan 31, 2025 at 1:31 PM SrIzan10 ***@***.***> wrote: it works well for me — Reply to this email directly, view it on GitHub <#151 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/ASJS6OZSUWZNJQBJ674ZWVD2NPFQLAVCNFSM6AAAAABWDYFDDWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDMMRYGE3DENRTGA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[jacoobes]

i will test this week. stuff got pushed

[EvolutionX-10]

LGTM

[jacoobes]

API proposal: theres this proposal adds watchCommand which allows user to run any script they want after succesful build, and also the implicit start command. why not this: if watchCommand is supplied, then it overrides the default <packageManager> start. I can implement this in the branch. How does that sound?

@EvolutionX-10 @SrIzan10

[SrIzan10]

i think it already does that. if there's a watch command run that, else the start command

[jacoobes]

i think it already does that. if there's a watch command run that, else the start command

ahhh i see okay

[jacoobes]

lets push a dev release, how does that sound? @SrIzan10 @EvolutionX-10 that way we can let the community test it.

[jacoobes]

before that i need to implement debouncing on build calls to avoid extra calls

[jacoobes]

note: esbuild local npm links do not work with watch command