Skip to content

✨ Released Admin 7 page chrome - #30499

Closed
peterzimon wants to merge 1 commit into
TryGhost:mainfrom
peterzimon:codex/remove-admin7-page-chrome-flag
Closed

✨ Released Admin 7 page chrome#30499
peterzimon wants to merge 1 commit into
TryGhost:mainfrom
peterzimon:codex/remove-admin7-page-chrome-flag

Conversation

@peterzimon

Copy link
Copy Markdown
Contributor

What changed

Milestone 1 of the Admin 7.0 project has completed private dogfooding. This makes the new desktop Admin shell and typography the default for eligible users.

  • Removes admin7PageChrome from the Core Labs allowlist and Admin private-features UI.
  • Removes the Labs config dependency from the Admin 7 rollout boundary.
  • Deletes the obsolete flag-off assertions and updates coverage to prove the shell works while Labs config is pending or unavailable.
  • Preserves the existing mobile, contributor, and Settings page-chrome boundaries.

Testing

  • Admin acceptance: sidebar navigation (29 tests)
  • Admin acceptance: Settings layout (7 tests)
  • Admin acceptance: advanced settings (21 tests)
  • Admin acceptance: analytics, post analytics, and members import (33 tests)
  • Core Labs unit tests (19 tests)
  • Core Admin config and settings API tests (36 tests)
  • Targeted Admin/Core ESLint and dependency checks
  • Formatting check

Package-wide Admin lint/typecheck remains blocked by existing errors in unchanged src/settings/membership/tiers/tier-checkout-collection.tsx.

  • I have read and followed the Contributor Guide
  • I have explained my change
  • I have written automated tests to prove my change works

no ref

Milestone 1 has completed private dogfooding, so the refreshed desktop shell and typography should now be available to every eligible Admin user. Remove the Labs registration and toggle, make the rollout boundary independent of Core config, and keep the existing mobile, contributor, and Settings layout boundaries.
@coderabbitai

coderabbitai Bot commented Sep 3, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: QUIET

Plan: Team

Run ID: d518b4d9-a899-47dc-8acb-4a3f966a049c

📥 Commits

Reviewing files that changed from the base of the PR and between 8477639 and 64a36c4.

📒 Files selected for processing (12)
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/admin-layout.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/posts/analytics/post-analytics.acceptance.test.tsx
  • apps/admin/src/settings/advanced/advanced.acceptance.test.tsx
  • apps/admin/src/settings/advanced/labs/private-features.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • ghost/core/core/shared/labs.js
  • ghost/core/test/e2e-api/admin/config.test.js
  • ghost/core/test/unit/shared/labs.test.js
💤 Files with no reviewable changes (6)
  • ghost/core/test/e2e-api/admin/config.test.js
  • apps/admin/src/settings/advanced/labs/private-features.tsx
  • ghost/core/core/shared/labs.js
  • apps/admin/src/posts/analytics/post-analytics.acceptance.test.tsx
  • ghost/core/test/unit/shared/labs.test.js
  • apps/admin/src/settings/advanced/advanced.acceptance.test.tsx

Included review availability: Your plan provides up to 10 included reviews per hour; 9 remain after this review.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (12)
  • GitHub Check: Tinybird required tests passed or skipped
  • GitHub Check: Legacy tests (Node 24.20.0, mysql8)
  • GitHub Check: Unit tests (Node 24.20.0)
  • GitHub Check: Legacy tests (Node 22.23.1, mysql8)
  • GitHub Check: Lint
  • GitHub Check: Unit tests (Node 22.23.1)
  • GitHub Check: Acceptance tests (Node 24.20.0, mysql8)
  • GitHub Check: Check app version bump
  • GitHub Check: Build Admin
  • GitHub Check: App Playwright Acceptance Tests (@tryghost/admin)
  • GitHub Check: Acceptance tests (Node 22.23.1, mysql8)
  • GitHub Check: Build Docker Images
🧰 Additional context used
📓 Path-based instructions (7)
Review Admin UI for existing Shade reuse, correct component layer, semantic tokens, accessible interaction states, and whole-sentence translations.

⚙️ CodeRabbit configuration file

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/layout/admin-layout.tsx
Review whether tests prove changed behaviour, meaningful error/edge paths, and externally observable contracts without coupling to implementation details.

⚙️ CodeRabbit configuration file

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
Review lens: "where does this data become trusted?" Boundary data (HTTP input, external API/SDK responses, env/config, DB/filesystem reads, queue/webhook/event payloads) is `unknown` until validated — Zod by default.

⚙️ CodeRabbit configuration file

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/layout/admin-layout.tsx
Prioritise concrete correctness, security, data-integrity, compatibility, and regression risks.

⚙️ CodeRabbit configuration file

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/layout/admin-layout.tsx
Type-safe boundaries: Fail only if the PR: consumes boundary data (HTTP input, external API/SDK responses, env/config, DB/filesystem reads, queue/webhook/event payloads) without validating it first — Zod by default, another format only wher...

📄 CodeRabbit inference engine (Custom checks)

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/layout/admin-layout.tsx
Build new features in React, use `admin-x-framework` for APIs, and use Shade for UI.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/layout/admin-layout.tsx
Always use `pnpm`, never npm or Yarn.

📄 CodeRabbit inference engine (AGENTS.md)

Files:

  • apps/admin/src/members/import-members-gate.acceptance.test.tsx
  • apps/admin/src/analytics/analytics.acceptance.test.tsx
  • apps/admin/src/layout/sidebar.acceptance.test.tsx
  • apps/admin/src/settings/layout.acceptance.test.tsx
  • apps/admin/src/layout/use-admin7.ts
  • apps/admin/src/layout/admin-layout.tsx

Walkthrough

The admin7PageChrome Labs flag was removed. The useAdmin7 hook now enables the Admin 7 shell without configuration lookup. Acceptance tests now verify default Admin 7 rendering across analytics, settings, sidebar, posts, and member import flows. A local layout constant was renamed.

Suggested reviewers: 9larsons

Merge Risk: ⚪ Minimal · up to 64a36

Eligible desktop Admin users receive the Admin 7 shell and typography by default without waiting for Labs configuration, while the existing mobile and eligibility boundaries remain in place. The supplied coverage indicates no remaining merge-blocking risk.

🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the release of the Admin 7 page chrome, which is the main change in the pull request.
Description check ✅ Passed The description directly explains the Admin 7 rollout, flag removal, rollout boundary changes, test updates, and preserved behavior.
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.
Type-Safe Boundaries ✅ Passed PASS: The PR removes the useBrowseConfig import and the config.config.labs.admin7PageChrome read from useAdmin7; it does not add a new boundary-data read. The remaining production changes are bo…
New Files Are Typescript ✅ Passed The pull request adds no files. git diff --diff-filter=A HEAD^ HEAD returns no paths, and the three changed .js files are marked M with confirmed parent entries. The check only fails for newly a…
Full details: Type-Safe Boundaries

Explanation

PASS: The PR removes the useBrowseConfig import and the config.config.labs.admin7PageChrome read from useAdmin7; it does not add a new boundary-data read. The remaining production changes are boolean calculations and a class-name rename. The diff adds no any, unchecked as, @ts-ignore, @ts-nocheck, or hand-written schema-duplicate type. Changed tests and incidental JavaScript files are explicitly exempt.

Full details: New Files Are Typescript

Explanation

The pull request adds no files. git diff --diff-filter=A HEAD^ HEAD returns no paths, and the three changed .js files are marked M with confirmed parent entries. The check only fails for newly added .js/.jsx/.cjs/.mjs source files, so this pull request passes.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

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

@nx-cloud

nx-cloud Bot commented Sep 3, 2026

Copy link
Copy Markdown

🤖 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 64a36c4

Command Status Duration Result
nx run @tryghost/admin:test:acceptance ❌ Failed 8m 17s View ↗
nx run ghost:test:ci:integration ✅ Succeeded 4m 36s View ↗
nx run ghost:test:integration ✅ Succeeded 3m 48s View ↗
nx run ghost:test:ci:e2e ✅ Succeeded 4m 14s View ↗
nx run ghost:test:legacy ✅ Succeeded 3m 13s View ↗
nx run-many -t test:unit -p @tryghost/admin,ghost ✅ Succeeded 3m 32s View ↗
nx run ghost:test:e2e ✅ Succeeded 2m 59s View ↗
nx run ghost-monorepo:lint:boundaries ✅ Succeeded 22s View ↗
Additional runs (4) ✅ Succeeded ... View ↗

💡 Dealing with memory or CPU issues? See memory and CPU details with the resource usage add-on ↗.


☁️ Nx Cloud last updated this comment at 2026-09-03 12:31:15 UTC

@codecov

codecov Bot commented Sep 3, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 67.46%. Comparing base (8477639) to head (64a36c4).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #30499      +/-   ##
==========================================
- Coverage   67.49%   67.46%   -0.03%     
==========================================
  Files        1664     1664              
  Lines       60060    60060              
  Branches    10386    10386              
==========================================
- Hits        40535    40518      -17     
- Misses      17236    17254      +18     
+ Partials     2289     2288       -1     
Flag Coverage Δ
e2e-tests 70.24% <ø> (-0.04%) ⬇️

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.

@peterzimon

Copy link
Copy Markdown
Contributor Author

Cancelled this PR to separate concerns

@peterzimon peterzimon closed this Sep 3, 2026
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