fix: small review fixes

Author: astandrik

.github/copilot-instructions.md

@@ -1,3 +1,11 @@
+# Copilot Instructions
+
+## Internationalization (i18n)
+
+- Do NOT hardcode user-facing strings.
+- ALWAYS use the component i18n keysets.
+- For key naming, see `i18n-naming-ruleset.md` in the repo root.
+
 # GitHub Copilot Instructions for YDB Embedded UI
 
 > **Note**: This file contains project-specific instructions for GitHub Copilot code review and assistance.

AGENTS.md

@@ -232,21 +232,7 @@ Uses BEM naming convention with `cn()` utility from `utils/cn`. Create a block f
 
 ### Internationalization (i18n)
 
-All user-facing text must be internationalized using the i18n system. Follow the naming rules from `i18n-naming-ruleset.md`:
-
-- **Component Structure**: Each component has an `i18n/` folder with `en.json` and `index.ts`
-- **Registration**: Use `registerKeysets()` with a unique component name
-- **Key Format**: Follow `<context>_<content>` pattern (e.g., `action_save`, `field_name`, `alert_error`)
-- **Context Prefixes**:
-  - `action_` - buttons, links, menu items
-  - `field_` - form fields, table columns
-  - `title_` - page/section titles
-  - `alert_` - notifications, errors
-  - `context_` - descriptions, hints
-  - `confirm_` - confirmation dialogs
-  - `value_` - status values, options
-- **NEVER** use hardcoded strings in UI components
-- **ALWAYS** create i18n entries for all user-visible text
+See `i18n-naming-ruleset.md` in the repo root for all i18n conventions (naming and usage).
 
 ### Performance Considerations
 

CLAUDE.md

@@ -232,21 +232,7 @@ Uses BEM naming convention with `cn()` utility from `utils/cn`. Create a block f
 
 ### Internationalization (i18n)
 
-All user-facing text must be internationalized using the i18n system. Follow the naming rules from `i18n-naming-ruleset.md`:
-
-- **Component Structure**: Each component has an `i18n/` folder with `en.json` and `index.ts`
-- **Registration**: Use `registerKeysets()` with a unique component name
-- **Key Format**: Follow `<context>_<content>` pattern (e.g., `action_save`, `field_name`, `alert_error`)
-- **Context Prefixes**:
-  - `action_` - buttons, links, menu items
-  - `field_` - form fields, table columns
-  - `title_` - page/section titles
-  - `alert_` - notifications, errors
-  - `context_` - descriptions, hints
-  - `confirm_` - confirmation dialogs
-  - `value_` - status values, options
-- **NEVER** use hardcoded strings in UI components
-- **ALWAYS** create i18n entries for all user-visible text
+See `i18n-naming-ruleset.md` in the repo root for all i18n conventions (naming and usage).
 
 ### Performance Considerations
 

src/components/nodesColumns/columns.tsx

@@ -114,7 +114,7 @@ export function getRackColumn<T extends {Rack?: string}>(): Column<T> {
 export function getPileNameColumn<T extends {PileName?: string}>(): Column<T> {
     return {
         name: NODES_COLUMNS_IDS.PileName,
-        header: i18n('PileName'),
+        header: i18n('field_pile-name'),
         align: DataTable.LEFT,
         render: ({row}) => row.PileName || EMPTY_DATA_PLACEHOLDER,
         width: 100,

src/components/nodesColumns/constants.ts

@@ -132,7 +132,7 @@ export const NODES_COLUMNS_TITLES = {
         return i18n('pdisks');
     },
     get PileName() {
-        return i18n('PileName');
+        return i18n('field_pile-name');
     },
 } as const satisfies Record<NodesColumnId, string>;
 
@@ -183,7 +183,7 @@ const NODES_COLUMNS_GROUP_BY_TITLES = {
         return i18n('ping-time');
     },
     get PileName() {
-        return i18n('PileName');
+        return i18n('field_pile-name');
     },
 } as const satisfies Record<NodesGroupByField, string>;
 

src/components/nodesColumns/i18n/en.json

@@ -20,7 +20,7 @@
   "pdisks": "PDisks",
   "field_memory-used": "Memory used",
   "field_memory-limit": "Memory limit",
-  "PileName": "Pile Name",
+  "field_pile-name": "Pile Name",
   "system-state": "System State",
   "connect-status": "Connect Status",
   "utilization": "Utilization",

src/containers/Cluster/ClusterInfo/ClusterInfo.scss

@@ -21,11 +21,6 @@
         margin-left: 5px;
     }
 
-    &__details-layout {
-        align-items: flex-start;
-        gap: var(--g-spacing-6);
-    }
-
     &__bridge-table {
         flex: 0 0 360px; // do not shrink, fixed basis so stats don't take all space
 

src/containers/Cluster/ClusterInfo/ClusterInfo.tsx

@@ -20,8 +20,6 @@ import {getInfo, getStorageGroupStats} from './utils/utils';
 
 import './ClusterInfo.scss';
 
-const GROUPS_SECTION_GAP = 10;
-
 interface ClusterInfoProps {
     cluster?: TClusterInfo;
     loading?: boolean;
@@ -101,7 +99,7 @@ export const ClusterInfo = ({
         }
         return (
             <InfoSection>
-                <Flex gap={GROUPS_SECTION_GAP} width="full">
+                <Flex gap={10} width="full">
                     <Flex direction="column" gap={2}>
                         <Text as="div" variant="subheader-2" className={b('section-title')}>
                             {i18n('title_storage-groups')}{' '}

src/containers/Cluster/ClusterOverview/components/BridgeInfoTable.tsx

@@ -19,26 +19,26 @@ export function BridgeInfoTable({piles}: BridgeInfoTableProps) {
         () => [
             {
                 name: 'Name',
-                header: i18n('label_name'),
+                header: i18n('field_name'),
                 width: 160,
                 align: DataTable.LEFT,
             },
             {
                 name: 'IsPrimary',
-                header: i18n('label_primary'),
+                header: i18n('field_primary'),
                 width: 110,
                 align: DataTable.LEFT,
                 render: ({row}) => (row.IsPrimary ? i18n('value_yes') : i18n('value_no')),
             },
             {
                 name: 'State',
-                header: i18n('label_state'),
+                header: i18n('field_state'),
                 width: 160,
                 align: DataTable.LEFT,
             },
             {
                 name: 'Nodes',
-                header: i18n('label_nodes'),
+                header: i18n('field_nodes'),
                 width: 100,
                 align: DataTable.RIGHT,
                 render: ({row}) =>

src/containers/Cluster/i18n/en.json

@@ -20,10 +20,10 @@
   "title_bridge": "Bridge Piles",
   "label_overview": "Overview",
   "label_load": "Load",
-  "label_name": "Name",
-  "label_primary": "Primary",
-  "label_state": "State",
-  "label_nodes": "Nodes",
+  "field_name": "Name",
+  "field_primary": "Primary",
+  "field_state": "State",
+  "field_nodes": "Nodes",
   "value_yes": "Yes",
   "value_no": "No",
   "context_of": "of",