Fix translation issue

[Jack251970]

Use the number for translation:

/// <remarks>
/// e.g.: <code>
/// 
/// Flow detected you have installed {0} plugins, which will require {1} to run. Would you like to download {1}?
/// {2}{2}
/// Click no if it's already installed, and you will be prompted to select the folder that contains the {1} executable
/// 
/// </code>
/// </remarks>
public static string runtimePluginInstalledChooseRuntimePrompt(object? arg0, object? arg1, object? arg2) => string.Format(Flow.Launcher.Core.PublicApi.Instance.GetTranslation("runtimePluginInstalledChooseRuntimePrompt"), arg0, arg1, arg2);

Resolve #4255

---

Summary by cubic

Fixes the runtime prompt translation and improves setup by counting how many plugins use each language, then showing that count in the message. Skips setup when none need the environment.

• Bug Fixes
  • Count plugins per language; return [] if none.
  • Update localized prompt to use numeric placeholders and include plugin count.
  • Use [] for empty returns; change SetPathForPluginPairs to return List.

^{Written for commit 517293a. Summary will update on new commits.}

[gitstream-cm]

🥷 Code experts: no user but you matched threshold 10

Jack251970 has most 👩‍💻 activity in the files.
Jack251970 has most 🧠 knowledge in the files.

See details

Flow.Launcher.Core/ExternalPlugins/Environments/AbstractPluginEnvironment.cs

Activity based on git-commit:

	Jack251970
FEB
JAN
DEC
NOV
OCT
SEP	17 additions & 28 deletions

Knowledge based on git-blame:
Jack251970: 100%

✨ Comment /gs review for LinearB AI review. Learn how to automate it here.

[gitstream-cm]

Be a legend 🏆 by adding a before and after screenshot of the changes you made, especially if they are around UI/UX.

[cubic-dev-ai]

No issues found across 1 file

[coderabbitai]

📝 Walkthrough

Walkthrough

This change refactors AbstractPluginEnvironment.cs to use a plugin count heuristic instead of language existence checks, standardizes empty collection returns, and narrows the SetPathForPluginPairs method signature from IEnumerable<PluginPair> to List<PluginPair>, updating all call sites accordingly.

Changes

Cohort / File(s)	Summary
Plugin Environment Refactoring Flow.Launcher.Core/ExternalPlugins/Environments/AbstractPluginEnvironment.cs	Replaced language existence check with plugin count heuristic; updated user prompts to pass plugin count argument; changed empty-return patterns to consistent empty arrays/lists; narrowed SetPathForPluginPairs return type from IEnumerable<PluginPair> to List<PluginPair> with corresponding call-site adjustments.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~12 minutes

Possibly related PRs

• Localize startup messages about missing Python/NodeJS #2769: Localizes the startup messages in AbstractPluginEnvironment.cs that this PR modifies for prompt construction and return types.

Suggested reviewers

• jjw24
• taooceros
• Yusyuriv

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (1 warning, 1 inconclusive)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.
Title check	❓ Inconclusive	The title 'Fix translation issue' is vague and generic; while it relates to the changeset, it does not clearly describe the specific improvement being made (counting plugins per language and updating prompt formatting).	Provide a more specific title such as 'Count plugins per language and fix translation placeholders' to better convey the primary changes.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Linked Issues check	✅ Passed	The PR addresses issue #4255 by implementing numeric placeholders in the translation string and counting plugins per language, which resolves the Chinese translation mistake where counts were incorrectly formatted.
Out of Scope Changes check	✅ Passed	All changes in the PR are directly related to fixing the translation issue and improving the plugin environment setup flow; no out-of-scope modifications were detected.
Description check	✅ Passed	The PR description is directly related to the changeset, specifying a translation improvement using numeric placeholders in localized strings.

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

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment
• Commit unit tests in branch environment_localization

---

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

[Copilot]

Pull request overview

This PR fixes a translation bug where the Language property (e.g., "Python") was being incorrectly passed as the plugin count parameter to a translation string, resulting in incorrect messages like "Python个插件" in Chinese translations. The fix calculates the actual plugin count and passes it to the translation function instead.

Changes:

• Calculate plugin count before checking if any plugins use the language
• Pass plugin count instead of language name to the translation function
• Update return statements to use modern collection expression syntax

---

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

[jjw24]

The issue is with translation I believe.

English version:
'Flow detected you have installed {Python/TypeScript} plugins, which will require {Python/Node.js} to run. Would you like to download {Python/Node.js}?'

Changing to count requires update to the message to read better:
'Flow detected you have installed {5} plugins that will require {Python/Node.js} to run. Would you like to download {Python/Node.js}?'

[Jack251970]

The issue is with translation I believe.

English version: 'Flow detected you have installed {Python/TypeScript} plugins, which will require {Python/Node.js} to run. Would you like to download {Python/Node.js}?'

Changing to count requires update to the message to read better: 'Flow detected you have installed {5} plugins that will require {Python/Node.js} to run. Would you like to download {Python/Node.js}?'

Oh, yeah. I will fix this translation in Crowdin.