Pre-approve esm.sh network calls for easy access to transpiled modules during local-run

[filmaj]

Should help address issues like e.g. slack-samples/deno-triage-rotation#32

Closes #56

Try it out by adding the following to a sample app's slack.json:

"start" : "deno run -q --config=deno.jsonc --allow-read --allow-net --allow-run --allow-env /Users/fmaj/src/deno-slack-runtime/src/local-run.ts "

[codecov]

Codecov Report

Merging #57 (91dec45) into main (97cf8f5) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##             main      #57   +/-   ##
=======================================
  Coverage   95.08%   95.09%           
=======================================
  Files          14       14           
  Lines         549      550    +1     
  Branches       82       82           
=======================================
+ Hits          522      523    +1     
  Misses         27       27           

Files	Coverage Δ
src/local-run.ts	89.28% <100.00%> (+0.09%)	⬆️

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[WilliamBergamin]

LGTM, one concern on this relates to where we draw the line on adding domains here?
Now that Deno supports imports from npm with the npm: specifier and we fallback to bundling with esbuild should we favor the npm specifier over other alternatives?

[filmaj]

Really good point @WilliamBergamin! Maybe this isn't needed after all?

[filmaj]

Testing out what you suggested, indeed, not needed! npm: specifiers work great on their own in local-run mode, and your latest change to the hooks repo lands support for npm: specifiers during bundling. Excellent!

Gonna close this down.