gemini-cli
gemini-cli copied to clipboard
Migrate console to coreEvents.emitFeedback or debugLogger
Summary
As title suggests, also added a no-console rule to eslint to prevent usage of console.* unless there's a valid reason.
There were also other console.* statements picked up by my rule that wasn't a sub issue, I used my best judgement + Gemini CLI to figure out whether it should be kept or be a user/non-user facing error
Details
Related Issues
Closes all the sub-issues in #10761 except migrating CLI appEvents to Core coreEvents
Closes #11838 Closes #11841 Closes #11852 Closes #11854 Closes #11862 Closes #11872 Closes #11876 Closes #11880 Closes #11883 Closes #11884 Closes #11885 Closes #11887 Closes #11891 Closes #11897 Closes #11898 Closes #11899 Closes #11901 Closes #11906
How to Validate
Pre-Merge Checklist
- [ ] Updated relevant documentation and README (if needed)
- [ ] Added/updated tests (if needed)
- [ ] Noted breaking changes (if any)
- [ ] Validated on required platforms/methods:
- [ ] MacOS
- [X] npm run
- [ ] npx
- [ ] Docker
- [ ] Podman
- [ ] Seatbelt
- [ ] Windows
- [ ] npm run
- [ ] npx
- [ ] Docker
- [ ] Linux
- [ ] npm run
- [ ] npx
- [ ] Docker
- [ ] MacOS