Skip to content

Remove proprietary information from security skills#2

Merged
biplav-crl merged 1 commit intomainfrom
skill/security-and-governance/expand-security-skills
Mar 9, 2026
Merged

Remove proprietary information from security skills#2
biplav-crl merged 1 commit intomainfrom
skill/security-and-governance/expand-security-skills

Conversation

@biplav-crl
Copy link
Contributor

  • Remove zendesk-ticket-analysis.md (internal planning doc with Snowflake table names and support ticket volume metrics)
  • Anonymize sample-report.md: replace real cluster ID, cluster name, and employee username with generic placeholders

What does this PR do?

Related Issue

Closes #

Type of Change

  • New skill
  • Skill update/improvement
  • Documentation update
  • Infrastructure/tooling change
  • Bug fix

Skill Compliance Checklist

  • SKILL.md includes required frontmatter fields (name, description)
  • Skill name follows naming conventions (lowercase, hyphens, gerund form preferred)
  • Skill name matches directory name exactly
  • Description is specific and includes "when to use" trigger keywords
  • Description is max 1024 characters
  • Skill is under 500 lines (or uses references/ for detailed content)
  • References official CockroachDB documentation (not duplicate content)
  • Includes safety guardrails for risky operations (if applicable)
  • Tested with at least one AI agent (manual validation)
  • No time-sensitive information (version numbers, dates, "currently")
  • Directory structure follows specification (only scripts/, references/, assets/ subdirs)
  • No reserved words in skill name ("anthropic", "claude")
  • Local validation passes: python scripts/validate-spec.py skills/

Documentation Updates

  • README.md
  • CONTRIBUTING.md
  • Skill SKILL.md files
  • Other (specify):

Testing

Manual testing:

  • Tested with AI agent (specify which one):
  • Verified skill is discoverable with appropriate prompts
  • Validated technical accuracy against CockroachDB docs
  • Tested any scripts or commands included in the skill

Automated testing:

  • Local validation script passes: python scripts/validate-spec.py skills/
  • CI validation will run automatically on this PR

Additional Context

Reviewer Notes


By submitting this PR, I confirm:

- Remove zendesk-ticket-analysis.md (internal planning doc with
  Snowflake table names and support ticket volume metrics)
- Anonymize sample-report.md: replace real cluster ID, cluster name,
  and employee username with generic placeholders

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@biplav-crl biplav-crl merged commit e82ac72 into main Mar 9, 2026
1 check passed
@biplav-crl biplav-crl deleted the skill/security-and-governance/expand-security-skills branch March 9, 2026 14:50
biplav-crl added a commit that referenced this pull request Mar 9, 2026
…e/expand-security-skills

Remove proprietary information from security skills
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.

1 participant