fix(slack): skip webhook registration when integration token is gone

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code

[KrisBraun]

Problem

Queue/error-tracking was flooded with retried failures:

No integration found for authorization slack:<actorId>
outcome: failure_retry, attempts: 3

Thrown from network.tscreateSlackWebhook, reached via the queued
setupChannelWebhook task (onChannelEnabled→runTask / startSync).

Root cause

The Slack connector's activate() writes store key "auth" (the Authorization)
once and never clears it. removeAuth clears the real token
auth_token:slack:<actorId> when an account is disconnected / re-authed with a
different account / cleared after a failure, but leaves "auth" in place.

setupChannelWebhook guarded only on this.get("auth") (still present), so it
proceeded to createWebhook → createSlackWebhook, which does a raw direct
auth_token lookup, finds nothing, and throws. The webhook queue then retried
the task 3× per event.

Fix

After the "auth" check, confirm a live token via
this.tools.integrations.get(channelId) — the same canonical resolver getApi
already uses, which also flags the connection for re-auth when the token is
missing. If there's no usable token, log and return instead of registering. No
throw, no retry storm, and the user is prompted to reconnect.

Tests

Adds a vitest harness to the Slack connector (mirrors gmail: vitest.config.ts

• test/test:watch scripts + vitest devDep). src/slack.test.ts constructs
  new Slack(id, fakeToolShed) with mocked store/integrations/network and covers:

• token removed → does not register, no throw
• usable token → registers + persists channel_webhook_<id>
• no stored "auth" → does nothing

3/3 green; tsc build + plot lint clean.

Connector change only (not twister/src) → no changeset required.

🤖 Generated with Claude Code