coplan-engine 0.4.0 → 1.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: fc0c571fae079ecc474da55dea08af61a198ca761f6f3c9b5c3b672afe3b838c
-  data.tar.gz: 791565182e62dab38561ae29eeb9794cf0f0087222449b4ef5842525b5e28424
+  metadata.gz: 25dc4e49ba0c60553ed7d736f692711ed95908f904ced5ab0eab166a6afd5454
+  data.tar.gz: 9081eee65fbcadaef5bfb8af91a50bf67a1825382ff8763de0d3b1cb38a00ac4
 SHA512:
-  metadata.gz: 973c44900f288a2ba876c9ee2df96e47e13668f271c3b5e1de187ddf4d935b4fef63de2dac2753cf1acc82628a4a4e3987c8e8e3ea8cd3e38a9ecd123e7b734e
-  data.tar.gz: e7866646881f7c217cb5b23e0adac89651e6fa47facfe9f4f33049a4e2a7362f8eb209f40dc161dbd43bdd689e643248c050be0044821efeae6abcb97c13aa08
+  metadata.gz: 3d75ee11d917e817192149b2a69a33b1089333312ebd8ac4f7b64142479d5108c5b70e801c6b4ae3b64f78e9190d123164f4e5b87f56ced1aad7df0d9b72cb93
+  data.tar.gz: c6b96bd3fe16455118551bd01ae3769914bcc2f41d0b2bd59bf159ee8eddd6e84a7fb24ebb74461d522d07802b242b08bc8f691acf9772dabcc0864ab4fdfaed

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

@@ -21,6 +21,8 @@
   --color-status-developing: #3b82f6;
   --color-status-live: #10b981;
   --color-status-abandoned: #6b7280;
+--color-agent: #4338ca;
+--color-agent-bg: #e0e7ff;
 
   /* Spacing */
   --space-xs: 0.25rem;
@@ -184,6 +186,87 @@ img, svg {
   font-weight: 700;
 }
 
+/* Plan viewers / presence */
+.plan-viewers {
+  display: flex;
+  align-items: center;
+  gap: var(--space-sm);
+}
+
+.plan-viewers__avatars {
+  display: flex;
+  flex-direction: row-reverse;
+}
+
+.plan-viewers__avatar {
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  width: 2rem;
+  height: 2rem;
+  border-radius: 50%;
+  background: var(--color-primary);
+  color: #fff;
+  font-size: 0.7rem;
+  font-weight: 600;
+  border: 2px solid var(--color-surface);
+  margin-left: -0.5rem;
+  cursor: default;
+  position: relative;
+}
+
+.plan-viewers__avatar--you {
+  background: var(--color-success);
+}
+
+.plan-viewers__avatar::after {
+  content: attr(data-tooltip);
+  position: absolute;
+  bottom: calc(100% + 6px);
+  left: 50%;
+  transform: translateX(-50%);
+  background: var(--color-text);
+  color: var(--color-surface);
+  padding: 4px 8px;
+  border-radius: var(--radius);
+  font-size: 0.75rem;
+  font-weight: 500;
+  white-space: nowrap;
+  pointer-events: none;
+  opacity: 0;
+  transition: opacity 0.15s;
+  z-index: 10;
+}
+
+.plan-viewers__avatar:hover::after {
+  opacity: 1;
+}
+
+.plan-viewers__avatars > :last-child {
+  margin-left: 0;
+}
+
+.plan-viewers__overflow {
+  display: flex;
+  align-items: center;
+  justify-content: center;
+  width: 2rem;
+  height: 2rem;
+  border-radius: 50%;
+  background: var(--color-border);
+  color: var(--color-text-muted);
+  font-size: 0.7rem;
+  font-weight: 600;
+  border: 2px solid var(--color-surface);
+  margin-left: -0.5rem;
+}
+
+.plan-viewers__label {
+  font-size: var(--text-sm);
+  color: var(--color-text-muted);
+  white-space: nowrap;
+}
+
 /* Flash messages */
 .flash {
   padding: var(--space-md) var(--space-lg);
@@ -283,6 +366,7 @@ img, svg {
 .badge--todo        { background: #dbeafe; color: var(--color-status-developing); }
 .badge--discarded   { background: #f3f4f6; color: var(--color-status-abandoned); }
 .badge--resolved    { background: #d1fae5; color: var(--color-status-live); }
+.badge--agent       { background: var(--color-agent-bg); color: var(--color-agent); font-size: 0.65rem; padding: 1px 6px; vertical-align: middle; }
 
 /* Forms */
 .form-group {
@@ -722,6 +806,18 @@ img, svg {
   min-width: 0;
 }
 
+#plan-threads {
+  position: absolute;
+  width: 0;
+  height: 0;
+  overflow: hidden;
+  pointer-events: none;
+}
+
+#plan-threads .thread-popover {
+  pointer-events: auto;
+}
+
 /* Comment form */
 .comment-form {
   position: absolute;
@@ -1143,3 +1239,194 @@ body:has(.comment-toolbar) .main-content {
 .comment-toolbar__toggle input[type="checkbox"] {
   cursor: pointer;
 }
+
+/* Content navigation sidebar */
+.content-nav {
+  position: sticky;
+  top: calc(var(--nav-height) + var(--space-lg));
+  align-self: flex-start;
+  width: 220px;
+  flex-shrink: 0;
+  max-height: calc(100vh - var(--nav-height) - var(--space-xl) * 2);
+  transition: width 0.2s ease, opacity 0.2s ease;
+  margin-right: var(--space-lg);
+}
+
+.content-nav--hidden {
+  width: 0;
+  opacity: 0;
+  overflow: hidden;
+  margin-right: 0;
+}
+
+.content-nav__header {
+  display: flex;
+  align-items: center;
+  justify-content: space-between;
+  padding-bottom: var(--space-sm);
+  margin-bottom: var(--space-sm);
+  border-bottom: 1px solid var(--color-border);
+}
+
+.content-nav__title {
+  font-size: var(--text-sm);
+  font-weight: 600;
+  color: var(--color-text-muted);
+  text-transform: uppercase;
+  letter-spacing: 0.05em;
+}
+
+.content-nav__toggle {
+  background: none;
+  border: none;
+  cursor: pointer;
+  color: var(--color-text-muted);
+  padding: 2px;
+  font-size: var(--text-sm);
+  line-height: 1;
+  border-radius: var(--radius);
+  transition: color 0.15s;
+}
+
+.content-nav__toggle:hover {
+  color: var(--color-text);
+}
+
+.content-nav__list {
+  list-style: none;
+  padding: 0;
+  margin: 0;
+  overflow-y: auto;
+  max-height: calc(100vh - var(--nav-height) - 8rem);
+  scrollbar-width: thin;
+  scrollbar-color: var(--color-border) transparent;
+}
+
+.content-nav__item {
+  margin: 0;
+}
+
+.content-nav__item--h1 {
+  /* top level — no indent */
+}
+
+.content-nav__item--h2 {
+  padding-left: var(--space-md);
+}
+
+.content-nav__item--h3 {
+  padding-left: var(--space-xl);
+}
+
+.content-nav__link {
+  display: flex;
+  align-items: center;
+  gap: var(--space-xs);
+  padding: 3px var(--space-sm);
+  font-size: 0.8rem;
+  line-height: 1.4;
+  color: var(--color-text-muted);
+  text-decoration: none;
+  border-radius: var(--radius);
+  border-left: 2px solid transparent;
+  transition: color 0.15s, border-color 0.15s, background 0.15s;
+  word-break: break-word;
+}
+
+.content-nav__link:hover {
+  color: var(--color-text);
+  background: var(--color-bg);
+  text-decoration: none;
+}
+
+.content-nav__link--active {
+  color: var(--color-primary);
+  border-left-color: var(--color-primary);
+  font-weight: 600;
+}
+
+.content-nav__link-text {
+  flex: 1;
+  min-width: 0;
+}
+
+.content-nav__badge {
+  flex-shrink: 0;
+  font-size: 0.65rem;
+  font-weight: 700;
+  min-width: 16px;
+  height: 16px;
+  display: inline-flex;
+  align-items: center;
+  justify-content: center;
+  border-radius: 8px;
+  color: white;
+  line-height: 1;
+}
+
+.content-nav__badge--pending {
+  background: var(--color-status-considering);
+}
+
+.content-nav__badge--todo {
+  background: var(--color-status-developing);
+}
+
+/* Toggle button when sidebar is hidden — shown outside the sidebar */
+.content-nav-show-btn {
+  position: sticky;
+  top: calc(var(--nav-height) + var(--space-lg));
+  align-self: flex-start;
+  flex-shrink: 0;
+  background: var(--color-surface);
+  border: 1px solid var(--color-border);
+  border-radius: var(--radius);
+  padding: var(--space-xs) var(--space-sm);
+  cursor: pointer;
+  color: var(--color-text-muted);
+  font-size: var(--text-sm);
+  box-shadow: var(--shadow);
+  transition: color 0.15s, box-shadow 0.15s;
+  margin-right: var(--space-sm);
+  display: none;
+  white-space: nowrap;
+}
+
+.content-nav-show-btn:hover {
+  color: var(--color-text);
+  box-shadow: 0 2px 6px rgba(0, 0, 0, 0.1);
+}
+
+.content-nav-show-btn kbd {
+  font-family: var(--font-mono);
+  font-size: 0.7rem;
+  background: var(--color-bg);
+  border: 1px solid var(--color-border);
+  border-radius: 3px;
+  padding: 0 4px;
+  color: var(--color-text-muted);
+}
+
+.content-nav--hidden + .content-nav-show-btn {
+  display: block;
+}
+
+/* Add scroll-margin to headings so they clear the sticky nav */
+.markdown-rendered h1,
+.markdown-rendered h2,
+.markdown-rendered h3,
+.markdown-rendered h4,
+.markdown-rendered h5,
+.markdown-rendered h6 {
+  scroll-margin-top: calc(var(--nav-height) + var(--space-lg));
+}
+
+/* Hide sidebar on small screens */
+@media (max-width: 1024px) {
+  .content-nav {
+    display: none;
+  }
+  .content-nav-show-btn {
+    display: none !important;
+  }
+}

data/app/channels/coplan/plan_presence_channel.rb

@@ -0,0 +1,45 @@
+module CoPlan
+  class PlanPresenceChannel < ActionCable::Channel::Base
+    def subscribed
+      @plan = Plan.find_by(id: params[:plan_id])
+      policy = @plan && PlanPolicy.new(current_user, @plan)
+      unless @plan && policy.show?
+        reject
+        return
+      end
+
+      PlanViewer.track(plan: @plan, user: current_user)
+      broadcast_viewers
+    end
+
+    def unsubscribed
+      return unless @plan
+
+      PlanViewer.expire(plan: @plan, user: current_user)
+      broadcast_viewers
+    end
+
+    def ping
+      return unless @plan
+
+      PlanViewer.track(plan: @plan, user: current_user)
+      broadcast_viewers
+    end
+
+    private
+
+    def current_user
+      connection.current_user
+    end
+
+    def broadcast_viewers
+      viewers = PlanViewer.active_viewers_for(@plan)
+      Broadcaster.replace_to(
+        @plan,
+        target: "plan-viewers",
+        partial: "coplan/plans/viewers",
+        locals: { viewers: viewers }
+      )
+    end
+  end
+end

data/app/controllers/coplan/api/v1/comments_controller.rb

@@ -34,7 +34,7 @@ module CoPlan
           }, status: :created
 
         rescue ActiveRecord::RecordInvalid => e
-          render json: { error: e.message }, status: :unprocessable_entity
+          render json: { error: e.message }, status: :unprocessable_content
         end
 
         def resolve
@@ -98,7 +98,7 @@ module CoPlan
           }, status: :created
 
         rescue ActiveRecord::RecordInvalid => e
-          render json: { error: e.message }, status: :unprocessable_entity
+          render json: { error: e.message }, status: :unprocessable_content
         end
 
         private

data/app/controllers/coplan/api/v1/operations_controller.rb

@@ -10,12 +10,12 @@ module CoPlan
           base_revision = params[:base_revision]&.to_i
 
           unless base_revision.present?
-            render json: { error: "base_revision is required" }, status: :unprocessable_entity
+            render json: { error: "base_revision is required" }, status: :unprocessable_content
             return
           end
 
           unless operations.is_a?(Array) && operations.any?
-            render json: { error: "operations must be a non-empty array" }, status: :unprocessable_entity
+            render json: { error: "operations must be a non-empty array" }, status: :unprocessable_content
             return
           end
 
@@ -27,7 +27,7 @@ module CoPlan
             apply_direct(operations, base_revision)
           end
         rescue Plans::OperationError => e
-          render json: { error: e.message }, status: :unprocessable_entity
+          render json: { error: e.message }, status: :unprocessable_content
         end
 
         private

data/app/controllers/coplan/api/v1/plans_controller.rb

@@ -32,7 +32,7 @@ module CoPlan
             current_revision: plan.current_revision
           ), status: :created
         rescue ActiveRecord::RecordInvalid => e
-          render json: { error: e.message }, status: :unprocessable_entity
+          render json: { error: e.message }, status: :unprocessable_content
         end
 
         def update
@@ -61,7 +61,7 @@ module CoPlan
             current_revision: @plan.current_revision
           )
         rescue ActiveRecord::RecordInvalid => e
-          render json: { error: e.record.errors.full_messages.join(", ") }, status: :unprocessable_entity
+          render json: { error: e.record.errors.full_messages.join(", ") }, status: :unprocessable_content
         end
 
         def versions

data/app/controllers/coplan/api/v1/sessions_controller.rb

@@ -11,7 +11,7 @@ module CoPlan
         def create
           actor_type = params[:actor_type].presence || ApiToken::HOLDER_TYPE
           unless EditSession::ACTOR_TYPES.include?(actor_type)
-            render json: { error: "Invalid actor_type" }, status: :unprocessable_entity
+            render json: { error: "Invalid actor_type" }, status: :unprocessable_content
             return
           end
           ttl = actor_type == "cloud_persona" ? EditSession::CLOUD_PERSONA_TTL : EditSession::LOCAL_AGENT_TTL
@@ -56,7 +56,7 @@ module CoPlan
 
           render json: response
         rescue Plans::CommitSession::SessionNotOpenError => e
-          render json: { error: e.message }, status: :unprocessable_entity
+          render json: { error: e.message }, status: :unprocessable_content
         rescue Plans::CommitSession::StaleSessionError => e
           render json: { error: e.message }, status: :conflict
         rescue Plans::CommitSession::SessionConflictError => e

data/app/controllers/coplan/plans_controller.rb

@@ -11,6 +11,7 @@ module CoPlan
     def show
       authorize!(@plan, :show?)
       @threads = @plan.comment_threads.includes(:comments, :created_by_user).order(:created_at)
+      PlanViewer.track(plan: @plan, user: current_user)
     end
 
     def edit

data/app/controllers/coplan/settings/tokens_controller.rb

@@ -20,7 +20,7 @@ module CoPlan
       rescue ActiveRecord::RecordInvalid => e
         @api_tokens = current_user.api_tokens.order(created_at: :desc)
         flash.now[:alert] = e.message
-        render :index, status: :unprocessable_entity
+        render :index, status: :unprocessable_content
       end
 
       def destroy

data/app/helpers/coplan/application_helper.rb

@@ -1,5 +1,6 @@
 module CoPlan
   module ApplicationHelper
+    include MarkdownHelper
     FAVICON_COLORS = {
       "production"  => { start: "#3B82F6", stop: "#1E40AF" },
       "staging"     => { start: "#F59E0B", stop: "#D97706" },
@@ -37,6 +38,18 @@ module CoPlan
       tag.link(rel: "icon", type: "image/svg+xml", href: data_uri)
     end
 
+    def plan_og_description(plan)
+      status = plan.status.capitalize
+      author = plan.created_by_user.name
+      prefix = "#{status} · by #{author}"
+      content = plan.current_content
+      return prefix if content.blank?
+
+      plain = markdown_to_plain_text(content)
+      truncated = truncate(plain, length: 200, omission: "…")
+      "#{prefix} — #{truncated}"
+    end
+
     def coplan_environment_badge
       return if Rails.env.production?
 

data/app/helpers/coplan/comments_helper.rb

@@ -1,20 +1,21 @@
 module CoPlan
   module CommentsHelper
     def comment_author_name(comment)
-      case comment.author_type
+      user_name = case comment.author_type
       when "human"
         CoPlan::User.find_by(id: comment.author_id)&.name || "Unknown"
       when "local_agent"
-        user_name = CoPlan::User
+        CoPlan::User
           .joins(:api_tokens)
           .where(coplan_api_tokens: { id: comment.author_id })
           .pick(:name) || "Agent"
-        comment.agent_name.present? ? "#{user_name} (#{comment.agent_name})" : user_name
       when "cloud_persona"
         AutomatedPlanReviewer.find_by(id: comment.author_id)&.name || "Reviewer"
       else
         comment.author_type
       end
+
+      comment.agent_name.present? ? "#{comment.agent_name} (via #{user_name})" : user_name
     end
   end
 end

data/app/helpers/coplan/markdown_helper.rb

@@ -17,11 +17,16 @@ module CoPlan
     ALLOWED_ATTRIBUTES = %w[id class href src alt title].freeze
 
     def render_markdown(content)
-      html = Commonmarker.to_html(content.to_s.encode("UTF-8"), plugins: { syntax_highlighter: nil })
+      html = Commonmarker.to_html(content.to_s.encode("UTF-8"), options: { render: { unsafe: true } }, plugins: { syntax_highlighter: nil })
       sanitized = sanitize(html, tags: ALLOWED_TAGS, attributes: ALLOWED_ATTRIBUTES)
       tag.div(sanitized, class: "markdown-rendered")
     end
 
+    def markdown_to_plain_text(content)
+      html = Commonmarker.to_html(content.to_s.encode("UTF-8"), plugins: { syntax_highlighter: nil })
+      Nokogiri::HTML::DocumentFragment.parse(html).text.squish
+    end
+
     def render_line_view(content)
       lines = content.to_s.split("\n", -1)
       line_divs = lines.each_with_index.map do |line, index|