From 017f35b0f49b44b93e2a511090b2c0c554f76259 Mon Sep 17 00:00:00 2001
From: Brian Schwartz <brian.schwartz@centricconsulting.com>
Date: Thu, 16 Jul 2026 16:08:14 +0000
Subject: [PATCH] feat: show optional experience location on resume date lines

Append role location after the date range in DOCX/PDF resumes, tighten template spacing, and render education as a single pipe-separated line.

Co-authored-by: Cursor <cursoragent@cursor.com>
---
 data/profile-dev.sample.json          |  3 ++
 data/profile.sample.json              |  3 ++
 lib/__tests__/resume-format.test.ts   | 36 +++++++++++++++
 lib/__tests__/resume-template.test.ts | 66 +++++++++++++++++++++++++++
 lib/docx-resume.ts                    |  3 +-
 lib/profile.ts                        |  1 +
 lib/resume-format.ts                  |  5 +-
 lib/resume-template.ts                | 31 ++++++-------
 8 files changed, 128 insertions(+), 20 deletions(-)

diff --git a/data/profile-dev.sample.json b/data/profile-dev.sample.json
index 1a939e9..d513def 100644
--- a/data/profile-dev.sample.json
+++ b/data/profile-dev.sample.json
@@ -15,6 +15,7 @@
     {
       "company": "Clarion Systems",
       "role": "Staff Software Engineer",
+      "location": "Remote",
       "start": "2022-07",
       "end": null,
       "bullets": [
@@ -28,6 +29,7 @@
     {
       "company": "Beacon Labs",
       "role": "Senior Software Engineer",
+      "location": "Austin, TX",
       "start": "2019-04",
       "end": "2022-06",
       "bullets": [
@@ -40,6 +42,7 @@
     {
       "company": "Trove Digital",
       "role": "Software Engineer",
+      "location": "Austin, TX",
       "start": "2016-08",
       "end": "2019-03",
       "bullets": [
diff --git a/data/profile.sample.json b/data/profile.sample.json
index 150fbd1..6d89812 100644
--- a/data/profile.sample.json
+++ b/data/profile.sample.json
@@ -15,6 +15,7 @@
     {
       "company": "Meridian Software",
       "role": "Director of Product Design",
+      "location": "Remote",
       "start": "2023-03",
       "end": null,
       "bullets": [
@@ -28,6 +29,7 @@
     {
       "company": "Lumen Analytics",
       "role": "Senior Product Designer",
+      "location": "Chicago, IL",
       "start": "2020-06",
       "end": "2023-02",
       "bullets": [
@@ -41,6 +43,7 @@
     {
       "company": "Clearpath Digital",
       "role": "UX Designer",
+      "location": "Chicago, IL",
       "start": "2017-08",
       "end": "2020-05",
       "bullets": [
diff --git a/lib/__tests__/resume-format.test.ts b/lib/__tests__/resume-format.test.ts
index bbd37ac..439595a 100644
--- a/lib/__tests__/resume-format.test.ts
+++ b/lib/__tests__/resume-format.test.ts
@@ -96,6 +96,42 @@ describe("formatDateRange", () => {
     // Assert
     expect(range).toBe("December 2011 – January 2012");
   });
+
+  it("omits the location segment entirely when location is undefined", () => {
+    // Arrange
+    const start = "2024-10";
+    const end = null;
+
+    // Act
+    const range = formatDateRange(start, end, undefined);
+
+    // Assert
+    expect(range).toBe("October 2024 – Present");
+  });
+
+  it("omits the location segment entirely when location is an empty string", () => {
+    // Arrange
+    const start = "2024-10";
+    const end = null;
+
+    // Act
+    const range = formatDateRange(start, end, "");
+
+    // Assert
+    expect(range).toBe("October 2024 – Present");
+  });
+
+  it("appends a single location after the date range separated by ' | '", () => {
+    // Arrange
+    const start = "2023-01";
+    const end = "2024-01";
+
+    // Act
+    const range = formatDateRange(start, end, "Chicago, IL");
+
+    // Assert
+    expect(range).toBe("January 2023 – January 2024 | Chicago, IL");
+  });
 });
 
 describe("linesToBullets", () => {
diff --git a/lib/__tests__/resume-template.test.ts b/lib/__tests__/resume-template.test.ts
index 6d9b0c0..527dd39 100644
--- a/lib/__tests__/resume-template.test.ts
+++ b/lib/__tests__/resume-template.test.ts
@@ -50,6 +50,43 @@ describe("buildResumeHtml", () => {
     expect(html).toContain("<li>Shipped the thing.</li>");
   });
 
+  it("appends the role's location after the date range when location is present", () => {
+    // Arrange
+    const profile: ProfileData = {
+      experience: [
+        {
+          company: "Anthropic",
+          role: "Staff Designer",
+          location: "Chicago, IL",
+          start: "2023-01",
+          end: null,
+          bullets: [],
+        },
+      ],
+    };
+
+    // Act
+    const html = buildResumeHtml(profile, candidate);
+
+    // Assert
+    expect(html).toContain("January 2023 – Present | Chicago, IL");
+  });
+
+  it("omits the location segment when the role has no location", () => {
+    // Arrange
+    const profile: ProfileData = {
+      experience: [
+        { company: "Anthropic", role: "Staff Designer", start: "2023-01", end: null, bullets: [] },
+      ],
+    };
+
+    // Act
+    const html = buildResumeHtml(profile, candidate);
+
+    // Assert
+    expect(html).toContain("January 2023 – Present</span>");
+  });
+
   it("renders multiple experience roles in the given order", () => {
     // Arrange
     const profile: ProfileData = {
@@ -309,6 +346,35 @@ describe("buildResumeHtml", () => {
     expect(html).not.toContain("<script>");
   });
 
+  it("renders each education entry as a single line joining institution, degree, honors, and date with ' | '", () => {
+    // Arrange
+    const profile: ProfileData = {
+      education: [
+        { institution: "State University", degree: "B.A. Design", graduated: "2010-05", honors: "Cum Laude" },
+      ],
+    };
+
+    // Act
+    const html = buildResumeHtml(profile, candidate);
+
+    // Assert
+    expect(html).toMatch(/<b>State University<\/b>\s*\|\s*B\.A\. Design\s*\|\s*Cum Laude\s*\|\s*2010-05/);
+  });
+
+  it("gives the first section header a larger top margin than subsequent section headers", () => {
+    // Arrange
+    const profile: ProfileData = {
+      summary: "Fifteen years of design leadership.",
+      strengths: ["Design Systems"],
+    };
+
+    // Act
+    const html = buildResumeHtml(profile, candidate);
+
+    // Assert
+    expect(html).toMatch(/h2:first-of-type\s*\{[^}]*margin-top:\s*18px/);
+  });
+
   it("embeds Inter as a real base64 font-face, not a bare font-family reference", () => {
     // Arrange
     const profile: ProfileData = {};
diff --git a/lib/docx-resume.ts b/lib/docx-resume.ts
index d5786d8..67fd6b5 100644
--- a/lib/docx-resume.ts
+++ b/lib/docx-resume.ts
@@ -22,6 +22,7 @@ export interface ProfileData {
   experience?: {
     company: string;
     role: string;
+    location?: string;
     start: string;
     end: string | null;
     bullets: string[];
@@ -207,7 +208,7 @@ export async function generateResumeDOCX(
           children: [
             run(job.company, { bold: true, color: body }),
             new TextRun({ children: ["\t"], font: { ascii: font, hAnsi: font } }),
-            run(formatDateRange(job.start, job.end), { color: meta, size: 18 }),
+            run(formatDateRange(job.start, job.end, job.location), { color: meta, size: 18 }),
           ],
           tabStops: [{ type: TabStopType.RIGHT, position: contentWidth }],
           spacing: { after: 60 },
diff --git a/lib/profile.ts b/lib/profile.ts
index 8b8411b..58a4f82 100644
--- a/lib/profile.ts
+++ b/lib/profile.ts
@@ -1,6 +1,7 @@
 export interface ExperienceEntry {
   company: string;
   role: string;
+  location?: string;
   start: string;
   end: string | null;
   bullets: string[];
diff --git a/lib/resume-format.ts b/lib/resume-format.ts
index 49dc4ac..f9be0d8 100644
--- a/lib/resume-format.ts
+++ b/lib/resume-format.ts
@@ -1,4 +1,4 @@
-export function formatDateRange(start: string, end: string | null): string {
+export function formatDateRange(start: string, end: string | null, location?: string): string {
   const fmt = (d: string) => {
     const [y, m] = d.split("-");
     return new Date(Number(y), Number(m) - 1).toLocaleDateString("en-US", {
@@ -6,7 +6,8 @@ export function formatDateRange(start: string, end: string | null): string {
       year: "numeric",
     });
   };
-  return `${fmt(start)} – ${end ? fmt(end) : "Present"}`;
+  const range = `${fmt(start)} – ${end ? fmt(end) : "Present"}`;
+  return location ? `${range} | ${location}` : range;
 }
 
 export function companySlug(company: string): string {
diff --git a/lib/resume-template.ts b/lib/resume-template.ts
index ad4afab..318b1dd 100644
--- a/lib/resume-template.ts
+++ b/lib/resume-template.ts
@@ -61,26 +61,26 @@ function renderStyle(): string {
   * { box-sizing: border-box; }
   body { font-family: 'Inter', -apple-system, sans-serif; color: var(--body); margin: 0; padding: 0; }
   h1 { font-size: 27px; font-weight: 700; color: var(--ink); letter-spacing: -0.01em; line-height: 1.2; margin: 0; }
-  .headline { font-size: 14px; font-weight: 500; color: var(--accent); letter-spacing: 0.01em; margin: 4px 8px 0 0; text-align: justify;}
-  .contact { font-size: 12px; font-weight: 400; color: var(--secondary); line-height: 1.3; margin: 6px 6px 0 0; }
+  .headline { font-size: 13px; font-weight: 500; color: var(--accent); letter-spacing: 0.01em; margin: 6px 0 0; }
+  .contact { font-size: 12px; font-weight: 400; color: var(--secondary); line-height: 1.6; margin: 8px 0 0; }
   h2 {
     font-size: 11px; font-weight: 600; color: var(--accent); text-transform: uppercase;
-    letter-spacing: 0.12em; padding-bottom: 5px; border-bottom: 1px solid var(--rule); margin: 17px 0 8px;
+    letter-spacing: 0.12em; padding-bottom: 5px; border-bottom: 1px solid var(--rule); margin: 16px 0 7px;
   }
+  h2:first-of-type { margin-top: 18px; }
   h3 { font-size: 13.5px; font-weight: 700; color: var(--ink); line-height: 1.4; margin: 0; }
   .role-row { display: flex; justify-content: space-between; align-items: baseline; gap: 16px; margin-top: 2px; }
   .employer { font-size: 12.5px; font-weight: 500; color: var(--secondary); }
   .dates { font-size: 12px; font-weight: 400; color: var(--muted); white-space: nowrap; }
-  .role-block { margin-bottom: 16px; }
+  .role-block { margin-bottom: 12px; }
   .role-head { break-inside: avoid; }
-  ul { padding-left: 18px; margin: 5px 0 0; }
-  li { font-size: 13px; line-height: 1.45; margin-bottom: 4px; }
-  p.body-text { font-size: 13px; line-height: 1.45; margin: 0; }
-  .skills-line { font-size: 13px; line-height: 1.3; margin: 4px 0; }
+  ul { padding-left: 18px; margin: 4px 0 0; }
+  li { font-size: 12.25px; line-height: 1.5; margin-bottom: 4px; }
+  p.body-text { font-size: 12.25px; line-height: 1.5; margin: 0; }
+  .skills-line { font-size: 12.25px; line-height: 1.5; margin: 4px 0; }
   .skills-line b { font-weight: 600; color: var(--ink); }
-  .edu-entry { break-inside: avoid; margin-bottom: 10px; }
-  .edu-entry b { font-weight: 700; color: var(--ink); font-size: 13px; }
-  .edu-entry .details { font-size: 12px; color: var(--secondary); }
+  .edu-entry { break-inside: avoid; margin-bottom: 10px; font-size: 12.25px; }
+  .edu-entry b { font-weight: 700; color: var(--ink); }
 </style>`;
 }
 
@@ -138,7 +138,7 @@ function renderExperienceSection(profile: ProfileData, tailoredBullets?: Record<
     <h3>${escapeHtml(job.role)}</h3>
     <div class="role-row">
       <span class="employer">${escapeHtml(job.company)}</span>
-      <span class="dates">${formatDateRange(job.start, job.end)}</span>
+      <span class="dates">${escapeHtml(formatDateRange(job.start, job.end, job.location))}</span>
     </div>
   </div>
   <ul>${bullets.map((b) => `<li>${escapeHtml(b)}</li>`).join("")}</ul>
@@ -155,11 +155,8 @@ function renderAwardsSection(profile: ProfileData): string {
 function renderEducationSection(profile: ProfileData): string {
   if (!profile.education?.length) return "";
   const entries = profile.education.map((edu) => {
-    const details = [edu.degree, edu.honors, edu.graduated].filter(Boolean).map(String).map(escapeHtml).join(" | ");
-    return `<div class="edu-entry">
-  <div><b>${escapeHtml(edu.institution)}</b></div>
-  <div class="details">${details}</div>
-</div>`;
+    const rest = [edu.degree, edu.honors, edu.graduated].filter(Boolean).map(String).map(escapeHtml).join(" | ");
+    return `<div class="edu-entry"><b>${escapeHtml(edu.institution)}</b>${rest ? ` | ${rest}` : ""}</div>`;
   });
   return `<h2>Education</h2>\n${entries.join("\n")}`;
 }