fix(refs): resolve unjoined-but-public channels by name

[lmjabreu]

Summary

• resolveChannelRef's name branch only called client.channels.getChannels({ workspaceId }), which is membership-scoped and excludes public channels the current user hasn't joined.
• Effect: tw channel <action> "Public Channel I Haven't Joined" fails with CHANNEL_NOT_FOUND, while id:N and URL refs work because getChannel(id) isn't membership-filtered.
• Fix merges getChannels with workspaces.getPublicChannels and dedupes by id. Same shape as src/commands/channel/list.ts:177-180 already uses for the --scope public path.

Why this exists

The bug was first suspected to be "active vs archived" — that turned out to be wrong: getChannels({ workspaceId }) already returns both states (Twist's API is tri-state — see list.ts:163-169). The real axis is joined vs unjoined, which I confirmed live and which also holds for non-admin users (getPublicChannels is not admin-gated). Scale check in Doist workspace 1585: 998 public channels, 829 of which I haven't joined — so the bug bites in production, not just edge cases.

Test plan

• npm run lint — clean
• npm run type-check — no errors in refs.ts / refs.test.ts (pre-existing @doist/cli-core export errors elsewhere on main are unrelated)
• npx vitest run src/lib/refs.test.ts — 71/71 pass (4 new tests: unjoined-public exact match, unjoined-public substring match, dedup of joined-and-public, ambiguity across joined+public)
• Live smoke in workspace 1767 (Testing Environment), against a fresh test channel — no existing channels touched:
  1. tw channel create tw-cli-resolver-smoke-<ts> --workspace 1767 → id 837258
  2. Left the channel via SDK removeUser
  3. tw channel archive "tw-cli-resolver-smoke-<ts>" --workspace 1767 → { id: 837258, archived: true } ✅ (previously: CHANNEL_NOT_FOUND)
  4. tw channel delete id:837258 --workspace 1767 --yes → cleaned up

Smoke required #246's channel create/archive/delete commands to exercise an action on the resolved channel; they were checked out temporarily and reverted after.

Out of scope

• Help-text cleanup on tw channel unarchive in feat(channel): add create, delete, archive, unarchive subcommands #246's src/commands/channel/index.ts — separate trivial follow-up once both this and feat(channel): add create, delete, archive, unarchive subcommands #246 land.
• id: and URL branches — already work for archived and unjoined channels via getChannel(id).

🤖 Generated with Claude Code

[scottlovegrove]

@doistbot /review

[doistbot]

This PR successfully expands name-based channel resolution to include unjoined public channels, fixing a frustrating gap in how the CLI handles larger workspaces. The approach is solid and well-tested, ensuring users can interact with any public channel regardless of their membership status. A few refinements are noted regarding aligning the deduplication logic with existing codebase patterns, deferring the public channel API fetch until necessary to reduce overhead, and tweaking the test suite to better prove deduplication without hardcoding the underlying fetch strategy.

_{Share Feedback • Review Logs}

[Bloomca]

The code change looks good. I do agree that it can add a bit of noise (in general I assume most of the time we want to look at joined channels), but let's try and see.

[lmjabreu]

Thanks @Bloomca. Agreed it's worth watching. If the substring noise shows up on the bigger workspaces, the lazy-fetch fallback doistbot suggested (joined first, widen to public only on a miss) is the natural next step. Merging.

[doist-release-bot]

🎉 This PR is included in version 2.44.2 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀