tina4ruby 3.13.7 → 3.13.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6526c615d8d8dd6c8bdba089bceb5969917b2c4400d44886f3c3d757ad4ef14f
-  data.tar.gz: ddebcbd9f847f2660b6ed32e6f7ef86b7e3bd5aa1df3f37f2827d4a2549d9efb
+  metadata.gz: a23c9445f57fd1c84a7bed4e5325a26f94f2234d703f889ffea769faefa5622e
+  data.tar.gz: 7082c808b1d72968997c69167b734cea292e2616c086b46b05831254f30f0353
 SHA512:
-  metadata.gz: 066d4124d585f5e79c3b41e3a763dc655c68d45e5c4afb19c01e0c5dc50fec29f50e21c0fe2eb3b6c8a9941664e45134eb46ef21c42827298c729a77801e766e
-  data.tar.gz: '0148be259f7b00d815abfc04b35f7bbca09e0e5561ff24e72e4a94a4e9e2c7d94236f67949e31d48daba76d383550a12f0c67d690a03a107db7642ef2a9e5570'
+  metadata.gz: f9896ecf33201656f977ceb7bdf4739f2585b14ec32e2c969f61b5ae3c24d69b007ce02d79b053dd96be680af7a5eed8bac8f262fab43e0fc1c51720ff3b2cae
+  data.tar.gz: 9722548ba9dcc819407c20d426154eab388f092424e2d1a9bcd253a877bf79579e5937ee98d319adf2f736aceee0ce57e24c939755a38db070109df9bca76819

data/lib/tina4/ai.rb

@@ -135,6 +135,118 @@ module Tina4
       # @param tool [Hash] tool entry from AI_TOOLS
       # @param context [String] generated context content
       # @return [Array<String>] list of created/updated relative file paths
+      # ── v3.13.9: non-destructive context-file writer ─────────────────
+      #
+      # Pre-v3.13.9 the installer wrote a full developer guide to
+      # CLAUDE.md (and the other context files) on every run, clobbering
+      # whatever the user had put there. Now it writes only a marker-
+      # bracketed Tina4 skill block -- pointing the assistant at
+      # .claude/skills/tina4-*/SKILL.md -- and leaves the rest alone.
+
+      # Return [start, end] markers for the given context file.
+      def markers_for(context_file)
+        if context_file.downcase.end_with?(".md")
+          ["<!-- tina4-skills:start -->", "<!-- tina4-skills:end -->"]
+        else
+          ["# tina4-skills:start", "# tina4-skills:end"]
+        end
+      end
+
+      # Return the marker-bracketed Tina4 skill registration block.
+      def skill_block(context_file)
+        start, finish = markers_for(context_file)
+        body = if context_file.downcase.end_with?(".md")
+          "## Tina4 Skills\n\n" \
+          "When working on this Tina4 project, these skills give the assistant project-aware behaviour:\n\n" \
+          "- **tina4-developer** -- Read `.claude/skills/tina4-developer/SKILL.md` before building features.\n" \
+          "- **tina4-js** -- Read `.claude/skills/tina4-js/SKILL.md` for frontend work.\n" \
+          "- **tina4-maintainer** -- Read `.claude/skills/tina4-maintainer/SKILL.md` for framework-level changes.\n\n" \
+          "See https://tina4.com for full docs."
+        else
+          "Tina4 Skills -- read these files before working on this project:\n" \
+          "  .claude/skills/tina4-developer/SKILL.md   (feature development)\n" \
+          "  .claude/skills/tina4-js/SKILL.md          (frontend / tina4-js)\n" \
+          "  .claude/skills/tina4-maintainer/SKILL.md  (framework-level changes)\n" \
+          "Docs: https://tina4.com"
+        end
+        "#{start}\n#{body}\n#{finish}"
+      end
+
+      # True iff both start and end markers appear in order.
+      def has_markers?(existing, start, finish)
+        s_idx = existing.index(start)
+        return false unless s_idx
+        !existing.index(finish, s_idx + start.length).nil?
+      end
+
+      # Replace the bracketed block in `existing` with `block`.
+      def replace_marker_block(existing, block, start, finish)
+        s_idx = existing.index(start)
+        return existing.rstrip + "\n\n" + block + "\n" unless s_idx
+        e_idx = existing.index(finish, s_idx + start.length)
+        return existing.rstrip + "\n\n" + block + "\n" unless e_idx
+        before = existing[0...s_idx].rstrip
+        after = existing[(e_idx + finish.length)..].sub(/\A\n+/, "")
+        glue_before = before.empty? ? "" : "\n\n"
+        glue_after = after.empty? ? "\n" : "\n" + after
+        "#{before}#{glue_before}#{block}#{glue_after}"
+      end
+
+      # Headers the pre-v3.13.9 installer wrote at the top of CLAUDE.md.
+      OLD_FRAMEWORK_HEADERS = [
+        "# Tina4 Python",
+        "# Tina4 PHP",
+        "# Tina4 Ruby",
+        "# CLAUDE.md -- AI Developer Guide for tina4-nodejs",
+        "# CLAUDE.md — AI Developer Guide for tina4-nodejs",
+      ].freeze
+
+      def looks_like_old_framework_install?(existing)
+        head = existing.lstrip[0, 400] || ""
+        OLD_FRAMEWORK_HEADERS.any? { |h| head.start_with?(h) }
+      end
+
+      # Write the context file non-destructively. Returns a human-readable
+      # action verb for the caller's log line.
+      #
+      # Four branches:
+      #   1. Doesn't exist  -> write framework guide + skill block
+      #   2. Has markers    -> refresh just the skill block (idempotent)
+      #   3. Old header     -> migrate: replace old dump with new guide + block
+      #   4. User content   -> append the skill block, preserve everything else
+      def write_or_merge(context_path, context_file, framework_guide)
+        # Force UTF-8 — CLAUDE.md and the framework guide both contain
+        # non-ASCII (em-dashes, ✓, etc.). Without this, File.read may
+        # return ASCII-8BIT and string concat raises CompatibilityError.
+        block = skill_block(context_file).dup.force_encoding("UTF-8")
+        guide = framework_guide.dup.force_encoding("UTF-8")
+        start, finish = markers_for(context_file)
+
+        unless File.exist?(context_path)
+          File.write(context_path, guide.rstrip + "\n\n" + block + "\n", encoding: "UTF-8")
+          return "Installed"
+        end
+
+        existing = File.read(context_path, encoding: "UTF-8")
+
+        if has_markers?(existing, start, finish)
+          File.write(context_path, replace_marker_block(existing, block, start, finish), encoding: "UTF-8")
+          return "Refreshed skill block in"
+        end
+
+        if looks_like_old_framework_install?(existing)
+          head = existing.lstrip
+          preamble = existing[0, existing.length - head.length] || ""
+          new_content = (preamble.strip.empty? ? "" : preamble.rstrip + "\n\n") +
+                        guide.rstrip + "\n\n" + block + "\n"
+          File.write(context_path, new_content, encoding: "UTF-8")
+          return "Migrated (replaced old framework dump in)"
+        end
+
+        File.write(context_path, existing.rstrip + "\n\n" + block + "\n", encoding: "UTF-8")
+        "Appended skill block to"
+      end
+
       def install_for_tool(root, tool, context)
         created = []
         context_path = File.join(root, tool[:context_file])
@@ -145,9 +257,8 @@ module Tina4
         end
         FileUtils.mkdir_p(File.dirname(context_path))
 
-        # Always overwrite -- user chose to install
-        action = File.exist?(context_path) ? "Updated" : "Installed"
-        File.write(context_path, context)
+        # v3.13.9: non-destructive write -- see write_or_merge below.
+        action = write_or_merge(context_path, tool[:context_file], context)
         rel = context_path.sub("#{root}/", "")
         created << rel
         puts "  \e[32m✓\e[0m #{action} #{rel}"

data/lib/tina4/orm.rb

@@ -302,13 +302,27 @@ module Tina4
       def create_table
         return true if db.table_exists?(table_name)
 
+        # v3.13.11 (BooleanField parity): pick each engine's native
+        # bool type where it's reliable. SQLite has no native bool;
+        # Firebird's driver round-trip for native BOOLEAN is uneven —
+        # both stay on INTEGER. PG/MySQL/MSSQL use their native types
+        # so Python ``True``/Ruby ``true`` bind cleanly without
+        # ``operator does not exist: boolean = integer`` errors.
+        engine = (db.respond_to?(:get_database_type) ? db.get_database_type : "").to_s.downcase
+        bool_sql = case engine
+                   when "postgres", "postgresql" then "BOOLEAN"
+                   when "mysql" then "BOOLEAN" # alias for TINYINT(1)
+                   when "mssql", "sqlserver" then "BIT"
+                   else "INTEGER" # sqlite, firebird, odbc, anything else
+                   end
+
         type_map = {
           integer: "INTEGER",
           string: "VARCHAR(255)",
           text: "TEXT",
           float: "REAL",
           decimal: "REAL",
-          boolean: "INTEGER",
+          boolean: bool_sql,
           date: "DATE",
           datetime: "DATETIME",
           timestamp: "TIMESTAMP",
@@ -419,10 +433,18 @@ module Tina4
         setter = "#{key}="
         __send__(setter, value) if respond_to?(setter)
       end
-      # Set defaults
+      # Set defaults.
+      # v3.13.11 (issue #50.1): when the default is a Proc/lambda
+      # (``default: -> { Time.now }``), call it per-instance so
+      # per-row timestamps actually differ. Class objects are
+      # excluded — ``default: Integer`` is almost never intended
+      # to mean ``Integer.new`` (and Integer has no zero-arg
+      # constructor anyway).
       self.class.field_definitions.each do |name, opts|
         if __send__(name).nil? && opts[:default]
-          __send__("#{name}=", opts[:default])
+          d = opts[:default]
+          d = d.call if d.respond_to?(:call) && !d.is_a?(Class)
+          __send__("#{name}=", d)
         end
       end
     end

data/lib/tina4/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Tina4
-  VERSION = "3.13.7"
+  VERSION = "3.13.11"
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: tina4ruby
 version: !ruby/object:Gem::Version
-  version: 3.13.7
+  version: 3.13.11
 platform: ruby
 authors:
 - Tina4 Team
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-06-10 00:00:00.000000000 Z
+date: 2026-06-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rack