Clean code

.gitignore

@@ -1,6 +1,6 @@
 node_modules/
 reports/
-build/
+dist/
 generated/prisma/
 
 ### JetBrains template

cucumber.js

@@ -0,0 +1,9 @@
+let common = [
+  "features/**/*.feature",
+  "--require-module ts-node/register",
+  "--require features/**/*.ts",
+].join(" ");
+
+module.exports = {
+  default: common,
+};

eslint.config.js → eslint.config.mjs

@@ -1,10 +1,10 @@
 import eslint from "@eslint/js";
-import prettierConfig from "eslint-config-prettier";
 import tseslint from "typescript-eslint";
+import prettierConfig from "eslint-config-prettier";
 
 export default tseslint.config(
   {
-    ignores: ["build/**"],
+    ignores: ["dist/**", "node_modules/**", "**/*.js"],
   },
   eslint.configs.recommended,
   tseslint.configs.recommended,

features/steps/common_steps.ts

@@ -1,10 +1,10 @@
 import { Given } from "@cucumber/cucumber";
-import { User } from "../../src/Domain/Models/User.js";
-import { Vehicle } from "../../src/Domain/Models/Vehicle.js";
-import { InMemoryFleetRepository } from "../../src/Secondary/Repositories/InMemoryFleetRepository.js";
-import { initializeFleetForUser } from "./shared/initializeFleetForUser.js";
-import { registerVehicleInUserFleet } from "./register_vehicle_steps.js";
-import { generateFrenchPlateNumber } from "../../tests/Utils/generateFrenchPlateNumber.js";
+import { User } from "../../src/Domain/Models/User";
+import { Vehicle } from "../../src/Domain/Models/Vehicle";
+import { InMemoryFleetRepository } from "../../src/Infra/Secondary/Repositories/InMemoryFleetRepository";
+import { initializeFleetForUser } from "./shared/initializeFleetForUser";
+import { registerVehicleInUserFleet } from "./register_vehicle_steps";
+import { generateFrenchPlateNumber } from "../../tests/Utils/generateFrenchPlateNumber";
 
 Given("my fleet", async function (): Promise<void> {
   const user: User = User.create(crypto.randomUUID());

features/steps/park_vehicle_steps.ts

@@ -3,12 +3,12 @@ import { Given, Then, When } from "@cucumber/cucumber";
 import { expect } from "chai";
 
 // Second group: Domain
-import { Location } from "../../src/Domain/Models/Location.js";
-import { VehicleAlreadyParkedAtThisLocationError } from "../../src/Domain/Errors/VehicleAlreadyParkedAtThisLocationError.js";
+import { Location } from "../../src/Domain/Models/Location";
+import { VehicleAlreadyParkedAtThisLocationError } from "../../src/Domain/Errors/VehicleAlreadyParkedAtThisLocationError";
 
 // Third group: Helpers
-import { parkVehicleAtLocation } from "./shared/parkVehicleAtLocation.js";
-import { retrieveLocation } from "./shared/retrieveLocation.js";
+import { parkVehicleAtLocation } from "./shared/parkVehicleAtLocation";
+import { retrieveLocation } from "./shared/retrieveLocation";
 
 Given("a location", async function (): Promise<void> {
   this.context.location = Location.create(48.8566, 2.3522);
@@ -71,9 +71,7 @@ Then(
     const expected = new VehicleAlreadyParkedAtThisLocationError(
       this.context.vehicle.plateNumber,
       this.context.fleetId,
-      this.context.location.latitude,
-      this.context.location.longitude,
-      this.context.location.altitude,
+      this.context.location,
     );
 
     expect(this.context.parkingAttemptError).to.deep.equal(expected);

features/steps/register_vehicle_steps.ts

@@ -4,17 +4,17 @@ import { World } from "cucumber";
 import { expect } from "chai";
 
 // Second group: Domain
-import { User } from "../../src/Domain/Models/User.js";
-import { Vehicle } from "../../src/Domain/Models/Vehicle.js";
-import { VehicleAlreadyRegisteredError } from "../../src/Domain/Errors/VehicleAlreadyRegisteredError.js";
+import { User } from "../../src/Domain/Models/User";
+import { Vehicle } from "../../src/Domain/Models/Vehicle";
+import { VehicleAlreadyRegisteredError } from "../../src/Domain/Errors/VehicleAlreadyRegisteredError";
 
 // Third group: Helpers
-import { initializeFleetForUser } from "./shared/initializeFleetForUser.js";
-import { retrieveFleet } from "./shared/retrieveFleet.js";
+import { initializeFleetForUser } from "./shared/initializeFleetForUser";
+import { retrieveFleet } from "./shared/retrieveFleet";
 import {
   RegisterVehicle,
   RegisterVehicleHandler,
-} from "../../src/App/Commands/registerVehicle.js";
+} from "../../src/App/Commands/registerVehicle";
 
 Given("the fleet of another user", async function (): Promise<void> {
   this.context.otherUser = User.create(crypto.randomUUID());

features/steps/shared/initializeFleetForUser.ts

@@ -1,9 +1,9 @@
-import { FleetRepository } from "../../../src/Domain/Repositories/FleetRepository.js";
-import { User } from "../../../src/Domain/Models/User.js";
+import { FleetRepository } from "../../../src/Domain/Ports/FleetRepository";
 import {
   InitializeFleet,
   InitializeFleetHandler,
-} from "../../../src/App/Commands/initializeFleet.js";
+} from "../../../src/App/Commands/initializeFleet";
+import { User } from "../../../src/Domain/Models/User";
 
 export async function initializeFleetForUser(
   repository: FleetRepository,

features/steps/shared/parkVehicleAtLocation.ts

@@ -1,9 +1,9 @@
-import { Location } from "../../../src/Domain/Models/Location.js";
-import { FleetRepository } from "../../../src/Domain/Repositories/FleetRepository.js";
+import { Location } from "../../../src/Domain/Models/Location";
+import { FleetRepository } from "../../../src/Domain/Ports/FleetRepository";
 import {
   ParkVehicle,
   ParkVehicleHandler,
-} from "../../../src/App/Commands/parkVehicle.js";
+} from "../../../src/App/Commands/parkVehicle";
 
 export async function parkVehicleAtLocation(
   repository: FleetRepository,

features/steps/shared/retrieveFleet.ts

@@ -1,9 +1,6 @@
-import { Fleet } from "../../../src/Domain/Models/Fleet.js";
-import { FleetRepository } from "../../../src/Domain/Repositories/FleetRepository.js";
-import {
-  GetFleet,
-  GetFleetHandler,
-} from "../../../src/App/Queries/getFleet.js";
+import { Fleet } from "../../../src/Domain/Models/Fleet";
+import { FleetRepository } from "../../../src/Domain/Ports/FleetRepository";
+import { GetFleet, GetFleetHandler } from "../../../src/App/Queries/getFleet";
 
 export async function retrieveFleet(
   repository: FleetRepository,

features/steps/shared/retrieveLocation.ts

@@ -1,9 +1,9 @@
-import { Location } from "../../../src/Domain/Models/Location.js";
-import { FleetRepository } from "../../../src/Domain/Repositories/FleetRepository.js";
+import { Location } from "../../../src/Domain/Models/Location";
+import { FleetRepository } from "../../../src/Domain/Ports/FleetRepository";
 import {
   GetLocation,
   GetLocationHandler,
-} from "../../../src/App/Queries/getLocation.js";
+} from "../../../src/App/Queries/getLocation";
 
 export async function retrieveLocation(
   repository: FleetRepository,

package.json

@@ -1,16 +1,15 @@
 {
-  "type": "module",
   "name": "fulll-backend",
   "version": "1.0.0",
   "license": "ISC",
   "dependencies": {
+    "@eslint/js": "^9.27.0",
     "@prisma/client": "^6.8.2",
     "chalk": "^5.4.1",
     "yargs": "^17.7.2"
   },
   "devDependencies": {
     "@cucumber/cucumber": "^11.2.0",
-    "@eslint/js": "^9.25.1",
     "@types/chai": "^5.2.1",
     "@types/cucumber": "^6.0.1",
     "@types/node": "^22.14.1",
@@ -21,17 +20,16 @@
     "prettier": "^3.5.3",
     "prisma": "^6.8.2",
     "ts-node": "^10.9.2",
-    "tsx": "^4.19.4",
     "typescript": "^5.8.3",
     "typescript-eslint": "^8.31.0",
     "vitest": "^3.1.2"
   },
   "scripts": {
     "generate": "prisma generate --no-hints",
     "build": "tsc",
-    "prepare": "npm run generate && npm run build",
-    "fleet": "npm run prepare && tsx ./build/src/cli.js",
-    "cucumber": "cucumber-js --config cucumber.json",
+    "prepare": "rm -rf dist/ && npm run generate && npm run build",
+    "fleet": "npm run prepare && ts-node ./dist/src/cli.js",
+    "cucumber": "cucumber-js",
     "cucumber:critical": "cucumber-js --tags \"@critical\"",
     "lint": "eslint .",
     "test": "vitest"

src/App/Commands/initializeFleet.ts

@@ -1,6 +1,6 @@
-import { Command, CommandHandler } from "./command.js";
-import { Fleet } from "../../Domain/Models/Fleet.js";
-import { FleetRepository } from "../../Domain/Repositories/FleetRepository.js";
+import { Command, CommandHandler } from "./command";
+import { Fleet } from "../../Domain/Models/Fleet";
+import { FleetRepository } from "../../Domain/Ports/FleetRepository";
 
 export class InitializeFleet implements Command {
   constructor(public readonly userId: string) {}

src/App/Commands/parkVehicle.ts

@@ -1,7 +1,7 @@
-import { Command, CommandHandler } from "./command.js";
-import { Location } from "../../Domain/Models/Location.js";
-import { FleetRepository } from "../../Domain/Repositories/FleetRepository.js";
-import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError.js";
+import { Command, CommandHandler } from "./command";
+import { Location } from "../../Domain/Models/Location";
+import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError";
+import { FleetRepository } from "../../Domain/Ports/FleetRepository";
 
 export class ParkVehicle implements Command {
   constructor(

src/App/Commands/registerVehicle.ts

@@ -1,6 +1,6 @@
-import { Command, CommandHandler } from "./command.js";
-import { FleetRepository } from "../../Domain/Repositories/FleetRepository.js";
-import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError.js";
+import { Command, CommandHandler } from "./command";
+import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError";
+import { FleetRepository } from "../../Domain/Ports/FleetRepository";
 
 export class RegisterVehicle implements Command {
   constructor(

src/App/Queries/getFleet.ts

@@ -1,7 +1,7 @@
-import { Query, QueryHandler } from "./query.js";
-import { Fleet } from "../../Domain/Models/Fleet.js";
-import { FleetRepository } from "../../Domain/Repositories/FleetRepository.js";
-import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError.js";
+import { Query, QueryHandler } from "./query";
+import { Fleet } from "../../Domain/Models/Fleet";
+import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError";
+import { FleetRepository } from "../../Domain/Ports/FleetRepository";
 
 export class GetFleet implements Query {
   constructor(public readonly fleetId: string) {}

src/App/Queries/getLocation.ts

@@ -1,7 +1,7 @@
-import { Query, QueryHandler } from "./query.js";
-import { Location } from "../../Domain/Models/Location.js";
-import { FleetRepository } from "../../Domain/Repositories/FleetRepository.js";
-import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError.js";
+import { Query, QueryHandler } from "./query";
+import { Location } from "../../Domain/Models/Location";
+import { FleetNotFoundError } from "../../Domain/Errors/FleetNotFoundError";
+import { FleetRepository } from "../../Domain/Ports/FleetRepository";
 
 export class GetLocation implements Query {
   constructor(

src/Domain/Errors/VehicleAlreadyParkedAtThisLocationError.ts

@@ -1,13 +1,9 @@
+import { Location } from "../Models/Location.js";
+
 export class VehicleAlreadyParkedAtThisLocationError extends Error {
-  constructor(
-    vehiclePlateNumber: string,
-    fleetId: string,
-    latitude: number,
-    longitude: number,
-    altitude: number,
-  ) {
+  constructor(vehiclePlateNumber: string, fleetId: string, location: Location) {
     super(
-      `Error: Vehicle ${vehiclePlateNumber} of ${fleetId} is already parked at latitude: ${latitude}, longitude: ${longitude}, altitude: ${altitude}`,
+      `Error: Vehicle ${vehiclePlateNumber} of ${fleetId} is already parked at latitude: ${location.latitude}, longitude: ${location.longitude}, altitude: ${location.altitude}`,
     );
     this.name = "VehicleAlreadyParkedAtThisLocationError";
   }