Closed
Conversation
Contributor
There was a problem hiding this comment.
This PR is a clean structural refactor — renaming files to kebab-case and updating all import paths consistently. The changes are mechanical and well-executed, with two minor naming inconsistencies worth addressing: the public getMetadata export key in lib/index.js is now misaligned with the renamed compileMetadata function, and the interopCSN export alias diverges from the newly renamed internal function interopCsn, leaving the codebase with mixed capitalisation conventions for the same symbol.
PR Bot Information
Version: 1.17.99 | 📖 Documentation | 🚨 Create Incident | 💬 Feedback
- Correlation ID:
bbb350a0-1c6d-11f1-89e1-ce8792caa807 - Event Trigger:
pull_request.opened - LLM:
anthropic--claude-4.6-sonnet - Agent Instructions:
3ad5ac0 to
b442277
Compare
c1dc151 to
0a372d3
Compare
47caf7e to
2031700
Compare
4a34167 to
5117f46
Compare
5117f46 to
6ea0fc7
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Improve Project Structure & Rename Files to kebab-case Convention
Refactor
♻️ Renamed several
lib/source files and__tests__/test files to follow a consistent kebab-case naming convention, and simplified module exports where applicable.Changes
lib/extendOrdWithCustom.js→lib/extend-ord-with-custom.js: Renamed to kebab-case; simplified module export from a named export object to a direct default export.lib/integrationDependency.js→lib/integration-dependency.js: Renamed to kebab-case.lib/interopCsn.js→lib/interop-csn.js: Renamed to kebab-case; internal function renamed frominteropCSNtointeropCsn(exported under the originalinteropCSNname for backward compatibility); minor whitespace cleanup.lib/metaData.js→lib/compile-metadata.js: Renamed to kebab-case with a more descriptive name; main function renamed fromgetMetadatatocompileMetadata.lib/ord.js: Updated imports to reference renamed modules (integration-dependency,extend-ord-with-custom) and adjusted destructuring for the simplified export.lib/ord-service.js: Updated import to referencecompile-metadata.lib/threads/compile.js: Updated import to referencecompile-metadata.lib/index.js: UpdatedgetMetadataexport to point to the renamedcompile-metadatamodule.__tests__/unit/extend-ord-with-custom.test.js: Renamed (wasextendOrdWithCustom.test.js); updated import to use the new module path and default export.__tests__/unit/integration-dependency.test.js: Renamed; updated import path.__tests__/unit/interop-csn.test.js: Renamed; updated import path.__tests__/unit/auth/authentication.test.js,cf-mtls.test.js,mtls-endpoint-service.test.js: Moved into anauth/subdirectory; updated relative import paths accordingly.__tests__/unit/threads/compile.test.js: Updated mock paths to referencecompile-metadata.__tests__/unit/__snapshots__/extend-ord-with-custom.test.js.snap,interop-csn.test.js.snap: Renamed to match test file renames..github/copilot-instructions.md: Updated file references to reflect renamed modules.memory-bank/systemPatterns.md,memory-bank/techContext.md: Updated file map references to kebab-case filenames.CHANGELOG.md: Minor text fix (interopCSN→interopCsn).📬 Subscribe to the Hyperspace PR Bot DL to get the latest announcements and pilot features!
PR Bot Information
Version:
1.17.99| 📖 Documentation | 🚨 Create Incident | 💬 Feedbackpull_request.openedanthropic--claude-4.6-sonnetbbb350a0-1c6d-11f1-89e1-ce8792caa807