Skip to content

refactor(codegen): import toKebabCase from inflekt, drop komoji dep#922

Merged
pyramation merged 2 commits intomainfrom
devin/1774655458-drop-komoji-from-codegen
Mar 28, 2026
Merged

refactor(codegen): import toKebabCase from inflekt, drop komoji dep#922
pyramation merged 2 commits intomainfrom
devin/1774655458-drop-komoji-from-codegen

Conversation

@pyramation
Copy link
Copy Markdown
Contributor

Summary

Switches all 6 toKebabCase imports in graphql/codegen from komoji to inflekt (which re-exports it from komoji as of v0.7.0). Removes komoji as a direct dependency of the codegen package and bumps inflekt from ^0.6.0 to ^0.7.0.

The implementation is identical — inflekt re-exports toKebabCase directly from komoji with no wrapping. This eliminates codegen's last direct komoji import.

Files changed:

  • cli/command-map-generator.ts
  • cli/custom-command-generator.ts
  • cli/docs-generator.ts
  • cli/table-command-generator.ts
  • hooks-docs-generator.ts
  • orm/docs-generator.ts
  • package.json (bump inflekt, remove komoji)

The lockfile diff is large but almost entirely pnpm YAML formatting changes; the only semantic changes are the inflekt version bump and komoji removal from codegen's direct deps.

Review & Testing Checklist for Human

  • Confirm inflekt@0.7.0 is published on npm and includes the toKebabCase export (it was published earlier in this session)
  • Run pnpm test in graphql/codegen to verify codegen tests pass (workspace build passed but unit tests are a separate signal)
  • Grep for any remaining from 'komoji' imports in graphql/codegen/ to confirm none were missed

Notes

  • komoji remains a transitive dependency (via inflekt), so it's still in the lockfile — just no longer a direct dep of codegen.

Link to Devin session: https://app.devin.ai/sessions/01424b7bd5a54c55acac65af34236896
Requested by: @pyramation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@pyramation pyramation merged commit a236c7f into main Mar 28, 2026
44 checks passed
@pyramation pyramation deleted the devin/1774655458-drop-komoji-from-codegen branch March 28, 2026 00:05
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.

1 participant