Added DEBUG env passthrough for yarn dev:forward

[kevinansfield]

no issue

• allows for debug logging, e.g. DEBUG=knex:query,knex:bindings yarn dev:forward

[coderabbitai]

Walkthrough

A new environment variable entry DEBUG: ${DEBUG:-} was added to the ghost-dev service in compose.dev.yaml. This variable uses parameter expansion to default to an empty string if the DEBUG environment variable is not provided. The addition is placed alongside existing environment variable declarations without altering other service configuration settings or dependencies.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Specific areas requiring attention:
  • Verify the DEBUG environment variable name matches the intended framework or library requirements
  • Confirm the parameter expansion syntax ${DEBUG:-} produces the intended default behavior (empty string vs. unset)
  • Check whether this DEBUG variable aligns with existing development configuration practices in the project

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately describes the main change: adding DEBUG environment variable passthrough for yarn dev:forward, which matches the changeset.
Description check	✅ Passed	The description is directly related to the changeset, explaining the purpose of the DEBUG env variable addition and providing a usage example.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch dev-forward-debug-passthrough

---

📜 Recent review details

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 62c7fe4 and be83a19.

📒 Files selected for processing (1)

• compose.dev.yaml (1 hunks)

🧰 Additional context used

🧠 Learnings (4)

📓 Common learnings

Learnt from: CR
Repo: TryGhost/Ghost PR: 0
File: AGENTS.md:0-0
Timestamp: 2025-11-25T14:28:50.351Z
Learning: Use `yarn dev:forward` for hybrid Docker + host development with backend in Docker and frontend dev servers on host

Learnt from: jonatansberg
Repo: TryGhost/Ghost PR: 25485
File: compose.dev.yaml:0-0
Timestamp: 2025-11-25T13:09:33.918Z
Learning: In the Ghost Docker Compose development setup (compose.dev.yaml), the analytics service (ghost/traffic-analytics:1.0.20) requires `platform: linux/amd64` to be explicitly set, as this platform specification is necessary for now.

Learnt from: jonatansberg
Repo: TryGhost/Ghost PR: 25552
File: e2e/helpers/environment/service-managers/dev-ghost-manager.ts:210-247
Timestamp: 2025-12-01T08:42:35.320Z
Learning: In e2e/helpers/environment/service-managers/dev-ghost-manager.ts, the hardcoded volume name 'ghost-dev_shared-config' at line 231 is intentional. E2E tests run under the 'ghost-dev-e2e' project namespace but deliberately mount the shared-config volume from the main 'ghost-dev' project to access Tinybird credentials created by yarn dev:forward. This is cross-project volume sharing by design.

📚 Learning: 2025-11-25T13:09:33.918Z

Learnt from: jonatansberg
Repo: TryGhost/Ghost PR: 25485
File: compose.dev.yaml:0-0
Timestamp: 2025-11-25T13:09:33.918Z
Learning: In the Ghost Docker Compose development setup (compose.dev.yaml), the analytics service (ghost/traffic-analytics:1.0.20) requires `platform: linux/amd64` to be explicitly set, as this platform specification is necessary for now.

Applied to files:

• compose.dev.yaml

📚 Learning: 2025-12-01T08:42:35.320Z

Learnt from: jonatansberg
Repo: TryGhost/Ghost PR: 25552
File: e2e/helpers/environment/service-managers/dev-ghost-manager.ts:210-247
Timestamp: 2025-12-01T08:42:35.320Z
Learning: In e2e/helpers/environment/service-managers/dev-ghost-manager.ts, the hardcoded volume name 'ghost-dev_shared-config' at line 231 is intentional. E2E tests run under the 'ghost-dev-e2e' project namespace but deliberately mount the shared-config volume from the main 'ghost-dev' project to access Tinybird credentials created by yarn dev:forward. This is cross-project volume sharing by design.

Applied to files:

• compose.dev.yaml

📚 Learning: 2025-06-19T22:57:05.880Z

Learnt from: cmraible
Repo: TryGhost/Ghost PR: 23941
File: .github/workflows/ci.yml:911-914
Timestamp: 2025-06-19T22:57:05.880Z
Learning: In Ghost, when NODE_ENV is set to testing-mysql, Ghost uses config.testing-mysql.json configuration which sets the server port to 2369 instead of the default 2368. This also applies to other testing environments like testing and testing-browser.

Applied to files:

• compose.dev.yaml

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: Build & Push Docker Image

🔇 Additional comments (1)

compose.dev.yaml (1)

72-72: LGTM! Clean implementation of DEBUG passthrough.

The syntax is correct, and the empty string default ensures no impact when the DEBUG environment variable isn't set. This follows the established pattern used elsewhere in the file and enables valuable debugging capabilities for development workflows.

---

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 and usage tips.}