Skip to content

feat: skill updates#104

Open
brandonjpierce wants to merge 5 commits into
mainfrom
feat/skill-updates
Open

feat: skill updates#104
brandonjpierce wants to merge 5 commits into
mainfrom
feat/skill-updates

Conversation

@brandonjpierce
Copy link
Copy Markdown
Collaborator

@brandonjpierce brandonjpierce commented May 18, 2026

  • Add skill-creator to repo
  • Update existing 3rd party skills and lockfile
  • Add two new references to two skills
  • Run skill-creator eval loop twice against accelint-react-best-practices

Copy link
Copy Markdown
Collaborator

@belsrc belsrc left a comment

Choose a reason for hiding this comment

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

Obviously fine with the .agents/.claude stuff. And the React addition. Slight concerned with the addition is best-practice. When auditkit is released, the versions in this code base will:

  • Need to be deprecated as the skills are a part of it now (preferred)
  • Will need to keep both in sync (more maintenance)

Also slightly torn on placement given the direct call outs to Next, Webpack and similar. And the likely need to have some sort of auditing specifically for bundling in general that doesn't exactly fit in with strict coding rules.

Thoughts?

@brandonjpierce
Copy link
Copy Markdown
Collaborator Author

Obviously fine with the .agents/.claude stuff. And the React addition. Slight concerned with the addition is best-practice. When auditkit is released, the versions in this code base will:

  • Need to be deprecated as the skills are a part of it now (preferred)
  • Will need to keep both in sync (more maintenance)

Also slightly torn on placement given the direct call outs to Next, Webpack and similar. And the likely need to have some sort of auditing specifically for bundling in general that doesn't exactly fit in with strict coding rules.

Thoughts?

I think the hard part here is that with essentially any modern js setup you have some bundler in play no matter what. So this is meant to be sort of a catch all since all bundlers currently behave the same way with regards to this example.

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.

2 participants