Skip to content

fix(skill): correct duplicate Option numbering in auth section#1161

Merged
ctate merged 1 commit into
vercel-labs:mainfrom
Marshall-Sun:fix/skill-md-option-numbering
Apr 7, 2026
Merged

fix(skill): correct duplicate Option numbering in auth section#1161
ctate merged 1 commit into
vercel-labs:mainfrom
Marshall-Sun:fix/skill-md-option-numbering

Conversation

@Marshall-Sun

Copy link
Copy Markdown
Contributor

What

Fix duplicate Option numbering in the authentication section of skills/agent-browser/SKILL.md.

The current file has two headings both labeled "Option 4":

  • Line 85: "Option 4: Session name"
  • Line 96: "Option 4: Auth vault"

This causes "Option 5: State file" to also be off-by-one.

Fix

Renumbered sequentially:

Before After
Option 4: Session name Option 4: Session name (unchanged)
Option 4: Auth vault Option 5: Auth vault
Option 5: State file Option 6: State file

Scope

Text-only, 3 lines changed. No logic or structural changes.

@vercel

vercel Bot commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

@Marshall-Sun is attempting to deploy a commit to the Vercel Labs Team on Vercel.

A member of the Team first needs to authorize it.

@ctate

ctate commented Apr 7, 2026

Copy link
Copy Markdown
Collaborator

Thank you @Marshall-Sun!

@ctate ctate merged commit e4e2fe8 into vercel-labs:main Apr 7, 2026
3 of 5 checks passed
@Marshall-Sun

Copy link
Copy Markdown
Contributor Author

@ctate Thanks for the merge!

While working on this fix I noticed a broader issue: SKILL.md is currently 700+ lines, which exceeds Anthropic's recommended 500-line limit for skill files. The main contributors to the length are:

  1. Auth content duplication — three subsections under "Common Patterns" largely repeat what's already covered in "Handling Authentication" (~70 lines of overlap)
  2. Essential Commands — the full command reference could be moved to references/commands.md, keeping only the top ~15 commands inline (~60 lines saved)

Would you be open to a follow-up PR from me that trims the file down to ~450 lines via deduplication and offloading reference content to the existing references/ directory?

@ctate ctate mentioned this pull request Apr 12, 2026
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