@forwardimpact/pathway 0.25.12 → 0.25.20

package/src/formatters/job/description.js

@@ -13,6 +13,80 @@ import Mustache from "mustache";
 import { BEHAVIOUR_MATURITY_ORDER } from "@forwardimpact/map/levels";
 import { trimValue, trimFields } from "../shared.js";
 
+/**
+ * Build expectations paragraph from job expectations
+ * @param {Object|undefined} expectations
+ * @returns {string}
+ */
+function buildExpectationsParagraph(expectations) {
+  if (!expectations) return "";
+  const exp = expectations;
+  const sentences = [];
+
+  if (exp.impactScope) {
+    sentences.push(`This role encompasses ${exp.impactScope.toLowerCase()}.`);
+  }
+  if (exp.autonomyExpectation) {
+    let autonomySentence = `You will ${exp.autonomyExpectation.toLowerCase()}`;
+    if (exp.influenceScope) {
+      autonomySentence +=
+        `, ${exp.influenceScope.toLowerCase()}` +
+        (exp.influenceScope.endsWith(".") ? "" : ".");
+    } else {
+      autonomySentence += exp.autonomyExpectation.endsWith(".") ? "" : ".";
+    }
+    sentences.push(autonomySentence);
+  } else if (exp.influenceScope) {
+    sentences.push(
+      exp.influenceScope + (exp.influenceScope.endsWith(".") ? "" : "."),
+    );
+  }
+  if (exp.complexityHandled) {
+    sentences.push(`You will handle ${exp.complexityHandled.toLowerCase()}.`);
+  }
+  return sentences.length > 0 ? sentences.join(" ") : "";
+}
+
+/**
+ * Build capability skill sections at the highest proficiency
+ * @param {Object} job
+ * @returns {Array}
+ */
+function buildCapabilitySkills(job) {
+  const derivedResponsibilities = job.derivedResponsibilities || [];
+  if (derivedResponsibilities.length === 0) return [];
+
+  const highestProficiency = derivedResponsibilities[0].proficiency;
+  const topResponsibilities = derivedResponsibilities.filter(
+    (r) => r.proficiency === highestProficiency,
+  );
+
+  const skillsByCapability = {};
+  for (const skill of job.skillMatrix) {
+    if (skill.proficiency !== highestProficiency) continue;
+    if (!skillsByCapability[skill.capability]) {
+      skillsByCapability[skill.capability] = [];
+    }
+    skillsByCapability[skill.capability].push(skill);
+  }
+
+  return topResponsibilities
+    .filter((r) => skillsByCapability[r.capability]?.length > 0)
+    .map((r) => {
+      const skills = [...skillsByCapability[r.capability]].sort((a, b) =>
+        (a.skillName || "").localeCompare(b.skillName || ""),
+      );
+      return {
+        capabilityHeading: r.capabilityName.toUpperCase(),
+        responsibilityDescription: r.responsibility,
+        skills: skills.map((s) => ({
+          skillName: s.skillName,
+          proficiencyDescription: s.proficiencyDescription || "",
+        })),
+      };
+    });
+}
+
 /**
  * Prepare job data for template rendering
  * @param {Object} params
@@ -29,42 +103,7 @@ function prepareJobDescriptionData({ job, discipline, level, track }) {
   roleSummary = roleSummary.replace(/\{roleTitle\}/g, roleTitle);
   roleSummary = roleSummary.replace(/\{specialization\}/g, specialization);
 
-  // Build expectations paragraph
-  let expectationsParagraph = "";
-  if (job.expectations) {
-    const exp = job.expectations;
-    const expectationSentences = [];
-
-    if (exp.impactScope) {
-      expectationSentences.push(
-        `This role encompasses ${exp.impactScope.toLowerCase()}.`,
-      );
-    }
-    if (exp.autonomyExpectation) {
-      let autonomySentence = `You will ${exp.autonomyExpectation.toLowerCase()}`;
-      if (exp.influenceScope) {
-        autonomySentence +=
-          `, ${exp.influenceScope.toLowerCase()}` +
-          (exp.influenceScope.endsWith(".") ? "" : ".");
-      } else {
-        autonomySentence += exp.autonomyExpectation.endsWith(".") ? "" : ".";
-      }
-      expectationSentences.push(autonomySentence);
-    } else if (exp.influenceScope) {
-      expectationSentences.push(
-        exp.influenceScope + (exp.influenceScope.endsWith(".") ? "" : "."),
-      );
-    }
-    if (exp.complexityHandled) {
-      expectationSentences.push(
-        `You will handle ${exp.complexityHandled.toLowerCase()}.`,
-      );
-    }
-
-    if (expectationSentences.length > 0) {
-      expectationsParagraph = expectationSentences.join(" ");
-    }
-  }
+  const expectationsParagraph = buildExpectationsParagraph(job.expectations);
 
   // Sort behaviours by maturity level (highest first)
   const sortedBehaviours = [...job.behaviourProfile].sort((a, b) => {
@@ -76,45 +115,7 @@ function prepareJobDescriptionData({ job, discipline, level, track }) {
     return indexB - indexA;
   });
 
-  // Build capability skill sections at the highest skill proficiency
-  let capabilitySkills = [];
-  const derivedResponsibilities = job.derivedResponsibilities || [];
-  if (derivedResponsibilities.length > 0) {
-    // derivedResponsibilities is sorted: highest proficiency first, then by skill count
-    const highestProficiency = derivedResponsibilities[0].proficiency;
-
-    // Filter responsibilities to only the highest proficiency
-    const topResponsibilities = derivedResponsibilities.filter(
-      (r) => r.proficiency === highestProficiency,
-    );
-
-    // Group skill matrix entries by capability at the highest level
-    const skillsByCapability = {};
-    for (const skill of job.skillMatrix) {
-      if (skill.proficiency !== highestProficiency) continue;
-      if (!skillsByCapability[skill.capability]) {
-        skillsByCapability[skill.capability] = [];
-      }
-      skillsByCapability[skill.capability].push(skill);
-    }
-
-    // Build capability sections in skill count order
-    capabilitySkills = topResponsibilities
-      .filter((r) => skillsByCapability[r.capability]?.length > 0)
-      .map((r) => {
-        const skills = [...skillsByCapability[r.capability]].sort((a, b) =>
-          (a.skillName || "").localeCompare(b.skillName || ""),
-        );
-        return {
-          capabilityHeading: r.capabilityName.toUpperCase(),
-          responsibilityDescription: r.responsibility,
-          skills: skills.map((s) => ({
-            skillName: s.skillName,
-            proficiencyDescription: s.proficiencyDescription || "",
-          })),
-        };
-      });
-  }
+  const capabilitySkills = buildCapabilitySkills(job);
 
   // Build qualification summary with placeholder replacement
   const qualificationSummary =

package/src/formatters/job/dom.js

@@ -33,6 +33,107 @@ import { createToolkitTable } from "../toolkit/dom.js";
  * @param {string} [options.jobTemplate] - Mustache template for job description
  * @returns {HTMLElement}
  */
+/**
+ * Build a job data object from a view for template rendering
+ * @param {Object} view
+ * @returns {Object}
+ */
+function buildJobFromView(view) {
+  return {
+    title: view.title,
+    skillMatrix: view.skillMatrix,
+    behaviourProfile: view.behaviourProfile,
+    expectations: view.expectations,
+    derivedResponsibilities: view.derivedResponsibilities,
+  };
+}
+
+/**
+ * Create the tables section for a job detail view
+ * @param {Object} view
+ * @returns {HTMLElement}
+ */
+function createJobTablesSection(view) {
+  return div(
+    { className: "job-tables-section" },
+    createDetailSection({
+      title: "Behaviour Profile",
+      content: createBehaviourProfile(view.behaviourProfile),
+    }),
+    createDetailSection({
+      title: "Skill Matrix",
+      content: createSkillMatrix(view.skillMatrix, {
+        capabilityOrder: view.capabilityOrder,
+      }),
+    }),
+    view.toolkit && view.toolkit.length > 0
+      ? createDetailSection({
+          title: "Tool Kit",
+          content: createToolkitTable(view.toolkit),
+        })
+      : null,
+  );
+}
+
+/**
+ * Create the page header with breadcrumb links
+ * @param {Object} view
+ * @param {boolean} showBackLink
+ * @returns {HTMLElement}
+ */
+function createJobHeader(view, showBackLink) {
+  return div(
+    { className: "page-header" },
+    showBackLink
+      ? createBackLink("/job-builder", "← Back to Job Builder")
+      : null,
+    h1({ className: "page-title" }, view.title),
+    div(
+      { className: "page-description" },
+      "Generated from: ",
+      a({ href: `#/discipline/${view.disciplineId}` }, view.disciplineName),
+      " × ",
+      a({ href: `#/level/${view.levelId}` }, view.levelId),
+      " × ",
+      a({ href: `#/track/${view.trackId}` }, view.trackName),
+    ),
+  );
+}
+
+/**
+ * Create the expectations section if expectations exist
+ * @param {Object} view
+ * @returns {HTMLElement|null}
+ */
+function createExpectationsSection(view) {
+  if (!view.expectations || Object.keys(view.expectations).length === 0) {
+    return null;
+  }
+  return createDetailSection({
+    title: "Expectations",
+    content: createExpectationsCard(view.expectations),
+  });
+}
+
+/**
+ * Create the radar charts section
+ * @param {Object} view
+ * @returns {HTMLElement}
+ */
+function createRadarSection(view) {
+  return div(
+    { className: "section auto-grid-lg" },
+    createBehaviourRadar(view.behaviourProfile, {
+      title: "Behaviours Radar",
+      size: 420,
+    }),
+    createSkillRadar(view.skillMatrix, {
+      title: "Skills Radar",
+      size: 420,
+    }),
+  );
+}
+
 export function jobToDOM(view, options = {}) {
   const {
     showBackLink = true,
@@ -46,107 +147,22 @@ export function jobToDOM(view, options = {}) {
   } = options;
 
   const hasEntities = discipline && level && jobTemplate;
+  const job = hasEntities ? buildJobFromView(view) : null;
+  const descParams = hasEntities
+    ? { job, discipline, level, track, template: jobTemplate }
+    : null;
 
   return div(
     { className: "job-detail-page" },
-    // Header
-    div(
-      { className: "page-header" },
-      showBackLink
-        ? createBackLink("/job-builder", "← Back to Job Builder")
-        : null,
-      h1({ className: "page-title" }, view.title),
-      div(
-        { className: "page-description" },
-        "Generated from: ",
-        a({ href: `#/discipline/${view.disciplineId}` }, view.disciplineName),
-        " × ",
-        a({ href: `#/level/${view.levelId}` }, view.levelId),
-        " × ",
-        a({ href: `#/track/${view.trackId}` }, view.trackName),
-      ),
-    ),
-
-    // Expectations card
-    view.expectations && Object.keys(view.expectations).length > 0
-      ? createDetailSection({
-          title: "Expectations",
-          content: createExpectationsCard(view.expectations),
-        })
+    createJobHeader(view, showBackLink),
+    createExpectationsSection(view),
+    createRadarSection(view),
+    showJobDescriptionHtml && descParams
+      ? createJobDescriptionHtml(descParams)
       : null,
-
-    // Radar charts
-    div(
-      { className: "section auto-grid-lg" },
-      createBehaviourRadar(view.behaviourProfile, {
-        title: "Behaviours Radar",
-        size: 420,
-      }),
-      createSkillRadar(view.skillMatrix, {
-        title: "Skills Radar",
-        size: 420,
-      }),
-    ),
-
-    // Job Description HTML (for print view)
-    showJobDescriptionHtml && hasEntities
-      ? createJobDescriptionHtml({
-          job: {
-            title: view.title,
-            skillMatrix: view.skillMatrix,
-            behaviourProfile: view.behaviourProfile,
-            expectations: view.expectations,
-            derivedResponsibilities: view.derivedResponsibilities,
-          },
-          discipline,
-          level,
-          track,
-          template: jobTemplate,
-        })
-      : null,
-
-    // Behaviour profile, Skill matrix, Toolkit, Driver coverage tables
-    showTables
-      ? div(
-          { className: "job-tables-section" },
-          // Behaviour profile table
-          createDetailSection({
-            title: "Behaviour Profile",
-            content: createBehaviourProfile(view.behaviourProfile),
-          }),
-
-          createDetailSection({
-            title: "Skill Matrix",
-            content: createSkillMatrix(view.skillMatrix, {
-              capabilityOrder: view.capabilityOrder,
-            }),
-          }),
-
-          // Toolkit (after skill matrix)
-          view.toolkit && view.toolkit.length > 0
-            ? createDetailSection({
-                title: "Tool Kit",
-                content: createToolkitTable(view.toolkit),
-              })
-            : null,
-        )
-      : null,
-
-    // Job Description (copyable markdown)
-    showJobDescriptionMarkdown && hasEntities
-      ? createJobDescriptionSection({
-          job: {
-            title: view.title,
-            skillMatrix: view.skillMatrix,
-            behaviourProfile: view.behaviourProfile,
-            expectations: view.expectations,
-            derivedResponsibilities: view.derivedResponsibilities,
-          },
-          discipline,
-          level,
-          track,
-          template: jobTemplate,
-        })
+    showTables ? createJobTablesSection(view) : null,
+    showJobDescriptionMarkdown && descParams
+      ? createJobDescriptionSection(descParams)
       : null,
   );
 }

package/src/formatters/level/dom.js

@@ -35,14 +35,98 @@ import { createJsonLdScript, levelToJsonLd } from "../json-ld.js";
  * @param {boolean} [options.showBackLink=true] - Whether to show back navigation link
  * @returns {HTMLElement}
  */
+/**
+ * Create a proficiency row for the base skill table
+ * @param {string} label
+ * @param {string} badgeClass
+ * @param {string|undefined} proficiency
+ * @returns {HTMLElement}
+ */
+function createProficiencyRow(label, badgeClass, proficiency) {
+  return tr(
+    {},
+    td({}, span({ className: `badge ${badgeClass}` }, label)),
+    td(
+      {},
+      proficiency
+        ? createLevelDots(
+            SKILL_PROFICIENCY_ORDER.indexOf(proficiency),
+            SKILL_PROFICIENCY_ORDER.length,
+          )
+        : span({ className: "text-muted" }, "—"),
+    ),
+  );
+}
+
+/**
+ * Create the base skill proficiencies and behaviour maturity section
+ * @param {Object} view
+ * @returns {HTMLElement}
+ */
+function createBaseProfileSection(view) {
+  const profs = view.baseSkillProficiencies || {};
+  return div(
+    { className: "detail-section" },
+    div(
+      { className: "content-columns" },
+      div(
+        { className: "column" },
+        heading2({ className: "section-title" }, "Base Skill Proficiencies"),
+        table(
+          { className: "level-table" },
+          thead({}, tr({}, th({}, "Type"), th({}, "Level"))),
+          tbody(
+            {},
+            createProficiencyRow("Primary", "badge-primary", profs.primary),
+            createProficiencyRow(
+              "Secondary",
+              "badge-secondary",
+              profs.secondary,
+            ),
+            createProficiencyRow("Broad", "badge-broad", profs.broad),
+          ),
+        ),
+      ),
+      div(
+        { className: "column" },
+        heading2({ className: "section-title" }, "Base Behaviour Maturity"),
+        view.baseBehaviourMaturity
+          ? table(
+              { className: "level-table" },
+              thead({}, tr({}, th({}, "Maturity"), th({}, "Level"))),
+              tbody(
+                {},
+                tr(
+                  {},
+                  td(
+                    {},
+                    view.baseBehaviourMaturity.charAt(0).toUpperCase() +
+                      view.baseBehaviourMaturity.slice(1),
+                  ),
+                  td(
+                    {},
+                    createLevelDots(
+                      BEHAVIOUR_MATURITY_ORDER.indexOf(
+                        view.baseBehaviourMaturity,
+                      ),
+                      BEHAVIOUR_MATURITY_ORDER.length,
+                    ),
+                  ),
+                ),
+              ),
+            )
+          : p({ className: "text-muted" }, "—"),
+      ),
+    ),
+  );
+}
+
 export function levelToDOM(level, { framework, showBackLink = true } = {}) {
   const view = prepareLevelDetail(level);
   const emoji = framework ? getConceptEmoji(framework, "level") : "📊";
   return div(
     { className: "detail-page level-detail" },
-    // JSON-LD structured data
     createJsonLdScript(levelToJsonLd(level)),
-    // Header
     div(
       { className: "page-header" },
       showBackLink ? createBackLink("/level", "← Back to Levels") : null,
@@ -68,7 +152,6 @@ export function levelToDOM(level, { framework, showBackLink = true } = {}) {
         : null,
     ),
 
-    // Titles section
     view.professionalTitle || view.managementTitle
       ? div(
           { className: "detail-section" },
@@ -93,7 +176,6 @@ export function levelToDOM(level, { framework, showBackLink = true } = {}) {
         )
       : null,
 
-    // Expectations
     view.expectations && Object.keys(view.expectations).length > 0
       ? div(
           { className: "detail-section" },
@@ -111,103 +193,6 @@ export function levelToDOM(level, { framework, showBackLink = true } = {}) {
         )
       : null,
 
-    // Base Skill Proficiencies and Base Behaviour Maturity in two columns
-    div(
-      { className: "detail-section" },
-      div(
-        { className: "content-columns" },
-        // Base Skill Proficiencies column
-        div(
-          { className: "column" },
-          heading2({ className: "section-title" }, "Base Skill Proficiencies"),
-          table(
-            { className: "level-table" },
-            thead({}, tr({}, th({}, "Type"), th({}, "Level"))),
-            tbody(
-              {},
-              tr(
-                {},
-                td({}, span({ className: "badge badge-primary" }, "Primary")),
-                td(
-                  {},
-                  view.baseSkillProficiencies?.primary
-                    ? createLevelDots(
-                        SKILL_PROFICIENCY_ORDER.indexOf(
-                          view.baseSkillProficiencies.primary,
-                        ),
-                        SKILL_PROFICIENCY_ORDER.length,
-                      )
-                    : span({ className: "text-muted" }, "—"),
-                ),
-              ),
-              tr(
-                {},
-                td(
-                  {},
-                  span({ className: "badge badge-secondary" }, "Secondary"),
-                ),
-                td(
-                  {},
-                  view.baseSkillProficiencies?.secondary
-                    ? createLevelDots(
-                        SKILL_PROFICIENCY_ORDER.indexOf(
-                          view.baseSkillProficiencies.secondary,
-                        ),
-                        SKILL_PROFICIENCY_ORDER.length,
-                      )
-                    : span({ className: "text-muted" }, "—"),
-                ),
-              ),
-              tr(
-                {},
-                td({}, span({ className: "badge badge-broad" }, "Broad")),
-                td(
-                  {},
-                  view.baseSkillProficiencies?.broad
-                    ? createLevelDots(
-                        SKILL_PROFICIENCY_ORDER.indexOf(
-                          view.baseSkillProficiencies.broad,
-                        ),
-                        SKILL_PROFICIENCY_ORDER.length,
-                      )
-                    : span({ className: "text-muted" }, "—"),
-                ),
-              ),
-            ),
-          ),
-        ),
-        // Base Behaviour Maturity column
-        div(
-          { className: "column" },
-          heading2({ className: "section-title" }, "Base Behaviour Maturity"),
-          view.baseBehaviourMaturity
-            ? table(
-                { className: "level-table" },
-                thead({}, tr({}, th({}, "Maturity"), th({}, "Level"))),
-                tbody(
-                  {},
-                  tr(
-                    {},
-                    td(
-                      {},
-                      view.baseBehaviourMaturity.charAt(0).toUpperCase() +
-                        view.baseBehaviourMaturity.slice(1),
-                    ),
-                    td(
-                      {},
-                      createLevelDots(
-                        BEHAVIOUR_MATURITY_ORDER.indexOf(
-                          view.baseBehaviourMaturity,
-                        ),
-                        BEHAVIOUR_MATURITY_ORDER.length,
-                      ),
-                    ),
-                  ),
-                ),
-              )
-            : p({ className: "text-muted" }, "—"),
-        ),
-      ),
-    ),
+    createBaseProfileSection(view),
   );
 }

package/src/formatters/questions/markdown.js

@@ -223,10 +223,43 @@ function formatTable(view) {
 }
 
 /**
- * Format single source deep dive
- * @param {Object} view - Questions view
- * @returns {string}
+ * Format a single question's detail lines
+ * @param {string[]} lines
+ * @param {Object} q - Flattened question
  */
+function formatQuestionDetail(lines, q) {
+  lines.push(`  • [${q.id}] ${q.text}`);
+  lines.push(`    Duration: ${q.expectedDurationMinutes} min`);
+  if (q.context) {
+    lines.push(`    Context: ${q.context}`);
+  }
+  if (q.simulationPrompts && q.simulationPrompts.length > 0) {
+    lines.push("    Steer the simulation:");
+    for (const prompt of q.simulationPrompts) {
+      lines.push(`      - ${prompt}`);
+    }
+  }
+  if (q.decompositionPrompts && q.decompositionPrompts.length > 0) {
+    lines.push("    Guide candidate thinking:");
+    for (const prompt of q.decompositionPrompts) {
+      lines.push(`      - ${prompt}`);
+    }
+  }
+  if (q.lookingFor.length > 0) {
+    lines.push("    Looking for:");
+    for (const item of q.lookingFor) {
+      lines.push(`      - ${item}`);
+    }
+  }
+  if (q.followUps.length > 0) {
+    lines.push("    Follow-ups:");
+    for (const fu of q.followUps) {
+      lines.push(`      → ${fu}`);
+    }
+  }
+  lines.push("");
+}
+
 function formatSingleSource(view) {
   const lines = [];
   const { questions, stats } = view;
@@ -259,36 +292,7 @@ function formatSingleSource(view) {
 
     lines.push(level.toUpperCase());
     for (const q of byLevel[level]) {
-      lines.push(`  • [${q.id}] ${q.text}`);
-      lines.push(`    Duration: ${q.expectedDurationMinutes} min`);
-      if (q.context) {
-        lines.push(`    Context: ${q.context}`);
-      }
-      if (q.simulationPrompts && q.simulationPrompts.length > 0) {
-        lines.push("    Steer the simulation:");
-        for (const prompt of q.simulationPrompts) {
-          lines.push(`      - ${prompt}`);
-        }
-      }
-      if (q.decompositionPrompts && q.decompositionPrompts.length > 0) {
-        lines.push("    Guide candidate thinking:");
-        for (const prompt of q.decompositionPrompts) {
-          lines.push(`      - ${prompt}`);
-        }
-      }
-      if (q.lookingFor.length > 0) {
-        lines.push("    Looking for:");
-        for (const item of q.lookingFor) {
-          lines.push(`      - ${item}`);
-        }
-      }
-      if (q.followUps.length > 0) {
-        lines.push("    Follow-ups:");
-        for (const fu of q.followUps) {
-          lines.push(`      → ${fu}`);
-        }
-      }
-      lines.push("");
+      formatQuestionDetail(lines, q);
     }
   }