Skip to content

perf(index): skip toCamelCase for auto-named plugins - #307

Merged
Tony133 merged 4 commits into
mainfrom
perf/skip-cc
Sep 12, 2026
Merged

Tony133 merged 4 commits into
mainfrom
perf/skip-cc

Conversation

@Fdawgs

@Fdawgs Fdawgs commented Sep 10, 2026

Copy link
Copy Markdown
Member

toCamelCase() ran for every plugin but its result was discarded when autoName was true.

It should only be called when autoName is false, so a costly regex replace is only paid for plugins that get an alias.

Checklist

Copilot AI 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.

🟢 Approval recommended

The focused optimization preserves existing behavior while avoiding unnecessary regex work.

Pull request overview

Avoids unnecessary camel-case conversion for automatically named plugins.

Changes:

  • Calls toCamelCase() only for explicitly named plugins.
  • Preserves named-import alias behavior.
File summaries
File Description
index.js Moves camel-case conversion inside the explicit-name branch.
Review details
  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Balanced

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@Tony133
Tony133 merged commit 56f07ad into main Sep 12, 2026
20 checks passed
@Tony133
Tony133 deleted the perf/skip-cc branch September 12, 2026 10:32
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