#16792: fix(skills): use subsystem logger instead of console.log for debug output
agents
stale
size: XS
Cluster:
Skill Improvements and Fixes
Fixes #16661
## Problem
src/agents/skills/workspace.ts uses raw console.log for debug output (lines 57, 62), polluting stdout on every message when skill filters are configured, regardless of log level settings.
## Solution
Replace console.log with skillsLogger.verbose to respect configured log levels.
## Changes
- Line 57: console.log → skillsLogger.verbose
- Line 62: console.log → skillsLogger.verbose
## Impact
- Debug output now respects log level configuration
- No more stdout pollution in production
- Messages only appear when verbose logging is enabled
<!-- greptile_comment -->
<h3>Greptile Summary</h3>
Replaces raw `console.log` calls in `filterSkillEntries()` with `skillsLogger.verbose()` to respect configured log levels. However, the `SubsystemLogger` type does not expose a `verbose` method — its available methods are `trace`, `debug`, `info`, `warn`, `error`, `fatal`, and `raw`. This will throw a `TypeError` at runtime when skill filters are configured. As noted in previous review comments, `skillsLogger.debug()` should be used instead.
<h3>Confidence Score: 1/5</h3>
- This PR will crash at runtime — do not merge without fixing the method name.
- The `SubsystemLogger` type defines `trace`, `debug`, `info`, `warn`, `error`, `fatal`, and `raw` methods — there is no `verbose` method. Both changed lines (57 and 62) call `skillsLogger.verbose()`, which will throw `TypeError: skillsLogger.verbose is not a function` at runtime whenever skill filters are configured. The fix is straightforward (use `debug` instead of `verbose`), but as-is this PR introduces a regression.
- `src/agents/skills/workspace.ts` — both changed lines use a non-existent method
<sub>Last reviewed commit: 4a99799</sub>
<!-- greptile_other_comments_section -->
<!-- /greptile_comment -->
Most Similar PRs
#19664: fix(skills): log skill YAML parsing diagnostics with skill name
by orchidsun · 2026-02-18
80.9%
#12076: fix(skills): recursive directory filtering to actually exclude venv...
by xiaoyaner0201 · 2026-02-08
80.5%
#12956: fix: guard .trim() calls on potentially undefined workspaceDir
by omair445 · 2026-02-10
78.4%
#11250: fix: expand skills watcher ignore list and improve session repair l...
by zhangzhefang-github · 2026-02-07
77.6%
#21839: fix(skills): allowBundled: [] now blocks all bundled skills
by hydro13 · 2026-02-20
76.9%
#22525: [Bug]: Session snapshot not reloading skills after gateway restart ...
by zwffff · 2026-02-21
76.6%
#19707: fix(agents): apply per-agent skills filter to all run paths
by mcaxtr · 2026-02-18
76.5%
#8873: fix: document skill loading locations in system prompt
by ytfh44 · 2026-02-04
76.5%
#9221: fix(skills): use skillKey for env config lookup in snapshots
by gavinbmoore · 2026-02-05
76.1%
#22198: fix(skills): treat empty allowBundled array as block-all
by haitao-sjsu · 2026-02-20
76.0%