🧹 Janitor: Refactor format selection logic to use a map #74
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🧹 Code Quality Improvement
📊 What Changed
Refactored the output format selection logic in
api/index.go. Theswitchstatement that handled "html", "md", and "json" formats has been replaced with a map of handler functions.🎯 Why This Helps
This change improves code quality in several ways:
handlerfunction is now simpler and its intent is clearer. It delegates the formatting task instead of containing all the implementation details.🔍 Before/After
Before:
switchstatement directly inside the mainhandlerfunction managed all formatting logic.caseto theswitch, increasing the function's complexity.After:
map[string]formatHandlercalledformatterscleanly maps format strings to dedicated handler functions.handlerfunction now performs a simple map lookup to find the correct formatter.✅ Verification
go test ./...)📝 Notes
This is a pure refactoring and introduces no changes to functionality.
PR created automatically by Jules for task 848828935292256217 started by @lucasew