Almirah 0.4.4 → 0.4.5

data/lib/almirah/project.rb

@@ -2,7 +2,6 @@
 
 require 'fileutils'
 require_relative 'doc_fabric'
-require_relative 'doc_items/work_item'
 require_relative 'navigation_pane'
 require_relative 'doc_types/traceability'
 require_relative 'doc_types/index'
@@ -53,7 +52,6 @@ class Project
     link_all_risks
     check_wrong_specification_referenced
     build_link_registry
-    link_work_items
     create_index
     render_all_specifications(@project_data.specifications)
     render_all_specifications(@project_data.traceability_matrices)
@@ -62,7 +60,6 @@ class Project
     render_all_source_files
     render_all_specifications(@project_data.implementation_matrices) # intentionally after source file rendering
     render_decisions_overview
-    render_critical_chain_page
     render_all_decisions
     render_all_risk_records
     render_risk_registry_pages
@@ -70,7 +67,6 @@ class Project
     render_index
     create_search_data
     report_broken_links
-    report_kit_violations
     report_rendered
   end
 
@@ -87,7 +83,6 @@ class Project
     link_all_risks
     check_wrong_specification_referenced
     build_link_registry
-    link_work_items
     create_index
     render_all_specifications(@project_data.specifications)
     render_all_specifications(@project_data.traceability_matrices)
@@ -96,7 +91,6 @@ class Project
     render_all_source_files
     render_all_specifications(@project_data.implementation_matrices) # intentionally after source file rendering
     render_decisions_overview
-    render_critical_chain_page
     render_all_decisions
     render_all_risk_records
     render_risk_registry_pages
@@ -104,7 +98,6 @@ class Project
     render_index
     create_search_data
     report_broken_links
-    report_kit_violations
     report_rendered
   end
 
@@ -124,111 +117,6 @@ class Project
     broken.each { |b| puts ConsoleReporter.warn_detail("  #{b[:document] || '?'}: #{b[:target]}") }
   end
 
-  # Builds the per-row WorkItem dependency network (ADR-194): registers every
-  # Scope-row work item, fills the intra-record step-order edges, then resolves
-  # each Depends On reference globally (LinkRegistry) to the activity-type-aligned
-  # work item of the target record, tagging each cross-record edge in-group or
-  # cross-group against the decision_groups boundary (ADR-197). Unresolved
-  # references are collected for report_kit_violations. Runs after
-  # build_link_registry (so every record resolves) and before rendering (so the
-  # overview Kit column is ready).
-  def link_work_items
-    @kit_unresolved = []
-    @project_data.decisions.each do |d|
-      d.scope_work_items.each { |wi| @project_data.work_items[wi.id] = wi }
-    end
-    link_intra_record_steps
-    link_cross_record_dependencies
-  end
-
-  # Each row's lower-numbered same-record steps are its predecessors; equal step
-  # numbers are concurrent (no edge). These edges are in-group by definition.
-  def link_intra_record_steps
-    @project_data.decisions.each do |d|
-      items = d.scope_work_items
-      items.each do |wi|
-        items.each do |other|
-          next if other.equal?(wi) || other.step >= wi.step
-
-          wi.add_predecessor(other, cross_group: false)
-          other.add_successor(wi)
-        end
-      end
-    end
-  end
-
-  def link_cross_record_dependencies
-    @project_data.decisions.each do |d|
-      d.scope_work_items.each do |wi|
-        wi.depends_on_refs.each { |ref| link_dependency(d, wi, ref) }
-      end
-    end
-  end
-
-  def link_dependency(record, work_item, ref)
-    target = @project_data.link_registry.find_by_id(ref)
-    unless target.is_a?(Decision)
-      @kit_unresolved << { record: record.id, target: ref }
-      return
-    end
-    prereq = aligned_work_item(target, work_item.activity)
-    return if prereq.nil? || prereq.equal?(work_item)
-
-    cross = decision_group_name(record) != decision_group_name(target)
-    work_item.add_predecessor(prereq, cross_group: cross)
-    prereq.add_successor(work_item)
-    anchor = target.scope_table&.step_column? ? prereq.row_anchor : nil
-    work_item.add_resolved_dependency(ref, target, anchor, prereq.id)
-  end
-
-  # The target record's work item whose activity (Item) matches `activity`,
-  # falling back to the nearest earlier activity by canonical phase order, then
-  # (when the target has only later activities) to its earliest row. nil only
-  # when the target has no Scope rows.
-  def aligned_work_item(target, activity)
-    items = target.scope_work_items
-    return nil if items.empty?
-
-    exact = items.select { |t| t.activity == activity }.min_by(&:step)
-    return exact if exact
-
-    rank = WorkItem::ACTIVITY_ORDER.index(activity) || WorkItem::ACTIVITY_ORDER.length
-    earlier = items.select { |t| t.activity_rank <= rank }
-    return items.min_by { |t| [t.activity_rank, t.step] } if earlier.empty?
-
-    earlier.min_by { |t| [-t.activity_rank, t.step] }
-  end
-
-  # The planning-group name (first-level decisions/ folder) a record belongs to,
-  # read from the decision_groups collection (ADR-197).
-  def decision_group_name(doc)
-    group = @project_data.decision_groups.find { |g| g.values.first.include?(doc) }
-    group&.keys&.first
-  end
-
-  # Reports the two kit gates and unresolved Depends On references (ADR-194), all
-  # as non-failing console warnings alongside report_broken_links.
-  def report_kit_violations
-    @kit_unresolved ||= []
-    phase = @project_data.work_items.each_value.select(&:phase_order_violation?)
-    cross = @project_data.work_items.each_value.select(&:cross_record_violation?)
-    total = phase.length + cross.length + @kit_unresolved.length
-    return if total.zero?
-
-    ConsoleReporter.warn('kit violations', total)
-    phase.each do |wi|
-      blocking = wi.intra_record_predecessors.reject(&:done?).map(&:id).join(', ')
-      puts ConsoleReporter.warn_detail("  phase order: #{wi.id} started before #{blocking}")
-    end
-    cross.each do |wi|
-      blocking = wi.cross_record_predecessors.reject(&:done?).map(&:id).join(', ')
-      puts ConsoleReporter.warn_detail("  not kitted: #{wi.id} needs #{blocking}")
-    end
-    @kit_unresolved.each do |u|
-      puts ConsoleReporter.warn_detail("  unresolved Depends On: #{u[:record]} -> #{u[:target]}")
-    end
-  end
-
   # Assigns each document its generated output path (relative to the build root)
   # and registers it for cross-document link resolution (ADR-186). Runs after all
   # documents are parsed and before any rendering, so link targets are known.
@@ -611,16 +499,6 @@ class Project
     doc.to_html("#{path}/build/decisions/")
   end
 
-  def render_critical_chain_page
-    return if @project_data.decisions.empty?
-
-    path = @configuration.project_root_directory
-    FileUtils.mkdir_p("#{path}/build/decisions")
-
-    doc = DocFabric.create_critical_chain_page(@project)
-    doc.to_html("#{path}/build/decisions/")
-  end
-
   def render_all_decisions
     return if @project_data.decisions.empty?
 

data/lib/almirah/project_configuration.rb

@@ -1,11 +1,6 @@
 require 'yaml'
-require 'date'
 
 class ProjectConfiguration
-  DEFAULT_WIP_LIMIT = 2
-  DEFAULT_BUFFER_RATIO = 0.5
-  DEFAULT_HOURS_PER_DAY = 8
-
   attr_accessor :project_root_directory, :parameters
 
   def initialize(path)
@@ -34,78 +29,6 @@ class ProjectConfiguration
     []
   end
 
-  def get_wip_limit
-    return DEFAULT_WIP_LIMIT unless @parameters.is_a?(Hash)
-
-    planning = @parameters['planning']
-    return DEFAULT_WIP_LIMIT unless planning.is_a?(Hash)
-
-    value = planning['wip_limit']
-    return DEFAULT_WIP_LIMIT unless value.is_a?(Integer) && value.positive?
-
-    value
-  end
-
-  # The CCPM project-buffer ratio (ADR-195): a fraction in (0, 1] cutting the
-  # aggregated chain safety. Absent or out-of-range falls back to the 0.5 default.
-  def get_buffer_ratio
-    return DEFAULT_BUFFER_RATIO unless @parameters.is_a?(Hash)
-
-    planning = @parameters['planning']
-    return DEFAULT_BUFFER_RATIO unless planning.is_a?(Hash)
-
-    value = planning['buffer_ratio']
-    return DEFAULT_BUFFER_RATIO unless value.is_a?(Numeric) && value.positive? && value <= 1
-
-    value
-  end
-
-  # The calendar anchor for working day 1 (ADR-205), a Date parsed from a
-  # DD-MM-YYYY planning.start_date. Absent or unparseable falls back to today,
-  # so an unconfigured project still renders (a moving anchor).
-  def get_start_date
-    date = parse_planning_date(planning_value('start_date'))
-    date || Date.today
-  end
-
-  # Per-group planning start dates (ADR-211): a map from a decision group's
-  # first-level folder name (under decisions/) to its start Date, read from
-  # planning.groups as DD-MM-YYYY entries. Non-string or unparseable values are
-  # dropped; empty when unset. A group absent from the map sequences after the
-  # previous group rather than carrying a declared start.
-  def get_group_start_dates
-    value = planning_value('groups')
-    return {} unless value.is_a?(Hash)
-
-    value.each_with_object({}) do |(name, raw), acc|
-      date = parse_planning_date(raw)
-      acc[name.to_s] = date if date
-    end
-  end
-
-  # The non-working holiday dates (ADR-205) from planning.holidays, each a
-  # DD-MM-YYYY entry; unparseable entries are dropped. Empty when unset.
-  def get_holidays
-    value = planning_value('holidays')
-    return [] unless value.is_a?(Array)
-
-    value.filter_map { |entry| parse_planning_date(entry) }
-  end
-
-  # Working hours per day (ADR-196): converts logged effort hours into the
-  # working-day unit the estimates use. Absent or non-positive falls back to 8.
-  def get_hours_per_day
-    return DEFAULT_HOURS_PER_DAY unless @parameters.is_a?(Hash)
-
-    planning = @parameters['planning']
-    return DEFAULT_HOURS_PER_DAY unless planning.is_a?(Hash)
-
-    value = planning['hours_per_day']
-    return DEFAULT_HOURS_PER_DAY unless value.is_a?(Numeric) && value.positive?
-
-    value
-  end
-
   # The ordered register-column list for a risk registry folder (ADR-216),
   # read from the risks: root — a list of { folder:, columns: } entries.
   # nil when the registry carries no configuration; such a registry renders
@@ -164,26 +87,4 @@ class ProjectConfiguration
   def numeric_threshold(value)
     value.is_a?(Numeric) ? value : nil
   end
-
-  # A value under the planning: key, or nil when planning is absent.
-  def planning_value(key)
-    return nil unless @parameters.is_a?(Hash)
-
-    planning = @parameters['planning']
-    planning.is_a?(Hash) ? planning[key] : nil
-  end
-
-  # Parse a planning date that may already be a Date (YAML ISO form) or a
-  # DD-MM-YYYY string; nil when neither.
-  def parse_planning_date(value)
-    return value if value.is_a?(Date)
-    return nil unless value.is_a?(String)
-
-    match = /\A(\d{2})-(\d{2})-(\d{4})\z/.match(value.strip)
-    return nil unless match
-
-    Date.new(match[3].to_i, match[2].to_i, match[1].to_i)
-  rescue ArgumentError
-    nil
-  end
 end

data/lib/almirah/templates/css/main.css

@@ -147,163 +147,6 @@ table.controlled td.item_text{
     max-width: 100%;
     max-height: 300px;
 }
-.workitem_gantt {
-    --gantt-owner-width: 120px;
-    --gantt-day-width: 26px;
-    --gantt-month-h: 18px;
-    overflow: auto;
-    max-height: 420px;
-    margin-bottom: 16px;
-    border: 1px solid #e1e4e8;
-}
-.workitem_gantt .gantt_grid {
-    display: grid;
-    gap: 0;
-    width: max-content;
-    padding: 4px;
-}
-.workitem_gantt .gantt_corner,
-.workitem_gantt .gantt_owner,
-.workitem_gantt .gantt_owner_tracking,
-.workitem_gantt .gantt_band_corner,
-.workitem_gantt .gantt_buffer {
-    position: sticky;
-    left: 0;
-    background: #fff;
-    padding: 2px 8px;
-    font-weight: 600;
-    white-space: nowrap;
-    border-right: 1px solid #e1e4e8;
-    z-index: 2;
-}
-.workitem_gantt .gantt_corner {
-    top: 0;
-    border-bottom: 1px solid #bbb;
-    z-index: 3;
-}
-.workitem_gantt .gantt_month_head {
-    position: sticky;
-    top: 0;
-    background: #fff;
-    text-align: center;
-    font-size: 11px;
-    font-weight: 600;
-    color: #57606a;
-    border-bottom: 1px solid #e1e4e8;
-    border-left: 1px solid #d0d7de;
-    white-space: nowrap;
-    overflow: hidden;
-    z-index: 1;
-}
-.workitem_gantt .gantt_day_head {
-    position: sticky;
-    top: var(--gantt-month-h);
-    background: #fff;
-    text-align: center;
-    font-size: 11px;
-    color: #57606a;
-    border-bottom: 1px solid #bbb;
-    z-index: 1;
-}
-/* Weekday-holiday highlight in the Chart.js palette red (ADR-205 amendment):
-   the column carries the buffer bar's diagonal stripe texture, in red; the
-   header stays a solid tint so the day number reads. */
-.workitem_gantt .gantt_day_head.gantt_nonworking { background: rgba(255, 99, 132, 0.22); color: #b23a52; }
-.workitem_gantt .gantt_day_head.gantt_friday { background: rgb(225, 241, 250); }
-.workitem_gantt .gantt_nonworking_col {
-    background: repeating-linear-gradient(45deg,
-        rgba(255, 99, 132, 0.20), rgba(255, 99, 132, 0.20) 5px,
-        rgba(255, 99, 132, 0.07) 5px, rgba(255, 99, 132, 0.07) 10px);
-}
-.workitem_gantt .gantt_friday_col { background: rgb(225, 241, 250); }
-.workitem_gantt .gantt_bar,
-.workitem_gantt .gantt_track_bar {
-    font-size: 11px;
-    line-height: 1.5;
-    margin: 1px;
-    padding: 1px 5px;
-    border-radius: 3px;
-    overflow: hidden;
-    white-space: nowrap;
-    text-overflow: ellipsis;
-    color: #fff;
-    cursor: default;
-}
-.workitem_gantt .gantt_todo { background: #9aa5b1; }
-.workitem_gantt .gantt_inprogress { background: #ffffdd; color: #333; box-shadow: inset 0 0 0 1px #bbb; }
-.workitem_gantt .gantt_done { background: #cfc; color: #57606a; box-shadow: inset 0 0 0 1px #e1e4e8; }
-.workitem_gantt .gantt_blocked { box-shadow: inset 0 0 0 1px #bbb; }
-.workitem_gantt .gantt_critical { outline: 1px solid #9a6700; outline-offset: -1px; } /* buffer-bar amber (dark) */
-.workitem_gantt .gantt_release_band {
-    text-align: center;
-    font-size: 11px;
-    font-weight: 600;
-    color: #57606a;
-    background: #eef1f4;
-    border-bottom: 1px solid #e1e4e8;
-    border-left: 1px solid #d0d7de;
-    padding: 2px 4px;
-    white-space: nowrap;
-    overflow: hidden;
-    text-overflow: ellipsis;
-}
-.workitem_gantt .gantt_buffer { color: #9a6700; }
-.critical_chain { margin-bottom: 16px; }
-.critical_chain h2 { font-size: 1.1em; margin-bottom: 6px; }
-.critical_chain .cc_group { margin: 8px 0; }
-.critical_chain .cc_group h3 { font-size: 0.95em; margin: 4px 0; }
-.critical_chain table.cc_chain { border-collapse: collapse; margin: 4px 0; }
-.critical_chain table.cc_chain th,
-.critical_chain table.cc_chain td { border: 1px solid #e1e4e8; padding: 2px 8px; text-align: left; font-size: 12px; }
-.critical_chain .cc_buffer,
-.critical_chain .cc_consumed,
-.critical_chain .cc_projected,
-.critical_chain .cc_finish { margin: 2px 0; font-size: 12px; }
-.critical_chain .cc_consumed { color: #57606a; }
-.critical_chain .cc_finish { font-weight: 600; }
-.critical_chain .cc_unestimated { color: #9a6700; font-style: italic; font-size: 12px; }
-.critical_chain .cc_group_body { display: flex; flex-wrap: wrap; gap: 24px; align-items: flex-start; }
-.critical_chain .cc_plan { flex: 1 1 320px; }
-.critical_chain .cc_fever { flex: 1 1 360px; max-width: 480px; height: 320px; position: relative; }
-.critical_chain .cc_fever .fever_canvas { width: 100%; height: 100%; }
-.workitem_gantt .gantt_buffer_bar {
-    font-size: 11px;
-    line-height: 1.5;
-    margin: 1px;
-    padding: 1px 5px;
-    border-radius: 3px;
-    text-align: center;
-    overflow: hidden;
-    white-space: nowrap;
-    text-overflow: ellipsis;
-    color: #9a6700;
-    background: repeating-linear-gradient(45deg, #ffe9b3, #ffe9b3 5px, #fff3d4 5px, #fff3d4 10px);
-    box-shadow: inset 0 0 0 1px #e0c068;
-    cursor: default;
-}
-.workitem_gantt .gantt_today { border-left: 2px solid #d33; z-index: 5; pointer-events: none; }
-/* Plan-vs-actual tracking lanes (ADR-213): hidden until the toolbar toggle adds
-   show-actuals; a hidden lane's grid tracks collapse to zero height. */
-.workitem_gantt .gantt_tracking_lane { display: none; }
-.workitem_gantt.show-actuals .gantt_tracking_lane { display: block; }
-.workitem_gantt .gantt_owner_tracking { font-weight: 400; font-style: italic; color: #57606a; }
-.workitem_gantt .gantt_track_committed { background: #b0b6bd; }
-.workitem_gantt .gantt_track_logged { background: #36a2eb; }
-.gantt_container .gantt_toolbar {
-    display: flex;
-    justify-content: flex-end;
-    gap: 6px;
-    padding: 4px 0;
-}
-.gantt_container .gantt_actuals_toggle {
-    font-size: 12px;
-    padding: 3px 10px;
-    border: 1px solid #d0d7de;
-    border-radius: 4px;
-    background: #f6f8fa;
-    cursor: pointer;
-}
-.gantt_container .gantt_actuals_toggle:hover { background: #eef1f4; }
 table.controlled:not(.odd-even) tbody tr:nth-child(odd) { background-color:#f6f7f8; }
 table.controlled:not(.odd-even) tbody tr:nth-child(even) { background-color: #fff; }
 table.controlled:not(.odd-even) tbody tr:nth-child(odd):hover,

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: Almirah
 version: !ruby/object:Gem::Version
-  version: 0.4.4
+  version: 0.4.5
 platform: ruby
 authors:
 - Oleksandr Ivanov
@@ -77,13 +77,10 @@ files:
 - lib/almirah/doc_items/source_code_paragraph.rb
 - lib/almirah/doc_items/text_line.rb
 - lib/almirah/doc_items/todo_block.rb
-- lib/almirah/doc_items/work_item.rb
 - lib/almirah/doc_parser.rb
 - lib/almirah/doc_types/base_document.rb
 - lib/almirah/doc_types/coverage.rb
-- lib/almirah/doc_types/critical_chain_page.rb
 - lib/almirah/doc_types/decision.rb
-- lib/almirah/doc_types/decision_grouping.rb
 - lib/almirah/doc_types/decisions_overview.rb
 - lib/almirah/doc_types/implementation.rb
 - lib/almirah/doc_types/index.rb
@@ -103,12 +100,8 @@ files:
 - lib/almirah/link_registry.rb
 - lib/almirah/navigation_pane.rb
 - lib/almirah/project.rb
-- lib/almirah/project/critical_chain.rb
 - lib/almirah/project/doc_linker.rb
-- lib/almirah/project/fever_chart.rb
 - lib/almirah/project/project_data.rb
-- lib/almirah/project/work_item_scheduler.rb
-- lib/almirah/project/working_calendar.rb
 - lib/almirah/project_configuration.rb
 - lib/almirah/project_template.rb
 - lib/almirah/project_utility.rb

data/lib/almirah/doc_items/work_item.rb

@@ -1,129 +0,0 @@
-# frozen_string_literal: true
-
-# A single Scope-table row of a Decision Record, modelled as a node in the
-# per-row dependency network (ADR-194). Each work item carries its canonical
-# identity (`<record>.<step>.<activity>`), the bounded per-row Status it reads
-# its readiness from (ADR-193), and predecessor / successor edges spanning both
-# intra-record (step order) and cross-record (Depends On) links.
-#
-# `predecessors` / `successors` follow the house list-of-single-key-hashes shape
-# (ADR-197): each entry maps a readable `record.step.activity` label to the
-# resolved WorkItem. Every cross-record edge additionally records whether it
-# stays within a decision_groups folder (in-group) or crosses to another
-# (cross-group); that tag is inert here and consumed by the critical-chain step
-# (ADR-195). Readiness ignores it — a cross-group predecessor blocks exactly
-# like an in-group one.
-class WorkItem
-  # Canonical phase order used only as the fallback for activity-type-aligned
-  # resolution when the prerequisite has no row of the dependent's exact Item.
-  ACTIVITY_ORDER = %w[Analysis Requirements Code Tests].freeze
-
-  attr_reader :record_id, :step, :activity, :owner, :status, :depends_on_refs,
-              :focused_estimate, :safe_estimate, :record_sequence, :start_date, :target_date
-  attr_accessor :predecessors, :successors, :cross_group_predecessor_labels, :resolved_dependencies
-
-  def initialize(record_id:, step:, activity:, owner:, status:, depends_on_refs:, # rubocop:disable Metrics/ParameterLists
-                 focused_estimate: 0.0, safe_estimate: 0.0, record_sequence: 0,
-                 start_date: nil, target_date: nil)
-    @record_id = record_id
-    @step = step
-    @activity = activity.to_s.strip
-    @owner = owner.to_s.strip
-    @status = status.to_s.strip
-    @depends_on_refs = depends_on_refs # array of record ref strings, e.g. ["ADR-193"]
-    # The row's authored committed window (ADR-213): the Scope Start Date and
-    # Target Date as parsed Dates (nil when absent/unparseable). Distinct from the
-    # computed schedule -- these are the author's intent, drawn on the tracking lane.
-    @start_date = start_date
-    @target_date = target_date
-    # Per-row scheduling estimates in working days (ADR-195); the focused estimate
-    # is the scheduling duration, safe - focused the safety aggregated into the
-    # project buffer. record_sequence is the owning record's number, used only as
-    # a deterministic priority tiebreak in the critical-chain scheduler.
-    @focused_estimate = focused_estimate
-    @safe_estimate = safe_estimate
-    @record_sequence = record_sequence
-    @predecessors = []
-    @successors = []
-    @cross_group_predecessor_labels = []
-    # One entry per resolved Depends On reference, keyed by the authored ref:
-    # { ref => { doc:, anchor:, label: } }. `anchor` is the target's resolved
-    # work-item row anchor (nil when the target has no `#` step column, so the
-    # link opens the record page); `label` is the resolved work item's id, for
-    # the link tooltip. Used to render the Depends On cell as deep links.
-    @resolved_dependencies = {}
-  end
-
-  def id
-    "#{@record_id}.#{@step}.#{@activity}"
-  end
-
-  # The anchor of this work item's row in its rendered Scope table — the deep-link
-  # target a dependent record points at. Namespaced with `.scope.` so it never
-  # collides with the Affected Documents controlled table, whose rows anchor on
-  # the same `<record>.<step>` scheme on the same page.
-  def row_anchor
-    "#{@record_id}.scope.#{@step}"
-  end
-
-  def add_resolved_dependency(ref, doc, anchor, label)
-    @resolved_dependencies[ref] = { doc: doc, anchor: anchor, label: label }
-  end
-
-  def activity_rank
-    ACTIVITY_ORDER.index(@activity) || ACTIVITY_ORDER.length
-  end
-
-  # A row is "started" once it leaves To Do; both In-Progress and Done count, so
-  # a Done row whose predecessor never finished is still flagged as a violation.
-  def started?
-    @status == 'In-Progress' || @status == 'Done'
-  end
-
-  def done?
-    @status == 'Done'
-  end
-
-  def add_predecessor(work_item, cross_group:)
-    @predecessors << { work_item.id => work_item }
-    @cross_group_predecessor_labels << work_item.id if cross_group
-  end
-
-  def add_successor(work_item)
-    @successors << { work_item.id => work_item }
-  end
-
-  def predecessor_items
-    @predecessors.map { |edge| edge.values.first }
-  end
-
-  def successor_items
-    @successors.map { |edge| edge.values.first }
-  end
-
-  # Same-record, lower-numbered steps — the intra-record (phase-order) edges.
-  def intra_record_predecessors
-    predecessor_items.select { |p| p.record_id == @record_id }
-  end
-
-  # Resolved Depends On edges into other records (in-group or cross-group alike).
-  def cross_record_predecessors
-    predecessor_items.reject { |p| p.record_id == @record_id }
-  end
-
-  # Kitted when every predecessor — intra- and cross-record, regardless of the
-  # in-group / cross-group tag — is Done; trivially kitted when it has none.
-  def fully_kitted?
-    predecessor_items.all?(&:done?)
-  end
-
-  # A started row with an unfinished lower-numbered step in the same record.
-  def phase_order_violation?
-    started? && intra_record_predecessors.any? { |p| !p.done? }
-  end
-
-  # A started row whose resolved cross-record predecessor is not yet Done.
-  def cross_record_violation?
-    started? && cross_record_predecessors.any? { |p| !p.done? }
-  end
-end