Skip to content

skills: force-load adapter-author when creating a command - #390

Open
ankitranjan7 wants to merge 1 commit into
mainfrom
skills/force-adapter-author
Open

skills: force-load adapter-author when creating a command#390
ankitranjan7 wants to merge 1 commit into
mainfrom
skills/force-adapter-author

Conversation

@ankitranjan7

Copy link
Copy Markdown
Contributor

Description

Closes #386.

webcmd-usage already force-loads webcmd-browser before raw browser work. It now force-loads webcmd-adapter-author before creating, revising, or privately overriding a command.

Also:

  • Authoring notes go through site-memory, not a hand-written file next to the adapter.
  • Verification is webcmd verify / webcmd browser verify, not a sibling verify command.
  • args keys match the declared name (args['note-id'], not args.noteId).

Published skill names no eval fixture.

Type of Change

  • 📝 Documentation

Checklist

  • I ran the checks relevant to this PR
  • I updated tests or docs if needed
  • If I edited skill-src/, I ran make build and committed skills/
npx vitest run --project unit src/skills.test.ts: 15 passed

Creating, revising, or privately overriding a command requires
webcmd-adapter-author. Notes go through site-memory; verify is
webcmd verify / browser verify; args keys match declared name.

Closes #386
@github-actions

Copy link
Copy Markdown
Contributor

🟠 Maintainer review suggested — low confidence

The automated review could not reach a fully supported conclusion.

Limitations

  • The automated review returned an invalid structured result.

This review is advisory and does not block merging.

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.

skills(usage): creating or customizing a command requires webcmd-adapter-author

1 participant