Skip to content

Comments

refactor(zeph-skills): rename requires-secrets to x-requires-secrets vendor extension#693

Merged
bug-ops merged 1 commit intomainfrom
feat/688-vendor-prefix
Feb 21, 2026
Merged

refactor(zeph-skills): rename requires-secrets to x-requires-secrets vendor extension#693
bug-ops merged 1 commit intomainfrom
feat/688-vendor-prefix

Conversation

@bug-ops
Copy link
Owner

@bug-ops bug-ops commented Feb 21, 2026

Summary

  • x-requires-secrets is now the canonical frontmatter field per JSON Schema vendor extension convention
  • Old requires-secrets form still accepted with tracing::warn! deprecation message
  • Canonical form takes precedence when both are present
  • All docs, tests, and READMEs updated

Closes #688

Breaking changes

  • requires-secrets in SKILL.md is deprecated in favor of x-requires-secrets

Test plan

  • cargo nextest run --workspace --lib --bins — 2229 tests pass
  • cargo clippy --workspace -- -D warnings — clean
  • cargo +nightly fmt --check — clean

@github-actions github-actions bot added refactor size/M documentation Improvements or additions to documentation skills SKILL.md system rust core and removed size/M labels Feb 21, 2026
…vendor extension

Use x-requires-secrets as canonical field per JSON Schema vendor extension
convention. Old requires-secrets form still accepted with deprecation
warning. Canonical form takes precedence when both are present.

Closes #688
@bug-ops bug-ops force-pushed the feat/688-vendor-prefix branch from 5d9bbbb to fa131d8 Compare February 21, 2026 01:00
@bug-ops bug-ops enabled auto-merge (squash) February 21, 2026 01:01
@bug-ops bug-ops merged commit 7dfe4b8 into main Feb 21, 2026
22 checks passed
@bug-ops bug-ops deleted the feat/688-vendor-prefix branch February 21, 2026 01:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core documentation Improvements or additions to documentation refactor rust size/M skills SKILL.md system

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Namespace requires-secrets as vendor extension (x-requires-secrets)

1 participant