Skip to content

docs(source-salesforce): fix changelog errors and clarify BULK API object list#75904

Draft
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1774974524-salesforce-docs-update
Draft

docs(source-salesforce): fix changelog errors and clarify BULK API object list#75904
devin-ai-integration[bot] wants to merge 1 commit intomasterfrom
devin/1774974524-salesforce-docs-update

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration bot commented Mar 31, 2026

Documentation Confidence Assessment

Overall Confidence: 3/5

Dimension Score Rationale
Code Comprehension 2/5 Python CDK with complex multi-file logic, though the triggering change (spec.yaml OAuth config) is fully declarative.
API Documentation Quality 4/5 Salesforce has comprehensive official docs on OAuth, BULK API, and REST API with minor gaps in edge cases.
Change Scope & Risk 5/5 Under 10 lines changed: changelog date corrections, PR number fix, duplicate removal, wording clarification.
Existing Doc Maturity 5/5 Mature 434-line doc with comprehensive coverage of prerequisites, setup, streams, troubleshooting, and changelog.
Connector Sensitivity 1/5 Salesforce is a top-tier certified connector (ql=400, sl=300) — errors affect many users.
Triggering Context 5/5 Triggered from a small merged PR adding declarative OAuth config with a version bump.

What I Verified vs. What I Inferred

Areas of Concern

  • The BULK API unsupported objects list remains incomplete relative to the code. I clarified the wording rather than expanding the list, since maintaining an exhaustive list in docs is fragile. Reviewer may prefer expanding the list instead.
  • The v2.6.2 changelog entry has a pre-existing empty PR link ([](https://github.com/airbytehq/airbyte/pull/)) that I did not fix because I could not determine the correct PR number.

What

Fixes several factual errors in the Salesforce source connector documentation, triggered by review of commit c6eb0bb (PR #75579).

How

Changelog corrections:

  • v2.7.19: Date corrected from 2026-03-30 to 2026-03-31 (verified against merge commit timestamp)
  • v2.7.17: Date corrected from 2026-02-10 to 2026-02-11 (verified against merge commit timestamp)
  • v2.7.16-rc.1: PR link text corrected from [66136] to [67509] — the URL already pointed to PR source-salesforce: performance tuning #67509 ("performance tuning"), but the display text incorrectly showed chore: salesforce cdk 7 migration #66136 (which is the CDK v7 migration, correctly referenced in v2.7.12)
  • v2.6.5: Removed duplicate changelog row

BULK API object list:

  • Clarified that the listed objects unsupported by the BULK API are not exhaustive. The source code (UNSUPPORTED_BULK_API_SALESFORCE_OBJECTS in api.py) contains significantly more objects than what was listed in the docs. Changed wording from "This includes the following objects" to "Commonly encountered examples include the following objects, but this is not an exhaustive list" to avoid misleading users.

Review guide

  1. docs/integrations/sources/salesforce.md — all changes are in this single file

Key items to verify:

  • Are the corrected changelog dates accurate? (Verified via git log --format="%H %aI %s" for each PR's merge commit.)
  • Is the BULK API list wording change appropriate, or would you prefer the list be updated to match the full code list instead?

User Impact

Users reading the Salesforce connector docs will see correct release dates, correct PR links, no duplicate entries, and an accurate description of the BULK API object list scope. No functional changes.

Can this PR be safely reverted and rolled back?

  • YES 💚

Note: I am an AI assistant (Devin) and have proposed these documentation updates based on a review of the connector source code and third-party API documentation. Reviewers may merge, modify, or close this PR as they see fit.

Link to Devin session: https://app.devin.ai/sessions/689e679bbe47409e89385ff747d893a8

…ject list

Co-Authored-By: bot_apk <apk@cognition.ai>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@github-actions
Copy link
Copy Markdown
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
  • ❇️ AI Testing and Review (internal link: AI-SDLC Docs):
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
    • /bump-progressive-rollout-version - Bumps connector version with an RC suffix (2.16.10-rc.1) for progressive rollouts (enableProgressiveRollout: true).
      • Example: /bump-progressive-rollout-version changelog="Add new feature for progressive rollout"
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
  • 🐍 Python connectors:
    • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
    • /poe source example lock - Alias for /poe connector source-example lock.
    • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
    • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@devin-ai-integration devin-ai-integration bot added area/documentation Improvements or additions to documentation team/documentation labels Mar 31, 2026
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

Phase 2 note: Docusaurus build verification was skipped because the environment has Node v22.12.0, which does not satisfy the engines.node constraint >=20 <22 in docusaurus/package.json. The changes are limited to a single markdown file with changelog corrections and minor wording adjustments, so broken links or build errors from these changes are unlikely.


Devin session

@github-actions
Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

✅ Preview
https://airbyte-docs-55nq35mn0-airbyte-growth.vercel.app

Built with commit 9365cef.
This pull request is being automatically deployed with vercel-action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/documentation Improvements or additions to documentation team/documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

0 participants