feat(content-uploader): Implement permission error handling uploads

[dealwith]

5/5 PR in the queue:

1. feat(uploads-manager): integrate shared feature into ContentUploader #4573
2. feat(content-uploader): Implement cancel all retry all handlers #4577
3. feat(content-uploader): Implement cancelled state uploads manager #4578
4. feat(content-uploader): Implement cancel all confirmation modal #4579
5. 👉 feat(content-uploader): Implement permission error handling uploads #4580

Summary by CodeRabbit

• Bug Fixes

  • Improved upload error handling so users see a clear, localized "insufficient permissions" message when they lack permission to upload to a folder.

• Localization

  • Added a specific localized message for insufficient-permission upload failures.

• Tests

  • Added/updated tests to verify the new permission-specific error message is displayed.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: c6666b5b-ea91-4552-96a0-be7a3e2da61d

📥 Commits

Reviewing files that changed from the base of the PR and between 6caedd9 and be7fa97.

📒 Files selected for processing (3)

• src/api/uploads/MultiputUpload.js
• src/api/uploads/__tests__/MultiputUpload.test.js
• src/elements/content-uploader/__tests__/CellRenderer.test.tsx

🚧 Files skipped from review as they are similar to previous changes (3)

• src/api/uploads/tests/MultiputUpload.test.js
• src/elements/content-uploader/tests/CellRenderer.test.tsx
• src/api/uploads/MultiputUpload.js

---

Walkthrough

Adds a new exported error constant and i18n message for upload "insufficient permissions", replaces a hard-coded error string in MultiputUpload with the constant, and updates UI rendering and tests to show the localized permission-denied message.

Changes

Upload insufficient permissions error handling

Layer / File(s)	Summary
Error constant and message definition src/constants.js, src/elements/common/messages.js	Add ERROR_CODE_UPLOAD_INSUFFICIENT_PERMISSIONS = 'access_denied_insufficient_permissions' and uploadsInsufficientPermissionsErrorMessage i18n entry.
API error handling and tests src/api/uploads/MultiputUpload.js, src/api/uploads/__tests__/MultiputUpload.test.js	Import and compare errorData.code to ERROR_CODE_UPLOAD_INSUFFICIENT_PERMISSIONS in create-session forbidden handling; update tests to use the constant.
UI rendering and tests src/elements/content-uploader/progressCellRenderer.tsx, src/elements/content-uploader/__tests__/CellRenderer.test.tsx	Import the new constant and add a switch case in getErrorMessage to return uploadsInsufficientPermissionsErrorMessage; add test asserting the permission-denied message renders for that error code.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Suggested labels

ready-to-merge

Suggested reviewers

• jpan-box
• olehrybak

Poem

🐰 A tiny constant hops in place,

No more strings to hide its face,
From API to UI it leaps,
Tests awake from restful sleeps,
Permission messages now find their space.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description lacks substantive PR context; it only contains queue position information and template instructions without explaining what changes were made or why.	Add a clear explanation of the changes, what problem this solves, and implementation details beyond the template boilerplate.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically summarizes the main change: implementing permission error handling for uploads with dedicated error messaging.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

📝 Generate docstrings

• Create stacked PR
• Commit on current branch

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch implement-permission-error-handling-uploads

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

The base branch was changed.

[mergify]

Merge Queue Status

• ✅ Entered queue — 2026-05-29 15:52 UTC · Rule: Automatic strict merge
• ✅ Checks skipped · PR is already up-to-date
• ✅ Merged — 2026-05-29 15:53 UTC · at be7fa975b49211cdc22f87bc102805d37150c5cf · squash

This pull request spent 38 seconds in the queue, including 4 seconds running CI.

Required conditions to merge

• #approved-reviews-by >= 1 [🛡 GitHub branch protection]
  • feat(content-uploader): Implement permission error handling uploads #4580
• #review-threads-unresolved = 0 [🛡 GitHub branch protection]
  • feat(content-uploader): Implement permission error handling uploads #4580
• github-review-decision = APPROVED [🛡 GitHub branch protection]
  • feat(content-uploader): Implement permission error handling uploads #4580
• any of [🛡 GitHub branch protection]:
  • check-success = Summary
  • check-neutral = Summary
  • check-skipped = Summary
• any of [🛡 GitHub branch protection]:
  • check-success = lint_test_build
  • check-neutral = lint_test_build
  • check-skipped = lint_test_build
• any of [🛡 GitHub branch protection]:
  • check-success = license/cla
  • check-neutral = license/cla
  • check-skipped = license/cla
• any of [🛡 GitHub branch protection]:
  • check-success = lint_pull_request
  • check-neutral = lint_pull_request
  • check-skipped = lint_pull_request