Feature/464 add soft delete feature

[david-monichi]

No description provided.

[Copilot]

Pull Request Overview

This PR introduces a soft-delete mechanism for SCL files by adding an is_deleted flag, updating repository implementations to respect it, and making the feature togglable via configuration.

• Add is_deleted column to the scl_file table with a Flyway migration.
• Introduce SoftDeleteCompasSclDataPostgreSQLRepository and a producer to switch implementations based on a feature flag.
• Update all queries to exclude soft-deleted records and document the new column.

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 2 comments.

Show a summary per file

File	Description
repository/pom.xml	Added quarkus-arc dependency to support CDI in Quarkus configuration.
repository-postgresql/src/main/resources/.../V1_5__add_soft_delete_flag.sql	Migration to add the is_deleted boolean column with default false.
repository-postgresql/src/main/java/.../SoftDeleteCompasSclDataPostgreSQLRepository.java	New repository variant overriding delete to set the soft-delete flag instead of hard delete.
repository-postgresql/src/main/java/.../CompasSclDataPostgreSQLRepositoryConfiguration.java	CDI producer that selects between the default and soft-delete repository based on a config property.
repository-postgresql/src/main/java/.../CompasSclDataPostgreSQLRepository.java	Added AND scl_file.is_deleted = false to existing queries to filter out soft-deleted records.
doc/postgresql.md	Documented the new is_deleted column in the table schema.
doc/development.md	New guide for running the application locally with Keycloak and PostgreSQL.
app/src/test/resources/docker-compose.yml	Docker Compose setup for Keycloak and PostgreSQL to support local development/testing.
app/src/main/resources/application.properties	Declared the soft-delete-enabled feature flag property.
app/src/main/resources/application-local.properties	Enabled soft delete and Flyway migrations for local development profile.
app/src/main/resources/application-dev-postgresql.properties	Changed default DB credentials for the dev PostgreSQL profile to compas/compas.
README.md	Added link to the new development documentation.

Comments suppressed due to low confidence (2)

doc/postgresql.md:40

• The 'Null' column is marked as 'False', but the migration does not enforce NOT NULL. Either update this to 'True' or add NOT NULL in the migration.

| is_deleted    | boolean      | False    | Flag indicating whether the record is marked as deleted            |

repository-postgresql/src/main/java/org/lfenergy/compas/scl/data/repository/postgresql/SoftDeleteCompasSclDataPostgreSQLRepository.java:27

• [nitpick] Consider adding unit or integration tests to verify that the delete method correctly sets is_deleted and that filtered queries exclude soft-deleted records.

    public void delete(SclFileType type, UUID id) {

[sonarqubecloud]

Quality Gate passed

Issues
2 New issues
0 Accepted issues

Measures
0 Security Hotspots
82.4% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud