refactor: resolve eslint warnings (max-statements, complexity) - src/lib/assets/index.ts

[btlghrants]

Describe what should be investigated or refactored

Refactor of src/lib/assets/index.ts to reduce complexity warnings:

/pepr/src/lib/assets/index.ts
  82:23  warning  Async method 'generateHelmChart' has a complexity of 12. Maximum allowed is 10        complexity
  82:23  warning  Async method 'generateHelmChart' has too many statements (58). Maximum allowed is 20  max-statements

Additional context

In support of #1248

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 79.12%. Comparing base (b75ac42) to head (102ab83).
Report is 1 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1497   +/-   ##
=======================================
  Coverage   79.12%   79.12%           
=======================================
  Files          39       39           
  Lines        1720     1720           
  Branches      388      404   +16     
=======================================
  Hits         1361     1361           
+ Misses        357      330   -27     
- Partials        2       29   +27     

see 7 files with indirect coverage changes

[btlghrants]

Got all the complexity warnings fixed and shrunk number of statements to just one more than the limit where I then stopped because any further statement squashing would have made the function more difficult to grok.

This method has the (rather large) responsibility for laying out files on disk in a format that it doesn't own -- a helm chart's expected structure -- and that feels rather larger in scope than the max-statements eslint rule was intended to apply to, so... after I shrank the function as much as I thought was advisable I added a comment-based config to tell eslint that this method is (likely) as minimal as it's going to get.

[samayer12]

Discussion points have been resolved, good work on addressing some complex code!