Skip to content

fix(cli): enable vector buckets by default#5421

Merged
jgoux merged 2 commits into
developfrom
codex/enable-vector-buckets-default
Jun 2, 2026
Merged

fix(cli): enable vector buckets by default#5421
jgoux merged 2 commits into
developfrom
codex/enable-vector-buckets-default

Conversation

@jgoux
Copy link
Copy Markdown
Contributor

@jgoux jgoux commented Jun 2, 2026

Summary

Enable local storage vector bucket support by default in generated CLI config.

This makes fresh and missing storage.vector.enabled config paths opt in to vector buckets automatically, while still allowing users to explicitly disable the feature with enabled = false.

The local start happy path now expects the default vector bucket seed call so the test coverage matches the new behavior.

@jgoux jgoux marked this pull request as ready for review June 2, 2026 09:40
@jgoux jgoux requested a review from a team as a code owner June 2, 2026 09:40
@coveralls
Copy link
Copy Markdown

coveralls commented Jun 2, 2026

Coverage Report for CI Build 26812244842

Warning

No base build found for commit 8680362 on develop.
Coverage changes can't be calculated without a base build.
If a base build is processing, this comment will update automatically when it completes.

Coverage: 63.886%

Details

  • Patch coverage: No coverable lines changed in this PR.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

Requires a base build to compare against. How to fix this →


Coverage Stats

Coverage Status
Relevant Lines: 15778
Covered Lines: 10080
Line Coverage: 63.89%
Coverage Strength: 7.08 hits per line

💛 - Coveralls

@jgoux jgoux enabled auto-merge (squash) June 2, 2026 10:03
@jgoux jgoux merged commit 39bf0e7 into develop Jun 2, 2026
30 checks passed
@jgoux jgoux deleted the codex/enable-vector-buckets-default branch June 2, 2026 10:08
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.

4 participants