@conduction/nextcloud-vue 2.70.0 → 2.72.0

package/src/components/CnFlowStepOutcomes/CnFlowStepOutcomes.vue

@@ -0,0 +1,250 @@
+<!--
+  CnFlowStepOutcomes: who a send step reached, and who it did not, by name.
+
+  A messaging step (send email, send notification, post to Talk) writes its
+  per-recipient outcome onto its run-log entry as `report.messaging`: one
+  bucket per outcome, each `{count, sample}`. Nothing rendered it, so the run
+  log could not show that a person was skipped because they opted out, or
+  because the opt-out check did not answer.
+
+  Only buckets with someone in them are listed. A bucket this component does
+  not know is still shown, under its own key made readable, so a new outcome
+  on the server is never hidden by an older library.
+
+  SPDX-FileCopyrightText: 2026 Conduction B.V. <info@conduction.nl>
+  SPDX-License-Identifier: EUPL-1.2
+-->
+<template>
+	<div v-if="rows.length" class="cn-flow-step-outcomes" data-testid="flow-step-outcomes">
+		<dl class="cn-flow-step-outcomes__list" :aria-label="t('nextcloud-vue', 'Who this step reached')">
+			<div v-for="row in rows"
+				:key="row.key"
+				class="cn-flow-step-outcomes__row"
+				:class="`cn-flow-step-outcomes__row--${row.tone}`"
+				:data-testid="`flow-step-outcome-${row.key}`">
+				<dt class="cn-flow-step-outcomes__name">
+					{{ row.label }}
+				</dt>
+				<dd class="cn-flow-step-outcomes__value">
+					<span class="cn-flow-step-outcomes__count">{{ row.count }}</span>
+					<span v-if="row.sample.length" class="cn-flow-step-outcomes__sample"> · {{ row.sample.join(', ') }}</span>
+				</dd>
+			</div>
+		</dl>
+		<p v-if="report && report.truncated"
+			class="cn-flow-step-outcomes__truncated"
+			data-testid="flow-step-outcomes-truncated">
+			{{ t('nextcloud-vue', 'Each list shows its first entries only.') }}
+		</p>
+	</div>
+</template>
+
+<script>
+import { translate as t } from '@nextcloud/l10n'
+
+/**
+ * The buckets a messaging report carries, in reading order: who got it first,
+ * then each reason someone did not. `tone` marks the row for styling only.
+ */
+const KNOWN_BUCKETS = [
+	{ key: 'delivered', tone: 'success' },
+	{ key: 'optedOut', tone: 'neutral' },
+	{ key: 'authorityUnavailable', tone: 'warning' },
+	{ key: 'skippedByPreference', tone: 'neutral' },
+	{ key: 'skippedByKillSwitch', tone: 'neutral' },
+	{ key: 'rateLimited', tone: 'warning' },
+	{ key: 'refusedRecipients', tone: 'warning' },
+	{ key: 'unknownRecipients', tone: 'warning' },
+	{ key: 'failed', tone: 'error' },
+]
+
+/** Report keys that are facts about the send, not outcome buckets. */
+const NOT_BUCKETS = ['channel', 'actor', 'recipients', 'truncated']
+
+export default {
+	name: 'CnFlowStepOutcomes',
+
+	props: {
+		/**
+		 * A step's messaging report: the run-log entry's `report.messaging`.
+		 * Each outcome bucket is `{count, sample}`; a `refusedRecipients`
+		 * sample entry is `{recipient, reason}`. `null` renders nothing.
+		 */
+		report: {
+			type: Object,
+			default: null,
+		},
+
+		/**
+		 * Names to use instead of the built-in ones, keyed by bucket
+		 * (`{ optedOut: 'Unsubscribed' }`). Pass already-translated text.
+		 */
+		labels: {
+			type: Object,
+			default: () => ({}),
+		},
+	},
+
+	computed: {
+		/**
+		 * The non-empty buckets, known ones first in reading order, then any
+		 * bucket the server added that this component does not know.
+		 *
+		 * @return {Array<object>} One row per bucket: key, label, count, sample, tone.
+		 */
+		rows() {
+			const report = this.report
+			if (!report || typeof report !== 'object') {
+				return []
+			}
+
+			const known = KNOWN_BUCKETS.map((bucket) => bucket.key)
+			const extra = Object.keys(report).filter((key) => !known.includes(key) && !NOT_BUCKETS.includes(key))
+			const ordered = [
+				...KNOWN_BUCKETS,
+				...extra.map((key) => ({ key, tone: 'neutral' })),
+			]
+
+			return ordered
+				.map(({ key, tone }) => ({ key, tone, bucket: report[key] }))
+				.filter(({ bucket }) => bucket && typeof bucket === 'object' && Number(bucket.count) > 0)
+				.map(({ key, tone, bucket }) => ({
+					key,
+					tone,
+					label: this.labelFor(key),
+					count: Number(bucket.count),
+					sample: (Array.isArray(bucket.sample) ? bucket.sample : []).map((entry) => this.describeEntry(entry)),
+				}))
+		},
+	},
+
+	methods: {
+		t,
+
+		/**
+		 * The readable name of a bucket: the host's, then the built-in one,
+		 * then the key itself (`heldForReview` reads "Held for review").
+		 *
+		 * @param {string} key The bucket key.
+		 * @return {string} The name.
+		 */
+		labelFor(key) {
+			if (this.labels && this.labels[key]) {
+				return String(this.labels[key])
+			}
+
+			const builtIn = {
+				delivered: t('nextcloud-vue', 'Delivered'),
+				optedOut: t('nextcloud-vue', 'Opted out'),
+				authorityUnavailable: t('nextcloud-vue', 'Not sent, the opt-out check did not answer'),
+				skippedByPreference: t('nextcloud-vue', 'Skipped by their notification settings'),
+				skippedByKillSwitch: t('nextcloud-vue', 'Skipped, sending is switched off'),
+				rateLimited: t('nextcloud-vue', 'Held back by the send limit'),
+				refusedRecipients: t('nextcloud-vue', 'Refused by the step’s address rule'),
+				unknownRecipients: t('nextcloud-vue', 'Not a known user or group'),
+				failed: t('nextcloud-vue', 'Failed'),
+			}
+			if (builtIn[key]) {
+				return builtIn[key]
+			}
+
+			const words = String(key).replace(/([a-z0-9])([A-Z])/g, '$1 $2').replace(/[-_]+/g, ' ').trim().toLowerCase()
+
+			return words.charAt(0).toUpperCase() + words.slice(1)
+		},
+
+		/**
+		 * One sample entry as text. A refused address carries its reason.
+		 *
+		 * @param {string|object} entry The sample entry.
+		 * @return {string} The text.
+		 */
+		describeEntry(entry) {
+			if (entry && typeof entry === 'object') {
+				const who = String(entry.recipient ?? entry.id ?? '')
+				const reason = this.reasonFor(entry.reason)
+
+				return reason ? `${who} (${reason})` : who
+			}
+
+			return String(entry)
+		},
+
+		/**
+		 * Why the step's address rule refused an address, in words.
+		 *
+		 * @param {string} [code] The refusal code.
+		 * @return {string} The reason, or the code when it is not known.
+		 */
+		reasonFor(code) {
+			const reasons = {
+				'external-recipients-off': t('nextcloud-vue', 'external recipients are off'),
+				'not-on-item': t('nextcloud-vue', 'not on the item'),
+				'invalid-address': t('nextcloud-vue', 'not a valid address'),
+			}
+
+			return code ? (reasons[code] || String(code)) : ''
+		},
+	},
+}
+</script>
+
+<style scoped>
+.cn-flow-step-outcomes {
+	margin: 4px 0 0;
+	font-size: 0.9em;
+}
+
+.cn-flow-step-outcomes__list {
+	margin: 0;
+	padding: 0;
+}
+
+/* Stacked, name over value: the run sidebar is narrow, and a side-by-side
+   row wrapped mid-list. The host's dt/dd margins and alignment are reset. */
+.cn-flow-step-outcomes__row {
+	display: block;
+	margin-block: 4px;
+	padding-block: 2px;
+	padding-inline-start: 8px;
+	border-inline-start: 3px solid var(--color-border);
+	text-align: start;
+}
+
+.cn-flow-step-outcomes__row--success {
+	border-inline-start-color: var(--color-success);
+}
+
+.cn-flow-step-outcomes__row--warning {
+	border-inline-start-color: var(--color-warning);
+}
+
+.cn-flow-step-outcomes__row--error {
+	border-inline-start-color: var(--color-error);
+}
+
+.cn-flow-step-outcomes__name {
+	display: block;
+	white-space: normal;
+	overflow-wrap: anywhere;
+	margin: 0;
+	padding: 0;
+	font-weight: bold;
+	text-align: start;
+}
+
+.cn-flow-step-outcomes__value {
+	display: block;
+	white-space: normal;
+	margin: 0;
+	padding: 0;
+	text-align: start;
+	color: var(--color-main-text);
+	overflow-wrap: anywhere;
+}
+
+.cn-flow-step-outcomes__truncated {
+	margin: 4px 0 0;
+	color: var(--color-text-maxcontrast);
+}
+</style>

package/src/components/CnFlowStepOutcomes/index.js

@@ -0,0 +1,8 @@
+/**
+ * SPDX-FileCopyrightText: 2026 Conduction B.V. <info@conduction.nl>
+ * SPDX-License-Identifier: EUPL-1.2
+ */
+import CnFlowStepOutcomes from './CnFlowStepOutcomes.vue'
+
+export default CnFlowStepOutcomes
+export { CnFlowStepOutcomes }

package/src/components/CnFormDialog/CnFormDialog.vue

@@ -250,6 +250,17 @@
 									<!-- @binding {object} option The option being rendered (`{ id, label }`). -->
 									<slot :name="'field-' + field.key + '-option'" v-bind="optionProps" />
 								</template>
+								<!-- NcSelect's own option splits a label in two spans
+								     (NcEllipsisedOption, keeping its last characters
+								     visible), and in a narrow dropdown the halves broke
+								     onto two lines mid-word: "Agricu lture". A plain
+								     label wraps at spaces only. NcSelectUsers keeps its
+								     avatar rendering. -->
+								<template
+									v-else-if="!isUserField(field)"
+									#option="optionProps">
+									<span class="cn-form-dialog__option">{{ optionText(optionProps) }}</span>
+								</template>
 								<template
 									v-if="$slots['field-' + field.key + '-selected-option']"
 									#selected-option="optionProps">
@@ -284,11 +295,17 @@
 									#option="optionProps">
 									<slot :name="'field-' + field.key + '-option'" v-bind="optionProps" />
 								</template>
+								<template v-else #option="optionProps">
+									<span class="cn-form-dialog__option">{{ optionText(optionProps) }}</span>
+								</template>
 								<template
 									v-if="$slots['field-' + field.key + '-selected-option']"
 									#selected-option="optionProps">
 									<slot :name="'field-' + field.key + '-selected-option'" v-bind="optionProps" />
 								</template>
+								<template v-else #selected-option="optionProps">
+									<span class="cn-form-dialog__option">{{ optionText(optionProps) }}</span>
+								</template>
 							</NcSelect>
 							<CnFieldHelper
 								:text="field.description"
@@ -318,11 +335,17 @@
 									#option="optionProps">
 									<slot :name="'field-' + field.key + '-option'" v-bind="optionProps" />
 								</template>
+								<template v-else #option="optionProps">
+									<span class="cn-form-dialog__option">{{ optionText(optionProps) }}</span>
+								</template>
 								<template
 									v-if="$slots['field-' + field.key + '-selected-option']"
 									#selected-option="optionProps">
 									<slot :name="'field-' + field.key + '-selected-option'" v-bind="optionProps" />
 								</template>
+								<template v-else #selected-option="optionProps">
+									<span class="cn-form-dialog__option">{{ optionText(optionProps) }}</span>
+								</template>
 							</NcSelect>
 							<CnFieldHelper
 								:text="field.description"
@@ -2452,6 +2475,26 @@ export default {
 		 * @param {object} field The field definition
 		 * @return {boolean}
 		 */
+		/**
+		 * The text a select option shows: its label, else a display name, a
+		 * name or the id. A plain string option (a tag) is its own text.
+		 *
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/schema-utilities/spec.md
+		 * @param {object|string} option The option (slot props).
+		 * @return {string} The text.
+		 */
+		optionText(option) {
+			if (option === null || option === undefined) {
+				return ''
+			}
+			if (typeof option !== 'object') {
+				return String(option)
+			}
+			const text = [option.label, option.displayName, option.name, option.id]
+				.find((v) => v !== undefined && v !== null && v !== '')
+			return text === undefined ? '' : String(text)
+		},
+
 		isUserField(field) {
 			return !!(field && field.userPicker && !field.userPicker.multiple)
 		},
@@ -3519,8 +3562,15 @@ export default {
 			if (this.isIdPickerField(field)) {
 				// Reference / user fields store the chosen id (UUID / UID),
 				// not the full option. Cache its label so the selection displays.
+				// A user option can carry only `displayName` (NcSelectUsers' own
+				// shape), and without that fallback the chip showed the uid.
 				if (option && option.id) {
-					this.referenceLabels = { ...this.referenceLabels, [option.id]: option.label || String(option.id) }
+					const name = option.label || option.displayName || this.referenceLabels[option.id]
+					if (name) {
+						this.referenceLabels = { ...this.referenceLabels, [option.id]: name }
+					} else if (this.isUserField(field)) {
+						this.resolveUserLabel(option.id)
+					}
 				}
 				this.updateField(field.key, option ? option.id : null)
 				// Reference options are label-only ({id,label}) — pass the id so
@@ -3587,7 +3637,7 @@ export default {
 				const labels = {}
 				for (const o of list) {
 					if (o && o.id) {
-						labels[o.id] = o.label || String(o.id)
+						labels[o.id] = o.label || o.displayName || this.referenceLabels[o.id] || String(o.id)
 					}
 				}
 				if (Object.keys(labels).length > 0) {
@@ -3861,6 +3911,16 @@ export default {
 </script>
 
 <style scoped>
+/* A select option breaks between words only, never inside one. */
+.cn-form-dialog__option {
+	display: block;
+	min-width: 0;
+	white-space: normal;
+	overflow-wrap: normal;
+	word-break: normal;
+	hyphens: manual;
+}
+
 .cn-form-dialog__draft-actions {
 	display: flex;
 	gap: 8px;

package/src/components/CnIndexPage/CnIndexPage.vue

@@ -488,7 +488,7 @@
 					:includeColumns="includeColumns"
 					:columnOverrides="columnOverrides"
 					:rowClass="rowClass"
-					:filterable="headerFilters && !!effectiveSchema"
+					:filterable="tableHeaderFilters"
 					:activeFilters="effectiveActiveFilters"
 					:filterRegister="typeof register === 'string' ? register : ''"
 					@columnFilter="onColumnFilterEvent"
@@ -3873,6 +3873,28 @@ export default {
 		},
 
 		/** Resolved schema OBJECT (for column generation / icons / labels). */
+		/**
+		 * Whether the table shows its header filters. A self-fetching page
+		 * applies them itself, with or without a resolved schema (the columns
+		 * then say how each one filters, else a text filter on contains). A
+		 * host-fed table shows them when a schema came along, as before, or
+		 * when the host listens for `filter-change`; a filter nobody applies
+		 * would be a control that does nothing.
+		 *
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-data-table/spec.md
+		 * @return {boolean} True when header filters show.
+		 */
+		tableHeaderFilters() {
+			if (!this.headerFilters) {
+				return false
+			}
+			if (this.effectiveSchema || this.isSelfFetchMode) {
+				return true
+			}
+			const vnodeProps = (this.$ && this.$.vnode && this.$.vnode.props) || {}
+			return typeof vnodeProps.onFilterChange === 'function' || Array.isArray(vnodeProps.onFilterChange)
+		},
+
 		effectiveSchema() {
 			if (this.isSelfFetchMode) {
 				return this.list.schema.value

package/src/components/CnObjectSidebar/CnNotesTab.vue

@@ -279,7 +279,7 @@ export default {
 		 * @param {(suggestions: Array<object>) => void} callback Receives the suggestion array.
 		 */
 		async fetchMentionSuggestions(search, callback) {
-			const users = await searchNextcloudUsers(search)
+			const users = await searchNextcloudUsers(search, { includeCurrentUser: false })
 			callback(users.map((user) => ({
 				id: user.id,
 				label: user.label,

package/src/components/CnSetupWizard/CnSetupWizard.vue

@@ -123,6 +123,13 @@
 
 				<!-- config-fields -->
 				<template v-else-if="step.type === 'config-fields'">
+					<!-- The step's intro, drawn like choice and run-action steps.
+					     It was left out here, so pipelinq's organisation step
+					     ("Your organisation details…") showed fields with no
+					     word about what they were for. -->
+					<NcNoteCard v-if="step.body" type="info" data-testid="cn-setup-step-intro">
+						{{ stepBody(step) }}
+					</NcNoteCard>
 					<div
 						v-for="field in fieldsFor(step)"
 						:key="field.key"

package/src/components/CnWalkthrough/CnWalkthrough.vue

@@ -172,6 +172,12 @@ export default {
 		tourId: { type: String, default: '' },
 		/** Optional resume token `{ tourId, stepId }` (refresh / cross-app hand-off). */
 		resume: { type: Object, default: null },
+		/**
+		 * Whether a first-visit tour may start on its own. The host turns it
+		 * off while the user's tour is paused, so a paused tour stays hidden
+		 * across page loads until the user picks "Continue".
+		 */
+		autoStart: { type: Boolean, default: true },
 		/** Stacking order of the overlay. */
 		zIndex: { type: Number, default: 10000 },
 		/** Next button label. */
@@ -571,6 +577,9 @@ export default {
 				this.wt.start(this.tourId)
 				return
 			}
+			if (!this.autoStart) {
+				return
+			}
 			const auto = this.wt.autoStartTour.value
 			if (!auto) {
 				return
@@ -1171,31 +1180,43 @@ export default {
 		},
 
 		/**
-		 * End the tour for good from the corner close button: mark it complete so
-		 * the seen-version is persisted and it does not auto-show again.
+		 * The corner close button PAUSES the tour, like ESC and the dim. It
+		 * used to complete it, and the host then wiped the saved step, so a
+		 * restart began at step 1 again. Only Finish on the last step (and a
+		 * host-supplied Skip) completes the tour; "Start over" in the user
+		 * settings stays the way back to step 1.
 		 *
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
 		 * @return {void}
 		 */
 		close() {
-			this.wt.complete()
-			this.$emit('complete')
+			this.onBackdrop()
 		},
 
 		/**
-		 * Pause the tour from a backdrop click or ESC. The tour hides and
-		 * keeps its step; only Skip (the close button) and Finish end it.
+		 * Pause the tour from the corner X, a backdrop click or ESC. The tour
+		 * hides and keeps its step; only Finish (or a host Skip) ends it.
 		 * Pressing ESC used to dismiss it, and the host recorded that as
-		 * "seen", so the tour never came back.
+		 * "seen", so the tour never came back. The `pause` event carries
+		 * where the user was, so the host can keep a paused tour hidden on
+		 * the next page load.
 		 *
 		 * @spec openspec/changes/walkthrough-advance-pause-resume/specs/cn-walkthrough/spec.md
+		 * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
 		 * @return {void}
 		 */
 		onBackdrop() {
+			const where = {
+				tourId: (this.wt.activeTour.value && this.wt.activeTour.value.id) || '',
+				stepId: (this.step && this.step.id) || '',
+				index: this.index,
+			}
 			this.wt.pause()
 			/**
-			 * @event pause Emitted when the user hides the tour (backdrop / ESC). The tour can continue later.
+			 * @event pause Emitted when the user hides the tour (X, backdrop, ESC). The tour can continue later.
+			 * @type {{ tourId: string, stepId: string, index: number }}
 			 */
-			this.$emit('pause')
+			this.$emit('pause', where)
 		},
 	},
 }

package/src/components/index.js

@@ -191,6 +191,7 @@ export { CnDeltaWidgetForm } from './CnDeltaWidgetForm/index.js'
 export { CnGaugeWidget } from './CnGaugeWidget/index.js'
 export { CnGaugeWidgetForm } from './CnGaugeWidgetForm/index.js'
 export { CnFlowRunsWidget } from './CnFlowRunsWidget/index.js'
+export { CnFlowStepOutcomes } from './CnFlowStepOutcomes/index.js'
 export { CnFlowRunsWidgetForm } from './CnFlowRunsWidgetForm/index.js'
 export { CnTasksWidget } from './CnTasksWidget/index.js'
 export { CnTasksWidgetForm } from './CnTasksWidgetForm/index.js'

package/src/composables/useWalkthrough.js

@@ -216,11 +216,13 @@ export function walkthroughProgressKey(configKey) {
 
 /**
  * Normalise stored progress (an object, or the JSON string a preference
- * holds) into `{ tourId, stepId, index, version }`, or null.
+ * holds) into `{ tourId, stepId, index, version }`, plus `paused: true`
+ * when the user paused the tour, or null.
  *
  * @spec openspec/changes/walkthrough-advance-pause-resume/specs/cn-walkthrough/spec.md
+ * @spec openspec/changes/audit-round-lib-fixes/specs/cn-walkthrough/spec.md
  * @param {unknown} value The stored value.
- * @return {{tourId: string, stepId: string, index: number, version: string}|null} The progress.
+ * @return {{tourId: string, stepId: string, index: number, version: string, paused: (boolean|undefined)}|null} The progress.
  */
 export function normaliseWalkthroughProgress(value) {
 	let data = value
@@ -237,12 +239,18 @@ export function normaliseWalkthroughProgress(value) {
 	if (!isPlainObject(data) || typeof data.tourId !== 'string' || data.tourId === '') {
 		return null
 	}
-	return {
+	const progress = {
 		tourId: data.tourId,
 		stepId: typeof data.stepId === 'string' ? data.stepId : '',
 		index: Number.isInteger(data.index) && data.index >= 0 ? data.index : 0,
 		version: typeof data.version === 'string' ? data.version : '',
 	}
+	// A paused tour stays hidden on the next visit until the user picks
+	// "Continue". Only written when true, so older values read unchanged.
+	if (data.paused === true) {
+		progress.paused = true
+	}
+	return progress
 }
 
 /**

package/src/css/dashboard.css

@@ -129,3 +129,47 @@
 .cn-requires-app.cn-requires-app .empty-content__action {
 	margin-top: var(--default-grid-baseline, 4px);
 }
+
+/* NcEmptyContent forces its icon's svg to 64px with !important while the
+ * rule above lets the icon box shrink, so the svg spilled out of its box and
+ * over the name. Hold the svg to the size the caller asked for. */
+.cn-requires-app.cn-requires-app .empty-content__icon svg {
+	width: 32px !important;
+	height: 32px !important;
+	max-width: 32px !important;
+	max-height: 32px !important;
+}
+
+/* The dashboard's compact "Install <app>" placeholder: two lines of text and
+ * the button, nothing else, so it fits a two-row tile even under a title row.
+ * `safe center` keeps the top in view when a tile is smaller still, and the
+ * body scrolls rather than clipping the button. */
+.cn-requires-app--compact {
+	display: flex;
+	flex-direction: column;
+	align-items: center;
+	justify-content: safe center;
+	gap: calc(2 * var(--default-grid-baseline, 4px));
+	box-sizing: border-box;
+	height: 100%;
+	min-height: 0;
+	overflow: auto;
+	padding: calc(2 * var(--default-grid-baseline, 4px));
+	text-align: center;
+}
+
+.cn-requires-app--compact .cn-requires-app__description {
+	display: -webkit-box;
+	margin: 0;
+	overflow: hidden;
+	color: var(--color-text-maxcontrast);
+	font-size: 0.9em;
+	line-height: 1.4;
+	-webkit-box-orient: vertical;
+	-webkit-line-clamp: 2;
+	line-clamp: 2;
+}
+
+.cn-requires-app--compact .button-vue {
+	flex-shrink: 0;
+}