Use singular S3 key prefixes for changelog/bundle artifacts

[cotti]

Why

The changelog automation uploads artifacts to S3 under plural folder prefixes ({product}/changelogs/ and {product}/bundles/), but everywhere else in the codebase these entities are singular (ArtifactType.Changelog, ArtifactType.Bundle). This inconsistency is a small mishap worth correcting now, before any consumer reads the published layout from the CDN.

What

Artifacts now upload under {product}/changelog/ and {product}/bundle/, and the scrubber Lambda detects bundles via the /bundle/ path segment. Upload-target and incremental-uploader tests were updated to assert the new keys.

This changes where new objects are written in the elastic-docs-v3-changelog-bundles[-private] buckets. There is no consumer of these prefixes yet, so the only follow-up is housekeeping: optionally migrate/relabel any objects already written under the old plural prefixes (or let them age out). The S3 bucket names and the local :::{changelog} directive folder convention (changelog/bundles/ in a docset) are unchanged.

Companion docs-only PRs update the matching references in docs-infra and docs-actions.

Made with Cursor

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Enterprise

Run ID: e9717acc-17fe-45f8-b973-e2194aa47e2a

📥 Commits

Reviewing files that changed from the base of the PR and between 4856b14 and 089e3ab.

📒 Files selected for processing (6)

• src/infra/docs-lambda-changelog-scrubber/Program.cs
• src/infra/docs-lambda-changelog-scrubber/README.md
• src/services/Elastic.Changelog/Uploading/ChangelogUploadService.cs
• tests/Elastic.Changelog.Tests/Changelogs/LinkAllowlistSanitizerTests.cs
• tests/Elastic.Changelog.Tests/Uploading/ChangelogUploadServiceTests.cs
• tests/Elastic.Documentation.Integrations.Tests/S3/S3IncrementalUploaderTests.cs

---

📝 Walkthrough

Walkthrough

This PR renames S3 object key paths for changelog and bundle uploads from plural to singular directory conventions. The ChangelogUploadService now constructs paths using {product}/changelog/ and {product}/bundle/ instead of {product}/changelogs/ and {product}/bundles/. The Lambda scrubber is updated to recognize the new "/bundle/" pattern for routing, with corresponding README documentation updates. Test assertions across unit and integration test suites are updated to verify the new path formats.

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title accurately and concisely describes the main change: switching S3 key prefixes from plural to singular forms for changelog and bundle artifacts.
Description check	✅ Passed	The description clearly explains why the change is needed, what is being changed, and the scope of impact, all directly related to the changeset.
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

✨ Simplify code

• Create PR with simplified code
• Commit simplified code in branch changelog/singular-bucket-prefix

---

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.}