update gulpfile and dependencies for building i18n files

[cathysarisky]

This adds a dependency on theme-translations, so that Casper can pull new strings as they become available, and new gulp behavior to include them on build & watch.

[coderabbitai]

Walkthrough

The pull request introduces translation support to the project's build pipeline. A new development dependency (@tryghost/theme-translations) is added to package.json. In gulpfile.js, a new locales task is created to merge translation files from a ./locales-local directory into a ./locales output directory. A corresponding file watcher is added to monitor ./locales-local/**/*.json for changes and trigger the merging process. The localesWatcher is integrated into the main watcher configuration, and the locales task is included in the overall build sequence.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately summarizes the main changes: updating the gulpfile and dependencies to support internationalization (i18n) file building.
Description check	✅ Passed	The description clearly relates to the changeset, explaining the addition of the theme-translations dependency and new gulp behavior for including translations in build and watch tasks.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

• 📝 Generate docstrings (stacked PR)
• 📝 Generate docstrings (commit on current branch)

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch update-deps-and-gulp

---

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.}

[coderabbitai]

🧹 Nitpick comments (2)

package.json (1)

49-49: Consider wiring the package’s Renovate preset too.

Adding the devDependency enables the merge tooling, but the package’s own README treats automatic pickup of new shared strings as a separate step via github>TryGhost/Themes:packages/theme-translations/renovate-config. If the goal here is “pull new strings as they become available” without manual dependency refreshes, this repo still needs that extra Renovate extension. (github.com)

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@package.json` at line 49, The repo added "@tryghost/theme-translations" as a
devDependency but didn't enable the package's Renovate preset; add a Renovate
configuration that extends
"github>TryGhost/Themes:packages/theme-translations/renovate-config" so Renovate
will automatically import new shared strings, e.g., by adding an extends entry
in your Renovate config (or a "renovate" field in package.json) pointing to that
preset; ensure the extends string exactly matches
github>TryGhost/Themes:packages/theme-translations/renovate-config and keep the
existing devDependency "@tryghost/theme-translations" intact.

gulpfile.js (1)

95-107: Add livereload trigger for locale changes in watch mode.

The locales watcher rebuilds ./locales when locales-local/**/*.json changes, but doesn't trigger a browser reload like the CSS, JS, and HBS watchers do. This breaks the development experience when editing locale overrides that power template {{t}} calls.

Since @tryghost/theme-translations exposes mergeLocales as a Gulp task helper, you can use it directly and compose a reload step:

♻️ Proposed fix

-function locales(done) {
-    mergeLocales({
-        local: './locales-local',
-        output: './locales'
-    })(done);
-}
+const locales = mergeLocales({
+    local: './locales-local',
+    output: './locales'
+});
+
+function reloadLocales(done) {
+    pump([
+        src('locales/**/*.json'),
+        livereload()
+    ], handleError(done));
+}
 
 const cssWatcher = () => watch('assets/css/**', css);
 const jsWatcher = () => watch('assets/js/**', js);
 const hbsWatcher = () => watch(['*.hbs', 'partials/**/*.hbs'], hbs);
-const localesWatcher = () => watch('./locales-local/**/*.json', locales);
+const localesWatcher = () => watch('./locales-local/**/*.json', series(locales, reloadLocales));
 const watcher = parallel(cssWatcher, jsWatcher, hbsWatcher, localesWatcher);
 const build = series(css, js, locales);

🤖 Prompt for AI Agents

Verify each finding against the current code and only fix it if needed.

In `@gulpfile.js` around lines 95 - 107, The locales watcher rebuilds ./locales
but does not trigger browser reload; update the locales task/watcher so locale
changes also call the livereload step used by css/js/hbs watchers: use the
existing mergeLocales helper (mergeLocales(...)) as the locales Gulp task (or
wrap it) and compose it with the same reload step used elsewhere (i.e., make
locales a series of mergeLocales(...) followed by the reload function), and
change localesWatcher to invoke that composed task (or
watch('./locales-local/**/*.json', series(locales, reload))) so edits to
locales-local/**/*.json trigger a browser reload like css/js/hbs.

🤖 Prompt for all review comments with AI agents

Verify each finding against the current code and only fix it if needed.

Nitpick comments:
In `@gulpfile.js`:
- Around line 95-107: The locales watcher rebuilds ./locales but does not
trigger browser reload; update the locales task/watcher so locale changes also
call the livereload step used by css/js/hbs watchers: use the existing
mergeLocales helper (mergeLocales(...)) as the locales Gulp task (or wrap it)
and compose it with the same reload step used elsewhere (i.e., make locales a
series of mergeLocales(...) followed by the reload function), and change
localesWatcher to invoke that composed task (or
watch('./locales-local/**/*.json', series(locales, reload))) so edits to
locales-local/**/*.json trigger a browser reload like css/js/hbs.

In `@package.json`:
- Line 49: The repo added "@tryghost/theme-translations" as a devDependency but
didn't enable the package's Renovate preset; add a Renovate configuration that
extends "github>TryGhost/Themes:packages/theme-translations/renovate-config" so
Renovate will automatically import new shared strings, e.g., by adding an
extends entry in your Renovate config (or a "renovate" field in package.json)
pointing to that preset; ensure the extends string exactly matches
github>TryGhost/Themes:packages/theme-translations/renovate-config and keep the
existing devDependency "@tryghost/theme-translations" intact.

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 336d030f-2388-42b3-a9ee-9545e207ea12

📥 Commits

Reviewing files that changed from the base of the PR and between 4950217 and 1de4486.

📒 Files selected for processing (2)

• gulpfile.js
• package.json