Skip to content

fix: preserve article casing in prefixed skill references#52

Merged
pbakaus merged 1 commit intopbakaus:mainfrom
Gujiassh:fix/preserve-prefixed-article-casing
Mar 21, 2026
Merged

fix: preserve article casing in prefixed skill references#52
pbakaus merged 1 commit intopbakaus:mainfrom
Gujiassh:fix/preserve-prefixed-article-casing

Conversation

@Gujiassh
Copy link
Copy Markdown
Contributor

Summary

  • preserve the matched article casing when prefixing the {skill} skill references, so The audit skill becomes The i-audit skill
  • keep the existing lowercase article behavior and /skillname prefixing unchanged
  • update the focused utils regression expectations to cover the mixed-case phrase path

Validation

  • bun test tests/lib/utils.test.js
  • bun test

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)

Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@Gujiassh Gujiassh requested a review from pbakaus as a code owner March 19, 2026 02:53
Copy link
Copy Markdown
Owner

@pbakaus pbakaus left a comment

Choose a reason for hiding this comment

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

Nice catch -- the case-preserving fix is clean and correct. LGTM.

@pbakaus pbakaus merged commit 998dd0c into pbakaus:main Mar 21, 2026
1 check 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.

2 participants