Skip to content

feat(lifecycle): add optional bounded Firecrawl company capture - #1017

Merged
blove merged 1 commit into
mainfrom
blove/growth-firecrawl-v1
Sep 5, 2026
Merged

feat(lifecycle): add optional bounded Firecrawl company capture#1017
blove merged 1 commit into
mainfrom
blove/growth-firecrawl-v1

Conversation

@blove

@blove blove commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Company enrichment can lose homepage evidence to redirects or response size limits. Add an explicitly configured Firecrawl homepage capture that returns the existing bounded evidence schema, while keeping direct capture as the default and email delivery independent of enrichment.

The adapter makes one request with a 15-second total deadline, a 2 MiB response cap, code-only failures, and bounded capture diagnostics. It validates requested/final provenance and public hostnames; Firecrawl owns remote DNS and intermediate redirect safety. No new agent workflow, fallback chain, or SDK dependency.

Validation: lifecycle and growth-research test, check, lint, and build targets passed; 52 new adapter/selector tests passed. Six fresh public homepage responses replayed through the actual adapter produced schema-valid evidence. Those experiments used keyless access and do not verify an authenticated account. Production remains default-direct until an account key is configured and authenticated capture is verified.

@vercel

vercel Bot commented Sep 5, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
threadplane Building Building Preview Sep 5, 2026 7:49pm UTC

Request Review

@blove
blove enabled auto-merge (squash) September 5, 2026 19:49
@github-actions

github-actions Bot commented Sep 5, 2026

Copy link
Copy Markdown
Contributor

Claude finished @blove's task in 0s —— View job


I'll analyze this and get back to you.

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval: this PR received an intelligent (AI) code review. See the review comments on this PR.

@blove
blove merged commit 50c70c9 into main Sep 5, 2026
34 of 35 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant