Remove unnecessary properties

Author: quen2404

src/main/java/com/qdesrame/openapi/diff/compare/schemadiffresult/SchemaDiffResult.java

@@ -15,6 +15,7 @@
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
+import java.util.stream.Collectors;
 
 import static com.qdesrame.openapi.diff.utils.ChangedUtils.isChanged;
 
@@ -51,8 +52,6 @@ public Optional<ChangedSchema> diff(HashSet<String> refSet, Components leftCompo
         Map<String, Schema> leftProperties = null == left ? null : left.getProperties();
         Map<String, Schema> rightProperties = null == right ? null : right.getProperties();
         MapKeyDiff<String, Schema> propertyDiff = MapKeyDiff.diff(leftProperties, rightProperties);
-        Map<String, Schema> increasedProp = propertyDiff.getIncreased();
-        Map<String, Schema> missingProp = propertyDiff.getMissing();
 
         for (String key : propertyDiff.getSharedKey()) {
             Optional<ChangedSchema> resultSchema = openApiDiff.getSchemaDiff().diff(refSet, leftProperties.get(key), rightProperties.get(key), context);
@@ -61,11 +60,22 @@ public Optional<ChangedSchema> diff(HashSet<String> refSet, Components leftCompo
 
         compareAdditionalProperties(refSet, left, right, context);
 
-        changedSchema.getIncreasedProperties().putAll(increasedProp);
-        changedSchema.getMissingProperties().putAll(missingProp);
+        changedSchema.getIncreasedProperties().putAll(filterProperties(propertyDiff.getIncreased(), context));
+        changedSchema.getMissingProperties().putAll(filterProperties(propertyDiff.getMissing(), context));
         return isChanged(changedSchema);
     }
 
+    private Map<String, Schema> filterProperties(Map<String, Schema> properties, DiffContext context) {
+        return properties.entrySet().stream()
+                .filter(entry -> isPropertyApplicable(entry.getValue(), context))
+                .collect(Collectors.toMap(Map.Entry::getKey, Map.Entry::getValue));
+    }
+
+    private boolean isPropertyApplicable(Schema schema, DiffContext context) {
+        return !(context.isResponse() && Boolean.TRUE.equals(schema.getWriteOnly()))
+                && !(context.isRequest() && Boolean.TRUE.equals(schema.getReadOnly()));
+    }
+
     private void compareAdditionalProperties(HashSet<String> refSet, Schema leftSchema, Schema rightSchema, DiffContext context) {
         Object left = leftSchema.getAdditionalProperties();
         Object right = rightSchema.getAdditionalProperties();

src/main/java/com/qdesrame/openapi/diff/output/MarkdownRender.java

@@ -216,15 +216,6 @@ private String schema(ChangedSchema schema) {
         return schema(1, schema);
     }
 
-    private static boolean isDisplayed(ChangedSchema schema) {
-        return isDisplayed(schema.getNewSchema(), schema.getContext());
-    }
-
-    private static boolean isDisplayed(Schema schema, DiffContext context) {
-        return !(Boolean.TRUE.equals(schema.getWriteOnly()) && context.isResponse())
-                || !(Boolean.TRUE.equals(schema.getReadOnly()) && context.isRequest());
-    }
-
     private String oneOfSchema(int deepness, ChangedOneOfSchema schema, String discriminator) {
         StringBuilder sb = new StringBuilder("");
         sb.append(format("%sSwitch `%s`:\n", indent(deepness), discriminator));
@@ -250,9 +241,6 @@ private String required(int deepness, String title, List<String> required) {
 
     private String schema(int deepness, ChangedSchema schema) {
         StringBuilder sb = new StringBuilder("");
-        if (!isDisplayed(schema)) {
-            return sb.toString();
-        }
         if (schema.isDiscriminatorPropertyChanged()) {
             LOGGER.debug("Discriminator property changed");
         }
@@ -292,9 +280,6 @@ private String schema(int deepness, ComposedSchema schema, DiffContext context)
 
     private String schema(int deepness, Schema schema, DiffContext context) {
         StringBuilder sb = new StringBuilder("");
-        if (!isDisplayed(schema, context)) {
-            return sb.toString();
-        }
         sb.append(listItem(deepness, "Enum", schema.getEnum()));
         sb.append(properties(deepness, "Property", schema.getProperties(), true, context));
         if (schema instanceof ComposedSchema) {