Skip to content

Conversation

@PierreDemailly
Copy link
Member

No description provided.

@PierreDemailly PierreDemailly requested a review from fraxken June 24, 2025 20:01
@fraxken fraxken requested a review from Copilot June 24, 2025 21:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR refactors the project by splitting the codebase into separate workspaces, reorganizing imports, and updating file paths to reflect the new structure.

  • Updated import paths for internal dependencies across server and cache workspaces.
  • Removed deprecated or redundant code such as unused mocks and extraneous configuration options.
  • Added new configuration options (projectRootDir and componentsDir) for improved path resolution.

Reviewed Changes

Copilot reviewed 31 out of 43 changed files in this pull request and generated no comments.

Show a summary per file
File Description
workspaces/server/test/httpServer.test.js Updated imports and adjusted options passed to buildServer.
workspaces/server/test/config.test.js Updated import paths to reference the new config location.
workspaces/server/test/bodyPaser.test.js Corrected import path for bodyParser; file name still appears misspelled.
workspaces/server/src/* Changes reflect updated dependency paths and added configuration options for flexibility.
workspaces/cache/* Refactored cache module imports to use the new workspace structure.
src/commands/* Updated HTTP/server command imports to reference workspaces, removing deprecated code.
package.json, README.md, workflows Adjusted workspace definitions and CI configurations.
Comments suppressed due to low confidence (2)

workspaces/server/test/bodyPaser.test.js:6

  • The file name 'bodyPaser.test.js' appears to be a typo; consider renaming it to 'bodyParser.test.js' for clarity.
import { bodyParser } from "../src/middlewares/bodyParser.js";

workspaces/server/README.md:35

  • The code snippet for setting the data file path appears to be missing a closing quote in the string literal; please update it to include the missing quote.
  "nsecure-result.json

@PierreDemailly PierreDemailly force-pushed the refactor-codebase-into-workspaces branch from a07d138 to 28ff50d Compare June 25, 2025 05:42
@fraxken fraxken merged commit bfd5649 into master Jun 25, 2025
13 checks passed
@fraxken fraxken deleted the refactor-codebase-into-workspaces branch June 25, 2025 13:15
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.

3 participants