nodebb-plugin-onekite-calendar 1.0.4 → 1.0.5

package/lib/api.js

@@ -3,50 +3,14 @@
 const crypto = require('crypto');
 
 const meta = require.main.require('./src/meta');
+const emailer = require.main.require('./src/emailer');
 const nconf = require.main.require('nconf');
 const user = require.main.require('./src/user');
 const groups = require.main.require('./src/groups');
 const db = require.main.require('./src/database');
-// logger available if you want to debug locally; we avoid noisy logs in prod.
+const logger = require.main.require('./src/logger');
 
 const dbLayer = require('./db');
-const { sendEmail } = require('./email');
-const log = require('./log');
-
-// Ultra-perf: short-lived in-memory cache for the events feed.
-// We cache a "base" event list without per-user privileges, then decorate
-// it per request. This reduces DB load when many users/widgets refresh.
-const EVENTS_CACHE = new Map();
-const EVENTS_CACHE_TTL_MS = 30 * 1000;
-const EVENTS_CACHE_MAX = 50;
-
-function eventsCacheKey(startTs, endTs) {
-  // Keep the key stable and short.
-  return `${Math.max(0, Number(startTs) || 0)}:${Math.max(0, Number(endTs) || 0)}`;
-}
-
-function getEventsCache(key) {
-  const hit = EVENTS_CACHE.get(key);
-  if (!hit) return null;
-  if ((Date.now() - hit.ts) > EVENTS_CACHE_TTL_MS) {
-    EVENTS_CACHE.delete(key);
-    return null;
-  }
-  return hit.data;
-}
-
-function setEventsCache(key, data) {
-  EVENTS_CACHE.set(key, { ts: Date.now(), data });
-  if (EVENTS_CACHE.size > EVENTS_CACHE_MAX) {
-    // Drop oldest entries.
-    const entries = Array.from(EVENTS_CACHE.entries());
-    entries.sort((a, b) => (a[1]?.ts || 0) - (b[1]?.ts || 0));
-    const toDrop = Math.max(1, EVENTS_CACHE.size - EVENTS_CACHE_MAX);
-    for (let i = 0; i < toDrop; i++) {
-      EVENTS_CACHE.delete(entries[i][0]);
-    }
-  }
-}
 
 // Fast membership check without N calls to groups.isMember.
 // NodeBB's groups.getUserGroups([uid]) returns an array (per uid) of group objects.
@@ -88,7 +52,36 @@ function normalizeAllowedGroups(raw) {
 const helloasso = require('./helloasso');
 const discord = require('./discord');
 
-// Email helper is in lib/email.js (no logs here).
+// Email helper: NodeBB's Emailer signature differs across versions.
+// We try the common forms. Any failure is logged for debugging.
+async function sendEmail(template, toEmail, subject, data) {
+  if (!toEmail) return;
+  try {
+    // NodeBB core signature (historically):
+    //   Emailer.sendToEmail(template, email, language, params[, callback])
+    // Subject is not a positional arg; it must be injected (either by NodeBB itself
+    // or via filter:email.modify). We always pass it in params.subject.
+    const language = (meta && meta.config && (meta.config.defaultLang || meta.config.defaultLanguage)) || 'fr';
+    const params = Object.assign({}, data || {}, subject ? { subject } : {});
+    if (typeof emailer.sendToEmail === 'function') {
+      await emailer.sendToEmail(template, toEmail, language, params);
+      return;
+    }
+    // Fallback for older/unusual builds (rare)
+    if (typeof emailer.send === 'function') {
+      // Some builds accept (template, email, language, params)
+      if (emailer.send.length >= 4) {
+        await emailer.send(template, toEmail, language, params);
+        return;
+      }
+      // Some builds accept (template, email, params)
+      await emailer.send(template, toEmail, params);
+    }
+  } catch (err) {
+    // eslint-disable-next-line no-console
+    console.warn('[calendar-onekite] Failed to send email', { template, toEmail, err: String(err) });
+  }
+}
 
 function normalizeBaseUrl(meta) {
   // Prefer meta.config.url, fallback to nconf.get('url')
@@ -109,8 +102,7 @@ function normalizeCallbackUrl(configured, meta) {
   let url = (configured || '').trim();
   if (!url) {
     // Default webhook endpoint (recommended): namespaced under /plugins
-    // Prefer the new namespace; legacy endpoint remains available.
-    url = base ? `${base}/plugins/onekite-calendar/helloasso` : '';
+    url = base ? `${base}/plugins/calendar-onekite/helloasso` : '';
   }
   if (url && url.startsWith('/') && base) {
     url = `${base}${url}`;
@@ -336,7 +328,7 @@ api.getEvents = async function (req, res) {
   const startTs = toTs(req.query.start) || 0;
   const endTs = toTs(req.query.end) || (Date.now() + 365 * 24 * 3600 * 1000);
 
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const canMod = req.uid ? await canValidate(req.uid, settings) : false;
   const canSpecialCreate = req.uid ? await canCreateSpecial(req.uid, settings) : false;
   const canSpecialDelete = req.uid ? await canDeleteSpecial(req.uid, settings) : false;
@@ -344,89 +336,20 @@ api.getEvents = async function (req, res) {
   // Fetch a wider window because an event can start before the query range
   // and still overlap.
   const wideStart = Math.max(0, startTs - 366 * 24 * 3600 * 1000);
-
-  const cacheKey = eventsCacheKey(startTs, endTs);
-  let base = getEventsCache(cacheKey);
-  if (!base) {
-    const baseOut = [];
-
-    const ids = await dbLayer.listReservationIdsByStartRange(wideStart, endTs, 5000);
-    // Batch fetch = major perf win when there are many reservations.
-    const reservations = await dbLayer.getReservations(ids);
-    for (const r of (reservations || [])) {
-      if (!r) continue;
-      if (!['pending', 'awaiting_payment', 'paid'].includes(r.status)) continue;
-      const rStart = parseInt(r.start, 10);
-      const rEnd = parseInt(r.end, 10);
-      if (!(rStart < endTs && startTs < rEnd)) continue;
-
-      const evs = eventsFor(r);
-      const hasPay = (r.status === 'awaiting_payment' && r.paymentUrl && (/^https?:\/\//i).test(String(r.paymentUrl)));
-      for (const ev of (evs || [])) {
-        const p = ev.extendedProps || {};
-        baseOut.push({
-          id: ev.id,
-          title: ev.title,
-          backgroundColor: ev.backgroundColor,
-          borderColor: ev.borderColor,
-          textColor: ev.textColor,
-          allDay: ev.allDay,
-          start: ev.start,
-          end: ev.end,
-          extendedProps: {
-            type: 'reservation',
-            rid: p.rid,
-            status: p.status,
-            uid: p.uid,
-            _username: r.username ? String(r.username) : '',
-            _hasPayment: !!hasPay,
-          },
-        });
-      }
-    }
-
-    // Special events
-    try {
-      const specialIds = await dbLayer.listSpecialIdsByStartRange(wideStart, endTs, 5000);
-      const specials = await dbLayer.getSpecialEvents(specialIds);
-      for (const sev of (specials || [])) {
-        if (!sev) continue;
-        const sStart = parseInt(sev.start, 10);
-        const sEnd = parseInt(sev.end, 10);
-        if (!(sStart < endTs && startTs < sEnd)) continue;
-        const full = eventsForSpecial(sev);
-        baseOut.push({
-          id: full.id,
-          title: full.title,
-          allDay: full.allDay,
-          start: full.start,
-          end: full.end,
-          backgroundColor: full.backgroundColor,
-          borderColor: full.borderColor,
-          textColor: full.textColor,
-          extendedProps: {
-            type: 'special',
-            eid: sev.eid,
-            uid: sev.uid,
-            _username: sev.username ? String(sev.username) : '',
-          },
-        });
-      }
-    } catch (e) {
-      // ignore
-    }
-
-    base = baseOut;
-    setEventsCache(cacheKey, baseOut);
-  }
-
+  const ids = await dbLayer.listReservationIdsByStartRange(wideStart, endTs, 5000);
   const out = [];
-  const uidStr = (req.uid || req.uid === 0) ? String(req.uid) : '';
-  for (const ev of (base || [])) {
-    if (!ev) continue;
-    const p = ev.extendedProps || {};
-    if (p.type === 'reservation') {
-      const ownerOrMod = !!(p.uid && uidStr && String(p.uid) === uidStr) || canMod;
+  // Batch fetch = major perf win when there are many reservations.
+  const reservations = await dbLayer.getReservations(ids);
+  for (const r of (reservations || [])) {
+    if (!r) continue;
+    // Only show active statuses
+    if (!['pending', 'awaiting_payment', 'paid'].includes(r.status)) continue;
+    const rStart = parseInt(r.start, 10);
+    const rEnd = parseInt(r.end, 10);
+    if (!(rStart < endTs && startTs < rEnd)) continue; // overlap check
+    const evs = eventsFor(r);
+    for (const ev of evs) {
+      const p = ev.extendedProps || {};
       const minimal = {
         id: ev.id,
         title: ev.title,
@@ -444,30 +367,53 @@ api.getEvents = async function (req, res) {
           canModerate: canMod,
         },
       };
-      if (p._username && ownerOrMod) minimal.extendedProps.username = String(p._username);
-      if (p._hasPayment && ownerOrMod) minimal.extendedProps.hasPayment = true;
+      // Only expose username on the event list to owner/moderators.
+      if (r.username && ((req.uid && String(req.uid) === String(r.uid)) || canMod)) {
+        minimal.extendedProps.username = String(r.username);
+      }
+      // Let the UI decide if a "Payer" button might exist, without exposing the URL in list.
+      if (r.status === 'awaiting_payment' && r.paymentUrl && (/^https?:\/\//i).test(String(r.paymentUrl))) {
+        if ((req.uid && String(req.uid) === String(r.uid)) || canMod) {
+          minimal.extendedProps.hasPayment = true;
+        }
+      }
       out.push(minimal);
-    } else if (p.type === 'special') {
-      const ownerOrPriv = canMod || canSpecialDelete || (p.uid && uidStr && String(p.uid) === uidStr);
+    }
+  }
+
+  // Special events
+  try {
+    const specialIds = await dbLayer.listSpecialIdsByStartRange(wideStart, endTs, 5000);
+    const specials = await dbLayer.getSpecialEvents(specialIds);
+    for (const sev of (specials || [])) {
+      if (!sev) continue;
+      const sStart = parseInt(sev.start, 10);
+      const sEnd = parseInt(sev.end, 10);
+      if (!(sStart < endTs && startTs < sEnd)) continue;
+      const full = eventsForSpecial(sev);
       const minimal = {
-        id: ev.id,
-        title: ev.title,
-        allDay: ev.allDay,
-        start: ev.start,
-        end: ev.end,
-        backgroundColor: ev.backgroundColor,
-        borderColor: ev.borderColor,
-        textColor: ev.textColor,
+        id: full.id,
+        title: full.title,
+        allDay: full.allDay,
+        start: full.start,
+        end: full.end,
+        backgroundColor: full.backgroundColor,
+        borderColor: full.borderColor,
+        textColor: full.textColor,
         extendedProps: {
           type: 'special',
-          eid: p.eid,
+          eid: sev.eid,
           canCreateSpecial: canSpecialCreate,
           canDeleteSpecial: canSpecialDelete,
         },
       };
-      if (p._username && ownerOrPriv) minimal.extendedProps.username = String(p._username);
+      if (sev.username && (canMod || canSpecialDelete || (req.uid && String(req.uid) === String(sev.uid)))) {
+        minimal.extendedProps.username = String(sev.username);
+      }
       out.push(minimal);
     }
+  } catch (e) {
+    // ignore
   }
 
   // Stable ordering -> stable ETag
@@ -482,8 +428,7 @@ api.getEvents = async function (req, res) {
 
   const etag = computeEtag(out);
   res.setHeader('ETag', etag);
-  // Short caching + ETag revalidation keeps the feed snappy and reduces bandwidth.
-  res.setHeader('Cache-Control', 'private, max-age=15, must-revalidate');
+  res.setHeader('Cache-Control', 'private, max-age=0, must-revalidate');
   if (String(req.headers['if-none-match'] || '') === etag) {
     return res.status(304).end();
   }
@@ -495,7 +440,7 @@ api.getReservationDetails = async function (req, res) {
   const uid = req.uid;
   if (!uid) return res.status(401).json({ error: 'not-logged-in' });
 
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const canMod = await canValidate(uid, settings);
 
   const rid = String(req.params.rid || '').trim();
@@ -537,7 +482,7 @@ api.getSpecialEventDetails = async function (req, res) {
   const uid = req.uid;
   if (!uid) return res.status(401).json({ error: 'not-logged-in' });
 
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const canMod = await canValidate(uid, settings);
   const canSpecialDelete = await canDeleteSpecial(uid, settings);
 
@@ -566,7 +511,7 @@ api.getSpecialEventDetails = async function (req, res) {
 };
 
 api.getCapabilities = async function (req, res) {
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const uid = req.uid || 0;
   const canMod = uid ? await canValidate(uid, settings) : false;
   res.json({
@@ -577,7 +522,7 @@ api.getCapabilities = async function (req, res) {
 };
 
 api.createSpecialEvent = async function (req, res) {
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   if (!req.uid) return res.status(401).json({ error: 'not-logged-in' });
   const ok = await canCreateSpecial(req.uid, settings);
   if (!ok) return res.status(403).json({ error: 'not-allowed' });
@@ -613,7 +558,7 @@ api.createSpecialEvent = async function (req, res) {
 };
 
 api.deleteSpecialEvent = async function (req, res) {
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   if (!req.uid) return res.status(401).json({ error: 'not-logged-in' });
   const ok = await canDeleteSpecial(req.uid, settings);
   if (!ok) return res.status(403).json({ error: 'not-allowed' });
@@ -624,7 +569,7 @@ api.deleteSpecialEvent = async function (req, res) {
 };
 
 api.getItems = async function (req, res) {
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
 
   const env = settings.helloassoEnv || 'prod';
   const token = await helloasso.getAccessToken({
@@ -665,7 +610,7 @@ api.createReservation = async function (req, res) {
   const uid = req.uid;
   if (!uid) return res.status(401).json({ error: 'not-logged-in' });
 
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const startPreview = toTs(req.body.start);
   const ok = await canRequest(uid, settings, startPreview);
   if (!ok) return res.status(403).json({ error: 'not-allowed' });
@@ -762,7 +707,7 @@ api.createReservation = async function (req, res) {
 
         for (const md of (usersData || [])) {
           if (md && md.email) {
-            await sendEmail('onekite-calendar_pending', md.email, 'Location matériel - Demande de réservation', {
+            await sendEmail('calendar-onekite_pending', md.email, 'Location matériel - Demande de réservation', {
               username: md.username,
               requester: requester.username,
               itemName: itemsLabel,
@@ -777,7 +722,7 @@ api.createReservation = async function (req, res) {
       }
     }
   } catch (e) {
-    log.warn('Failed to send pending email', e && e.message ? e.message : e);
+    console.warn('[calendar-onekite] Failed to send pending email', e && e.message ? e.message : e);
   }
 
   // Discord webhook (optional)
@@ -801,7 +746,7 @@ api.createReservation = async function (req, res) {
 api.approveReservation = async function (req, res) {
   const uid = req.uid;
   if (!uid) return res.status(401).json({ error: 'not-logged-in' });
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const ok = await canValidate(uid, settings);
   if (!ok) return res.status(403).json({ error: 'not-allowed' });
 
@@ -828,7 +773,7 @@ api.approveReservation = async function (req, res) {
   }
   // Create HelloAsso payment link on validation
   try {
-    const settings2 = await meta.settings.get('onekite-calendar');
+    const settings2 = await meta.settings.get('calendar-onekite');
     const token = await helloasso.getAccessToken({ env: settings2.helloassoEnv || 'prod', clientId: settings2.helloassoClientId, clientSecret: settings2.helloassoClientSecret });
     const payer = await user.getUserFields(r.uid, ['email']);
     const year = yearFromTs(r.start);
@@ -843,7 +788,7 @@ api.approveReservation = async function (req, res) {
       totalAmount: (() => {
         const cents = Math.max(0, Math.round((Number(r.total) || 0) * 100));
         if (!cents) {
-          log.warn('HelloAsso totalAmount is 0 (approve API)', { rid, total: r.total });
+          console.warn('[calendar-onekite] HelloAsso totalAmount is 0 (approve API)', { rid, total: r.total });
         }
         return cents;
       })(),
@@ -852,7 +797,7 @@ api.approveReservation = async function (req, res) {
       // Can be overridden via ACP setting `helloassoCallbackUrl`.
       callbackUrl: normalizeReturnUrl(meta),
       webhookUrl: normalizeCallbackUrl(settings2.helloassoCallbackUrl, meta),
-      itemName: 'Réservation matériel Onekite',
+      itemName: 'Réservation matériel OneKite',
       containsDonation: false,
       metadata: { reservationId: String(rid) },
     });
@@ -864,7 +809,7 @@ api.approveReservation = async function (req, res) {
         r.checkoutIntentId = checkoutIntentId;
       }
     } else {
-      log.warn('HelloAsso payment link not created (approve API)', { rid });
+      console.warn('[calendar-onekite] HelloAsso payment link not created (approve API)', { rid });
     }
   } catch (e) {
     // ignore payment link errors, admin can retry
@@ -880,7 +825,7 @@ api.approveReservation = async function (req, res) {
     const mapUrl = (Number.isFinite(latNum) && Number.isFinite(lonNum))
       ? `https://www.openstreetmap.org/?mlat=${encodeURIComponent(String(latNum))}&mlon=${encodeURIComponent(String(lonNum))}#map=18/${encodeURIComponent(String(latNum))}/${encodeURIComponent(String(lonNum))}`
       : '';
-    await sendEmail('onekite-calendar_approved', requester.email, 'Location matériel - Réservation validée', {
+    await sendEmail('calendar-onekite_approved', requester.email, 'Location matériel - Réservation validée', {
       username: requester.username,
       itemName: (Array.isArray(r.itemNames) ? r.itemNames.join(', ') : (r.itemName || '')),
         itemNames: (Array.isArray(r.itemNames) ? r.itemNames : (r.itemName ? [r.itemName] : [])),
@@ -904,7 +849,7 @@ api.approveReservation = async function (req, res) {
 api.refuseReservation = async function (req, res) {
   const uid = req.uid;
   if (!uid) return res.status(401).json({ error: 'not-logged-in' });
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const ok = await canValidate(uid, settings);
   if (!ok) return res.status(403).json({ error: 'not-allowed' });
 
@@ -919,7 +864,7 @@ api.refuseReservation = async function (req, res) {
 
   const requester = await user.getUserFields(r.uid, ['username', 'email']);
   if (requester && requester.email) {
-    await sendEmail('onekite-calendar_refused', requester.email, 'Location matériel - Demande de réservation', {
+    await sendEmail('calendar-onekite_refused', requester.email, 'Location matériel - Demande de réservation', {
       username: requester.username,
       itemName: (Array.isArray(r.itemNames) ? r.itemNames.join(', ') : (r.itemName || '')),
         itemNames: (Array.isArray(r.itemNames) ? r.itemNames : (r.itemName ? [r.itemName] : [])),
@@ -939,7 +884,7 @@ api.cancelReservation = async function (req, res) {
   const uid = req.uid;
   if (!uid) return res.status(401).json({ error: 'not-logged-in' });
 
-  const settings = await meta.settings.get('onekite-calendar');
+  const settings = await meta.settings.get('calendar-onekite');
   const rid = String(req.params.rid || '').trim();
   if (!rid) return res.status(400).json({ error: 'missing-rid' });
 

package/lib/controllers.js

@@ -3,8 +3,8 @@
 const controllers = {};
 
 controllers.renderCalendar = async function (req, res) {
-  res.render('onekite-calendar', {
-    title: 'Calendrier',
+  res.render('calendar-onekite', {
+    title: 'Calendar',
   });
 };
 

package/lib/db.js

@@ -2,13 +2,13 @@
 
 const db = require.main.require('./src/database');
 
-const KEY_ZSET = 'onekite-calendar:reservations';
-const KEY_OBJ = (rid) => `onekite-calendar:reservation:${rid}`;
-const KEY_CHECKOUT_INTENT_TO_RID = 'onekite-calendar:helloasso:checkoutIntentToRid';
+const KEY_ZSET = 'calendar-onekite:reservations';
+const KEY_OBJ = (rid) => `calendar-onekite:reservation:${rid}`;
+const KEY_CHECKOUT_INTENT_TO_RID = 'calendar-onekite:helloasso:checkoutIntentToRid';
 
 // Special events (non-reservation events shown in a different colour)
-const KEY_SPECIAL_ZSET = 'onekite-calendar:special';
-const KEY_SPECIAL_OBJ = (eid) => `onekite-calendar:special:${eid}`;
+const KEY_SPECIAL_ZSET = 'calendar-onekite:special';
+const KEY_SPECIAL_OBJ = (eid) => `calendar-onekite:special:${eid}`;
 
 // Helpers
 function reservationKey(rid) {

package/lib/discord.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const https = require('https');
-const log = require('./log');
 const { URL } = require('url');
 
 function isEnabled(v, defaultValue) {
@@ -37,7 +36,7 @@ function postWebhook(webhookUrl, payload) {
         headers: {
           'Content-Type': 'application/json',
           'Content-Length': body.length,
-          'User-Agent': 'nodebb-plugin-onekite-calendar',
+          'User-Agent': 'nodebb-plugin-calendar-onekite',
         },
       }, (res) => {
         const ok = res.statusCode && res.statusCode >= 200 && res.statusCode < 300;
@@ -89,7 +88,7 @@ function buildReservationMessage(kind, reservation) {
 function buildWebhookPayload(kind, reservation) {
   // Discord "regroupe" visuellement les messages consécutifs d'un même auteur.
   // En utilisant un username différent par action, on obtient un message bien distinct.
-  const webhookUsername = kind === 'paid' ? 'Onekite • Paiement' : 'Onekite • Réservation';
+  const webhookUsername = kind === 'paid' ? 'OneKite • Paiement' : 'OneKite • Réservation';
 
   const calUrl = 'https://www.onekite.com/calendar';
   const username = reservation && reservation.username ? String(reservation.username) : '';
@@ -125,7 +124,7 @@ function buildWebhookPayload(kind, reservation) {
           ? 'Un paiement a été reçu pour une réservation.'
           : 'Une nouvelle demande de réservation a été créée.',
         fields,
-        footer: { text: 'Onekite • Calendrier' },
+        footer: { text: 'OneKite • Calendrier' },
         timestamp: new Date().toISOString(),
       },
     ],
@@ -141,7 +140,7 @@ async function notifyReservationRequested(settings, reservation) {
     await postWebhook(url, buildWebhookPayload('request', reservation));
   } catch (e) {
     // eslint-disable-next-line no-console
-    log.warn('Discord webhook failed (request)', e && e.message ? e.message : String(e));
+    console.warn('[calendar-onekite] Discord webhook failed (request)', e && e.message ? e.message : String(e));
   }
 }
 
@@ -154,7 +153,7 @@ async function notifyPaymentReceived(settings, reservation) {
     await postWebhook(url, buildWebhookPayload('paid', reservation));
   } catch (e) {
     // eslint-disable-next-line no-console
-    log.warn('Discord webhook failed (paid)', e && e.message ? e.message : String(e));
+    console.warn('[calendar-onekite] Discord webhook failed (paid)', e && e.message ? e.message : String(e));
   }
 }
 

package/lib/helloasso.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const https = require('https');
-const log = require('./log');
 
 function requestJson(method, url, headers = {}, bodyObj = null) {
   return new Promise((resolve) => {
@@ -285,11 +284,11 @@ async function listCatalogItems({ env, token, organizationSlug, formType, formSl
 async function createCheckoutIntent({ env, token, organizationSlug, formType, formSlug, totalAmount, payerEmail, callbackUrl, webhookUrl, itemName, containsDonation, metadata }) {
   if (!token || !organizationSlug) return null;
   if (!callbackUrl || !/^https?:\/\//i.test(String(callbackUrl))) {
-    log.warn('HelloAsso invalid return/back/error URL', { callbackUrl });
+    console.warn('[calendar-onekite] HelloAsso invalid return/back/error URL', { callbackUrl });
     return null;
   }
   if (webhookUrl && !/^https?:\/\//i.test(String(webhookUrl))) {
-    log.warn('HelloAsso invalid webhook URL', { webhookUrl });
+    console.warn('[calendar-onekite] HelloAsso invalid webhook URL', { webhookUrl });
   }
   // Checkout intents are created at organization level.
   const url = `${baseUrl(env)}/v5/organizations/${encodeURIComponent(organizationSlug)}/checkout-intents`;
@@ -312,7 +311,7 @@ async function createCheckoutIntent({ env, token, organizationSlug, formType, fo
   // Log the error payload to help diagnose configuration issues (slug, env, urls, amount, etc.)
   try {
     // eslint-disable-next-line no-console
-    log.warn('HelloAsso checkout-intent failed', { status, json });
+    console.warn('[calendar-onekite] HelloAsso checkout-intent failed', { status, json });
   } catch (e) { /* ignore */ }
   return null;
 }

package/lib/helloassoWebhook.js

@@ -1,7 +1,6 @@
 'use strict';
 
 const crypto = require('crypto');
-const log = require('./log');
 
 const db = require.main.require('./src/database');
 const meta = require.main.require('./src/meta');
@@ -20,10 +19,10 @@ const dbLayer = require('./db');
 const helloasso = require('./helloasso');
 const discord = require('./discord');
 
-const SETTINGS_KEY = 'onekite-calendar';
+const SETTINGS_KEY = 'calendar-onekite';
 
 // Replay protection: store processed payment ids.
-const PROCESSED_KEY = 'onekite-calendar:helloasso:processedPayments';
+const PROCESSED_KEY = 'calendar-onekite:helloasso:processedPayments';
 
 async function sendEmail(template, toEmail, subject, data) {
   const uidFromData = data && Number.isInteger(data.uid) ? data.uid : null;
@@ -68,7 +67,7 @@ async function sendEmail(template, toEmail, subject, data) {
     // fall back to sendToEmail only.
   } catch (err) {
     // eslint-disable-next-line no-console
-    log.warn('Failed to send email (webhook)', { template, toEmail, err: String(err && err.message || err) });
+    console.warn('[calendar-onekite] Failed to send email (webhook)', { template, toEmail, err: String(err && err.message || err) });
   }
 }
 
@@ -282,7 +281,7 @@ async function handler(req, res, next) {
 
     if (allowedIps.length && clientIp && !allowedIps.includes(clientIp)) {
       // eslint-disable-next-line no-console
-      log.warn('HelloAsso webhook blocked by IP allowlist', { ip: clientIp, allowed: allowedIps });
+      console.warn('[calendar-onekite] HelloAsso webhook blocked by IP allowlist', { ip: clientIp, allowed: allowedIps });
       return res.status(403).json({ ok: false, error: 'ip-not-allowed' });
     }
 
@@ -325,7 +324,7 @@ async function handler(req, res, next) {
     }
     if (!resolvedRid) {
       // eslint-disable-next-line no-console
-      log.warn('HelloAsso webhook missing reservationId in metadata', { eventType: payload && payload.eventType, paymentId, checkoutIntentId });
+      console.warn('[calendar-onekite] HelloAsso webhook missing reservationId in metadata', { eventType: payload && payload.eventType, paymentId, checkoutIntentId });
       // Do NOT mark as processed: if metadata/config is fixed later, a manual replay may be possible.
       return res.json({ ok: true, processed: false, missingReservationId: true });
     }
@@ -348,14 +347,14 @@ async function handler(req, res, next) {
     // Real-time notify: refresh calendars for all viewers (owner + validators/admins)
     try {
       if (io && io.sockets && typeof io.sockets.emit === 'function') {
-        io.sockets.emit('event:onekite-calendar.reservationUpdated', { rid: r.rid, status: r.status });
+        io.sockets.emit('event:calendar-onekite.reservationUpdated', { rid: r.rid, status: r.status });
       }
     } catch (e) {}
 
     // Notify requester
     const requester = await user.getUserFields(r.uid, ['username', 'email']);
     if (requester && requester.email) {
-      await sendEmail('onekite-calendar_paid', requester.email, 'Location matériel - Paiement reçu', {
+      await sendEmail('calendar-onekite_paid', requester.email, 'Location matériel - Paiement reçu', {
         uid: parseInt(r.uid, 10),
         username: requester.username,
         itemName: (Array.isArray(r.itemNames) ? r.itemNames.join(', ') : (r.itemName || '')),