@alliance-droid/status-feedback-system 4.1.4 → 4.1.6

package/CHANGELOG.md

@@ -1,5 +1,17 @@
 # Changelog
 
+## 4.1.6
+
+### Fixed
+
+- Board status badges: for privileged viewers (Admin / ProductOwner), **every** item now shows its real internal status (e.g. "QA / UAT"), not just the first one. Previously the list badge derived its label solely from the per-item transitions cache, which is only populated lazily (on menu open) plus a one-time privilege probe of `items[0]` — so exactly the first item showed its real status and the rest stayed on the coarse public projection ("In progress"). The badge now derives the privileged label from each item's own `status` (already present on every board item) via a new `internalStatusLabel`, so all rows resolve consistently once privilege is confirmed. Non-privileged viewers are unchanged (they still see the public status bucket).
+
+## 4.1.5
+
+### Fixed
+
+- `<DevFeedback>` console-error capture: arguments are now serialized properly before being attached to a bug report. Previously `console.error(obj)` / `console.error('msg', { detail })` were captured via `String()`, which produced `[object Object]` — so the resulting ADO work item listed each console error as `[object Object]`. Errors now serialize to their stack, plain objects/arrays to JSON (with circular-reference safety), and primitives via `String()`. Extracted to `formatConsoleArg` / `formatConsoleArgs` with unit coverage.
+
 ## 4.1.4
 
 ### Changed

package/dist/components/DevFeedback.svelte

@@ -1,6 +1,7 @@
 <script lang="ts">
 	import { createStatusClient, type StatusClient } from '../api.js';
 	import { FEEDBACK_CATEGORIES } from '../types.js';
+	import { formatConsoleArgs } from '../console-format.js';
 	import PasteImageInput from './shared/PasteImageInput.svelte';
 	import {
 		Card,
@@ -209,7 +210,7 @@
 		// Capture recent console errors to attach to bug reports.
 		const originalConsoleError = console.error;
 		console.error = (...args: unknown[]) => {
-			consoleErrors.push(args.map(String).join(' '));
+			consoleErrors.push(formatConsoleArgs(args));
 			if (consoleErrors.length > maxConsoleErrors) consoleErrors.shift();
 			originalConsoleError.apply(console, args);
 		};

package/dist/components/board/StatusBadgeMenu.svelte

@@ -21,7 +21,7 @@
 	import { onMount } from 'svelte';
 	import type { BoardItem, BoardTransition } from '../../api.js';
 	import type { FeedbackBoardState } from './state.svelte.js';
-	import { getStatusBadgeVariant, publicStatusLabel } from './utils.js';
+	import { getStatusBadgeVariant, publicStatusLabel, internalStatusLabel } from './utils.js';
 
 	interface Props {
 		board: FeedbackBoardState;
@@ -45,7 +45,15 @@
 
 	// Read from the board's shared cache so detail + list reuse the same fetch.
 	let txns = $derived(board.transitions[item.id] ?? null);
-	let badgeLabel = $derived(txns?.current.display ?? publicStatusLabel(item.publicStatus));
+	// Label priority: the authoritative loaded transitions display (if this item's
+	// transitions have been fetched) → for privileged viewers, the item's own
+	// internal status (present on every board item, so ALL rows show their real
+	// status, not just the one the privilege probe happened to load) → otherwise
+	// the coarse public projection shown to everyone else.
+	let badgeLabel = $derived(
+		txns?.current.display ??
+			(board.canManageStatus ? internalStatusLabel(item.status) : publicStatusLabel(item.publicStatus))
+	);
 	// Changes whenever the menu's content (and thus its size) changes — drives a
 	// reposition so the teleported menu stays anchored after it loads/expands.
 	let posKey = $derived(`${menuOpen}:${loading}:${pendingTarget?.status ?? ''}:${txns?.targets.length ?? 0}`);

package/dist/components/board/utils.d.ts

@@ -3,5 +3,11 @@ export declare function timeAgo(timestamp: number): string;
 import type { PublicStatus } from '../../types.js';
 /** Human label for a public board status. */
 export declare function publicStatusLabel(status: PublicStatus): string;
+/**
+ * Human label for an item's internal status. Falls back to a title-cased
+ * version of an unknown status value (e.g. a status added to the backend after
+ * this package was built) so the badge always shows something readable.
+ */
+export declare function internalStatusLabel(status: string): string;
 /** Map a public board status to a badge variant. */
 export declare function getStatusBadgeVariant(status: PublicStatus): 'success' | 'neutral' | 'info' | 'warning';

package/dist/components/board/utils.js

@@ -30,6 +30,42 @@ const PUBLIC_STATUS_VARIANTS = {
 export function publicStatusLabel(status) {
     return PUBLIC_STATUS_LABELS[status] ?? status;
 }
+/**
+ * Display labels for the backend's internal statuses. Mirrors `STATUS_META`
+ * (display) in status-feedback-admin's `status.ts`. Privileged users (who can
+ * manage status) see these precise labels; the public sees the coarser
+ * `publicStatus` bucket instead. `internalStatusLabel` degrades gracefully for
+ * any status not in this map, so drift only loses polish — never breaks.
+ */
+const INTERNAL_STATUS_LABELS = {
+    open: 'Open',
+    considering: 'Considering',
+    accepted: 'Accepted',
+    investigating: 'Investigating',
+    waiting_on_reply: 'Waiting on reply',
+    ready_to_start: 'Ready to start',
+    in_progress: 'In progress',
+    technical_review: 'Technical review',
+    qa_uat: 'QA / UAT',
+    reopened: 'Reopened',
+    resolved: 'Resolved',
+    wont_do: "Won't do",
+    duplicate: 'Duplicate',
+    cannot_reproduce: "Can't reproduce"
+};
+/**
+ * Human label for an item's internal status. Falls back to a title-cased
+ * version of an unknown status value (e.g. a status added to the backend after
+ * this package was built) so the badge always shows something readable.
+ */
+export function internalStatusLabel(status) {
+    if (INTERNAL_STATUS_LABELS[status])
+        return INTERNAL_STATUS_LABELS[status];
+    if (!status)
+        return '';
+    const spaced = status.replace(/_/g, ' ');
+    return spaced.charAt(0).toUpperCase() + spaced.slice(1);
+}
 /** Map a public board status to a badge variant. */
 export function getStatusBadgeVariant(status) {
     return PUBLIC_STATUS_VARIANTS[status] ?? 'neutral';

package/dist/console-format.d.ts

@@ -0,0 +1,12 @@
+/**
+ * Serialize console.error arguments into useful strings for bug-report capture.
+ *
+ * The naive `args.map(String).join(' ')` turns any object argument into the
+ * useless `"[object Object]"` and drops an Error's stack — so a captured
+ * `console.error(err)` or `console.error('msg', { detail })` reaches the ADO
+ * work item as `[object Object]`. These helpers serialize each argument by shape.
+ */
+/** Serialize a single console.error argument to a useful string. */
+export declare function formatConsoleArg(arg: unknown): string;
+/** Serialize a console.error argument list into a single line. */
+export declare function formatConsoleArgs(args: unknown[]): string;

package/dist/console-format.js

@@ -0,0 +1,46 @@
+/**
+ * Serialize console.error arguments into useful strings for bug-report capture.
+ *
+ * The naive `args.map(String).join(' ')` turns any object argument into the
+ * useless `"[object Object]"` and drops an Error's stack — so a captured
+ * `console.error(err)` or `console.error('msg', { detail })` reaches the ADO
+ * work item as `[object Object]`. These helpers serialize each argument by shape.
+ */
+/** Serialize a single console.error argument to a useful string. */
+export function formatConsoleArg(arg) {
+    if (typeof arg === 'string')
+        return arg;
+    if (arg instanceof Error)
+        return arg.stack || `${arg.name}: ${arg.message}`;
+    if (arg === null)
+        return 'null';
+    if (arg === undefined)
+        return 'undefined';
+    if (typeof arg === 'object') {
+        const seen = new WeakSet();
+        try {
+            const json = JSON.stringify(arg, (_key, value) => {
+                if (value instanceof Error)
+                    return value.stack || `${value.name}: ${value.message}`;
+                if (typeof value === 'object' && value !== null) {
+                    if (seen.has(value))
+                        return '[Circular]';
+                    seen.add(value);
+                }
+                return value;
+            });
+            // JSON.stringify returns undefined for unserializable roots and "{}"
+            // for objects with no enumerable props — fall back to String() then.
+            if (json && json !== '{}')
+                return json;
+        }
+        catch {
+            /* fall through to String() */
+        }
+    }
+    return String(arg);
+}
+/** Serialize a console.error argument list into a single line. */
+export function formatConsoleArgs(args) {
+    return args.map(formatConsoleArg).join(' ');
+}

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@alliance-droid/status-feedback-system",
-	"version": "4.1.4",
+	"version": "4.1.6",
 	"type": "module",
 	"description": "Drop-in system status pages and user feedback forms for SvelteKit apps",
 	"svelte": "./dist/index.js",