Cleaned up stale references to the removed posts and stats apps#29327
Conversation
ref https://linear.app/ghost/issue/PLA-246/post-dissolution-sweep-stale-docs-comments-referencing-appsposts-and - docs and comments still pointed at apps/posts and apps/stats after those packages dissolved into apps/admin domain folders — AGENTS.md even directed new admin features to apps/posts - updated the duplicated-file NOTEs in the automations email utils to point at their real admin-x-settings/apps-admin counterparts so the keep-in-sync constraint survives - dropped the shared-views TODOs whose "once the apps are merged" premise is satisfied (the layout hook already delegates to the members parser) - comment/doc-only; no behavior change
|
It looks like this PR contains a migration 👀 General requirements
Schema changes
Data changes
|
WalkthroughUpdated repository guidance and documentation to reflect the consolidated Admin React shell and its Possibly related PRs
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
|
| Command | Status | Duration | Result |
|---|---|---|---|
nx run @tryghost/admin-x-settings:test:acceptance |
✅ Succeeded | 10m 13s | View ↗ |
nx run-many --target=build --projects=tag:publi... |
✅ Succeeded | 1s | View ↗ |
nx run-many -t test:unit -p @tryghost/adapter-b... |
✅ Succeeded | 7m 59s | View ↗ |
nx run ghost:test:ci:integration |
✅ Succeeded | 2m 41s | View ↗ |
nx run ghost:test:integration |
✅ Succeeded | 2m 53s | View ↗ |
nx run @tryghost/admin:build |
✅ Succeeded | 3m 8s | View ↗ |
nx run ghost-monorepo:lint:boundaries |
✅ Succeeded | 7s | View ↗ |
nx run ghost:test:legacy |
✅ Succeeded | 2m 50s | View ↗ |
Additional runs (9) |
✅ Succeeded | ... | View ↗ |
💡 Verify your cache is correct by running tasks in a sandbox. Read docs ↗
☁️ Nx Cloud last updated this comment at 2026-07-14 19:21:20 UTC
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #29327 +/- ##
==========================================
- Coverage 73.95% 73.95% -0.01%
==========================================
Files 1579 1579
Lines 136893 136893
Branches 16570 16569 -1
==========================================
- Hits 101244 101242 -2
+ Misses 34640 34609 -31
- Partials 1009 1042 +33
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|

ref https://linear.app/ghost/issue/PLA-246/
The posts/stats packages dissolved into
apps/admindomain folders, but a number of docs and code comments still pointed at the deletedapps/posts/apps/statspaths — including AGENTS.md directing new admin features toapps/posts.Comment/doc-only; no behavior change.
@sourcelists (now matchingapps/admin/src/index.css), and the "where to build new admin features" guidanceadmin-x-activitypubdirectory@exampleheader named a deleted config file; made genericapps/admin/src/members/member-query-params.ts(and no longer agetCurrentSubscriptionhelper)Deliberately left:
apps/shade'sposts-statstransitional subpath and the docs describing it — that subpath still exists and is removed separately.Verification
eslintclean on all touched JS/TS files (also enforced by lint-staged on commit)grep -rn 'apps/posts\|apps/stats'across the repo now only matches the intentionally-kept shadeposts-statsfiles