Skip to content

i18n(fr): update how-it-works#183

Merged
dreyfus92 merged 2 commits into
withstudiocms:mainfrom
ArmandPhilippot:i18n/fr-update-how-it-works
Dec 20, 2025
Merged

i18n(fr): update how-it-works#183
dreyfus92 merged 2 commits into
withstudiocms:mainfrom
ArmandPhilippot:i18n/fr-update-how-it-works

Conversation

@ArmandPhilippot
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot commented Dec 20, 2025

Description

Adds changes from #177 to the French translations in how-it-works (except cli.mdx already updated in #178)

Note

The failing slug should be resolved once #182 is merged.

Summary by CodeRabbit

  • Documentation

    • Updated French documentation with "Updated" badges.
    • Revised SDK documentation to reference an internal database client.
    • Removed Astro DB references from content management descriptions.
    • Added supplementary reading section to SDK documentation.
  • Features

    • SDK now exports new public utilities: dbService, cache, AUTH, CONFIG, MIDDLEWARES, PLUGINS, and additional modules.

✏️ Tip: You can customize this high-level summary in your review settings.

@ArmandPhilippot ArmandPhilippot requested a review from a team as a code owner December 20, 2025 17:21
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 20, 2025

📝 Walkthrough

Walkthrough

French documentation for how StudioCMS works has been updated: badges added to sidebar metadata, Astro DB references removed from descriptions, and SDK public API documentation revised to reflect new exports (dbService, cache, AUTH, CONFIG, and others replacing db).

Changes

Cohort / File(s) Change Summary
French how-it-works documentation updates
src/content/docs/fr/how-it-works/index.mdx, src/content/docs/fr/how-it-works/restapi.mdx, src/content/docs/fr/how-it-works/sdk.mdx
Added "Mis à jour" badge to sidebar in index.mdx and sdk.mdx. Removed Astro DB references from opening descriptions across all three files. Updated sdk.mdx SDK export documentation to reflect public API changes: removed db, added dbService, cache, AUTH, CONFIG, diffTracking, MIDDLEWARES, notificationSettings, PLUGINS, resetTokenBucket, UTIL. Added "Lectures complémentaires" section to sdk.mdx.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify SDK public API changes (removed db, added dbService and others) are documented accurately
  • Confirm Astro DB references are appropriately removed across all three files
  • Check badge frontmatter syntax and consistency

Possibly related PRs

  • #141: Modifies the same French SDK documentation and SDKCoreJs export destructuring patterns
  • #179: Related to public API changes removing db and introducing dbService configuration
  • #178: Introduces Kysely-based database layer that contextualizes the SDK API surface changes documented here

Suggested labels

i18n

Suggested reviewers

  • dreyfus92
  • Adammatthiesen

Poem

🐰 A bunny hops through docs so fine,
Old Astro DB left behind,
New badges gleam, "Mis à jour" shine,
dbService dances in perfect line,
French knowledge now completely redesigned!

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately reflects the main change: updating French translations in the how-it-works documentation section.
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

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

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

@studiocms-no-reply
Copy link
Copy Markdown
Member

studiocms-no-reply commented Dec 20, 2025

Lunaria Status Overview

🌕 This pull request will trigger status changes.

Learn more

By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.

You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

File Note
fr/how-it-works/index.mdx Localization changed, will be marked as complete.
fr/how-it-works/restapi.mdx Localization changed, will be marked as complete.
fr/how-it-works/sdk.mdx Localization changed, will be marked as complete.
Warnings reference
Icon Description
🔄️ The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 1

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e168b89 and d30057e.

📒 Files selected for processing (3)
  • src/content/docs/fr/how-it-works/index.mdx (1 hunks)
  • src/content/docs/fr/how-it-works/restapi.mdx (1 hunks)
  • src/content/docs/fr/how-it-works/sdk.mdx (2 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: ArmandPhilippot
Repo: withstudiocms/docs PR: 149
File: src/content/docs/fr/guides/upgrade/version-guides/0-1-0-beta-21.mdx:13-15
Timestamp: 2025-07-24T11:57:15.225Z
Learning: In the withstudiocms/docs repository, when reviewing French translation files, maintain consistency with the English version even if it means keeping unused imports. Structural parity between language versions is prioritized over removing unused code.
Learnt from: ArmandPhilippot
Repo: withstudiocms/docs PR: 149
File: src/content/docs/fr/guides/upgrade/version-guides/0-1-0-beta-20.mdx:18-18
Timestamp: 2025-07-24T12:00:35.014Z
Learning: In the withstudiocms/docs repository, while translations should maintain structural parity with English versions, improving readability during translation is acceptable when the original text has clear redundancy or readability issues that would also impact the translation quality. The maintainer ArmandPhilippot confirmed this approach is reasonable when the technical meaning is preserved.
Learnt from: ArmandPhilippot
Repo: withstudiocms/docs PR: 130
File: src/content/docs/fr/guides/upgrade/version-guides/0-1-0-beta-17.mdx:1-11
Timestamp: 2025-04-26T12:29:17.995Z
Learning: When reviewing translations in the withstudiocms/docs repository, compare with the corresponding English version as the source of truth, not with other files in the same target language. French translations should maintain structural parity with their English counterparts.
Learnt from: ArmandPhilippot
Repo: withstudiocms/docs PR: 140
File: src/content/docs/fr/guides/upgrade/version-guides/0-1-0-beta-19.mdx:24-24
Timestamp: 2025-07-21T12:16:32.374Z
Learning: In the withstudiocms/docs repository, documentation URLs in the package-catalog may maintain legacy path names for stability and consistency across language versions, even when the actual package names have been updated. Always verify that corresponding files exist before flagging URLs as broken.
🪛 LanguageTool
src/content/docs/fr/how-it-works/restapi.mdx

[style] ~11-~11: Ce verbe peut être considéré comme familier dans un contexte formel.
Context: ...le offre un moyen simple et flexible de gérer et de diffuser votre contenu. L’API RE...

(VERBES_FAMILIERS_PREMIUM)

src/content/docs/fr/how-it-works/index.mdx

[style] ~14-~14: Ce verbe peut être considéré comme familier dans un contexte formel.
Context: ...ant conçu pour Astro qui vous permet de gérer et de diffuser votre contenu. Il est co...

(VERBES_FAMILIERS_PREMIUM)


[style] ~14-~14: Ce verbe peut être considéré comme familier dans un contexte formel.
Context: ... utiliser et suffisamment flexible pour gérer tout type de contenu. Étendre StudioCM...

(VERBES_FAMILIERS_PREMIUM)

src/content/docs/fr/how-it-works/sdk.mdx

[style] ~14-~14: Ce verbe peut être considéré comme familier dans un contexte formel.
Context: ...mble de fonctions et d’utilitaires pour gérer et diffuser votre contenu via notre pro...

(VERBES_FAMILIERS_PREMIUM)


[style] ~14-~14: Cette structure peut être modifiée afin de devenir plus percutante.
Context: ...r votre contenu via notre propre client de base de données interne [Kysely](https://kys...

(DE_BASE3)


[style] ~14-~14: Utilisé hors du contexte militaire, on préfèrera d’autres tournures à « baser sur ».
Context: ...ase du tableau de bord StudioCMS et est basé sur Effect. ## U...

(BASER_SUR)

⏰ 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). (2)
  • GitHub Check: build-and-push
  • GitHub Check: Run Starlight Link Check
🔇 Additional comments (6)
src/content/docs/fr/how-it-works/index.mdx (2)

6-9: LGTM! Badge addition is properly formatted.

The "Mis à jour" badge with success variant is correctly added with proper YAML indentation.


14-18: LGTM! Translation updates align with PR objectives.

The revised descriptions successfully remove Astro DB references while maintaining clear and accurate French technical documentation. The text flows naturally and preserves the intended meaning.

Note: The static analysis hints flagging "gérer" as informal are false positives—this verb is perfectly appropriate in French technical documentation.

src/content/docs/fr/how-it-works/sdk.mdx (3)

6-9: LGTM! Badge addition is consistent and properly formatted.

The "Mis à jour" badge matches the pattern used in index.mdx and follows correct YAML formatting.


14-14: LGTM! Documentation updated to reflect Kysely-based client.

The description accurately reflects the shift from Astro DB to an internal Kysely-based database client, aligning with the broader API refactor documented in this PR.


52-70: LGTM! API surface changes properly documented.

The destructuring update accurately reflects the public API refactor:

  • Removed: db
  • Added: dbService, cache, AUTH, CONFIG, diffTracking, MIDDLEWARES, notificationSettings, PLUGINS, resetTokenBucket, UTIL

This documents a significant shift in the SDK's public interface.

src/content/docs/fr/how-it-works/restapi.mdx (1)

11-11: Translation maintains correct parity with English source.

The French restapi.mdx correctly lacks a badge because its English counterpart also does not have one. Both index.mdx and sdk.mdx (in English and French) consistently include the "Updated" / "Mis à jour" badge. The file structure is consistent with the source material.

Comment on lines +73 to +77
## Lectures complémentaires

<ReadMore>
Le SDK de StudioCMS est alimenté par le [paquet `@withstudiocms/sdk`](/fr/utils/kysely-and-sdk/#le-paquet-sdk), qui fournit des fonctionnalités et des utilitaires supplémentaires pour travailler avec StudioCMS. Pour plus d'informations, consultez la [documentation de Kysely et du SDK](/fr/utils/kysely-and-sdk/).
</ReadMore>
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Verify the referenced path exists in the French documentation

# Check if the Kysely and SDK documentation exists in French
echo "=== Searching for kysely-and-sdk in French docs ==="
fd -t f "kysely-and-sdk" src/content/docs/fr

echo -e "\n=== Checking utils directory structure ==="
fd -t d -d 3 "utils" src/content/docs/fr

echo -e "\n=== Checking if utils directory exists and listing contents ==="
if [ -d "src/content/docs/fr/utils" ]; then
  find src/content/docs/fr/utils -type f -name "*.mdx" | head -20
fi

Repository: withstudiocms/docs

Length of output: 289


🏁 Script executed:

#!/bin/bash
# Search for Kysely and SDK documentation in the entire docs structure

echo "=== Searching for any Kysely references ==="
fd -t f "kysely" src/content/docs

echo -e "\n=== Checking English utils directory ==="
fd -t f . src/content/docs/en/utils/ 2>/dev/null || echo "English utils directory contents check"

echo -e "\n=== Searching for SDK references in French docs ==="
rg "kysely\|sdk" src/content/docs/fr --type mdx -l

Repository: withstudiocms/docs

Length of output: 379


🏁 Script executed:

#!/bin/bash
# Let's also check if there's a pattern for French translations of English docs

echo "=== List all French utils docs ==="
ls -la src/content/docs/fr/utils/ 2>/dev/null || echo "Directory structure"

echo -e "\n=== List all English utils docs ==="
ls -la src/content/docs/en/utils/ 2>/dev/null || echo "Directory structure"

echo -e "\n=== Check if the file is referenced elsewhere ==="
rg "kysely-and-sdk" src/content/docs --no-text

Repository: withstudiocms/docs

Length of output: 1293


The referenced path /fr/utils/kysely-and-sdk/ does not exist and will result in broken links.

The French documentation at src/content/docs/fr/utils/ only contains rendering.mdx. The Kysely and SDK documentation currently exists only in English at /en/utils/kysely-and-sdk/. Either translate the documentation to French or link to the English version using /en/utils/kysely-and-sdk/ instead.

🤖 Prompt for AI Agents
In src/content/docs/fr/how-it-works/sdk.mdx around lines 73 to 77, the link
targets /fr/utils/kysely-and-sdk/ which does not exist and will create broken
links; update the link to point to the existing English page
/en/utils/kysely-and-sdk/ (or add/translate the corresponding French page under
src/content/docs/fr/utils/kysely-and-sdk.mdx and adjust the path accordingly)
and ensure the ReadMore block and any surrounding references use the corrected
path.

Copy link
Copy Markdown
Member

@dreyfus92 dreyfus92 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM ✅

@dreyfus92 dreyfus92 merged commit 6d04372 into withstudiocms:main Dec 20, 2025
8 of 9 checks passed
@ArmandPhilippot ArmandPhilippot deleted the i18n/fr-update-how-it-works branch December 20, 2025 18:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants