Skip to content

refactor: Zod-based config loader #45

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 76 commits into
base: main
Choose a base branch
from

Conversation

fabiovincenzi
Copy link
Collaborator

Replace JSON-Schema with Zod for config validation and default injection
Add typed config export, loadConfig(), validate() and setConfigFile()
Update index.ts to use the new loader and pass the validated config.

renovate bot and others added 20 commits April 28, 2025 07:47
fix(deps): update npm - li-cli - experimental/li-cli/package.json
chore(deps): update github-actions - workflows - .github/workflows/dependency-review.yml
chore: bump by minor to v1.11.0
chore: switch @typescript-eslint/no-explicit-any to off instead of warn
…y-plugin-samples - plugins/git-proxy-plugin-samples/package.json
…ples-manager

chore(deps): update dependency @finos/git-proxy to ^1.11.0 - git-proxy-plugin-samples - plugins/git-proxy-plugin-samples/package.json
fix(deps): update dependency axios to ^1.9.0 - git-proxy-cli - packages/git-proxy-cli/package.json
fix(deps): update npm - website - website/package.json
@codecov-commenter
Copy link

codecov-commenter commented Apr 30, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 48.53333% with 193 lines in your changes missing coverage. Please review.

Please upload report for BASE (main@3718943). Learn more about missing BASE report.

Files with missing lines Patch % Lines
src/proxy/processors/push-action/gitleaks.ts 11.62% 76 Missing ⚠️
src/config/ConfigLoader.ts 62.22% 59 Missing and 9 partials ⚠️
src/config/index.ts 50.00% 16 Missing ⚠️
src/proxy/index.ts 36.36% 14 Missing ⚠️
src/service/index.js 22.22% 14 Missing ⚠️
src/config/file.ts 50.00% 5 Missing ⚠️

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #45   +/-   ##
=======================================
  Coverage        ?   48.22%           
=======================================
  Files           ?       52           
  Lines           ?     2117           
  Branches        ?      241           
=======================================
  Hits            ?     1021           
  Misses          ?     1052           
  Partials        ?       44           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

06kellyjac and others added 30 commits May 14, 2025 16:03
ci: generate js and definitions from ts and place in original location
add meeting minutes GitHub Issue template
fix(deps): update dependency eslint to ^9.27.0 - website - website/package.json
chore(deps): update github-actions - workflows - .github/workflows/dependency-review.yml
feat: converted to typescript

fix: config loader clone command issue

fix: adds input validation, uses array arguments, prevented shell spawn

fix: adds failsafe checking for directory location and structure

fix: env-paths change to v2.2.1 which support require and minor code fix

fix: improves test coverage

Adds additional tests for better cove

fix: fixed creating cache directory
Default value should be false, and set to true when cert path is configured
feat: implements config loader to enable remote or external configs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants