Jm/improved sync mdx by JonathanMatthey · Pull Request #9 · interledger/interledger.org-v5

JonathanMatthey · 2026-02-05T22:43:54Z

improves sync scripts
adds tests
major refactor for lifecycles
also improve summit styling

JonathanMatthey · 2026-02-06T11:18:03Z

sync import locale seems broken, possibly because contentId of locale wont match the new generated english version of the newly imported entry.

possibly add the contentId in the english FrontMatter

Copilot

Pull request overview

This PR implements significant improvements to the sync infrastructure and adds i18n support across the codebase. It transitions from SSR to SSG for foundation and summit pages, refactors CMS lifecycles into reusable utilities, modularizes sync scripts, adds test coverage, and improves summit page styling.

Changes:

Converts foundation and summit pages from server-side rendering (prerender=false) to static site generation with getStaticPaths
Refactors CMS lifecycle hooks into reusable factory functions (mdxLifecycle.ts, navigationLifecycle.ts) with shared MDX utilities
Modularizes sync-mdx script into testable components with added test coverage
Adds comprehensive i18n plugin options to all CMS components and content types
Updates styling for summit header (extracted CSS classes) and footer components

Reviewed changes

Copilot reviewed 69 out of 71 changed files in this pull request and generated 7 comments.

Show a summary per file

File	Description
src/pages/[...page].astro	Changed to SSG with getStaticPaths, filters out 'home' slug
src/pages/summit/[...page].astro	Changed to SSG with getStaticPaths, added background styling
src/pages/developers/blog/preview.astro	Fixed import paths (added '../' prefix)
src/content/summit/*.mdx	Added new summit pages (faq, code-of-conduct) and Spanish localizations
src/content/foundation-pages/*.mdx	Removed 'page.mdx', added financial-services, education, ambassadors, updated about-us
src/content/es/foundation-pages/sobre-nosotros.mdx	Added Spanish localized about-us page (content needs translation)
src/components/pages/SummitHeader.astro	Refactored inline styles to CSS class variables
src/components/pages/SummitFooter.astro	Updated styling with improved colors and spacing
cms/src/utils/*.ts	New lifecycle utilities (mdxLifecycle, navigationLifecycle, mdx, gitSync)
cms/src/api/*/lifecycles.ts	Updated to use new factory functions
cms/src/components/*/.json	Added i18n localized: true to all fields
cms/scripts/sync-mdx/*.cjs	Modularized sync script into separate concerns (scan, sync, strapi, markdown, mdx)
cms/scripts/sync-mdx/tests/*.js	Added comprehensive test coverage for sync functionality
cms/scripts/sync-navigation.cjs	New script to sync navigation JSON to Strapi
package.json	Disabled ESLint warnings (--max-warnings=9999), removed export scripts
eslint.config.js	Disabled multiple TypeScript linting rules
astro.config.mjs	Made link validation conditional on CI or env variable
.github/workflows/*.yml	Updated to use Bun, added missing test steps
cms/README.md, cms/QUICKSTART.md	Updated documentation to reference Bun instead of npm

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot · 2026-02-06T12:26:41Z

cms/src/utils/gitSync.ts

 export async function gitCommitAndPush(
-  filepath: string,
+  filepath: string | string[],
  message: string
 ): Promise<void> {
  if (process.env.STRAPI_DISABLE_GIT_SYNC === 'true') {


The git workflow should pull before committing to avoid race conditions. The current workflow of add → commit → pull --rebase → push can lose commits if conflicts occur during rebase. Consider changing to: pull → add → commit → push to ensure changes are integrated before committing.

Copilot · 2026-02-06T12:26:41Z

cms/scripts/sync-mdx/env.cjs

+  // Ensure cms/.env takes precedence
+  delete process.env.STRAPI_API_TOKEN;
+  delete process.env.STRAPI_PREVIEW_TOKEN;
+
+  const envPath = path.join(projectRoot, 'cms', '.env');
+  if (fs.existsSync(envPath)) {
+    require('dotenv').config({ path: envPath });


Deleting environment variables from process.env before loading the .env file can cause issues in certain deployment environments where these variables are set via the platform. Consider using dotenv's override option instead of manually deleting variables, or check if variables exist before deleting them.

Suggested change

// Ensure cms/.env takes precedence

delete process.env.STRAPI_API_TOKEN;

delete process.env.STRAPI_PREVIEW_TOKEN;

const envPath = path.join(projectRoot, 'cms', '.env');

if (fs.existsSync(envPath)) {

require('dotenv').config({ path: envPath });

// Ensure cms/.env takes precedence by overriding existing values

const envPath = path.join(projectRoot, 'cms', '.env');

if (fs.existsSync(envPath)) {

require('dotenv').config({ path: envPath, override: true });

Copilot · 2026-02-06T12:26:41Z

src/content/es/foundation-pages/sobre-nosotros.mdx

+The Interledger Foundation (ILF) is a mission-driven nonprofit ensuring that no one is left behind in the digital economy. We support inclusive innovation and financial infrastructure that connects people, communities, and entire economies globally.
+
+## Our role?
+
+We collaborate with entrepreneurs, organizations, and governments to [advocate](/policy-and-advocacy) for equitable access to financial systems and steward the interoperable technologies that power the Internet of Opportunity (IoO). To advance this work we fund [financial services](/financial-services) projects, [education](/education) initiatives, and [ambassador](/ambassadors) programs.
+
+## The Problem We’re Solving
+
+The global financial system is broken, fragmented, and slow.
+
+Billions are excluded from full participation due to:
+
+- High transaction costs and slow settlements  
+- Disconnected local and global payment systems  
+- Inaccessible remittance infrastructure  
+- Limited digital financial tools for creators, freelancers, and the unbanked  
+
+The Interledger Foundation is changing that.
+
+## Our Vision: The Internet of Opportunity
+
+Imagine a world where everyone can send a payment as easily as an email.
+
+With the Internet of Opportunity, powered by Interledger, we make that vision real:
+
+**Open Access**  
+Anyone can participate in the digital economy, regardless of location or income.
+
+**Seamless Interoperability**  
+Payments move instantly across platforms, currencies, and borders.
+
+**Programmable Transactions**  
+New business models become possible, from real-time creator tips to pay-as-you-go utilities.


The Spanish localized page (sobre-nosotros.mdx) has a Spanish slug and heroTitle but the content body is in English. This should be translated to Spanish to provide a proper localized experience for Spanish-speaking users.

Copilot · 2026-02-06T12:26:42Z

.github/workflows/test-build.yml

+  push:
  pull_request:


The workflow now runs on every push to any branch, not just pull requests to main. This can cause unnecessary CI runs for personal development branches and increase CI costs. Consider keeping the trigger limited to pull requests targeting main and pushes to main/production branches.

Suggested change

push:

pull_request:

push:

branches:

- main

- production

pull_request:

branches:

- main

Copilot · 2026-02-06T12:26:42Z

.github/workflows/test-build.yml

+        run: bun run test
      - name: Test build website
        run: bun run build
+      - name: Install Playwright browsers
+        run: bunx playwright install --with-deps
+      - name: E2E tests
+        run: bun run test:e2e


The workflow references bun run test and bun run test:e2e scripts, but these are not defined in package.json. The CI build will fail when it tries to run these commands.

Suggested change

run: bun run test

- name: Test build website

run: bun run build

- name: Install Playwright browsers

run: bunx playwright install --with-deps

- name: E2E tests

run: bun run test:e2e

run: bun test

- name: Test build website

run: bun run build

- name: Install Playwright browsers

run: bunx playwright install --with-deps

- name: E2E tests

run: bunx playwright test

Copilot · 2026-02-06T12:26:42Z

eslint.config.js

+      'no-unused-vars': 'off',
+      '@typescript-eslint/no-unused-vars': 'off',
+      '@typescript-eslint/no-explicit-any': 'off',
+      '@typescript-eslint/no-empty-function': 'off',
+      '@typescript-eslint/no-non-null-assertion': 'off',
+      '@typescript-eslint/ban-ts-comment': 'off',
+      '@typescript-eslint/ban-types': 'off',
+      '@typescript-eslint/prefer-namespace-keyword': 'off',
+      'astro/no-set-text-directive': 'off',


The ESLint configuration has disabled many useful TypeScript and JavaScript linting rules. Disabling rules like @typescript-eslint/no-unused-vars, @typescript-eslint/no-explicit-any, @typescript-eslint/ban-ts-comment, and others significantly reduces code quality enforcement. Consider re-enabling these rules and fixing the underlying issues instead of suppressing all warnings.

Copilot · 2026-02-06T12:26:43Z

package.json

-    "import:nav": "npx tsx scripts/import-navigation.ts --all"
+    "check": "astro check",
+    "format": "prettier --write '**/*.{js,mjs,ts,tsx,json,md,mdx,astro}'",
+    "lint": "eslint --max-warnings=9999 ."


The lint script has been changed to allow up to 9999 warnings (--max-warnings=9999), which effectively disables warning limits. This defeats the purpose of linting. Consider setting this to a reasonable number (e.g., 0) and addressing the warnings properly.

JonathanMatthey added 12 commits February 5, 2026 22:19

run cms on bun - not npm

e3ac65c

update readme

fb036a1

fix git sync and page lifecycles

fb71ce0

update lifecycles for blog posts and summit pages

097e324

refactor lifecycle files

8533d7e

remove events from sync mdx

8cebad3

refactor sync, simpler to manager and add tests

e136024

sync navigation scripts

424af52

add faq summit page

1b35dee

add internationalization to blog posts

2a288b0

summit nav color

73273ae

nav colors

8600714

JonathanMatthey requested a review from JoblersTune February 5, 2026 22:43

JonathanMatthey self-assigned this Feb 5, 2026

JonathanMatthey added 16 commits February 5, 2026 23:51

refactor summit header / simplify tailwind

5909b14

add summit code of conduct

7f1076f

improve summit footer

32c155d

add i18n to comopnents

17ab9ae

sync summit apges

9488b92

summit page bg

faa7a8c

fix summit page sync

2b9c0c1

code of conduct mdx

8c9e43f

add about us page

b6b88fa

add more sync tests

23292bc

rm deprecated tailwind

4ad78b7

fix swirl ref

f466e70

fix refs

7c3417b

coming soon pages

5fe3190

looser lint for now

3d56405

lint

f89bf8e

JonathanMatthey added 2 commits February 6, 2026 00:54

static paths and link validation improvements

acaa660

delete

e70efc1

bosbaber requested a review from Copilot February 6, 2026 12:22

Copilot started reviewing on behalf of bosbaber February 6, 2026 12:22 View session

Copilot AI reviewed Feb 6, 2026

View reviewed changes

export contentId in lifecycle

cf85006

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Jm/improved sync mdx#9

Jm/improved sync mdx#9
JonathanMatthey wants to merge 31 commits intomainfrom
jm/improved-sync-mdx

JonathanMatthey commented Feb 5, 2026

Uh oh!

JonathanMatthey commented Feb 6, 2026 •

edited

Loading

Uh oh!

Copilot AI left a comment

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Copilot AI Feb 6, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

-  push:
-  pull_request:
+  push:
+    branches:
+      - main
+      - production
+  pull_request:
+    branches:
+      - main

Conversation

JonathanMatthey commented Feb 5, 2026

Uh oh!

JonathanMatthey commented Feb 6, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Reviewed changes

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Feb 6, 2026

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

JonathanMatthey commented Feb 6, 2026 •

edited

Loading