let5see/clawdbot
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix(doctor): avoid ack reaction migration without config (#1087)

Browse Source 
Thanks @YuriNachos.

Co-authored-by: Yuri Chukhlib <YuriNachos@users.noreply.github.com>
This commit is contained in:
Peter Steinberger
2026-01-17 18:07:06 +00:00
parent 1045b032a2
commit 50c8e74230
3 changed files with 13 additions and 22 deletions
Download Patch File Download Diff File Expand all files Collapse all files

27
src/commands/doctor-legacy-config.test.ts
View File

@@ -57,7 +57,7 @@ describe("normalizeLegacyConfigValues", () => {
]);
});
it("copies legacy ack reaction when whatsapp auth exists", () => {
it("does not add whatsapp config when only auth exists (issue #900)", () => {
const credsDir = path.join(tempOauthDir ?? "", "whatsapp", "default");
writeCreds(credsDir);
@@ -65,14 +65,11 @@ describe("normalizeLegacyConfigValues", () => {
messages: { ackReaction: "👀", ackReactionScope: "group-mentions" },
});
expect(res.config.channels?.whatsapp?.ackReaction).toEqual({
emoji: "👀",
direct: false,
group: "mentions",
});
expect(res.config.channels?.whatsapp).toBeUndefined();
expect(res.changes).toEqual([]);
});
it("copies legacy ack reaction when legacy auth exists", () => {
it("does not add whatsapp config when only legacy auth exists (issue #900)", () => {
const credsPath = path.join(tempOauthDir ?? "", "creds.json");
fs.writeFileSync(credsPath, JSON.stringify({ me: {} }));
@@ -80,14 +77,11 @@ describe("normalizeLegacyConfigValues", () => {
messages: { ackReaction: "👀", ackReactionScope: "group-mentions" },
});
expect(res.config.channels?.whatsapp?.ackReaction).toEqual({
emoji: "👀",
direct: false,
group: "mentions",
});
expect(res.config.channels?.whatsapp).toBeUndefined();
expect(res.changes).toEqual([]);
});
it("copies legacy ack reaction when non-default auth exists", () => {
it("does not add whatsapp config when only non-default auth exists (issue #900)", () => {
const credsDir = path.join(tempOauthDir ?? "", "whatsapp", "work");
writeCreds(credsDir);
@@ -95,11 +89,8 @@ describe("normalizeLegacyConfigValues", () => {
messages: { ackReaction: "👀", ackReactionScope: "group-mentions" },
});
expect(res.config.channels?.whatsapp?.ackReaction).toEqual({
emoji: "👀",
direct: false,
group: "mentions",
});
expect(res.config.channels?.whatsapp).toBeUndefined();
expect(res.changes).toEqual([]);
});
it("copies legacy ack reaction when authDir override exists", () => {