Replace `print` with `os.Logger` so failures are findable
I maintainer di solito rispondono entro 1 giorno
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
Labels: enhancement, good first issue · Difficulty: easy (about 1 to 2 hours)
Problem
CatNotch is a GUI app, so print output goes nowhere a user or a bug reporter can see it. Five real failure paths log with print:
CatNotchCore/CatNotchStore.swift:178([store] write failed)CatNotchCore/ClaudeHooks.swift:421([claude] no free port)CatNotchCore/ClaudeHooks.swift:422(hook sync failed)CatNotch/GlobalShortcut.swift:30(shortcut already taken)CatNotch/SettingsWindow.swift:447(open at login failed)
A failed store write or a missing hook port is exactly what a bug report needs, and today it vanishes.
Expected outcome
These messages go to the unified log through os.Logger, visible in Console.app and log stream, with no behavior change.
Relevant files
- The five locations above (re-run
grep -rnE '\bprint\(' CatNotch CatNotchCoreto confirm the list).
Acceptance criteria
- No
print(remains inCatNotch/orCatNotchCore/. - One
Loggerper file or subsystem, using the bundle IDcom.thecharydev.catnotchas the subsystem. Do not log tokens, file contents or full paths of the user's files. - No change to what the app does. The existing tests still pass.
- The PR description shows the Console.app filter to find the messages.
- Lingua principale
- Swift
- Stelle
- 1
- Fork
- 1
- Merge medio
- 14m
- PR unite (30g)
- 15
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di thecharydev/catnotch
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 83/100
thecharydev/catnotch#41 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation good first issue
Difficoltà 2/5 1-3 ore Idoneità per principianti 87/100
thecharydev/catnotch#40 ·
I maintainer di solito rispondono entro 1 giorno
-
documentation good first issue
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 86/100
thecharydev/catnotch#39 ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement needs-triage
Difficoltà 3/5 1-2 giorni Idoneità per principianti 57/100
thecharydev/catnotch#51 ·
I maintainer di solito rispondono entro 1 giorno
-
bug needs-triage
Difficoltà 3/5 1-2 giorni Idoneità per principianti 67/100
thecharydev/catnotch#49 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di thecharydev/catnotch
Issue simili
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
ls1intum/artemis-ios#571 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
ml-explore/mlx-swift-lm#791 ·
I maintainer di solito rispondono entro 2 giorni
-
a: plugins d: docs/ fyi-macos P2 platform-ios platform-macos team-ios
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
swiftlang/github-workflows#352 ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
I maintainer di solito rispondono entro 1 giorno