@nyaruka/temba-components 0.156.15 → 0.156.17

package/src/flow/revision-summary.ts

@@ -0,0 +1,62 @@
+export interface RevisionChanges {
+  tags: string[];
+}
+
+const TAG_LABELS: Record<string, { label: string; order: number }> = {
+  metadata: { label: 'metadata', order: 0 },
+  nodes: { label: 'nodes', order: 1 },
+  routing: { label: 'routing', order: 2 },
+  actions: { label: 'actions', order: 3 },
+  stickies: { label: 'stickies', order: 5 },
+  layout: { label: 'layout', order: 6 }
+};
+
+function tagToLabel(tag: string): { label: string; order: number } | null {
+  if (Object.prototype.hasOwnProperty.call(TAG_LABELS, tag)) {
+    return TAG_LABELS[tag];
+  }
+  if (tag.startsWith('localization:')) {
+    return { label: 'translations', order: 4 };
+  }
+  return null;
+}
+
+export function labelsFor(
+  changes: RevisionChanges | null | undefined
+): Set<string> {
+  const result = new Set<string>();
+  for (const tag of changes?.tags || []) {
+    const entry = tagToLabel(tag);
+    if (entry) result.add(entry.label);
+  }
+  return result;
+}
+
+export function summarizeChanges(
+  changes: RevisionChanges | null | undefined
+): string {
+  if (!changes) return '';
+  const tags = changes.tags || [];
+  if (tags.length === 0) return '';
+
+  const orders = new Map<string, number>();
+  for (const tag of tags) {
+    const entry = tagToLabel(tag);
+    if (entry && !orders.has(entry.label)) {
+      orders.set(entry.label, entry.order);
+    }
+  }
+  if (orders.size === 0) return '';
+
+  const labels = Array.from(orders.entries())
+    .sort((a, b) => a[1] - b[1])
+    .map(([label]) => label);
+
+  return `Changed ${joinNaturally(labels)}`;
+}
+
+function joinNaturally(parts: string[]): string {
+  if (parts.length <= 1) return parts.join('');
+  if (parts.length === 2) return `${parts[0]} and ${parts[1]}`;
+  return `${parts.slice(0, -1).join(', ')}, and ${parts[parts.length - 1]}`;
+}

package/src/flow/types.ts

@@ -50,8 +50,7 @@ export const CONTEXT_MENU_SHORTCUTS: Record<FlowType, ContextMenuShortcut[]> = {
 export const Features = {
   AI: 'ai',
   AIRTIME: 'airtime',
-  LOCATIONS: 'locations',
-  AUTO_TRANSLATE: 'auto_translate'
+  LOCATIONS: 'locations'
 } as const;
 
 export type Feature = (typeof Features)[keyof typeof Features];

package/src/form/RichEditor.ts

@@ -97,6 +97,7 @@ export class RichEditor extends FieldElement {
         overflow-y: auto;
         min-height: var(--textarea-min-height, 100px);
         resize: none;
+        position: relative;
       }
 
       :host(:not([textarea])) {
@@ -119,6 +120,12 @@ export class RichEditor extends FieldElement {
         content: attr(data-placeholder);
         color: var(--color-placeholder, #999);
         pointer-events: none;
+        /* Take the placeholder out of flow so the caret sits at offset 0
+           visually, not after the placeholder text (Firefox honors flow
+           strictly here while Chrome happens to overlay the caret). */
+        position: absolute;
+        inset: 0;
+        padding: inherit;
       }
 
       /* Token styles (shared) */
@@ -302,6 +309,11 @@ export class RichEditor extends FieldElement {
 
     if (this.disableCompletion) {
       div.textContent = text || '';
+      if (text && text.endsWith('\n')) {
+        const br = document.createElement('br');
+        br.setAttribute('data-sentinel', '');
+        div.appendChild(br);
+      }
       return;
     }
 
@@ -359,6 +371,15 @@ export class RichEditor extends FieldElement {
     // Ensure there's at least an empty text node for cursor placement
     if (!text || text === '') {
       div.appendChild(document.createTextNode(''));
+    } else if (text.endsWith('\n')) {
+      // A trailing "\n" text node is collapsed by Firefox in contenteditable,
+      // making the empty final line invisible. A sentinel <br> forces the
+      // line break to render. The data-sentinel attribute distinguishes it
+      // from browser-inserted <br>s (which represent real newlines and must
+      // be preserved by the caret/text utilities).
+      const br = document.createElement('br');
+      br.setAttribute('data-sentinel', '');
+      div.appendChild(br);
     }
   }
 

package/web-test-runner.config.mjs

@@ -140,13 +140,13 @@ const checkScreenshot = async (filename, excluded, threshold) => {
   });
 };
 
-const wireScreenshots = async (page, context, wait, replaceScreenshots) => {
-  // clear out any past tests
-  const diffs = path.resolve(SCREENSHOTS, DIFF);
-  const tests = path.resolve(SCREENSHOTS, TEST);
+// clear out any past tests once per process — clearing per-page would race
+// with other concurrent pages that have already written test screenshots and
+// are about to read them back, causing intermittent ENOENT failures.
+rimraf.sync(path.resolve(SCREENSHOTS, DIFF));
+rimraf.sync(path.resolve(SCREENSHOTS, TEST));
 
-  rimraf.sync(diffs);
-  rimraf.sync(tests);
+const wireScreenshots = async (page, context, wait, replaceScreenshots) => {
 
   await page.exposeFunction(
     'matchPageSnapshot',
@@ -335,7 +335,7 @@ const wireScreenshots = async (page, context, wait, replaceScreenshots) => {
 
 export default {
   rootDir: './',
-  files: '**/test/**/*.test.ts',
+  files: ['**/test/**/*.test.ts', '!**/test/utils.test.ts'],
   nodeResolve: true,
   concurrency: 4,
   filterBrowserLogs(log) {