Removed @tryghost/parse-email-address from pnpm dev fan-out

[9larsons]

@tryghost/parse-email-address is a 20-line workspace package wrapping the npm parse-email-address module. Its dev script runs tsc --watch continuously, emitting build/index.js and build/index.d.ts (~145 MB RSS for the tsc process, plus ~145 MB of Nx run-executor + pnpm wrapper overhead).

None of that output is consumed in dev. The package's package.json exports declares:

"exports": {
  ".": {
    "source": "./src/index.ts",
    "types": "./build/index.d.ts",
    "default": "./build/index.js"
  }
}

ghost/core runs with node --conditions=source --import=tsx (see ghost/core/nodemon.json), so the source condition wins and ghost/core imports the .ts source directly via tsx. The build/ output is purely a production artifact, never read at dev time. tsc --watch was running for nothing.

Change

Drop @tryghost/parse-email-address from docker:dev's dependsOn list in the root package.json. Saves ~290 MB RSS and one process per pnpm dev session.

pnpm --filter @tryghost/parse-email-address dev still works for solo work on the package itself. CI builds (which need build/) are unaffected since they invoke build targets directly.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 842e913b-0062-4bdc-912a-110da2f66b39

📥 Commits

Reviewing files that changed from the base of the PR and between f9c264e and b0009a0.

📒 Files selected for processing (2)

• ghost/parse-email-address/package.json
• package.json

---

Walkthrough

A prepare script (pnpm build) is added to ghost/parse-email-address/package.json so the package builds automatically on install. Correspondingly, @tryghost/parse-email-address is removed from the nx.targets.docker:dev.dependsOn projects list in the root package.json, since the build is now handled by the lifecycle script rather than the nx pipeline.

Possibly related PRs

• TryGhost/Ghost#28842: Modifies the same ghost/parse-email-address/package.json export configuration for the same package.
• TryGhost/Ghost#28844: Removes the root pnpm build step from pnpm setup, coordinating with where package builds are triggered across the workspace.
• TryGhost/Ghost#28902: Also modifies nx.targets.docker:dev.dependsOn in the root package.json.

Suggested reviewers

• EvanHahn
• jonatansberg

🚥 Pre-merge checks | ✅ 4

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly matches the main change: removing @tryghost/parse-email-address from the pnpm dev fan-out.
Description check	✅ Passed	The description is directly related to the change and explains why the package was removed from dev fan-out.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch drop-parse-email-address-from-dev-fanout

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands.}

[nx-cloud]

🤖 Nx Cloud AI Fix

Ensure the fix-ci command is configured to always run in your CI pipeline to get automatic fixes in future runs. For more information, please see https://nx.dev/ci/features/self-healing-ci

---

View your CI Pipeline Execution ↗ for commit b0009a0

Command	Status	Duration	Result
nx run ghost:test:legacy	✅ Succeeded	1m 24s	View ↗

💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗

---

☁️ Nx Cloud last updated this comment at 2026-06-29 19:59:19 UTC

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.31%. Comparing base (41355a4) to head (b0009a0).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main   #28969      +/-   ##
==========================================
+ Coverage   74.28%   74.31%   +0.03%     
==========================================
  Files        1562     1562              
  Lines      135348   135348              
  Branches    16447    16450       +3     
==========================================
+ Hits       100548   100589      +41     
+ Misses      33802    33761      -41     
  Partials      998      998              

Flag	Coverage Δ
e2e-tests	76.46% <ø> (+0.03%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[9larsons]

I think this breaks Docker build as-is. Looking. edit: specifically the prepare, but I don't think we need that since the lint command runs build for this package...