Skip to content

[BB-8] Transactional Repositories#13

Merged
Ansi007 merged 37 commits intomainfrom
anns/bb-8-transactions-repositories
Dec 16, 2025
Merged

[BB-8] Transactional Repositories#13
Ansi007 merged 37 commits intomainfrom
anns/bb-8-transactions-repositories

Conversation

@Ansi007
Copy link
Collaborator

@Ansi007 Ansi007 commented Dec 12, 2025

Pull Request Title

[BB-8] Transactional Repositories #13


Description

This PR adds the support of transactions in db, it sets up the docker for mongo with replicas and provide implementation and code generation for transactional access to db.


Type of Change

Select all that apply:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that could cause existing functionality to not work as expected)
  • Refactor / Chore (code cleanup, dependencies, tooling, etc.)

How Has This Been Tested?

Manually tested.


Breaking Changes

Re-write repositories.


Checklist

  • My code follows the project’s code style.
  • I have run all linter.
  • I have checked that my changes do not introduce breaking changes.
  • I have commented my code where needed, especially complex logic.

Screenshots / Videos

N/A


Additional Notes

N/A

@github-actions
Copy link

❌ Linting issues found in changed files:

api (apps/api) Linting Issues


/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/plugin.mjs
  0:0  error  Parsing error: /home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/plugin.mjs was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject

/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/require-transactional.mjs
  0:0  error  Parsing error: /home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/require-transactional.mjs was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject

/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/src/modules/crud/repositories/mongoose/crud.mongoose-entity.ts
  10:34  warning  Insert `⏎·`  prettier/prettier

✖ 3 problems (2 errors, 1 warning)
  0 errors and 1 warning potentially fixable with the `--fix` option.

web (apps/web) Linting Issues


/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/web/app/crud-demo/page.tsx
   28:32  warning  Replace `·?·"hover:text-green-400"` with `⏎····?·"hover:text-green-400"⏎···`                                                      prettier/prettier
   31:30  warning  Replace `isMongoose·?·"findAllMongo"·:·"findAllPrisma"` with `⏎····isMongoose·?·"findAllMongo"·:·"findAllPrisma"⏎··`              prettier/prettier
   39:32  warning  Replace `isMongoose·?·"createCrudMongo"·:·"createCrudPrisma"` with `⏎····isMongoose·?·"createCrudMongo"·:·"createCrudPrisma"⏎··`  prettier/prettier
   41:23  warning  Replace `isMongoose·?·"findAllMongo"·:·"findAllPrisma"` with `⏎········isMongoose·?·"findAllMongo"·:·"findAllPrisma"⏎······`      prettier/prettier
   46:32  warning  Replace `isMongoose·?·"deleteCrudMongo"·:·"deleteCrudPrisma"` with `⏎····isMongoose·?·"deleteCrudMongo"·:·"deleteCrudPrisma"⏎··`  prettier/prettier
   47:21  warning  Insert `⏎·····`                                                                                                                   prettier/prettier
   50:32  warning  Replace `isMongoose·?·"updateCrudMongo"·:·"updateCrudPrisma"` with `⏎····isMongoose·?·"updateCrudMongo"·:·"updateCrudPrisma"⏎··`  prettier/prettier
   52:23  warning  Replace `isMongoose·?·"findAllMongo"·:·"findAllPrisma"` with `⏎········isMongoose·?·"findAllMongo"·:·"findAllPrisma"⏎······`      prettier/prettier
  255:69  warning  Insert `⏎···········`                                                                                                             prettier/prettier

✖ 9 problems (0 errors, 9 warnings)
  0 errors and 9 warnings potentially fixable with the `--fix` option.

ESLint found too many warnings (maximum: 0).

Please fix these issues before merging.

@Ansi007 Ansi007 changed the title Anns/bb 8 transactions repositories [BB-8] Transactional Repositories Dec 16, 2025
@Ansi007 Ansi007 self-assigned this Dec 16, 2025
@Ansi007
Copy link
Collaborator Author

Ansi007 commented Dec 16, 2025

/linter

@Ansi007 Ansi007 marked this pull request as ready for review December 16, 2025 05:51
@Ansi007 Ansi007 requested a review from a team as a code owner December 16, 2025 05:51
@github-actions
Copy link

❌ Linting issues found in changed files:

api (apps/api) Linting Issues


/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/plugin.mjs
  0:0  error  Parsing error: /home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/plugin.mjs was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject

/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/require-transaction.mjs
  0:0  error  Parsing error: /home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/eslint-rules/require-transaction.mjs was not found by the project service. Consider either including it in the tsconfig.json or including it in allowDefaultProject

✖ 2 problems (2 errors, 0 warnings)

Please fix these issues before merging.

@Ansi007
Copy link
Collaborator Author

Ansi007 commented Dec 16, 2025

/linter

@github-actions
Copy link

✅ All changed files passed linting checks!

@Ansi007
Copy link
Collaborator Author

Ansi007 commented Dec 16, 2025

/linter

@github-actions
Copy link

✅ All changed files passed linting checks!

@Ansi007
Copy link
Collaborator Author

Ansi007 commented Dec 16, 2025

/linter

@github-actions
Copy link

❌ Linting issues found in changed files:

api (apps/api) Linting Issues


/home/runner/work/be-full-stack-boilerplate/be-full-stack-boilerplate/apps/api/src/modules/auth/auth.service.ts
  8:8  warning  Service class 'AuthService' is missing @AutoTransaction decorator  custom/require-transactional

✖ 1 problem (0 errors, 1 warning)

ESLint found too many warnings (maximum: 0).

Please fix these issues before merging.

@Ansi007
Copy link
Collaborator Author

Ansi007 commented Dec 16, 2025

/linter

@github-actions
Copy link

✅ All changed files passed linting checks!

@Ansi007 Ansi007 merged commit 86bf44d into main Dec 16, 2025
@Ansi007 Ansi007 deleted the anns/bb-8-transactions-repositories branch December 16, 2025 06:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant