Localize debugger labels and show enabled breakpoint counts (#101)

* Clarify breakpoint counts and localize debugger labels

* fix: retain readable limit errors and focus native label coverage

---------

Co-authored-by: Claude Opus 5.5 <noreply@anthropic.com>

Author: nutyworks

docs/verification/features/breakpoints.md

@@ -4,7 +4,7 @@
 
 Deleting from the condition modal waits for server acknowledgement before closing
 back to its existing parent (including Source). Pending, rejected and unavailable
-requests remain visible in the same modal. The Active breakpoint list now only
+requests remain visible in the same modal. The breakpoint list now only
 navigates: its row overflow, toggle, condition, delete and Undo actions are removed.
 Function entries open their original source line and matching stage; block entries
 open an exact matching recorded/static stage in the current pause's Flow. Matching
@@ -62,8 +62,16 @@ appear on hover/focus; disabled markers also appear only on hover/focus, while
 enabled breakpoints remain visible. Flow keeps both saved disabled markers and unset breakpoint-capable stage affordances
 visible as neutral hollow circles/diamonds, including run and terminal function stages.
 Rendering an unset affordance does not create a saved breakpoint. The
-list retains enabled and disabled definitions with an explicit state label. The toolbar count and source stage summaries
-still count enabled breakpoints. Disabling preserves the saved condition; use the
+list retains enabled and disabled definitions with an explicit state label. Its header
+shows the total definitions and enabled count. The toolbar's short count and source
+stage summaries still count enabled breakpoints; the toolbar tooltip explicitly shows
+enabled and total counts. `CodonLabelsTest` checks enabled-only, disabled-only, mixed
+and filtered captions in English/Korean, alongside all Watch-kind prefix permutations.
+`CodonLabelsGameTest` retains one mixed snapshot per language at the 320×240 logical
+viewport, including the maximum-count header width, representative Watch Details
+and rotation. This produces eight screenshots rather than repeating every permutation.
+Limit failures retain client translation keys with a readable English fallback for
+console/RCON sources without client language assets. Disabling preserves the saved condition; use the
 list to navigate to its source, then hover/focus its original marker to enable it again. The whole-command
 marker at the front of the command-block editor is always visible, including when
 unused or disabled. Text selection in the
@@ -110,7 +118,7 @@ requirement, the loaded command-block path, `list` output and `/help` usage, wit
 test-registered `execute` tree instead of vanilla's. It does not execute a
 native loaded function or prove execution beyond the Source response limits.
 
-The active list retains widget identity only for currently rendered controls.
+The breakpoint list retains widget identity only for currently rendered controls.
 Replacement authoritative snapshots, scrolling and empty/disconnected lists release
 obsolete labels, tooltips and actions; unchanged visible targets retain widget identity
 and focus restoration. `BreakpointListCacheTest` exercises 100 acknowledged replacement

docs/verification/features/watches-and-nbt.md

@@ -17,6 +17,12 @@ edge, with no inline management controls. `WatchPanelLayoutTest` covers the same
 name/value layout at narrow and wide widths; `DebuggerCompactWatchGameTest`
 exercises the actual screen events and Details actions.
 
+Watch Details uses localized kind prefixes and literal objective/storage/path identifiers.
+`CodonLabelsTest` checks EN/KO key and placeholder parity and all three Watch kinds,
+including `Health` and `Pos[0]`. `CodonLabelsGameTest` captures their actual English/Korean
+Details rendering and the localized yaw/pitch labels at normal and 320×240 logical
+viewports. These use presentation fixtures and do not establish server read execution.
+
 ## User path and expected result
 
 Regular Watches default to 280 logical pixels (previously 360), bounded by the same

src/client/java/works/nuty/codon/client/ui/BreakpointListScreen.java

@@ -198,13 +198,19 @@ private void navigate(BreakpointTarget target) {
         graphics.fill(0, 0, width, height, DebuggerTheme.modalColor(0x70000000));
         graphics.fill(left, top, left + panelWidth, top + panelHeight, DebuggerTheme.modalColor(PANEL));
         graphics.outline(left, top, panelWidth, panelHeight, DebuggerTheme.modalColor(BORDER));
-        WatchUi.line(graphics, font, tr(targets == null ? "codon.breakpoint.list_header" : "codon.breakpoint.saved_definitions_header", displayed.size()),
+        WatchUi.line(graphics, font, countHeader(),
             left + 8, top + 10, panelWidth - 16, TEXT);
         if (displayed.isEmpty()) WatchUi.line(graphics, font, tr("codon.breakpoint.list_empty"), left + 12, top + 43,
             panelWidth - 24, MUTED);
         super.extractRenderState(graphics, mouseX, mouseY, partialTick);
     }
 
+    private String countHeader() {
+        if (targets != null) return tr("codon.breakpoint.saved_definitions_header", displayed.size());
+        long enabled = displayed.stream().filter(BreakpointDefinition::enabled).count();
+        return tr("codon.breakpoint.list_header", displayed.size(), enabled);
+    }
+
     @Override public boolean mouseScrolled(double x, double y, double scrollX, double scrollY) {
         if (x < left || x >= left + panelWidth || y < top + 30 || y >= top + 30 + rows * ROW_HEIGHT
             || scrollY == 0) return super.mouseScrolled(x, y, scrollX, scrollY);

src/client/java/works/nuty/codon/client/ui/DebuggerOverlay.java

@@ -386,7 +386,7 @@ private void renderHeader(GuiGraphicsExtractor graphics, DebuggerLayout layout,
             () -> { if (client.gui.screen() != null) client.gui.setScreen(new BreakpointListScreen(client.gui.screen(), state)); })
             .withTextIcon(DebuggerIcon.BREAKPOINT_LIST)
             .setTooltip(Tooltip.create(Component.translatable("codon.breakpoint.toolbar",
-                breakpointCount)));
+                breakpointCount, state.breakpoints().definitions().size())));
 
     }
 
@@ -731,8 +731,9 @@ private void renderSourceDetails(GuiGraphicsExtractor graphics, Bounds area, int
         }
         y = sourceValueRows(graphics, area, y, SourceDetailsFormatting.position(source),
             SourceDetailsFormatting.previousPosition(before, source));
-        sourceValueRows(graphics, area, y, SourceDetailsFormatting.rotation(source),
-            SourceDetailsFormatting.previousRotation(before, source));
+        String yaw = tr("codon.ui.rotation.yaw"), pitch = tr("codon.ui.rotation.pitch");
+        sourceValueRows(graphics, area, y, SourceDetailsFormatting.rotation(source, yaw, pitch),
+            SourceDetailsFormatting.previousRotation(before, source, yaw, pitch));
     }
 
     private int sourceValueRows(GuiGraphicsExtractor graphics, Bounds area, int y,

src/client/java/works/nuty/codon/client/ui/SourceDetailsFormatting.java

@@ -14,8 +14,8 @@ public static String position(PauseSource source) {
             source.anchor().x(), source.anchor().y(), source.anchor().z());
     }
 
-    public static String rotation(PauseSource source) {
-        return String.format(Locale.ROOT, "yaw %.1f° / pitch %.1f°", source.yaw(), source.pitch());
+    public static String rotation(PauseSource source, String yaw, String pitch) {
+        return String.format(Locale.ROOT, "%s %.1f° / %s %.1f°", yaw, source.yaw(), pitch, source.pitch());
     }
 
     public static String previousPosition(@Nullable PauseSource before, PauseSource after) {
@@ -25,10 +25,10 @@ public static String previousPosition(@Nullable PauseSource before, PauseSource
             new String[] { "X", "Y", "Z" }, "%.2f");
     }
 
-    public static String previousRotation(@Nullable PauseSource before, PauseSource after) {
+    public static String previousRotation(@Nullable PauseSource before, PauseSource after, String yaw, String pitch) {
         if (before == null) return "";
         return previous(new double[] { before.yaw(), before.pitch() },
-            new double[] { after.yaw(), after.pitch() }, new String[] { "yaw", "pitch" }, "%.1f°");
+            new double[] { after.yaw(), after.pitch() }, new String[] { yaw, pitch }, "%.1f°");
     }
 
     private static String previous(double[] oldValues, double[] newValues, String[] labels, String format) {

src/client/java/works/nuty/codon/client/ui/WatchDetailsScreen.java

@@ -127,7 +127,7 @@ private void rebuildLines() {
         retry.active = !entry.automatic() && !state.watchReadsFailed() && state.watches().canRetry(entryId);
         edit.active = expanded;
         add(WatchUi.text("details.expression"), TEAL);
-        add(Component.literal(expression(entry.spec())), TEXT);
+        add(expression(entry.spec()), TEXT);
         blank();
         add(WatchUi.text("details.binding"), TEAL);
         if (entry.spec().kind() == WatchSpec.Kind.STORAGE_NBT) add(WatchUi.text("details.storage"), TEXT);
@@ -151,12 +151,14 @@ private String fixedBinding(ClientWatchState.Entry entry) {
         return WatchUi.text("details.fixed", name + entry.spec().executor()).getString();
     }
 
-    private static String expression(WatchSpec spec) {
-        return switch (spec.kind()) {
-            case SCORE -> "score: " + spec.target();
-            case ENTITY_NBT -> "entity NBT: " + spec.path();
-            case STORAGE_NBT -> "storage NBT: " + spec.target() + " / " + spec.path();
+    private static Component expression(WatchSpec spec) {
+        String value = switch (spec.kind()) {
+            case SCORE -> spec.target();
+            case ENTITY_NBT -> spec.path();
+            case STORAGE_NBT -> spec.target() + " / " + spec.path();
         };
+        return WatchUi.text("kind." + spec.kind().name().toLowerCase(java.util.Locale.ROOT))
+            .copy().append(": ").append(Component.literal(value));
     }
 
     private static String value(WatchResult result) {

src/clientTest/java/works/nuty/codon/client/state/ClientBreakpointStateTest.java

@@ -14,6 +14,22 @@
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
 class ClientBreakpointStateTest {
+    @Test
+    void acknowledgedMixedSnapshotKeepsDisabledDefinitionsAndEnabledCountDistinct() {
+        var state = new ClientBreakpointState();
+        var first = BreakpointDefinition.plain(BreakpointTarget.whole(
+            new SourceLocation.Block(new BlockLocation(1, 64, 0, "minecraft:overworld"))));
+        var second = BreakpointDefinition.plain(BreakpointTarget.whole(
+            new SourceLocation.Block(new BlockLocation(2, 64, 0, "minecraft:overworld"))));
+        assertTrue(state.acceptPage(1, 0, true, List.of(first, second.withEnabled(false))));
+        assertEquals(2, state.definitions().size());
+        assertEquals(1, state.definitions().stream().filter(BreakpointDefinition::enabled).count());
+        assertFalse(state.get(second.target()).enabled());
+        assertTrue(state.acceptPage(2, 0, true, List.of(first.withEnabled(false), second.withEnabled(false))));
+        assertEquals(2, state.definitions().size());
+        assertEquals(0, state.definitions().stream().filter(BreakpointDefinition::enabled).count());
+    }
+
     @Test
     void rejectsAnOversizedPagedSnapshotWithoutPublishingIt() {
         ClientBreakpointState state = new ClientBreakpointState();

src/clientTest/java/works/nuty/codon/client/ui/SourceDetailsFormattingTest.java

@@ -12,31 +12,41 @@ void coordinateChangeDoesNotProduceRotationHistory() {
         var before = source(7.5, -59.5, 5.5, 220.2f, 0);
         var after = source(-4.73, -60, 11.77, 220.2f, 0);
         assertEquals("X -4.73 / Y -60.00 / Z 11.77", SourceDetailsFormatting.position(after));
-        assertEquals("yaw 220.2° / pitch 0.0°", SourceDetailsFormatting.rotation(after));
-        assertEquals("", SourceDetailsFormatting.previousRotation(before, after));
+        assertEquals("yaw 220.2° / pitch 0.0°", SourceDetailsFormatting.rotation(after, "yaw", "pitch"));
+        assertEquals("", SourceDetailsFormatting.previousRotation(before, after, "yaw", "pitch"));
         assertEquals("← X 7.50 / Y -59.50 / Z 5.50", SourceDetailsFormatting.previousPosition(before, after));
     }
 
     @Test
     void eachAxisAndAngleIsComparedIndependently() {
         var before = source(1, 2, 3, 45, -10);
         assertEquals("← X - / Y 2.00 / Z -", SourceDetailsFormatting.previousPosition(before, source(1, 4, 3, 45, -10)));
-        assertEquals("← yaw - / pitch -10.0°", SourceDetailsFormatting.previousRotation(before, source(1, 2, 3, 45, 0)));
+        assertEquals("← yaw - / pitch -10.0°", SourceDetailsFormatting.previousRotation(before, source(1, 2, 3, 45, 0), "yaw", "pitch"));
         assertEquals("", SourceDetailsFormatting.previousPosition(before, source(1, 2, 3, 90, 0)));
-        assertEquals("← yaw 45.0° / pitch -10.0°", SourceDetailsFormatting.previousRotation(before, source(1, 2, 3, 90, 0)));
+        assertEquals("← yaw 45.0° / pitch -10.0°", SourceDetailsFormatting.previousRotation(before, source(1, 2, 3, 90, 0), "yaw", "pitch"));
     }
 
     @Test
     void missingParentOrUnchangedTransformDoesNotProduceHistoryRow() {
         var source = source(1, 2, 3, 45, -10);
         assertEquals("", SourceDetailsFormatting.previousPosition(null, source));
-        assertEquals("", SourceDetailsFormatting.previousRotation(null, source));
-        assertEquals("", SourceDetailsFormatting.previousRotation(source, source));
+        assertEquals("", SourceDetailsFormatting.previousRotation(null, source, "yaw", "pitch"));
+        assertEquals("", SourceDetailsFormatting.previousRotation(source, source, "yaw", "pitch"));
         assertEquals("", SourceDetailsFormatting.previousPosition(source, source));
         var otherDimension = new PauseSource(source.anchor(), source.pitch(), source.yaw(), null, "minecraft:the_nether");
         assertEquals("", SourceDetailsFormatting.previousPosition(source, otherDimension));
     }
 
+    @Test
+    void localizedAnglesPreserveValuesAndIndependentHistory() {
+        var before = source(1, 2, 3, 45, -10);
+        var after = source(1, 2, 3, 45, 0);
+        assertEquals("수평각 45.0° / 수직각 0.0°", SourceDetailsFormatting.rotation(after, "수평각", "수직각"));
+        assertEquals("← 수평각 - / 수직각 -10.0°",
+            SourceDetailsFormatting.previousRotation(before, after, "수평각", "수직각"));
+        assertEquals("", SourceDetailsFormatting.previousRotation(null, after, "수평각", "수직각"));
+    }
+
     private static PauseSource source(double x, double y, double z, float yaw, float pitch) {
         return new PauseSource(new Vec3d(x, y, z), pitch, yaw, null, "minecraft:overworld");
     }