Skip to content

[Hackathon] Debug diagnostics for suppressed skill-nudge errors #285

Description

@yuhaouno

Summary

This is a narrow, independently reviewable subtask of #186 covering the two remaining src/lib/skill-nudge.ts silent catch paths.

The global --debug flag is not currently passed into the skill-nudge pre-action, so a failed profile lookup silently skips the nudge and an unreadable managed skill target is silently treated as absent. Those failures leave operators without a diagnostic even when they requested debug output.

Expected behavior

  • With --debug, write an actionable diagnostic to stderr for each swallowed failure.
  • Without --debug, preserve the existing warning text, output mode behavior, and exit status exactly.
  • Never pollute JSON stdout or let a failing diagnostic sink break the command.

Scope

  • Thread the parsed global debug flag into the pre-action nudge context.
  • Add debug-only diagnostics for the profile-lookup and managed-target read failures.
  • Add regression coverage for debug output and byte-identical non-debug behavior.

Relationship

Part of #186; deliberately split so this PR can close a concrete, scoped issue without closing the broader tracker.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

acceptedTriaged and accepted — a PR for this will be reviewedin-progressAssigned and actively being worked on

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions