Skip to content

fix(config): use zoo-code-settings.json for settings export/import - #1177

Open
Rafael-Silva-Oliveira wants to merge 1 commit into
Zoo-Code-Org:mainfrom
Rafael-Silva-Oliveira:fix-roo-export-name
Open

fix(config): use zoo-code-settings.json for settings export/import#1177
Rafael-Silva-Oliveira wants to merge 1 commit into
Zoo-Code-Org:mainfrom
Rafael-Silva-Oliveira:fix-roo-export-name

Conversation

@Rafael-Silva-Oliveira

@Rafael-Silva-Oliveira Rafael-Silva-Oliveira commented Aug 6, 2026

Copy link
Copy Markdown

Related GitHub Issue

Closes: #1176

Description

Renames the default settings export/import filename from roo-code-settings.json to zoo-code-settings.json.

Three places:

  • exportSettings and importSettings in src/core/config/importExport.ts — the filename passed to resolveDefaultSaveUri.
  • The autoImportSettingsPath example path in all 18 package.nls.*.json files.
  • The corresponding assertions in importExport.spec.ts.

This only changes the name the save/open dialog pre-fills. Nothing reads or matches on the filename, so existing exports keep importing fine regardless of what they're called.

Test Procedure

npx vitest run core/config/__tests__/importExport.spec.ts — 49/49 pass.

Manually: Settings → About Zoo Code → Manage Settings → Export, and the save dialog now suggests zoo-code-settings.json. Import shows the same default. Re-importing a file still named roo-code-settings.json works as before.

Pre-Submission Checklist

  • Issue Linked: This PR is linked to an approved GitHub Issue (see "Related GitHub Issue" above).
  • Scope: My changes are focused on the linked issue (one major feature/fix per PR).
  • Self-Review: I have performed a thorough self-review of my code.
  • Testing: New and/or updated tests have been added to cover my changes (if applicable).
  • Visual Snapshot (UI changes only): No webview UI changes — the filename comes from a native VS Code dialog.
  • Documentation Impact: I have considered if my changes require documentation updates (see "Documentation Updates" section below).
  • Contribution Guidelines: I have read and agree to the Contributor Guidelines.

Documentation Updates

None needed — no documented behavior changes, only the suggested default filename.

Summary by CodeRabbit

  • Bug Fixes

    • Updated the default settings import and export filename to zoo-code-settings.json.
    • Corrected the default Downloads location and related file references.
  • Documentation

    • Updated automatic settings import descriptions across all supported languages to use the new filename.
  • Tests

    • Updated import/export coverage to reflect the revised default filename.

The settings export and import dialogs still defaulted to the legacy
"roo-code-settings.json" filename, as did the autoImportSettingsPath
example in the localized setting descriptions. Rename the default to
"zoo-code-settings.json" to match the Zoo Code branding.

This only changes the default filename suggested in the save/open
dialogs; existing files with any name can still be imported.
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 2f7d5547-7663-4be9-8cfc-c53d297a0966

📥 Commits

Reviewing files that changed from the base of the PR and between f149073 and 302c163.

📒 Files selected for processing (20)
  • src/core/config/__tests__/importExport.spec.ts
  • src/core/config/importExport.ts
  • src/package.nls.ca.json
  • src/package.nls.de.json
  • src/package.nls.es.json
  • src/package.nls.fr.json
  • src/package.nls.hi.json
  • src/package.nls.id.json
  • src/package.nls.it.json
  • src/package.nls.ja.json
  • src/package.nls.json
  • src/package.nls.ko.json
  • src/package.nls.nl.json
  • src/package.nls.pl.json
  • src/package.nls.pt-BR.json
  • src/package.nls.ru.json
  • src/package.nls.tr.json
  • src/package.nls.vi.json
  • src/package.nls.zh-CN.json
  • src/package.nls.zh-TW.json

📝 Walkthrough

Walkthrough

The import and export dialogs now suggest zoo-code-settings.json. Export tests and automatic-import descriptions in all supported locales use the same filename.

Changes

Settings filename update

Layer / File(s) Summary
Import/export defaults and validation
src/core/config/importExport.ts, src/core/config/__tests__/importExport.spec.ts
Import and export defaults, fixtures, and expectations now use zoo-code-settings.json.
Localized filename descriptions
src/package.nls*.json
Automatic settings import descriptions now reference zoo-code-settings.json.

Estimated code review effort: 1 (Trivial) | ~5 minutes

Possibly related PRs

Suggested labels: awaiting-review

Suggested reviewers: edelauna

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the filename change for settings import and export.
Description check ✅ Passed The description includes the linked issue, implementation details, test results, manual verification, and checklist status.
Linked Issues check ✅ Passed The changes update both default filenames, localized example paths, and tests while preserving imports of existing files, satisfying issue #1176.
Out of Scope Changes check ✅ Passed All changes directly support the filename rename in issue #1176, with no unrelated code or behavior changes.
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

Warning

There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure.

🔧 ESLint

If the error stems from missing dependencies, add them to the package.json file. For unrecoverable errors (e.g., due to private dependencies), disable the tool in the CodeRabbit configuration.

src/core/config/__tests__/importExport.spec.ts

ESLint skipped: missing config or dependency (missing-dependency). The ESLint configuration references a package that is not available in the sandbox.

src/core/config/importExport.ts

ESLint skipped: the ESLint configuration for this file references a package that is not available in the sandbox.


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.

@github-actions github-actions Bot added the awaiting-review PR changes are ready and waiting for maintainer re-review label Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR changes are ready and waiting for maintainer re-review

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Settings export defaults to old roo-code-settings.json filename

1 participant