gemini-cli icon indicating copy to clipboard operation
gemini-cli copied to clipboard

Migrate console to coreEvents.emitFeedback or debugLogger

Open Adib234 opened this issue 3 weeks ago • 1 comments

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

Adib234 avatar Dec 17 '25 21:12 Adib234