Skip to content

Hide skill lifecycle hooks from agent docs - #186

Merged
sklinglernv merged 1 commit into
mainfrom
fix/hide-skill-lifecycle-docs
Aug 21, 2026
Merged

Hide skill lifecycle hooks from agent docs#186
sklinglernv merged 1 commit into
mainfrom
fix/hide-skill-lifecycle-docs

Conversation

@furgalep

@furgalep furgalep commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • mark the base Skill.attach() and Skill.detach() lifecycle hooks with the existing @hidden decorator
  • verify inherited lifecycle hooks stay out of model-facing agent docs
  • avoid adding any new agentdoc visibility mechanism or changing spec(..., hidden=False) semantics

Verification

  • uv run ruff check src/nooa/skill.py tests/agentdoc/test_skill_lifecycle_visibility.py
  • uv run pytest tests/agentdoc -q (687 passed)

@furgalep
furgalep force-pushed the fix/hide-skill-lifecycle-docs branch from 689c06a to bffe605 Compare August 21, 2026 10:54
@furgalep
furgalep changed the base branch from dev/tui to main August 21, 2026 10:54
@furgalep
furgalep force-pushed the fix/hide-skill-lifecycle-docs branch 2 times, most recently from 27ae6dc to 54c3978 Compare August 21, 2026 10:56
Signed-off-by: Paul Furgale <pfurgale@nvidia.com>
@furgalep
furgalep force-pushed the fix/hide-skill-lifecycle-docs branch from 54c3978 to 70eb052 Compare August 21, 2026 10:57

@sklinglernv sklinglernv left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sklinglernv
sklinglernv merged commit 97f52de into main Aug 21, 2026
6 checks passed
@sklinglernv
sklinglernv deleted the fix/hide-skill-lifecycle-docs branch August 21, 2026 14:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants