Skip to content

i18n(fr): update package-catalog#187

Merged
dreyfus92 merged 3 commits into
withstudiocms:mainfrom
ArmandPhilippot:i18n/fr-update-package-catalog
Dec 21, 2025
Merged

i18n(fr): update package-catalog#187
dreyfus92 merged 3 commits into
withstudiocms:mainfrom
ArmandPhilippot:i18n/fr-update-package-catalog

Conversation

@ArmandPhilippot
Copy link
Copy Markdown
Member

@ArmandPhilippot ArmandPhilippot commented Dec 21, 2025

Description

Adds changes from #177 to the French translations in package-catalog.

Summary by CodeRabbit

  • Documentation
    • Removed French Web Vitals integration documentation
    • Updated StudioCMS DevApps configuration examples with simplified code snippets
    • Revised integration import statements in setup guides
    • Removed libSQL viewer tool references from documentation

✏️ 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 21, 2025 14:42
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Dec 21, 2025

📝 Walkthrough

Walkthrough

This PR deletes French Web Vitals documentation and updates the StudioCMS devApps plugin documentation to replace @astrojs/db references with studiocms imports, remove libSQL tooling sections, and enable Steps wrappers in code examples.

Changes

Cohort / File(s) Change Summary
Deletion of legacy documentation
src/content/docs/fr/package-catalog/community-plugins/web-vitals.mdx
Removed entire French Web Vitals integration documentation including front matter (i18nReady, title, type, catalogEntry, description, sidebar) and MDX content covering Présentation, Installation (two methods), and Gallery sections.
StudioCMS devApps plugin documentation updates
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx
Enabled Steps wrappers in installation section by uncommenting tags. Removed twoslash directive from code fence. Replaced all @astrojs/db imports and db() function calls with studiocms imports and studiocms() calls across multiple code examples. Removed libSQLViewer configuration option and eliminated entire Visionneuse libSQL tooling section from documentation.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Documentation-only changes with straightforward import/configuration replacements
  • Pattern repetition across multiple code examples (db → studiocms) reduces cognitive load
  • Deletion of one document and structural cleanup of another

Possibly related PRs

Suggested reviewers

  • Adammatthiesen

Poem

🐰 A web-vital farewell, swept to the past,
As studiocms shines, strong and steadfast!
LibSQL tools fade, new steps take the stage,
French docs rewritten on a cleaner page.

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 directly describes the main change: updating French translation files in the package-catalog section, as confirmed by the file modifications (removing one file and updating another).
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 21, 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/package-catalog/community-plugins/web-vitals.mdx Localization removed, will be marked as missing.
fr/package-catalog/studiocms-plugins/studiocms-devapps.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: 0

🧹 Nitpick comments (1)
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx (1)

29-29: Optional: Consider alternative wording for stylistic precision.

The static analysis tool suggests that "Mettez à jour" could be replaced with a potentially more precise alternative. While the current phrasing is correct and clear, you may want to consider using "Modifiez" or another variant if you feel it better fits the context.

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between a6a49e6 and ddc5b50.

📒 Files selected for processing (2)
  • src/content/docs/fr/package-catalog/community-plugins/web-vitals.mdx (0 hunks)
  • src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx (2 hunks)
💤 Files with no reviewable changes (1)
  • src/content/docs/fr/package-catalog/community-plugins/web-vitals.mdx
🧰 Additional context used
🧠 Learnings (3)
📓 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: 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: 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: 185
File: src/content/docs/fr/utils/template-lang.mdx:1-187
Timestamp: 2025-12-21T14:44:48.926Z
Learning: In the withstudiocms/docs repository, when translating documentation (particularly French translations), variable names in code examples can and should be localized to the target language (e.g., `utilisateur` instead of `user`, `commande` instead of `order`) to improve readability and understanding for users of that language. This applies to pedagogical documentation examples, not production code.
📚 Learning: 2025-07-24T11:57:15.225Z
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.

Applied to files:

  • src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx
📚 Learning: 2025-12-21T14:44:48.926Z
Learnt from: ArmandPhilippot
Repo: withstudiocms/docs PR: 185
File: src/content/docs/fr/utils/template-lang.mdx:1-187
Timestamp: 2025-12-21T14:44:48.926Z
Learning: In French documentation under withstudiocms/docs, localize code example identifiers to the target language for pedagogical content (e.g., use 'utilisateur' instead of 'user', 'commande' instead of 'order') to improve readability. Do this only in documentation/examples meant for learning, and do not apply to production source code. Apply this guideline consistently across French docs.

Applied to files:

  • src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx
🪛 LanguageTool
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx

[style] ~29-~29: Un autre mot peut sembler plus précis et percutant.
Context: ...o" args="add @studiocms/devapps" /> 2. Mettez à jour votre fichier de configuration comme in...

(METTRE_A_JOUR)

⏰ 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: Run Starlight Link Check
  • GitHub Check: build-and-push
🔇 Additional comments (2)
src/content/docs/fr/package-catalog/studiocms-plugins/studiocms-devapps.mdx (2)

24-24: LGTM! Steps wrappers properly enabled.

The Steps component wrappers have been correctly uncommented in both installation methods, improving the documentation structure and user experience.

Also applies to: 30-30, 33-33, 53-53


40-52: No action needed. The French translation at lines 40-52 already matches the English source with correct localization ("REQUIS" for "REQUIRED"). The twoslash directives remain in both versions at lines 63 and 81; they are not present in the section under review. The architectural changes from studiocms are synchronized across both language versions.

Likely an incorrect or invalid review comment.

@dreyfus92 dreyfus92 merged commit 4e9b3d6 into withstudiocms:main Dec 21, 2025
7 of 9 checks passed
@ArmandPhilippot ArmandPhilippot deleted the i18n/fr-update-package-catalog branch December 21, 2025 20:19
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