-
- <% if plan.tags.any? %>
-
- <% plan.tags.each do |tag| %>
- <%= link_to tag.name, plans_path(params.permit(:scope, :status, :plan_type).merge(tag: tag.name)), class: "badge badge--tag #{'badge--tag-active' if params[:tag] == tag.name}", data: { turbo_frame: "_top" } %>
- <% end %>
-
+
+
+ <% if summary.present? %>
+
<%= summary %>
<% end %>
+
- <%= user_avatar(plan.created_by_user) %> <%= plan.created_by_user.name %> · v<%= plan.current_revision %> · updated <%= time_ago_in_words(plan.updated_at) %> ago
+
+ <%= user_avatar(plan.created_by_user) %>
+ <%= plan.created_by_user.name %>
+
+ v<%= plan.current_revision %>
+ updated <%= time_ago_in_words(plan.updated_at) %> ago
+ <% if plan.tags.any? %>
+
+ <% plan.tags.each do |tag| %>
+ <%= link_to tag.name,
+ plans_path(params.permit(:scope, :status, :plan_type).merge(tag: tag.name)),
+ class: "badge badge--tag #{'badge--tag-active' if params[:tag] == tag.name}",
+ data: { turbo_frame: "_top" } %>
+ <% end %>
+
+ <% end %>
-
+
+ <%= link_to "Mine", plans_path(params.permit(:status, :plan_type, :tag)),
+ class: "status-filter #{'status-filter--active' if @scope == 'mine'}" %>
+ <%= link_to "All", plans_path(params.permit(:status, :plan_type, :tag).merge(scope: "all")),
+ class: "status-filter #{'status-filter--active' if @scope == 'all'}" %>
+
+ <%= link_to "Any status", plans_path(params.permit(:scope, :plan_type, :tag)),
+ class: "status-filter #{'status-filter--active' if params[:status].blank?}" %>
+ <% CoPlan::Plan::STATUSES.each do |status| %>
+ <%= link_to status.titleize, plans_path(params.permit(:scope, :plan_type, :tag).merge(status: status)),
+ class: "status-filter status-filter--#{status} #{'status-filter--active' if params[:status] == status}" %>
+ <% end %>
+
-
- <%= link_to "All Plans", plans_path(params.permit(:status, :tag)), class: "status-filter #{'status-filter--active' if params[:scope].blank?}" %>
- <%= link_to "My Plans", plans_path(params.permit(:status, :tag).merge(scope: "mine")), class: "status-filter #{'status-filter--active' if params[:scope] == 'mine'}" %>
-
+ <% if @plan_types.any? %>
+
+ <%= link_to "Any type", plans_path(params.permit(:scope, :status, :tag)),
+ class: "status-filter #{'status-filter--active' if params[:plan_type].blank?}" %>
+ <% @plan_types.each do |pt| %>
+ <%= link_to pt.name, plans_path(params.permit(:scope, :status, :tag).merge(plan_type: pt.id)),
+ class: "status-filter #{'status-filter--active' if params[:plan_type] == pt.id}" %>
+ <% end %>
+
+ <% end %>
-
- <%= link_to "All", plans_path(params.permit(:scope, :plan_type, :tag)), class: "status-filter #{'status-filter--active' if params[:status].blank?}" %>
- <% CoPlan::Plan::STATUSES.each do |status| %>
- <%= link_to status.titleize, plans_path(params.permit(:scope, :plan_type, :tag).merge(status: status)), class: "status-filter status-filter--#{status} #{'status-filter--active' if params[:status] == status}" %>
+ <% if params[:tag].present? %>
+
+ Filtered by tag: <%= params[:tag] %>
+ <%= link_to "✕ Clear", plans_path(params.permit(:scope, :status, :plan_type)), class: "active-filter__clear" %>
+
<% end %>
-<% if @plan_types.any? %>
-
- <%= link_to "All Types", plans_path(params.permit(:scope, :status, :tag)), class: "status-filter #{'status-filter--active' if params[:plan_type].blank?}" %>
- <% @plan_types.each do |pt| %>
- <%= link_to pt.name, plans_path(params.permit(:scope, :status, :tag).merge(plan_type: pt.id)), class: "status-filter #{'status-filter--active' if params[:plan_type] == pt.id}" %>
- <% end %>
-
-<% end %>
-
-<% if params[:tag].present? %>
-
- Filtered by tag: <%= params[:tag] %>
- <%= link_to "✕ Clear", plans_path(params.permit(:scope, :status, :plan_type)), class: "active-filter__clear" %>
-
-<% end %>
-
<% if @plans.any? %>
- <%= render partial: "coplan/plans/plan_page", locals: { plans: @plans, plan_unread_counts: @plan_unread_counts, page: @page, has_next_page: @has_next_page } %>
+ <%= render partial: "coplan/plans/plan_page", locals: {
+ plans: @plans,
+ plan_unread_counts: @plan_unread_counts,
+ page: @page,
+ has_next_page: @has_next_page,
+ grouped_by_status: @grouped_by_status,
+ } %>
<% else %>
-
No plans yet. Plans are created via the API.
+ <% if @scope == "mine" %>
+
You haven't created any plans yet. Plans are created via the API.
+ <% else %>
+
No plans yet. Plans are created via the API.
+ <% end %>
<% end %>
diff --git a/engine/app/views/coplan/welcome/_default_landing.html.erb b/engine/app/views/coplan/welcome/_default_landing.html.erb
index c0a1619..533b376 100644
--- a/engine/app/views/coplan/welcome/_default_landing.html.erb
+++ b/engine/app/views/coplan/welcome/_default_landing.html.erb
@@ -63,7 +63,7 @@
<% end %>
diff --git a/spec/factories/plans.rb b/spec/factories/plans.rb
index c3ffb0e..fa64024 100644
--- a/spec/factories/plans.rb
+++ b/spec/factories/plans.rb
@@ -20,5 +20,17 @@
trait :brainstorm do
status { "brainstorm" }
end
+
+ trait :developing do
+ status { "developing" }
+ end
+
+ trait :live do
+ status { "live" }
+ end
+
+ trait :abandoned do
+ status { "abandoned" }
+ end
end
end
diff --git a/spec/helpers/plans_helper_spec.rb b/spec/helpers/plans_helper_spec.rb
new file mode 100644
index 0000000..0857b41
--- /dev/null
+++ b/spec/helpers/plans_helper_spec.rb
@@ -0,0 +1,37 @@
+require "rails_helper"
+
+RSpec.describe CoPlan::PlansHelper, type: :helper do
+ describe "#plan_content_preview" do
+ let(:plan) { create(:plan, :considering) }
+
+ it "strips markdown formatting and returns a plain-text preview" do
+ plan.current_plan_version.update!(
+ content_markdown: "# Heading\n\nA **bold** intro with [a link](https://example.com)."
+ )
+ preview = helper.plan_content_preview(plan)
+ expect(preview).to include("Heading")
+ expect(preview).to include("A bold intro with a link")
+ expect(preview).not_to include("**")
+ expect(preview).not_to include("](")
+ end
+
+ it "truncates to the requested limit" do
+ plan.current_plan_version.update!(content_markdown: "word " * 100)
+ preview = helper.plan_content_preview(plan, limit: 40)
+ expect(preview.length).to be <= 41 # 40 chars + ellipsis
+ expect(preview).to end_with("…")
+ end
+
+ it "returns nil when the plan has no content" do
+ plan.current_plan_version.update_columns(content_markdown: "", content_sha256: Digest::SHA256.hexdigest(""))
+ plan.reload
+ expect(helper.plan_content_preview(plan)).to be_nil
+ end
+
+ it "returns nil when the plan has no version" do
+ plan.update_columns(current_plan_version_id: nil)
+ plan.reload
+ expect(helper.plan_content_preview(plan)).to be_nil
+ end
+ end
+end
diff --git a/spec/requests/plans_spec.rb b/spec/requests/plans_spec.rb
index 7233aad..2048924 100644
--- a/spec/requests/plans_spec.rb
+++ b/spec/requests/plans_spec.rb
@@ -139,10 +139,10 @@
expect(response).to redirect_to(plan_path(plan))
end
- it "index hides other users brainstorm plans" do
+ it "index hides other users brainstorm plans on the All scope" do
brainstorm_plan # alice's brainstorm
sign_in_as(bob)
- get plans_path
+ get plans_path(scope: "all")
expect(response).to have_http_status(:success)
expect(response.body).not_to include(brainstorm_plan.title)
end
@@ -154,6 +154,64 @@
expect(response.body).to include(brainstorm_plan.title)
end
+ describe "default scope" do
+ it "defaults to 'mine' and hides other users' plans" do
+ plan # alice's
+ bobs_plan = create(:plan, :considering, created_by_user: bob, title: "Bobs Roadmap")
+ get plans_path
+ expect(response.body).to include(plan.title)
+ expect(response.body).not_to include(bobs_plan.title)
+ end
+
+ it "scope=all shows everyone's published plans" do
+ plan # alice's
+ bobs_plan = create(:plan, :considering, created_by_user: bob, title: "Bobs Roadmap")
+ get plans_path(scope: "all")
+ expect(response.body).to include(plan.title)
+ expect(response.body).to include(bobs_plan.title)
+ end
+
+ it "groups My Plans by status with section headers" do
+ create(:plan, :developing, created_by_user: alice, title: "Developing Plan")
+ create(:plan, :considering, created_by_user: alice, title: "Considering Plan")
+ create(:plan, :brainstorm, created_by_user: alice, title: "Brainstorm Plan")
+ get plans_path
+ expect(response.body).to include("plans-list__section")
+ # active work appears before brainstorm
+ expect(response.body.index("Developing Plan")).to be < response.body.index("Brainstorm Plan")
+ expect(response.body.index("Considering Plan")).to be < response.body.index("Brainstorm Plan")
+ end
+
+ it "does not group when filtered to a single status" do
+ create(:plan, :developing, created_by_user: alice, title: "Developing Plan")
+ get plans_path(status: "developing")
+ expect(response.body).not_to include("plans-list__section")
+ expect(response.body).to include("Developing Plan")
+ end
+
+ it "scope=all does not group by status" do
+ create(:plan, :developing, created_by_user: alice, title: "Developing Plan")
+ get plans_path(scope: "all")
+ expect(response.body).not_to include("plans-list__section")
+ end
+ end
+
+ describe "content preview on cards" do
+ it "renders a markdown-stripped preview when there is no AI summary" do
+ plan.current_plan_version.update!(content_markdown: "# Heading\n\nThis is the **plan body** with [links](https://example.com).")
+ get plans_path
+ expect(response.body).to include("plans-list__summary")
+ expect(response.body).to include("This is the plan body with links")
+ expect(response.body).not_to include("**plan body**")
+ end
+
+ it "omits the summary block when the plan has no content" do
+ plan.current_plan_version.update_columns(content_markdown: "", content_sha256: Digest::SHA256.hexdigest(""))
+ get plans_path
+ expect(response.body).not_to include("plans-list__summary")
+ end
+ end
+
it "can view brainstorm plan as non-author" do
sign_in_as(bob)
get plan_path(brainstorm_plan)