Make UI server more CLI/library friendly
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- TypeScript
- Stars
- 431
- Forks
- 179
- Avg merge
- 2d 9h
- Merged PRs (30d)
- 71
Description
Describe the solution you'd like
UI server currently dumps things to stdout/stderr which cannot be controlled from what I can see. Please make the following changes:
- Allow the Echo logger to be customized or at least disabled
- Allow
HideBannerto be set to not show the banner - Remove (or use logger for) the
fmt.Printlnstatements for "Starting" and "Stopping" UI server (and anywhere else they may be used) - Do not call
s.httpServer.Logger.FatalonStart, this kills the entire process. Just log an error if one is returned, otherwise, the caller can decide whether to kill the process
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the UI server entry point and its Echo logger, HideBanner setting, Start and stop paths, and the fmt.Println calls named in the issue. Trace how logging and startup errors reach callers. Done means logging and the banner are configurable, startup and shutdown messages use the chosen logger, and Start reports errors without terminating the process.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- go
- Domain
- backend
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100