Ayuriel/openclaw
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Fix LINE doctor warnings for file-backed credentials

Browse Source
This commit is contained in:
xaeon2026 2026-03-09 09:50:45 -04:00
parent 14bbcad169
commit 73e641c1b4
2 changed files with 55 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

51
extensions/line/src/channel.startup.test.ts
View File

@ -129,3 +129,54 @@ describe("linePlugin gateway.startAccount", () => {
await task;
});
});
describe("linePlugin status", () => {
it("does not report missing token or secret when snapshot came from file-backed config", async () => {
const snapshot = await linePlugin.status?.buildAccountSnapshot?.({
account: {
accountId: "default",
name: "Default",
enabled: true,
channelAccessToken: "token-from-file",
channelSecret: "secret-from-file",
tokenSource: "file",
config: {} as ResolvedLineAccount["config"],
} as never,
cfg: {} as OpenClawConfig,
runtime: undefined,
probe: undefined,
audit: undefined,
});
expect(snapshot?.configured).toBe(true);
expect(linePlugin.status?.collectStatusIssues?.([snapshot as never])).toEqual([]);
});
it("keeps per-field warnings when only one credential is missing", async () => {
const snapshot = await linePlugin.status?.buildAccountSnapshot?.({
account: {
accountId: "default",
name: "Default",
enabled: true,
channelAccessToken: " ",
channelSecret: "secret-from-file",
tokenSource: "file",
config: {} as ResolvedLineAccount["config"],
} as never,
cfg: {} as OpenClawConfig,
runtime: undefined,
probe: undefined,
audit: undefined,
});
expect(snapshot?.configured).toBe(false);
expect(linePlugin.status?.collectStatusIssues?.([snapshot as never])).toEqual([
{
channel: "line",
accountId: "default",
kind: "config",
message: "LINE channel access token not configured",
},
]);
});
});

6
extensions/line/src/channel.ts
View File

@ -569,7 +569,7 @@ export const linePlugin: ChannelPlugin<ResolvedLineAccount> = {
const issues: ChannelStatusIssue[] = [];
for (const account of accounts) {
const accountId = account.accountId ?? DEFAULT_ACCOUNT_ID;
if (!account.channelAccessToken?.trim()) {
if (account.channelAccessTokenConfigured === false) {
issues.push({
channel: "line",
accountId,
@ -577,7 +577,7 @@ export const linePlugin: ChannelPlugin<ResolvedLineAccount> = {
message: "LINE channel access token not configured",
});
}
if (!account.channelSecret?.trim()) {
if (account.channelSecretConfigured === false) {
issues.push({
channel: "line",
accountId,
@ -605,6 +605,8 @@ export const linePlugin: ChannelPlugin<ResolvedLineAccount> = {
});
return {
...base,
channelAccessTokenConfigured: Boolean(account.channelAccessToken?.trim()),
channelSecretConfigured: Boolean(account.channelSecret?.trim()),
tokenSource: account.tokenSource,
mode: "webhook",
};