Skip to content

fix: resolve requested channels directly for targeted archived thread sync - #150

Merged
steipete merged 4 commits into
mainfrom
pr120-rebased
Jul 27, 2026
Merged

fix: resolve requested channels directly for targeted archived thread sync#150
steipete merged 4 commits into
mainfrom
pr120-rebased

Conversation

@steipete

Copy link
Copy Markdown
Collaborator

Rebased and completed from #120 by @robinbraemer.

Targeted archived-thread sync now resolves every requested channel directly with GET /channels/{id} before broad thread-catalog discovery. This preserves the targeted path even when archived thread discovery cannot find the requested channel globally.

The completed implementation handles requested channels owned by multiple guilds, performs only one direct lookup per requested channel, rejects DMs from the guild archived-thread path, and adds regression coverage for direct resolution, lookup caching, multi-guild ownership, and DM rejection.

This PR supersedes #120 and closes it while preserving @robinbraemer's authorship credit in the squash commit.

@clawsweeper

clawsweeper Bot commented Jul 27, 2026

Copy link
Copy Markdown
Contributor

ClawSweeper status: review started.

I am starting a fresh review of this pull request: fix: resolve requested channels directly for targeted archived thread sync This is item 1/1 in the current shard. Shard 0/1.

This placeholder means the worker is alive and reading the current context. I will edit this same comment with the actual review when the claws are done clicking.

Crustacean status: shell secured, claws on keyboard, evidence pebbles being sorted.

@steipete
steipete merged commit 3d066a5 into main Jul 27, 2026
11 checks passed
@steipete
steipete deleted the pr120-rebased branch July 27, 2026 05:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants