coplan-engine 0.2.0 → 1.0.0

data/app/services/coplan/plans/markdown_text_extractor.rb

@@ -0,0 +1,142 @@
+module CoPlan
+  module Plans
+    # Extracts plain text from markdown using the Commonmarker AST, returning
+    # [stripped_string, position_map] where position_map[i] is the character
+    # index in the raw content of stripped_string[i].
+    #
+    # This handles all markdown constructs: inline formatting (`**`, `*`, `` ` ``,
+    # `~~`), tables, links, images, lists, blockquotes, headings, etc.
+    #
+    # Usage:
+    #   stripped, pos_map = MarkdownTextExtractor.call("Hello **world**")
+    #   # stripped  => "Hello world"
+    #   # pos_map   => [0, 1, 2, 3, 4, 5, 8, 9, 10, 11, 12]
+    class MarkdownTextExtractor
+      def self.call(content)
+        new(content).call
+      end
+
+      def initialize(content)
+        @content = content
+      end
+
+      def call
+        doc = Commonmarker.parse(@content)
+        byte_to_char = build_byte_to_char_map
+        line_byte_offsets = build_line_byte_offsets
+        stripped = +""
+        pos_map = []
+
+        extract_text_nodes(doc, line_byte_offsets, byte_to_char, stripped, pos_map)
+
+        [stripped, pos_map]
+      end
+
+      private
+
+      # Builds a map from byte offset to character index. Commonmarker reports
+      # source positions using byte-based columns, but Ruby string indexing
+      # uses character positions.
+      def build_byte_to_char_map
+        map = {}
+        byte_offset = 0
+        @content.each_char.with_index do |char, char_idx|
+          map[byte_offset] = char_idx
+          byte_offset += char.bytesize
+        end
+        map
+      end
+
+      # Builds an array mapping 1-based line numbers to byte offsets.
+      # line_byte_offsets[line_number] = byte offset of the first byte on that line.
+      def build_line_byte_offsets
+        offsets = [nil, 0] # index 0 unused; line 1 starts at byte 0
+        byte_offset = 0
+        @content.each_char do |char|
+          byte_offset += char.bytesize
+          offsets << byte_offset if char == "\n"
+        end
+        offsets
+      end
+
+      # Block-level node types that should be separated by newlines.
+      BLOCK_TYPES = %i[paragraph heading table table_row item block_quote list code_block].to_set.freeze
+
+      # Recursively walks the AST, appending text content to `stripped` and
+      # character-index mappings to `pos_map`. Inserts whitespace between
+      # block elements and table cells to match browser DOM text behavior.
+      def extract_text_nodes(node, line_byte_offsets, byte_to_char, stripped, pos_map)
+        prev_was_block = false
+
+        node.each do |child|
+          # Insert a space between adjacent table cells, and a newline
+          # between block-level siblings (paragraphs, rows, items, etc.).
+          if child.type == :table_cell
+            append_separator(stripped, pos_map, " ") if prev_was_block
+            prev_was_block = true
+          elsif BLOCK_TYPES.include?(child.type)
+            append_separator(stripped, pos_map, "\n") if prev_was_block
+            prev_was_block = true
+          end
+
+          case child.type
+          when :text
+            pos = child.source_position
+            start_byte = line_byte_offsets[pos[:start_line]] + pos[:start_column] - 1
+            char_idx = byte_to_char[start_byte]
+            child.string_content.each_char.with_index do |char, i|
+              stripped << char
+              pos_map << (char_idx + i)
+            end
+          when :code
+            # source_position includes backtick delimiters; find the inner
+            # content start by scanning past them in the raw string.
+            pos = child.source_position
+            start_byte = line_byte_offsets[pos[:start_line]] + pos[:start_column] - 1
+            node_char_start = byte_to_char[start_byte]
+            end_byte = line_byte_offsets[pos[:end_line]] + pos[:end_column] - 1
+            node_char_end = byte_to_char[end_byte]
+            text = child.string_content
+            node_char_len = node_char_end - node_char_start + 1
+            tick_len = (node_char_len - text.length) / 2
+            content_char_start = node_char_start + tick_len
+            text.each_char.with_index do |char, i|
+              stripped << char
+              pos_map << (content_char_start + i)
+            end
+          when :code_block
+            # Fenced code blocks: source_position spans from the opening
+            # fence to the closing fence. The string_content is the inner
+            # text (excluding fences). Content starts on the line after
+            # the opening fence.
+            pos = child.source_position
+            text = child.string_content
+            content_line = pos[:start_line] + 1
+            if content_line <= line_byte_offsets.length - 1
+              content_byte = line_byte_offsets[content_line]
+              char_idx = byte_to_char[content_byte]
+              text.each_char.with_index do |char, i|
+                stripped << char
+                pos_map << (char_idx + i)
+              end
+            end
+          when :softbreak, :linebreak
+            pos = child.source_position
+            start_byte = line_byte_offsets[pos[:start_line]] + pos[:start_column] - 1
+            stripped << "\n"
+            pos_map << byte_to_char[start_byte]
+          else
+            extract_text_nodes(child, line_byte_offsets, byte_to_char, stripped, pos_map)
+          end
+        end
+      end
+
+      # Appends a synthetic separator character to the stripped text.
+      # Maps it to -1 since it doesn't correspond to any raw source position.
+      def append_separator(stripped, pos_map, char)
+        stripped << char
+        pos_map << -1
+      end
+    end
+  end
+end

data/app/services/coplan/plans/position_resolver.rb

@@ -20,6 +20,8 @@ module CoPlan
           resolve_insert_under_heading
         when "delete_paragraph_containing"
           resolve_delete_paragraph_containing
+        when "replace_section"
+          resolve_replace_section
         else
           raise OperationError, "Unknown operation: #{@op["op"]}"
         end
@@ -153,6 +155,115 @@ module CoPlan
         paragraphs
       end
 
+      def resolve_replace_section
+        heading = @op["heading"]
+        raise OperationError, "replace_section requires 'heading'" if heading.blank?
+        raise OperationError, "replace_section requires 'new_content'" if @op["new_content"].nil?
+
+        include_heading = @op.fetch("include_heading", true)
+        # Normalize: accept both string and boolean
+        include_heading = include_heading != false && include_heading != "false"
+
+        headings = parse_headings(@content)
+        matches = headings.select { |h| h[:text] == heading }
+
+        if matches.empty?
+          raise OperationError, "replace_section: heading_not_found — no heading matching '#{heading}'"
+        end
+
+        if matches.length > 1
+          match_details = matches.map { |m| { heading: m[:text], line: m[:line] } }
+          raise OperationError, "replace_section: ambiguous_heading — found #{matches.length} headings matching '#{heading}': #{match_details.inspect}"
+        end
+
+        match = matches.first
+        target_level = match[:level]
+
+        # Section starts at the heading line start
+        section_start = match[:line_start]
+
+        # Section ends at the next heading of equal or higher level, or EOF
+        next_heading = headings.find { |h| h[:line_start] > match[:line_start] && h[:level] <= target_level }
+        section_end = next_heading ? next_heading[:line_start] : @content.length
+
+        # Strip all trailing newlines from the section range so the separator
+        # between sections falls outside the replaced range. This ensures
+        # replacement content won't merge into the next heading.
+        section_end = section_end.to_i
+        while section_end > section_start && @content[section_end - 1] == "\n"
+          section_end -= 1
+        end
+
+        range = if include_heading
+          [section_start, section_end]
+        else
+          # Skip past the heading line itself
+          heading_line_end = @content.index("\n", section_start)
+          if heading_line_end
+            body_start = heading_line_end + 1
+            # Skip blank line after heading
+            while body_start < section_end && @content[body_start] == "\n"
+              body_start += 1
+            end
+            # When trailing newlines are stripped, section_end can retreat
+            # behind body_start. Use an empty range at body_start to avoid
+            # an inverted range and keep the insertion point after the heading newline.
+            [body_start, [body_start, section_end].max]
+          else
+            # Heading is the only line — body is empty
+            [section_end, section_end]
+          end
+        end
+
+        Resolution.new(op: "replace_section", ranges: [range])
+      end
+
+      # Parse markdown headings, respecting code fences (``` blocks).
+      # Returns an array of hashes: { text:, level:, line:, line_start:, line_end: }
+      def parse_headings(content)
+        headings = []
+        in_code_fence = false
+        fence_char = nil
+        fence_length = 0
+        line_number = 0
+        pos = 0
+
+        content.each_line do |line|
+          line_number += 1
+          line_start = pos
+          line_end = pos + line.length
+          stripped = line.chomp
+
+          fence_match = stripped.match(/\A(`{3,}|~{3,})(.*)\z/)
+          if fence_match
+            fence_chars = fence_match[1]
+            info_string = fence_match[2]
+            if in_code_fence
+              # Close only if fence char and length match, and no info string
+              if fence_chars[0] == fence_char && fence_chars.length >= fence_length && info_string.strip.empty?
+                in_code_fence = false
+              end
+            else
+              in_code_fence = true
+              fence_char = fence_chars[0]
+              fence_length = fence_chars.length
+            end
+          elsif !in_code_fence && (m = stripped.match(/\A(\#{1,6})\s+(.+)/))
+            headings << {
+              level: m[1].length,
+              text: stripped,
+              line: line_number,
+              line_start: line_start,
+              line_end: line_end
+            }
+          end
+
+          pos = line_end
+        end
+
+        headings
+      end
+
       # Determine the character range to delete so that removing
       # content[range[0]...range[1]] produces clean output with
       # correct paragraph spacing.

data/app/views/coplan/comment_threads/_reply_form.html.erb

@@ -1,7 +1,7 @@
 <div class="comment-thread__reply" id="<%= dom_id(thread, :reply_form) %>">
   <%= form_with url: plan_comment_thread_comments_path(plan, thread), method: :post, data: { action: "turbo:submit-end->coplan--text-selection#resetReplyForm" } do |f| %>
     <div class="form-group">
-      <textarea name="comment[body_markdown]" rows="2" placeholder="Reply..." required
+      <textarea name="comment[body_markdown]" rows="2" placeholder="Press r to reply" required
         data-controller="coplan--comment-form" data-action="keydown->coplan--comment-form#submitOnEnter"></textarea>
     </div>
     <button type="submit" class="btn btn--secondary btn--sm">Reply</button>

data/app/views/coplan/comment_threads/_thread_popover.html.erb

@@ -7,7 +7,7 @@
 
   <div popover="auto" id="<%= dom_id(thread) %>_popover" class="thread-popover">
     <div class="thread-popover__header">
-      <span class="badge badge--<%= thread.open? ? 'live' : 'abandoned' %>"><%= thread.status %></span>
+      <span class="badge badge--<%= thread.status %>"><%= thread.status %></span>
       <% if thread.out_of_date? %>
         <span class="badge badge--abandoned">out of date</span>
       <% end %>
@@ -32,7 +32,7 @@
       <div class="thread-popover__reply">
         <%= form_with url: plan_comment_thread_comments_path(plan, thread), method: :post, data: { action: "turbo:submit-end->coplan--text-selection#resetReplyForm" } do |f| %>
           <div class="form-group">
-            <textarea name="comment[body_markdown]" rows="2" placeholder="Reply..." required
+            <textarea name="comment[body_markdown]" rows="2" placeholder="Press r to reply" required
               data-controller="coplan--comment-form" data-action="keydown->coplan--comment-form#submitOnEnter"></textarea>
           </div>
           <button type="submit" class="btn btn--secondary btn--sm">Reply</button>
@@ -42,9 +42,9 @@
       <div class="thread-popover__actions">
         <% if is_plan_author %>
           <% if thread.status == "pending" %>
-            <%= button_to "Accept", accept_plan_comment_thread_path(plan, thread), method: :patch, class: "btn btn--secondary btn--sm" %>
+            <%= button_to "Accept (a)", accept_plan_comment_thread_path(plan, thread), method: :patch, class: "btn btn--secondary btn--sm", form: { "data-action-name": "accept" } %>
           <% end %>
-          <%= button_to "Discard", discard_plan_comment_thread_path(plan, thread), method: :patch, class: "btn btn--secondary btn--sm" %>
+          <%= button_to "Discard (d)", discard_plan_comment_thread_path(plan, thread), method: :patch, class: "btn btn--secondary btn--sm", form: { "data-action-name": "discard" } %>
         <% end %>
       </div>
     <% else %>

data/app/views/coplan/comments/_comment.html.erb

@@ -1,6 +1,9 @@
 <div class="comment" id="<%= dom_id(comment) %>">
   <div class="comment__header text-sm text-muted">
     <strong><%= comment_author_name(comment) %></strong>
+    <% if comment.agent? %>
+      <span class="badge badge--agent">agent</span>
+    <% end %>
     · <%= time_ago_in_words(comment.created_at) %> ago
   </div>
   <div class="comment__body">

data/app/views/coplan/plans/_header.html.erb

@@ -6,6 +6,7 @@
     </div>
   </div>
   <div class="page-header__actions">
+    <%= render partial: "coplan/plans/viewers", locals: { viewers: CoPlan::PlanViewer.active_viewers_for(plan), current_user: current_user } %>
     <% if CoPlan::AutomatedPlanReviewer.enabled.any? %>
       <div class="dropdown" data-controller="coplan--dropdown">
         <button class="btn btn--secondary" data-action="coplan--dropdown#toggle">

data/app/views/coplan/plans/_viewers.html.erb

@@ -0,0 +1,16 @@
+<div id="plan-viewers" class="plan-viewers">
+  <% if viewers.any? %>
+    <div class="plan-viewers__avatars">
+      <% viewers.first(8).each do |viewer| %>
+        <% is_you = defined?(current_user) && current_user&.id == viewer.id %>
+        <span class="plan-viewers__avatar<%= ' plan-viewers__avatar--you' if is_you %>" data-tooltip="<%= is_you ? "#{viewer.name} (you)" : viewer.name %>">
+          <%= viewer.name.split.map { |w| w[0] }.first(2).join.upcase %>
+        </span>
+      <% end %>
+      <% if viewers.size > 8 %>
+        <span class="plan-viewers__overflow">+<%= viewers.size - 8 %></span>
+      <% end %>
+    </div>
+    <span class="plan-viewers__label"><%= viewers.size %> viewing</span>
+  <% end %>
+</div>

data/app/views/coplan/plans/show.html.erb

@@ -1,12 +1,34 @@
+<% content_for(:title, "#{@plan.title} — CoPlan") %>
+<% content_for(:head) do %>
+  <meta property="og:title" content="<%= @plan.title %>">
+  <meta property="og:description" content="<%= plan_og_description(@plan) %>">
+  <meta property="og:type" content="article">
+  <meta property="og:site_name" content="CoPlan">
+  <meta name="twitter:card" content="summary">
+  <meta name="twitter:title" content="<%= @plan.title %>">
+  <meta name="twitter:description" content="<%= plan_og_description(@plan) %>">
+<% end %>
+
 <%= turbo_stream_from @plan %>
 
-<%= render partial: "coplan/plans/header", locals: { plan: @plan } %>
+<div data-controller="coplan--presence" data-coplan--presence-plan-id-value="<%= @plan.id %>">
+  <%= render partial: "coplan/plans/header", locals: { plan: @plan } %>
+</div>
 
 <div class="plan-content card">
   <% if @plan.current_content.present? %>
-    <div class="plan-layout" data-controller="coplan--text-selection" data-coplan--text-selection-plan-id-value="<%= @plan.id %>">
+    <div class="plan-layout" data-controller="coplan--text-selection coplan--content-nav" data-coplan--text-selection-plan-id-value="<%= @plan.id %>" data-action="keydown.esc@document->coplan--text-selection#dismiss">
+      <nav class="content-nav" data-coplan--content-nav-target="sidebar" aria-label="Document outline">
+        <div class="content-nav__header">
+          <span class="content-nav__title">Contents</span>
+          <button type="button" class="content-nav__toggle" data-action="coplan--content-nav#toggle" data-coplan--content-nav-target="toggleBtn" aria-label="Hide table of contents" title="Toggle (])">✕</button>
+        </div>
+        <ul class="content-nav__list" data-coplan--content-nav-target="list"></ul>
+      </nav>
+      <button type="button" class="content-nav-show-btn" data-action="coplan--content-nav#toggle" data-coplan--content-nav-target="showBtn" title="Show table of contents (])">☰ <kbd>]</kbd></button>
+
       <div class="plan-layout__margin" data-coplan--text-selection-target="margin"></div>
-      <div class="plan-layout__content" data-coplan--text-selection-target="content">
+      <div class="plan-layout__content" data-coplan--text-selection-target="content" data-coplan--content-nav-target="content">
         <%= render_markdown(@plan.current_content) %>
 
         <div class="comment-popover" data-coplan--text-selection-target="popover" style="display: none;">

data/app/views/layouts/coplan/application.html.erb

@@ -5,6 +5,7 @@
     <meta name="viewport" content="width=device-width,initial-scale=1">
     <%= csrf_meta_tags %>
     <%= csp_meta_tag %>
+    <%= coplan_favicon_tag %>
     <%= yield :head %>
     <link rel="preconnect" href="https://fonts.googleapis.com">
     <link rel="preconnect" href="https://fonts.gstatic.com" crossorigin>
@@ -18,6 +19,7 @@
         <%= link_to coplan.root_path, class: "site-nav__brand" do %>
           <%= image_tag "coplan/coplan-logo-sm.png", alt: "CoPlan", class: "site-nav__logo" %>
           CoPlan
+          <%= coplan_environment_badge %>
         <% end %>
         <ul class="site-nav__links">
           <% if signed_in? %>

data/db/migrate/20260327000000_create_coplan_plan_viewers.rb

@@ -0,0 +1,15 @@
+class CreateCoplanPlanViewers < ActiveRecord::Migration[8.1]
+  def change
+    create_table :coplan_plan_viewers, id: { type: :string, limit: 36 } do |t|
+      t.string :plan_id, limit: 36, null: false
+      t.string :user_id, limit: 36, null: false
+      t.datetime :last_seen_at, null: false
+      t.timestamps
+    end
+
+    add_index :coplan_plan_viewers, [:plan_id, :user_id], unique: true
+    add_index :coplan_plan_viewers, :last_seen_at
+    add_foreign_key :coplan_plan_viewers, :coplan_plans, column: :plan_id
+    add_foreign_key :coplan_plan_viewers, :coplan_users, column: :user_id
+  end
+end

data/lib/coplan/version.rb

@@ -1,3 +1,3 @@
 module CoPlan
-  VERSION = "0.2.0"
+  VERSION = "1.0.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: coplan-engine
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 1.0.0
 platform: ruby
 authors:
 - Block
@@ -142,6 +142,7 @@ extra_rdoc_files: []
 files:
 - app/assets/images/coplan/coplan-logo-sm.png
 - app/assets/stylesheets/coplan/application.css
+- app/channels/coplan/plan_presence_channel.rb
 - app/controllers/coplan/api/v1/base_controller.rb
 - app/controllers/coplan/api/v1/comments_controller.rb
 - app/controllers/coplan/api/v1/leases_controller.rb
@@ -161,8 +162,10 @@ files:
 - app/helpers/coplan/markdown_helper.rb
 - app/javascript/controllers/coplan/comment_form_controller.js
 - app/javascript/controllers/coplan/comment_nav_controller.js
+- app/javascript/controllers/coplan/content_nav_controller.js
 - app/javascript/controllers/coplan/dropdown_controller.js
 - app/javascript/controllers/coplan/line_selection_controller.js
+- app/javascript/controllers/coplan/presence_controller.js
 - app/javascript/controllers/coplan/text_selection_controller.js
 - app/jobs/coplan/application_job.rb
 - app/jobs/coplan/automated_review_job.rb
@@ -179,6 +182,7 @@ files:
 - app/models/coplan/plan.rb
 - app/models/coplan/plan_collaborator.rb
 - app/models/coplan/plan_version.rb
+- app/models/coplan/plan_viewer.rb
 - app/models/coplan/user.rb
 - app/policies/coplan/application_policy.rb
 - app/policies/coplan/comment_thread_policy.rb
@@ -189,6 +193,7 @@ files:
 - app/services/coplan/plans/apply_operations.rb
 - app/services/coplan/plans/commit_session.rb
 - app/services/coplan/plans/create.rb
+- app/services/coplan/plans/markdown_text_extractor.rb
 - app/services/coplan/plans/operation_error.rb
 - app/services/coplan/plans/position_resolver.rb
 - app/services/coplan/plans/review_prompt_formatter.rb
@@ -204,6 +209,7 @@ files:
 - app/views/coplan/plan_versions/index.html.erb
 - app/views/coplan/plan_versions/show.html.erb
 - app/views/coplan/plans/_header.html.erb
+- app/views/coplan/plans/_viewers.html.erb
 - app/views/coplan/plans/edit.html.erb
 - app/views/coplan/plans/index.html.erb
 - app/views/coplan/plans/show.html.erb
@@ -219,6 +225,7 @@ files:
 - db/migrate/20260226200000_create_coplan_schema.rb
 - db/migrate/20260313210000_expand_content_markdown_to_mediumtext.rb
 - db/migrate/20260320145453_migrate_comment_thread_statuses.rb
+- db/migrate/20260327000000_create_coplan_plan_viewers.rb
 - lib/coplan.rb
 - lib/coplan/configuration.rb
 - lib/coplan/engine.rb