Skip to content

refactor: rename hey-openapi to hey-api#78

Merged
michaelbe812 merged 2 commits intomainfrom
fix/76-rename-hey-openapi-to-hey-api
Jan 21, 2026
Merged

refactor: rename hey-openapi to hey-api#78
michaelbe812 merged 2 commits intomainfrom
fix/76-rename-hey-openapi-to-hey-api

Conversation

@michaelbe812
Copy link
Contributor

Summary

Renames all occurrences of hey-openapi to hey-api as requested in issue #76.

Changes

  • Renamed package directory from plugin-hey-openapi to plugin-hey-api
  • Updated npm package name to @nx-plugin-openapi/plugin-hey-api
  • Updated nx project name to plugin-hey-api
  • Renamed HeyOpenApiGenerator class to HeyApiGenerator
  • Renamed HeyOpenApiOptions interface to HeyApiOptions
  • Renamed HeyOpenApiPlugin export to HeyApiPlugin
  • Updated generator name property from 'hey-openapi' to 'hey-api'
  • Updated BUILTIN_PLUGIN_MAP to use 'hey-api' key
  • Updated demo project to use 'hey-api' generator
  • Added path mapping in tsconfig.base.json
  • Updated all related tests

Test plan

  • Run nx test plugin-hey-api - all 3 tests pass
  • Run nx test core - all 33 tests pass
  • Run nx lint plugin-hey-api - passes
  • Run nx lint core - passes
  • Verified no remaining references to hey-openapi or HeyOpenApi in codebase

Fixes #76

🤖 Generated with Claude Code

…-api

Step 1 of issue #76: Physical file/directory renames
- Updated npm package name to @nx-plugin-openapi/plugin-hey-api
- Updated nx project name to plugin-hey-api
- Renamed HeyOpenApiGenerator to HeyApiGenerator
- Updated BUILTIN_PLUGIN_MAP to use 'hey-api' as the key
- Updated demo project.json to use 'hey-api' generator
- Added path mapping in tsconfig.base.json

Fixes #76
@michaelbe812
Copy link
Contributor Author

Code Review Summary ✅

Verification Results

  • ✅ All 3 tests in plugin-hey-api pass
  • ✅ All 154 tests in core pass
  • ✅ Linting passes for both packages
  • ✅ Build succeeds
  • ✅ No remaining references to hey-openapi or HeyOpenApi in the codebase
  • ✅ Old plugin-hey-openapi directory was properly deleted

Code Quality Assessment

File Renames & Structure:

  • Directory properly renamed from plugin-hey-openapi to plugin-hey-api
  • All file renames follow the convention (hey-openapi-generator.tshey-api-generator.ts)
  • tsconfig.base.json path mapping added correctly

Code Changes:

  • BUILTIN_PLUGIN_MAP correctly updated to use 'hey-api' key
  • Class rename from HeyOpenApiGenerator to HeyApiGenerator is consistent
  • Interface rename from HeyOpenApiOptions to HeyApiOptions is consistent
  • Exports updated properly in index.ts
  • Logger messages updated to use hey-api

Configuration Updates:

  • package.json name updated to @nx-plugin-openapi/plugin-hey-api
  • project.json project name and paths updated
  • jest.config.ts display name and coverage directory updated
  • Demo project.json generator option updated

Tests:

  • Test file properly renamed and updated
  • All test assertions updated to reflect new naming
  • Tests pass successfully

Conclusion

The PR is well-executed and thoroughly addresses issue #76. All naming conventions have been consistently applied throughout the codebase. Ready to merge.

🤖 Reviewed with Claude Code

@michaelbe812 michaelbe812 merged commit 06111d3 into main Jan 21, 2026
1 check passed
@michaelbe812 michaelbe812 deleted the fix/76-rename-hey-openapi-to-hey-api branch January 21, 2026 12:48
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.

Rename hey-openapi to hey-api

1 participant