Skip to content

fix(services/onedrive): align service semantics#7393

Draft
suyanhanx wants to merge 7 commits intomainfrom
fix/onedrive-api-alignment
Draft

fix(services/onedrive): align service semantics#7393
suyanhanx wants to merge 7 commits intomainfrom
fix/onedrive-api-alignment

Conversation

@suyanhanx
Copy link
Copy Markdown
Member

Which issue does this PR close?

N/A (no existing issue; review-driven OneDrive cleanup)

Rationale for this change

The OneDrive backend had a few mismatches between Microsoft Graph behavior and OpenDAL's expected primitives. Some unsupported options were silently ignored, create_dir did not ensure missing ancestors, and version listing mixed current entries with historical versions.

What changes are included in this PR?

  • treat upload-session final 200 OK as a successful completion
  • make create_dir recursively ensure missing parent directories
  • keep the current file entry separate from historical versions in list_with_versions
  • tighten OneDrive capabilities around versioned stat
  • return Unsupported for operations and options that OneDrive cannot reliably support (read/stat/delete with version, conditional write options, and copy with if_not_exists)
  • refresh the lockfile after dependency cleanup

Are there any user-facing changes?

Yes. The OneDrive service now fails fast for unsupported options instead of silently ignoring them, create_dir behaves recursively, and version listing reports the current item separately from historical versions.

AI Usage Statement

This PR was prepared with OpenAI Codex assistance for implementation and review.

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