Skip to content

feat: moved validation pattern to shared reference#157

Open
RuiMiguel wants to merge 3 commits intomainfrom
feat/shared-validation-pattern-to-reference
Open

feat: moved validation pattern to shared reference#157
RuiMiguel wants to merge 3 commits intomainfrom
feat/shared-validation-pattern-to-reference

Conversation

@RuiMiguel
Copy link
Copy Markdown
Contributor

Description

#145

This block appears nearly word-for-word in 3 skills:

build/SKILL.md Phase 2 Step 3 (lines 108-119)
hotfix/SKILL.md Phase 3 Step 3 (lines 99-109)
review/SKILL.md Step 4 auto-fix section (implicit)
Fix: Extract to a shared reference file references/validate-and-fix.md. Each skill references it instead of inlining. Saves ~200 words across skills.

Type of Change

  • New feature (feat)
  • Bug fix (fix)
  • Code refactor (refactor)
  • Documentation (docs)
  • CI change (ci)
  • Chore (chore)

@RuiMiguel RuiMiguel self-assigned this Apr 10, 2026
@RuiMiguel RuiMiguel requested a review from a team as a code owner April 10, 2026 08:06
@RuiMiguel RuiMiguel marked this pull request as draft April 10, 2026 08:18
@RuiMiguel RuiMiguel marked this pull request as ready for review April 10, 2026 08:32
@RuiMiguel RuiMiguel added the performance Changes that improve performance label Apr 10, 2026
Copy link
Copy Markdown
Contributor

@ryzizub ryzizub left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Changes that improve performance

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants