fix(lifecycle): own enrichment provenance in code; fix(mailbox-poller): drop numeric separators (#985)

* fix(mailbox-poller): drop numeric separators so the Apps Script editor can parse Code.gs

The Apps Script editor rejects numeric separator literals (8_000, 4_000,
1_000_000) with 'Unexpected token ILLEGAL' at line 321 and refuses to save.
Plain literals keep the same values.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* fix(lifecycle): own enrichment provenance in code so structured output stops failing

Every production enrich job failed. The Anthropic structured-output grammar
drops array-length, numeric, and enum bounds from the JSON schema, and the
model was asked to echo provenance verbatim. In practice it returned one
draft instead of three, dropped milliseconds from retrieved_at, cited score
identifiers as sources, and fabricated placeholder sources in neutral mode.
Each of those tripped the strict artifact parse or a verifier and the job
went terminal after two attempts.

The wire schema now carries only what the model judges: summary,
confidence, cited signals, company profile, recommended angle, and drafts,
with no grammar-unenforceable bounds. The code derives sources from the
bounded evidence for cited ids, copies score metadata from the input, drops
signals that cite anything outside the evidence, nulls drafts that point at
a dropped source or an unknown angle, pads or truncates drafts to the three
slots, and strips all company claims in neutral mode. The prompt states the
slot count, the allowed angle ids, and which ids are citable.

Verified live against the API: five of five runs succeed across company and
neutral inputs, where zero succeeded before.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

* fix(lifecycle): survive unusable company pages, keep angles distinct, close evidence sequences

Three findings from vetting enrichment against live sites:

- One oversized, missing, or slow company page aborted evidence for the
  whole company, and the enrich job then failed after retries. The fetcher
  now skips such a page and keeps the rest; only the caller's abort and
  SSRF violations (now CompanyFetchSecurityError) propagate.
- Nothing stopped the model from selecting the same angle in two slots,
  which would send the same email twice. The normalizer keeps the first
  use of an angle and nulls repeats so those slots fall back to default
  copy; the prompt asks for distinct angles.
- The default day-8 copy says it is the last automated follow-up but the
  evidence variants did not. renderEvidenceCampaignTemplate takes a
  finalStep option and the dispatcher sets it for step 3.

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5.1 <noreply@anthropic.com>

Author: blove

apps/lifecycle/src/campaign/send.spec.ts

@@ -248,6 +248,40 @@ describe('prepareCampaignMessage', () => {
     ).toMatchObject({ status: 'ready', subject: 'A practical place to start' });
   });
 
+  it('closes the sequence on the final step even when evidence copy is selected', () => {
+    const cited = artifact({
+      cited_signals: [
+        { signal: 'Bounded source fact', source_ids: ['source-1'] },
+      ],
+      sources: [
+        {
+          id: 'source-1',
+          url: 'https://example.com/about',
+          retrieved_at: '2026-09-01T12:00:00.000Z',
+          content_hash: 'a'.repeat(64),
+        },
+      ],
+      drafts: [
+        { angle_id: 'streaming_foundation', source_id: 'source-1' },
+        { angle_id: 'debugging_layers', source_id: 'source-1' },
+        { angle_id: 'event_state_boundary', source_id: 'source-1' },
+      ],
+    });
+
+    const message = prepareCampaignMessage({
+      context: context({ enrichmentArtifact: cited }),
+      job: job('send_step', { campaign_version: 'v1', step: 3 }),
+      now: new Date('2026-09-09T12:05:00.000Z'),
+      unsubscribeUrl: UNSUBSCRIBE,
+    });
+
+    expect(message).toMatchObject({
+      status: 'ready',
+      subject: 'One event-state boundary',
+    });
+    expect(JSON.stringify(message)).toContain('last automated follow-up');
+  });
+
   it('falls back per fixed step when an artifact draft violates copy checks', () => {
     const invalid = artifact({
       drafts: [

apps/lifecycle/src/campaign/send.ts

@@ -240,7 +240,9 @@ function draftFor(
         source_ids.includes(selection.source_id)
       );
     if (selection !== null && cited) {
-      return renderEvidenceCampaignTemplate(selection.angle_id);
+      return renderEvidenceCampaignTemplate(selection.angle_id, {
+        finalStep: step === 3,
+      });
     }
   }
   return renderCampaignTemplate(STEP_NAMES[step]);

apps/lifecycle/src/campaign/templates.spec.ts

@@ -4,6 +4,7 @@ import {
   campaignDraftViolations,
   normalizeCampaignDraft,
   renderCampaignTemplate,
+  renderEvidenceCampaignTemplate,
 } from './templates.js';
 
 function wordCount(value: string): number {
@@ -34,6 +35,20 @@ describe('renderCampaignTemplate', () => {
     );
   });
 
+  it('appends the last-follow-up notice to an evidence template only for the final step', () => {
+    const final = renderEvidenceCampaignTemplate('event_state_boundary', {
+      finalStep: true,
+    });
+    const earlier = renderEvidenceCampaignTemplate('event_state_boundary');
+
+    expect(final.body).toContain('last automated follow-up');
+    expect(final.body.endsWith('This is my last automated follow-up.')).toBe(
+      true
+    );
+    expect(campaignDraftViolations(final)).toEqual([]);
+    expect(earlier.body).not.toContain('last automated follow-up');
+  });
+
   it('rejects an unknown campaign step at runtime', () => {
     expect(() => renderCampaignTemplate('day-30' as never)).toThrow();
   });

apps/lifecycle/src/campaign/templates.ts

@@ -194,8 +194,16 @@ export function renderCampaignTemplate(step: CampaignStep): CampaignDraft {
   return normalizeCampaignDraft(CAMPAIGN_TEMPLATES[parsedStep]);
 }
 
+const FINAL_STEP_NOTICE = 'This is my last automated follow-up.';
+
 export function renderEvidenceCampaignTemplate(
-  angle: CampaignEvidenceAngle
+  angle: CampaignEvidenceAngle,
+  options: { finalStep?: boolean } = {}
 ): CampaignDraft {
-  return normalizeCampaignDraft(EVIDENCE_TEMPLATES[angle]);
+  const template = EVIDENCE_TEMPLATES[angle];
+  return normalizeCampaignDraft(
+    options.finalStep
+      ? { ...template, body: `${template.body}\n\n${FINAL_STEP_NOTICE}` }
+      : template
+  );
 }