feat: guide Qdrant memory configuration - #11213
Merged
diegosouzapw merged 7 commits intoAug 23, 2026
Merged
diegosouzapw merged 7 commits into
diegosouzapw merged 7 commits into
Conversation
…ouzapw#189, diegosouzapw#190) Bumps: nanoid ^3.3.17 (was transitive, now overridden), dompurify ^3.4.13 (with monaco-editor scoped override). Closes Dependabot diegosouzapw#189, diegosouzapw#190. Remaining diegosouzapw#182-diegosouzapw#188 (js-yaml + mermaid) already closed by diegosouzapw#9651 merge — awaiting Dependabot re-scan. npm audit → 0 vulnerabilities.
…egosouzapw#190 Closes Dependabot diegosouzapw#189 (dompurify 3.4.13) and diegosouzapw#190 (nanoid 3.3.17). npm audit → 0.
_tasks is a SEPARATE nested git repo (gitignored). The pattern _tasks/ (trailing slash) ignores only a directory, not a SYMLINK named _tasks. A self-referential _tasks symlink can slip in via git add -A and, once pulled, checkout materializes it over the real _tasks repo (destroying plans/specs/hands-off). Anchored /_tasks ignores the symlink too, preventing re-capture.
…ropagated tests; planning docs dropped)
diegosouzapw
marked this pull request as ready for review
August 23, 2026 13:13
diegosouzapw
merged commit Aug 23, 2026
968fa96
into
diegosouzapw:release/v3.8.50
3 of 5 checks passed
4 of 5 tasks
maxmad64bis
pushed a commit
to maxmad64bis/OmniRoute
that referenced
this pull request
Aug 23, 2026
Bar NEW unused vars/args/catches outside the `_` escape hatch via
@typescript-eslint/no-unused-vars (args:"all", caughtErrors:"all",
ignoreRestSiblings), scoped to src/**/*.{ts,tsx,js,jsx} + open-sse/**/*.ts
+ tests/**/*.{ts,tsx,mjs} — scripts/, bin/ and docs/ stay out of scope.
Pre-existing violations (1393 across 1251 files) are frozen in
config/quality/eslint-suppressions.json via --suppress-rule, same
pattern as diegosouzapw#7879 (toNumber tier-by-tier). Only genuinely NEW unused
bindings fail lint from now on.
Also untracks two docs/superpowers/* planning files that slipped into
the tree via diegosouzapw#11213 while being gitignored; their presence makes the
tracked-artifacts pre-commit hook reject every new commit.
Refs diegosouzapw#9985
5 tasks done
diegosouzapw
pushed a commit
that referenced
this pull request
Aug 23, 2026
…n main return (#11228 follow-up) The #11228 hunk spliced the guided endpoint header into ProviderModelsModal's renderModelGroup map callback, replacing its return statement and leaving an unclosed <div> — dashboard typecheck failed on the release tip. Restore the modal's map return and render the header (title/subtitle//v1 URL/test link/ advanced-protocols note) at the top of the main component return, using the i18n keys the PR added (endpoint.title/subtitle/testEndpoint/advancedProtocols) and the existing displayBaseUrl variable. Also untrack two docs/superpowers planning files that leaked in via #11213; they belong to _tasks/ (copies preserved there) and their tracked presence trips check:tracked-artifacts on every commit. Validated: check:dashboard-typecheck OK (220 pre-existing, all within frozen baseline; zero syntax errors in EndpointPageClient.tsx).
diegosouzapw
pushed a commit
that referenced
this pull request
Aug 23, 2026
…11247) Validated on the combined 12-PR batch board: typecheck:core clean, file-size/changelog/complexity/cognitive gates within baseline, eslint config smoke-tested. Freeze-then-ratchet for @typescript-eslint/no-unused-vars scoped to src/open-sse/tests (1393 pre-existing frozen, _ escape hatch, args:all), following the #7879 pattern — plus it untracks the two docs/superpowers planning files that leaked in via #11213 and were tripping check:tracked-artifacts for everyone. Thank you @maxmad64bis!
This was referenced Aug 23, 2026
diegosouzapw
pushed a commit
that referenced
this pull request
Aug 23, 2026
#11249/#11251/#11252/#10952 follow-ups) The gate (eslint full-tree, baseline 0) failed on the merged head with 26 unsuppressed errors, all introduced by same-day base merges — none by this PR. Each fixed at the cause (no new suppressions): 1. tests/integration/qdrant-routes.test.ts (#11249/#11213): 20x no-explicit-any on route-invocation casts. Fix: typed asNextRequest() adapter (Request -> NextRequest) replacing every "as any". 2. tests/unit/effort-tiers-loop-catalog-e2e.test.ts (#11252): unused after/beforeEach imports — newly error-level under the no-unused-vars ratchet #11247 shipped the same day. Fix: trim the import (the file uses test.after/test.beforeEach method forms). 3. EditConnectionModal.tsx (#11251): react-hooks/set-state-in-effect on the modal-open setFormData init. The pattern (sync form state with the loaded connection when isOpen flips) is the codebase's sanctioned exception — same justified eslint-disable-next-line already used in FreePoolTab.tsx and BatchConceptCard.tsx; a key-remount of the 30+ field form would be a behavior-risking restructure for a basereds PR. 4. tests/unit/usage-service-hardening.test.ts (shard 2/4): last stale Copilot wire-identity pin — #10952 re-based it on the live-captured CLI 1.0.81-6 (copilot-developer-cli integration id, API version 2026-08-01). Assertions aligned (UA, Editor-Version, Editor-Plugin-Version, X-GitHub-Api-Version). Validation: all four files eslint-clean with the frozen suppressions; usage-service-hardening 23/23 and effort-tiers 2/2 RED->GREEN (node --import tsx/esm --test). Refs #9985
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
Merged after conflict resolution (validated on the combined batch board + this branch): component integrated with diegosouzapw#10489's health-race semantics (stale-check invalidation, 3-state badge, correct useCallback deps — your validation gate and tutorial are preserved on top); the two pre-gate tests updated to validate-search-before-enable (contract propagation, commented); the docs/superpowers planning files were dropped — project rule keeps planning artifacts out of docs/ (they live in the private _tasks/ repo). Base was mistakenly main; retargeted. qdrant-config-card 7/7 + qdrant-routes integration 18/18. Thank you @rafacpti23 — the pre-enable real-search gate closes a real misconfiguration footgun!
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…n main return (diegosouzapw#11228 follow-up) The diegosouzapw#11228 hunk spliced the guided endpoint header into ProviderModelsModal's renderModelGroup map callback, replacing its return statement and leaving an unclosed <div> — dashboard typecheck failed on the release tip. Restore the modal's map return and render the header (title/subtitle//v1 URL/test link/ advanced-protocols note) at the top of the main component return, using the i18n keys the PR added (endpoint.title/subtitle/testEndpoint/advancedProtocols) and the existing displayBaseUrl variable. Also untrack two docs/superpowers planning files that leaked in via diegosouzapw#11213; they belong to _tasks/ (copies preserved there) and their tracked presence trips check:tracked-artifacts on every commit. Validated: check:dashboard-typecheck OK (220 pre-existing, all within frozen baseline; zero syntax errors in EndpointPageClient.tsx).
muhamadgalihsaputra
pushed a commit
to niyatna/NiyatnaRoute
that referenced
this pull request
Sep 27, 2026
…iegosouzapw#11247) Validated on the combined 12-PR batch board: typecheck:core clean, file-size/changelog/complexity/cognitive gates within baseline, eslint config smoke-tested. Freeze-then-ratchet for @typescript-eslint/no-unused-vars scoped to src/open-sse/tests (1393 pre-existing frozen, _ escape hatch, args:all), following the diegosouzapw#7879 pattern — plus it untracks the two docs/superpowers planning files that leaked in via diegosouzapw#11213 and were tripping check:tracked-artifacts for everyone. Thank you @maxmad64bis!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Adds a Qdrant setup tutorial to the Memory engine card and prevents enabling Qdrant until a real embedding search succeeds. This ensures the configured embedding model can search the selected Qdrant collection before it becomes the active vector store.
The health endpoint now reads collection metadata (existence, vector name, and vector dimension) without modifying Qdrant. The UI presents that data, explains that Qdrant retrieves relevant memory rather than compressing tokens, and includes setup/security guidance plus the requested credit.
Validation
node --import tsx/esm --test tests/integration/qdrant-routes.test.tsnpx vitest run tests/unit/ui/qdrant-config-card.test.tsxnpm run typecheck:coregit diff --check