Skip to content

feat(registry): add Skill context checks beside token usage - #530

Merged
harlan-zw merged 3 commits into
mainfrom
feat/skill-context-checks
Oct 7, 2026
Merged

harlan-zw merged 3 commits into
mainfrom
feat/skill-context-checks

Conversation

@harlan-zw

@harlan-zw harlan-zw commented Oct 7, 2026 •

Copy link
Copy Markdown
Collaborator

🔗 Linked issue

Related to Skit #52.

❓ Type of change

  • 📖 Documentation
  • 🐞 Bug fix
  • 👌 Enhancement
  • ✨ New feature
  • 🧹 Chore
  • ⚠️ Breaking change

📚 Description

Skill pages show token estimates without explaining metadata defects or Agent listing limits.
Show a compact description-size warning beside token usage. Expand it for the reason, metadata checks, and description limits.

Explain that Skills share space for their names and descriptions. Label the 1% threshold as a size suggestion.

The Claude comparison labels its 200k-context example and versioned source.
It accounts for when_to_use and disable-model-invocation without assuming the visitor's local settings.

Skill detail passes source metadata to context checks and displays warnings beside existing token estimates

🤖 AI disclosure: Harlan Agent Kit modified this description. My AI open-source policy.

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 7, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Agent browser check: the running Skill page shows context warnings beside token usage at desktop and 390px widths.
The mobile page has no horizontal overflow. The comparison labels the example allowance and source versions.

Desktop context checks
Mobile context checks

@harlan-github-agent harlan-github-agent Bot added harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit. and removed harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. labels Oct 7, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Agent browser check: context warnings now use one collapsed row below token usage.
Enter and click expand the warning, estimate, and source limits.
Checked the built page at desktop and 390px widths. No horizontal overflow.

Compact warnings on desktop
Expanded context on mobile

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 7, 2026
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 7, 2026
@harlan-zw

Copy link
Copy Markdown
Collaborator Author

🤖 Agent copy and browser check using Brundlefly's im-not-a-fly Skill.

The old text used “listing allowance” without explaining shared space.
The rewrite explains what the Agent reads and why shorter descriptions help.
It labels 1% as a size suggestion. Example settings and sources expand separately.

Checked the built page at 390px. Both expansions work. No horizontal overflow.

Plain-language description size warning

@harlan-zw harlan-zw added the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 7, 2026
@harlan-github-agent harlan-github-agent Bot removed the harlan-agent-review Pull request triage requires an adversarial Review. Adding this label manually always forces one. label Oct 7, 2026
@harlan-zw
harlan-zw merged commit 02f2090 into main Oct 7, 2026
7 checks passed
@harlan-zw
harlan-zw deleted the feat/skill-context-checks branch October 7, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

harlan-agent-review-required Pull request triage requires an adversarial Review for this head commit.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant