feat: allow multiple operation names in graph:hander

[dwwoelfel]

Summary

Allows multiple operation names to be specified in graph:handler, e.g. ntl graph:handler --codegen ns/handler OpA OpB.

Still supports showing a list of operations to select from if no operation name is provided.

[github-actions]

📊 Benchmark results

Comparing with 2d4b68c

Package size: 222 MB

(no change)

^  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB  222 MB 
│   ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐    ┌──┐  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
│   |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |  |    |▒▒|  
└───┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴────┴──┴──>
    T-12    T-11    T-10    T-9     T-8     T-7     T-6     T-5     T-4     T-3     T-2     T-1      T    

Legend

• T-30 (11e657c): 228 MB
• T-29 (50035ba): 222 MB
• T-28 (016b718): 222 MB
• T-27 (c90adde): 222 MB
• T-26 (c6eb30e): 222 MB
• T-25 (9fa41a5): 222 MB
• T-24 (94415f9): 222 MB
• T-23 (8e50080): 222 MB
• T-22 (886eebc): 222 MB
• T-21 (f1d1d91): 222 MB
• T-20 (56cfac8): 222 MB
• T-19 (3af976e): 222 MB
• T-18 (7a2af72): 222 MB
• T-17 (6fad2a8): 222 MB
• T-16 (1450e1e): 222 MB
• T-15 (8b8d8b3): 222 MB
• T-14 (dfe0365): 222 MB
• T-13 (7d792df): 222 MB
• T-12 (ba96774): 222 MB
• T-11 (48d581a): 222 MB
• T-10 (c0a4cad): 222 MB
• T-9 (ac4e0e4): 222 MB
• T-8 (3073dbb): 222 MB
• T-7 (2f9072b): 222 MB
• T-6 (091387f): 222 MB
• T-5 (a5bfa11): 222 MB
• T-4 (cfeea77): 222 MB
• T-3 (5342938): 222 MB
• T-2 (e0f1680): 222 MB
• T-1 (2d4b68c): 222 MB
• T (current commit): 222 MB

[eduardoboucas]

I don't see any async behaviour in generateHandlerByOperationName, so do you need to await here? And if not, you could use a forEach and avoid disabling the ESLint rule?

[sgrove]

Good catch, @eduardoboucas . All of the generateHandler need to be async now (since they're no longer pure functions) - @dwwoelfel is this PR missing some of the upstream changes?

[dwwoelfel]

@eduardoboucas Whenever Sean's submits his PR for https://github.com/netlify/cli/compare/sg/graph-async-handler-gen, generateHandlerByOperationName will be async and the await will be necessary.

I think it's fine to merge this now and not make his PR a dependency on this one.

[sgrove]

@dwwoelfel I'm fine to merge as-is if you want to follow up with the other changes separately.

[sgrove]

@dwwoelfel Async change PR is at https://github.com/netlify/cli/pull/4984/files