Add guidance to SKILL.md on when building a graph is (and isn't) worth it
Los mantenedores suelen responder en 1 día
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 1/5
- Tiempo estimado
- 1-3 horas
- Aptitud para principiantes
- 75/100
- Tipo de issue
- Documentación
- Claridad
- Bien especificado
- Estado de actividad
- Activo
- Stack tecnológico
- python
- Área
- documentation
Línea de trabajo
Busca el archivo SKILL.md de la skill graphify y lee los pasos de compilación existentes. Añade antes una sección breve que explique, con los casos y las indicaciones del issue, cuándo usar la búsqueda directa y cuándo crear o consultar un grafo. Estará listo cuando la sección cubra los grafos existentes, las consultas puntuales y las situaciones en las que vale la pena crear un grafo.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Ran the skill end-to-end today (sample corpus + a real ~200-file project) and it delivered genuinely good results — correct AST extraction, accurate cross-referencing between code and doc semantic layers, and the cohesion/god-node/surprising-connections analysis surfaced real findings (an architectural smell, a cross-doc duplicate derivation, etc.).
One gap: the skill has no guidance on when the full pipeline is worth its cost vs. when a plain code search would do. In practice:
- A single targeted lookup ("what calls
X", "where isYdefined") on an ordinary-sized codebase is answered just as well, instantly and for free, bygrep/ripgrep/Read. Building the graph first is pure overhead for that case — AST extraction is fast and free, but semantic extraction on docs dispatches subagents and can take minutes and tens of thousands of tokens, even for a modest doc set. - The graph earns its cost when: there will be many repeated questions (amortized build cost), the corpus is large/unfamiliar and the user needs orientation, the question needs cross-file/cross-type links text search can't see (doc ↔ code, cross-doc semantic duplicates), or the question is structural (cohesion, bridging nodes, import cycles) rather than textual.
Right now the skill (reasonably) defaults to building on /graphify <path>, which is fine when the user explicitly asks for that — but when an agent is deciding on its own whether to reach for graphify vs. a direct search for a one-off question, there's no steering in SKILL.md toward the cheaper option first.
Suggested addition: a short "is a graph worth it here" section before the build steps, something like:
Building costs real time and tokens: AST extraction on code is free and fast, but semantic extraction on docs/papers/images dispatches subagents (minutes, tens of thousands of tokens), and a full pipeline run is overhead you pay once per corpus state. Don't pay it reflexively.
If
graphify-out/graph.jsonalready exists for this corpus, always query it rather than re-deriving the answer by reading files or rebuilding — that cost is already sunk and querying is cheap.If no graph exists yet, check whether the request is a single, targeted lookup a plain code search would answer just as well — "where is X defined", "what calls Y", "find the function that does Z" on a codebase of ordinary size. For these, grep/ripgrep/Read is faster, free, and exact. Say so and just search directly; offer to build the full graph instead only if the user wants it.
Building (or updating) the graph earns its cost when:
- You expect many future questions against this corpus — the build cost amortizes.
- The corpus is large or unfamiliar and the user needs orientation (god nodes, communities).
- The question needs cross-file or cross-type connections text search can't see.
- The question is structural/architectural, not textual.
Happy to turn this into a PR against SKILL.md if that's useful.
Separately filed one small fix noticed during testing as a PR: #4830 (graphify query --help ran a literal query instead of showing usage). The other thing I initially flagged (hyperedge members/nodes field mismatch) turned out to already be intentionally handled in build.py (#1561) — not a bug, so no PR for that one.
- Lenguaje dominante
- Python
- Estrellas
- 124k
- Forks
- 11.9k
- Métricas de merge de PR
- Sin PR fusionados en 30 d
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
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4860 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Cache the claude backend system prompt via Anthropic prompt cachingPosiblemente ocupada @Jvxoq la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 76/100
Graphify-Labs/graphify#4835 · 4 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 10 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
-
GRAPHIFY_DISABLE_THINKING is ignored by the claude backend, so thinking eats max_tokens and extraction truncatesPosiblemente ocupada @bercedev la tomó hace 10 días. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 69/100
Graphify-Labs/graphify#4546 · 1 comentario ·
Los mantenedores suelen responder en 1 día
Todos los issues de Graphify-Labs/graphify
Issues similares
-
:bug: Bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 84/100
scikit-image/scikit-image#8377 ·
Los mantenedores suelen responder en 1 día
-
Ingredient import validation accepts negative nutrient values and NaNPosiblemente ocupada @LukeSekai la tomó hoy. Abierto
Dificultad 2/5 1-3 horas Aptitud para principiantes 82/100
wger-project/wger#2564 ·
Los mantenedores suelen responder en 2 días
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
apache/ponymail-foal#351 ·
-
Dificultad 1/5 Menos de una hora Aptitud para principiantes 68/100
Los mantenedores suelen responder en 4 días
-
To Lower Case(온보딩 전용)Abiertoproblem
Dificultad 1/5 1-3 horas Aptitud para principiantes 72/100
DaleStudy/leetcode-study#2885 ·