Skip to content

tech: enable watch mode & esbuild server in dev mode#674

Merged
fraxken merged 1 commit intoNodeSecure:masterfrom
ErwanRaulo:watch-mode
Mar 22, 2026
Merged

tech: enable watch mode & esbuild server in dev mode#674
fraxken merged 1 commit intoNodeSecure:masterfrom
ErwanRaulo:watch-mode

Conversation

@ErwanRaulo
Copy link
Contributor

@ErwanRaulo ErwanRaulo commented Mar 10, 2026

what it does

add esbuild dev config to watch and serve fle for auto reload on served files changes.

features

  • build, watch, serve and opens web app using an nsecure-result existing file or run a command to use generated result file.

tests

commands tested : nsecure auto
[x] tests pass

to validate:

  • 2 esbuild config VS 1 file with process arg (-dev) ? Or use base config ?

to fix:

  • searchbar not visible when opening the server ip address after payload extraction.

@clemgbld clemgbld requested review from PierreDemailly and fraxken and removed request for fraxken March 11, 2026 09:22
@fraxken
Copy link
Member

fraxken commented Mar 11, 2026

Not a big fan of duplicating code (http etc)

@ErwanRaulo
Copy link
Contributor Author

Not a big fan of duplicating code (http etc)

i ended up removing duplication.

@ErwanRaulo ErwanRaulo force-pushed the watch-mode branch 2 times, most recently from 2971aff to 83933fe Compare March 11, 2026 21:05
@ErwanRaulo ErwanRaulo force-pushed the watch-mode branch 4 times, most recently from ad135d8 to df46649 Compare March 21, 2026 20:25
@ErwanRaulo ErwanRaulo marked this pull request as ready for review March 21, 2026 20:32
@ErwanRaulo ErwanRaulo requested a review from fraxken March 21, 2026 20:34
@ErwanRaulo ErwanRaulo force-pushed the watch-mode branch 5 times, most recently from b008493 to 08553bb Compare March 21, 2026 21:29
@fraxken fraxken merged commit 40ac566 into NodeSecure:master Mar 22, 2026
9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants