Almirah 0.4.2 → 0.4.4

data/lib/almirah/doc_types/risks_overview.rb

@@ -0,0 +1,102 @@
+# frozen_string_literal: true
+
+require_relative 'base_document'
+require_relative 'rpn_rendering'
+
+# The all-registries summary page (ADR-219): build/risks/overview.html, the
+# target of the top-menu Risks button. One table row per registry, in
+# file-system order, with the columns Risk Registry (linked to the registry
+# page), Total Risks, Open Risks, Highest RPN and Average RPN — the RPN
+# aggregates computed over the registry's leading RPN group, ignoring records
+# whose group value is blank. The Risk Registry cell shows the registry
+# preface's frontmatter title, falling back to the folder name (ENH-221).
+class RisksOverview < BaseDocument
+  include RpnRendering
+
+  OPEN_EXCLUDED_STATUS = 'Closed'
+
+  attr_accessor :registries, :configuration, :prefaces
+
+  # `registries` is the ordered list of [name, records] pairs; `prefaces`
+  # maps a registry name to its parsed overview.md, when it has one.
+  def initialize(registries, configuration, prefaces = {})
+    super()
+    @registries = registries
+    @configuration = configuration
+    @prefaces = prefaces
+    @id = 'overview'
+    @title = 'Risk Registries'
+  end
+
+  def to_console
+    puts "\e[36mRisks Overview: #{@id}\e[0m"
+  end
+
+  def to_html(output_file_path)
+    html_rows = ['']
+    html_rows.append "<h1>#{@title}</h1>\n"
+    html_rows.append render_registries_table
+    save_html_to_file(html_rows, nil, output_file_path)
+  end
+
+  private
+
+  def render_registries_table
+    s = "<table class=\"controlled risks_overview\">\n"
+    s += "\t<thead>\n"
+    s += "\t\t<th>Risk Registry</th>\n"
+    s += "\t\t<th>Total Risks</th>\n"
+    s += "\t\t<th>Open Risks</th>\n"
+    s += "\t\t<th>Highest RPN</th>\n"
+    s += "\t\t<th>Average RPN</th>\n"
+    s += "</thead>\n"
+    @registries.each { |name, records| s += render_registry_row(name, records) }
+    s + "</table>\n"
+  end
+
+  def render_registry_row(name, records)
+    group = @configuration.get_risk_rpn_groups(name).first
+    values = group ? records.filter_map { |r| r.rpn_value(group) } : []
+    s = "\t<tr>\n"
+    s += "\t\t<td class=\"item_text\" style='padding: 5px;'>\
+<a name=\"#{name}\" id=\"#{name}\" href=\"./#{name}/overview.html\" class=\"external\" \
+title=\"Risk Registry\">#{registry_title(name)}</a></td>\n"
+    s += "\t\t<td class=\"item_rpn\">#{records.length}</td>\n"
+    s += "\t\t<td class=\"item_rpn\">#{open_count(records)}</td>\n"
+    s += render_highest_cell(values, group)
+    s += render_average_cell(values)
+    s + "\t</tr>\n"
+  end
+
+  # The registry preface's frontmatter title (ENH-221) — the same source the
+  # registry page heading uses — or the folder name when the registry has no
+  # preface or the preface carries no title.
+  def registry_title(name)
+    params = @prefaces[name]&.frontmatter&.parameters
+    (params && params['title']) || name
+  end
+
+  # Every record whose marked status is not Closed counts as open — including
+  # records with no current-status marker (ADR-219 keeps the count honest to
+  # the marker; another terminal status is the registry preface's to document).
+  def open_count(records)
+    records.count { |r| r.current_status.to_s.strip != OPEN_EXCLUDED_STATUS }
+  end
+
+  # The worst risk's verdict carries up: the cell keeps the leading group's
+  # threshold colouring. Blank without a group or computable values.
+  def render_highest_cell(values, group)
+    return "\t\t<td class=\"item_rpn\"></td>\n" if values.empty?
+
+    highest = values.max
+    classes = ['item_rpn', rpn_threshold_class(highest, group)].compact.join(' ')
+    "\t\t<td class=\"#{classes}\">#{format_rpn(highest)}</td>\n"
+  end
+
+  def render_average_cell(values)
+    return "\t\t<td class=\"item_rpn\"></td>\n" if values.empty?
+
+    average = (values.sum.to_f / values.length).round(1)
+    "\t\t<td class=\"item_rpn\">#{format_rpn(average)}</td>\n"
+  end
+end

data/lib/almirah/doc_types/rpn_rendering.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+# Shared RPN cell presentation (ADR-217, reused by the registries summary of
+# ADR-219): the threshold band class and the integer-when-whole formatting.
+module RpnRendering
+  # Acceptable at or below the acceptable bound, unacceptable at or above the
+  # unacceptable bound, caution between them (the ALARP band); a lone bound
+  # leaves the rest of the range as caution. nil when no thresholds configured.
+  def rpn_threshold_class(value, group)
+    if group[:acceptable] && value <= group[:acceptable]
+      'rpn_acceptable'
+    elsif group[:unacceptable] && value >= group[:unacceptable]
+      'rpn_unacceptable'
+    elsif group[:acceptable] || group[:unacceptable]
+      'rpn_caution'
+    end
+  end
+
+  # Whole values print as integers (8 * 3 -> 24, not 24.0).
+  def format_rpn(value)
+    value == value.to_i ? value.to_i.to_s : value.to_s
+  end
+end

data/lib/almirah/doc_types/source_file.rb

@@ -63,7 +63,7 @@ class SourceFile < PersistentDocument
     save_to_file(html_rows, nil, output_file_path)
   end
 
-  def save_to_file(html_rows, nav_pane, output_file_path) # rubocop:disable Metrics/AbcSize,Metrics/CyclomaticComplexity,Metrics/MethodLength,Metrics/PerceivedComplexity
+  def save_to_file(html_rows, nav_pane, output_file_path)
     gem_root = File.expand_path './../../..', File.dirname(__FILE__)
     template_file = "#{gem_root}/lib/almirah/templates/page.html"
 

data/lib/almirah/doc_types/traceability.rb

@@ -1,142 +1,133 @@
-require_relative "base_document"
+require_relative 'base_document'
 
 class Traceability < BaseDocument
-
-    attr_accessor :top_doc
-    attr_accessor :bottom_doc
-    attr_accessor :items
-    attr_accessor :is_agregated
-    attr_accessor :traced_items
-
-    def initialize(top_doc, bottom_doc)
-        super()
-        @top_doc = top_doc
-        @bottom_doc = bottom_doc
-        @is_agregated = if bottom_doc
-                          false
-                        else
-                          true
-                        end
-        @traced_items = {}
-
-        if @is_agregated 
-            @id = top_doc.id + "-all"
-        else
-            @id = top_doc.id + "-" + bottom_doc.id
-        end
-
-        @title = "Traceability Matrix: " + @id
-    end
-
-    def to_console
-        puts "\e[35m" + "Traceability: " + @id + "\e[0m"
+  attr_accessor :top_doc, :bottom_doc, :items, :is_agregated, :traced_items
+
+  def initialize(top_doc, bottom_doc)
+    super()
+    @top_doc = top_doc
+    @bottom_doc = bottom_doc
+    @is_agregated = if bottom_doc
+                      false
+                    else
+                      true
+                    end
+    @traced_items = {}
+
+    @id = if @is_agregated
+            top_doc.id + '-all'
+          else
+            top_doc.id + '-' + bottom_doc.id
+          end
+
+    @title = 'Traceability Matrix: ' + @id
+  end
+
+  def to_console
+    puts "\e[35m" + 'Traceability: ' + @id + "\e[0m"
+  end
+
+  def to_html(nav_pane, output_file_path)
+    html_rows = []
+
+    html_rows.append('')
+    s = "<h1>#{@title}</h1>\n"
+    s += "<table class=\"controlled\">\n"
+    s += "\t<thead> <th>#</th> <th style='font-weight: bold;'>#{@top_doc.title}</th> "
+    s += if @bottom_doc
+           "<th>#</th> <th style='font-weight: bold;'>#{@bottom_doc.title}</th> "
+         else
+           "<th>#</th> <th style='font-weight: bold;'>All References</th> "
+         end
+    s += "<th style='font-weight: bold;'>Document Section</th>"
+    s += "</thead>\n"
+    html_rows.append s
+
+    sorted_items = @top_doc.controlled_items.sort_by { |w| w.id }
+
+    sorted_items.each do |top_item|
+      row = render_table_row top_item
+      html_rows.append row
     end
-
-    def to_html(nav_pane, output_file_path)
-
-        html_rows = Array.new
-
-        html_rows.append('')
-        s = "<h1>#{@title}</h1>\n"
-        s += "<table class=\"controlled\">\n"
-        s += "\t<thead> <th>#</th> <th style='font-weight: bold;'>#{@top_doc.title}</th> "
-        if @bottom_doc
-            s += "<th>#</th> <th style='font-weight: bold;'>#{@bottom_doc.title}</th> "
-        else
-            s += "<th>#</th> <th style='font-weight: bold;'>All References</th> "
+    html_rows.append "</table>\n"
+
+    save_html_to_file(html_rows, nav_pane, output_file_path)
+  end
+
+  def render_table_row(top_item)
+    s = ''
+    top_f_text = top_item.format_string(top_item.text)
+    id_color = ''
+
+    if top_item.down_links
+
+      if @is_agregated
+
+        top_item_rendered = false
+
+        top_item.down_links.each do |bottom_item|
+          id_color = "style='background-color: ##{bottom_item.parent_doc.color};'"
+          bottom_f_text = bottom_item.format_string(bottom_item.text)
+          document_section = bottom_item.parent_heading.get_section_info_html
+          s += "\t<tr>\n"
+          s += "\t\t<td class=\"item_id\"><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
+          s += "\t\t<td class=\"item_id\" #{id_color}><a href=\"./../#{bottom_item.parent_doc.id}/#{bottom_item.parent_doc.id}.html##{bottom_item.id}\" class=\"external\">#{bottom_item.id}</a></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{bottom_f_text}</td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 16%;'>#{document_section}</td>\n"
+          s += "\t</tr>\n"
+          top_item_rendered = true
+          @traced_items[top_item.id.to_s.downcase] = top_item
         end
-        s += "<th style='font-weight: bold;'>Document Section</th>"
-        s += "</thead>\n"
-        html_rows.append s
-
-        sorted_items = @top_doc.controlled_items.sort_by { |w| w.id }
-
-        sorted_items.each do |top_item|
-            row = render_table_row top_item
-            html_rows.append row
+        unless top_item_rendered
+          s += "\t<tr>\n"
+          s += "\t\t<td class=\"item_id\"><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
+          s += "\t\t<td class=\"item_id\"></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 16%;'></td>\n"
+          s += "\t</tr>\n"
         end
-        html_rows.append "</table>\n"
 
-        self.save_html_to_file(html_rows, nav_pane, output_file_path)
-        
-    end
-
-    def render_table_row(top_item)
-        s = ""
-        top_f_text = top_item.format_string( top_item.text )
-        id_color = ""
-
-        if top_item.down_links
-
-            if @is_agregated
-                
-                top_item_rendered = false
-
-                top_item.down_links.each do |bottom_item|
-                    id_color = "style='background-color: ##{bottom_item.parent_doc.color};'"
-                    bottom_f_text = bottom_item.format_string( bottom_item.text )
-                    document_section = bottom_item.parent_heading.get_section_info_html
-                    s += "\t<tr>\n"
-                    s += "\t\t<td class=\"item_id\"><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
-                    s += "\t\t<td class=\"item_id\" #{id_color}><a href=\"./../#{bottom_item.parent_doc.id}/#{bottom_item.parent_doc.id}.html##{bottom_item.id}\" class=\"external\">#{bottom_item.id}</a></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{bottom_f_text}</td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 16%;'>#{document_section}</td>\n"
-                    s += "\t</tr>\n"
-                    top_item_rendered = true
-                    @traced_items[top_item.id.to_s.downcase] = top_item
-                end
-                unless top_item_rendered
-                    s += "\t<tr>\n"
-                    s += "\t\t<td class=\"item_id\"><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
-                    s += "\t\t<td class=\"item_id\"></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 34%;'></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 16%;'></td>\n"
-                    s += "\t</tr>\n"
-                end
-
-            else
-                top_item_rendered = false
-                top_item.down_links.each do |bottom_item|
-
-                    id_color = ""
-
-                    if bottom_item.parent_doc.id == @bottom_doc.id
-
-                        bottom_f_text = bottom_item.format_string( bottom_item.text )
-                        document_section = bottom_item.parent_heading.get_section_info_html
-
-                        s += "\t<tr>\n"
-                        s += "\t\t<td class=\"item_id\" #{id_color}><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
-                        s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
-                        s += "\t\t<td class=\"item_id\"><a href=\"./../#{bottom_item.parent_doc.id}/#{bottom_item.parent_doc.id}.html##{bottom_item.id}\" class=\"external\">#{bottom_item.id}</a></td>\n"
-                        s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{bottom_f_text}</td>\n"
-                        s += "\t\t<td class=\"item_text\" style='width: 16%;'>#{document_section}</td>\n"
-                        s += "\t</tr>\n"
-                        top_item_rendered = true
-                        @traced_items[top_item.id.to_s.downcase] = top_item
-                    end
-                end
-                unless top_item_rendered
-                    s += "\t<tr>\n"
-                    s += "\t\t<td class=\"item_id\" #{id_color}><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
-                    s += "\t\t<td class=\"item_id\"></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 34%;'></td>\n"
-                    s += "\t\t<td class=\"item_text\" style='width: 16%;'></td>\n"
-                    s += "\t</tr>\n"
-                end
-            end
-        else
-            s += "\t<tr>\n"
-            s += "\t\t<td class=\"item_id\"><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
-            s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
-            s += "\t\t<td class=\"item_id\"></td>\n"
-            s += "\t\t<td class=\"item_text\" style='width: 34%;'></td>\n"
-            s += "\t\t<td class=\"item_text\" style='width: 16%;'></td>\n"
-            s += "\t</tr>\n"
+      else
+        top_item_rendered = false
+        top_item.down_links.each do |bottom_item|
+          id_color = ''
+
+          next unless bottom_item.parent_doc.id == @bottom_doc.id
+
+          bottom_f_text = bottom_item.format_string(bottom_item.text)
+          document_section = bottom_item.parent_heading.get_section_info_html
+
+          s += "\t<tr>\n"
+          s += "\t\t<td class=\"item_id\" #{id_color}><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
+          s += "\t\t<td class=\"item_id\"><a href=\"./../#{bottom_item.parent_doc.id}/#{bottom_item.parent_doc.id}.html##{bottom_item.id}\" class=\"external\">#{bottom_item.id}</a></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{bottom_f_text}</td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 16%;'>#{document_section}</td>\n"
+          s += "\t</tr>\n"
+          top_item_rendered = true
+          @traced_items[top_item.id.to_s.downcase] = top_item
+        end
+        unless top_item_rendered
+          s += "\t<tr>\n"
+          s += "\t\t<td class=\"item_id\" #{id_color}><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
+          s += "\t\t<td class=\"item_id\"></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 34%;'></td>\n"
+          s += "\t\t<td class=\"item_text\" style='width: 16%;'></td>\n"
+          s += "\t</tr>\n"
         end
-        return s
+      end
+    else
+      s += "\t<tr>\n"
+      s += "\t\t<td class=\"item_id\"><a href=\"./../#{top_item.parent_doc.id}/#{top_item.parent_doc.id}.html##{top_item.id}\" class=\"external\">#{top_item.id}</a></td>\n"
+      s += "\t\t<td class=\"item_text\" style='width: 34%;'>#{top_f_text}</td>\n"
+      s += "\t\t<td class=\"item_id\"></td>\n"
+      s += "\t\t<td class=\"item_text\" style='width: 34%;'></td>\n"
+      s += "\t\t<td class=\"item_text\" style='width: 16%;'></td>\n"
+      s += "\t</tr>\n"
     end
+    s
+  end
 end

data/lib/almirah/dom/doc_section.rb

@@ -7,7 +7,7 @@ class DocSection
     @parent_section = nil
   end
 
-  def to_html # rubocop:disable Metrics/MethodLength
+  def to_html
     s = ''
     s += "\t<li onclick=\"nav_toggle_expand_list(this, event)\">" \
       '<span class="fa-li"><i class="fa fa-minus-square-o"> </i></span>'

data/lib/almirah/navigation_pane.rb

@@ -1,17 +1,13 @@
-
 class NavigationPane
+  attr_accessor :specifications
 
-    attr_accessor :specifications
+  def initialize(specification)
+    @doc = specification
+  end
 
-    def initialize(specification)
-        @doc = specification
-    end
+  def to_html
+    return @doc.dom.section_tree_to_html if @doc.dom
 
-    def to_html
-        if  @doc.dom
-            return @doc.dom.section_tree_to_html()
-        else
-            return ''
-        end
-    end
-end
+    ''
+  end
+end

data/lib/almirah/project/critical_chain.rb

@@ -0,0 +1,117 @@
+# frozen_string_literal: true
+
+require_relative 'work_item_scheduler'
+
+# Resource-levelled scheduler variant for the critical chain (ADR-195): each
+# row's duration is its focused estimate, and rows are prioritised by the longest
+# downstream chain of focused durations (ties: record sequence, then step). The
+# forward pass, resource levelling, binding-predecessor tracking, and chain
+# tracing are all inherited from WorkItemScheduler.
+class ChainScheduler < WorkItemScheduler
+  def duration_for(work_item)
+    work_item.focused_estimate
+  end
+
+  private
+
+  def priority_key(work_item, memo)
+    [-downstream_length(work_item, memo, []), work_item.record_sequence, work_item.step]
+  end
+
+  # The longest path of focused durations from this row through its in-scope
+  # successors to a sink. Memoised and cycle-guarded.
+  def downstream_length(work_item, memo, stack)
+    return memo[work_item] if memo.key?(work_item)
+    return duration_for(work_item) if stack.include?(work_item)
+
+    stack.push(work_item)
+    tail = scoped_successors(work_item).map { |s| downstream_length(s, memo, stack) }.max || 0
+    stack.pop
+    memo[work_item] = duration_for(work_item) + tail
+  end
+end
+
+# The Gantt's scheduler (ADR-201/195): real focused-estimate durations like the
+# critical-chain scheduler, but rounded up to whole day columns with a one-day
+# minimum so an unestimated row (focused 0) still shows a visible bar. The chain
+# and buffer keep their exact-duration math in ChainScheduler / CriticalChain.
+class GanttScheduler < ChainScheduler
+  def duration_for(work_item)
+    [work_item.focused_estimate.ceil, 1].max
+  end
+end
+
+# The critical chain and project buffer for one decision group's Scope rows
+# (ADR-195). This object serves two distinct jobs that want opposite treatment of
+# completed work, so it keeps two views of the same Scope rows:
+#
+# - the *scheduling* chain (`chain` / `buffer` / `projected_duration`) excludes
+#   Done rows as finished work — you do not re-schedule what is already done — and
+#   reports the duration of the work still remaining;
+# - the *baseline* chain (`baseline_chain` / `baseline_buffer`) keeps every row,
+#   Done included, as the plan was originally sized. Buffer-consumption accounting
+#   (the fever chart, ADR-196) reads this view so that a chain row which overran
+#   and then completed keeps consuming buffer instead of vanishing the moment it
+#   is marked Done, and so the consumption denominator stays a stable plan-time
+#   baseline rather than shrinking as rows finish (issue-207).
+#
+# A predecessor outside the given set (a row in another group) drops out and is
+# treated as an already-available input. The buffer aggregates the safety
+# (safe - focused, clamped at 0) along a chain and cuts it by buffer_ratio,
+# rounded up.
+class CriticalChain
+  def initialize(work_items, buffer_ratio: 0.5)
+    @all_items = work_items
+    @items = work_items.reject(&:done?)
+    @buffer_ratio = buffer_ratio
+    @scheduler = ChainScheduler.new(@items)
+    @baseline_scheduler = ChainScheduler.new(@all_items)
+  end
+
+  # The remaining chain (Done rows excluded) in start order — drives the chain
+  # table and the projected completion of the work still to do.
+  def chain
+    @scheduler.critical_chain
+  end
+
+  # The chain as originally planned, completed rows included (issue-207). Buffer
+  # consumption is accounted against this set so completed overruns stay visible.
+  def baseline_chain
+    @baseline_scheduler.critical_chain
+  end
+
+  # Chain length in working days (the latest finish).
+  def length
+    @scheduler.makespan
+  end
+
+  # ceil(buffer_ratio * Σ_chain max(safe - focused, 0)) over the remaining chain.
+  def buffer
+    buffer_for(chain)
+  end
+
+  # The plan-time buffer over the full baseline chain — the stable denominator for
+  # buffer-consumption %, unaffected by rows completing (issue-207).
+  def baseline_buffer
+    buffer_for(baseline_chain)
+  end
+
+  def projected_duration
+    length + buffer
+  end
+
+  # False when no row carries a positive focused estimate, so the overview marks
+  # the group "unestimated" rather than reporting a misleadingly short plan.
+  # Reads the baseline so a fully-completed group still reports as estimated and
+  # keeps its (100%-complete) fever chart instead of collapsing to "not sized".
+  def estimated?
+    @all_items.any? { |wi| wi.focused_estimate.positive? }
+  end
+
+  private
+
+  def buffer_for(rows)
+    safety = rows.sum { |wi| [wi.safe_estimate - wi.focused_estimate, 0].max }
+    (@buffer_ratio * safety).ceil
+  end
+end

data/lib/almirah/project/doc_linker.rb

@@ -17,7 +17,7 @@ class DocLinker
     result
   end
 
-  def self.link_protocol_to_spec(protocol, specification) # rubocop:disable Metrics/AbcSize,Metrics/CyclomaticComplexity,Metrics/MethodLength,Metrics/PerceivedComplexity
+  def self.link_protocol_to_spec(protocol, specification)
     top_document = specification
     bottom_document = protocol
 
@@ -44,7 +44,7 @@ class DocLinker
     end
   end
 
-  def self.link_decision_to_spec(decision, specification) # rubocop:disable Metrics/AbcSize,Metrics/CyclomaticComplexity,Metrics/MethodLength,Metrics/PerceivedComplexity
+  def self.link_decision_to_spec(decision, specification)
     top_document = specification
     bottom_document = decision
 
@@ -68,7 +68,7 @@ class DocLinker
     end
   end
 
-  def self.link_source_file_to_spec(source_file, specification) # rubocop:disable Metrics/AbcSize,Metrics/CyclomaticComplexity,Metrics/MethodLength,Metrics/PerceivedComplexity
+  def self.link_source_file_to_spec(source_file, specification)
     top_document = specification
     bottom_document = source_file
 
@@ -92,7 +92,7 @@ class DocLinker
     end
   end
 
-  def self.link_two_specifications(doc_a, doc_b) # rubocop:disable Metrics/AbcSize,Metrics/CyclomaticComplexity,Metrics/MethodLength,Metrics/PerceivedComplexity
+  def self.link_two_specifications(doc_a, doc_b)
     if doc_b.up_link_docs.key?(doc_a.id.to_s)
       top_document = doc_a
       bottom_document = doc_b