[2094] Ensure library documents are read-only

Bug: #2094
Signed-off-by: Gwendal Daniel <gwendal.daniel@obeosoft.com>

Author: gdaniel

CHANGELOG.adoc

@@ -22,6 +22,7 @@ As a result, the following GraphQL mutations have been removed `exposeRequiremen
 === Bug fixes
 
 - https://github.com/eclipse-syson/syson/issues/2045[#2045] [diagrams] In Interconnection View diagrams, fix an issue where the `parts` compartment of a `PartDefinition` graphical node was incorrectly revealed when creating a `PartUsage` from the `PartDefinition` graphical node, even if the `interconnection` compartment was already visible.
+- https://github.com/eclipse-syson/syson/issues/2094[#2094] [libraries] Ensure library documents are read-only.
 
 === Improvements
 

backend/application/syson-application-configuration/src/main/java/org/eclipse/syson/application/publication/SysONSysMLLibraryPublisher.java

@@ -29,6 +29,7 @@
 import org.eclipse.sirius.components.core.api.IEditingContext;
 import org.eclipse.sirius.components.core.api.IPayload;
 import org.eclipse.sirius.components.core.api.SuccessPayload;
+import org.eclipse.sirius.components.emf.ResourceMetadataAdapter;
 import org.eclipse.sirius.components.emf.services.api.IEMFEditingContext;
 import org.eclipse.sirius.components.events.ICause;
 import org.eclipse.sirius.components.representations.Message;
@@ -242,6 +243,14 @@ protected Optional<SemanticData> publishAsLibrary(final ICause parentCause, fina
         // should be determined by its import kind (reference or copy), and not whether it was imported from a textual
         // SysML file in the first place.
         resources.forEach(resource -> ElementUtil.setIsImported(resource, false));
+        // Ensure all the resources are flagged as read-only
+        for (Resource resource : resources) {
+            resource.eAdapters().stream()
+                    .filter(ResourceMetadataAdapter.class::isInstance)
+                    .map(ResourceMetadataAdapter.class::cast)
+                    .findFirst()
+                    .ifPresent(resourceMetadataAdapter -> resourceMetadataAdapter.setIsReadOnly(true));
+        }
 
         final ICause cause = new SysONPublishedLibrarySemanticDataCreationRequested(parentCause, libraryNamespace, libraryName, libraryVersion, libraryDescription);
         return this.createSemanticData(cause, resources, dependencies);

backend/application/syson-application/src/test/java/org/eclipse/syson/application/libraries/publication/SysONLibraryPublicationTests.java

@@ -52,6 +52,7 @@
 import org.eclipse.syson.application.publication.SysONLibraryPublicationHandler;
 import org.eclipse.syson.application.publication.SysONLibraryPublicationListener;
 import org.eclipse.syson.sysml.SysmlPackage;
+import org.eclipse.syson.sysml.util.ElementUtil;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.DisplayName;
 import org.junit.jupiter.api.Test;
@@ -124,11 +125,25 @@ public void givenProjectWhenLibraryIsPublishedThenLibraryExistsAndHasCorrectMeta
         TestTransaction.end();
         TestTransaction.start();
 
-        assertThat(this.librarySearchService.findByNamespaceAndNameAndVersion(SimpleProjectElementsTestProjectData.PROJECT_ID, SimpleProjectElementsTestProjectData.PROJECT_NAME, LIBRARY_VERSION))
+        Optional<Library> optionalLibrary = this.librarySearchService.findByNamespaceAndNameAndVersion(SimpleProjectElementsTestProjectData.PROJECT_ID, SimpleProjectElementsTestProjectData.PROJECT_NAME, LIBRARY_VERSION);
+        assertThat(optionalLibrary)
                 .isPresent()
                 .hasValueSatisfying(library -> assertThat(library.getName()).isEqualTo(SimpleProjectElementsTestProjectData.PROJECT_NAME))
                 .hasValueSatisfying(library -> assertThat(library.getVersion()).isEqualTo(LIBRARY_VERSION))
                 .hasValueSatisfying(library -> assertThat(library.getDescription()).isEqualTo(LIBRARY_DESCRIPTION));
+
+        // Load the editing context manually, we can't use ExecuteEditingContextFunctionInput because the editing context dispatcher doesn't allow mutations on libraries
+        Optional<IEditingContext> optionalEditingContext = this.editingContextSearchService.findById(optionalLibrary.get().getSemanticData().getId().toString());
+        assertThat(optionalEditingContext).isPresent();
+        IEditingContext editingContext = optionalEditingContext.get();
+        if (editingContext instanceof IEMFEditingContext emfEditingContext) {
+            List<Resource> properLibraryResources = emfEditingContext.getDomain().getResourceSet().getResources().stream()
+                    .filter(resource -> !ElementUtil.isStandardLibraryResource(resource))
+                    .toList();
+            assertThat(properLibraryResources).hasSize(1);
+            assertThat(properLibraryResources.getFirst().eAdapters())
+                    .anyMatch(adapter -> adapter instanceof ResourceMetadataAdapter resourceMetadataAdapter && resourceMetadataAdapter.isReadOnly());
+        }
     }
 
     @Test

doc/content/modules/user-manual/pages/release-notes/2026.5.0.adoc

@@ -14,6 +14,10 @@
 
 ** In _Interconnection View_ diagrams, fix an issue where the `parts` compartment of `PartDefinition` graphical node was incorrectly revealed when creating a `PartUsage` from the `PartDefinition` graphical node, even if the `interconnection` compartment was already visible.
 
+* In _Details_ view:
+
+** Fix an issue that allowed to edit library objects via the _Details_ view. 
+
 == Improvements
 
 == Technical details