@jsonresume/jobs 0.14.1 → 0.14.2

package/src/tui/SplitPane.js

@@ -0,0 +1,58 @@
+import { Box, Text } from 'ink';
+import TextInput from 'ink-text-input';
+import { h } from './h.js';
+import JobList from './JobList.js';
+
+// Inline find-as-you-type bar shown under the list view.
+export function InlineSearch({ query, onChange, onSubmit }) {
+  return h(
+    Box,
+    { paddingX: 1, gap: 1 },
+    h(Text, { color: 'yellow', bold: true }, 'Find:'),
+    h(TextInput, { value: query, onChange, onSubmit }),
+    h(Text, { dimColor: true }, '  Enter to apply, Esc to clear')
+  );
+}
+
+// Compact job list used as the left pane of the split-pane views.
+function leftPane(listProps) {
+  return h(
+    Box,
+    {
+      flexDirection: 'column',
+      width: '40%',
+      borderStyle: 'single',
+      borderColor: 'gray',
+      borderRight: true,
+      borderLeft: false,
+      borderTop: false,
+      borderBottom: false,
+    },
+    h(JobList, { ...listProps, compact: true, reservedRows: 8 })
+  );
+}
+
+/**
+ * Two-column layout: compact job list on the left, an arbitrary right pane.
+ * Used by both the detail and AI/dossier views.
+ *
+ * @param {object} p
+ * @param {any} p.header        rendered header element
+ * @param {any} p.statusBar     rendered status bar element
+ * @param {object} p.listProps  props forwarded to the left-pane JobList
+ * @param {any} p.right         rendered right-pane element
+ */
+export function SplitPane({ header, statusBar, listProps, right }) {
+  return h(
+    Box,
+    { flexDirection: 'column', height: process.stdout.rows || 40 },
+    header,
+    h(
+      Box,
+      { flexGrow: 1, flexDirection: 'row' },
+      leftPane(listProps),
+      h(Box, { flexDirection: 'column', width: '60%' }, right)
+    ),
+    statusBar
+  );
+}

package/src/tui/aiHelpers.js

@@ -0,0 +1,129 @@
+/**
+ * Pure helpers for the AI/dossier hook.
+ * Prompt builders, stream-json parsing, enrichment extraction and filename
+ * derivation are kept here so they can be unit-tested in isolation.
+ */
+
+// Compact resume text for the full dossier prompt.
+export function buildResumeText(resume) {
+  return [
+    resume?.basics?.name,
+    resume?.basics?.label,
+    resume?.basics?.summary,
+    ...(resume?.skills || []).map(
+      (s) => `${s.name}: ${(s.keywords || []).join(', ')}`
+    ),
+    ...(resume?.work || [])
+      .slice(0, 5)
+      .map(
+        (w) =>
+          `${w.position} at ${w.name}${
+            w.startDate ? ` (${w.startDate})` : ''
+          }: ${w.summary || ''}`
+      ),
+    ...(resume?.projects || [])
+      .slice(0, 3)
+      .map((p) => `Project: ${p.name} — ${p.description || ''}`),
+  ]
+    .filter(Boolean)
+    .join('\n');
+}
+
+// Full job text for the dossier prompt (includes the raw original posting).
+export function buildJobText(job, rawContent) {
+  return [
+    `Title: ${job.title}`,
+    `Company: ${job.company}`,
+    `Salary: ${job.salary || 'Not listed'}`,
+    `Remote: ${job.remote || 'Not specified'}`,
+    job.experience ? `Experience: ${job.experience}` : '',
+    job.type ? `Type: ${job.type}` : '',
+    job.url ? `HN Post: ${job.url}` : '',
+    `Description: ${job.description || ''}`,
+    `Skills: ${(job.skills || []).map((s) => s.name || s).join(', ')}`,
+    ...(job.qualifications || []).map((q) => `Qualification: ${q}`),
+    ...(job.responsibilities || []).map((r) => `Responsibility: ${r}`),
+    rawContent ? `\nFull original posting:\n${rawContent}` : '',
+  ]
+    .filter(Boolean)
+    .join('\n');
+}
+
+// Compact resume text used by the quick GPT summary/batch prompts.
+export function buildSummaryResumeText(resume) {
+  return [
+    resume?.basics?.label,
+    resume?.basics?.summary,
+    ...(resume?.skills || []).map(
+      (s) => `${s.name}: ${(s.keywords || []).join(', ')}`
+    ),
+  ]
+    .filter(Boolean)
+    .join('\n');
+}
+
+// Compact job text used by the quick GPT summary prompt.
+export function buildSummaryJobText(job) {
+  return [
+    `Title: ${job.title}`,
+    `Company: ${job.company}`,
+    `Salary: ${job.salary || 'Not listed'}`,
+    `Remote: ${job.remote || 'Not specified'}`,
+    `Description: ${job.description || ''}`,
+    `Skills: ${(job.skills || []).map((s) => s.name).join(', ')}`,
+  ].join('\n');
+}
+
+// Ranked list text used by the batch-review prompt (first 15 jobs).
+export function buildBatchJobsList(jobs) {
+  return jobs
+    .slice(0, 15)
+    .map(
+      (j, i) =>
+        `${i + 1}. [${j.similarity}] ${j.title} at ${j.company} | ${
+          j.salary || 'no salary'
+        } | ${j.remote || 'no remote info'}`
+    )
+    .join('\n');
+}
+
+// The full research-dossier prompt lives in its own module (large text blob).
+export { buildDossierPrompt } from './dossierPrompt.js';
+
+// Derive a human-readable status line from a Claude tool_use block.
+export function statusLineForTool(block) {
+  const name = block.name || 'tool';
+  const input = block.input || {};
+  if (name === 'WebSearch' || name === 'WebFetch') {
+    return `🔍 ${name}: ${input.query || input.url || ''}`;
+  }
+  return `⚙ Using ${name}…`;
+}
+
+// Compose visible dossier text from the running status line and result text.
+export function composeDossierText(statusLine, resultText) {
+  if (statusLine && resultText) return `${statusLine}\n\n${resultText}`;
+  return resultText || statusLine || '';
+}
+
+// Pull the enrichment JSON out of a finished dossier, or null if absent/invalid.
+export function extractEnrichment(finalResult) {
+  const match = finalResult.match(/```enrichment\s*\n([\s\S]*?)\n```/);
+  if (!match) return null;
+  try {
+    return JSON.parse(match[1]);
+  } catch {
+    return null;
+  }
+}
+
+// Build the on-disk filename for an exported dossier.
+export function dossierFilename(job) {
+  const company = (job?.company || 'unknown')
+    .replace(/[^a-zA-Z0-9]+/g, '-')
+    .replace(/-+/g, '-')
+    .replace(/^-|-$/g, '')
+    .toLowerCase()
+    .slice(0, 50);
+  return `dossier-${company}.md`;
+}

package/src/tui/aiHelpers.test.js

@@ -0,0 +1,200 @@
+import { describe, it, expect } from 'vitest';
+import {
+  buildResumeText,
+  buildJobText,
+  buildSummaryResumeText,
+  buildSummaryJobText,
+  buildBatchJobsList,
+  buildDossierPrompt,
+  statusLineForTool,
+  composeDossierText,
+  extractEnrichment,
+  dossierFilename,
+} from './aiHelpers.js';
+
+const resume = {
+  basics: { name: 'Ada Lovelace', label: 'Engineer', summary: 'Builds things' },
+  skills: [{ name: 'JS', keywords: ['react', 'node'] }, { name: 'Go' }],
+  work: [
+    { position: 'Eng', name: 'Acme', startDate: '2020', summary: 'Did work' },
+  ],
+  projects: [{ name: 'Proj', description: 'A project' }],
+};
+
+const job = {
+  title: 'Senior Dev',
+  company: 'BigCo',
+  salary: '$200k',
+  remote: 'Full',
+  experience: 'Senior',
+  type: 'Full-time',
+  url: 'https://news.ycombinator.com/item?id=1',
+  description: 'Build cool stuff',
+  skills: [{ name: 'JS' }, 'Python'],
+  qualifications: ['5y exp'],
+  responsibilities: ['Ship code'],
+};
+
+describe('buildResumeText', () => {
+  it('includes name, label, summary, skills, work and projects', () => {
+    const out = buildResumeText(resume);
+    expect(out).toContain('Ada Lovelace');
+    expect(out).toContain('Engineer');
+    expect(out).toContain('JS: react, node');
+    expect(out).toContain('Eng at Acme (2020): Did work');
+    expect(out).toContain('Project: Proj — A project');
+  });
+
+  it('skill without keywords renders empty keyword list', () => {
+    expect(buildResumeText(resume)).toContain('Go: ');
+  });
+
+  it('handles undefined resume without throwing', () => {
+    expect(buildResumeText(undefined)).toBe('');
+    expect(buildResumeText(null)).toBe('');
+  });
+
+  it('caps work at 5 and projects at 3', () => {
+    const big = {
+      work: Array.from({ length: 8 }, (_, i) => ({
+        position: `P${i}`,
+        name: `C${i}`,
+      })),
+      projects: Array.from({ length: 6 }, (_, i) => ({ name: `Pj${i}` })),
+    };
+    const out = buildResumeText(big);
+    expect(out).toContain('P4 at C4');
+    expect(out).not.toContain('P5 at C5');
+    expect(out).toContain('Project: Pj2');
+    expect(out).not.toContain('Project: Pj3');
+  });
+});
+
+describe('buildJobText', () => {
+  it('includes all listed fields and skills with name-or-string fallback', () => {
+    const out = buildJobText(job, '');
+    expect(out).toContain('Title: Senior Dev');
+    expect(out).toContain('Company: BigCo');
+    expect(out).toContain('Salary: $200k');
+    expect(out).toContain('Skills: JS, Python');
+    expect(out).toContain('Qualification: 5y exp');
+    expect(out).toContain('Responsibility: Ship code');
+  });
+
+  it('appends raw content when present', () => {
+    const out = buildJobText(job, 'ORIGINAL POSTING BODY');
+    expect(out).toContain('Full original posting:\nORIGINAL POSTING BODY');
+  });
+
+  it('omits raw content section when empty', () => {
+    expect(buildJobText(job, '')).not.toContain('Full original posting');
+  });
+
+  it('uses fallbacks for missing salary/remote', () => {
+    const out = buildJobText({ title: 't', company: 'c' }, '');
+    expect(out).toContain('Salary: Not listed');
+    expect(out).toContain('Remote: Not specified');
+  });
+});
+
+describe('buildSummaryResumeText / buildSummaryJobText', () => {
+  it('summary resume omits name but keeps label/summary/skills', () => {
+    const out = buildSummaryResumeText(resume);
+    expect(out).not.toContain('Ada Lovelace');
+    expect(out).toContain('Engineer');
+    expect(out).toContain('JS: react, node');
+  });
+
+  it('summary job uses skill.name only (no string fallback)', () => {
+    const out = buildSummaryJobText(job);
+    // 'Python' is a bare string → s.name is undefined
+    expect(out).toContain('Skills: JS, ');
+    expect(out).toContain('Title: Senior Dev');
+  });
+});
+
+describe('buildBatchJobsList', () => {
+  it('numbers jobs and caps at 15', () => {
+    const jobs = Array.from({ length: 20 }, (_, i) => ({
+      similarity: '0.9',
+      title: `T${i}`,
+      company: `C${i}`,
+    }));
+    const out = buildBatchJobsList(jobs);
+    expect(out).toContain('1. [0.9] T0 at C0 | no salary | no remote info');
+    expect(out).toContain('15. [0.9] T14 at C14');
+    expect(out).not.toContain('16. [0.9] T15');
+  });
+});
+
+describe('buildDossierPrompt', () => {
+  it('embeds resume and job text and the enrichment block', () => {
+    const out = buildDossierPrompt('RESUME_HERE', 'JOB_HERE');
+    expect(out).toContain('## Candidate Resume\nRESUME_HERE');
+    expect(out).toContain('## Job Posting\nJOB_HERE');
+    expect(out).toContain('```enrichment');
+    expect(out).toContain('AI-Forward / AI-Friendly');
+  });
+});
+
+describe('statusLineForTool', () => {
+  it('formats WebSearch with query', () => {
+    expect(
+      statusLineForTool({ name: 'WebSearch', input: { query: 'foo' } })
+    ).toBe('🔍 WebSearch: foo');
+  });
+  it('formats WebFetch with url', () => {
+    expect(
+      statusLineForTool({ name: 'WebFetch', input: { url: 'http://x' } })
+    ).toBe('🔍 WebFetch: http://x');
+  });
+  it('formats other tools generically', () => {
+    expect(statusLineForTool({ name: 'Bash' })).toBe('⚙ Using Bash…');
+    expect(statusLineForTool({})).toBe('⚙ Using tool…');
+  });
+});
+
+describe('composeDossierText', () => {
+  it('joins status and result when both present', () => {
+    expect(composeDossierText('STATUS', 'RESULT')).toBe('STATUS\n\nRESULT');
+  });
+  it('returns result alone when no status', () => {
+    expect(composeDossierText('', 'RESULT')).toBe('RESULT');
+  });
+  it('returns status alone when no result', () => {
+    expect(composeDossierText('STATUS', '')).toBe('STATUS');
+  });
+  it('returns empty string when neither present', () => {
+    expect(composeDossierText('', '')).toBe('');
+  });
+});
+
+describe('extractEnrichment', () => {
+  it('parses a valid enrichment block', () => {
+    const text = 'prose\n```enrichment\n{"salary":"$1"}\n```\nmore';
+    expect(extractEnrichment(text)).toEqual({ salary: '$1' });
+  });
+  it('returns null when no block present', () => {
+    expect(extractEnrichment('no block here')).toBeNull();
+  });
+  it('returns null on invalid JSON', () => {
+    expect(extractEnrichment('```enrichment\nnot json\n```')).toBeNull();
+  });
+});
+
+describe('dossierFilename', () => {
+  it('slugifies the company name', () => {
+    expect(dossierFilename({ company: 'Big Co, Inc.' })).toBe(
+      'dossier-big-co-inc.md'
+    );
+  });
+  it('falls back to unknown', () => {
+    expect(dossierFilename(null)).toBe('dossier-unknown.md');
+    expect(dossierFilename({})).toBe('dossier-unknown.md');
+  });
+  it('truncates to 50 chars of slug', () => {
+    const long = 'a'.repeat(80);
+    const out = dossierFilename({ company: long });
+    expect(out).toBe(`dossier-${'a'.repeat(50)}.md`);
+  });
+});

package/src/tui/appKeyHandler.js

@@ -0,0 +1,89 @@
+/**
+ * Keyboard dispatch for the main App view (list / detail / ai).
+ * Returns the (input, key) handler passed to Ink's useInput. All state lives in
+ * App; this module only routes keypresses to the supplied actions/setters so
+ * the routing rules stay in one cohesive, reviewable place.
+ */
+
+export function createAppKeyHandler(ctx) {
+  const {
+    view,
+    tab,
+    jobs,
+    cursor,
+    selectedJob,
+    inlineSearch,
+    confirmExit,
+    ai,
+    exit,
+    forceRefresh,
+    showToast,
+    setView,
+    setTab,
+    setCursor,
+    setSelectedJob,
+    setInlineSearch,
+    setSearchQuery,
+    setConfirmExit,
+    handleDossier,
+    nextTab,
+  } = ctx;
+
+  return (input, key) => {
+    if (view === 'filters' || view === 'searches' || view === 'help') return;
+    if (inlineSearch) return;
+
+    if (input === 'q' && view === 'list') {
+      if (ai.hasActiveProcess && !confirmExit) {
+        showToast(
+          'Claude dossier still running — press q again to quit',
+          'warning'
+        );
+        setConfirmExit(true);
+        return;
+      }
+      ai.cancel();
+      exit();
+    }
+    if (input !== 'q') setConfirmExit(false);
+    if (input === 'q' && view === 'detail') setView('list');
+    if (input === 'R' && (view === 'list' || view === 'detail')) {
+      forceRefresh();
+      showToast('Refreshing…', 'info');
+    }
+    if (input === 'f' && (view === 'list' || view === 'detail'))
+      setView('filters');
+    if (input === '/' && (view === 'list' || view === 'detail'))
+      setView('searches');
+    if (input === '?' && (view === 'list' || view === 'detail'))
+      setView('help');
+    if (input === 'n' && view === 'list') {
+      setInlineSearch(true);
+      setSearchQuery('');
+    }
+
+    // Enter toggles detail panel
+    if (key.return && view === 'list' && jobs[cursor]) {
+      setSelectedJob(jobs[cursor]);
+      setView('detail');
+    }
+    if (key.escape && view === 'detail') setView('list');
+    if (input === 'c' && view === 'detail' && selectedJob) {
+      handleDossier(selectedJob);
+    }
+
+    if (key.escape && view === 'ai') {
+      // Don't kill running dossier — just hide the panel
+      setView(selectedJob ? 'detail' : 'list');
+    }
+
+    if (key.tab && (view === 'list' || view === 'detail')) {
+      setTab(nextTab(tab, 1));
+      setCursor(0);
+    }
+    if (key.shift && key.tab && (view === 'list' || view === 'detail')) {
+      setTab(nextTab(tab, -1));
+      setCursor(0);
+    }
+  };
+}

package/src/tui/appKeyHandler.test.js

@@ -0,0 +1,144 @@
+import { describe, it, expect, vi } from 'vitest';
+import { createAppKeyHandler } from './appKeyHandler.js';
+import { nextTab } from './jobFilters.js';
+
+function makeCtx(overrides = {}) {
+  const ctx = {
+    view: 'list',
+    tab: 'all',
+    jobs: [{ id: 1 }, { id: 2 }],
+    cursor: 0,
+    selectedJob: null,
+    inlineSearch: false,
+    confirmExit: false,
+    ai: { hasActiveProcess: false, cancel: vi.fn() },
+    exit: vi.fn(),
+    forceRefresh: vi.fn(),
+    showToast: vi.fn(),
+    setView: vi.fn(),
+    setTab: vi.fn(),
+    setCursor: vi.fn(),
+    setSelectedJob: vi.fn(),
+    setInlineSearch: vi.fn(),
+    setSearchQuery: vi.fn(),
+    setConfirmExit: vi.fn(),
+    handleDossier: vi.fn(),
+    nextTab,
+    ...overrides,
+  };
+  return ctx;
+}
+
+const noKey = {};
+
+describe('createAppKeyHandler', () => {
+  it('ignores all input while in modal views', () => {
+    const ctx = makeCtx({ view: 'filters' });
+    createAppKeyHandler(ctx)('q', noKey);
+    expect(ctx.exit).not.toHaveBeenCalled();
+  });
+
+  it('ignores input while inline search is active', () => {
+    const ctx = makeCtx({ inlineSearch: true });
+    createAppKeyHandler(ctx)('q', noKey);
+    expect(ctx.exit).not.toHaveBeenCalled();
+  });
+
+  it('quits on q in list view when no active process', () => {
+    const ctx = makeCtx();
+    createAppKeyHandler(ctx)('q', noKey);
+    expect(ctx.ai.cancel).toHaveBeenCalled();
+    expect(ctx.exit).toHaveBeenCalled();
+  });
+
+  it('warns and arms confirmExit on q while dossier running', () => {
+    const ctx = makeCtx({ ai: { hasActiveProcess: true, cancel: vi.fn() } });
+    createAppKeyHandler(ctx)('q', noKey);
+    expect(ctx.showToast).toHaveBeenCalledWith(
+      expect.stringContaining('still running'),
+      'warning'
+    );
+    expect(ctx.setConfirmExit).toHaveBeenCalledWith(true);
+    expect(ctx.exit).not.toHaveBeenCalled();
+  });
+
+  it('quits on second q after confirm', () => {
+    const ctx = makeCtx({
+      ai: { hasActiveProcess: true, cancel: vi.fn() },
+      confirmExit: true,
+    });
+    createAppKeyHandler(ctx)('q', noKey);
+    expect(ctx.exit).toHaveBeenCalled();
+  });
+
+  it('q in detail view returns to list', () => {
+    const ctx = makeCtx({ view: 'detail' });
+    createAppKeyHandler(ctx)('q', noKey);
+    expect(ctx.setView).toHaveBeenCalledWith('list');
+  });
+
+  it('resets confirmExit on any non-q key', () => {
+    const ctx = makeCtx();
+    createAppKeyHandler(ctx)('R', noKey);
+    expect(ctx.setConfirmExit).toHaveBeenCalledWith(false);
+  });
+
+  it('R refreshes and toasts', () => {
+    const ctx = makeCtx();
+    createAppKeyHandler(ctx)('R', noKey);
+    expect(ctx.forceRefresh).toHaveBeenCalled();
+    expect(ctx.showToast).toHaveBeenCalledWith('Refreshing…', 'info');
+  });
+
+  it('f opens filters, / opens searches, ? opens help', () => {
+    const ctx = makeCtx();
+    const handler = createAppKeyHandler(ctx);
+    handler('f', noKey);
+    handler('/', noKey);
+    handler('?', noKey);
+    expect(ctx.setView).toHaveBeenCalledWith('filters');
+    expect(ctx.setView).toHaveBeenCalledWith('searches');
+    expect(ctx.setView).toHaveBeenCalledWith('help');
+  });
+
+  it('n opens inline search in list view', () => {
+    const ctx = makeCtx();
+    createAppKeyHandler(ctx)('n', noKey);
+    expect(ctx.setInlineSearch).toHaveBeenCalledWith(true);
+    expect(ctx.setSearchQuery).toHaveBeenCalledWith('');
+  });
+
+  it('Enter opens detail for the cursored job', () => {
+    const ctx = makeCtx();
+    createAppKeyHandler(ctx)('', { return: true });
+    expect(ctx.setSelectedJob).toHaveBeenCalledWith({ id: 1 });
+    expect(ctx.setView).toHaveBeenCalledWith('detail');
+  });
+
+  it('c in detail view triggers a dossier for the selected job', () => {
+    const job = { id: 9 };
+    const ctx = makeCtx({ view: 'detail', selectedJob: job });
+    createAppKeyHandler(ctx)('c', noKey);
+    expect(ctx.handleDossier).toHaveBeenCalledWith(job);
+  });
+
+  it('Escape in ai view returns to detail when a job is selected', () => {
+    const ctx = makeCtx({ view: 'ai', selectedJob: { id: 1 } });
+    createAppKeyHandler(ctx)('', { escape: true });
+    expect(ctx.setView).toHaveBeenCalledWith('detail');
+  });
+
+  it('Tab cycles forward and resets cursor', () => {
+    const ctx = makeCtx();
+    createAppKeyHandler(ctx)('', { tab: true });
+    expect(ctx.setTab).toHaveBeenCalledWith('new');
+    expect(ctx.setCursor).toHaveBeenCalledWith(0);
+  });
+
+  it('Shift+Tab cycles backward (last setTab wins)', () => {
+    const ctx = makeCtx({ tab: 'new' });
+    createAppKeyHandler(ctx)('', { tab: true, shift: true });
+    // both branches fire; backward call is last → 'all'
+    expect(ctx.setTab).toHaveBeenLastCalledWith('all');
+  });
+});