Skip to content

feat: add replacements skill#1

Open
43081j wants to merge 3 commits into
mainfrom
replacement-skill
Open

feat: add replacements skill#1
43081j wants to merge 3 commits into
mainfrom
replacement-skill

Conversation

@43081j

@43081j 43081j commented Jun 23, 2026

Copy link
Copy Markdown
Contributor

This basically vendors the module-replacements data set and adds an associated skill.

The reason for the vendoring is so the agent doesn't have to hit an external tool to query. Otherwise, it would have to install the CLI, or install module-replacements.

it may be worth seeing how the modern-web-guidance does this with their data set in case we can do similar.

@mcmxcdev mcmxcdev left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Feels like it's in good enough shape to ship as a first POC that can be iterated on!

Comment thread skills/module-replacements/SKILL.md

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Could benefit from some test coverage

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I tested this skill on a monorepo, and felt like I would rather use CLI/codemod that might be slightly wrong but provide instant feedback. AI consumed a lot of tokens and took many minutes to get something done. The changes generally were pretty clean, with some replacements that would have needed follow ups.

Comment thread skills/module-replacements/match.ts
Comment thread skills/module-replacements/SKILL.md Outdated
Comment thread skills/module-replacements/SKILL.md Outdated
Comment thread skills/module-replacements/SKILL.md Outdated
@43081j 43081j force-pushed the replacement-skill branch from 4866faa to a54f050 Compare June 25, 2026 08:44
@43081j 43081j force-pushed the replacement-skill branch from a54f050 to 4dd699b Compare June 25, 2026 08:47
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.

3 participants