Merge pull request #150 from vincentmakes/claude/fix-compact-print-urls-CHT94

Fix print_compact flag not reaching public dataset URLs

Author: vincentmakes

CHANGELOG.md

@@ -4,6 +4,11 @@ All notable changes to CV Manager will be documented in this file.
 
 Format follows [Keep a Changelog](https://keepachangelog.com/), versioning follows [Semantic Versioning](https://semver.org/).
 
+## [1.48.3] - 2026-04-23
+
+### Fixed
+- **Per-section "compact print" toggle is now honored when printing from public URLs.** The Skills / Projects compact-print flag (stored in `section_visibility.print_compact`) was correctly applied on the admin page and on the public page's live-DB fallback, but any public URL that renders from a saved dataset — the default dataset served at `/`, the versioned `/v/:slug` (+ `/v/:slug/:lang`) routes, and static-site exports — ignored it, because `print_compact` (and, same shape, `print_visible`) was dropped at three points in the dataset pipeline: `gatherCvData()` in `src/server.js` didn't include the fields when building `sectionOrder` entries, and the two client-side mappers in `public-readonly/index.html` (`loadDatasetPreview` and `loadStaticData`) further stripped them while reshaping the snapshot. `renderSectionsInOrder()` reads the flag correctly — it just never saw a value, so the `.print-compact` CSS class (and its inline-text layout in `public/shared/styles.css`) was never applied and printed pages always rendered the full card grid. Added the fields to `gatherCvData()`'s snapshot and to both client mappers. While in the area, also propagated the fields through `propagateStructure()` so language-sibling datasets share the same structural choice, and preserved them on `POST /api/datasets/:id/load` and `POST /api/import` (mirroring the `section.print_compact !== undefined ? … : existing` pattern already used by `PUT /api/sections/order`) so the roundtrip stays honest. Datasets saved before this release have no flag stored and render non-compact until re-saved, which is the correct default. No schema changes, no new endpoints, no i18n strings — the CSS rules and the admin UI were already in place. Covered by a new backend test that saves a public dataset with `print_compact=true` on Skills and asserts the flag reaches `GET /api/datasets/slug/:slug` on the public port.
+
 ## [1.48.2] - 2026-04-23
 
 ### Fixed

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "cv-manager",
-  "version": "1.48.2",
+  "version": "1.48.3",
   "description": "Professional CV Management System",
   "main": "src/server.js",
   "scripts": {

package.json

@@ -258,7 +258,7 @@ <h2 class="section-title" data-i18n="section.projects">Featured Projects</h2>
                     }
                     sectionOrder = data.sectionOrder.filter(s => s.visible !== false).map(s => {
                         const fallbackName = csNameMap[s.key] || s.key;
-                        return { key: s.key, visible: s.visible, name: s.display_name || s.name || fallbackName, default_name: s.default_name || s.name || fallbackName };
+                        return { key: s.key, visible: s.visible, print_visible: s.print_visible, print_compact: s.print_compact, name: s.display_name || s.name || fallbackName, default_name: s.default_name || s.name || fallbackName };
                     });
                     renderSectionsInOrder();
                     applySectionTitles(sectionOrder);
@@ -362,7 +362,7 @@ <h2 class="section-title" data-i18n="section.projects">Featured Projects</h2>
                     }
                     sectionOrder = datasetData.sectionOrder.filter(s => s.visible !== false).map(s => {
                         const fallbackName = csNameMap[s.key] || s.key;
-                        return { key: s.key, visible: s.visible, name: s.display_name || s.name || fallbackName, default_name: s.default_name || s.name || fallbackName };
+                        return { key: s.key, visible: s.visible, print_visible: s.print_visible, print_compact: s.print_compact, name: s.display_name || s.name || fallbackName, default_name: s.default_name || s.name || fallbackName };
                     });
                     renderSectionsInOrder();
                     applySectionTitles(sectionOrder);

public-readonly/index.html

@@ -701,6 +701,46 @@ describe('Backend API', () => {
             });
         });
 
+        it('saved datasets carry print_compact through to the public API', async () => {
+            // Turn on print_compact for skills in the live DB
+            await fetch(`${BASE_URL}/api/sections/skills/print-compact`, {
+                method: 'PUT',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ print_compact: true }),
+            });
+
+            // Create a public dataset and save the current live state into it
+            const createRes = await fetch(`${BASE_URL}/api/datasets`, {
+                method: 'POST',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ name: 'Compact Print Public' }),
+            });
+            const created = await createRes.json();
+            await fetch(`${BASE_URL}/api/datasets/${created.id}/save`, { method: 'POST' });
+            await fetch(`${BASE_URL}/api/datasets/${created.id}/public`, {
+                method: 'PUT',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ is_public: true }),
+            });
+
+            // Fetch the dataset via the PUBLIC server's slug endpoint —
+            // this is what /v/:slug's client uses to render the CV
+            const pubRes = await fetch(`${PUBLIC_URL}/api/datasets/slug/${created.slug}`);
+            assert.strictEqual(pubRes.status, 200);
+            const pub = await pubRes.json();
+            const pubSkills = pub.sectionOrder.find(s => s.key === 'skills');
+            assert.ok(pubSkills, 'skills entry present in public dataset sectionOrder');
+            assert.strictEqual(pubSkills.print_compact, true, 'print_compact reaches the public dataset payload');
+
+            // Clean up — delete dataset, reset flag
+            await fetch(`${BASE_URL}/api/datasets/${created.id}`, { method: 'DELETE' });
+            await fetch(`${BASE_URL}/api/sections/skills/print-compact`, {
+                method: 'PUT',
+                headers: { 'Content-Type': 'application/json' },
+                body: JSON.stringify({ print_compact: false }),
+            });
+        });
+
         // --- Reorder ---
         it('PUT /api/reorder/:type reorders items', async () => {
             // Create two experiences

src/server.js

@@ -1,4 +1,4 @@
 {
-  "version": "1.48.2",
+  "version": "1.48.3",
   "changelog": "https://github.com/vincentmakes/cv-manager/blob/main/CHANGELOG.md"
 }