You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
In-app support will let Ghost(Pro) users open Help in a sidebar inside Admin rather than a new tab. Admin will mark the Help links with an attribute when this flag is on, and the host script binds to those links to open the support app. Keeping it behind a private flag means the Admin changes can merge and deploy before the support app and host script are ready, with no change for anyone until the flag is enabled.
ref https://linear.app/ghost/issue/GVA-1144/add-labs-flag-to-ghost-for-in-app-support
In-app support will let Ghost(Pro) users open Help in a sidebar inside Admin
rather than a new tab. Admin will mark the Help links with an attribute when
this flag is on, and the host script binds to those links to open the support
app. Keeping it behind a private flag means the Admin changes can merge and
deploy before the support app and host script are ready, with no change for
anyone until the flag is enabled.
No actionable comments were generated in the recent review.
Walkthrough
The change adds inAppSupport to the private Labs feature list and to the Admin Labs settings list. The Admin entry is named “In-app support” and notes that it requires a host script to provide the support app.
Possibly related PRs
TryGhost/Ghost#32087: This PR also changes the Admin Help and support experience, adding a support-request form inside the Admin help chat.
Suggested reviewers: 9larsons
Merge Risk: Minimal · up to ff6bd
The lab flag registration and Admin setting are consistent and safe to merge.
Pre-merge checks: 7
Check name
Status
Explanation
Title check
The title clearly and concisely identifies the main change: adding a Labs flag for in-app support.
Description check
The description directly explains the in-app support behavior, the private flag, and the deployment purpose. It matches the changeset.
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Type-Safe Boundaries
PASS. The PR only adds the static inAppSupport flag to the typed Admin feature list and the private Labs allowlist. It does not consume HTTP, API, environment, database, filesystem, queue, or event …
New Files Are Typescript
The PR changes two pre-existing files: one .tsx file and one .js file. The diff reports both as modified (M), and both existed in the base ref. The PR adds no new .js, .jsx, .cjs, or `.mjs…
Tooling Ownership
PASS. The PR changes only Admin Labs UI and Ghost Core Labs policy. apps/admin/src/settings/advanced/labs/private-features.tsx places the toggle with the existing private feature entries, and `ghost…
Full details: Type-Safe Boundaries
Explanation
PASS. The PR only adds the static inAppSupport flag to the typed Admin feature list and the private Labs allowlist. It does not consume HTTP, API, environment, database, filesystem, queue, or event data. It adds no any, unchecked as, @ts-ignore, or @ts-nocheck, and it does not duplicate a Zod-described type.
Full details: New Files Are Typescript
Explanation
The PR changes two pre-existing files: one .tsx file and one .js file. The diff reports both as modified (M), and both existed in the base ref. The PR adds no new .js, .jsx, .cjs, or .mjs source file.
Full details: Tooling Ownership
Explanation
PASS. The PR changes only Admin Labs UI and Ghost Core Labs policy. apps/admin/src/settings/advanced/labs/private-features.tsx places the toggle with the existing private feature entries, and ghost/core/shared/labs.js places inAppSupport in PRIVATE_FEATURES, which feeds the writable allowlist and flag list. The existing settings write and input-filter paths consume that shared list. No scripts, tools, configuration, or misplaced scripts folder changed.
Finishing Touches
Generate docstrings
Commit to this branch
Create a new PR
Generate unit tests (beta)
Commit to this branch
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Tip
🎯 Customize your risk assessments
Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.
✅ Automatic approvals, with your rules
Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 9 remain after this review.
New source files must be TypeScript: flag new JS files as a required change unless exempt (DB migrations, tool/config files, tools/scripts/, docker/, generated code).
CodeRabbit configuration file
Files:
ghost/core/shared/labs.js
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.
Applicability reference: AGENTS.md: # AGENTS.md Start with: [Codebase direction](docs/codebase/direction.md) Source excerpt: Build new Admin UI in [`apps/admin/`](../../apps/admin/) with `admin-x-framework` for API access and Shade for UI.
Learnt from: troyciesco
Repo: TryGhost/Ghost PR: 29497
File: apps/admin/src/automations/components/canvas/off-value.tsx:4-4
Timestamp: 2026-07-21T19:57:01.324Z
Learning: Admin UI in Ghost is intentionally not localized. During code review, do not request adding i18n/translation hooks, wrappers, or new locale keys (e.g., updates to `packages/i18n/locales/en/ghost.json`) for Admin UI strings, including any React components under `apps/admin/src/`.
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
✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.88%. Comparing base (a654d53) to head (ff6bdb7). ⚠️ Report is 5 commits behind head on main.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
ref https://linear.app/ghost/issue/GVA-1144/add-labs-flag-to-ghost-for-in-app-support
In-app support will let Ghost(Pro) users open Help in a sidebar inside Admin rather than a new tab. Admin will mark the Help links with an attribute when this flag is on, and the host script binds to those links to open the support app. Keeping it behind a private flag means the Admin changes can merge and deploy before the support app and host script are ready, with no change for anyone until the flag is enabled.