fix: deliver receipts

Author: arespawn

docs/README.md

@@ -52,6 +52,7 @@ This repository currently pins the published npm package `@whiskeysockets/bailey
 - Fresh WhatsApp startup uses the Android Baileys browser profile by default for view-once media support. The `/pairwithcode` flow can temporarily restart into `macos-chrome` because pairing codes are more reliable with that profile.
 - When WA2DC sees registered WhatsApp auth without a matching saved browser-profile marker, it clears only the WhatsApp auth state and asks you to pair again. This prevents older sessions from being reused blindly under a different browser profile.
 - WA2DC also keeps rc11 reconnects on the initial-sync wait path when history sync is enabled, but skips Baileys startup buffering when history sync is disabled; this avoids packaged-startup heap spikes seen after switching existing sessions to the Android browser profile.
+- WA2DC keeps `markOnlineOnConnect: false` so your phone can keep receiving WhatsApp notifications, but patches rc11 so inbound non-newsletter messages still send normal delivered receipts instead of the downgraded `inactive` receipt.
 - WA2DC logs bounded WhatsApp startup memory probes and patches rc11 LID migration/tctoken pruning paths, so large auth stores do not trigger single huge heap allocations while the socket is coming online.
 - The Signal auth store seeds the newly required `lid-mapping`, `tctoken`, `device-list`, and `device-index` namespaces so rc11 can write those blobs safely.
 

docs/dev/runtime-and-layout.md

@@ -25,7 +25,7 @@ WhatsApp startup guardrails:
 - Do not eagerly call `groupFetchAllParticipating()` on WhatsApp reconnect or `/resync`. Group metadata is refreshed through live group events and `/resync` uses a lightweight `@g.us` participating-groups query that does not request every participant roster/description; full all-groups fetches can allocate very large Baileys response structures after pairing.
 - Pass Baileys a bounded logger wrapper instead of the root pino logger. Baileys errors can include bundled `data:text/javascript;base64...` stack traces and binary payloads; keep those summarized so `logs.txt` and `terminal.log` stay useful and do not drive heap pressure.
 - Keep WhatsApp startup memory probes around socket creation, WA connection, real initial buffer flushes, and LID session migration. These probes are intentionally small structured logs used to diagnose packaged OOM rollbacks.
-- The Baileys rc11 postinstall patch skips startup event buffering when history sync is disabled, avoids buffer-and-immediate-flush on the disabled-history path, and bounds tctoken pruning so large auth stores do not trigger single huge heap allocations while the socket is coming online.
+- The Baileys rc11 postinstall patch skips startup event buffering when history sync is disabled, avoids buffer-and-immediate-flush on the disabled-history path, preserves delivered receipts for inbound messages while WA2DC stays unavailable, and bounds tctoken pruning so large auth stores do not trigger single huge heap allocations while the socket is coming online.
 
 ## Developer quick start
 

docs/dev/testing-and-release.md

@@ -30,7 +30,7 @@ Release pipeline builds packaged binaries from a pkg-safe CJS runtime bundle:
 - release builds publish a signed `${binary}.runtime.tar.gz` archive for each packaged binary so `/update` can refresh the sidecar automatically
 - packaged startup may download that signed runtime archive on demand when a packaged install is missing `runtime/`
 - runtime may branch on `process.pkg` for packaged-vs-source behavior
-- `postinstall` patches Baileys rc11 for the Android browser profile, disabled-history startup buffering, LID migration probes, and bounded tctoken pruning; release builds must run after that patch has been applied
+- `postinstall` patches Baileys rc11 for the Android browser profile, disabled-history startup buffering, inbound delivered receipts while unavailable, LID migration probes, and bounded tctoken pruning; release builds must run after that patch has been applied
 
 ## Packaging-safe dependency rules
 

package-lock.json

@@ -1,6 +1,6 @@
 {
 	"name": "wa2dc",
-	"version": "2.4.0-alpha.2",
+	"version": "2.4.0-alpha.3",
 	"description": "WhatsAppToDiscord is a Discord bot that uses WhatsApp Web as a bridge between Discord and WhatsApp",
 	"type": "module",
 	"main": "src/runner.js",

package.json

@@ -261,6 +261,15 @@ const boundedTcTokenPruneAfter = `    const TC_TOKEN_PRUNE_BATCH_SIZE = 250;
         }
     }`;
 
+const preserveDeliveredReceiptWhileUnavailableBefore = `                        else if (!sendActiveReceipts) {
+                            type = 'inactive';
+                        }`;
+
+const preserveDeliveredReceiptWhileUnavailableAfter = `                        else if (!sendActiveReceipts) {
+                            // WA2DC stays unavailable on connect so the phone keeps getting notifications,
+                            // but inbound messages should still acknowledge delivery to the sender.
+                        }`;
+
 const replacements = [
 	{
 		file: "lib/Utils/browser-utils.js",
@@ -370,6 +379,12 @@ const replacements = [
 		before: boundedTcTokenPruneBefore,
 		after: boundedTcTokenPruneAfter,
 	},
+	{
+		file: "lib/Socket/messages-recv.js",
+		marker: "WA2DC stays unavailable on connect so the phone keeps getting notifications",
+		before: preserveDeliveredReceiptWhileUnavailableBefore,
+		after: preserveDeliveredReceiptWhileUnavailableAfter,
+	},
 ];
 
 const replaceOnce = async ({ file, marker, before, after }) => {

scripts/patchBaileysAndroidBrowser.js

@@ -106,3 +106,23 @@ test("Baileys rc11 tctoken prune is bounded during startup", () => {
 		/const allTokens = await authState\.keys\.get\('tctoken', jids\)/u,
 	);
 });
+
+test("Baileys rc11 preserves delivered receipts while WA2DC stays unavailable", () => {
+	const messagesRecvSource = fs.readFileSync(
+		"node_modules/@whiskeysockets/baileys/lib/Socket/messages-recv.js",
+		"utf8",
+	);
+
+	assert.match(
+		messagesRecvSource,
+		/WA2DC stays unavailable on connect so the phone keeps getting notifications/u,
+	);
+	assert.doesNotMatch(
+		messagesRecvSource,
+		/else if \(!sendActiveReceipts\) \{\s*type = 'inactive';\s*\}/u,
+	);
+	assert.match(
+		messagesRecvSource,
+		/await sendReceipt\(msg\.key\.remoteJid, participant, \[msg\.key\.id\], type\)/u,
+	);
+});