collavre 0.1.1 → 0.2.0

data/app/services/collavre/notion_service.rb

@@ -1,249 +0,0 @@
-module Collavre
-  require "digest"
-
-  class NotionService
-    def initialize(user:)
-      @user = user
-      @account = user.notion_account
-      raise NotionAuthError, "No Notion account found" unless @account
-    end
-
-    def client
-      @client ||= NotionClient.new(@account)
-    end
-
-    def search_pages(query: nil, start_cursor: nil, page_size: 10)
-      with_token_refresh { client.search_pages(query: query, start_cursor: start_cursor, page_size: page_size) }
-    end
-
-    def get_page(page_id)
-      with_token_refresh { client.get_page(page_id) }
-    end
-
-    def create_page(parent_id:, title:, blocks: [])
-      with_token_refresh { client.create_page(parent_id: parent_id, title: title, blocks: blocks) }
-    end
-
-    def update_page(page_id, properties: {}, blocks: nil)
-      with_token_refresh { client.update_page(page_id, properties: properties, blocks: blocks) }
-    end
-
-    def get_page_blocks(page_id, start_cursor: nil, page_size: 100)
-      with_token_refresh { client.get_page_blocks(page_id, start_cursor: start_cursor, page_size: page_size) }
-    end
-
-    def replace_page_blocks(page_id, blocks)
-      with_token_refresh { client.replace_page_blocks(page_id, blocks) }
-    end
-
-    def append_blocks(parent_id, blocks)
-      with_token_refresh { client.append_blocks(parent_id, blocks) }
-    end
-
-    def delete_block(block_id)
-      with_token_refresh { client.delete_block(block_id) }
-    end
-
-    def get_workspace
-      with_token_refresh { client.get_workspace }
-    end
-
-    # Create or update a page for a creative
-    def sync_creative(creative, parent_page_id: nil)
-      notion_link = find_or_create_page_link(creative, parent_page_id)
-
-      if notion_link.page_id.present?
-        # Update existing page
-        update_creative_page(creative, notion_link)
-      else
-        # Create new page
-        create_creative_page(creative, notion_link, parent_page_id)
-      end
-
-      notion_link.mark_synced!
-      notion_link
-    end
-
-    private
-
-    def with_token_refresh(&block)
-      yield
-    rescue NotionAuthError => e
-      if refresh_token!
-        @client = nil # Reset client with new token
-        yield
-      else
-        raise e
-      end
-    end
-
-    def refresh_token!
-      # Notion uses OAuth 2.0 but doesn't issue refresh tokens in the same way as Google
-      # For now, we'll just log the error and return false
-      # In a production app, you'd implement proper token refresh logic here
-      Rails.logger.error("Notion token refresh needed but not implemented")
-      false
-    end
-
-    def find_or_create_page_link(creative, parent_page_id)
-      @account.notion_page_links.find_or_initialize_by(creative: creative) do |link|
-        link.parent_page_id = parent_page_id
-      end
-    end
-
-    def create_creative_page(creative, notion_link, parent_page_id)
-      title = ActionController::Base.helpers.strip_tags(creative.description).strip.presence || "Untitled Creative"
-
-      # Export only the children - the page title serves as the root creative
-      children = creative.children.to_a
-      Rails.logger.info("NotionService: Exporting creative #{creative.id} as page title with #{children.count} children as blocks")
-
-      exporter = NotionCreativeExporter.new(creative)
-      blocks = []
-
-      # If no parent specified, search for a suitable workspace page
-      parent_page_id ||= find_default_parent_page
-
-      response = create_page(
-        parent_id: parent_page_id,
-        title: title,
-        blocks: blocks
-      )
-
-      notion_link.update!(
-        page_id: response["id"],
-        page_title: title,
-        page_url: response["url"],
-        parent_page_id: parent_page_id
-      )
-
-      sync_child_blocks(notion_link, creative, children, exporter)
-
-      response
-    end
-
-    def update_creative_page(creative, notion_link)
-      title = ActionController::Base.helpers.strip_tags(creative.description).strip.presence || "Untitled Creative"
-
-      # Update with only the children - page title serves as the root creative
-      children = creative.children.to_a
-      Rails.logger.info("NotionService: Updating creative #{creative.id} as page title with #{children.count} children as blocks")
-
-      exporter = NotionCreativeExporter.new(creative)
-
-      properties = {
-        title: {
-          title: [ { text: { content: title } } ]
-        }
-      }
-
-      update_page(notion_link.page_id, properties: properties)
-      notion_link.update!(page_title: title)
-
-      sync_child_blocks(notion_link, creative, children, exporter)
-    end
-
-    def find_default_parent_page
-      # Search for pages in the workspace to find a suitable parent
-      pages = search_pages(page_size: 1)
-      pages.dig("results")&.first&.dig("id") || raise(NotionError, "No accessible pages found in workspace")
-    end
-
-    def sync_child_blocks(notion_link, creative, children, exporter)
-      child_ids = children.map(&:id)
-      existing_links = notion_link.notion_block_links.includes(:creative).order(:created_at).to_a
-      existing_links_by_creative = existing_links.group_by(&:creative_id)
-
-      page_blocks = existing_links.any? ? fetch_all_page_blocks(notion_link.page_id) : []
-      page_block_ids = page_blocks.map { |block| block["id"] }
-
-      block_to_creative = {}
-      existing_links.each do |link|
-        block_to_creative[link.block_id] = link.creative_id if page_block_ids.include?(link.block_id)
-      end
-
-      existing_order = page_block_ids.map { |block_id| block_to_creative[block_id] }.compact.uniq
-      expected_order = child_ids.select { |id| existing_links_by_creative.key?(id) }
-      reorder_detected = existing_order != expected_order
-
-      removed_ids = existing_links_by_creative.keys - child_ids
-      changes_detected = removed_ids.any?
-
-      child_exports = children.map do |child|
-        exported_blocks = exporter.export_tree_blocks([ child ], 1, 0)
-        content_hash = Digest::SHA256.hexdigest(exported_blocks.to_json)
-        links = existing_links_by_creative[child.id] || []
-        missing_blocks = links.any? { |link| !page_block_ids.include?(link.block_id) }
-
-        if exported_blocks.empty?
-          changes_detected ||= links.present?
-        else
-          changes_detected ||= links.blank?
-          changes_detected ||= links.size != exported_blocks.size
-          changes_detected ||= links.first.content_hash != content_hash
-          changes_detected ||= missing_blocks
-        end
-
-        {
-          child: child,
-          exported_blocks: exported_blocks,
-          content_hash: content_hash
-        }
-      end
-
-      unless changes_detected || reorder_detected
-        return
-      end
-
-      blocks_to_clear = page_blocks.presence || fetch_all_page_blocks(notion_link.page_id)
-      blocks_to_clear.each do |block|
-        begin
-          delete_block(block["id"])
-        rescue NotionError => e
-          Rails.logger.warn("NotionService: Failed to delete Notion block #{block['id']} during resync: #{e.message}")
-        end
-      end
-
-      NotionBlockLink.transaction do
-        notion_link.notion_block_links.delete_all
-
-        child_exports.each do |data|
-          exported_blocks = data[:exported_blocks]
-          next if exported_blocks.empty?
-
-          response = append_blocks(notion_link.page_id, exported_blocks)
-          response_results = response.is_a?(Hash) ? response.fetch("results", []) : []
-          new_block_ids = response_results.filter_map { |result| result["id"] }
-
-          if new_block_ids.empty?
-            Rails.logger.warn("NotionService: Unable to determine new block ids for creative #{data[:child].id}")
-            next
-          end
-
-          new_block_ids.each do |block_id|
-            notion_link.notion_block_links.create!(
-              creative: data[:child],
-              block_id: block_id,
-              content_hash: data[:content_hash]
-            )
-          end
-        end
-      end
-    end
-
-    def fetch_all_page_blocks(page_id)
-      blocks = []
-      cursor = nil
-
-      loop do
-        response = get_page_blocks(page_id, start_cursor: cursor)
-        blocks.concat(response.fetch("results", []))
-        break unless response["has_more"]
-
-        cursor = response["next_cursor"]
-      end
-
-      blocks
-    end
-  end
-end

data/app/views/collavre/creatives/_notion_integration_modal.html.erb

@@ -1,90 +0,0 @@
-<div id="notion-integration-modal"
-     data-success-message="<%= t('collavre.creatives.index.notion_integration_saved', default: 'Notion integration saved successfully.') %>"
-     data-login-required="<%= t('collavre.creatives.index.notion_integration_login_required', default: 'Sign in with your Notion account to start the integration.') %>"
-     data-no-creative="<%= t('collavre.creatives.index.notion_integration_missing_creative', default: 'No Creative selected for integration.') %>"
-     data-existing-message="<%= t('collavre.creatives.index.notion_integration_existing_message', default: 'You\'re already connected to Notion pages below.') %>"
-     data-delete-confirm="<%= t('collavre.creatives.index.notion_integration_delete_confirm', default: 'Do you want to remove the Notion integration?') %>"
-     data-delete-success="<%= t('collavre.creatives.index.notion_integration_delete_success', default: 'Notion integration removed successfully.') %>"
-     data-delete-error="<%= t('collavre.creatives.index.notion_integration_delete_error', default: 'Failed to remove the Notion integration.') %>"
-     data-delete-button-label="<%= t('collavre.creatives.index.notion_integration_delete_button', default: 'Remove integration') %>"
-     data-export-success="<%= t('collavre.creatives.index.notion_export_success', default: 'Creative exported to Notion successfully.') %>"
-     data-sync-success="<%= t('collavre.creatives.index.notion_sync_success', default: 'Creative synced to Notion successfully.') %>"
-     style="display:none;position:fixed;top:0;left:0;width:100vw;height:100vh;z-index:1000;align-items:center;justify-content:center;">
-  <div class="popup-box" style="min-width:360px;max-width:90vw;">
-    <button type="button" id="close-notion-modal" class="popup-close-btn">&times;</button>
-    <h2><%= t('collavre.creatives.index.notion_integration_title', default: 'Configure Notion integration') %></h2>
-    <p id="notion-integration-status" class="notion-modal-status"></p>
-
-    <div class="notion-wizard-step" id="notion-step-connect">
-      <p id="notion-connect-message" class="notion-modal-subtext"><%= t('collavre.creatives.index.notion_integration_connect', default: 'Sign in with your Notion account to start exporting.') %></p>
-      <form id="notion-login-form" action="/auth/notion" method="get" target="notion-auth-window" style="display:none;">
-      </form>
-      <button type="button" id="notion-login-btn" class="btn btn-primary" data-window-width="620" data-window-height="720">
-        <%= t('collavre.creatives.index.notion_login_button', default: 'Sign in with Notion') %>
-      </button>
-      <div id="notion-existing-connections" style="display:none;margin-top:1.25em;">
-        <p style="margin-bottom:0.5em;"><%= t('collavre.creatives.index.notion_integration_existing_intro', default: 'Linked Notion pages:') %></p>
-        <ul id="notion-existing-page-list" style="padding-left:1.2em;margin-bottom:0.75em;color:var(--color-text);"></ul>
-        <div style="display:flex;gap:0.5em;">
-          <button type="button" id="notion-sync-btn" class="btn btn-secondary" style="display:none;">
-            <%= t('collavre.creatives.index.notion_sync_button', default: 'Sync to Notion') %>
-          </button>
-          <button type="button" id="notion-delete-btn" class="btn btn-danger" style="display:none;">
-            <%= t('collavre.creatives.index.notion_integration_delete_button', default: 'Remove link') %>
-          </button>
-        </div>
-      </div>
-    </div>
-
-    <div class="notion-wizard-step" id="notion-step-workspace" style="display:none;">
-      <p class="notion-modal-subtext"><%= t('collavre.creatives.index.notion_integration_workspace', default: 'Your Notion workspace is connected. Choose how to export your creative.') %></p>
-      <div id="notion-workspace-info" style="margin-bottom:1em;padding:1em;background:var(--color-bg-alt);border-radius:4px;">
-        <strong id="notion-workspace-name"></strong>
-      </div>
-      <div style="margin-bottom:1em;">
-        <label style="display:block;margin-bottom:0.5em;">
-          <%= t('collavre.creatives.index.notion_export_option', default: 'Export option:') %>
-        </label>
-        <div style="display:flex;flex-direction:column;gap:0.5em;">
-          <label style="display:flex;align-items:center;gap:0.5em;">
-            <input type="radio" name="notion-export-type" value="new-page" checked>
-            <%= t('collavre.creatives.index.notion_export_new_page', default: 'Create new page') %>
-          </label>
-          <label style="display:flex;align-items:center;gap:0.5em;">
-            <input type="radio" name="notion-export-type" value="select-parent">
-            <%= t('collavre.creatives.index.notion_export_under_page', default: 'Create as subpage under existing page') %>
-          </label>
-        </div>
-      </div>
-      <div id="notion-parent-page-section" style="display:none;margin-bottom:1em;">
-        <label for="notion-parent-page-select" style="display:block;margin-bottom:0.5em;">
-          <%= t('collavre.creatives.index.notion_parent_page', default: 'Parent page:') %>
-        </label>
-        <select id="notion-parent-page-select" style="width:100%;padding:0.5em;border:1px solid var(--color-border);border-radius:4px;">
-          <option value=""><%= t('collavre.creatives.index.notion_loading', default: 'Loading pages...') %></option>
-        </select>
-      </div>
-    </div>
-
-    <div class="notion-wizard-step" id="notion-step-summary" style="display:none;">
-      <p class="notion-modal-subtext"><%= t('collavre.creatives.index.notion_integration_summary', default: 'Ready to export your creative tree to Notion:') %></p>
-      <div id="notion-export-summary" style="margin:1em 0;padding:1em;background:var(--color-bg-alt);border-radius:4px;">
-        <div><strong><%= t('collavre.creatives.index.notion_creative_title', default: 'Root Creative:') %></strong> <span id="notion-creative-title"></span></div>
-        <div><strong><%= t('collavre.creatives.index.notion_workspace', default: 'Workspace:') %></strong> <span id="notion-workspace-summary"></span></div>
-        <div><strong><%= t('collavre.creatives.index.notion_export_as', default: 'Export as:') %></strong> <span id="notion-export-type-summary"></span></div>
-        <div id="notion-parent-summary" style="display:none;"><strong><%= t('collavre.creatives.index.notion_parent_page', default: 'Parent page:') %></strong> <span id="notion-parent-page-summary"></span></div>
-        <div style="margin-top:0.5em;color:var(--color-text-secondary);font-size:0.9em;"><%= t('collavre.creatives.index.notion_tree_note', default: 'This will export the selected creative and all its descendants as a structured document.') %></div>
-      </div>
-    </div>
-
-    <div id="notion-wizard-error" style="display:none;margin:0.5em 0;color:#c0392b;font-weight:bold;"></div>
-
-    <div class="notion-wizard-footer" style="display:flex;justify-content:space-between;gap:0.5em;margin-top:1.5em;">
-      <button type="button" id="notion-prev-btn" class="btn btn-secondary" style="display:none;"><%= t('app.previous', default: 'Previous') %></button>
-      <div style="margin-left:auto;display:flex;gap:0.5em;">
-        <button type="button" id="notion-next-btn" class="btn btn-primary" style="display:none;"><%= t('app.next', default: 'Next') %></button>
-        <button type="button" id="notion-export-btn" class="btn btn-primary" style="display:none;"><%= t('collavre.creatives.index.notion_export_button', default: 'Export to Notion') %></button>
-      </div>
-    </div>
-  </div>
-</div>

data/db/migrate/20241201000000_create_notion_integrations.rb

@@ -1,29 +0,0 @@
-class CreateNotionIntegrations < ActiveRecord::Migration[8.0]
-  def change
-    create_table :notion_accounts do |t|
-      t.references :user, null: false, foreign_key: true, index: { unique: true }
-      t.string :notion_uid, null: false
-      t.string :workspace_name
-      t.string :workspace_id
-      t.string :bot_id
-      t.string :token, null: false
-      t.datetime :token_expires_at
-      t.timestamps
-    end
-
-    create_table :notion_page_links do |t|
-      t.references :creative, null: false, foreign_key: true
-      t.references :notion_account, null: false, foreign_key: true
-      t.string :page_id, null: false
-      t.string :page_title
-      t.string :page_url
-      t.string :parent_page_id
-      t.datetime :last_synced_at
-      t.timestamps
-    end
-
-    add_index :notion_accounts, :notion_uid, unique: true
-    add_index :notion_page_links, :page_id, unique: true
-    add_index :notion_page_links, [ :creative_id, :page_id ], unique: true, name: "index_notion_links_on_creative_and_page"
-  end
-end

data/db/migrate/20250312000000_create_notion_block_links.rb

@@ -1,16 +0,0 @@
-class CreateNotionBlockLinks < ActiveRecord::Migration[8.0]
-  def change
-    create_table :notion_block_links do |t|
-      t.references :notion_page_link, null: false, foreign_key: true, index: false
-      t.references :creative, null: false, foreign_key: true, index: false
-      t.string :block_id, null: false
-      t.string :content_hash
-      t.timestamps
-    end
-
-    add_index :notion_block_links, :notion_page_link_id, name: "index_notion_block_links_on_notion_page_link_id"
-    add_index :notion_block_links, :creative_id
-    add_index :notion_block_links, [ :notion_page_link_id, :creative_id ], unique: true, name: "index_notion_block_links_on_page_link_and_creative"
-    add_index :notion_block_links, [ :notion_page_link_id, :block_id ], unique: true, name: "index_notion_block_links_on_page_link_and_block"
-  end
-end

data/db/migrate/20250312010000_allow_multiple_notion_blocks_per_creative.rb

@@ -1,5 +0,0 @@
-class AllowMultipleNotionBlocksPerCreative < ActiveRecord::Migration[8.0]
-  def change
-    remove_index :notion_block_links, name: "index_notion_block_links_on_page_link_and_creative"
-  end
-end