✨ Change frontend-only features automatic team assignation rules

The frontend-only features are now ignored from files and from teams
and they do not autoassigns automatically to team and file on file
creation or update operations.

backend/src/app/binfile/common.clj

@@ -445,8 +445,11 @@
                            (fn [features]
                              (let [features (cfeat/check-supported-features! features)]
                                (-> (::features cfg #{})
-                                   (set/difference cfeat/frontend-only-features)
+                                   (set/union features)
-                                   (set/union features))))))
+                                   ;; We never want to store
+                                   ;; frontend-only features on file
+                                   (set/difference cfeat/frontend-only-features))))))
+
 
         _      (when (contains? cf/flags :file-schema-validation)
                  (fval/validate-file-schema! file))

backend/src/app/binfile/v1.clj

@@ -514,6 +514,7 @@
                            (if (set? features)
                              (-> features
                                  (cfeat/migrate-legacy-features)
+                                 (set/difference cfeat/frontend-only-features)
                                  (set/difference cfeat/backend-only-features))
                              #{}))))
 

backend/src/app/rpc/commands/files_create.clj

@@ -111,13 +111,15 @@
         ;; features, because some features can be enabled
         ;; globally, but the team is still not migrated properly.
         features (-> (cfeat/get-team-enabled-features cf/flags team)
-                     (cfeat/check-client-features! (:features params)))
+                     (cfeat/check-client-features! (:features params))
+                     (set/difference cfeat/frontend-only-features))
 
         ;; We also include all no migration features declared by
         ;; client; that enables the ability to enable a runtime
         ;; feature on frontend and make it permanent on file
         features (-> (:features params #{})
                      (set/intersection cfeat/no-migration-features)
+                     (set/difference cfeat/frontend-only-features)
                      (set/union features))
 
         params   (-> params

backend/src/app/rpc/commands/files_update.clj

@@ -139,7 +139,8 @@
 
                           features (-> (cfeat/get-team-enabled-features cf/flags team)
                                        (cfeat/check-client-features! (:features params))
-                                       (cfeat/check-file-features! (:features file) (:features params)))
+                                       (cfeat/check-file-features! (:features file) (:features params))
+                                       (set/difference cfeat/frontend-only-features))
 
                           changes  (if changes-with-metadata
                                      (->> changes-with-metadata (mapcat :changes) vec)

backend/src/app/rpc/commands/teams.clj

@@ -30,7 +30,8 @@
    [app.storage :as sto]
    [app.util.services :as sv]
    [app.util.time :as dt]
-   [app.worker :as wrk]))
+   [app.worker :as wrk]
+   [clojure.set :as set]))
 
 ;; --- Helpers & Specs
 
@@ -416,6 +417,7 @@
                       ::quotes/profile-id profile-id})
 
   (let [features (-> (cfeat/get-enabled-features cf/flags)
+                     (set/difference cfeat/frontend-only-features)
                      (cfeat/check-client-features! (:features params)))
         params   (-> params
                      (assoc :profile-id profile-id)

frontend/src/app/main.cljs

@@ -30,6 +30,7 @@
    [app.util.i18n :as i18n]
    [app.util.theme :as theme]
    [beicon.v2.core :as rx]
+   [cuerdas.core :as str]
    [debug]
    [features]
    [potok.v2.core :as ptk]
@@ -38,11 +39,11 @@
 (log/setup! {:app :info})
 
 (when (= :browser cf/target)
-  (log/info :message "Welcome to penpot"
+  (log/info :version (:full cf/version)
-            :version (:full cf/version)
             :asserts *assert*
             :build-date cf/build-date
-            :public-uri (dm/str cf/public-uri)))
+            :public-uri (dm/str cf/public-uri))
+  (log/info :flags (str/join "," (map name cf/flags))))
 
 (declare reinit)
 

frontend/src/app/main/features.cljs

@@ -33,10 +33,13 @@
 
 (defn get-team-enabled-features
   [state]
-  (-> global-enabled-features
+  (let [runtime-features (:features-runtime state #{})
-      (set/union (:features-runtime state #{}))
+        team-features    (->> (:features-team state #{})
-      (set/intersection cfeat/no-migration-features)
+                              (into #{} cfeat/xf-remove-ephimeral))]
-      (set/union (:features-team state #{}))))
+    (-> global-enabled-features
+        (set/union runtime-features)
+        (set/intersection cfeat/no-migration-features)
+        (set/union team-features))))
 
 (def features-ref
   (l/derived get-team-enabled-features st/state =))
@@ -124,9 +127,9 @@
        (let [features (get-team-enabled-features state)]
          (if (contains? features "render-wasm/v1")
            (render.wasm/initialize true)
-           (render.wasm/initialize false)))
+           (render.wasm/initialize false))
 
-       (log/trc :hint "initialized features"
+         (log/inf :hint "initialized"
-                :team (str/join "," (:features-team state))
+                  :enabled (str/join "," features)
-                :runtime (str/join "," (:features-runtime state)))))))
+                  :runtime (str/join "," (:features-runtime state))))))))