gdk-toogle 1.0.2 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: cacce942effef205bff4a9e31cd15889c6e3faba254dd3efee2e238519940147
-  data.tar.gz: f1e506fe72c8dbe264382fa96544a66e232ea0b40fb0bfb4fb14772b3a0e9702
+  metadata.gz: 31e11a727853a15792bcdb3963066222752c19b30234d0dc63db07a360a44472
+  data.tar.gz: ddef44896dfda4944158d2e4204900b3df49d1c0c582973dc0efadc104122f57
 SHA512:
-  metadata.gz: aac129e17fb36ea58e72b2bd12dbf9d90b52d8055ab787146fc3fccaf0eec5cd178d6f1cf21672e0de6fd48aa9ab9ea15fa5a7d328e522ff430286ecf0c49d16
-  data.tar.gz: 83a9942634aab84f4058e743f900f538c5e5bc69c74f8c678c8738b544c96d377ef091cc96302164e33f03bc611db886cd90b10581b47b75a3b5e070ff32173f
+  metadata.gz: abcefb8b6457c45c10d5adb906deb4f25bc01f5202e0ac1cf17acc61da1d6d6c0ef4b32abd4aa8fbcb7e1fc00a6488d5ba0b429028d06e9649f24cbf4ce36052
+  data.tar.gz: d13585fdd22505d237f1166cc7d07b9c0233e66d1e6047ab7d945d68aa43dc08b09a67e8613b33298667e90c7049e5435d40e11d754756cd4ef92c4625e08cf5

data/README.md

@@ -81,7 +81,8 @@ Keep Ruby and gem versions at or below what GitLab itself runs (see
 One design goal of this little Rails engine was to keep it free of any dependencies, that GitLab uses today.
 Because, tomorrow we might decide to ditch one or the other tech we use today, and Toogle shouldn't break just because GitLab removes `sassc`, as an example (that actually happened during development).
 
-That's why Toogle only depends on the bare basics: `rails`, `sprockets` and `haml`.
+That's why Toogle only depends on the bare basics: `rails` and `haml`.
+It doesn't even use the host's asset pipeline: the engine serves its own CSS and JS files.
 
 ### Javascript
 

data/app/assets/javascript/toogle/application.js

@@ -33,25 +33,20 @@ document.addEventListener("alpine:init", () => {
     definitionsUrl,
 
     init() {
-      const featuresLoaded = this.fetchFeatures();
-
-      const definitionsLoaded = fetch(this.definitionsUrl, {
-        method: "GET",
-        headers: {
-          Accept: "application/json",
-        },
-      })
-        .then((response) => response.json())
-        .then((data) => {
-          this.definitions = data;
-        })
-        .catch(showFetchError);
+      const loading = this.reload();
 
       if (shareName) {
-        Promise.all([featuresLoaded, definitionsLoaded]).then(() =>
+        loading.then(() =>
           this.openDetails(shareName)
         );
       }
+
+      this.onReload = () => this.reload();
+      window.addEventListener("reload", this.onReload);
+    },
+
+    destroy() {
+      window.removeEventListener("reload", this.onReload);
     },
 
     openDetails(name) {
@@ -119,6 +114,27 @@ document.addEventListener("alpine:init", () => {
         .catch(showFetchError);
     },
 
+    fetchDefinitions() {
+      return fetch(this.definitionsUrl, {
+        method: "GET",
+        headers: {
+          Accept: "application/json",
+        },
+      })
+        .then((response) => response.json())
+        .then((data) => {
+          this.definitions = data;
+        })
+        .catch(showFetchError);
+    },
+
+    reload() {
+      const featuresLoaded = this.fetchFeatures();
+      const definitionsLoaded = this.fetchDefinitions();
+
+      return Promise.all([featuresLoaded, definitionsLoaded])
+    },
+
     // Animates the layout change, e.g. a row moving between the sections.
     // Only the moving row gets a view-transition-name (see the view), so the
     // browser snapshots one row, not all ~700 — keeping toggles fast no
@@ -246,4 +262,31 @@ document.addEventListener("alpine:init", () => {
       },
     },
   }));
+
+  Alpine.data("gitBranchWatcher", (branch, branchUrl) => ({
+    branch,
+    branchUrl,
+
+    init() {
+      this.$watch("branch", () => {
+        this.$dispatch("reload")
+      });
+
+      window.addEventListener("focus", () => this.fetchCurrentBranch())
+    },
+
+    fetchCurrentBranch() {
+      fetch(this.branchUrl, {
+        method: "GET",
+        headers: {
+          Accept: "application/json",
+        },
+      })
+        .then((response) => response.json())
+        .then((data) => {
+          this.branch = data;
+        })
+        .catch(showFetchError);
+    },
+  }))
 });

data/app/assets/stylesheets/toogle/application.css

@@ -1,10 +1,10 @@
-@import "variables";
+@import "variables.css";
 
-@import "elements";
-@import "layout";
-@import "utilities";
+@import "elements.css";
+@import "layout.css";
+@import "utilities.css";
 
-@import "components/toggle";
+@import "components/toggle.css";
 
 [x-cloak] {
   display: none !important;

data/app/assets/stylesheets/toogle/elements.css

@@ -15,6 +15,10 @@ body {
     position: sticky;
     top: 0;
     transition: 0.2s;
+
+    code {
+      color: var(--color-text-muted);
+    }
   }
 }
 

data/app/controllers/toogle/assets_controller.rb

@@ -0,0 +1,29 @@
+module Toogle
+  # Serves our own CSS and JS instead of going through the host's asset
+  # pipeline: hosts that only serve precompiled assets, or whose web server
+  # answers /assets/* itself, never deliver them otherwise.
+  class AssetsController < ApplicationController
+    ROOT = Engine.root.join("app/assets")
+
+    # Only these files are served, so no path from the URL ever reaches the
+    # filesystem.
+    FILES = Dir[ROOT.join("{stylesheets,javascript}/**/*.{css,js}")]
+      .index_by { Pathname(_1).relative_path_from(ROOT).to_s }
+
+    # Static public files: nothing to protect, and Rails' cross-origin check
+    # would otherwise refuse to serve JavaScript to a <script> tag.
+    skip_forgery_protection
+
+    def show
+      file = FILES[params[:path]]
+      return head :not_found unless file
+
+      # Send the content, not the file: GitLab enables X-Sendfile, which hands
+      # file bodies to Workhorse, and Workhorse may run on another host (as in
+      # Caproni) where this path doesn't exist.
+      send_data File.read(file),
+        type: Mime::Type.lookup_by_extension(File.extname(file).delete(".")),
+        disposition: :inline
+    end
+  end
+end

data/app/controllers/toogle/branches_controller.rb

@@ -0,0 +1,7 @@
+module Toogle
+  class BranchesController < ApplicationController
+    def show
+      render json: GitRepo.current_branch.to_json
+    end
+  end
+end

data/app/services/toogle/git_repo.rb

@@ -0,0 +1,7 @@
+module Toogle
+  class GitRepo
+    def self.current_branch
+      `git rev-parse --abbrev-ref HEAD`.chomp
+    end
+  end
+end

data/app/views/layouts/toogle/application.html.haml

@@ -11,15 +11,17 @@
     %title Feature flags
     = csrf_meta_tags
     = csp_meta_tag
-    = stylesheet_link_tag "toogle/application", media: "all"
-    = javascript_include_tag 'toogle/application'
+    %link(rel="stylesheet" media="all" href="#{asset_file_path("stylesheets/toogle/application.css", v: Toogle::VERSION)}")
+    %script(src="#{asset_file_path("javascript/toogle/application.js", v: Toogle::VERSION)}")
     %script(defer src="https://cdn.jsdelivr.net/npm/alpinejs@3.x.x/dist/cdn.min.js")
 
   %body
     %header
       %nav.d-flex.gap
-        = link_to main_app.root_path do
-          = render partial: "shared/logo", formats: :svg
+        %div(x-data="gitBranchWatcher(#{Toogle::GitRepo.current_branch.to_json}, '#{branch_url}')")
+          = link_to main_app.root_path do
+            = render partial: "shared/logo", formats: :svg
+          %code(x-text="branch" title="Current git branch")
 
       %button(x-data="darkModeSwitcher" x-bind="button")
         %svg(x-show="isDark" width="24" height="24" viewBox="0 0 24 24" fill="none" stroke="currentColor" stroke-width="2" stroke-linecap="round" stroke-linejoin="round")

data/config/routes.rb

@@ -1,4 +1,8 @@
 Toogle::Engine.routes.draw do
+  get "assets/*path", to: "assets#show", format: false, as: :asset_file
+
+  resource :branch, only: %i[show]
+
   resources :definitions, only: %i[index]
 
   # Keep this line last as it matches any URL

data/lib/toogle/version.rb

@@ -1,3 +1,3 @@
 module Toogle
-  VERSION = "1.0.2"
+  VERSION = "1.2.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: gdk-toogle
 version: !ruby/object:Gem::Version
-  version: 1.0.2
+  version: 1.2.0
 platform: ruby
 authors:
 - Thomas Hutterer
@@ -90,7 +90,6 @@ files:
 - MIT-LICENSE
 - README.md
 - Rakefile
-- app/assets/config/toogle_manifest.js
 - app/assets/javascript/toogle/application.js
 - app/assets/stylesheets/toogle/application.css
 - app/assets/stylesheets/toogle/components/toggle.css
@@ -99,10 +98,13 @@ files:
 - app/assets/stylesheets/toogle/utilities.css
 - app/assets/stylesheets/toogle/variables.css
 - app/controllers/toogle/application_controller.rb
+- app/controllers/toogle/assets_controller.rb
+- app/controllers/toogle/branches_controller.rb
 - app/controllers/toogle/definitions_controller.rb
 - app/controllers/toogle/features_controller.rb
 - app/models/toogle/definition.rb
 - app/models/toogle/feature.rb
+- app/services/toogle/git_repo.rb
 - app/views/layouts/toogle/application.html.haml
 - app/views/toogle/features/_details_button.html.haml
 - app/views/toogle/features/index.html.haml
@@ -132,7 +134,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.16
+rubygems_version: 4.0.22
 specification_version: 4
 summary: Toogle toggles feature flags
 test_files: []

data/app/assets/config/toogle_manifest.js

@@ -1,2 +0,0 @@
-//= link toogle/application.css
-//= link toogle/application.js