Skip to content

test: restore master baseline CI coverage#667

Open
rwmjhb wants to merge 1 commit intomasterfrom
fix/master-baseline-ci
Open

test: restore master baseline CI coverage#667
rwmjhb wants to merge 1 commit intomasterfrom
fix/master-baseline-ci

Conversation

@rwmjhb
Copy link
Copy Markdown
Collaborator

@rwmjhb rwmjhb commented Apr 19, 2026

Summary

  • add the missing embedder-ollama-batch-routing baseline entry to the CI manifest verifier
  • align the Ollama batch routing test with the existing /ollama provider-detection contract while keeping random-port mocks
  • reset plugin singleton registration state between recall and smart-extractor regression scenarios so later cases do not inherit stale config, hooks, or stores

Root causes fixed

  • scripts/verify-ci-test-manifest.mjs was not updated when test/embedder-ollama-batch-routing.test.mjs was added to the CI manifest
  • test/embedder-ollama-batch-routing.test.mjs assumed any random loopback port would hit the Ollama native-fetch path, but the implementation only treats :11434 or /ollama URLs as Ollama
  • test/recall-text-cleanup.test.mjs and test/smart-extractor-branches.mjs were reusing singleton plugin state across scenarios after the phase-2 singleton work, which caused stale retriever/LLM/store config to leak between cases

Verification

  • node scripts/verify-ci-test-manifest.mjs
  • node --test test/embedder-ollama-batch-routing.test.mjs
  • node --test test/recall-text-cleanup.test.mjs
  • node test/smart-extractor-branches.mjs
  • npm run test:packaging-and-workflow
  • npm run test:llm-clients-and-auth
  • npm run test:core-regression
  • npm test

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.

1 participant