Skip to content

fix: use handler discovery tools instead of inline docs#773

Closed
chubes4 wants to merge 2 commits intomainfrom
feat/704-publish-level-dedup
Closed

fix: use handler discovery tools instead of inline docs#773
chubes4 wants to merge 2 commits intomainfrom
feat/704-publish-level-dedup

Conversation

@chubes4
Copy link
Member

@chubes4 chubes4 commented Mar 9, 2026

Summary

  • remove the inline handler documentation dump from configure_flow_steps
  • direct the tool description to the existing discovery tools instead of embedding the full handler registry
  • delete the now-unused HandlerDocumentation helper

chubes4 added 2 commits March 7, 2026 23:08
…andler

Add DuplicateDetection utility class with fuzzy title matching to prevent
cross-flow duplicate publishing. When multiple flows process the same content
(e.g. same Reddit thread posted to multiple subreddits), the first flow
publishes and subsequent flows skip with action=duplicate_skipped.

- DuplicateDetection::findExistingPostByTitle() queries recent posts and
  runs fuzzy title comparison (core extraction, prefix matching, levenshtein)
- DuplicateDetection::titlesMatch() handles subreddit attributions,
  reaction suffixes, unicode dashes, and minor wording variations
- dedup_enabled and dedup_lookback_days settings added to PublishHandlerSettings
  (available to all publish handlers, default off)
- WordPress handler checks for duplicates before delegating to ability

Closes #704
@chubes4
Copy link
Member Author

chubes4 commented Mar 9, 2026

Superseded by #774, which uses a clean branch from main and avoids the duplicate branch-history conflict.

@chubes4 chubes4 closed this Mar 9, 2026
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.

1 participant