Skip to content

docs(skills): cover maintenance commands and verification - #2748

Open
rajanpanth wants to merge 1 commit into
Graphify-Labs:v8from
rajanpanth:docs/skill-command-and-failure-guidance
Open

docs(skills): cover maintenance commands and verification#2748
rajanpanth wants to merge 1 commit into
Graphify-Labs:v8from
rajanpanth:docs/skill-command-and-failure-guidance

Conversation

@rajanpanth

Copy link
Copy Markdown
Contributor

Summary

  • route blast-radius questions to \graphify affected\ and document its relation/depth controls
  • surface the remaining maintenance commands in every generated skill
  • add verification and recovery guidance for labeling, tree views, HTML limits, missing language extras, and OpenAI-compatible endpoints

Fixes #2535.

Validation

  • uv run python -m tools.skillgen --bless
  • uv run python -m tools.skillgen
  • uv run python -m tools.skillgen --check
  • uv run pytest tests/test_skillgen.py -q

@graphify-labs graphify-labs Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Graphify reviewed this change.

Looks safe to merge — no coupling regressions and no blocking issues, checked against the code graph (not a self-assessment).


Graphify review — findings

This pull request adds two new documentation sections to the graphify skill markdown files and their corresponding expected-output test fixtures. The first section documents the graphify affected command for blast-radius/dependency queries, and the second lists less-common maintenance commands (tree, global, check-update, diagnose multigraph, merge-driver). The identical content is applied across all agent variants (agents, amp, claw, codex, copilot, droid, kilo, kiro, opencode, etc.) and the paired tools_skillgen_expected_* fixtures.

No blocking issues surfaced. 1 lower-confidence candidate did not survive cross-model review.

Analysis details — impact, health, verification

Impact & health

Graphify review

Impact — 1136 functions depend on the 1136 functions this change touches.

Health — grade A; no new coupling hotspots.

Verification — 1136 functions in the blast radius were not formally verified this run (proofs are advisory here).

Gate & verification

graphify gate

PASS — objectively clean (no health regressions, tests not run — proofs not run this pass (advisory)). Grounded, not self-assessed.

Advisory (not blocking):

  • verification_scope: 1136 function(s) in the blast radius were not formally verified this run

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.

skill: six commands undocumented (incl. affected), and none of the silent-failure modes are covered

1 participant