Skill search returns stale results after all skills are disabled or removed
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 78/100
Línea de trabajo
Comienza con SkillSearchEngine._ensure_indexed() e inspecciona cómo afecta una lista de documentos vacía al recuperador BM25 y a la versión de la caché. Ejecuta test_skill_features.py, incluidos los casos de desactivación/eliminación y de índices secundarios híbridos; se considera terminado cuando las búsquedas no devuelven resultados una vez eliminadas todas las skills y vuelven a funcionar después de activar o añadir skills.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Description
After a search populates the skill index, disabling or removing all
skills does not clear the index. Subsequent searches still return
previously indexed skill IDs.
Steps to reproduce
- Create a SkillCatalog with enabled skills.
- Create a SkillSearchEngine using the BM25 backend.
- Search for a term matching an enabled skill.
- Disable or remove every skill in the catalog.
- Run the same search again.
Expected behavior
The search returns an empty list because no enabled skills remain.
Actual behavior
The search returns stale results from the previous index.
Environment
- OS: Windows
- Python: 3.12
- Commit: 1860d85a23dc507c9a7bb908bfdb83a04a6eeb18
Investigation
In SkillSearchEngine._ensure_indexed(), an empty document list skips
retriever indexing, but the cache version is still updated. This leaves
the previous index intact.
Resetting the retriever when there are no enabled skill documents
appears to resolve the problem.
I have a local fix and regression tests covering disabling/removing all
skills, clearing hybrid child indexes, and restoring search after skills
are enabled or added again. All 37 tests in test_skill_features.py pass
with the fix.
I would be happy to submit a PR.
- Lenguaje dominante
- Python
- Estrellas
- 4.4k
- Forks
- 532
- Merge medio
- 18 h 26 min
- PR fusionados (30 d)
- 10
Preparar el entorno
- Sin Dockerfile ni archivo de Docker Compose
- Tiene una plantilla de pull request
- Leer la guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de modelscope/ms-agent
-
fetch_page web tool performs unrestricted server-side HTTP fetch of agent-supplied URLs (SSRF)Posiblemente ocupada @SiyuanChenToDo la tomó hace 3 días. Abierto
Dificultad 4/5 3-5 días Aptitud para principiantes 45/100
modelscope/ms-agent#1018 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
modelscope/ms-agent#1011 ·
Los mantenedores suelen responder en 1 día
-
[tool] Document authenticated Streamable HTTP MCP setupPosiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 3/5 1-2 días Aptitud para principiantes 68/100
modelscope/ms-agent#1003 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
Dificultad 4/5 3-5 días Aptitud para principiantes 68/100
modelscope/ms-agent#990 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 5/5 Más de una semana Aptitud para principiantes 25/100
modelscope/ms-agent#911 ·
Los mantenedores suelen responder en 1 día
Todos los issues de modelscope/ms-agent
Issues similares
-
[Resource]: glyphcss-diagramsAbiertoresource-submission validation-passed
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
hesreallyhim/awesome-claude-code#3214 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
room-commons replace accepts an empty old string and silently prepends the new text to the pageAbierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
doobidoo/mcp-memory-service#1650 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 82/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
ml-explore/mlx-lm#2002 ·
Los mantenedores suelen responder en 2 días