Skip to content

fix: forward 100% BrightData-resolved backlinks to Mystique for bridge enrichment#2378

Open
miakobchuk wants to merge 1 commit intomainfrom
fix/brightdata-100pct-bridge-forward
Open

fix: forward 100% BrightData-resolved backlinks to Mystique for bridge enrichment#2378
miakobchuk wants to merge 1 commit intomainfrom
fix/brightdata-100pct-bridge-forward

Conversation

@miakobchuk
Copy link
Copy Markdown
Contributor

@miakobchuk miakobchuk commented Apr 13, 2026

⚠️ Merge after https://git.corp.adobe.com/experience-platform/mystique/pull/1592

Summary

  • Removes the early return when all broken backlinks are resolved by Bright Data
  • Instead forwards the message to Mystique with an empty brokenLinks array
  • Mystique's companion fast path (adobe/mystique#companion) detects the empty array, skips the LLM crew, and runs bridge enrichment via Subset 2 over the BrightData suggestions already in SpaceCat

Root cause: ~0.7% of broken-backlinks audits (37/5290 over 7 days in prod) hit the 100% BrightData path and skipped Mystique entirely, leaving fixable/fixabilityReason unset on those suggestions.

No breaking change. The message shape is unchanged. Mystique receives brokenLinks: [] — the companion PR handles this case.

Deploy order: Mystique companion PR must be deployed first. If this lands before Mystique, the old task receives brokenLinks: [], logs an error, and runs the LLM crew with empty input (no crash, just wasteful). Correct order: Mystique → audit-worker.

Test plan

  • New test: should forward to Mystique with empty brokenLinks when all links resolved via Bright Data — verifies sendMessage is called with brokenLinks: [] and logs "Forwarding to Mystique"
  • Existing 65 tests all pass
  • After deploy: search Coralogix prod for "Forwarding to Mystique" to confirm the path fires
  • Verify BrightData-only sites gain fixable/fixabilityReason on their suggestions

🤖 Generated with Claude Code

…e enrichment

When all broken backlinks are resolved by Bright Data, the audit worker previously
returned early and skipped Mystique entirely. This meant the autofix bridge never
fired for those sites, so `fixable`/`fixabilityReason` were never set on BrightData-
resolved suggestions.

Remove the early return and forward the empty `brokenLinks` array to Mystique.
Mystique's fast path (added in companion PR) detects the empty array, skips the LLM
crew, and runs only the Subset 2 bridge enrichment over the BrightData suggestions
already stored in SpaceCat.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 13, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@miakobchuk miakobchuk requested a review from BaDos April 13, 2026 18:06
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.

3 participants