docs(backlog): raise DEBT-032 to high, and say why it is not debt

[fvadicamo]

Documentation only. No code changes.

DOCS-009 (#106) found that the cleanup of an old index version does not exist: no endpoint, no flag, no script step, and no version-scoped delete on the VectorStoreProvider Protocol — while the module docstring tells the operator that "the operator ... then triggers cleanup of old-version chunks".

Why this is not "debt", and why it is high

REQ-064 spells the cleanup out as part of the requirement: "new chunks created with incremented version alongside old, atomic switch via config change, cleanup of old version afterwards". So it is not a suboptimal-but-working solution: it is an acceptance criterion of a shipped requirement that was never built, while the code advertises the capability. Code that promises something it cannot do is the same disease as BUG-023, one level up.

And it is live only because we fixed BUG-023. Until 2026-07-14, a reindex in Qdrant mode wrote nothing, so there was never an old version to clean up and the gap was harmless. The moment reindex started actually writing, every reindex began doubling a namespace's storage permanently, and the only exit became a hand-written destructive delete against the store — run at exactly the moment the operator is least certain about what is live.

This is the third gap in this family armed by its own fix:

1. DELETE /documents/{id} had no namespace binding — harmless while the delete was a no-op, a cross-namespace deletion once it worked (fixed in fix(index): route every chunk path through the vector store Protocol #102).
2. The vektra-app tests had never worked — nobody ran them, so nobody found out (fixed in test: close the local-env leak and run the tests nobody ran #104).
3. This one.

Added acceptance criterion

The endpoint refuses to delete the version the system is currently serving, and a test proves it.

The destructive failure mode here is not "an old version survives". It is "the live index is emptied".

🤖 Generated with Claude Code

[coderabbitai]

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (1)

• .s2s/BACKLOG.md is excluded by !.s2s/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: f288a47d-3984-4f5c-b82d-8dd02b51a986

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Commit unit tests in branch docs/promote-debt-032

---

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

[gemini-code-assist]

Code Review

This pull request updates the backlog item DEBT-032 by raising its priority to high, providing additional context on why this issue is critical, and adding a new acceptance criterion to ensure the endpoint refuses to delete the active index version. The reviewer suggested consolidating the new acceptance criterion with an existing one to avoid redundancy and focus specifically on the testing requirement and preventing the destructive failure mode.

Important

The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.