Skip to content

chore: Improve project structure & packages#384

Closed
mlakov wants to merge 1 commit intomainfrom
chore/improve-project-structure
Closed

chore: Improve project structure & packages#384
mlakov wants to merge 1 commit intomainfrom
chore/improve-project-structure

Conversation

@mlakov
Copy link
Copy Markdown
Contributor

@mlakov mlakov commented Mar 10, 2026

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.jslib/extend-ord-with-custom.js: Renamed to kebab-case; simplified module export from a named export object to a direct default export.
  • lib/integrationDependency.jslib/integration-dependency.js: Renamed to kebab-case.
  • lib/interopCsn.jslib/interop-csn.js: Renamed to kebab-case; internal function renamed from interopCSN to interopCsn (exported under the original interopCSN name for backward compatibility); minor whitespace cleanup.
  • lib/metaData.jslib/compile-metadata.js: Renamed to kebab-case with a more descriptive name; main function renamed from getMetadata to compileMetadata.
  • 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 reference compile-metadata.
  • lib/threads/compile.js: Updated import to reference compile-metadata.
  • lib/index.js: Updated getMetadata export to point to the renamed compile-metadata module.
  • __tests__/unit/extend-ord-with-custom.test.js: Renamed (was extendOrdWithCustom.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 an auth/ subdirectory; updated relative import paths accordingly.
  • __tests__/unit/threads/compile.test.js: Updated mock paths to reference compile-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 (interopCSNinteropCsn).
  • 🔄 Regenerate and Update Summary

📬 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 | 💬 Feedback

  • Event Trigger: pull_request.opened
  • LLM: anthropic--claude-4.6-sonnet
  • Output Template: Default Template
  • Correlation ID: bbb350a0-1c6d-11f1-89e1-ce8792caa807
  • Summary Prompt: Default Prompt

Copy link
Copy Markdown
Contributor

@hyperspace-insights hyperspace-insights Bot left a comment

Choose a reason for hiding this comment

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

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:

Comment thread lib/core/interop-csn.js Outdated
@mlakov mlakov force-pushed the chore/improve-project-structure branch 10 times, most recently from 3ad5ac0 to b442277 Compare March 10, 2026 11:23
@mlakov mlakov added run-e2e Trigger e2e test pipeline labels Mar 10, 2026
@mlakov mlakov force-pushed the chore/improve-project-structure branch 2 times, most recently from c1dc151 to 0a372d3 Compare March 10, 2026 11:27
@cap-js cap-js deleted a comment from hyperspace-insights Bot Mar 10, 2026
@mlakov mlakov requested a review from zongqichen March 10, 2026 11:32
@mlakov mlakov force-pushed the chore/improve-project-structure branch 3 times, most recently from 47caf7e to 2031700 Compare March 10, 2026 12:53
@mlakov mlakov added the retry-e2e When the e2e pipeline failed, add retry-e2e label Mar 10, 2026
@ord-e2e-bot ord-e2e-bot Bot removed the retry-e2e When the e2e pipeline failed, add retry-e2e label Mar 10, 2026
@mlakov mlakov requested a review from MariusPerleSAP March 10, 2026 16:32
@mlakov mlakov force-pushed the chore/improve-project-structure branch 3 times, most recently from 4a34167 to 5117f46 Compare March 10, 2026 17:25
@mlakov mlakov force-pushed the chore/improve-project-structure branch from 5117f46 to 6ea0fc7 Compare March 11, 2026 08:16
@mlakov mlakov closed this Mar 30, 2026
@mlakov mlakov deleted the chore/improve-project-structure branch March 30, 2026 12:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

run-e2e Trigger e2e test pipeline

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant