mirror of
https://github.com/AmruthPillai/Reactive-Resume.git
synced 2026-10-05 03:03:58 +10:00
* feat(import): parse a PDF resume without an AI provider Importing a PDF required a connected AI provider, so anyone without a paid API key could only import the three JSON formats. Almost nobody arrives with one of those files; they arrive with a PDF. The first thing a new user tries to do was blocked behind bringing their own key. Adds a deterministic parser that reads the text out of the PDF in the browser and prefills the builder. It pulls the contact block, segments the body on conventional headings, and maps entries to real items, reusing the ATS period parser for dates so a date range is not mistaken for a phone number. Nothing is thrown away: header parts that do not map to a field go into the description, and unrecognized headings become custom sections. The imported sections are placed on the page so the result renders straight away. Output is validated against the resume schema before it is returned. Text extraction groups items by baseline rather than trusting hasEOL, and turns wide column gaps into a double space, which is what lets a row split into company, position and location. The AI path still runs when a provider is connected. Word import is unchanged and still requires one. Closes #3334 * fix(import): keep every section and entry the PDF actually contains Review found three ways the parser lost or mangled content, all of them reproducible. A document whose first heading was not one of the known aliases never started a section, because unknown-heading detection was gated on a section already being open. Everything after it was swallowed as contact header text. The header block is now bounded by where the contact details stop, so a heading is recognized wherever it appears. An entry spreading company, position and dates over three lines was imported as two malformed items. A line that introduces an entry now merges into the open entry instead of starting a second one. An uppercase company such as ACME CORPORATION was read as a section heading and fragmented the entry. A heading candidate followed by a date line is now treated as an entry header, which is what it is. Also escape single quotes, and construct the PDF worker inside the try so the nested worker is terminated even if construction throws. Title-case headings are deliberately still not treated as headings: company and school names are title case too, and splitting on them would fragment real entries. Such a section stays in the preceding one with its text intact rather than risking loss. * fix(import): look past a multi-line preamble before calling a line a heading The previous guard only inspected the next line, so an uppercase company followed by a separate role line and then the dates was still read as a section heading. The experience or education entry was moved into a custom section and lost. Heading detection now scans a two-line window for the date that marks an entry, and stops early at a bullet so a genuine heading whose section opens with bullet points is still recognized. The window can suppress a real heading whose first entry puts a bare date two lines below it. That is the deliberate direction to fail in: a missed heading leaves the text in the preceding section, while a misread entry fragments structured content. * fix(import): collect an entry preamble until its dates appear An entry that spread company, role, location and dates over four lines was imported as two broken items: the company with no dates, and the location carrying the period. The cause was in entry grouping rather than heading detection. Lines before a date were only folded into the entry header when the date sat on the very next line; anything earlier fell through to the description. Preamble lines are now collected into the entry header until the dates turn up, bounded by the same lookahead and stopping at a bullet, so an undated section cannot swallow itself. The heading lookahead widens to four lines to match, which is the realistic maximum for company, role, location and dates. * fix(import): harden local PDF resume parsing * chore(import): document audited HTML construction --------- Co-authored-by: Amruth Pillai <im.amruth@gmail.com>
322 lines
12 KiB
TypeScript
322 lines
12 KiB
TypeScript
import { describe, expect, it } from "vitest";
|
|
import { resumeDataSchema } from "@reactive-resume/schema/resume/data";
|
|
import { parseResumeText } from "./plain-text";
|
|
|
|
const SAMPLE = `Ada Lovelace
|
|
Senior Software Engineer
|
|
Berlin, Germany | ada@example.com | +44 20 7946 0100 | https://ada.dev
|
|
|
|
SUMMARY
|
|
Engineer with 10 years building analytical systems.
|
|
|
|
WORK EXPERIENCE
|
|
Analytical Engines Senior Engineer Berlin
|
|
Jan 2020 - Present
|
|
• Led the difference engine rewrite
|
|
• Mentored four junior engineers
|
|
Babbage Ltd Engineer London
|
|
Mar 2016 - Dec 2019
|
|
• Built the punch card pipeline
|
|
|
|
EDUCATION
|
|
University of London BSc Mathematics
|
|
2012 - 2016
|
|
|
|
SKILLS
|
|
TypeScript, Rust, PostgreSQL
|
|
|
|
LANGUAGES
|
|
English (Native)
|
|
German (B2)
|
|
|
|
CERTIFICATIONS
|
|
AWS Solutions Architect Amazon 2021
|
|
`;
|
|
|
|
describe("parseResumeText", () => {
|
|
const data = parseResumeText(SAMPLE);
|
|
|
|
it("always returns schema-valid resume data", () => {
|
|
expect(() => resumeDataSchema.parse(data)).not.toThrow();
|
|
});
|
|
|
|
it("reads the contact block", () => {
|
|
expect(data.basics).toMatchObject({
|
|
name: "Ada Lovelace",
|
|
headline: "Senior Software Engineer",
|
|
email: "ada@example.com",
|
|
phone: "+44 20 7946 0100",
|
|
location: "Berlin, Germany",
|
|
});
|
|
expect(data.basics.website.url).toBe("https://ada.dev");
|
|
});
|
|
|
|
it("reads the summary as rich text", () => {
|
|
expect(data.summary.content).toBe("<p>Engineer with 10 years building analytical systems.</p>");
|
|
});
|
|
|
|
it("splits experience into one entry per role", () => {
|
|
expect(data.sections.experience.items).toHaveLength(2);
|
|
expect(data.sections.experience.items[0]).toMatchObject({
|
|
company: "Analytical Engines",
|
|
position: "Senior Engineer",
|
|
location: "Berlin",
|
|
period: "Jan 2020 - Present",
|
|
});
|
|
expect(data.sections.experience.items[1]).toMatchObject({
|
|
company: "Babbage Ltd",
|
|
position: "Engineer",
|
|
period: "Mar 2016 - Dec 2019",
|
|
});
|
|
});
|
|
|
|
it("keeps bullets as a list in the description", () => {
|
|
expect(data.sections.experience.items[0]?.description).toBe(
|
|
"<ul><li>Led the difference engine rewrite</li><li>Mentored four junior engineers</li></ul>",
|
|
);
|
|
});
|
|
|
|
it("reads education", () => {
|
|
expect(data.sections.education.items[0]).toMatchObject({
|
|
school: "University of London",
|
|
degree: "BSc Mathematics",
|
|
period: "2012 - 2016",
|
|
});
|
|
});
|
|
|
|
it("splits a comma separated skills line", () => {
|
|
expect(data.sections.skills.items.map((item) => item.name)).toEqual(["TypeScript", "Rust", "PostgreSQL"]);
|
|
});
|
|
|
|
it("splits a language from its fluency", () => {
|
|
expect(data.sections.languages.items).toMatchObject([
|
|
{ language: "English", fluency: "Native" },
|
|
{ language: "German", fluency: "B2" },
|
|
]);
|
|
});
|
|
|
|
it("reads a trailing year as the certification date", () => {
|
|
expect(data.sections.certifications.items[0]).toMatchObject({
|
|
title: "AWS Solutions Architect",
|
|
issuer: "Amazon",
|
|
date: "2021",
|
|
});
|
|
});
|
|
|
|
it("places every populated section on the page in document order", () => {
|
|
expect(data.metadata.layout.pages[0]?.main).toEqual([
|
|
"summary",
|
|
"experience",
|
|
"education",
|
|
"skills",
|
|
"languages",
|
|
"certifications",
|
|
]);
|
|
});
|
|
});
|
|
|
|
describe("parseResumeText edge cases", () => {
|
|
it("returns usable data for empty input", () => {
|
|
const data = parseResumeText("");
|
|
expect(() => resumeDataSchema.parse(data)).not.toThrow();
|
|
expect(data.basics.name).toBe("");
|
|
expect(data.metadata.layout.pages[0]?.main).toEqual([]);
|
|
});
|
|
|
|
it("does not mistake a date range for a phone number", () => {
|
|
const data = parseResumeText("Ada Lovelace\nBerlin\n2016 - 2019\n");
|
|
expect(data.basics.phone).toBe("");
|
|
});
|
|
|
|
it("keeps an unrecognized heading as a custom section", () => {
|
|
const data = parseResumeText("Ada\n\nSKILLS\nRust\n\nSPEAKING\nGave a talk at a conference\n");
|
|
expect(data.customSections).toHaveLength(1);
|
|
expect(data.customSections[0]).toMatchObject({ type: "summary", title: "SPEAKING" });
|
|
expect(data.metadata.layout.pages[0]?.main).toContain(data.customSections[0]?.id);
|
|
});
|
|
|
|
it("keeps unclassified header parts in the description rather than dropping them", () => {
|
|
const data = parseResumeText("EXPERIENCE\nAcme Engineer Berlin Remote Contract\n2020 - 2022\n");
|
|
expect(data.sections.experience.items[0]?.description).toContain("Remote");
|
|
expect(data.sections.experience.items[0]?.description).toContain("Contract");
|
|
});
|
|
|
|
it("escapes markup found in the source text", () => {
|
|
const data = parseResumeText("SUMMARY\nI write <script>alert(1)</script> safely\n");
|
|
expect(data.summary.content).toContain("<script>");
|
|
expect(data.summary.content).not.toContain("<script>");
|
|
});
|
|
|
|
it("treats a heading with a trailing colon as a heading", () => {
|
|
const data = parseResumeText("Ada\n\nSkills:\nRust, Go\n");
|
|
expect(data.sections.skills.items.map((item) => item.name)).toEqual(["Rust", "Go"]);
|
|
});
|
|
});
|
|
|
|
describe("parseResumeText review findings", () => {
|
|
it("keeps a section whose heading is the first one in the document", () => {
|
|
const data = parseResumeText(
|
|
"Ada Lovelace\nada@example.com\n\nCAREER HIGHLIGHTS\nShipped the difference engine\nMentored the team\n",
|
|
);
|
|
|
|
expect(data.customSections).toHaveLength(1);
|
|
expect(data.customSections[0]).toMatchObject({ title: "CAREER HIGHLIGHTS" });
|
|
expect(JSON.stringify(data)).toContain("Shipped the difference engine");
|
|
});
|
|
|
|
it("keeps one entry when company, position and dates sit on separate lines", () => {
|
|
const data = parseResumeText(
|
|
"EXPERIENCE\nAnalytical Engines\nSenior Engineer\nJan 2020 - Present\n• Led the rewrite\n",
|
|
);
|
|
|
|
expect(data.sections.experience.items).toHaveLength(1);
|
|
expect(data.sections.experience.items[0]).toMatchObject({
|
|
company: "Analytical Engines",
|
|
position: "Senior Engineer",
|
|
period: "Jan 2020 - Present",
|
|
});
|
|
});
|
|
|
|
it("does not turn an uppercase company name into a section heading", () => {
|
|
const data = parseResumeText("EXPERIENCE\nACME CORPORATION\nJan 2020 - Present\n• Did the work\n");
|
|
|
|
expect(data.customSections).toHaveLength(0);
|
|
expect(data.sections.experience.items[0]).toMatchObject({ company: "ACME CORPORATION" });
|
|
});
|
|
|
|
it("escapes single quotes in extracted text", () => {
|
|
const data = parseResumeText("SUMMARY\nIt's a resume\n");
|
|
expect(data.summary.content).toContain("'");
|
|
});
|
|
});
|
|
|
|
describe("parseResumeText multi-line entry preambles", () => {
|
|
it("keeps an uppercase company followed by a separate role line as one entry", () => {
|
|
const data = parseResumeText(
|
|
"EXPERIENCE\nACME CORPORATION\nSenior Engineer\nJan 2020 - Present\n• Led the rewrite\n",
|
|
);
|
|
|
|
expect(data.customSections).toHaveLength(0);
|
|
expect(data.sections.experience.items).toHaveLength(1);
|
|
expect(data.sections.experience.items[0]).toMatchObject({
|
|
company: "ACME CORPORATION",
|
|
position: "Senior Engineer",
|
|
period: "Jan 2020 - Present",
|
|
});
|
|
});
|
|
|
|
it("keeps an uppercase school followed by a separate degree line as one entry", () => {
|
|
const data = parseResumeText("EDUCATION\nUNIVERSITY OF LONDON\nBSc Mathematics\n2012 - 2016\n");
|
|
|
|
expect(data.customSections).toHaveLength(0);
|
|
expect(data.sections.education.items).toHaveLength(1);
|
|
expect(data.sections.education.items[0]).toMatchObject({
|
|
school: "UNIVERSITY OF LONDON",
|
|
degree: "BSc Mathematics",
|
|
period: "2012 - 2016",
|
|
});
|
|
});
|
|
|
|
it("still recognizes a real heading whose section starts with bullets", () => {
|
|
const data = parseResumeText("Ada\nada@example.com\n\nCAREER HIGHLIGHTS\n• Shipped in 2019\n• Grew the team\n");
|
|
|
|
expect(data.customSections).toHaveLength(1);
|
|
expect(data.customSections[0]).toMatchObject({ title: "CAREER HIGHLIGHTS" });
|
|
});
|
|
});
|
|
|
|
describe("parseResumeText four-line entry preambles", () => {
|
|
it("keeps company, role, location and dates as one entry", () => {
|
|
const data = parseResumeText(
|
|
"EXPERIENCE\nACME CORPORATION\nSenior Engineer\nBerlin, Germany\nJan 2020 - Present\n• Led the rewrite\n",
|
|
);
|
|
|
|
expect(data.customSections).toHaveLength(0);
|
|
expect(data.sections.experience.items).toHaveLength(1);
|
|
expect(data.sections.experience.items[0]).toMatchObject({
|
|
company: "ACME CORPORATION",
|
|
position: "Senior Engineer",
|
|
location: "Berlin, Germany",
|
|
period: "Jan 2020 - Present",
|
|
});
|
|
});
|
|
|
|
it("keeps school, degree, location and dates as one entry", () => {
|
|
const data = parseResumeText("EDUCATION\nUNIVERSITY OF LONDON\nBSc Mathematics\nLondon, UK\n2012 - 2016\n");
|
|
|
|
expect(data.customSections).toHaveLength(0);
|
|
expect(data.sections.education.items).toHaveLength(1);
|
|
expect(data.sections.education.items[0]).toMatchObject({
|
|
school: "UNIVERSITY OF LONDON",
|
|
degree: "BSc Mathematics",
|
|
location: "London, UK",
|
|
period: "2012 - 2016",
|
|
});
|
|
});
|
|
});
|
|
|
|
describe("parseResumeText keeps content the layout hides", () => {
|
|
it("recognizes isolated title-case custom section headings", () => {
|
|
const initial = parseResumeText(
|
|
"Ada Lovelace\nada@example.com\n\nConferences\nReactConf\nBerlin\n2021\nSpoke about parsers\n",
|
|
);
|
|
const subsequent = parseResumeText(
|
|
"Ada Lovelace\nada@example.com\n\nEXPERIENCE\nAcme Engineer\n2020 - 2022\nBuilt products.\n\nConferences\nReactConf\nBerlin\n2021\nSpoke about parsers\n",
|
|
);
|
|
|
|
expect(initial.customSections[0]).toMatchObject({ title: "Conferences" });
|
|
expect(subsequent.customSections[0]).toMatchObject({ title: "Conferences" });
|
|
expect(subsequent.sections.experience.items).toHaveLength(1);
|
|
});
|
|
|
|
it("keeps a dated custom section that opens the body", () => {
|
|
const data = parseResumeText(
|
|
"Ada Lovelace\nada@example.com\n\nCONFERENCES\nReactConf\nBerlin\n2021\nSpoke about parsers\n",
|
|
);
|
|
|
|
expect(data.basics.headline).not.toBe("CONFERENCES");
|
|
expect(data.customSections).toHaveLength(1);
|
|
expect(data.customSections[0]).toMatchObject({ title: "CONFERENCES" });
|
|
|
|
const content = data.customSections[0]?.items[0]?.content ?? "";
|
|
for (const line of ["ReactConf", "Berlin", "2021", "Spoke about parsers"]) {
|
|
expect(content).toContain(line);
|
|
}
|
|
});
|
|
|
|
it("keeps unbulleted descriptions with their own role", () => {
|
|
const data = parseResumeText(
|
|
"EXPERIENCE\nAcme Engineer Berlin\nJan 2020 - Present\nBuilt the thing end to end.\nWorked with a team of five.\nBabbage Ltd Engineer London\nMar 2016 - Dec 2019\nDid other work.\n",
|
|
);
|
|
|
|
expect(data.sections.experience.items).toHaveLength(2);
|
|
expect(data.sections.experience.items[0]?.description).toContain("Built the thing end to end.");
|
|
expect(data.sections.experience.items[0]?.description).toContain("Worked with a team of five.");
|
|
expect(data.sections.experience.items[1]).toMatchObject({ company: "Babbage Ltd", location: "London" });
|
|
});
|
|
|
|
it("does not split an entry on a bare year in a section that ignores single dates", () => {
|
|
const data = parseResumeText(
|
|
"EXPERIENCE\nAcme Engineer\nJan 2020 - Present\nGrew the team.\nMore work here.\n2022\n",
|
|
);
|
|
|
|
expect(data.sections.experience.items).toHaveLength(1);
|
|
|
|
const description = data.sections.experience.items[0]?.description ?? "";
|
|
for (const line of ["Grew the team.", "More work here.", "2022"]) {
|
|
expect(description).toContain(line);
|
|
}
|
|
});
|
|
|
|
it("stays schema-valid when a section starts with its dates", () => {
|
|
const data = parseResumeText(
|
|
"EXPERIENCE\nJan 2020 - Present\nAcme Corp\nSenior Engineer\n\nEDUCATION\n2012 - 2016\nUniversity of London\n\nCERTIFICATIONS\n2021\n",
|
|
);
|
|
|
|
expect(() => resumeDataSchema.parse(data)).not.toThrow();
|
|
expect(data.sections.experience.items[0]).toMatchObject({ company: "Acme Corp", period: "Jan 2020 - Present" });
|
|
expect(data.sections.education.items[0]).toMatchObject({ school: "University of London" });
|
|
expect(data.sections.certifications.items).toHaveLength(0);
|
|
});
|
|
});
|