Skip to content

fix: database skills cleanup#63

Merged
markusha77 merged 2 commits intoChatAndBuild:mainfrom
markusha77:database-cleanup
Mar 29, 2026
Merged

fix: database skills cleanup#63
markusha77 merged 2 commits intoChatAndBuild:mainfrom
markusha77:database-cleanup

Conversation

@markusha77
Copy link
Copy Markdown
Collaborator

New Skill Contribution

Skill Info

  • Skill ID:
  • Category:
  • Description:

Checklist

  • SKILL.md has valid YAML frontmatter with all required fields
  • Skill ID matches the directory name
  • Category is one of: productivity, development, communication, writing, research, other
  • Instructions are clear and self-contained
  • Instructions are under 4000 tokens
  • No external URLs referenced in instructions
  • Tested the skill with an AI agent and it produces good results
  • Added usage examples in the frontmatter

Testing

Describe how you tested this skill:

Notes

Any additional context or notes for reviewers.

@greptile-apps
Copy link
Copy Markdown
Contributor

greptile-apps bot commented Mar 29, 2026

Greptile Summary

This PR performs a broad cleanup of the skills/databases directory, removing 42 placeholder skills that contained only auto-generated boilerplate and upgrading ~25 legitimate database skills with substantive, opinionated content. The changes are consistently applied across both the individual SKILL.md files and the central index.json registry.

Key changes:

  • 42 skills deleted — all had identical template content ("Step-by-step guidance for…", "Support X workflows with clear steps and best practices", generic Output section). Many were also miscategorized (e.g. culture-index, lokalise-local-dev-loop, windsurf-refactoring-large, tencent-cos) or were near-duplicates of retained skills (e.g. create-migration, database-migrations-sql-migrations, managing-database-migrations, db-migrations-and-schema-changes).
  • ~25 skills improved — upgraded from the same boilerplate to skills with real "Clarify First" prompts, domain-specific guidance, structured output sections, failure-mode catalogues, and a Boundaries clause discouraging the AI from pretending it has live database access. Frontmatter is also normalized to include requires: [] and populated examples: arrays.
  • index.json kept in sync — every deletion and update in the filesystem is mirrored in the index. No orphaned entries or missing registrations were found.
  • The PR checklist in the description is entirely unchecked, but this is a cleanup PR rather than a new-skill contribution, so those items are not applicable here.

Confidence Score: 5/5

Safe to merge — changes are content-only, well-scoped, and internally consistent between SKILL.md files and index.json.

All 42 deletions and ~25 updates are synchronized between the filesystem and index.json. No logic code is touched. Deleted skills contained only auto-generated template content with no unique value. Improved skills have materially better guidance. No P0 or P1 issues found.

No files require special attention.

Important Files Changed

Filename Overview
index.json Synchronized with SKILL.md changes: 42 low-quality entries removed and ~25 entries updated with richer descriptions, instructions, and example arrays. JSON structure is valid and consistent.
skills/databases/algolia-search/SKILL.md Upgraded from a generic 20-line template to a substantive skill with clarify-first prompts, review focus areas, output format, and common mistakes. Frontmatter now includes requires and examples.
skills/databases/create-database-migration/SKILL.md Significantly expanded with migration safety principles, drafting guidance, failure modes, and explicit boundaries around not simulating execution. Quality is well above the deleted create-migration template.
skills/databases/database-architect/SKILL.md Expanded from template to a thorough architectural skill covering topology choices, tenancy, consistency, and operational tradeoffs. Good coverage of failure modes and boundaries.
skills/databases/apollo-migration-deep-dive/SKILL.md Deleted. Was a generic 20-line template with no substantive content. Removal from both filesystem and index.json is consistent.
skills/databases/culture-index/SKILL.md Deleted. Culture Index is a personality assessment tool, not a database technology — was miscategorized. Removal is correct.
skills/databases/database-design/SKILL.md Upgraded with entity modeling guidance, normalization priorities, query-aware design principles, and common failure modes around JSON overuse and missing constraints.
skills/databases/database-migrations/SKILL.md Expanded with migration safety principles, rollout planning, backfill phases, and explicit watch-fors like long locks and nullability changes. Replaces three lower-quality duplicates.
skills/databases/dynamodb/SKILL.md Rewritten with DynamoDB-specific access-pattern-first design philosophy, partition-key guidance, and common single-table design pitfalls.
skills/databases/transaction-correctness/SKILL.md Upgraded with atomicity, isolation, idempotency, and race-condition review framework. Good focus on external side effects that can't be rolled back.

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A[skills/databases/* SKILL.md files] --> B{Content quality check}
    B -->|Template-only content\nor wrong category| C[Delete SKILL.md\nand directory]
    B -->|Legitimate database skill\nwith real value| D[Upgrade SKILL.md\nwith substantive content]
    C --> E[Remove entry from index.json]
    D --> F[Update entry in index.json\ndescription + instructions + examples]
    E --> G[42 skills removed]
    F --> H[~25 skills improved]
    G --> I[index.json synchronized]
    H --> I
Loading

Reviews (2): Last reviewed commit: "Merge remote-tracking branch 'upstream/m..." | Re-trigger Greptile

@markusha77 markusha77 merged commit 9ce063f into ChatAndBuild:main Mar 29, 2026
3 checks passed
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