Skip to content

refactor(growth): consolidate shared code and remove unused marketing scaffolding - #1049

Merged
blove merged 1 commit into
mainfrom
blove/growth-structure
Sep 7, 2026
Merged

refactor(growth): consolidate shared code and remove unused marketing scaffolding#1049
blove merged 1 commit into
mainfrom
blove/growth-structure

Conversation

@blove

@blove blove commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

Growth ownership was split across an unfinished marketing pipeline, duplicated identity configuration, and production research primitives stored under pilot. This change removes unused scaffolding and makes the shared boundaries explicit while preserving deployed graph identifiers, email policy, and service topology.

  • Remove unused marketing agent/metrics/approval stubs, disconnected website conversion helpers and campaign aggregate mapper; preserve working publishing adapters and campaign drafts.
  • Share keyring parsing in Growth, move reusable crawler capture into a private library, and move production research primitives into company. Keep the standalone research artifact free of local capture dependencies.
  • Add the Growth architecture/operator guide, consistent internal publishing package names and command aliases, configurable dry-run storage, and safe X refresh logging.
  • Enforce dependency boundaries and run publishing/capture checks in the existing CI lanes. Existing lifecycle imports of research protocol/cleanup helpers remain explicit exceptions.

Validation: fresh npm ci; Growth tests/lint/build; website tests/lint; scripts tests; 87 CI classifier tests; publishing tests/lint (72 tests); capture tests/check/lint (104 tests); lifecycle and research tests/check/lint/native builds. Standalone research artifact independently installed outside the workspace and all three graphs imported under Node 24. Native Vercel artifact import/auth checks passed. Existing lint warnings remain; no live messages or contact changes performed.

The live PostHog quality/reporting issues identified in the audit are separate follow-ups and are not claimed fixed here.

@vercel

vercel Bot commented Sep 7, 2026

Copy link
Copy Markdown

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

Project Deployment Actions Updated
threadplane Ready Ready Preview Sep 7, 2026 3:58pm UTC

Request Review

@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.

@github-actions

github-actions Bot commented Sep 7, 2026

Copy link
Copy Markdown
Contributor

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


I'll analyze this and get back to you.

@blove
blove enabled auto-merge (squash) September 7, 2026 15:56
@blove
blove merged commit 625a429 into main Sep 7, 2026
77 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