@mikitasazan/notify 1.10.1 → 1.11.0

package/dist/lint.js

@@ -31,7 +31,7 @@ const NOT_A_NAME = new Set([
     'null'
 ]);
 /** The five words the third tag is allowed to be, and there is no sixth. */
-const OUTCOMES = new Set(['ok', 'fail', 'off', 'unknown', 'news']);
+const OUTCOMES = new Set(['ok', 'fail', 'off', 'unknown', 'info']);
 /** Labels the template retired. Each one used to say what a neighbour said. */
 const RETIRED = ['Title', 'Number', 'State', 'Via', 'Check', 'Logs', 'Task', 'Id', 'Period'];
 /**
@@ -46,7 +46,7 @@ export const lintCard = (html) => {
         found.push(`line 1 is "${rows[0] ?? ''}" — it must be exactly three tags`);
     }
     else if (!OUTCOMES.has(tags[2].slice(1))) {
-        found.push(`the outcome tag is "${tags[2]}" — the vocabulary is ok, fail, off, unknown, news`);
+        found.push(`the outcome tag is "${tags[2]}" — the vocabulary is ok, fail, off, unknown, info`);
     }
     if (tags[1] === '#' || tags[1] === '#_') {
         found.push('the instance tag is empty — it groups nothing and pairs with nothing');
@@ -98,5 +98,17 @@ export const lintCard = (html) => {
             break;
         }
     }
+    // A red card must say where to look. Here — and only here — that can be
+    // judged completely: a log row, a command to run, or any address at all,
+    // including the one riding on line 2 where the standard puts it. Read back
+    // from Telegram the addresses are gone, so the same rule there accused a
+    // deploy card whose own second line was the link to the failed run.
+    // `#fail` only. A task that has gone silent is `#unknown`, and it has no log
+    // by definition — it never ran. Demanding one there would be demanding a
+    // thing that cannot exist.
+    const broke = (rows[0] ?? '').includes('#fail');
+    if (broke && !html.includes('<b>Log:</b>') && !html.includes('<b>To do:</b>') && !html.includes('<a href=')) {
+        found.push('a red card with no log, no command and no link — nowhere to look');
+    }
     return found;
 };

package/dist/render.d.ts

@@ -68,13 +68,13 @@ export declare const eventKey: (e: NotifyEvent) => string;
 /** Every icon the package can print — the key space of the outcome table. */
 type Icon = (typeof ICON)[keyof typeof ICON];
 /** The five words the third tag is allowed to be, and there is no sixth. */
-type OutcomeTag = 'ok' | 'fail' | 'off' | 'unknown' | 'news';
+type OutcomeTag = 'ok' | 'fail' | 'off' | 'unknown' | 'info';
 /**
  * The type is `Record` over EVERY icon, not over `string`. A new icon added to
  * `ICON` without a word here now fails to compile. Under the old loose type it
- * fell through a `?? 'news'` default instead: a card whose outcome nobody had
+ * fell through a `?? 'info'` default instead: a card whose outcome nobody had
  * decided was indistinguishable from a card that is genuinely just news, and
- * nothing anywhere went red. `news` is therefore written out for each icon
+ * nothing anywhere went red. `info` is therefore written out for each icon
  * that means it, never left to a fallback.
  */
 export declare const OUTCOME_TAG: Readonly<Record<Icon, OutcomeTag>>;

package/dist/render.js

@@ -694,9 +694,9 @@ export const eventKey = (e) => {
 /**
  * The type is `Record` over EVERY icon, not over `string`. A new icon added to
  * `ICON` without a word here now fails to compile. Under the old loose type it
- * fell through a `?? 'news'` default instead: a card whose outcome nobody had
+ * fell through a `?? 'info'` default instead: a card whose outcome nobody had
  * decided was indistinguishable from a card that is genuinely just news, and
- * nothing anywhere went red. `news` is therefore written out for each icon
+ * nothing anywhere went red. `info` is therefore written out for each icon
  * that means it, never left to a fallback.
  */
 export const OUTCOME_TAG = {
@@ -707,12 +707,14 @@ export const OUTCOME_TAG = {
     [ICON.ok]: 'ok',
     [ICON.landed]: 'ok',
     [ICON.approved]: 'ok',
-    // Something happened; no verdict was passed on it.
-    [ICON.fresh]: 'news',
-    [ICON.taken]: 'news',
-    [ICON.discarded]: 'news',
-    [ICON.changes]: 'news',
-    [ICON.info]: 'news'
+    // Something happened; no verdict was passed on it. The word is `info`, not
+    // `news`: the icon on every one of these cards is ℹ️, and the owner read
+    // `#news` under it and asked what news meant. One thing wore two words.
+    [ICON.fresh]: 'info',
+    [ICON.taken]: 'info',
+    [ICON.discarded]: 'info',
+    [ICON.changes]: 'info',
+    [ICON.info]: 'info'
 };
 export const outcomeTag = (e) => OUTCOME_TAG[iconFor(e)];
 const tagsLine = (e) => `#${TYPE_TAG[e.type]} #${esc(eventKey(e))} #${outcomeTag(e)}`;

package/dist/trend.d.ts

@@ -10,11 +10,15 @@
  * So the shape is not a sender's business any more. It lives here, one
  * implementation, and every report calls it:
  *
- *   trend(210, 207)        → '210 ▲3'
- *   trend(202, 207)        → '202 ▼5'
- *   trend(0, 0)            → '0 ='
+ *   trend(210, 207)        → '210 / 207 ▲3'
+ *   trend(202, 207)        → '202 / 207 ▼5'
+ *   trend(0, 0)            → '0 / 0 ='
  *   trend(37)              → '37'          nothing to compare to, so no mark
- *   trend(4.4, 3.6, '%')   → '4.4% ▲0.8'   the mark is in the unit left of it
+ *   trend(4.4, 3.6, '%')   → '4.4% / 3.6% ▲0.8'
+ *
+ * Both numbers are printed, now first and before first. The owner read
+ * `51 ▲5` and asked what the 5 was — the new value or the old one. Neither: it
+ * was the distance between two numbers, one of which the card never showed.
  *
  * The rule the owner asked for, in one line: where there is data to compare
  * against, the arrow is printed; where there is none, nothing is printed —

package/dist/trend.js

@@ -10,11 +10,15 @@
  * So the shape is not a sender's business any more. It lives here, one
  * implementation, and every report calls it:
  *
- *   trend(210, 207)        → '210 ▲3'
- *   trend(202, 207)        → '202 ▼5'
- *   trend(0, 0)            → '0 ='
+ *   trend(210, 207)        → '210 / 207 ▲3'
+ *   trend(202, 207)        → '202 / 207 ▼5'
+ *   trend(0, 0)            → '0 / 0 ='
  *   trend(37)              → '37'          nothing to compare to, so no mark
- *   trend(4.4, 3.6, '%')   → '4.4% ▲0.8'   the mark is in the unit left of it
+ *   trend(4.4, 3.6, '%')   → '4.4% / 3.6% ▲0.8'
+ *
+ * Both numbers are printed, now first and before first. The owner read
+ * `51 ▲5` and asked what the 5 was — the new value or the old one. Neither: it
+ * was the distance between two numbers, one of which the card never showed.
  *
  * The rule the owner asked for, in one line: where there is data to compare
  * against, the arrow is printed; where there is none, nothing is printed —
@@ -32,8 +36,9 @@ export const trend = (now, was, unit = '') => {
     if (was === undefined || !Number.isFinite(was)) {
         return head;
     }
+    const pair = `${head} / ${fmt(was)}${unit}`;
     if (same(now, was)) {
-        return `${head} =`;
+        return `${pair} =`;
     }
-    return now > was ? `${head} ▲${fmt(now - was)}` : `${head} ▼${fmt(was - now)}`;
+    return now > was ? `${pair} ▲${fmt(now - was)}` : `${pair} ▼${fmt(was - now)}`;
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mikitasazan/notify",
-  "version": "1.10.1",
+  "version": "1.11.0",
   "description": "Единая типизированная отправка Telegram-уведомлений (форум-темы, маршрутизация, ретраи) для всех проектов",
   "type": "module",
   "license": "MIT",