Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Пачка ftl и крошка фикса #168

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

percent-temp-percent
Copy link
Collaborator

@percent-temp-percent percent-temp-percent commented Mar 16, 2025

Максимальное количество одновременно открытых окон хранилищ увеличено с двух до 3-х.

Summary by CodeRabbit

  • New Features
    • Increased the storage window capacity, allowing an extra open window.
    • Added new localized chat emotes in Russian, enriching expression options.
    • Expanded bounty contracts with enhanced notifications and user feedback.
    • Introduced additional audio/visual customization options in the settings menu.
    • Launched creative new items in the uplink catalog for combat and deception.

Copy link

coderabbitai bot commented Mar 16, 2025

Walkthrough

This pull request updates a server-side configuration by increasing the maximum number of simultaneous storage windows from 2 to 3. It also adds multiple new Russian localization entries across various files for chat emotes, bounty contracts, escape menu options, and uplink catalog items. These changes adjust configuration values and expand localization support for enhanced UI text and in-game interactions.

Changes

File(s) Change Summary
Content.Shared/CCVar/CCVars.Interactions.cs Updated StorageLimit value from 2 to 3 with the same replication and server flags.
Resources/Locale/ru-RU/_Goobstation/emotes.ftl
Resources/Locale/ru-RU/_Impstation/chat/emotes.ftl
Added new Russian localization entries for chat emotes (e.g., names and messages for "spin", "jump", "bubble", "pop", "crack", and "reptilian hiss").
Resources/Locale/ru-RU/_NF/bounty-contracts/bounty-contracts.ftl Introduced several new keys for bounty contracts including announcements, UI feedback, and error messages.
Resources/Locale/ru-RU/escape-menu/ui/options-menu.ftl Added new entries for UI options covering sound settings, classic overlay, and speech bubble opacities.
Resources/Locale/ru-RU/store/uplink-catalog.ftl Added new uplink catalog items: “uplink-combat-bakery” and “uplink-fake-mindshield” with descriptive texts.

Suggested labels

size/M, YML

Poem

In a digital warren where updates thrive,
I hop with joy as new texts arrive,
Storage expands and emotes gleam,
Bounty alerts and options beam,
A rabbit’s delight in each code line’s art,
Hopping through changes with a jubilant heart!

Tip

⚡🧪 Multi-step agentic review comment chat (experimental)
  • We're introducing multi-step agentic chat in review comments. This experimental feature enhances review discussions with the CodeRabbit agentic chat by enabling advanced interactions, including the ability to create pull requests directly from comments.
    - To enable this feature, set early_access to true under in the settings.

📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1f76284 and ecb48b7.

📒 Files selected for processing (6)
  • Content.Shared/CCVar/CCVars.Interactions.cs (1 hunks)
  • Resources/Locale/ru-RU/_Goobstation/emotes.ftl (1 hunks)
  • Resources/Locale/ru-RU/_Impstation/chat/emotes.ftl (1 hunks)
  • Resources/Locale/ru-RU/_NF/bounty-contracts/bounty-contracts.ftl (4 hunks)
  • Resources/Locale/ru-RU/escape-menu/ui/options-menu.ftl (2 hunks)
  • Resources/Locale/ru-RU/store/uplink-catalog.ftl (1 hunks)
⏰ Context from checks skipped due to timeout of 90000ms (5)
  • GitHub Check: build (ubuntu-latest)
  • GitHub Check: build (ubuntu-latest)
  • GitHub Check: Test Packaging
  • GitHub Check: YAML map schema validator
  • GitHub Check: YAML Linter
🔇 Additional comments (9)
Content.Shared/CCVar/CCVars.Interactions.cs (1)

66-66: LGTM: Increased storage window limit successfully.

The change increases the maximum number of simultaneously open storage windows from 2 to 3 as stated in the PR objectives. This configuration remains server-controlled and properly replicated to clients.

Note: The inline comment "Frontier: 1<3" suggests a change from 1 to 3, while the PR description mentioned changing from 2 to 3. This minor discrepancy in documentation doesn't affect functionality.

Resources/Locale/ru-RU/_Goobstation/emotes.ftl (1)

1-5: Translations for emote names and messages look good.

The Russian translations for "spin" and "jump" emotes (both names and messages) are correct and consistent with the emote localization pattern.

Resources/Locale/ru-RU/_Impstation/chat/emotes.ftl (1)

1-10: Correctly organized emote localizations with proper formatting.

The localization entries for bubble, pop, crack, and reptilian hiss emotes are well structured with appropriate section comments and include the expected placeholder format for the crack emote's possessive adjective.

Resources/Locale/ru-RU/store/uplink-catalog.ftl (1)

303-306: New uplink item localizations follow existing pattern.

The new entries for combat bakery and fake mindshield items follow the established naming convention and translation style of the existing catalog items.

Resources/Locale/ru-RU/escape-menu/ui/options-menu.ftl (1)

264-267: New UI options properly localized.

The added options for the classic antagonist overlay and speech bubble opacity settings are correctly translated and maintain consistency with the existing UI option format.

Resources/Locale/ru-RU/_NF/bounty-contracts/bounty-contracts.ftl (4)

3-3: LGTM: Good addition for cases without job titles

The new localization entry properly handles cases when displaying an author's name without their job title, complementing the existing entry for authors with jobs.


19-27: LGTM: Well-structured localization entries for announcements

These new entries for command/public collections and various announcement types follow proper FTL syntax with appropriate placeholders for dynamic content. The translations are natural and consistent with the existing style.


39-40: LGTM: UI feedback messages added

Good addition of UI feedback messages for loading states and unknown authors, improving the user experience through proper localized interface elements.


59-61: LGTM: Error validation messages added

These error messages for input validation follow the same format as existing error messages and provide clear feedback to users when input fields exceed length limits.

✨ Finishing Touches
  • 📝 Generate Docstrings

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
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Hello @percent-temp-percent, I'm Gemini Code Assist1! I'm currently reviewing this pull request and will post my feedback shortly. In the meantime, here's a summary to help you and other reviewers quickly get up to speed!

Summary of Changes

This pull request primarily focuses on Russian localization updates and a minor gameplay tweak. Specifically, it increases the maximum number of simultaneously opened storage windows from two to three and introduces/updates several Russian translations for emotes, bounty contracts, options menu items, and uplink store items.

Highlights

  • Storage Limit Increase: The maximum number of simultaneously opened storage windows has been increased from 2 to 3, controlled by the control.storage_limit CVar.
  • New Emotes: Adds new emotes (spin, jump, bubble, pop, crack, reptilian-hiss) with corresponding names and messages in Russian.
  • Bounty Contract Localization: Expands Russian localization for bounty contract related texts, including author information, radio announcements, UI elements, and error messages.
  • Options Menu Localization: Adds Russian localization for new options menu items, including AHelp sound and speech bubble opacity settings.
  • Uplink Store Localization: Adds Russian localization for new uplink store items, including 'Combat Bakery' and 'Fake Mindshield'.

Changelog

Click here to see the changelog
  • Content.Shared/CCVar/CCVars.Interactions.cs
    • Increased the StorageLimit CVar default value from 2 to 3.
  • Resources/Locale/ru-RU/_Goobstation/emotes.ftl
    • Added localization for 'spin' and 'jump' emotes.
  • Resources/Locale/ru-RU/_Impstation/chat/emotes.ftl
    • Added localization for 'bubble', 'pop', 'crack', and 'reptilian-hiss' emotes.
  • Resources/Locale/ru-RU/_NF/bounty-contracts/bounty-contracts.ftl
    • Added localization for bounty contract author without a job.
    • Added localization for bounty contract announcements and collection names.
    • Added localization for bounty contract UI elements (loading, unknown author).
    • Added localization for bounty contract creation error messages (name, vessel, description too long).
  • Resources/Locale/ru-RU/escape-menu/ui/options-menu.ftl
    • Added localization for AHelp sound option.
    • Added localization for classic antagonist overlay and speech bubble opacity settings.
  • Resources/Locale/ru-RU/store/uplink-catalog.ftl
    • Added localization for 'Combat Bakery' and 'Fake Mindshield' uplink items.
Using Gemini Code Assist

The full guide for Gemini Code Assist can be found on our documentation page, here are some quick tips.

Invoking Gemini

You can request assistance from Gemini at any point in your pull request via creating an issue comment (i.e. comment on the pull request page) using either /gemini <command> or @gemini-code-assist <command>. Below is a summary of the supported commands.

Feature Command Description
Code Review /gemini review Performs a code review for the current pull request in its current state.
Pull Request Summary /gemini summary Provides a summary of the current pull request in its current state.
Comment @gemini-code-assist Responds in comments when explicitly tagged, both in issue comments and review comments.
Help /gemini help Displays a list of available commands.

Customization

To customize Gemini Code Assist for GitHub experience, repository maintainers can create a configuration file and/or provide a custom code review style guide (such as PEP-8 for Python) by creating and adding files to a .gemini/ folder in the base of the repository. Detailed instructions can be found here.

Limitations & Feedback

Gemini Code Assist is currently in preview and may make mistakes. Please leave feedback on any instances where its feedback is incorrect or counter productive. You can react with 👍 and 👎 on @gemini-code-assist comments to provide feedback.

You can also get AI-powered code generation, chat, as well as code reviews directly in the IDE at no cost with the Gemini Code Assist IDE Extension.


Did you know?

The Cyrillic alphabet, used for the Russian language, was developed in the First Bulgarian Empire during the 9th-10th centuries AD, based on the Greek uncial script.

Footnotes

  1. Review the Privacy Notices, Generative AI Prohibited Use Policy, Terms of Service, and learn how to configure Gemini Code Assist in GitHub here. Gemini can make mistakes, so double check it and use code with caution.

Copy link

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

The pull request includes an increase in the storage limit and several localization updates. The changes seem straightforward, but it's important to ensure that the increased storage limit doesn't negatively impact performance or game balance. The localization updates should also be verified for accuracy and consistency.

Summary of Findings

  • Increased Storage Limit: The storage limit has been increased from 2 to 3. Ensure this change is thoroughly tested and doesn't introduce any unexpected issues with the game's balance or performance.
  • Localization Updates: Several new localization strings have been added for emotes, bounty contracts, and UI elements. Verify that these translations are accurate and fit within the game's context.

Merge Readiness

The pull request appears to be in good shape for merging, but it's recommended to thoroughly test the increased storage limit and verify the accuracy of the localization updates before merging. I am unable to approve the pull request in any circumstance, and users should have others review and approve this code before merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant