Repository navigation
fix(source-linkedin-ads): replace deprecated MessageRepresentationAirbyteTracedErrors with AirbyteTracedException - #76040
Conversation
…byteTracedErrors with AirbyteTracedException Replace import and usage of the removed MessageRepresentationAirbyteTracedErrors class with AirbyteTracedException from airbyte_cdk.utils.traced_exception in test file. The MessageRepresentationAirbyteTracedErrors class was removed in CDK PR #927 (airbyte-python-cdk v7.10.2) without a backward-compatible alias, causing ImportError. Resolves airbytehq/oncall#11850 Co-Authored-By: bot_apk <apk@cognition.ai>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
👋 Greetings, Airbyte Team Member!Here are some helpful tips and reminders for your convenience. 💡 Show Tips and TricksPR Slash CommandsAirbyte Maintainers (that's you!) can execute the following slash commands on your PR:
📚 Show Repo GuidanceHelpful Resources
|
Co-Authored-By: bot_apk <apk@cognition.ai>
Co-Authored-By: bot_apk <apk@cognition.ai>
|
Deploy preview for airbyte-docs ready! ✅ Preview Built with commit ff5dd6b. |
|
|
/ai-prove-fix
|
Fix Validation EvidenceStatus: ✅ FIX VALIDATED — No regressions detected. Safe to merge. Evidence Summary
Key Findings
Pre-flight Checks
Evidence PlanProving Criteria: Regression tests show no behavioral difference between dev and control versions ✅ Test Approach: Since this is a test-only change with no runtime code modifications, live connection pinning would not exercise the fix code path. Regression tests (comparison mode) are the most appropriate evidence for this type of change. Recommendation
Session: https://app.devin.ai/sessions/1b07b363f040491395b5c9dc87f41648 |
|
|
Danylo Jablonski (@DanyloGL) It looks like the we're getting blocked on an expired refresh token. Is this something you can help with? |
GSM creds updated. |
…recated-import, bump to 5.6.7 Co-Authored-By: bot_apk <apk@cognition.ai>
The master-sync push (384073d) dropped these two rows while renumbering 5.7.0 -> 6.1.0. They are not duplicates: master carries each exactly once (airbytehq#76040 and airbytehq#75583, both dated 2026-04-06), so the branch was deleting real release history as a side effect of the renumber. A PR adding a stream has no business rewriting entries that predate it. The changelog diff is now purely the 6.1.0 row, and the whole docs page diff against master is additive: the Videos bullet, the Videos section and that one row, with zero deletions. Thanks to the AI review for catching it.
What
Replace the removed
MessageRepresentationAirbyteTracedErrorsclass with its base classAirbyteTracedExceptionin source-linkedin-ads unit tests.MessageRepresentationAirbyteTracedErrorswas removed fromairbyte_cdk.sources.streams.http.http_clientin CDK PR #927 (v7.10.2) without a backward-compatible alias, causingImportErrorwhen running the test suite on newer CDK versions.Resolves https://github.com/airbytehq/oncall/issues/11850:
How
airbyte_cdk.sources.streams.http.http_client.MessageRepresentationAirbyteTracedErrorstoairbyte_cdk.utils.traced_exception.AirbyteTracedExceptionpytest.raises(MessageRepresentationAirbyteTracedErrors)→pytest.raises(AirbyteTracedException)intest_should_retry_on_errorReview guide
unit_tests/test_source.py— import replacement andpytest.raisesupdatemetadata.yaml— version bump to 5.6.6docs/integrations/sources/linkedin-ads.md— changelog entryUser Impact
No user-facing behavior change. Fixes broken test suite on CDK >= v7.10.2.
Can this PR be safely reverted and rolled back?
Link to Devin session: https://app.devin.ai/sessions/83cdd5ae8ab140fa9b4c072d44e9163a