[Bug]: malformed learning sidecar data can hide valid annotations
Los mantenedores suelen responder en 1 día
Evaluación
- Dificultad
- 2/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 72/100
- Tipo de issue
- Error
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- python
- Área
- ai-infra-agents
Línea de trabajo
Empieza con graphify.reflect.load_learning_overlay y sigue cómo analiza el JSON del sidecar y lee las entradas individuales. Comprueba también cómo gestiona el cargador de MCP los errores del overlay. Trata una raíz que no sea un objeto como un overlay vacío y omite las filas mal formadas, conservando las entradas válidas; verifica las reproducciones con un sidecar que contenga null, [] y las entradas mixtas bad/good.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Reproduction
Confirmed on v8 at 59b5ab11 (0.9.102), Python 3.12 on Windows. No LLM or API credentials are needed.
Next to graph.json, save .graphify_learning.json containing {"nodes":{"bad":{"source_file":["app.py"]},"good":{"status":"preferred","score":2}}}. Call graphify.reflect.load_learning_overlay with the graph path. A second reproduction is a sidecar containing JSON null or [].
Actual behavior
A non-string source raises TypeError while resolving a path. The MCP loader catches the exception and drops the entire overlay, including healthy entries. A non-object JSON root raises AttributeError before entries are read.
Expected behavior
Treat a malformed root as an empty optional overlay, and keep healthy entries when an individual row is malformed.
The change concerns sidecar input handling. The staleness characterization in #4996 exercises the existing behavior for well-formed entries.
- Lenguaje dominante
- Python
- Estrellas
- 124k
- Forks
- 11.9k
- Merge medio
- 2 h 35 min
- PR fusionados (30 d)
- 6
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
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 Graphify-Labs/graphify
-
[Bug]: valid JSON with the wrong global-manifest shape bypasses recoveryPosiblemente ocupada @azizur100389 la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 64/100
Graphify-Labs/graphify#5020 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
[Bug]: graphify update skips or partially rebuilds a missing graph when the manifest survivesPosiblemente ocupada @azizur100389 la tomó hace 1 día. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 77/100
Graphify-Labs/graphify#5016 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 65/100
Graphify-Labs/graphify#5006 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
-
extract --out <dir>: semantic cache never saved, docs missing from manifest (relative source_file resolved against --out)Posiblemente ocupada Un pull request vinculado a esta issue está abierto o ya se fusionó. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4831 · 3 comentarios ·
Los mantenedores suelen responder en 1 día
-
hooks: last-resort `python3` probe launches the Microsoft Store redirector on Windows (visible console flash, exit 9009)Posiblemente ocupada @Deepjyoti-Sarmah la tomó hace 18 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4563 · 2 comentarios ·
Los mantenedores suelen responder en 1 día
Todos los issues de Graphify-Labs/graphify
Issues similares
-
bug good first issue
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
graphql-python/graphql-core#304 ·
Los mantenedores suelen responder en 1 día
-
docs size:S
Dificultad 1/5 Menos de una hora Aptitud para principiantes 88/100
Los mantenedores suelen responder en 1 día
-
Dificultad 1/5 1-3 horas Aptitud para principiantes 88/100
Gathin23/cleanyourtv#3464 ·
Los mantenedores suelen responder en 1 día
-
documentation
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
Los mantenedores suelen responder en 1 día