better_structure_sql 0.2.1 → 0.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: d309fdf24a779a29028226f4e0926d28087b9045fdff1b9625522bdac430f169
-  data.tar.gz: db6dbe765d3d385a1b8867d3ad1f54825f7cbeed887ab17b7e02a06f63d5ef48
+  metadata.gz: ef64a880cabbca1d370188561e604a27ecc2e8bd1040b68b123f9cf281b462ee
+  data.tar.gz: c4d5d2e7e366a0f768a8bc2933453958b10f46e72d5f168ea45af91d5b05473b
 SHA512:
-  metadata.gz: 6d2f15acedf160011639a0653dce7c639cf6e825c4e9a9638bf76e3262ee02d1cd76fc6726b3ff79c03a6054953f2310d6f61fd92af87974face17888574640c
-  data.tar.gz: c559251fe213ffabf576ab234ded58970dd54905813db0e6d3c9ee45a951a9435feac53d05eb93c4758df40829b6e91af509eebe647a72349eb036e12df52283
+  metadata.gz: 1d70b3713af668a419e7c9c5fcd66f516b36a50ec31e94b174b893207a7f2017e6ec273942345f191aa8860ab3f517d678eec04d62a32f25d36752c8a89a1b20
+  data.tar.gz: 39fcdb892436045a1579eeea41398fcf8ceb4b28407a192885d4aece4881b170e75724534b6cab30d7f0b58826aa1f5d03a7378f592cbf8d54f248ac98b080df

data/CHANGELOG.md

@@ -13,6 +13,14 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 
+## [0.2.2] - 2025-11-23
+
+### Fixed
+- Web UI now properly displays multi-file schema content for files under 1MB
+- Optimized schema version show page to avoid loading ZIP archives unnecessarily
+- Improved memory efficiency by explicitly selecting only needed columns
+- Better error messages distinguishing "too large" from "not available" content
+
 ## [0.2.1] - 2025-11-20
 
 ### Changed

data/README.md

@@ -16,7 +16,7 @@
 
 </div>
 
-> **⚠️ Beta Notice**: Version 0.2.1 is feature-complete and production-ready for **PostgreSQL**. Multi-database support (MySQL, SQLite) is implemented but considered experimental. APIs are stable but may see minor refinements before v1.0. We welcome feedback and contributions!
+> **⚠️ Beta Notice**: Version 0.2.2 is feature-complete and production-ready for **PostgreSQL**. Multi-database support (MySQL, SQLite) is implemented but considered experimental. APIs are stable but may see minor refinements before v1.0. We welcome feedback and contributions!
 
 ## ✨ Why BetterStructureSql?
 

data/app/controllers/better_structure_sql/schema_versions_controller.rb

@@ -41,20 +41,26 @@ module BetterStructureSql
     # @raise [ActiveRecord::RecordNotFound] if schema version not found
     # GET /better_structure_sql/schema_versions/:id
     def show
-      # Load metadata first
+      # Load metadata first (no content or ZIP to minimize memory usage)
       @schema_version = SchemaVersion
                         .select(:id, :pg_version, :format_type, :output_mode, :created_at,
                                 :content_size, :line_count, :file_count, :content_hash)
                         .find(params[:id])
 
-      # Only load content for small single-file versions
-      if @schema_version.output_mode == 'single_file' && @schema_version.content_size <= MAX_DISPLAY_SIZE
-        @schema_version = SchemaVersion.find(params[:id]) # Load with content
-      elsif @schema_version.output_mode == 'multi_file'
-        # Load content to extract manifest
-        full_version = SchemaVersion.select(:id, :content).find(params[:id])
-        @manifest = extract_manifest_from_content(full_version.content)
+      # Only load content for small files to display inline
+      # For large files, metadata is sufficient
+      if @schema_version.content_size <= MAX_DISPLAY_SIZE
+        # Reload full record with content column
+        # Note: We don't select zip_archive to avoid loading large binary data
+        @schema_version = SchemaVersion.select(
+          :id, :pg_version, :format_type, :output_mode, :created_at,
+          :content_size, :line_count, :file_count, :content_hash, :content
+        ).find(params[:id])
       end
+
+      # NOTE: Manifest extraction intentionally skipped to avoid loading
+      # potentially large ZIP archives into memory. Manifest is primarily
+      # useful for tooling and can be extracted from the download if needed.
     rescue ActiveRecord::RecordNotFound
       render plain: 'Schema version not found', status: :not_found
     end
@@ -177,37 +183,6 @@ module BetterStructureSql
       end
     end
 
-    # Extracts embedded manifest JSON from multi-file schema content
-    #
-    # Manifest is stored between MANIFEST_JSON_START and MANIFEST_JSON_END markers
-    # as SQL comments. Parses and returns the manifest hash.
-    #
-    # @param content [String] the schema content containing embedded manifest
-    # @return [Hash, nil] parsed manifest hash or nil if not found/invalid
-    def extract_manifest_from_content(content)
-      # Manifest is embedded in content between MANIFEST_JSON_START and MANIFEST_JSON_END markers
-      return nil unless content.include?('MANIFEST_JSON_START')
-
-      # Extract JSON from between markers, removing comment prefixes
-      start_marker = '-- MANIFEST_JSON_START'
-      end_marker = '-- MANIFEST_JSON_END'
-
-      start_pos = content.index(start_marker)
-      end_pos = content.index(end_marker)
-
-      return nil unless start_pos && end_pos
-
-      manifest_section = content[(start_pos + start_marker.length)..(end_pos - 1)]
-      manifest_json = manifest_section.lines
-                                      .map { |line| line.sub(/^--\s?/, '') }
-                                      .join
-
-      JSON.parse(manifest_json)
-    rescue JSON::ParserError => e
-      Rails.logger.debug { "Failed to parse manifest: #{e.message}" }
-      nil
-    end
-
     # Streams large file content from database in chunks
     #
     # Sets appropriate headers for streaming downloads and disables proxy buffering.

data/app/views/better_structure_sql/schema_versions/show.html.erb

@@ -121,7 +121,7 @@
           Download File
         <% end %>
       </a>
-      <% if @schema_version.respond_to?(:content) && @schema_version.content.present? %>
+      <% if @schema_version.content.present? %>
         <button type="button" class="btn btn-outline-secondary btn-lg" onclick="copyToClipboard()">
           <i class="bi bi-clipboard"></i>
           Copy to Clipboard
@@ -130,19 +130,16 @@
     </div>
 
     <!-- Multi-file Info -->
-    <% if @schema_version.multi_file? && @manifest %>
+    <% if @schema_version.multi_file? %>
       <div class="alert alert-info mb-4">
         <h6 class="alert-heading">
           <i class="bi bi-info-circle"></i>
           Multi-File Schema
         </h6>
-        <p class="mb-2">This schema was generated in multi-file format with <%= @schema_version.file_count %> files across organized directories.</p>
-        <% if @manifest['directories'] %>
-          <p class="mb-0 small">
-            <strong>Directories:</strong>
-            <%= @manifest['directories'].keys.sort.join(', ') %>
-          </p>
-        <% end %>
+        <p class="mb-0">
+          This schema was generated in multi-file format with <%= @schema_version.file_count %> files across organized directories.
+          Download the ZIP archive to explore the directory structure and individual files.
+        </p>
       </div>
     <% end %>
 
@@ -158,14 +155,18 @@
         </h5>
       </div>
       <div class="card-body p-0">
-        <% if @schema_version.respond_to?(:content) && @schema_version.content.present? %>
+        <% if @schema_version.content.present? %>
           <pre class="code-block m-0" id="schema-content"><%= @schema_version.content %></pre>
         <% else %>
           <div class="alert alert-warning m-3">
             <i class="bi bi-exclamation-triangle"></i>
-            <strong>File too large to display</strong>
+            <strong>Content not loaded</strong>
             <p class="mb-0 mt-2">
-              This schema file is too large to display in the browser (limit: 1 MB).
+              <% if @schema_version.content_size && @schema_version.content_size > 1048576 %>
+                This schema file is too large to display in the browser (limit: 1 MB, actual size: <%= number_to_human_size(@schema_version.content_size) %>).
+              <% else %>
+                The schema content is not available for inline display.
+              <% end %>
               Please use the "Download" button above to download and view it locally.
             </p>
           </div>

data/lib/better_structure_sql/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module BetterStructureSql
-  VERSION = '0.2.1'
+  VERSION = '0.2.2'
 end

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: better_structure_sql
 version: !ruby/object:Gem::Version
-  version: 0.2.1
+  version: 0.2.2
 platform: ruby
 authors:
 - sebyx07
 bindir: exe
 cert_chain: []
-date: 2025-11-20 00:00:00.000000000 Z
+date: 1980-01-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: pg
@@ -294,7 +294,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.6.2
+rubygems_version: 3.7.2
 specification_version: 4
 summary: Clean database schema dumps for Rails (PostgreSQL, MySQL, SQLite) without
   external tool dependencies