This is a comment left during a code review.
Path: extensions/slack/src/channel.ts
Line: 24-27

Comment:
**Alternative: import from existing `runtime-api.js` re-export**

`runtime-api.ts` already re-exports both `listSlackDirectoryPeersFromConfig` and `listSlackDirectoryGroupsFromConfig` from `./directory-config.js` (lines 12–15 in `runtime-api.ts`). Adding these two names to the existing `runtime-api.js` import block in `channel.ts` (which already imports several sibling utilities from there) would consolidate imports and avoid introducing a new import source for a file that is otherwise accessed through `runtime-api`.

```suggestion
import {
  buildComputedAccountStatusSnapshot,
  DEFAULT_ACCOUNT_ID,
  listSlackDirectoryGroupsFromConfig,
  listSlackDirectoryPeersFromConfig,
  looksLikeSlackTargetId,
  normalizeSlackMessagingTarget,
  PAIRING_APPROVED_MESSAGE,
  projectCredentialSnapshotFields,
  resolveConfiguredFromRequiredCredentialStatuses,
  type ChannelPlugin,
  type OpenClawConfig,
} from "./runtime-api.js";
```
(and remove the separate `./directory-config.js` import block)

This is purely a style suggestion — the current direct import is functionally correct.

How can I resolve this? If you propose a fix, please make it concise.

---

This is a comment left during a code review.
Path: extensions/slack/src/channel.test.ts
Line: 174-194

Comment:
**`listGroups` has no regression coverage**

The PR description mentions that both `directory.listPeers` and `directory.listGroups` threw at runtime due to the missing import. The new test only exercises `listPeers`. A parallel test for `listGroups` (using `channels` keys instead of `dms` keys) would close the regression gap for the second fixed path and guard against the same issue re-appearing for groups.

```suggestion
  it("lists configured groups without throwing a ReferenceError", async () => {
    const listGroups = slackPlugin.directory?.listGroups;
    expect(listGroups).toBeDefined();

    await expect(
      listGroups!({
        cfg: {
          channels: {
            slack: {
              channels: {
                "C456": {},
              },
            },
          },
        },
        runtime: undefined,
      }),
    ).resolves.toBeDefined();
  });
```

How can I resolve this? If you propose a fix, please make it concise.